Skip to content

Fix: remove hook runtime context tmp .json files#104

Merged
davidabram merged 5 commits into
mainfrom
fix/remove-hook-runtime-context-tmp-artifacts
Jun 30, 2026
Merged

Fix: remove hook runtime context tmp .json files#104
davidabram merged 5 commits into
mainfrom
fix/remove-hook-runtime-context-tmp-artifacts

Conversation

@ivke995

@ivke995 ivke995 commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

No description provided.

ivke995 and others added 5 commits June 29, 2026 15:38
Stop diff-trace hook intake from writing parsed payload artifacts under context/tmp, leaving AgentTraceDb as the only persistence path for diff-trace payloads. Update current-state context to reflect the DB-only runtime behavior.

Plan: remove-hook-runtime-context-tmp-artifacts

Task: T01

Co-authored-by: SCE <sce@crocoder.dev>
Remove the remaining post-commit hook trace writer and its unused collision-safe JSON artifact helpers so hook runtime no longer writes post-commit artifacts under context/tmp.

Update current-state context to reflect DB-only hook persistence and record completion evidence for the artifact-removal plan.

Plan: remove-hook-runtime-context-tmp-artifacts
Task: T02

Co-authored-by: SCE <sce@crocoder.dev>
Record completion evidence for manually removed context/tmp hook-runtime artifacts while preserving sce.log and its creation behavior.

Plan: remove-hook-runtime-context-tmp-artifacts

Task: T03

Co-authored-by: SCE <sce@crocoder.dev>
is AgentTraceDb-only with no context/tmp parsed-payload artifacts.

- overview.md: Describe diff-trace as AgentTraceDb-only without
  context/tmp parsed-payload artifacts
- patterns.md: Clarify unresolved attribution persists as NULL
  in AgentTraceDb rather than referencing artifacts
- sce/claude-raw-hook-capture.md: Describe nullable attribution
  persisted to AgentTraceDb without raw hook artifacts
- plans/remove-hook-runtime-context-tmp-artifacts.md: Mark T04
  complete with completion metadata (date, files, evidence)

Co-authored-by: SCE <sce@crocoder.dev>
…oval

Final validation task for the remove-hook-runtime-context-tmp-artifacts
plan. All checks pass (nix flake check, pkl-check-generated), stale
hook-runtime artifact references are absent from code and context docs,
residual artifacts cleaned from context/tmp/, and SCE logs preserved.
All 9 success criteria confirmed met.

Plan: remove-hook-runtime-context-tmp-artifacts
Tasks: T05

Co-authored-by: SCE <sce@crocoder.dev>
@davidabram davidabram merged commit dfa05d9 into main Jun 30, 2026
6 checks passed
@davidabram davidabram deleted the fix/remove-hook-runtime-context-tmp-artifacts branch June 30, 2026 14:28
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