docs(stores): add multi-repo implementation flow - #1491
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
📝 WalkthroughWalkthroughThe user guide adds a cross-repository workflow. It explains shared contract planning, store context boundaries, ChangesCross-repository store workflow
Estimated code review effort: 1 (Trivial) | ~3 minutes Possibly related issues
Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@docs/stores-beta/user-guide.md`:
- Around line 218-221: Qualify the `store: team-plans` pointer description in
the user guide to state that it applies only when no higher-precedence explicit
`--store` option or nearest local `openspec/` root overrides it. Preserve the
existing clarification that OpenSpec does not split tasks by the directory where
`apply` was invoked or route tasks to repositories.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro Plus
Run ID: 880b9cab-aec9-4e83-8708-8ec8825b7d57
📒 Files selected for processing (1)
docs/stores-beta/user-guide.md
alfred-openspec
left a comment
There was a problem hiding this comment.
Please qualify the store: pointer paragraph before merge: an explicit --store and a nearer local planning root take precedence, so the pointer does not make every command target team-plans unconditionally. The rest of the two-layer flow matches current Stores behavior, and the focused pointer/reference/workset tests pass.
|
Thanks for the review. Addressed in cb9b08c: the paragraph now makes the store: pointer conditional on there being no explicit --store or nearer local openspec/ root, while retaining the clarification that OpenSpec does not route tasks to repositories. CodeRabbit's follow-up found no actionable comments. The CI and Security workflows are currently awaiting maintainer approval (�ction_required), with no executed check failure reported. |
alfred-openspec
left a comment
There was a problem hiding this comment.
The pointer-precedence qualification now matches the actual root-selection behavior while preserving the important no-routing clarification. The rest of the two-repo flow remains coherent; approved at cb9b08c.
Summary
Add a concrete stores-beta walkthrough for a feature spanning one service repo and one frontend repo.
The guide now answers the two questions from #1257:
store:pointer selects the same central root for every command and does not divide tasks by caller directory. Teams that need independent component apply/review cycles can keep shared requirements in the store while each repo retains a local OpenSpec root with a read-onlyreferences:entry.The example follows that supported two-layer flow through central contract planning, local service/frontend changes, separate apply cycles, and independent review/archive.
Closes #1257.
Why this shape
The current root/action-context model scopes a workflow to its selected planning root, references expose store specs as read-only upstream context, and worksets only reopen folders together. Documenting those boundaries avoids implying repo-sliced apply behavior that OpenSpec does not currently implement.
Validation
corepack pnpm run build— passed.git diff --check— passed.Scope and risk
Documentation only. No CLI, schema, generated skill, or runtime behavior changes. Repo-sliced task routing remains explicitly deferred; the guide describes the currently supported pattern instead.
Summary by CodeRabbit