Skip to content

runtime: make Full Skin switching transactional - #19

Closed
rwang23 wants to merge 1 commit into
Codex/audit-pr01-hermetic-testsfrom
Codex/audit-pr04-runtime-transaction
Closed

runtime: make Full Skin switching transactional#19
rwang23 wants to merge 1 commit into
Codex/audit-pr01-hermetic-testsfrom
Codex/audit-pr04-runtime-transaction

Conversation

@rwang23

@rwang23 rwang23 commented Jul 26, 2026

Copy link
Copy Markdown
Owner

Summary

  • stage candidate Full Skin injection before replacing the current session
  • roll back partial target injection and preserve the previous session on candidate failure
  • expose healthy, recovering, degraded, and stopped monitor states in the runtime and bilingual UI

Verification

  • npm run desktop:check
  • npm run test:source
  • node --check apps/theme-manager/src/renderer/app.js
  • git diff --check

Independent acceptance review

Pass with risks: targeted Rust regressions prove rollback ordering and monitor-state transitions. A real isolated Stable/Beta package + CDP switch smoke remains pending, so this PR does not claim live-app verification.

Closes #7

@rwang23

rwang23 commented Jul 27, 2026

Copy link
Copy Markdown
Owner Author

Superseded by #26, now merged into main at e2ed6e8. This branch head is already included in main; closing the draft avoids duplicate review and merge work.

@rwang23 rwang23 closed this Jul 27, 2026
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.

1 participant