Skip to content

Module cleanup: batch (run 25569662252)#394

Open
github-actions[bot] wants to merge 16 commits into
mainfrom
module-cleanup-batch-25569662252
Open

Module cleanup: batch (run 25569662252)#394
github-actions[bot] wants to merge 16 commits into
mainfrom
module-cleanup-batch-25569662252

Conversation

@github-actions
Copy link
Copy Markdown

@github-actions github-actions Bot commented May 8, 2026

Automated module-cleanup batch.

Modules in this batch

  • executors:testing
  • finagle-http-23.11:javaagent
  • finatra-2.9:javaagent

Cleanup for executors:testing

  • Narrow executor testing helper visibility to package scope where sibling callers are in the same package

Co-authored-by: Copilot 223556219+Copilot@users.noreply.github.com

Cleanup for finagle-http-23.11:javaagent

  • Move shared test JVM arguments and system properties into withType().configureEach
  • Remove redundant isMethod() matchers from named method advice matchers
  • Reduce FinagleClientExtension visibility to package-private test scope

Co-authored-by: Copilot 223556219+Copilot@users.noreply.github.com

Cleanup for finatra-2.9:javaagent

  • Qualify FinatraSingletons helper-method calls instead of static importing non-accessor helpers

Co-authored-by: Copilot 223556219+Copilot@users.noreply.github.com

trask and others added 16 commits May 6, 2026 11:45
The AWF api-proxy in firewall v0.25.40 collapses Copilot traffic to a single endpoint, which appears to break copilot-cli's /responses-API routing negotiation and blocks newer models like gpt-5.5. Disabling the agent sandbox lets copilot connect directly to api.githubcopilot.com, matching how the original (non-gh-aw) module-cleanup workflow ran successfully with gpt-5.5.
- Narrow executor testing helper visibility to package scope where sibling callers are in the same package

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Move shared test JVM arguments and system properties into withType<Test>().configureEach
- Remove redundant isMethod() matchers from named method advice matchers
- Reduce FinagleClientExtension visibility to package-private test scope

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Qualify FinatraSingletons helper-method calls instead of static importing non-accessor helpers

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@github-actions github-actions Bot added the module cleanup Automated module cleanup batch PR label May 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

module cleanup Automated module cleanup batch PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant