Skip to content

Keep checkpoint selection stable across history refreshes - #1134

Merged
blove merged 3 commits into
codex/langgraph-checkpoint-routingfrom
codex/checkpoint-selection
Sep 23, 2026
Merged

blove merged 3 commits into
codex/langgraph-checkpoint-routingfrom
codex/checkpoint-selection

Conversation

@blove

@blove blove commented Sep 23, 2026 •

Copy link
Copy Markdown
Contributor

Forking from an older checkpoint prepends new history entries. The time-travel example tracked its selection by row index, so the highlight moved to a different checkpoint even though the fork executed correctly.

Store the selected checkpoint ID in local component state, derive row and badge highlighting from that ID, and key rows by checkpoint identity. Select remains a branch pointer; Fork remains the explicit run command. Update the guide to explain selection across history refreshes.

Stacked on #1132, which fixes checkpoint wire routing. This diff only changes the example, its guide and the selection assertion.

Validation:

  • The real-server fork e2e fails on the previous index-based selection and passes with this change; both time-travel e2e tests pass.
  • Production example build and 30 documentation/content/snippet tests pass.
  • Chrome DevTools MCP and Codex in-app browser both repeated the fork flow. After history grows from six to nine entries, exactly one row remains selected by the original ID at position seven; keyboard focus stays on that checkpoint's Fork button. Both browser consoles have no warning/error messages in this run. Screenshots inspected.
  • Checked generator inputs: agent context reads version/templates, narrative generation consumes API docs, and neither is affected. ExampleCode regions load directly from the updated source.
  • Inventory and source boundaries pass, along with 288 migration infrastructure tests. The guide fingerprint was refreshed without changing capability assignments.
  • Parent audited state ownership and exact diff. Independent local agent review is unavailable after thread capacity exhaustion. Hosted workflow 35802695852 on bae3dc17e returned is_error: true despite a green job badge and produced no substantive review.

@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 12:44am UTC

Request Review

@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
blove merged commit fc8e1a7 into codex/langgraph-checkpoint-routing Sep 23, 2026
29 checks passed
blove added a commit that referenced this pull request Sep 23, 2026
* test(runtime): prove application-owned thread session lifetime

* fix(langgraph): route checkpoint ID through SDK run payloads

* test(langgraph): prove checkpoint forks against the API server

* chore: refresh checkpoint transport inventory facts

* Keep checkpoint selection stable across history refreshes (#1134)

* fix(cockpit): retain selected checkpoint across history refreshes

* chore: refresh checkpoint guide inventory facts

This branch was successfully deployed

1 active deployment
Preview – threadplane — bae3dc17 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