feat(journal): add compact resume handoff summary (#1027)

[shaun0927]

Progress / Review status

Auto-refreshed 2026-05-13 — owner comments cleaned up to reduce review noise.

Field	Value
Branch	feat/1027-journal-handoff → develop
Draft	no
CI	❌ 0/9 passing — 9 failing
Mergeable	✅ MERGEABLE
Review decision	—
Codex (latest)	—
Other reviewers (latest)	—
Head	ff42afd — Help agents resume long browser sessions safely
Commits	1

_{Owner comment cleanup: 0 issue + 0 inline review comments deleted. Outstanding feedback from automated/external reviewers above is unchanged.}

---

Summary

• Adds oc_journal action handoff_summary, a compact JSON read model over sanitized journal entries plus the existing current checkpoint file.
• Includes period/scope metadata, current checkpoint tab evidence, completed milestones, grouped recent failures, pending steps, bounded recovery recommendations, and explicit limits/unavailable fields.
• Treats tool results with isError: true as failed in the task journal, so non-throwing tool failures can appear in recovery summaries.
• Adds deterministic unit coverage and a real OpenChrome E2E restart scenario proving the handoff remains available from persisted artifacts.

Direction / overlap review

• Closes feat(journal): compact resume handoff summary for long-running sessions #1027.
• This stays read-only and evidence-first: it does not add a planner, autonomous retry loop, or duplicate feat(core): oc_task_ledger — persistent async task table with cancel & wait (mcp-browser-use adoption A) #855 task-ledger / feat(core): oc_context — portable context handle: export/import only (browserbase adoption C) #873 portable-context ownership.
• Open PR overlap was reviewed. This complements feat(harness): add opt-in run lifecycle ledger (#1021) #1086/feat(task-run): persist goal-level lifecycle state (#1039) #1083 lifecycle-ledger work by providing a caller-facing compact resume view over existing journal/checkpoint artifacts rather than a new lifecycle authority.
• checkpointId is intentionally limited: the current checkpoint store only has current, so non-current ids are reported in limits instead of inventing state.

Validation

• npm run build
• ./node_modules/.bin/eslint src/journal/handoff-summary.ts src/tools/journal.ts src/tools/checkpoint.ts src/mcp-server.ts --max-warnings=0
• ./node_modules/.bin/jest --runTestsByPath tests/journal/handoff-summary.test.ts tests/tools/journal.test.ts tests/journal/task-journal.test.ts --runInBand
• ./node_modules/.bin/jest --config tests/e2e/jest.e2e.config.js --runTestsByPath tests/e2e/scenarios/journal-handoff.e2e.ts --runInBand

Real OpenChrome verification covered

• Starts a real OpenChrome MCP server with isolated HOME.
• Navigates to a fixture, saves an oc_checkpoint, records a non-throwing failed tool result, calls oc_journal handoff_summary, verifies redaction and bounded recovery options, restarts the MCP process, and verifies the summary can still be produced from persisted artifacts.

[gemini-code-assist]

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

[qodo-code-review]

Qodo reviews are paused for this user.

Troubleshooting steps vary by plan Learn more →

On a Teams plan?
Reviews resume once this user has a paid seat and their Git account is linked in Qodo.
Link Git account →

Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center?
These require an Enterprise plan - Contact us
Contact us →

[chatgpt-codex-connector]

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Repo admins can enable using credits for code reviews in their settings.