Skip to content

fix(autosave): include branchName in persisted snapshot#36

Merged
johannesjo merged 1 commit into
mainfrom
claude/review-previous-changes-lOQNM
Mar 27, 2026
Merged

fix(autosave): include branchName in persisted snapshot#36
johannesjo merged 1 commit into
mainfrom
claude/review-previous-changes-lOQNM

Conversation

@johannesjo
Copy link
Copy Markdown
Owner

updateTaskBranch (used by MergeDialog's branch mismatch fix) updates
branchName in the store, but the autosave snapshot didn't track it.
If branchName was the only field that changed, the debounced save
would not trigger, potentially losing the update on app restart.

https://claude.ai/code/session_01McqHYwdaCNNXqPQWGBTGS4

updateTaskBranch (used by MergeDialog's branch mismatch fix) updates
branchName in the store, but the autosave snapshot didn't track it.
If branchName was the only field that changed, the debounced save
would not trigger, potentially losing the update on app restart.

https://claude.ai/code/session_01McqHYwdaCNNXqPQWGBTGS4
@johannesjo johannesjo merged commit 39aff3c into main Mar 27, 2026
2 checks passed
brooksc added a commit to brooksc/parallel-code that referenced this pull request May 11, 2026
…johannesjo#39

- johannesjo#33: restart round-trip — hydrateTask rewrites config with new subtaskToken
  (not coordinator token) when setMCPServerInfo has already run; and
  waitForIdle resolves after agent output fires post-hydration
- johannesjo#36: mcpConfigPath directory scoping — path traversal rejected, wrong-dir
  rejected, correct host tmpdir accepted, Docker dirname(serverPath) accepted,
  Docker wrong-dir rejected
- johannesjo#39: per-task close isolation — closing task-1 deletes only its config file;
  task-2 config and task entry remain untouched

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
brooksc added a commit to brooksc/parallel-code that referenced this pull request May 14, 2026
…johannesjo#39

- johannesjo#33: restart round-trip — hydrateTask rewrites config with new subtaskToken
  (not coordinator token) when setMCPServerInfo has already run; and
  waitForIdle resolves after agent output fires post-hydration
- johannesjo#36: mcpConfigPath directory scoping — path traversal rejected, wrong-dir
  rejected, correct host tmpdir accepted, Docker dirname(serverPath) accepted,
  Docker wrong-dir rejected
- johannesjo#39: per-task close isolation — closing task-1 deletes only its config file;
  task-2 config and task entry remain untouched

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
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