Conversation
There was a problem hiding this comment.
Pull request overview
Codifies a “now required vs later” MVP readiness split for the guardrails workstream, and aligns the repo’s local issue-pack documentation with the current GitHub epic/issue structure.
Changes:
- Adds an MVP readiness register documenting what blocks an MVP claim vs what is post-MVP maturity work.
- Updates guardrails planning/tracking docs to reference the new MVP readiness epic and current issue.
- Adds missing local issue briefs (004–009) and updates the migration inventory + issue pack index to match.
Reviewed changes
Copilot reviewed 11 out of 11 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| packages/guardrails/README.md | Updates “planned next slices” links to include the MVP readiness epic and post-MVP issues. |
| docs/ai-guardrails/README.md | Introduces the MVP readiness split section and updates tracking/current-issue references. |
| docs/ai-guardrails/mvp-readiness.md | New: MVP readiness register defining “now required” vs “later” scope with rationale. |
| docs/ai-guardrails/migration/claude-code-skills-inventory.md | Updates “Immediate Implications” mapping to the local issue-brief slugs and the now/later split. |
| docs/ai-guardrails/issues/README.md | Extends the issue-pack index through 009 and documents the MVP split boundary. |
| docs/ai-guardrails/issues/004-safe-agents-and-commands.md | New: brief for safe agents + workflow commands as MVP-floor work. |
| docs/ai-guardrails/issues/005-provider-admission-policy.md | New: brief for provider admission policy as MVP-floor work. |
| docs/ai-guardrails/issues/006-scenario-and-replay-harness.md | New: brief for scenario/replay harness as MVP-floor work. |
| docs/ai-guardrails/issues/007-plugin-hardening-wave-2.md | New: brief for MVP-floor plugin hardening wave 2. |
| docs/ai-guardrails/issues/008-authoritative-ci-and-release-gates.md | New: brief for post-MVP authoritative CI/release gates. |
| docs/ai-guardrails/issues/009-broader-claude-asset-migration.md | New: brief for post-MVP broader asset migration. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
|
||
| ## Dependencies | ||
|
|
||
| - `docs/ai-guardrails/adr/002-provider-admission-lanes.md` |
There was a problem hiding this comment.
docs/ai-guardrails/adr/002-provider-admission-lanes.md is listed as a dependency, but there is no 002-* ADR in docs/ai-guardrails/adr/ (only 001, 003, 004). This makes the issue brief’s dependency list non-actionable. Either add the missing ADR 002 (recommended if lanes are a key architectural decision) or update this dependency to point at the correct existing ADR(s).
| - `docs/ai-guardrails/adr/002-provider-admission-lanes.md` | |
| - relevant existing ADRs in `docs/ai-guardrails/adr/` |
Summary
Verification
Closes #15