Skip to content

docs(specs): enumerate sub-agent transcripts as 4th persistence surface in spec-072 - #6316

Merged
bug-ops merged 1 commit into
mainfrom
feat/issue-6311/spec-072-4-c1-fr-008-ac-5-does
Jul 16, 2026
Merged

docs(specs): enumerate sub-agent transcripts as 4th persistence surface in spec-072#6316
bug-ops merged 1 commit into
mainfrom
feat/issue-6311/spec-072-4-c1-fr-008-ac-5-does

Conversation

@bug-ops

@bug-ops bug-ops commented Jul 16, 2026

Copy link
Copy Markdown
Owner

Summary

  • spec-072 §4 (invariant C1, FR-008, AC-5, threat model) previously enumerated only three persistence surfaces requiring MessagePart::Image stripping: SQLite parts_json, Qdrant embeddings, and the durable JSONL session log.
  • PR fix(subagent): strip MessagePart::Image from transcript writes #6310 added a fourth strip site — sub-agent transcripts (TranscriptWriter::append in crates/zeph-subagent/src/transcript.rs, calling MessagePart::strip_images) — but the spec text was never updated to reflect it.
  • This is a doc-only change: updates all five affected sections of spec.md (C1 enumeration, C1 scope, FR-008, AC-5, threat model) plus a CHANGELOG.md entry. No source files touched.

Closes #6311

Test plan

  • Verified TranscriptWriter::append calls MessagePart::strip_images before writing (crates/zeph-subagent/src/transcript.rs:127-129, crates/zeph-llm/src/provider.rs:369)
  • Verified zeph-subagent does not route through Agent::persist_message, confirming the "structurally separate path" claim
  • Grepped the full spec file for stale "three persistence surfaces" language — none remains
  • git diff --stat confirms only specs/072-multimodal-mcp-passthrough/spec.md and CHANGELOG.md changed
  • Line-wrap convention verified for the edited C1 paragraph (all lines within surrounding ~90-102 char convention)

@github-actions github-actions Bot added documentation Improvements or additions to documentation size/S Small PR (11-50 lines) labels Jul 16, 2026
…ce in spec-072

Sub-agent transcripts (TranscriptWriter::append) were added as a fourth
MessagePart::Image strip site by PR #6310, but spec-072 section 4 (C1,
FR-008, AC-5, threat model) still only listed three persistence surfaces.
@bug-ops
bug-ops enabled auto-merge (squash) July 16, 2026 17:24
@bug-ops
bug-ops force-pushed the feat/issue-6311/spec-072-4-c1-fr-008-ac-5-does branch from f05ea51 to 567d606 Compare July 16, 2026 17:24
@bug-ops
bug-ops merged commit bdabe50 into main Jul 16, 2026
23 checks passed
@bug-ops
bug-ops deleted the feat/issue-6311/spec-072-4-c1-fr-008-ac-5-does branch July 16, 2026 17:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation size/S Small PR (11-50 lines)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

spec-072 §4 C1/FR-008/AC-5 does not enumerate sub-agent transcripts as a 4th persistence surface

1 participant