fix(telemetry): fix commands importing buildCommand directly from @stricli/core

trace/list, trace/view, log/view, and api.ts were importing buildCommand
directly from @stricli/core, bypassing the telemetry wrapper in lib/command.ts
that captures flag and arg context for Sentry. Switch all four to the local
wrapper. Adds a prominent comment in command.ts documenting this requirement.

help.ts is intentionally left on @stricli/core since it also needs `run`
and has no meaningful flags to capture.

Author: BYK

src/commands/api.ts

@@ -5,9 +5,9 @@
  * Similar to 'gh api' for GitHub.
  */
 
-import { buildCommand } from "@stricli/core";
 import type { SentryContext } from "../context.js";
 import { rawApiRequest } from "../lib/api-client.js";
+import { buildCommand } from "../lib/command.js";
 import type { Writer } from "../types/index.js";
 
 type HttpMethod = "GET" | "POST" | "PUT" | "DELETE" | "PATCH";

src/commands/log/view.ts

@@ -4,14 +4,14 @@
  * View detailed information about a Sentry log entry.
  */
 
-import { buildCommand } from "@stricli/core";
 import type { SentryContext } from "../../context.js";
 import { getLog } from "../../lib/api-client.js";
 import {
   parseOrgProjectArg,
   parseSlashSeparatedArg,
 } from "../../lib/arg-parsing.js";
 import { openInBrowser } from "../../lib/browser.js";
+import { buildCommand } from "../../lib/command.js";
 import { ContextError, ValidationError } from "../../lib/errors.js";
 import { formatLogDetails, writeJson } from "../../lib/formatters/index.js";
 import {

src/commands/trace/list.ts

@@ -4,10 +4,10 @@
  * List recent traces from Sentry projects.
  */
 
-import { buildCommand } from "@stricli/core";
 import type { SentryContext } from "../../context.js";
 import { listTransactions } from "../../lib/api-client.js";
 import { validateLimit } from "../../lib/arg-parsing.js";
+import { buildCommand } from "../../lib/command.js";
 import {
   formatTraceRow,
   formatTracesHeader,

src/commands/trace/view.ts

@@ -4,7 +4,6 @@
  * View detailed information about a distributed trace.
  */
 
-import { buildCommand } from "@stricli/core";
 import type { SentryContext } from "../../context.js";
 import { getDetailedTrace } from "../../lib/api-client.js";
 import {
@@ -13,6 +12,7 @@ import {
   spansFlag,
 } from "../../lib/arg-parsing.js";
 import { openInBrowser } from "../../lib/browser.js";
+import { buildCommand } from "../../lib/command.js";
 import { ContextError, ValidationError } from "../../lib/errors.js";
 import {
   computeTraceSummary,

src/lib/command.ts

@@ -2,7 +2,15 @@
  * Command Builder with Telemetry
  *
  * Wraps Stricli's buildCommand to automatically capture flag usage for telemetry.
- * Commands should import buildCommand from this module instead of @stricli/core.
+ *
+ * ALL commands MUST import `buildCommand` from this module, NOT from `@stricli/core`.
+ * Importing directly from `@stricli/core` silently bypasses flag/arg telemetry capture.
+ *
+ * Correct:   import { buildCommand } from "../../lib/command.js";
+ * Incorrect: import { buildCommand } from "@stricli/core";  // skips telemetry!
+ *
+ * Exception: `help.ts` may import from `@stricli/core` because it also needs `run`,
+ * and the help command has no meaningful flags to capture.
  */
 
 import {