docs: add MCP Inspector: Our AI Software Factory - #2498
Merged
Merged
Conversation
Adds docs/ai-software-factory.md — an end-to-end explanation of how contributions work since v2.0.0: the rules/skills split, the issue lifecycle, the PR flow (including how a working session gets started and Claude Code's /goal mode), the three automated sweeps that replaced Dependabot PRs, the human-gated security-advisory track, the quality gate, and where the process is headed. Backed by verified numbers: 268 PRs merged and 9 releases shipped in the ~8 weeks since v2.0.0, 265 of those PRs (99%) agent-authored. Closes #2497 Signed-off-by: Bob Dickinson <bob.dickinson@gmail.com>
There was a problem hiding this comment.
Copilot review overview
🟡 Changes recommended
Several statements inaccurately describe issue intake automation and the quality gate’s guarantees.
Get a fresh assessment by requesting another Copilot review.
Review effort: Balanced
Findings: 4
Open (4)
What changed in this PR
Documents Inspector’s AI-assisted contribution model and links it from the README.
Changes:
- Explains issue, PR, security, automation, and quality-gate workflows.
- Adds metrics covering releases and agent-authored PRs.
- Adds the guide to the documentation index.
| File | Description |
|---|---|
README.md |
Links the new guide. |
docs/ai-software-factory.md |
Documents the software-factory process. |
- Distinguish the two issue intake paths in the v2 lifecycle: a maintainer-created issue (via issue-create) is approved by definition and boards straight into Todo, while everything else (outside reporters, or a maintainer's own hand-opened issue) lands unmilestoned in Incoming. Mark the triage/approval steps as Incoming-path only. - Correct the quality-gate stage-list reference: one stage (smoke:web:firefox) is local-only, not two. - Correct the 'Where this is headed' section: scoring and boarding an issue still requires someone to invoke the issue-triage sweep today; UI-filed and sweep-filed issues don't board themselves. npm run local:gate passes (docs-only change). Signed-off-by: Bob Dickinson <bob.dickinson@gmail.com>
Contributor
Author
|
Thanks for the review — all four findings were accurate and are addressed in 0a942dd (replies posted on each thread). No further review needed from here; this is ready for a maintainer to merge. |
This was referenced Sep 27, 2026
cliffhall
requested changes
Sep 27, 2026
- An issue that arrives outside issue-create isn't on the board at all until issue-triage is triggered over it — it doesn't land in Incoming automatically. - Board #28 has a GitHub Projects workflow enabled that closes the linked issue automatically once its card reaches Done. The manual step on merge is moving the card to Done; nothing separately closes the issue by hand. Corrected the v2 lifecycle, the PR-flow bullet, and the launch-session paragraph to match. - Mention the skills eval (npm run skills:eval) alongside the Claude Code / Copilot CLI parity claim: it measures both first-move (prompt-triggered / model self-directed) and hand-off (second-order) skill firing, against both agents. npm run local:gate passes (docs-only change). Signed-off-by: Bob Dickinson <bob.dickinson@gmail.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.
Closes #2497
Adds
docs/ai-software-factory.md, an end-to-end explanation of how contributions actually happen since v2.0.0: the rules/skills split, the issue lifecycle, the PR flow (including how a working session gets started and Claude Code's/goalmode), the three automated sweeps that replaced Dependabot PRs, the human-gated security-advisory track, the quality gate, and where the process is headed.Backed by verified numbers rather than just describing the intent: 268 PRs merged and 9 releases shipped in the ~8 weeks since v2.0.0, 265 of those PRs (99%) agent-authored.
Also adds a row to the README's Documentation table linking the new doc.
Docs-only change.
npm run local:gatepasses.