docs: replace stale repo-map references with repo-intel#20
Merged
Conversation
4 references in README and 2 in SKILL.md pointed at the renamed repo-map plugin / repo-map.json cache file. Updated to repo-intel / repo-intel.json.
Pre-push hook expects npm test, but package.json had no scripts. Mirror the smoke-test pattern used by drift-detect and repo-intel (require entry module). Matches the graceful fallback in the shared ci-node workflow.
Contributor
There was a problem hiding this comment.
Code Review
This pull request renames references from repo-map to repo-intel across the documentation and adds basic test and validation scripts to package.json. A review comment points out a potential inconsistency in the documentation update for SKILL.md, noting that the underlying implementation may still be using repo-map.json as its primary cache file.
The primary symbol cache loaded via ensureRepoMapSync in lib/collectors/docs-patterns.js is repo-map.json. repo-intel.json is the raw data source from agent-analyzer that gets converted into repo-map.json. Step 5 should reference repo-map.json. Step 6 correctly references repo-intel.json for the doc-drift query that calls the agent-analyzer binary directly.
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
repo-mapreferences in README withrepo-intel(plugin was renamed).skills/sync-docs/SKILL.md(including therepo-map.jsoncache file name, nowrepo-intel.json).npm testandnpm run validatescripts so the pre-push hook and sharedci-nodeworkflow have a smoke test to run (matchesdrift-detectandrepo-intel).Why
Detected during a workspace-wide repo-intel doc-drift scan.
Test plan
Docs-only change (plus the small tooling fix needed to unblock the pre-push hook).
Note
Low Risk
Low risk: primarily documentation wording updates plus adding simple
npm test/validatescripts that onlyrequire('./lib')and shouldn’t affect runtime behavior.Overview
Updates docs to replace stale
repo-mapreferences withrepo-intel(including CLI flag/help text, optionalast-grepintegration notes, and related plugin links).Adds
npmtestandvalidatescripts that run a lightweight Node require check (require('./lib')) to give CI/pre-push hooks a basic smoke test.Reviewed by Cursor Bugbot for commit 2bdd09e. Configure here.