Skip to content

Fix checkpoint execution and preserve history selection - #1132

Merged
blove merged 14 commits into
mainfrom
codex/langgraph-checkpoint-routing
Sep 23, 2026
Merged

blove merged 14 commits into
mainfrom
codex/langgraph-checkpoint-routing

Conversation

@blove

@blove blove commented Sep 23, 2026 •

Copy link
Copy Markdown
Contributor

Streaming with the documented checkpointId option silently ran at the thread tip because the locked LangGraph SDK serializes only checkpoint on that endpoint. Normalize the alias once in the existing pure payload builder for streamed and queued runs. An explicit checkpoint object or null wins, caller options remain unchanged, and the synthesized root checkpoint uses an empty map accepted by the example's local API.

The time-travel example now keeps its selected checkpoint ID when new history entries prepend. Rows are keyed by checkpoint identity, preserving the selected row and keyboard focus. Select remains a branch pointer; Fork explicitly starts a run. This includes the selection correction reviewed and merged green in #1134.

Validation:

  • Ten real-SDK HTTP regressions cover streaming/queue aliases, explicit objects/null, undefined fallback, absence, exact payloads and caller immutability. Six cases fail before the transport fix.
  • Two real-server time-travel e2e tests pass. The new fork scenario verifies the serialized checkpoint and saved server messages, excluding the later original turn. Its selection assertion fails with index-based selection and passes with checkpoint identity.
  • Chrome MCP and the Codex in-app browser independently repeated the fork and selection flow against local LangGraph API 0.7.96 with deterministic model responses. Both show the correct fork and retained selection/focus after history insertion; final selection-run consoles are clean.
  • Combined local integration passes 588 runtime tests, strict runtime/public types, lint, 442 infrastructure tests, fresh foundation builds, inventory and source/emitted-boundary checks. Packed React and Angular consumers each pass all 21 browser scenarios, with exact request/abort checks and no page errors or unexpected requests.
  • Thirty documentation/content/snippet tests and the example production build pass. The inventory's transport and guide fingerprints were explicitly refreshed; capability assignments remain unchanged.

This fixes the existing public Angular transport/example. It does not expose checkpoint execution through the private neutral session; replay, tool identity and continuation ownership still need a separate design. No argument validation or new state store is introduced.

Review provenance: parent source/type/SDK/lifecycle audits completed. Independent local review was unavailable after thread capacity exhaustion. Recent hosted Claude jobs returned internal errors despite green job badges and supplied no substantive code review.

Integration: #1131 is merged green on main. Merging that commit changed no files: the resulting tree was identical to the prepared combined tree. A later main update changed only two website instrumentation files; its seven focused tests pass, and checkpoint/runtime files are unchanged. Current head is 6fa81f0cbd331a2aedabfcd0c2ae932f881e71a2; inventory remains valid. Required CI is evaluated on this exact head before main delivery.

@vercel

vercel Bot commented Sep 23, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
threadplane Ready Ready Preview Sep 23, 2026 4:25am UTC

Request Review

@github-actions

github-actions Bot commented Sep 23, 2026 •

Copy link
Copy Markdown
Contributor

Claude finished @blove's task in 0s —— View job


I'll analyze this and get back to you.

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated approval: this PR received an intelligent (AI) code review. See the review comments on this PR.

@github-actions

github-actions Bot commented Sep 23, 2026 •

Copy link
Copy Markdown
Contributor

Claude finished @blove's task in 0s —— View job


I'll analyze this and get back to you.

@blove
blove marked this pull request as ready for review September 23, 2026 00:21

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated approval: this PR received an intelligent (AI) code review. See the review comments on this PR.

@github-actions

github-actions Bot commented Sep 23, 2026 •

Copy link
Copy Markdown
Contributor

Claude finished @blove's task in 0s —— View job


I'll analyze this and get back to you.

@github-actions

github-actions Bot commented Sep 23, 2026 •

Copy link
Copy Markdown
Contributor

Claude finished @blove's task in 0s —— View job


I'll analyze this and get back to you.

blove and others added 3 commits September 22, 2026 17:44
* fix(cockpit): retain selected checkpoint across history refreshes

* chore: refresh checkpoint guide inventory facts
@github-actions

github-actions Bot commented Sep 23, 2026 •

Copy link
Copy Markdown
Contributor

Claude finished @blove's task in 0s —— View job


I'll analyze this and get back to you.

@blove blove changed the title Fix checkpoint ID routing for LangGraph runs Fix checkpoint execution and preserve history selection Sep 23, 2026
@github-actions

github-actions Bot commented Sep 23, 2026 •

Copy link
Copy Markdown
Contributor

Claude finished @blove's task in 0s —— View job


I'll analyze this and get back to you.

@github-actions

github-actions Bot commented Sep 23, 2026 •

Copy link
Copy Markdown
Contributor

Claude finished @blove's task in 0s —— View job


I'll analyze this and get back to you.

@blove
blove enabled auto-merge (squash) September 23, 2026 05:09
@blove
blove merged commit ea15ba3 into main Sep 23, 2026
77 checks passed

This branch was successfully deployed

1 active deployment
Preview – threadplane — 6fa81f0c Deployed Sep 23, 2026 by vercel[bot]
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