feat(cli): Emit cli_run once per invocation from the runner

[thecodedrift]

Phase 1 of the telemetry rework (stacked on the proposal docs).

Adds the cli_run denominator: the index.ts runner now emits exactly one cli_run per invocation — on both success and failure — with { command, cli_version, success, durationMs, anonymous, loggedIn }. On failure it first emits cli_error { command, code } (CliError gains an optional CliErrorCode, falling back to INTERNAL_ERROR). The telemetry client exposes identity so the runner can stamp anonymous/loggedIn.

The runner logic is extracted to telemetry-run.ts so it's unit-testable without executing the CLI entry point — test/cli-run.test.ts covers it.

Purely additive: legacy per-command events still emit (removed in phases 2–4), so the suite stays green.

Part of the restructure-cli-telemetry OpenSpec change.

---

• main
  • docs(openspec): Propose restructure-cli-telemetry change #33
    • feat(cli): Emit cli_run once per invocation from the runner 👈
      • feat(cli): Emit concrete rule events; drop rule start/_completed pairs #34
        • feat(cli): Concrete auth + lifecycle events; drop start/_completed pairs #35
          • feat(cli): Collapse help_* into cli_help { topic }; drop cli_info #36
            • chore(openspec): Finalize + archive restructure-cli-telemetry #37

^{Stack generated by Git Town}

[Copilot]

Pull request overview

Adds a per-invocation telemetry “denominator” (cli_run) emitted exactly once from the CLI runner, plus a failure-side cli_error event, as Phase 1 of the CLI telemetry taxonomy restructure (stacked PRs).

Changes:

• Extract runner-side telemetry helpers (resolveCommandName, resolveCwd, emitRunEvents) into a new testable module and call them from src/index.ts in a finally block.
• Extend telemetry + error plumbing to support cli_error { command, code } via an optional CliError.code and a TelemetryClient.identity surface for anonymous/logged-in stamping.
• Add unit tests for the new runner telemetry behavior and document the phased rollout plan in OpenSpec change tasks.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 4 comments.

Show a summary per file

File	Description
packages/cli/src/index.ts	Emits cli_run (and cli_error on failure) once per invocation from the top-level runner.
packages/cli/src/telemetry-run.ts	New extracted runner telemetry helpers (command/cwd resolution + event emission).
packages/cli/src/telemetry.ts	Exposes TelemetryClient.identity and wires it into the initialized/no-op clients.
packages/cli/src/util/cli-error.ts	Adds optional CliErrorCode to CliError for stable failure attribution.
packages/cli/test/cli-run.test.ts	Adds unit tests covering resolveCommandName and runner event emission.
openspec/changes/restructure-cli-telemetry/tasks.md	Adds the phased implementation task plan for the telemetry restructure change.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull request overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.

[thecodedrift]

4 comments, some more significant than others. I'm not sold on the CLI vs Cli but I believe we need to be consistent

The identity stuff IS a concern though, we want to ensure we actually have this captured if possible so we can trace errors back to a service user when required