docs(persistence): record #2 spike findings (resume + git identity) #11

Closed

claude-desktop wants to merge 0 commits from boss/2 into main

claude-desktop commented

2026-04-16 23:20:33 +00:00

Collaborator

Summary

Captures the decisions from the #2 timeboxed spike so #3 can start without re-litigating.

SDK resume: session_id read from SDKSystemMessage (subtype: 'init'); stored under ~/.claude/projects/<sanitized-cwd>/… — so resume is bound to the project dir, but the SDK treats git worktrees of the same repo as the same project bucket (exactly what the worktree design wants).
Git identity: option A — pass GIT_AUTHOR_* / GIT_COMMITTER_* via query({ env }). No per-worktree config file needed.
Git auth: URL rewrite via GIT_CONFIG_COUNT + url.<base>.insteadOf env injection. Token never touches .git/config and only lives in the query() child process env.

Spike code itself is throwaway per the acceptance criteria; only the decision doc is merged.

Closes #2.

Test plan

bunx tsc --noEmit — clean (no code changes)
bunx biome check src/ — clean
bunx biome format src/ — clean
bun test — 16 pass
Reviewer confirms the resume / identity / auth decisions are sound before #3 starts.

## Summary Captures the decisions from the #2 timeboxed spike so #3 can start without re-litigating. - **SDK resume:** `session_id` read from `SDKSystemMessage` (`subtype: 'init'`); stored under `~/.claude/projects/<sanitized-cwd>/…` — so resume is bound to the project dir, but the SDK treats git worktrees of the same repo as the same project bucket (exactly what the worktree design wants). - **Git identity:** option **A** — pass `GIT_AUTHOR_*` / `GIT_COMMITTER_*` via `query({ env })`. No per-worktree config file needed. - **Git auth:** URL rewrite via `GIT_CONFIG_COUNT` + `url.<base>.insteadOf` env injection. Token never touches `.git/config` and only lives in the `query()` child process env. Spike code itself is throwaway per the acceptance criteria; only the decision doc is merged. Closes #2. ## Test plan - [x] `bunx tsc --noEmit` — clean (no code changes) - [x] `bunx biome check src/` — clean - [x] `bunx biome format src/` — clean - [x] `bun test` — 16 pass - [ ] Reviewer confirms the resume / identity / auth decisions are sound before #3 starts.

claude-desktop added 1 commit

2026-04-16 23:20:33 +00:00

docs(persistence): record spike findings for SDK resume + git identity

qa / qa (pull_request) Successful in 44s

Details

a9db68fdf4

Captures the decisions from the #2 spike so #3 can start without
re-litigating:

- SDK session_id is read from SDKSystemMessage (subtype 'init').
- Resume is local-filesystem only; worktrees of the same repo share a
  project bucket, so resume works across dispatches that land in
  different worktrees.
- Git identity: pass GIT_AUTHOR_*/GIT_COMMITTER_* via query({ env })
  instead of per-worktree config.
- Git auth: GIT_CONFIG_COUNT + url.<base>.insteadOf env injection, so
  the token never lands in .git/config.

Closes #2.