Skip to content

CM-71772: increase required CLI version to 3.21.0 - #155

Merged
omer-roth merged 1 commit into
mainfrom
CM-71772-cli-min-version-3.21.0
Aug 30, 2026
Merged

CM-71772: increase required CLI version to 3.21.0#155
omer-roth merged 1 commit into
mainfrom
CM-71772-cli-min-version-3.21.0

Conversation

@omer-roth

@omer-roth omer-roth commented Aug 30, 2026

Copy link
Copy Markdown
Collaborator

Why

Parallel scans spawn one cycode CLI process per scan type, and every process does a non-atomic read-modify-write on ~/.cycode/credentials.yaml. Concurrent writes corrupt the file, cycode status then crashes, the plugin reports "not authenticated", and the browser login that follows succeeds server-side but is lost at write-back — an auth loop the user cannot escape. See CM-71772.

Fixed in cycode-cli v3.21.0 (0% corruption / 0 reader crashes vs 96.7% on 3.18.0), published 2026-08-25 with all per-OS assets.

What

Required CLI version bumped to 3.21.0. CLI_GITHUB.TAG derives from the constant, so the download tag follows. Plugin version + CHANGELOG bumped alongside.

The exact-match pin behaviour is unchanged — the plugin still installs exactly this version.

Out of scope

CLI-side items from the ticket: serialising device-auth initiation across processes, and surfacing write-back failures to the user.

🤖 Generated with Claude Code

Comment thread pnpm-lock.yaml Outdated
@omer-roth
omer-roth force-pushed the CM-71772-cli-min-version-3.21.0 branch from 8b30bf4 to 53f09ab Compare August 30, 2026 08:37
@omer-roth omer-roth changed the title CM-71772: require cycode-cli 3.21.0 and treat the pin as a minimum CM-71772: increase required CLI version to 3.21.0 Aug 30, 2026
Concurrent CLI processes spawned by parallel scans corrupt
~/.cycode/credentials.yaml, which leaves the user in an auth loop.
That is fixed in cycode-cli 3.21.0, but the extension pinned 3.12.2,
so no user could receive the fix.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@omer-roth
omer-roth force-pushed the CM-71772-cli-min-version-3.21.0 branch from 53f09ab to ee92295 Compare August 30, 2026 09:11
@omer-roth
omer-roth enabled auto-merge (squash) August 30, 2026 09:12
@omer-roth
omer-roth merged commit a156b7a into main Aug 30, 2026
7 checks passed
@omer-roth
omer-roth deleted the CM-71772-cli-min-version-3.21.0 branch August 30, 2026 09:14
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