Phase 1: split source vs published refs and verify dual publish#2085
Merged
aaronpowell merged 2 commits intoJun 22, 2026
Merged
Conversation
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
There was a problem hiding this comment.
Pull request overview
This PR updates the repository’s “source vs published” branch wiring to support Phase 1 of the migration to a dedicated marketplace published branch while preserving the existing contributor flow.
Changes:
- Introduces explicit constants for source-content vs published-artifact refs and base URLs used for raw-link generation.
- Updates README/plugin docs to surface “source” vs “published” guidance, including a tool-facing published manifest URL on
marketplace. - Adds a post-publish parity check in
publish.ymlto fail the workflow ifmainandmarketplacepublished outputs diverge.
Show a summary per file
| File | Description |
|---|---|
| eng/constants.mjs | Adds source/published branch constants and corresponding raw base URLs; updates exports. |
| eng/update-readme.mjs | Switches install badge URL generation to use source/published base URLs; adds published manifest + source guidance to the plugins section output. |
| docs/README.plugins.md | Documents the published manifest URL on marketplace and a link to source plugin content. |
| CONTRIBUTING.md | Adds a note pointing Phase 2 gating to Issue #1368 rollout status. |
| .github/workflows/publish.yml | Adds a post-push parity check between main and marketplace branches. |
Copilot's findings
- Files reviewed: 5/5 changed files
- Comments generated: 2
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Pull Request Checklist
npm startand verified thatREADME.mdis up to date.stagedbranch for this pull request.Description
Phase 1 needs to preserve contributor flow while making published artifacts resolvable from
marketplace. This PR finishes the repo-side wiring so source-content and published-artifact refs are explicit, and dual-publish output stays verifiable.marketplace.publish.ymlto fail ifmainandmarketplacediverge during the compatibility window.Fixes: #1368
Type of Contribution
Additional Notes
Validation run in this branch covered
npm run buildandnpm run plugin:validate.npm run skill:validatestill reports a pre-existing repository issue inskills/aws-cloudwatch-investigation(invalid skillnameformat), unrelated to this change set.By submitting this pull request, I confirm that my contribution abides by the Code of Conduct and will be licensed under the MIT License.