Skip to content

Module cleanup: batch (run 25543623593)#324

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

Module cleanup: batch (run 25543623593)#324
github-actions[bot] wants to merge 22 commits into
mainfrom
module-cleanup-batch-25543623593

Conversation

@github-actions
Copy link
Copy Markdown

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

Automated module-cleanup batch.

Modules in this batch

  • apache-dubbo-2.7:testing
  • apache-httpclient-2.0:javaagent
  • apache-httpclient-5.0:javaagent
  • armeria-1.3:library
  • aws-lambda-core-1.0:javaagent
  • aws-lambda-events-2.2:javaagent
  • aws-lambda-events-common-2.2:library
  • aws-sdk-1.11:library
  • aws-sdk-1.11:library-autoconfigure

Cleanup for apache-dubbo-2.7:testing

  • Fix typo in Dubbo test bootstrap comments

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

Cleanup for apache-httpclient-2.0:javaagent

  • Move AdviceScope scope creation into the constructor to match the repository's preferred simple advice-scope pattern

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

Cleanup for apache-httpclient-5.0:javaagent

  • Mark advice return values nullable when exit advice also runs on thrown methods

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

Cleanup for armeria-1.3:library

  • Wire library semconv metadata collection into all test tasks

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

Cleanup for aws-lambda-core-1.0:javaagent

  • Clarify AWS Lambda runtime package comments in handler matchers

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

Cleanup for aws-lambda-events-2.2:javaagent

  • Fix the SQSEvent dependency comment grammar so the Gradle rationale reads correctly, following the general review rule to correct inaccurate or incorrect comments.

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

Cleanup for aws-lambda-events-common-2.2:library

  • Mark nullable SerializationUtil JSON inputs that can receive null handler values
  • Rename an unused fallback catch variable to ignored

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

Cleanup for aws-sdk-1.11:library

  • Restrict SnsAttributesExtractor to package-private visibility to follow the style guide's minimal visibility rule after confirming it has no sibling-module callers.\n\nCo-authored-by: Copilot 223556219+Copilot@users.noreply.github.com

Cleanup for aws-sdk-1.11:library-autoconfigure

  • Mark AWS SDK error callback response as nullable to match the AWS error path and sibling handler contract

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

trask and others added 22 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.
- Fix typo in Dubbo test bootstrap comments

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Move AdviceScope scope creation into the constructor to match the repository's preferred simple advice-scope pattern

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Mark advice return values nullable when exit advice also runs on thrown methods

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Wire library semconv metadata collection into all test tasks

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Clarify AWS Lambda runtime package comments in handler matchers

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Fix the SQSEvent dependency comment grammar so the Gradle rationale reads correctly, following the general review rule to correct inaccurate or incorrect comments.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Mark nullable SerializationUtil JSON inputs that can receive null handler values
- Rename an unused fallback catch variable to ignored

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Restrict SnsAttributesExtractor to package-private visibility to follow the style guide's minimal visibility rule after confirming it has no sibling-module callers.\n\nCo-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Mark AWS SDK error callback response as nullable to match the AWS error path and sibling handler contract

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