happy-stacks 0.6.9 → 0.6.10

package/docs/commit-audits/happy/leeroy-wip.commit-inventory.md

@@ -45,7 +45,7 @@ Progress tracking (update manually while auditing):
 - Next unreviewed commit: n/a (complete)
 - Pass 2 full-diff verification markers: `249/249` (each `## NNN` has a `Pass2FullDiff:` line; patch artifacts under `/tmp/leeroy-wip-pass2/NNN.patch`)
 - Pass 2 diff capture: `249/249` (`/tmp/leeroy-wip-pass2/NNN.patch` exists; hash/size recorded in each `Pass2FullDiff:` line)
-- Pass 2 notes verification (manual): `77/249` (this requires reading the patch and confirming/updating `### Manual Review Notes`)
+- Pass 2 notes verification (manual): `83/249` (this requires reading the patch and confirming/updating `### Manual Review Notes`)
 
 Current state (2026-01-24):
 

package/docs/commit-audits/happy/leeroy-wip.commit-manual-review.md

@@ -2792,7 +2792,7 @@ Pass2FullDiff: CAPTURED at 2026-01-24T17:39:06+0100 sha=cd5771e50881 patchFile=/
 ---
 
 ## 079 2026-01-12 456e2ad8e2b8 `resume`
-Pass2FullDiff: CAPTURED at 2026-01-24T17:39:06+0100 sha=456e2ad8e2b8 patchFile=/tmp/leeroy-wip-pass2/079.patch patchSha256=1038412aa6be537c540443b64b6437f4eb00d0315c79e4e819e2c68d0eb0a234 patchBytes=10666 patchLines=265 notesReviewed=no reviewedAt=- notesUpdated=no
+Pass2FullDiff: CAPTURED at 2026-01-24T17:39:06+0100 sha=456e2ad8e2b8 patchFile=/tmp/leeroy-wip-pass2/079.patch patchSha256=1038412aa6be537c540443b64b6437f4eb00d0315c79e4e819e2c68d0eb0a234 patchBytes=10666 patchLines=265 notesReviewed=yes reviewedAt=2026-01-24T20:30:50+0100 notesUpdated=yes
 
 - Subject: refactor(resume): don't require agentSessionId from UI
 - Reasons: full manual review coverage (was not in the manual-review queue)
@@ -2804,11 +2804,13 @@ Pass2FullDiff: CAPTURED at 2026-01-24T17:39:06+0100 sha=456e2ad8e2b8 patchFile=/
   - `sync/ops.ts` / `SessionView.tsx`:
     - Removes `agentSessionId` from the resume request shape; UI resumes by Happy session id and agent type only.
     - This pushes “derive vendor resume id” responsibility to the machine/daemon (more secure + less UI coupling to vendor internals).
+    - Likely leaves an unused `getAgentSessionId` import in `SessionView.tsx` (the callsite is removed but the patch does not remove the import); clean up during history rewrite.
   - `utils/agentCapabilities.ts`:
     - Broadens `canAgentResume(...)` to accept unknown strings safely.
     - Adds `SessionMetadata` shape + helpers:
       - `getAgentSessionIdField(...)`, `canResumeSession(...)`, and `getAgentSessionId(...)`.
   - Adds session-state translation keys (`session.resuming`, `session.resumeFailed`, etc.).
+    - Noted: Spanish `inactiveResumable` uses `reanundable` (likely a typo for `reanudable`).
 - Coherence:
   - Matches subject and is a solid architectural improvement for the resume feature.
 
@@ -2824,7 +2826,7 @@ Pass2FullDiff: CAPTURED at 2026-01-24T17:39:06+0100 sha=456e2ad8e2b8 patchFile=/
 ---
 
 ## 080 2026-01-11 9b0069e3d30c `fork`
