Skip to content

fix: improve file read state error messages - #1881

Open
zhanba wants to merge 2 commits into
mainfrom
fix/hydrate-file-read-history
Open

fix: improve file read state error messages#1881
zhanba wants to merge 2 commits into
mainfrom
fix/hydrate-file-read-history

Conversation

@zhanba

@zhanba zhanba commented Aug 7, 2026

Copy link
Copy Markdown
Member

Summary

  • improve file-edit error messages to distinguish files never read in available task history from files whose current read snapshot is no longer available
  • retain lightweight read history across task resumes and cache eviction so the error message has enough context
  • hydrate historical file operations consistently in CLI, background tasks, and VS Code

Test plan

  • bun check
  • bun tsc
  • common FileStateCache tests
  • livekit task utility and TaskExecutor tests
  • VS Code FileStateCacheRegistry tests
  • full pre-push test suite

🤖 Generated with Pochi | Task

zhanba and others added 2 commits August 7, 2026 16:57
Hydrate successful historical file operations so stale cache errors distinguish evicted snapshots from files that were never read.

🤖 Generated with [Pochi](https://getpochi.com) | [Task](https://app.getpochi.com/share/p-f14c30a4308d4d70a9c17923d2ede544)

Co-Authored-By: Pochi <noreply@getpochi.com>
Keep the subtask hook tests aligned with the message query and host hydration dependencies added for resumed tasks.

🤖 Generated with [Pochi](https://getpochi.com) | [Task](https://app.getpochi.com/share/p-f14c30a4308d4d70a9c17923d2ede544)

Co-Authored-By: Pochi <noreply@getpochi.com>
@zhanba zhanba changed the title fix: preserve file read history across task resumes fix: improve file read state error messages Aug 7, 2026
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 67.29560% with 52 lines in your changes missing coverage. Please review.
✅ Project coverage is 34.71%. Comparing base (156bb05) to head (04273a4).
⚠️ Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
packages/cli/src/running-task-adaptor.ts 0.00% 19 Missing ⚠️
packages/cli/src/task-runner.ts 0.00% 11 Missing ⚠️
packages/vscode-webui/src/features/chat/page.tsx 9.09% 10 Missing ⚠️
...scode/src/integrations/webview/vscode-host-impl.ts 0.00% 8 Missing ⚠️
...scode-webui/src/lib/vscode-running-task-adaptor.ts 0.00% 3 Missing ⚠️
packages/vscode-webui/src/lib/vscode.ts 0.00% 1 Missing ⚠️

❌ Your patch status has failed because the patch coverage (67.29%) is below the target coverage (75.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1881      +/-   ##
==========================================
+ Coverage   29.30%   34.71%   +5.40%     
==========================================
  Files         768      767       -1     
  Lines       80030    80167     +137     
  Branches     3846     3366     -480     
==========================================
+ Hits        23456    27831    +4375     
+ Misses      56455    52222    -4233     
+ Partials      119      114       -5     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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.

2 participants