Add MCP integration and Developer Environments skill (v1.2.0)#6
Add MCP integration and Developer Environments skill (v1.2.0)#6scottisloud wants to merge 10 commits into
Conversation
|
Adjusted scope slightly for expected agent behaviours to account for differences in how Cursor Agents work compared to some other agent clients. Ready for review. Testing is a bit fiddly... |
scottisloud
force-pushed
the
scoot/feature/add-mcp-integration
branch
from
3a7a6e0 to
87be6b7
Compare
Wire the 1Password desktop MCP server into the Cursor plugin manifest, add branding assets, document installation and usage in the README, and ship the 1password-environments agent skill with reference material for import-and-mount workflows.
Block direct .env and MCP doc reads that bypass the skill, nudge agents toward MCP-based imports, and require reading the skill before 1Password MCP tool calls.
scottisloud
force-pushed
the
scoot/feature/add-mcp-integration
branch
from
87be6b7 to
081607a
Compare
JillRegan
left a comment
JillRegan
left a comment
There was a problem hiding this comment.
Tested and works as expected! Left some comments.
Another thing I want to mention is the .cmd wrapper on Windows (you'll notice we have this for validate-mounted-env-files which was implemented here). Essentially cursor tries to run the bash hook commands, and without .cmd file it can attempt the bash script and fail badly which opens the file in the editor. The .cmd stub just returns allow since mounts aren’t supported on Windows anyway.
So if on Windows and the plugin is installed and these new hooks execute it will be an unpleasant user experience because the bash scripts will fail and open in the editor. While I don't think we should address in this PR I do think we should work on a quick follow up. But we may need more discussion with the Cursor team to see how other folks are handling this, as a .cmd file for every hook is not ideal.
|
Confirmed, Cursor seems to not have recognized it was failing to correctly load the plugin while I was developing it. As as a result, it implemented a TON of additional scaffolding that isn't required. I'll make local changes to the marketplace-installed version of the plugin (which Cursor abides by) so I can accurately assess what's required for the desired behaviour, pull out what isn't, and port that over here. |
Remove MCP skill-read gates, doc-deny hooks, import nudges, and the agent rule. Update SKILL.md, reference.md, and README to document only what the plugin ships. Co-authored-by: Cursor <cursoragent@cursor.com>
Bring over duplicate environment name handling in the skill, richer plugin manifest description, and simplified hooks.json from mcp-plugin-test-marketplace. Co-authored-by: Cursor <cursoragent@cursor.com>
2 participants
Summary
1passwordCursor plugin at v1.2.0mcp.jsonpointing at the local1password-mcpbinaryskills/1password-environments/SKILL.mdfor agent MCP workflows.gitignoreTest plan
/add-plugin 1passwordor local copybeforeShellExecutionhook appears in Settings → Hooksecho hello); verify allow/deny in Execution Log