feat(tui): display TPS metric + minimal ripgrep path fix#13892
feat(tui): display TPS metric + minimal ripgrep path fix#13892Daltonganger wants to merge 4 commits intoanomalyco:devfrom
Conversation
|
Thanks for your contribution! This PR doesn't have a linked issue. All PRs must reference an existing issue. Please:
See CONTRIBUTING.md for details. |
|
@rekram1-node This PR is a direct follow-up to #12721. It carries over all feature changes from #12721 and adds one minimal ripgrep resolver fix aimed at the e2e failures. Cross-reference: |
|
The following comment was made by an LLM, it may be inaccurate: Based on my search results, the PR described appears to be a follow-up to #12721, which is the original implementation. Here's what I found: Related PR:
Why it's related: PR #13892 (the current PR) explicitly states it includes all changes from #12721 and adds a minimal follow-up fix for ripgrep path resolution. This is a direct continuation, not a duplicate. Additional context:
Conclusion: PR #13892 is not a duplicate—it's a documented follow-up to #12721 that builds upon the TPS display feature. |
Adds TPS calculation and display to message footers. Tracks firstToken timestamp during streaming and calculates throughput for completed text responses. Filters out tool calls and fast responses to avoid noise. Key features: - Shows TPS next to duration: "3.4s · 45 tok/s" - Includes both output and reasoning tokens - 250ms minimum threshold to filter noise - Comprehensive test coverage (34 tests) Tested with Kimi K2.5 showing ~131 tok/s. Fixes anomalyco#5374, Closes anomalyco#6096
Daltonganger
force-pushed
the
daltonganger/pr-12721-plus-rgfix-v2
branch
from
fbadbb3 to
6718543
Compare
|
@rekram1-node Quick update: I refreshed this branch on top of the latest Current status:
|
2 participants
Summary
packages/opencode/src/file/ripgrep.tsrgresolution safer by rejecting invalid Windows POSIX-style paths and probing resolved binaries before spawnWhy
PR #12721 shows three failing checks where two are real e2e failures caused by invalid/unusable
rgpaths and the third is a gate job failure. This keeps scope minimal and targets only ripgrep resolution robustness.Linked PRs
Closes #6096
Fixes #5374