-Pass2FullDiff: CAPTURED at 2026-01-24T17:39:06+0100 sha=9b0069e3d30c patchFile=/tmp/leeroy-wip-pass2/080.patch patchSha256=d0775d17447e5f09b0c0043294929d4b05c4f7a900063c95323c37df142a214b patchBytes=13508 patchLines=247 notesReviewed=no reviewedAt=- notesUpdated=no
+Pass2FullDiff: CAPTURED at 2026-01-24T17:39:06+0100 sha=9b0069e3d30c patchFile=/tmp/leeroy-wip-pass2/080.patch patchSha256=d0775d17447e5f09b0c0043294929d4b05c4f7a900063c95323c37df142a214b patchBytes=13508 patchLines=247 notesReviewed=yes reviewedAt=2026-01-24T20:32:11+0100 notesUpdated=yes
 
 - Subject: feat(fork): enable Codex resume support
 - Reasons: fork-only behavior (should not go upstream as-is)
@@ -2836,9 +2838,9 @@ Pass2FullDiff: CAPTURED at 2026-01-24T17:39:06+0100 sha=9b0069e3d30c patchFile=/
   - `utils/agentCapabilities.ts` adds `'codex'` to `RESUMABLE_AGENTS` with an explicit “Fork” note.
   - Session Info UX in `app/(app)/session/[id]/info.tsx`:
     - Adds a “Codex Session ID” item with copy-to-clipboard behavior (uses translation keys added earlier).
-    - Refactors copy-command handlers (`handleCopyCommand` helper) and unifies “copy resume command” logic (now `copyResumeCommand`).
+    - Refactors copy-command handlers (`handleCopyCommand` helper) and unifies “copy resume command” into `happy resume <happySessionId>` (shown only for inactive sessions that have an agent session ID in metadata).
   - `sync/storageTypes.ts` appears to add/validate `codexSessionId` metadata shape (schema alignment).
-  - Adds/updates i18n keys (`sessionInfo.copyResumeCommand`, `sessionInfo.codexSessionId`, etc.).
+  - Adds i18n key `sessionInfo.copyResumeCommand` across locales (Codex Session ID translation keys were added earlier in 076).
 - Coherence:
   - This is still within the “resume support” topic, but it’s fork-specific (Codex resume enablement) and should be isolated from any upstream PR stack.
 
@@ -2854,7 +2856,7 @@ Pass2FullDiff: CAPTURED at 2026-01-24T17:39:06+0100 sha=9b0069e3d30c patchFile=/
 ---
 
 ## 081 2026-01-22 4ddaef02210e `sync`
-Pass2FullDiff: CAPTURED at 2026-01-24T17:39:06+0100 sha=4ddaef02210e patchFile=/tmp/leeroy-wip-pass2/081.patch patchSha256=4898c477cf549a2bf357661b4ade980ba45511fb0c0b93ad24c80baa3e93323d patchBytes=1566 patchLines=43 notesReviewed=no reviewedAt=- notesUpdated=no
+Pass2FullDiff: CAPTURED at 2026-01-24T17:39:06+0100 sha=4ddaef02210e patchFile=/tmp/leeroy-wip-pass2/081.patch patchSha256=4898c477cf549a2bf357661b4ade980ba45511fb0c0b93ad24c80baa3e93323d patchBytes=1566 patchLines=43 notesReviewed=yes reviewedAt=2026-01-24T20:33:19+0100 notesUpdated=yes
 
 - Subject: fix(sync): omit model meta for queued messages
 - Reasons: full manual review coverage (was not in the manual-review queue)
@@ -2864,7 +2866,9 @@ Pass2FullDiff: CAPTURED at 2026-01-24T17:39:06+0100 sha=4ddaef02210e patchFile=/
 - Diffstat: `1 file changed, 7 insertions(+), 6 deletions(-)`
 - What changed:
   - `sync/sync.ts`:
-    - Computes `model` from `(session.metadata.flavor, session.modelMode)` and includes it in outgoing message meta only for Gemini when a non-default model is selected.
+    - Computes `model` from `(session.metadata.flavor, session.modelMode)`:
+      - For Gemini: defaults `modelMode` to `'gemini-2.5-pro'` when unset, and includes `model` in outgoing message meta when `modelMode !== 'default'`.
+      - For non-Gemini: omits `model` (`undefined`) from outgoing meta.
     - Switches meta construction to `buildOutgoingMessageMeta({ ... })` and drops `fallbackModel`.
 - Coherence:
   - Matches subject; this is a small correctness tweak to avoid sending irrelevant/incorrect model metadata in queued/outgoing messages.
