Skip to content

Parse Pi transcripts during attach and explain - #1860

Merged
gtrrz-victor merged 1 commit into
mainfrom
fix-pi-attach-transcript-parsing
Jul 27, 2026
Merged

Parse Pi transcripts during attach and explain#1860
gtrrz-victor merged 1 commit into
mainfrom
fix-pi-attach-transcript-parsing

Conversation

@entire

@entire entire Bot commented Jul 26, 2026

Copy link
Copy Markdown

https://entire.io/gh/entireio/cli/trails/944

This draft pull request was opened by Entire after CI was requested for the linked trail. Feel free to edit the title or body — the link above is what keeps the trail and PR connected.

Entire-Checkpoint: 01KYG1T1J303QQV463F3W16ZW3
@dipree
dipree marked this pull request as ready for review July 27, 2026 06:49
@dipree
dipree requested a review from a team as a code owner July 27, 2026 06:49
Copilot AI review requested due to automatic review settings July 27, 2026 06:49

Copilot AI 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.

Pull request overview

This PR adds first-class handling for Pi’s native v3 JSONL transcripts so attach and explain can extract/format prompts and models correctly, and summarization can condense Pi sessions the same way other agents are handled.

Changes:

  • Extend transcript condensation to support agent.AgentTypePi, including native Pi JSONL → compact → condensed entries.
  • Improve attach metadata extraction by deferring to agent-native prompt/model extractors when available.
  • Add unit tests covering Pi native transcript parsing for summarization, explain formatting, and attach metadata extraction.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
cmd/entire/cli/summarize/summarize.go Adds Pi-specific condensed-transcript builder and routes Pi agent type to it.
cmd/entire/cli/summarize/summarize_test.go Adds summarization coverage for native Pi JSONL and parallelizes an existing compact fallback test.
cmd/entire/cli/explain_test.go Adds coverage that explain formatting no longer fails on native Pi JSONL.
cmd/entire/cli/attach.go Switches attach transcript metadata extraction to an agent-aware extraction path.
cmd/entire/cli/attach_transcript.go Introduces agent-aware metadata extraction using prompt/model extractor capabilities.
cmd/entire/cli/attach_test.go Adds coverage ensuring Pi-native prompt/model metadata is extracted during attach.

Comment thread cmd/entire/cli/summarize/summarize.go
Comment thread cmd/entire/cli/attach_test.go
@gtrrz-victor
gtrrz-victor merged commit 0a18d91 into main Jul 27, 2026
12 checks passed
@gtrrz-victor
gtrrz-victor deleted the fix-pi-attach-transcript-parsing branch July 27, 2026 08:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants