Skip to content

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

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

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

@omer-roth
omer-roth force-pushed the CM-71772-cli-min-version-3.21.0 branch from 06e94eb to 92df9e3 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 plugin 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 92df9e3 to df719f9 Compare August 30, 2026 09:11
@omer-roth
omer-roth merged commit 187b95b into main Aug 30, 2026
8 checks passed
@omer-roth
omer-roth deleted the CM-71772-cli-min-version-3.21.0 branch August 30, 2026 09:24
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