Skip to content

feat: add PR attribution and document Claude Code setup#30

Merged
scottschreckengaust merged 1 commit intomainfrom
extra-known-marketplaces
Feb 17, 2026
Merged

feat: add PR attribution and document Claude Code setup#30
scottschreckengaust merged 1 commit intomainfrom
extra-known-marketplaces

Conversation

@scottschreckengaust
Copy link
Member

@scottschreckengaust scottschreckengaust commented Feb 17, 2026

Summary

  • Adds attribution.pr to .claude/settings.json so Claude Code automatically appends the required contributor statement to PR descriptions, preventing CI failures from the contributorStatement check
  • Adds a note in CONTRIBUTING.md pointing Claude Code users to the Development Guide
  • Documents the attribution.pr setting in the Claude Code Setup section of the Development Guide

Test plan

  • Create a PR using Claude Code and verify the description includes both the Claude Code attribution and the contributor statement
  • Verify the contributorStatement CI check passes on this PR

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the project license.

@scottschreckengaust scottschreckengaust requested a review from a team February 17, 2026 12:45
@scottschreckengaust scottschreckengaust requested a review from a team as a code owner February 17, 2026 12:45
@theagenticguy
Copy link
Contributor

theagenticguy commented Feb 17, 2026

Do we expect the agent plugins for aws to have plugins/skills/agents etc. that enhance the development of the marketplace itself? would be good to capture what a user "job to be done" would look like in terms of contribution to the repo vs. using the plugins separate from the repo itself.

@scottschreckengaust scottschreckengaust changed the title feat: register marketplace via extraKnownMarketplaces feat: claude code project settings update Feb 17, 2026
@scottschreckengaust scottschreckengaust changed the title feat: claude code project settings update feat: add PR attribution with required contributor statement Feb 17, 2026
@scottschreckengaust
Copy link
Member Author

Do we expect the agent plugins for aws to have plugins/skills/agents etc. that enhance the development of the marketplace itself? would be good to capture what a user "job to be done" would look like in terms of contribution to the repo vs. using the plugins separate from the repo itself.

Agreed and removed for a future state if necessary. Only keeping the contribution statement for the pull request validation GitHub Workflow.

@theagenticguy
Copy link
Contributor

Do we expect the agent plugins for aws to have plugins/skills/agents etc. that enhance the development of the marketplace itself? would be good to capture what a user "job to be done" would look like in terms of contribution to the repo vs. using the plugins separate from the repo itself.

Agreed and removed for a future state if necessary. Only keeping the contribution statement for the pull request validation GitHub Workflow.

One more thing (sorry for the churn!). Should we indicate in the readme or CONTRIBUTING docs anywhere? e.g. if you use Claude Code to contribute, we have .claude folder ?

@scottschreckengaust
Copy link
Member Author

scottschreckengaust commented Feb 17, 2026

Do we expect the agent plugins for aws to have plugins/skills/agents etc. that enhance the development of the marketplace itself? would be good to capture what a user "job to be done" would look like in terms of contribution to the repo vs. using the plugins separate from the repo itself.

Agreed and removed for a future state if necessary. Only keeping the contribution statement for the pull request validation GitHub Workflow.

One more thing (sorry for the churn!). Should we indicate in the readme or CONTRIBUTING docs anywhere? e.g. if you use Claude Code to contribute, we have .claude folder ?

Good call. The Development Guide already has a "Claude Code Setup" section (line 20) that covers marketplace registration and the .claude/settings.json plugin configuration. We could add a brief note there about attribution.pr auto-including the contributor statement, but I'd keep it minimal since the setting is transparent — contributors don't need to do anything for it to work.

For the CONTRIBUTING.md, I'd suggest keeping it tool-agnostic (it currently is) and letting the Development Guide be the Claude Code-specific onboarding doc. We can add a one-liner to CONTRIBUTING.md pointing Claude Code users to the dev guide if that feels useful. Happy to do either as a follow-up.

Adds attribution.pr to .claude/settings.json so Claude Code PRs
automatically include the required contributor statement. Documents
the Claude Code project configuration in CONTRIBUTING.md and the
Development Guide.
@scottschreckengaust scottschreckengaust changed the title feat: add PR attribution with required contributor statement feat: add PR attribution and document Claude Code setup Feb 17, 2026
@scottschreckengaust
Copy link
Member Author

@theagenticguy Added the doc updates in this PR:

  • CONTRIBUTING.md now points Claude Code users to the Claude Code Setup section in the Development Guide
  • Development Guide documents the attribution.pr setting that auto-includes the contributor statement in Claude Code PRs

@scottschreckengaust scottschreckengaust requested review from a team February 17, 2026 14:37
@scottschreckengaust scottschreckengaust added documentation Improvements or additions to documentation enhancement New feature or request labels Feb 17, 2026
@scottschreckengaust scottschreckengaust added this pull request to the merge queue Feb 17, 2026
Merged via the queue into main with commit c2c183c Feb 17, 2026
46 of 48 checks passed
@scottschreckengaust scottschreckengaust deleted the extra-known-marketplaces branch February 17, 2026 14:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants