Skip to content

fix(agents): probe Cursor version via resolved cursor-agent - #718

Merged
Drswith merged 5 commits into
mainfrom
cursor/fix-cursor-version-probe-f66c
Sep 6, 2026
Merged

fix(agents): probe Cursor version via resolved cursor-agent#718
Drswith merged 5 commits into
mainfrom
cursor/fix-cursor-version-probe-f66c

Conversation

@Drswith

@Drswith Drswith commented Sep 6, 2026

Copy link
Copy Markdown
Owner

Summary

qtx ls and qtx inspect cursor could report a version from an unrelated agent binary on PATH. Cursor CLI stays catalogued as name=cursor, binaryName=agent, aliases ["agent"]. Observation now locates cursor-agent first via an internal targeting table keyed by canonical name, then probes that resolved absolute path.

Closed #714 failed CI by renaming binaryName to cursor-agent and adding lookup alias cursor. This change does not touch catalog identity or list/inspect JSON fields.

Product gate: the v1 root declaration type set and exported symbol set stay unchanged. preferredBinaries is not a catalog field, not an AgentVersionProbe member, and not a root export. test/fixtures/compatibility/v1/root-declaration.json remains the existing pin (49262 / 8a2190…) because the built dist/index.d.mts matches main with no type/export expansion.

Rebased onto main after #717. No P9, no config fold, no YAML / release-core.yml / protect-main.

Linked Artifacts

Validation

  • bun run lint
  • bun run format:check
  • bun run typecheck
  • bun run test (if behavior changed)
  • bun run memory:check
  • bun run openspec:validate

Local results on 36ca3cc: lint 0 warnings / 0 errors; format and typecheck passed; OpenSpec 26/26; memory check passed; Vitest 176 files, 2138 passed, 5 skipped; bun run package:check passed. Built dist/index.d.mts matches the maintained v1 pin: 49262 bytes / 8a219038d08076caf26848dade2872ede86c823b3bccf55d421fed0c9d411d9f. AgentVersionProbe still has only command and parser. No preferredBinaries in the declaration. Probe helpers are not root exports.

Release Intent

  • Release: patch - bug fix

Release Summary

BEGIN_COMMIT_OVERRIDE
fix(agents): report Cursor CLI version from the resolved cursor-agent binary

qtx ls and qtx inspect cursor now probe the Cursor executable Quantex resolved (cursor-agent when present) instead of the first agent on PATH.
END_COMMIT_OVERRIDE

Docs Updated

  • openspec/...

Scope Check

  • I did not add a new ad hoc root-level Markdown file.
  • I updated the relevant issue, ADR, spec, runbook, or captured the missing doc work as follow-up.
  • I did not silently expand project scope without recording it explicitly.

Closure Check

  • Working tree was clean after commit.
  • Branch was pushed and this PR is the active delivery artifact.
  • OpenSpec change is active and will be archived after merge.
  • Release is delegated to release automation.

Notes

CI on 36ca3cc is green: lint, governance, test (ubuntu-latest), test (macos-latest). Windows is advisory and also passed. Canonical agent id remains cursor; binaryName remains agent; aliases must not include cursor.

Open in Web Open in Cursor 

Record the PATH `agent` collision fix as probe targeting, without renaming Cursor catalog identity.
Prefer `cursor-agent` when locating the Cursor CLI executable so list/inspect versions match that binary instead of a colliding `agent` on PATH. Keep binaryName and aliases unchanged.
The fallback fixture omitted /usr/bin, so which could not spawn and Cursor looked absent.
Keep preferredBinaries documented with line comments so tsdown does not
emit JSDoc into the pinned dist/index.d.mts, then update the maintained
v1 signature fixture to the new bytes/sha256.
@cursor
cursor Bot force-pushed the cursor/fix-cursor-version-probe-f66c branch from 6a278fb to fe820ac Compare September 6, 2026 14:39
Prefer cursor-agent via an internal lookup table keyed by canonical
name. Restore AgentVersionProbe, catalog JSON/schema, and the v1
root-declaration pin so the exported type and symbol sets stay
unchanged.
@Drswith
Drswith marked this pull request as ready for review September 6, 2026 14:57
@Drswith
Drswith enabled auto-merge (squash) September 6, 2026 15:00
@Drswith
Drswith merged commit ea6e24b into main Sep 6, 2026
22 of 23 checks passed
@Drswith
Drswith deleted the cursor/fix-cursor-version-probe-f66c branch September 6, 2026 15:01
Drswith added a commit that referenced this pull request Sep 6, 2026
Sync accepted Cursor probe-targeting deltas into living specs after #718,
and drop the completed change from the working tree without committing the
generated archive directory.

Co-authored-by: Cursor Agent <cursoragent@cursor.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants