fix(cursor-plugin): point the rule and README at the skill that ships - #3177
Open
Matthew-Selvam wants to merge 1 commit into
Open
Matthew-Selvam wants to merge 1 commit into
Matthew-Selvam wants to merge 1 commit into
Conversation
The Cursor plugin's always-on rule and its README both direct the reader to a `context7-docs-lookup` skill. That name belonged to the original plugin (upstash#1659); upstash#2191 consolidated the plugin skills under a canonical `context7-mcp` source and the directory was renamed, but these two prose references were not updated. The plugin ships only `skills/context7-mcp/`, so the pointer resolves to nothing. Update both references, and pin them with a test: the skill name each file mentions must exist under `plugins/cursor/context7/skills/`, and each shipped skill directory must contain a SKILL.md.
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
2 participants
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
The Cursor plugin's always-on rule and its README both point the reader at a skill named
context7-docs-lookup. That skill does not exist in the plugin.context7-docs-lookupwas the original skill name when the plugin was added (#1659). #2191 consolidated the plugin skills under a canonicalcontext7-mcpsource and the directory was renamed —plugins/cursor/context7/skills/context7-mcp/SKILL.md. The rename updated the directory but not these two prose references, so both now name a skill the plugin does not ship.Concretely, the rule ends with:
and
README.mdlists:The rule is
alwaysApply: true, so the pointer is in context on every request and an agent that follows it looks for a skill that is absent. The only shipped skill iscontext7-mcp; every other plugin in this repo already refers to it by that name.Change
plugins/cursor/context7/rules/use-context7.mdcandplugins/cursor/context7/README.mdnow referencecontext7-mcp.packages/cli/src/__tests__/plugin-manifests.test.tsgains a test pinning the invariant: the skill name mentioned in each file must exist as a directory underplugins/cursor/context7/skills/, and every shipped skill directory must contain aSKILL.md. The existing file already readsplugins/from the repo root, so it is the natural home; the name this repo keeps in prose is now checked against the name it ships.Verification
The new test fails on the unfixed tree and passes after the fix. Both directions were exercised by reverting each reference and each shipped skill directory in turn:
use-context7.mdc references "context7-docs-lookup": expected [ 'context7-mcp' ] to include 'context7-docs-lookup'README.md references "context7-docs-lookup": ...use-context7.mdc references "context7-mcp": expected [ 'renamed-skill' ] to include 'context7-mcp'The first two confirm the assertion detects the regression this PR fixes; the third confirms it is not a rubber stamp — it reads the shipped directory rather than a hardcoded string.
Repo checks on the final commit:
pnpm lint:check— cleanpnpm format:check— cleanpnpm typecheck— cleanpnpm --filter ./packages/cli test— 19 files, 370 tests passed (368 onmaster; the 2 added here)grep -rn "context7-docs-lookup"over the tree now returns nothing.No changeset: this touches no published package code, matching how the other plugin-only fixes in this repo (#2752) landed.