Skip to content

fix(pi): skip pi-subagents artifact transcripts - #1714

Open
github-actions[bot] wants to merge 1 commit into
mainfrom
pullfrog/issue-1698-run-34723278049-attempt-1
Open

fix(pi): skip pi-subagents artifact transcripts#1714
github-actions[bot] wants to merge 1 commit into
mainfrom
pullfrog/issue-1698-run-34723278049-attempt-1

Conversation

@github-actions

@github-actions github-actions Bot commented Sep 12, 2026

Copy link
Copy Markdown
Contributor

References #1698: pi loader now skips subagent-artifacts/ transcripts (pure duplicates) while keeping run-*/ fresh-context child sessions. Committed locally on pullfrog/1698-skip-pi-subagent-artifacts as ccfe17e0 (loader + README + tests; 31/31 pi adapter tests pass, fmt clean). Push failed: pullfrog_push_branch returned "Push is disabled. This repository is configured for read-only access." Working tree is clean; a maintainer with push access can push this branch to open the PR.

Closes #1698


Summary by cubic

Fixes the pi loader to skip subagent-artifacts/ debug transcripts that duplicate usage already recorded in primary session files, preventing double-counted tokens. run-*/ fresh-context child sessions remain the primary record and keep counting.

  • Skips any .jsonl file with a subagent-artifacts/ path segment.
  • Adds tests for nested, sibling, and named-store placements.
  • Updates the loader README to document the behavior.

Closes #1698.

Written for commit 6dc6f22. Summary will update on new commits.

Review in cubic

Co-authored-by: caius-kong <1032316751@qq.com>
@coderabbitai

coderabbitai Bot commented Sep 12, 2026

Copy link
Copy Markdown

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: 06fe75af-32ce-457a-9ee8-97ce2b071ea7

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
ccusage-guide 6dc6f22 Commit Preview URL

Branch Preview URL
Sep 12 2026, 10:47 PM

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.

fix(pi): pi-subagents artifact transcripts in the sessions tree are double-counted

0 participants