Add arch plugin (architecture + modernization skills)#2253
Merged
Conversation
Add the `arch` plugin with two skills: - `arch:document` — produce a single, cited architecture document for a locally-cloned repo, reading files on disk only. - `arch:modernize` — generate a phased modernization plan, auto-running the document workflow first when no architecture doc exists. Skill sources live in top-level skills/ and are referenced declaratively from plugins/arch/.github/plugin/plugin.json, per the repo's plugin model. Regenerated docs/README.plugins.md, docs/README.skills.md and marketplace.json. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
🔒 PR Risk Scan ResultsScanned 8 changed file(s).
|
Contributor
🔍 Vally Lint Results⛔ Findings need attention
Summary
Full linter output
|
Contributor
There was a problem hiding this comment.
Pull request overview
Adds a new in-repo plugin, arch, that bundles two new skills to (1) generate a cited architecture document from a local checkout and (2) produce an adaptive, phase-based modernization plan (including supporting reference materials). Updates the generated plugin/skill docs and marketplace index so the new resources are discoverable and installable.
Changes:
- Added two new skills:
document(architecture research) andmodernize(modernization planning with safety-ladder framing). - Added
plugins/archwith plugin manifest + README bundling the two skills. - Updated generated documentation (
docs/README.*) and.github/plugin/marketplace.jsonto include the new plugin/skills.
Show a summary per file
| File | Description |
|---|---|
| skills/modernize/SKILL.md | Defines the modernization planning workflow, phases, and governance model. |
| skills/modernize/references/migration-hazards.md | Adds a reusable hazard catalog (H1–H8) to red-team migration phases. |
| skills/modernize/references/copilot-instructions.template.md | Adds a template for .github/copilot-instructions.md to execute the plan. |
| skills/document/SKILL.md | Defines the local-checkout architecture research workflow and output structure. |
| plugins/arch/README.md | Documents the new arch plugin and its included skills. |
| plugins/arch/.github/plugin/plugin.json | Registers the new arch plugin and bundles the two skills. |
| docs/README.skills.md | Adds entries for the new document and modernize skills. |
| docs/README.plugins.md | Adds the new arch plugin to the plugin index. |
| .github/plugin/marketplace.json | Adds arch to the generated marketplace listing. |
Review details
- Files reviewed: 9/9 changed files
- Comments generated: 4
- Review effort level: Low
Collapse the document and modernize skills into one standalone skill (doc-and-modernize) with Documentation and Modernization modes, keeping the plugin named arch. Modernization mode now runs the Documentation workflow inline instead of invoking a separate arch:document skill, fixing standalone-install cross-skill references. Reframe Documentation mode as local-first (remote/API lookups are a flagged last resort) rather than local-only. Regenerate docs and marketplace. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Use 'doc-and-modernize' (repo convention) instead of the namespaced 'arch:doc-and-modernize', noting it surfaces as arch:doc-and-modernize when installed via the plugin. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Change plain-text <PROJECT NAME> to [PROJECT NAME] in the header and first paragraph so Markdown renderers don't parse it as an HTML tag and drop it. The code-span `<N>` on the phase line is unaffected. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Fix codespell: pre-empts -> preempts in the instructions template - Consistent terminology: replace 'research step/workflow' with 'Documentation mode' in SKILL.md, README, and plugin.json description - Fix run-on: add 'that' before 'Modernization mode must surface' - Regenerate docs/marketplace for the updated plugin description Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Instruct redacting credentials/tokens from git remote URLs before recording - CI enforcement: ask user or mark [UNVERIFIED]; remote lookup is flagged last resort Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
aaronpowell
approved these changes
Jul 10, 2026
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.
What this adds
A new curated in-repo plugin,
arch, providing a single consolidated skill for working with locally-cloned repositories:doc-and-modernize(surfaces asarch:doc-and-modernizewhen installed via the plugin) — one skill with two complementary modes:[UNVERIFIED]/sourced-remotely), not the default path. Great for onboarding docs.Structure
Following the repo's declarative plugin model, the skill source lives in the top-level
skills/directory and is referenced fromplugins/arch/.github/plugin/plugin.json; CI materializes it into the installable plugin. The two workflows were consolidated into one standalone skill (rather than two) because top-levelskills/are published/validated as standalone, so the modes now cross-reference each other inline instead of invoking a separate skill.plugins/arch/.github/plugin/plugin.json+plugins/arch/README.mdskills/doc-and-modernize/SKILL.md(+references/)docs/README.plugins.md,docs/README.skills.md,.github/plugin/marketplace.jsonValidation
npm run skill:validate✅npm run plugin:validate✅npm run build(regenerated docs + marketplace) ✅bash eng/fix-line-endings.sh✅