Skip to content

report anonymously whether the review comment could be posted - #195

Merged
reuvenharrison merged 1 commit into
mainfrom
telemetry/review-delivery
Aug 8, 2026
Merged

report anonymously whether the review comment could be posted#195
reuvenharrison merged 1 commit into
mainfrom
telemetry/review-delivery

Conversation

@reuvenharrison

@reuvenharrison reuvenharrison commented Aug 8, 2026

Copy link
Copy Markdown
Contributor

When the action produces a review link it either posts it as a PR comment or falls back to the job summary (no token, read-only token, or an API error). The action now sends oasdiff.com one anonymous event per run saying which of the three happened, plus its own version ref, so we can see how often users land on the fallback and improve the defaults.

No repo, organization, or user identity is sent. Best-effort with a 5 second cap; a failure never affects the run. Skipped when OASDIFF_INTERNAL=1, the same marker the CLI uses for internal test runs. Same change in breaking and changelog.

🤖 Generated with Claude Code

When the action produces a review link it either posts it as a PR
comment or falls back to the job summary (no token, read-only token, or
an API error). The action now sends oasdiff.com one anonymous event per
run saying which of the three happened, plus its own version ref, so
the maintainers can see how often users land on the fallback and
improve the defaults accordingly.

It carries no repo, organization, or user identity, is best-effort
(5 second cap, a failure never affects the run), and is skipped when
OASDIFF_INTERNAL=1, the same marker the CLI uses for internal test
runs. Same change in breaking and changelog.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@reuvenharrison reuvenharrison changed the title telemetry: report how the review link was delivered report anonymously whether the review comment could be posted Aug 8, 2026
@reuvenharrison
reuvenharrison force-pushed the telemetry/review-delivery branch from 6badbde to 10d2664 Compare August 8, 2026 06:41
@reuvenharrison
reuvenharrison merged commit b803cbd into main Aug 8, 2026
31 checks passed
@reuvenharrison
reuvenharrison deleted the telemetry/review-delivery branch August 8, 2026 06:45
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