Open
Conversation
Entire-Checkpoint: 9b668a38f4bf
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Comment @cursor review or bugbot run to trigger another review on this PR
Reviewed by Cursor Bugbot for commit 7189770. Configure here.
Contributor
There was a problem hiding this comment.
Pull request overview
Adds Codex rollout transcript support to the explain --generate summarization flow by enabling checkpoint-scoped parsing/compaction and condensed transcript extraction.
Changes:
- Expanded Codex format detection to recognize checkpoint-scoped Codex JSONL that may not include
session_meta. - Added Codex handling in summarization condensation by compacting Codex rollouts into the shared compact transcript schema and extracting user/assistant/tool entries.
- Updated transcript scoping for Codex in
explainand added unit tests for Codex summarization/scoping.
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| cmd/entire/cli/transcript/compact/codex.go | Broadens Codex JSONL detection to handle mid-file (scoped) transcripts. |
| cmd/entire/cli/summarize/summarize.go | Adds Codex-specific condensed transcript building via compact.Compact and parses compact JSONL into summary entries. |
| cmd/entire/cli/summarize/summarize_test.go | Adds a unit test validating Codex condensed transcript extraction. |
| cmd/entire/cli/explain.go | Scopes Codex transcripts using response_item-based slicing before generating summaries/intents. |
| cmd/entire/cli/explain_test.go | Adds a unit test for Codex checkpoint transcript scoping behavior. |
| cmd/entire/cli/agent/codex/transcript.go | Introduces SliceFromResponseItem helper used for Codex transcript scoping. |
Entire-Checkpoint: a1c5ef641832
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

Adds support to generate summary from codex session with
entire explain -c <checkpointID> --generateAvoids error
failed to generate summary: transcript has no content to summarizeNote
Medium Risk
Adds a new parsing/scoping path for Codex transcripts used by
explain --generate, which could affect checkpoint transcript slicing and summary accuracy for Codex sessions but is localized to transcript processing.Overview
explain --generateand transcript rendering now support Codex sessions by scoping checkpoint transcripts using Codexresponse_itemoffsets (via newcodex.SliceFromResponseItem) instead of generic line slicing.summarize.BuildCondensedTranscriptFromBytesgains a Codex path that compacts Codex rollout JSONL into the shared condensed transcript format (including tool calls), andcompactCodex format detection is broadened to recognize additional Codex line types. New tests cover Codex scoping and condensed-transcript extraction, and tool-detail extraction is hardened against nil inputs.Reviewed by Cursor Bugbot for commit 7189770. Configure here.