Repository navigation
docs(maproom): replace INTERIM cross-repo guidance with native 0.3.0 flags (plugin 0.10.3) - #68
Open
danielbushman wants to merge 2 commits into
Open
danielbushman wants to merge 2 commits into
danielbushman wants to merge 2 commits into
Conversation
…cross-repo flags Replace the <!-- INTERIM --> section in maproom-researcher.md with native --all-repos / repeatable --repo guidance (maproom 0.3.0). Update version floor 0.2.0 → 0.3.0. Add Native Cross-Repo Search section to multi-repo-guide.md with per-repo loop demoted to pre-0.3.0 fallback note. Update SKILL.md Cross-Repo Search Workflow with single-invocation examples. Bump plugin.json 0.10.2 → 0.10.3. Verified live: --all-repos and repeatable --repo confirmed working; vector-search exits 2 on multi-repo scope as documented. verify-docs.sh 11/11. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…gression coverage - Remove fabricated '10× faster' claim; replace with 'substantially faster' (matching multi-repo-guide.md) - Clarify scope rules: --all-repos and --repo are two-way mutually exclusive; --repo is one repeatable flag, not two separate scope types - Fix terminology: 'vector-search' is a subcommand, 'hybrid'/'vector' are --mode values on search; document both paths that exit 2 on multi-repo scope - Apply same scope-rules and mode-terminology fixes to multi-repo-guide.md for consistency - Add 4 new checks to verify-docs.sh (checks 12-15): INTERIM absence, 0.3.0 version floor, --all-repos presence, search-budget note — closes TEST ADEQUACY gap flagged in review - verify-docs.sh now passes 15/15 Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ 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 |
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.
Summary
<!-- INTERIM -->"Multi-repo tasks" subsection inmaproom-researcher.mdwith native guidance for--all-reposand repeatable--repo(maproom 0.3.0+)multi-repo-guide.md; supersedesmaproom-allwrapperverify-docs.shfrom 11 to 15 checks, adding automated regression coverage for the new cross-repo contentMerge after cc#70 — this PR depends on maproom 0.3.0 (cross-repo flags) shipped via cc#70.
What changed
plugins/maproom/agents/maproom-researcher.md--all-repos/ repeatable--repoguidance--kcap semantics stated;repo:file:lineformat notedvector-searchsubcommand andsearch --mode hybrid/--mode vectordocumented as exiting code 2 on multi-repo scope--all-reposcall = one search (prefer over N single-repo calls)plugins/maproom/skills/maproom-search/references/multi-repo-guide.md# Pre-0.3.0 fallback onlycommentmaproom-allwrapper noted as supersededplugins/maproom/scripts/verify-docs.sh--all-repospresence, search-budget noteTOTAL_CHECKSupdated from 11 to 15Spec
/workspace/_SPECS/crewchief/research/maproom-eco-plugin-cross-repo-docs-spec.md
Validation evidence
Review findings resolved / deferred
MUST-FIX — all resolved:
maproom-researcher.mdline 74: Removed fabricated "10× faster" claim; replaced with "substantially faster" (matching multi-repo-guide.md line 178)verify-docs.shzero cross-repo coverage: added 4 new checks (12–15) covering INTERIM removal, 0.3.0 floor,--all-repospresence, search-budget noteSHOULD-FIX — all resolved:
--all-reposvs--repoas two-way mutual exclusion;--repois one repeatable flag, not two distinct scope types; applied to bothmaproom-researcher.mdandmulti-repo-guide.mdvector-searchis a subcommand,hybrid/vectorare--modevalues onsearch; now documented distinctly; notes thatsearch --mode vectoralso exits 2 (not only hybrid)NOTE — no action needed:
SKILL.mdline 264 cross-repo note is intentionally minimal (skill-level doc, not agent-level); agent and guide docs carry the complete restriction listMerge-order notes
This branch is stacked on
maproom-plugin-docs-refresh. Merge cc#70 (maproom 0.3.0 binary) andmaproom-plugin-docs-refreshbefore merging this PR.Non-goals
plugins/maproom/)🤖 Generated with Claude Code