Skip to content

Docs: add v1.7.2.4.1 roadmap entry, fix stale lock-file limitation - #13

Merged
Wewoc merged 1 commit into
mainfrom
claude/changelog-adjustments-ideit9
Sep 23, 2026
Merged

Wewoc merged 1 commit into
mainfrom
claude/changelog-adjustments-ideit9

Conversation

@Wewoc

@Wewoc Wewoc commented Sep 23, 2026

Copy link
Copy Markdown
Owner

… lock-file limitation

New ROADMAP.md entry for v1.7.2.4.1 (planned) — self-updater startup handshake + auto-rollback via _update_backup/, replacing today's silent-failure behavior in updater_helper.ps1's GUI-triggered path. Handshake shape (app writes a success flag once up and interactive, helper polls with a generous timeout) chosen over a plain PID-liveness timeout after review: a bare timeout races against first-launch AV scanning and doesn't distinguish 'alive' from 'actually started correctly'. Entry documents an open design point (the existing migration confirmation dialog must not gate the flag, or a user who hasn't clicked yet triggers a false rollback) and a correctly-scoped data-safety caveat (mcp_cache.db is an additive-only, rebuildable proxy cache outside ExeDir — the real analogous risk is the archive structure migration behind _check_migration(), already interactive/ opt-in, out of scope for this round).

Placed directly after 'Currently stable — v1.7.2.4', ahead of v1.7.2.5/v1.7.2.6 — it is a patch on v1.7.2.4, not a successor to v1.7.2.6, so version order puts it first regardless of write order.

Also fixes a stale-docs inconsistency found during the same review: CHANGELOG.md's v1.7.2.4 'Known limitations' and ROADMAP.md both still claimed no lock file existed against concurrent update triggers, but updater_helper.ps1 already has one (_update_helper.lock), added within that same v1.7.2.4 commit. Removed both stale mentions.

Claude-Session: https://claude.ai/code/session_011Z4cpStY3EYYqyd1CLsGhJ

… lock-file limitation

New ROADMAP.md entry for v1.7.2.4.1 (planned) — self-updater startup
handshake + auto-rollback via _update_backup/, replacing today's
silent-failure behavior in updater_helper.ps1's GUI-triggered path.
Handshake shape (app writes a success flag once up and interactive,
helper polls with a generous timeout) chosen over a plain PID-liveness
timeout after review: a bare timeout races against first-launch AV
scanning and doesn't distinguish 'alive' from 'actually started
correctly'. Entry documents an open design point (the existing
migration confirmation dialog must not gate the flag, or a user who
hasn't clicked yet triggers a false rollback) and a correctly-scoped
data-safety caveat (mcp_cache.db is an additive-only, rebuildable
proxy cache outside ExeDir — the real analogous risk is the archive
structure migration behind _check_migration(), already interactive/
opt-in, out of scope for this round).

Placed directly after 'Currently stable — v1.7.2.4', ahead of
v1.7.2.5/v1.7.2.6 — it is a patch on v1.7.2.4, not a successor to
v1.7.2.6, so version order puts it first regardless of write order.

Also fixes a stale-docs inconsistency found during the same review:
CHANGELOG.md's v1.7.2.4 'Known limitations' and ROADMAP.md both still
claimed no lock file existed against concurrent update triggers, but
updater_helper.ps1 already has one (_update_helper.lock), added within
that same v1.7.2.4 commit. Removed both stale mentions.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_011Z4cpStY3EYYqyd1CLsGhJ
@coderabbitai

coderabbitai Bot commented Sep 23, 2026

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: 6325ec5c-6cb5-4394-8ca2-19aa871589c3

📥 Commits

Reviewing files that changed from the base of the PR and between 1cacd9d and 494dde7.

📒 Files selected for processing (2)
  • src/docs/CHANGELOG.md
  • src/docs/ROADMAP.md
 _________________________________________________
< Let's pair program: you type, I judge lovingly. >
 -------------------------------------------------
  \
   \   (\__/)
       (•ㅅ•)
       /   づ
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

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.

@Wewoc
Wewoc merged commit 87fe28e into main Sep 23, 2026
1 of 2 checks passed
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