chore: sync with upstream pingdotgg/t3code main - #52
Conversation
Co-authored-by: maria-rcks <254055478+maria-rcks@users.noreply.github.com> Co-authored-by: Julius Marminge <julius0216@outlook.com>
…#12304) Co-authored-by: maria-rcks <254055478+maria-rcks@users.noreply.github.com>
…pstream-20260917-1 Signed-off-by: Yordis Prieto <yordis.prieto@gmail.com>
PR SummaryHigh Risk Overview Mobile icons centralize Web chat timeline makes nested “Thought” traces inside tool groups collapsible (preview when collapsed, state keyed by message id + row anchor for virtualizer recycling). Reviewed by Cursor Bugbot for commit 3c84c91. Bugbot is set up for automated code reviews on this repo. Configure here. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (9)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughThe pull request updates mobile symbol mappings and shared types, preserves usage data across transcript cleanup and restart, and adds collapsible reasoning traces with disclosure state and scroll anchors. ChangesMobile symbol mappings and typing
Usage retention and scan caching
Reasoning trace disclosure
Priority: ➖ Normal Estimated code review effort: 4 (Complex) | ~45 minutes Change: Other Sequence Diagram(s)sequenceDiagram
participant UsageService
participant FileSystem
participant ScanCache
participant UsageMerge
UsageService->>FileSystem: resolve directories and scan transcripts
UsageService->>ScanCache: load retained source records
UsageService->>ScanCache: persist updated source metadata
UsageService->>UsageService: aggregate and deduplicate usage
UsageService->>UsageMerge: merge environment usage
UsageMerge-->>UsageService: return merged usage
Suggested reviewers: Merge Risk: ⚪ Minimal · up to The symbol, usage-retention, and reasoning-disclosure changes appear mergeable after normal checks. 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches 💡 1📝 Generate docstrings 💡
🧪 Generate unit tests (beta)
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. Comment |
Thread transfer impact✅ Thread transfer remains within every enforced ceiling.
Baseline: Scenario and decoded snapshot size10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.
Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed. |
No divergence collided with this batch, so nothing was re-applied by hand and every file the merge touched is byte-identical to upstream except one.
Upstream reworked the usage-scanning region that 0016 adapts. The fork keeps its wrapper there: usage reads only locate provider home directories, so an instance value is used as written and a secret reference can never be reported as unresolved from a usage scan.
Upstream's collapsed thoughts land in the component 0003 extends. Upstream's shape is taken whole, because the subagent rows the fork adds sit elsewhere in the file and share none of the state upstream moved into the new disclosure.
Summary by CodeRabbit