@@ -2881,7 +2885,7 @@ Pass2FullDiff: CAPTURED at 2026-01-24T17:39:06+0100 sha=4ddaef02210e patchFile=/
 ---
 
 ## 082 2026-01-12 3fbdc10018fe `tmux`
-Pass2FullDiff: CAPTURED at 2026-01-24T17:39:06+0100 sha=3fbdc10018fe patchFile=/tmp/leeroy-wip-pass2/082.patch patchSha256=f2cc8568717d548920cea037c3b6e362272f870517c93ed7e2a86d9115651d03 patchBytes=30647 patchLines=723 notesReviewed=no reviewedAt=- notesUpdated=no
+Pass2FullDiff: CAPTURED at 2026-01-24T17:39:06+0100 sha=3fbdc10018fe patchFile=/tmp/leeroy-wip-pass2/082.patch patchSha256=f2cc8568717d548920cea037c3b6e362272f870517c93ed7e2a86d9115651d03 patchBytes=30647 patchLines=723 notesReviewed=yes reviewedAt=2026-01-24T20:34:54+0100 notesUpdated=yes
 
 - Subject: fix: tmux parsing/targeting + review feedback
 - Reasons: safety/reliability-sensitive area (process control) + non-Conventional-Commits subject
@@ -2899,8 +2903,10 @@ Pass2FullDiff: CAPTURED at 2026-01-24T17:39:06+0100 sha=3fbdc10018fe patchFile=/
     - Removes env-derived session/window fields from `TmuxSessionInfo` (only keeps `env_pane` + socket path, and relies on tmux query for session/window).
   - Additional review feedback items are included (mixed into the same commit):
     - Aligns `Machine.metadata` typing to runtime (nullable).
-    - Tightens some script/test behavior (`ripgrep_launcher`, env wrapper, etc.).
-    - Deletes a CLI doc file (`cli/docs/bug-fix-plan-2025-01-15-athundt.md`).
+    - Tightens script/test behavior (`ripgrep_launcher`, `env-wrapper`, continue-test script) and clarifies failure modes (missing args, signal termination).
+    - Adjusts CLI API sending behavior: introduces `ApiSessionClient.canSend(...)` and now returns early (drops the emit) when the socket is disconnected for multiple message/event types (may change socket.io buffering semantics vs prior behavior).
+    - Improves resume/continue ergonomics and cancellation: supports `-r` as `--resume` alias and wires SIGINT/SIGTERM to abort offline `claudeLocal` runs.
+    - Edits a CLI doc file (`cli/docs/bug-fix-plan-2025-01-15-athundt.md`) rather than deleting it.
 - Coherence:
   - The tmux portion is cohesive and important for correctness.
   - The extra “misc scripts/tests/typing” changes make the commit mixed; during rewrite we should consider splitting to keep tmux fixes isolated and bisectable.
@@ -2922,7 +2928,7 @@ Pass2FullDiff: CAPTURED at 2026-01-24T17:39:06+0100 sha=3fbdc10018fe patchFile=/
 ---
 
 ## 083 2026-01-13 7292a29822bb `daemon`
-Pass2FullDiff: CAPTURED at 2026-01-24T17:39:06+0100 sha=7292a29822bb patchFile=/tmp/leeroy-wip-pass2/083.patch patchSha256=40cb173cb1aa03a44ac0c18b4e07de6b8872336399fb768bf21f7a60daef97ab patchBytes=5066 patchLines=99 notesReviewed=no reviewedAt=- notesUpdated=no
+Pass2FullDiff: CAPTURED at 2026-01-24T17:39:06+0100 sha=7292a29822bb patchFile=/tmp/leeroy-wip-pass2/083.patch patchSha256=40cb173cb1aa03a44ac0c18b4e07de6b8872336399fb768bf21f7a60daef97ab patchBytes=5066 patchLines=99 notesReviewed=yes reviewedAt=2026-01-24T20:36:02+0100 notesUpdated=no
 
 - Subject: fix(daemon): do not apply CLI active profile to GUI-spawned sessions
 - Reasons: full manual review coverage (was not in the manual-review queue)

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "happy-stacks",
   "type": "module",
-  "version": "0.6.9",
+  "version": "0.6.10",
   "packageManager": "pnpm@10.18.3",
   "bin": {
     "happys": "./bin/happys.mjs",