[NOT MERGE] chore: remove reportCliUsage calls from create, docs, link

[jwfing]

Summary

• Remove reportCliUsage calls and imports from create, docs, and link commands
• These commands no longer report to the OSS /api/usage/mcp endpoint

Test plan

• Verify insforge create, insforge docs, insforge link still work normally
• TypeScript build passes

🤖 Generated with Claude Code

Note

Remove reportCliUsage calls from create, docs, and link commands

Removes all reportCliUsage telemetry calls from three CLI commands. The create command no longer emits cli.create, docs no longer emits cli.docs, and link no longer emits cli.link or cli.link_direct on success or failure paths.

^{Macroscope summarized 2b2dda2.}

Summary by CodeRabbit

• Chores
  • Removed internal telemetry calls from create, docs, and project link commands while maintaining full command functionality.

[coderabbitai]

Walkthrough

This PR removes reportCliUsage telemetry function calls from three command handlers (create.ts, docs.ts, link.ts) and their corresponding imports, while preserving all remaining command logic and control flow.

Changes

Cohort / File(s)	Summary
Telemetry Call Removal src/commands/create.ts, src/commands/docs.ts, src/commands/projects/link.ts	Removed reportCliUsage function calls and imports from command handlers; all other command flow (skill installation, auth checks, error handling) remains unchanged.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Possibly related PRs

• track connection #11: Directly inverse—this PR adds the reportCliUsage calls and exports that are now being removed from these exact command files.
• feat(cli): add alternative authentication for link command #24: Modifies the same CLI command handlers (create.ts, link.ts) with related telemetry and auth flow adjustments.
• feat(link): report agent-connected event after project linking #48: Updates src/commands/projects/link.ts with telemetry-related changes, replacing reportCliUsage patterns.

Suggested reviewers

• tonychang04

Poem

🐰 Hopping through the code with glee,
Telemetry tales set finally free,
No more reporting where the signals once flew,
Clean command paths—a rabbit's debut! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the main change: removing reportCliUsage calls from three commands (create, docs, link). It is concise, specific, and clearly summarizes the primary objective of the changeset.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch chore/remove-report-cli-usage

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}