feat(cli): Collapse help_* into cli_help { topic }; drop cli_info

[thecodedrift]

Phase 4 of the telemetry rework (stacked on the auth/lifecycle phase).

• help.ts: replace help_index / help_<topic> / help_unknown with a single cli_help { topic } — the served topic, "(index)" for a no-arg invocation, or the attempted topic when unknown. Help intent is now one event you filter by topic (the wrong-topic re-routing funnel still derives from the cli_help / cli_run sequence).
• info.ts: drop the bespoke cli_info / cli_info_completed events (covered by cli_run) and the now-unused getTelemetry import.

detect.ts (cli_detect) is not on this branch's lineage — it lives in the unmerged local-rule-routing stack, so there's nothing to change here; it'll be reconciled when both stacks land.

Adds test/help-telemetry.test.ts asserting cli_help { topic } across served / index / unknown cases and that no legacy help_* event fires. Full suite green (259).

Part of the restructure-cli-telemetry OpenSpec change. Phase 5 (finalize + archive) is next.

---

• main
  • docs(openspec): Propose restructure-cli-telemetry change #33
    • feat(cli): Emit cli_run once per invocation from the runner #32
      • 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* 👈
            • chore(openspec): Finalize + archive restructure-cli-telemetry #37

^{Stack generated by Git Town}

[Copilot]

Pull request overview

Phase 4 of the CLI telemetry rework: consolidates the help_* event family into a single cli_help { topic } signal and removes the bespoke cli_info(_completed) events now that cli_run is the invocation denominator.

Changes:

• Update help command telemetry to emit only cli_help with a topic value for index/served/unknown requests.
• Remove cli_info / cli_info_completed emissions (and the unused getTelemetry import) from info.
• Add a new unit test validating cli_help { topic } behavior and preventing legacy help_* events.
• Mark Phase 4 tasks as completed in the OpenSpec change task list.

Reviewed changes

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

File	Description
packages/cli/src/commands/help.ts	Replaces help_index / help_<topic> / help_unknown with cli_help { topic }.
packages/cli/src/commands/info.ts	Drops cli_info(_completed) telemetry and removes unused telemetry import.
packages/cli/test/help-telemetry.test.ts	Adds tests for cli_help across served/index/unknown flows.
openspec/changes/restructure-cli-telemetry/tasks.md	Updates Phase 4 checklist items to completed and clarifies detect.ts lineage note.

---

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

[thecodedrift]

just weighing in on the same feedback as copilot. But I specifically think over-testing would be a bad design here. I'd much rather see us just confidently say after this lands there's no help_ events being emitted