Add merge props migration driver - #8254
Open
joshblack wants to merge 7 commits into
Open
Conversation
|
joshblack
changed the title
joshblack
force-pushed
the
migrate/merge-props-migrator
branch
from
July 31, 2026 18:35
63f3f06 to
2dd85ec
Compare
github-actions
Bot
added
the
integration-tests: recommended
Contributor
|
joshblack
changed the base branch from
copilot/update-style-docs-prop-merging
to
migrate/merge-props-eslint-tooling
August 10, 2026 20:05
joshblack
force-pushed
the
migrate/merge-props-migrator
branch
from
August 10, 2026 20:05
2dd85ec to
db46abb
Compare
joshblack
had a problem deploying
to
github-pages
August 10, 2026 20:09 — with 
GitHub Actions
Failure
GitHub Actions
Failure
joshblack
marked this pull request as ready for review
August 10, 2026 20:11
joshblack
added
the
integration-tests: skipped manually
Contributor
There was a problem hiding this comment.
Pull request overview
Adds temporary migration coordination tooling to drive staged adoption of ADR-025 (prop merging conventions) across @primer/react components via bounded, stacked batches.
Changes:
- Adds the
migrate-to-merge-propscoordinator skill with stack ownership rules, batching workflow, and cleanup criteria. - Adds a
migrate-to-merge-props-batchagent definition to implement a single assigned batch and report validation + migration status evidence.
Show a summary per file
| File | Description |
|---|---|
.github/skills/migrate-to-merge-props/SKILL.md |
Defines the coordinator workflow, stack ownership boundaries, batching rules, and cleanup steps for ADR-025 adoption. |
.github/agents/migrate-to-merge-props-batch.agent.md |
Introduces a bounded batch agent contract for implementing one migration batch and returning evidence to the coordinator. |
Review details
Tip
Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.
- Files reviewed: 2/2 changed files
- Comments generated: 1
- Review effort level: Lite
|
|
||
| - Included: findings reported in `packages/react/src/**/*.{ts,tsx}` by | ||
| `primer/prefer-merge-props`. | ||
| - Excluded: tests, stories, nested JSX, generated output, `.agents/**/*`, and |
joshblack
force-pushed
the
migrate/merge-props-migrator
branch
from
August 10, 2026 21:10
db46abb to
3d07672
Compare
joshblack
force-pushed
the
migrate/merge-props-migrator
branch
from
August 10, 2026 21:10
db46abb to
3d07672
Compare
joshblack
had a problem deploying
to
github-pages
August 10, 2026 21:14 — with
GitHub Actions
Failure
GitHub Actions
Failure
joshblack
force-pushed
the
migrate/merge-props-migrator
branch
from
August 10, 2026 21:19
3d07672 to
4132b01
Compare
joshblack
force-pushed
the
migrate/merge-props-migrator
branch
from
August 11, 2026 21:43
4132b01 to
0e7f754
Compare
joshblack
force-pushed
the
migrate/merge-props-migrator
branch
from
August 11, 2026 22:01
0e7f754 to
0ae3ad3
Compare
TylerJDev
approved these changes
Aug 12, 2026
Refine the adoption report, add validated migration examples, and introduce the coordinator skill and bounded batch agent for ADR-025. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 00bfd295-6ed9-4119-8d63-103a0b3c63e8
Move implementation and adoption-tool changes out of the driver entry and document stack ownership for foundation, driver, batch, and cleanup work. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 00bfd295-6ed9-4119-8d63-103a0b3c63e8
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 9541ca7f-0281-4223-a839-39d75393d922
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 9541ca7f-0281-4223-a839-39d75393d922
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 9541ca7f-0281-4223-a839-39d75393d922
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 9541ca7f-0281-4223-a839-39d75393d922
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 84a0d8ca-13d3-4cb2-ab24-18fa2d9a8a14
joshblack
force-pushed
the
migrate/merge-props-migrator
branch
from
August 13, 2026 21:25
55487f9 to
273dcf3
Compare
joshblack
had a problem deploying
to
github-pages
August 13, 2026 21:28 — with
GitHub Actions
Failure
GitHub Actions
Failure
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.
This PR adds the driver skill for migrating Primer React components to ADR-025.
Changelog
New
migrate-to-merge-propscoordinator skill.migrate-to-merge-props-batchagent.Changed
Removed
Rollout strategy
This is temporary migration coordination tooling and has no public package impact.
Testing & Reviewing
Please verify the coordinator always routes adoption-tool refinements to PR #8248, keeps this entry limited to the skill and agent, and creates one stack entry per implementation batch.