Skip to content

docs(devlog): release-readiness note for the cursor-call integration - #1996

Merged
lidge-jun merged 1 commit into
devfrom
codex/cursor-call-release-note
Aug 18, 2026
Merged

docs(devlog): release-readiness note for the cursor-call integration#1996
lidge-jun merged 1 commit into
devfrom
codex/cursor-call-release-note

Conversation

@lidge-jun

@lidge-jun lidge-jun commented Aug 18, 2026

Copy link
Copy Markdown
Owner

Summary

Release-readiness note for the cursor-call integration campaign (#1993 / #1994 / #1995, merged as 4f72d6f75).

Records the five release gates run against the merged dev SHA rather than the pre-merge tip — a merge commit is a tree nobody has tested until it is tested — and recommends cutting 2.25.0 as a minor, because the externally observable behaviour of a failed turn changed.

Says two things plainly instead of burying them: the merges were owner-authorized admin merges with CI waived, which is owner authority and not compliance with MAINTAINERS.md:48-49; and publishing is not one command away, since scripts/release.ts waits for Cross-platform CI and Service lifecycle at the exact release SHA.

Carries the five open follow-ups, led by the one the campaign's own docs previously overstated: the Cursor tool-result image encoder is correct and does not reach production, because every Cursor model is in noVisionModels and the vision sidecar strips images before the adapter runs.

Verification

Docs-only change — no source, tests, or build inputs touched.

The gates this note records ran on ssh lidge at 4f72d6f7555013ab231de78233d2aa95bd1e439c in a worktree pinned to that SHA, with HEAD re-asserted before each:

bun x tsc --noEmit          exit 0
bun run privacy:scan        Privacy scan passed
bun run audit:high          No vulnerabilities found (root + gui)
bun run build:gui           built in 205ms
bun test --isolate tests    green

Checklist

  • Scope stays focused and avoids unrelated cleanup.
  • Docs or release notes were updated when needed. (This PR is that update.)
  • Security-sensitive changes were reviewed for secrets, auth, and unsafe defaults. (Docs only; privacy:scan green at the recorded SHA.)

Summary by CodeRabbit

  • Documentation
    • Added release-readiness guidance for the cursor-call integration.
    • Documented validation requirements, governance considerations, cross-platform support, and service lifecycle checks.
    • Recorded the current version status and recommendation for version 2.25.0.
    • Listed promotion prerequisites, follow-up items, and known limitations.

The five gates ran against the MERGED dev SHA 4f72d6f, not the pre-merge tip — a
merge commit is a tree nobody has tested until it is tested. All green in a lidge
worktree pinned to that SHA, with HEAD re-asserted before each gate.

Recommends 2.25.0, a minor rather than a patch, because the externally observable
behaviour of a FAILED turn changed: a turn that came back completed with a vanished
tool call now reports failed, an unrequested CANCEL is now a typed failure, and a
truncated compaction turn no longer installs half-written replacement history.
Correct behaviour, but new behaviour for anything keying on "the proxy said
completed". The maintainer decides; the note recommends.

Says two things plainly rather than burying them. The merges were owner-authorized
admin merges with CI waived — that is owner authority over one's own repository, not
compliance with MAINTAINERS.md:48-49, and lidge is Linux-only while CI spans three
platforms. And publishing is not one command away: scripts/release.ts waits for a
successful Cross-platform CI run AND a Service lifecycle run at the exact release
SHA, so a green gate table alone does not mean bun run release will work.

Carries the five open follow-ups, led by the one the campaign's own docs previously
overstated: the Cursor tool-result image encoder is correct and does not reach
production, because every Cursor model is in noVisionModels and the vision sidecar
strips images before the adapter runs.
@lidge-jun
lidge-jun merged commit aad8e26 into dev Aug 18, 2026
6 checks passed
@github-actions

Copy link
Copy Markdown
Contributor

Deterministic PR hygiene checks passed.

@github-actions github-actions Bot added the documentation Improvements or additions to documentation label Aug 18, 2026
@coderabbitai

coderabbitai Bot commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: c3c29af6-f480-492c-8a72-558383de3db6

📥 Commits

Reviewing files that changed from the base of the PR and between 4f72d6f and 45ae409.

📒 Files selected for processing (1)
  • devlog/_plan/260818_cursor_call_integration/060_release_readiness.md

📝 Walkthrough

Walkthrough

The release-readiness note documents the recommended 2.25.0 release, merged-SHA validation, promotion prerequisites, version state, unresolved integration gaps, and issue disposition.

Changes

Cursor-call release readiness

Layer / File(s) Summary
Release evidence and version decision
devlog/_plan/260818_cursor_call_integration/060_release_readiness.md
Lines 1–81 record merged PR ancestry, validation gates, governance conditions, platform requirements, live versions, and the 2.25.0 recommendation.
Promotion prerequisites and sequence
devlog/_plan/260818_cursor_call_integration/060_release_readiness.md
Lines 83–101 define an unexecuted dev-to-preview and dev-to-main promotion sequence using required workflows and scripts/release.ts.
Remaining gaps and issue disposition
devlog/_plan/260818_cursor_call_integration/060_release_readiness.md
Lines 102–132 list unresolved integration gaps and summarize the status of related issues.

Estimated code review effort: 1 (Trivial) | ~3 minutes

Suggested reviewers: ingwannu

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch codex/cursor-call-release-note

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@lidge-jun
lidge-jun deleted the codex/cursor-call-release-note branch August 18, 2026 08:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant