docs(openspec): Propose restructure-cli-telemetry change - #33
Merged
Conversation
Establish the contract for reworking the CLI's PostHog taxonomy: a single
cli_run denominator emitted once per invocation, cli_* reserved for
concrete state transitions (cli_rule_created, cli_error, cli_check_completed,
...), and cli_help { topic } replacing the help_* trio. Modifies the
analytics capability; a hard cut with no dual-emit window.
Base of a stack; implementation lands in phased PRs on top.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Contributor
There was a problem hiding this comment.
Pull request overview
Defines an OpenSpec change contract for a redesigned CLI telemetry taxonomy (PostHog), centered around a single cli_run denominator event per invocation and a smaller set of concrete cli_* state-transition events.
Changes:
- Adds a new requirement for exactly one
cli_runevent per CLI invocation (emitted centrally). - Redefines the CLI analytics event taxonomy (hard cut) including
cli_help { topic }andcli_error. - Adds proposal/design docs and OpenSpec metadata for the change package.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 8 comments.
| File | Description |
|---|---|
| openspec/changes/restructure-cli-telemetry/specs/analytics/spec.md | Specifies the new telemetry event taxonomy and required properties/events. |
| openspec/changes/restructure-cli-telemetry/proposal.md | Summarizes motivation, breaking change scope, and impacted areas for implementation PRs. |
| openspec/changes/restructure-cli-telemetry/design.md | Records key design decisions (centralized cli_run, concrete transition events, hard cut). |
| openspec/changes/restructure-cli-telemetry/.openspec.yaml | Declares the change package schema and created date. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Bring the stack-aware OpenSpec archive check (originally #31, landed in the local-rule-routing stack) into the telemetry stack so PRs below the tip show the check as skipped and only the tip runs it. Identical content to the local-rule-routing copy, so the two stacks won't conflict when both reach main. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
ci(openspec): Skip the archive check on non-tip stacked PRs
This was referenced Jun 13, 2026
thecodedrift
marked this pull request as ready for review
June 13, 2026 17:25
thecodedrift
commented
Jun 13, 2026
thecodedrift
left a comment
thecodedrift
left a comment
Member
Author
There was a problem hiding this comment.
spec change already reviewed in depth, deferring to copilot review
PR #33 review: - Drop cli_version from cli_run's property list across proposal/design/spec; the CLI version rides on the standard cliVersion property (no second field). - Define cli_help's topic precisely: served topic, the literal "(index)" for no-arg, and the attempted topic for unknown; add an unknown-topic scenario. - cliVersion is embedded at build time (no runtime read), matching the CLI spec. - Clarify loggedIn vs anonymous as two distinct run-context dimensions, neither of which is the independent --anonymous flag. - Align cli_check_completed counts on findings (matches the implementation). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Keep the proposed analytics spec/design prose in sync with the source rename — the CLI acronym is uppercase in CLIError/CLIErrorCode. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Base of the telemetry-rework stack — the OpenSpec change contract.
Reworks the CLI's PostHog taxonomy: a single
cli_rundenominator emitted once per invocation,cli_*reserved for concrete state transitions (cli_rule_created,cli_error,cli_check_completed, …), andcli_help { topic }replacing thehelp_*trio. Modifies theanalyticscapability; a hard cut with no dual-emit window.Implementation lands in phased PRs stacked on top (cli_run → rule events → auth/lifecycle → cli_help → finalize+archive). See
openspec/changes/restructure-cli-telemetry/.Stack generated by Git Town