From bc24237948c0bd65d36e2a5adc05876f364d2d1a Mon Sep 17 00:00:00 2001 From: Claude Date: Tue, 14 Apr 2026 22:17:58 +0000 Subject: [PATCH 1/4] fix: correct 'License' to 'Licenses' in README The section heading should be plural since the project uses a single license but the heading conventionally reads 'Licenses'. https://claude.ai/code/session_017LdTZvZN3dae9ttj5BJoKx --- README.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/README.md b/README.md index 1e51475e..8632f711 100644 --- a/README.md +++ b/README.md @@ -340,6 +340,6 @@ GitHub environment and AWS account configuration are documented in the [administ - [Claude Code plugins documentation](https://code.claude.com/docs/en/plugins) - [Cursor plugins documentation](https://cursor.com/docs/plugins) -## License +## Licenses This project is licensed under the Apache-2.0 License. From 1767bcb7ee2c7849311fbcdcf1ab953d8ab6f54a Mon Sep 17 00:00:00 2001 From: Claude Date: Tue, 14 Apr 2026 22:25:34 +0000 Subject: [PATCH 2/4] docs: add PR requirements section to AGENTS.md Document the mandatory contributor statement for all PRs, the exact text required by the Require Contributor Statement CI check, and the URL mismatch pitfall when working in a fork (settings.json hardcodes awslabs/agent-plugins but the CI check uses github.repository). https://claude.ai/code/session_017LdTZvZN3dae9ttj5BJoKx --- AGENTS.md | 24 ++++++++++++++++++++++++ 1 file changed, 24 insertions(+) diff --git a/AGENTS.md b/AGENTS.md index 0e596132..e0b8b6d6 100644 --- a/AGENTS.md +++ b/AGENTS.md @@ -204,6 +204,30 @@ git worktree remove .tmp/ All worktrees live under `.tmp/` (already in `.gitignore`). +## Pull Request Requirements + +Every PR body MUST include the contributor statement that the `Require Contributor Statement` +CI job enforces (see `.github/workflows/pull-request-lint.yml`). Always include both lines +verbatim: + +``` +Generated with [Claude Code](https://claude.com/claude-code) + +--- + +By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the [project license](https://github.com//blob/main/LICENSE). +``` + +Replace `` with the actual `owner/repo` of the repository the PR is being opened against +(e.g. `sbalswa/agent-plugins`). **Do not copy the URL from `.claude/settings.json` blindly** — +that file hardcodes `awslabs/agent-plugins`, but the CI check uses `${{ github.repository }}` +which resolves to the current repository. In a fork, these differ and the check will fail even +if the rest of the attribution text is present. + +> **Why it can still fail after adding the attribution:** The workflow uses a substring match on +> the exact URL. If the repo in the link does not match `github.repository`, the check fails. +> Always verify the `owner/repo` slug matches the repo the PR targets, not the upstream fork origin. + ## Boundaries - ALWAYS use `git worktree add .tmp/` for new work. NEVER switch branches in the main worktree. From c6dffa50d870140dab62278aee25cd4f5052bcc1 Mon Sep 17 00:00:00 2001 From: Claude Date: Tue, 14 Apr 2026 22:27:34 +0000 Subject: [PATCH 3/4] docs: fix example repo slug in PR requirements section MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Use awslabs/agent-plugins (the upstream) as the example since we are working in a fork — the example should reflect the target repo, not the fork. https://claude.ai/code/session_017LdTZvZN3dae9ttj5BJoKx --- AGENTS.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/AGENTS.md b/AGENTS.md index e0b8b6d6..07c1b900 100644 --- a/AGENTS.md +++ b/AGENTS.md @@ -219,7 +219,7 @@ By submitting this pull request, I confirm that you can use, modify, copy, and r ``` Replace `` with the actual `owner/repo` of the repository the PR is being opened against -(e.g. `sbalswa/agent-plugins`). **Do not copy the URL from `.claude/settings.json` blindly** — +(e.g. `awslabs/agent-plugins`). **Do not copy the URL from `.claude/settings.json` blindly** — that file hardcodes `awslabs/agent-plugins`, but the CI check uses `${{ github.repository }}` which resolves to the current repository. In a fork, these differ and the check will fail even if the rest of the attribution text is present. From 732f89ef57a5236366d17f04906c1830dbfabd18 Mon Sep 17 00:00:00 2001 From: Scott Schreckengaust Date: Fri, 23 Oct 2026 10:53:28 -0700 Subject: [PATCH 4/4] Remove generated message and confirmation from AGENTS.md Removed generated message and confirmation statement from AGENTS.md. Signed-off-by: Scott Schreckengaust --- AGENTS.md | 4 ---- 1 file changed, 4 deletions(-) diff --git a/AGENTS.md b/AGENTS.md index 46cdc722..766576c5 100644 --- a/AGENTS.md +++ b/AGENTS.md @@ -201,10 +201,6 @@ CI job enforces (see `.github/workflows/pull-request-lint.yml`). Always include verbatim: ``` -Generated with [Claude Code](https://claude.com/claude-code) - ---- - By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the [project license](https://github.com//blob/main/LICENSE). ```