Repository navigation
fix: automated fix in packages/opencode/src/altimate/telemetry - #1557
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 ⚙️ Run configuration
You can disable this status message by setting the Use the checkbox below for a quick retry:
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 |
|
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.
Changes
4 files changed, 45 lines added, 0 lines removed.
packages/opencode/src/altimate/telemetry/error-detail.tspackages/opencode/src/session/prompt.tspackages/opencode/src/tool/tool.tspackages/opencode/test/telemetry/user-rejection-not-tool-error.test.tsNew tests
packages/opencode/test/telemetry/user-rejection-not-tool-error.test.tsIndependent review: accepted, confidence 0.86
Verification
Summary by cubic
Stops telemetry from reporting declined permission prompts and dismissed questions as tool errors, since those are user decisions, not tool failures.
isUserRejection()to match the exact error messages forPermissionNextandQuestionrejections.Written for commit 23b450b. Summary will update on new commits.