Repository navigation
feat: simplify the built-in team shelf and keep first-project compatible - #864
Conversation
|
Note Currently processing new changes in this PR. This may take a few minutes, please wait... ⚙️ Run configuration
⛔ Files ignored due to path filters (1)
📒 Files selected for processing (51)
✨ Finishing Touches📝 Generate docstrings
🧪 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 |
aa101ed to
5e1d4a7
Compare
openrig-review
left a comment
There was a problem hiding this comment.
Approved at de57592 after two independent review rounds. The built-in team shelf now offers starter (a Claude builder delegating to a Codex reviewer), factory (lead, advisor, build, QA, design and two reviewers), and the specialist teams code-review, research and pm, with the kernel and its variants unchanged. The per-provider first-project variants, implementation-pair, product-team, demo and conveyor leave the shelf. first-project resolves to starter only after every exact library ID and name fails, prints that starter is a Claude builder and a Codex reviewer and that the OpenRig operator can adapt it on a Codex-only machine, and keeps the existing-rig and archived-rig checks for both names. The seven removed or renamed folders keep their original CULTURE.md bytes so rigs launched from them still restore after upgrade, with a restore test that fails without them; no rig.yaml stays beside them, so they don't return to the shelf. The implementer and QA defaults and the recommended skill routing no longer select test-driven development; the optional skill stays available to existing user specs. factory-rsi and secrets-manager keep their files as world-bundle.yaml for the move to openrig-world.
— dev60-planner@v-openrig-build
The built-in team shelf now offers
starter,factory,code-review,researchandpm, alongside the unchanged kernel. Starter has a Claude builder and Codex reviewer; factory has lead, advisor, build, QA, design and two review seats. Factory reuses the PM agent through an advisor profile and retains role skills and openrig-core. Implementer and QA defaults and recommended skill routing no longer select TDD; its library resource remains available to existing user specs.rig up first-projectresolves to the built-in starter and explains its providers and Codex-only adaptation on stderr. Exact library IDs/names and--existingretain precedence; existing and archived rigs under either name keep their recovery checks. Seven old spec folders retain their original culture bytes so stored startup paths still restore after upgrade, without restoring their catalog entries. Factory-RSI and secrets-manager retain their YAML asworld-bundle.yamlpending migration to world bundles.Validation covers catalog membership, seat/runtime/resource projection, alias routing, legacy culture restore (including a restore regression for a stored first-project culture file), skill mirrors and the skill index, topology, packing and TUI readers.