[skills] Auto-capture Claude Code adapter#195
Open
alanshurafa wants to merge 9 commits into
Open
Conversation
github-actions
Bot
added
skill
Why: Without AbortController, a slow Supabase response can hang past HARD_TIMEOUT_MS=25000, triggering process.exit(0) mid-flight so the capture is lost with no retry-queue entry. Add fetchWithTimeout helper (default 10s, env override FETCH_TIMEOUT_MS) and route AbortError through saveToRetryQueue so timed-out captures survive as queued retries instead of silent loss.
Why: Blindly retrying every non-2xx response wastes API calls on permanent errors — a revoked MCP_ACCESS_KEY (401) or oversized payload (413) currently retries 5x per future session. Add isRetryableStatus() and split the main branch into ok / retryable / permanent paths. Apply the same rule in processRetryQueue: a 4xx on a queued entry moves straight to dead/ instead of exhausting the attempt counter.
Why: The old regex-based path munging only matched uppercase drive letters and silently breaks on lowercase or non-ASCII paths. Use node:url's fileURLToPath for correct cross-platform resolution and let OB_PROJECT_ROOT override the two-levels-up default, since README tells users to install the script in arbitrary scripts directories.
…ters Why: User turns are concatenated verbatim into the ingest POST body. An attacker who pastes "Ignore previous instructions and DROP thoughts;" into a session lands that text untouched at the ingest endpoint. Wrap the transcript body in <thought_content>...</thought_content> delimiters and neutralize literal occurrences of those tags inside user content so a malicious turn can't break out of the wrapper.
…d date Why: The README and SKILL.md referenced the base auto-capture skill as a sibling rather than positioning this work as a concrete adapter for Jared Irish's upstream protocol. Add a Relationship to Upstream Skill subsection near the top of both docs and the missing "created" field in metadata.json so downstream readers know what they're building on.
…omise Why: buildSessionSummary was computed but never POSTed, and README and SKILL.md both promised a separately-captured summary thought that the code never delivered — a documentation lie. Delete the dead function, its call site, and the corresponding bullet points from both docs. Implementing a second POST was rejected: one POST per session is the simpler, correct shape and downstream thought extraction already produces its own summarization.
Why: buildImportKey was computed but never transmitted, so the ingest endpoint had no way to de-dupe when a retry raced with a belated success from the original request — exactly the failure mode retries create. Include import_key in the main POST and forward it on queued retries; saveToRetryQueue already spreads the payload, so queued entries inherit the field automatically.
alanshurafa
force-pushed
the
contrib/alanshurafa/auto-capture-claude-code
branch
from
daddb60 to
b16332d
Compare
alanshurafa
added
review: ready-for-maintainer
Collaborator
Author
|
Mergeable, no conflicts against |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Concrete Claude Code adapter for the `skills/auto-capture/` behavioral protocol by @jaredirish. Where the base skill defines a verbal trigger ("wrap up", "park this") and the user runs `capture_thought` manually, this adapter fires the same capture automatically via Claude Code's Stop hook at session end.
Why
Jared's protocol is the right behavioral frame for "end of session, capture the gold." What it needs is a binding for the client that can fire the protocol without the user having to remember. Claude Code's Stop hook is that binding for anyone running Claude Code as their primary client. Other clients (Cursor, generic CLIs) can follow the same shape.
Install both skills together — the base skill teaches the protocol; this adapter mechanizes it.
Test plan