Skip to content

feat(telemetry): record abandoned permission prompts and modes - #1202

Closed
TheGreatAxios wants to merge 2 commits into
mainfrom
cl-9013-permission-telemetry
Closed

TheGreatAxios wants to merge 2 commits into
mainfrom
cl-9013-permission-telemetry

Conversation

@TheGreatAxios

Copy link
Copy Markdown
Collaborator

Summary

  • Permission prompts abandoned with Esc record an abandoned decision instead of a synthetic deny; the action is still denied
  • permission_prompt events carry the approval mode (interactive) from the approval log
  • Unknown slash commands emit one bucketed slash_command event instead of none

Verification

  • bun run lint passes; targeted suites pass: telemetry-product-events (24), permission gate/queue plus approval-resume/check-gate plus command registry (139)
  • bun run typecheck reports one pre-existing vendor error (semver types missing from the install); check:dead-exports cannot run (ts-prune binary absent)
  • Seven existing overlay-level assertions still expect Esc to resolve a synthetic deny and need updating to the abandoned contract in a follow-up

Fixes CL-9013

An abandoned prompt is not a denial: Esc previously manufactured a deny outcome, hiding abandonment from telemetry. Settling with no outcome lets the gate deny the action while reporting what actually happened.
@linear-code

linear-code Bot commented Sep 28, 2026

Copy link
Copy Markdown

CL-9013

@TheGreatAxios

Copy link
Copy Markdown
Collaborator Author

Closing as telemetry-only: does not improve operator-facing output, and the PR is not CI-green + trivial after the 0333 merge wave. Reopen if product telemetry is prioritized separately.

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.

1 participant