Skip to content

feat(resume): compose checkpoints and journal into session resume guide#1095

Open
shaun0927 wants to merge 1 commit into
developfrom
feat/1004-resume-guide
Open

feat(resume): compose checkpoints and journal into session resume guide#1095
shaun0927 wants to merge 1 commit into
developfrom
feat/1004-resume-guide

Conversation

@shaun0927
Copy link
Copy Markdown
Owner

Summary

  • Extends session_resume to compose the existing session snapshot with checkpoint and journal artifacts.
  • Adds checkpoint task/completed/pending/current URL details, recent success/failure summaries, avoid guidance after failure, and optional evidence-bundle links.
  • Keeps the final recommendation conservative: verify live tab state before mutating the page.

Direction / necessity check

Validation

  • npm test -- --runTestsByPath tests/tools/session-resume.test.ts
  • npm run build:src
  • npm run lint:tier

Post-merge live verification with OpenChrome

  1. Run a browser task that writes ~/.openchrome/checkpoints/current-checkpoint.json with at least one completed and pending step.
  2. Call session_resume and verify the guide includes Checkpoint:, task description, completed steps, pending steps, and current URL.
  3. Produce one successful and one failed tool call so the journal has recent entries; call session_resume again and verify Recent tool activity, Last success, Last failure, and Avoid: repeating the last failed call... appear.
  4. If evidence bundle paths are passed by future callers, verify links are listed without embedding binary/blob payloads.
  5. Verify the recommended next action remains a non-mutating live-state check (read_page or tabs_context).

Closes #1004

Resume currently explains the saved snapshot but loses useful recovery context from checkpoint and journal artifacts. This composes those existing artifacts into a single guide that tells a host what was done, what failed, and what safe verification should come next.

Constraint: Resume output must summarize existing local artifacts without embedding large evidence blobs or requiring new persistence formats.

Rejected: Build a new resume state store | checkpoints, journal, and snapshot artifacts already exist and are safer to compose.

Confidence: high

Scope-risk: narrow

Directive: Keep resume guidance advisory and require live tab verification before mutation.

Tested: npm test -- --runTestsByPath tests/tools/session-resume.test.ts; npm run build:src; npm run lint:tier

Not-tested: Live OpenChrome resume after an actual long-running browser session.

Co-authored-by: OmX <omx@oh-my-codex.dev>
@qodo-code-review
Copy link
Copy Markdown

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 →

@gemini-code-assist
Copy link
Copy Markdown

Warning

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant