Repository navigation
fix(telemetry): stop reporting requested tool aborts as tool errors - #1627
altimate-software-factory[bot] wants to merge 1 commit into
Conversation
Factory lane: altimate_code. Agent-Authored-By: altimate-factory
|
Important Review skippedBot user detected. To trigger a single review, invoke the
|
|
This PR doesn't fully meet our contributing guidelines and PR template. What needs to be fixed:
Please edit this PR description to address the above within 2 hours, or it will be automatically closed. If you believe this was flagged incorrectly, please let a maintainer know. |
|
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. |
|
👋 This PR was automatically closed by our quality checks. Common reasons:
If you believe this was a mistake, please open an issue explaining your intended contribution and a maintainer will help you. |
Summary
Automated fix proposed by the Altimate software factory; a maintainer reviews every change.
The problem, fix and criteria below are the agent's own account; check them against the diff.
Problem
When a session is aborted, the processor closes running tools as errors with 'Tool execution aborted', and the prompt loop counted those requested cancellations as tool failures in error_fingerprint telemetry.
Fix
Error fingerprint telemetry now skips tools that were closed because the session itself was aborted, and still reports unrequested interruptions and real tool errors.
Acceptance criteria
Changes
3 files changed, 60 lines added, 0 lines removed.
packages/opencode/src/altimate/telemetry/error-detail.tspackages/opencode/src/session/prompt.tspackages/opencode/test/telemetry/error-fingerprint-requested-abort.test.tsNew tests
packages/opencode/test/telemetry/error-fingerprint-requested-abort.test.tsIndependent review: accepted, confidence 0.86
Verification
Summary by cubic
Stops reporting requested tool aborts as tool errors in
error_fingerprinttelemetry.isRequestedToolInterruptioninpackages/opencode/src/altimate/telemetry/error-detail.tsand a check inpackages/opencode/src/session/prompt.ts.Written for commit 4222260. Summary will update on new commits.