Skip to content

refactor(client): isolate presentation state - #943

Closed
chughtapan wants to merge 2 commits into
mainfrom
agent/harness-context-state
Closed

refactor(client): isolate presentation state#943
chughtapan wants to merge 2 commits into
mainfrom
agent/harness-context-state

Conversation

@chughtapan

Copy link
Copy Markdown
Owner

Outcome

Extract the existing in-memory conversation, message, agent-name, and viewer-context marker state from MoltZapService into one package-private PresentationState owner.

This is a behavior-preserving production refactor after #942. It creates the seam needed to move presentation/context ownership into HarnessClient without changing the current wire, public types, marker semantics, rendering, deduplication, history reads, or reply path.

Details

  • MoltZapService delegates existing cache and cross-conversation selection behavior to the internal presentation folder.
  • Rendering remains caller-owned and is injected as a pure message renderer.
  • Notification tracing moves into the existing notification folder.
  • FakeMoltZapService retains its original uncapped message-seeding behavior.
  • Generated client module docs are refreshed.

Verification

  • workspace precommit: 61/61 Nx targets
  • client unit tests: 221/221
  • focused service/context/raw tests: 57/57
  • client build, test typecheck, lint, and architecture checks
  • independent semantic review, including the uncapped fake-state regression fix
  • formatting and diff integrity

Tracks #926.

@mintlify

mintlify Bot commented Aug 4, 2026

Copy link
Copy Markdown

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
moltzap 🟢 Ready View Preview Aug 4, 2026, 5:32 AM

💡 Tip: Enable Workflows to automatically generate PRs for you.

@chughtapan

Copy link
Copy Markdown
Owner Author

Superseded by #974, which collapses this stack into one PR against main. This branch's commits are all contained in it — nothing is lost, and this thread stays readable. Closing to leave one place to review.

@chughtapan chughtapan closed this Aug 6, 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