capture follow-up: close-session SKILL veneer over the capture tool #62

New Issue

2026-06-23T05:44:30Z

mathias commented

2026-06-23 05:44:30 +00:00

Parent epic: #49 (capture, complete, v0.11.0) · deferred follow-up

Scope

Wire the existing close-session skill (skills/close-session/SKILL.md) to call the now-shipped capture capability instead of doing its brain/gitea writes inline. The skill becomes the trigger + harvest veneer over capture — exactly the role the architecture (#49) assigned it.

Context

capture shipped in #49 (PRs #56–#61, v0.11.0): one CaptureService behind POST /capture (REST) and an MCP capture tool. The MCP tool is live on the claude.ai connector. The close-session skill currently does its own Phase 4/5 writes (summary commit + brain note) directly; it should instead harvest the session into a capture call and let the server own persistence, the I1 gate, the I5 audit, and the supersession/staleness discipline.

What to do

Replace close-session Phase 4 (summary commit) + Phase 5 (brain note) with a single capture invocation (MCP tool on claude.ai), passing assembled insights + tickets + summary + context (harness claudeai-chat, fidelity: live-capture).
Phase 3 (issue actions) can route through capture's tickets array rather than direct gitea:* calls — consolidates the write path and gets the audit record for free.
Keep the harvest/gating prose (Phases 1–3) and the confirmation gates; only the persistence mechanism changes from inline tools → capture.
The skill no longer needs to carry the staleness rule (no-semantic-query-after-supersede) — capture owns it server-side. Simplify the Phase 5 prose accordingly.
Update fidelity handling: live-capture from claude.ai; note that a later transcript-parse capture supersedes it (the collision rule, now server-enforced).

Acceptance

close-session Phases 4–5 (and 3's writes) route through the capture MCP tool
Skill assembles context with harness + fidelity + (optional) classification
Staleness-discipline prose removed from the skill (server owns it now)
Confirmation gates preserved
Tested against a real session closeout (this is also the skill's first live test)
Committed to skills/close-session/SKILL.md

**Parent epic:** #49 (capture, complete, v0.11.0) · deferred follow-up ## Scope Wire the existing `close-session` skill (`skills/close-session/SKILL.md`) to call the now-shipped `capture` capability instead of doing its brain/gitea writes inline. The skill becomes the **trigger + harvest veneer** over `capture` — exactly the role the architecture (#49) assigned it. ## Context `capture` shipped in #49 (PRs #56–#61, v0.11.0): one `CaptureService` behind `POST /capture` (REST) and an MCP `capture` tool. The MCP tool is live on the claude.ai connector. The `close-session` skill currently does its own Phase 4/5 writes (summary commit + brain note) directly; it should instead **harvest** the session into a `capture` call and let the server own persistence, the I1 gate, the I5 audit, and the supersession/staleness discipline. ## What to do - Replace close-session Phase 4 (summary commit) + Phase 5 (brain note) with a single `capture` invocation (MCP tool on claude.ai), passing assembled `insights` + `tickets` + `summary` + `context` (harness `claudeai-chat`, `fidelity: live-capture`). - Phase 3 (issue actions) can route through `capture`'s `tickets` array rather than direct `gitea:*` calls — consolidates the write path and gets the audit record for free. - Keep the harvest/gating prose (Phases 1–3) and the confirmation gates; only the **persistence mechanism** changes from inline tools → `capture`. - The skill no longer needs to carry the staleness rule (no-semantic-query-after-supersede) — `capture` owns it server-side. Simplify the Phase 5 prose accordingly. - Update `fidelity` handling: live-capture from claude.ai; note that a later transcript-parse capture supersedes it (the collision rule, now server-enforced). ## Acceptance - [ ] close-session Phases 4–5 (and 3's writes) route through the `capture` MCP tool - [ ] Skill assembles `context` with harness + fidelity + (optional) classification - [ ] Staleness-discipline prose removed from the skill (server owns it now) - [ ] Confirmation gates preserved - [ ] Tested against a real session closeout (this is also the skill's first live test) - [ ] Committed to `skills/close-session/SKILL.md` ## Related - #49 (capture epic), `specs/capture-bdd-spec.md`, `specs/capture-implementation-report.md` - `skills/close-session/SKILL.md` (the veneer target)

mathias referenced this issue

2026-06-23 05:45:02 +00:00

capture follow-up: harvest adapters (transcript-parse / chat-reconstruct / agent-runlog) #64

mathias referenced this issue from a commit

2026-06-23 06:27:58 +00:00

feat(close-session): route closeout through the capture tool (#62 minimal)

mathias referenced this issue from a commit

2026-06-26 14:25:58 +00:00

docs(close-session): refresh classification gate post-#67 (#62)

mathias referenced a pull request that will close this issue

2026-06-26 14:26:11 +00:00

docs(close-session): refresh classification gate post-#67 (#62) #69