Skip to content

Consolidate thread runtime state and use typed UI snapshots - #53

Merged
VolkerChristian merged 5 commits into
masterfrom
polish/pr4-thread-runtime-and-typed-snapshots
Aug 27, 2026
Merged

Consolidate thread runtime state and use typed UI snapshots#53
VolkerChristian merged 5 commits into
masterfrom
polish/pr4-thread-runtime-and-typed-snapshots

Conversation

@VolkerChristian

Copy link
Copy Markdown
Member

Summary

  • consolidate per-thread hydration, settings, dispatch, resume, and recovery bookkeeping into one ThreadRuntimeState record
  • preserve narrow reconnect, reload, and thread-removal semantics while eliminating synchronized parallel-container cleanup
  • replace synthetic JSON and serialized fingerprints with typed optional snapshots in Shell, ThreadPane, and Inspector rendering
  • compare GitDiffSnapshot values directly instead of allocating a byte buffer and calculating SHA-256
  • refresh request thread labels after renames and Git presentation after metadata-only changes
  • top-align rich-text subagent results so surplus label height does not create a large blank gap

Regression coverage

  • request cards update their displayed thread title after thread.name.changed
  • Git Changes updates when only previously omitted additions and deletions metadata changes
  • agent Markdown result labels remain top-aligned

Validation

  • focused Application Layout, Shell Integration, and Git Changes Live checks passed
  • complete CTest suite: 7/7 passed
  • git diff --check passed

Scope

This remains a behavior-preserving ownership and type-safety cleanup. It adds no controllers, caches, repositories, event systems, or protocol-authority changes. HistoryWindow and PromptCoordinator ownership remain unchanged.

@VolkerChristian
VolkerChristian merged commit 934c742 into master Aug 27, 2026
1 check 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.

1 participant