Remove the harness-specific tool-failure buffer #1646

Closed
opened 2026-08-05 10:08:57 +00:00 by coilyco-ops · 1 comment
Member

Outcome

Remove the Claude transcript parser and Ward-owned local JSONL tool-failure buffer. Retain generic, secret-safe run artifacts as the supported evidence surface.

Scope

  • Delete agent_tool_failures.go, its tests, documentation, drain hook, sent-path handling, and harness transcript assumptions.
  • Delete the ~/.cache/agentic-os tool-failure output owned by this subsystem.
  • Do not generalize the parser or buffer into a new Ward subsystem in this issue.

Preserved boundary

  • Generic raw or redacted run artifacts and skill-produced summaries remain available where their owning contracts permit them.
  • Redaction and secret-safety requirements from #1582 remain mandatory.

Acceptance

  • Ward has no harness-specific transcript parsing or local tool-failure buffer.
  • No product documentation or runtime path refers to that buffer.
  • Retained generic artifacts still provide the supported diagnostic evidence without weakening redaction.

Coordinates with #1569 and #1582. Blocks documentation consolidation in #1624.

## Outcome Remove the Claude transcript parser and Ward-owned local JSONL tool-failure buffer. Retain generic, secret-safe run artifacts as the supported evidence surface. ## Scope * Delete `agent_tool_failures.go`, its tests, documentation, drain hook, sent-path handling, and harness transcript assumptions. * Delete the `~/.cache/agentic-os` tool-failure output owned by this subsystem. * Do not generalize the parser or buffer into a new Ward subsystem in this issue. ## Preserved boundary * Generic raw or redacted run artifacts and skill-produced summaries remain available where their owning contracts permit them. * Redaction and secret-safety requirements from #1582 remain mandatory. ## Acceptance * Ward has no harness-specific transcript parsing or local tool-failure buffer. * No product documentation or runtime path refers to that buffer. * Retained generic artifacts still provide the supported diagnostic evidence without weakening redaction. Coordinates with #1569 and #1582. Blocks documentation consolidation in #1624.
Author
Member

Decision resolved with Kai:

  • Remove only the Claude tool-failure producer, not every transcript-derived artifact.
  • Delete agent_tool_failures.go, its tests, the drain invocation, JSONL schema, classifier, fingerprinting, excerpts, buffer paths, and docs/tool-failures.md.
  • Remove the subsystem from the docs index and FEATURES.
  • Interpret the acceptance wording as no harness-specific tool-failure extraction. Preserve transcript capture, redaction, generic friction events, and secret-free skill-usage summaries.
  • Add no generalized parser, event stream, shipper, compatibility file, legacy warning, or tombstone behavior.
  • #1582 remains authoritative for persisted transcript and artifact safety. Any current allowance for raw artifacts does not survive #1582.
  • Stop creating or reading ~/.cache/ward/tool-failures/. Ward does not delete historical user cache files during upgrade or runtime.
  • Active checkout search found no downstream shipper consumer. Only a historical dispatch log mentions the older ~/.cache/agentic-os/tool-failures/ spelling.
  • Priority is P2 because this blocks #1624.
Decision resolved with Kai: * Remove only the Claude tool-failure producer, not every transcript-derived artifact. * Delete `agent_tool_failures.go`, its tests, the drain invocation, JSONL schema, classifier, fingerprinting, excerpts, buffer paths, and `docs/tool-failures.md`. * Remove the subsystem from the docs index and FEATURES. * Interpret the acceptance wording as no harness-specific tool-failure extraction. Preserve transcript capture, redaction, generic friction events, and secret-free skill-usage summaries. * Add no generalized parser, event stream, shipper, compatibility file, legacy warning, or tombstone behavior. * #1582 remains authoritative for persisted transcript and artifact safety. Any current allowance for raw artifacts does not survive #1582. * Stop creating or reading `~/.cache/ward/tool-failures/`. Ward does not delete historical user cache files during upgrade or runtime. * Active checkout search found no downstream shipper consumer. Only a historical dispatch log mentions the older `~/.cache/agentic-os/tool-failures/` spelling. * Priority is P2 because this blocks #1624.
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
coilyco-flight-deck/ward#1646
No description provided.