refactor(list): align all list commands to issue list standards (#453)

## Summary

Aligns all `list` subcommands to the standards established by `sentry
issue list`.

### Key Changes

1. **`sentry log list` — trace-id as positional arg**
   - Removed `--trace` flag entirely
- Accept trace-id positionally: `sentry log list <trace-id>`, `sentry
log list <org>/<trace-id>`
- Added `parseLogListArgs()` with `isTraceId()` disambiguation (32-char
hex → trace mode, otherwise project mode)
- Added `--period` / `-t` flag (default `90d`, trace mode auto-uses
`14d`)
- JSON output now returns `{ data, hasMore }` envelope instead of flat
array

2. **`withProgress` spinner in all list commands**
- `trace list`, `span list`, `trace logs`, `org list`, `trial list`,
`project list`
- Added to `org-list.ts` framework so `team list` and `repo list` get
spinners automatically

3. **JSON envelope alignment**
   - `trace logs` now returns `{ data, hasMore }` envelope

4. **Test improvements**
- Fixed pre-existing 5s timeout failures in `dispatchOrgScopedList`
tests (missing `resolveEffectiveOrg` mock)
   - Complete rewrite of `log/list.test.ts` (49 tests)
   - Added `isTraceId()` tests to `trace-id.test.ts`

### New helper

`isTraceId(value: string): boolean` — non-throwing check extracted from
`validateTraceId` core logic. Used by `log list` for positional
disambiguation.

### Files changed (15)

| File | Change |
|------|--------|
| `src/lib/trace-id.ts` | Add `isTraceId()` |
| `src/commands/log/list.ts` | Major: remove --trace, array positional,
--period, withProgress, JSON envelope |
| `src/commands/trace/list.ts` | Add withProgress |
| `src/commands/span/list.ts` | Add withProgress |
| `src/commands/trace/logs.ts` | Add withProgress + JSON envelope |
| `src/commands/org/list.ts` | Add withProgress |
| `src/commands/trial/list.ts` | Add withProgress |
| `src/commands/project/list.ts` | Add withProgress |
| `src/lib/org-list.ts` | Add withProgress to framework |
| `test/` files | Updated/new tests |

---------

Co-authored-by: betegon <miguelbetegongarcia@gmail.com>
Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

Author: 4 people

AGENTS.md

@@ -599,15 +599,15 @@ sentry team list --json
 
 View Sentry logs
 
-#### `sentry log list <org/project>`
+#### `sentry log list <org/project-or-trace-id...>`
 
 List logs from a project
 
 **Flags:**
 - `-n, --limit <value> - Number of log entries (1-1000) - (default: "100")`
 - `-q, --query <value> - Filter query (Sentry search syntax)`
 - `-f, --follow <value> - Stream logs (optionally specify poll interval in seconds)`
-- `--trace <value> - Filter logs by trace ID (32-character hex string)`
+- `-t, --period <value> - Time period (e.g., "90d", "14d", "24h"). Default: 90d (project mode), 14d (trace mode)`
 - `--fresh - Bypass cache, re-detect projects, and fetch fresh data`
 - `--json - Output as JSON`
 - `--fields <value> - Comma-separated fields to include in JSON output (dot.notation supported)`

plugins/sentry-cli/skills/sentry-cli/SKILL.md

@@ -135,7 +135,10 @@ export const listCommand = buildCommand({
     }
 
     const dashboards = await withProgress(
-      { message: `Fetching dashboards (up to ${flags.limit})...` },
+      {
+        message: `Fetching dashboards (up to ${flags.limit})...`,
+        json: flags.json,
+      },
       () => listDashboards(orgSlug, { perPage: flags.limit })
     );
     const url = buildDashboardsListUrl(orgSlug);

src/commands/dashboard/list.ts

@@ -814,7 +814,7 @@ async function handleOrgAllIssues(
   setContext([org], []);
 
   const { issues, nextCursor } = await withProgress(
-    { message: `Fetching issues (up to ${flags.limit})...` },
+    { message: `Fetching issues (up to ${flags.limit})...`, json: flags.json },
     (setMessage) =>
       fetchOrgAllIssues(org, flags, cursor, (fetched, limit) =>
         setMessage(
@@ -954,7 +954,7 @@ async function handleResolvedTargets(
       : "Fetching issues";
 
   const { results, hasMore } = await withProgress(
-    { message: `${baseMessage} (up to ${flags.limit})...` },
+    { message: `${baseMessage} (up to ${flags.limit})...`, json: flags.json },
     (setMessage) =>
       fetchWithBudget(
         activeTargets,