monday-cli 0.2.0 → 0.3.0

package/dist/commands/item/search.js

@@ -1,12 +1,33 @@
 /**
- * `monday item search --board <bid> --where ...` — column-value search
- * (`cli-design.md` §5.5, `v0.1-plan.md` §3 M4).
+ * `monday item search [--board <bid> | --workspace <wid> | --favorites]
+ * --where ...` — column-value search across one or many boards
+ * (`cli-design.md` §5.5 / §13 v0.3 entry, `v0.1-plan.md` §3 M4,
+ * `v0.3-plan.md` §3 M23).
  *
- * Backed by Monday's `items_page_by_column_values` endpoint — a
- * narrower surface than `items_page`'s `query_params.rules`:
- * value-equality only, AND across columns, OR within a column's
- * values. Items matching ANY of the listed values for a given
- * column count as a hit.
+ * **v0.1 single-board path (`--board <bid>` set).** Backed by
+ * Monday's `items_page_by_column_values` endpoint — a narrower
+ * surface than `items_page`'s `query_params.rules`: value-equality
+ * only, AND across columns, OR within a column's values. Items
+ * matching ANY of the listed values for a given column count as a
+ * hit.
+ *
+ * **v0.3-M23 cross-board path (`--board` omitted; `--workspace <wid>`,
+ * `--favorites`, or no scoping lever set — all-accessible-boards
+ * mode).** Uses a different shape on the wire: `boards(ids: [...])
+ * { items_page(query_params: { rules }) }` fan-out via
+ * `src/api/cross-board-search.ts`. Per-board cursors, per-board
+ * column resolution, `--max-boards 25` default cap (hard cap 100
+ * per Decision 5 closure `3a2f1db`). At most ONE of `--board` /
+ * `--workspace` / `--favorites` may be supplied — supplying two
+ * raises `usage_error` at the input-schema layer. **Runtime body
+ * shipped at `1f09a25`** — `--favorites` calls `fetchBoardFavorites`
+ * for the ID set; `--workspace <wid>` enumerates via
+ * `boards(workspace_ids:[wid])`; no scoping lever enumerates
+ * `boards(limit:N)` for all-accessible mode. Per-board column-
+ * resolution pre-pass surfaces `column_not_found_on_board` warnings
+ * for boards lacking the `--where` columns; the walker fans out
+ * over the surviving plans + surfaces `inaccessible_boards` /
+ * `cross_board_truncated` warnings per the empirical-probe findings.
  *
  * Why a separate command from `item list --where`: the endpoints
  * are different. `items_page_by_column_values` is purpose-built for
@@ -16,12 +37,13 @@
  * Monday's full filter DSL (any_of, contains_text, comparators,
  * is_empty) but pays the per-page complexity cost.
  *
- * v0.1 surface: only the `=` operator is supported via this command.
- * Multiple `--where status=A --where status=B` against the same
- * column merge into one entry with `[A, B]` (OR within column).
- * Multiple columns AND across entries. Anything else (`~=`, `<`,
- * `:is_empty`, etc.) raises `usage_error` — agents pick `item
- * list --where` for the richer surface.
+ * v0.1 single-board operator surface: only the `=` operator is
+ * supported via this command. Multiple `--where status=A --where
+ * status=B` against the same column merge into one entry with
+ * `[A, B]` (OR within column). Multiple columns AND across entries.
+ * Anything else (`~=`, `<`, `:is_empty`, etc.) raises `usage_error`
+ * — agents pick `item list --where` for the richer surface. Same
+ * operator surface applies to the cross-board path.
  *
  * Idempotent: yes.
  */
@@ -29,9 +51,12 @@ import { z } from 'zod';
 import { ensureSubcommand } from '../types.js';
 import { emitSuccess } from '../emit.js';
 import { resolveClient } from '../../api/resolve-client.js';
-import { BoardIdSchema } from '../../types/ids.js';
+import { BoardIdSchema, WorkspaceIdSchema } from '../../types/ids.js';
 import { parseArgv } from '../parse-argv.js';
-import { UsageError } from '../../utils/errors.js';
+import { ApiError, UsageError } from '../../utils/errors.js';
+import { DEFAULT_MAX_BOARDS, HARD_CAP_MAX_BOARDS, buildColumnNotFoundOnBoardWarning, crossBoardSearch, crossBoardSearchOutputSchema, } from '../../api/cross-board-search.js';
+import { fetchBoardFavorites, } from '../../api/board-favorites.js';
+import { SourceAggregator } from '../../api/source-aggregator.js';
 import { loadBoardMetadata, refreshBoardMetadata, } from '../../api/board-metadata.js';
 import { parseWhereSyntax } from '../../api/filters.js';
 import { resolveColumnsAcrossClauses } from '../../api/columns.js';
@@ -39,9 +64,8 @@ import { isMeToken } from '../../api/me-token.js';
 import { DEFAULT_PAGE_SIZE, paginate, } from '../../api/pagination.js';
 import { idFromRawItem, projectedItemSchema, } from '../../api/item-projection.js';
 import { ITEM_FIELDS_FRAGMENT, collectColumnHeads, projectFromRaw, resolveMeFactory, titleMap, } from '../../api/item-helpers.js';
-import { buildMeta } from '../../utils/output/envelope.js';
 import { selectOutput } from '../../utils/output/select.js';
-import { startNdjsonStream } from '../../utils/output/ndjson.js';
+import { buildStreamingTrailerMeta, startNdjsonStream, } from '../../utils/output/ndjson.js';
 import { collectSecrets } from '../../cli/envelope-out.js';
 const ITEMS_PAGE_BY_COLUMN_VALUES_QUERY = `
   query ItemsByColumnValues(
@@ -72,15 +96,97 @@ const ITEMS_BY_COLUMN_VALUES_NEXT_QUERY = `
   }
 `;
 export const itemSearchOutputSchema = z.array(projectedItemSchema);
+/**
+ * The command-registry-facing output schema. Per Codex round-2 P1-1,
+ * `monday item search`'s registry-level `outputSchema` is a union of
+ * the v0.1 single-board projection (`itemSearchOutputSchema` —
+ * `ProjectedItem[]`) AND the v0.3-M23 cross-board projection
+ * (`crossBoardSearchOutputSchema` — `CrossBoardItem[]`, each item
+ * carrying its source board id+name). The action body emits via
+ * the branch-specific schema (single-board uses
+ * `itemSearchOutputSchema` for the existing v0.1 path; cross-board
+ * uses `crossBoardSearchOutputSchema` at M23 implementation). The
+ * UNION here keeps `monday schema item.search` accurate to the
+ * runtime output across both branches.
+ *
+ * **Why a plain union and not a discriminated union.** The two
+ * shapes overlap on `id` / `name` but differ on the `board` slot
+ * (absent in single-board; required object in cross-board). Adding
+ * an explicit `cross_board: true` discriminator to one branch
+ * would carry-cost on every cross-board row for marginal agent
+ * value (the presence of the `board` slot itself discriminates).
+ * Agents that care can check `'board' in item`; agents that just
+ * want `id`/`name` consume both shapes uniformly.
+ */
+export const itemSearchCommandOutputSchema = z.union([
+    itemSearchOutputSchema,
+    crossBoardSearchOutputSchema,
+]);
 const inputSchema = z
     .object({
-    board: BoardIdSchema,
+    // v0.1 single-board scoping lever. v0.3-M23 makes this optional
+    // — when omitted, one of `workspace` / `favorites` / neither (=
+    // all-accessible-boards mode) picks the cross-board path.
+    board: BoardIdSchema.optional(),
+    // v0.3-M23 cross-board scoping levers (mutually exclusive with
+    // `--board` AND with each other per `.superRefine` below).
+    workspace: WorkspaceIdSchema.optional(),
+    favorites: z.boolean().optional(),
+    // v0.3-M23 cross-board fan-out cap (Decision 5 closure
+    // `3a2f1db`). Default 25, hard cap 100. Only meaningful when
+    // the cross-board path runs; with `--board` the flag is
+    // silently ignored. Codex P2-2 fix: the hard-cap enforcement
+    // is CONDITIONAL — applied only when `board` is absent (the
+    // cross-board path). On the single-board path the v0.1 user-
+    // facing contract is "flag is ignored", so we accept any
+    // positive integer there to honour that wording rather than
+    // rejecting agents that pass the flag harmlessly under
+    // `--board`.
+    maxBoards: z.coerce.number().int().positive().optional(),
     where: z.array(z.string()).min(1),
     all: z.boolean().optional(),
     limit: z.coerce.number().int().positive().max(10_000).optional(),
     pageSize: z.coerce.number().int().positive().max(500).optional(),
 })
-    .strict();
+    .strict()
+    .superRefine((value, ctx) => {
+    // v0.3-M23 mutual-exclusion rule per cli-design §13 v0.3 entry.
+    // At most ONE of `--board` / `--workspace` / `--favorites` may
+    // be supplied; supplying two surfaces a usage_error at the
+    // parse boundary rather than letting the runtime resolver
+    // surface a confusing error.
+    //
+    // Codex P2-3 fix: the path on the issue is empty (`[]`)
+    // because the conflict is about the COMBINATION of fields,
+    // not one specific field. The message + an explicit
+    // `conflicting_flags` slot in details carries the named
+    // levers for agent introspection.
+    const scopingLevers = [
+        value.board !== undefined ? 'board' : null,
+        value.workspace !== undefined ? 'workspace' : null,
+        value.favorites === true ? 'favorites' : null,
+    ].filter((s) => s !== null);
+    if (scopingLevers.length > 1) {
+        ctx.addIssue({
+            code: 'custom',
+            message: `at most one of --board / --workspace / --favorites may be supplied; got: ${scopingLevers.join(', ')}`,
+            path: [],
+            params: { conflicting_flags: scopingLevers },
+        });
+    }
+    // Codex P2-2 fix: cap only applies on the cross-board path.
+    // On the single-board path (board set), `--max-boards` is
+    // documented as ignored — accept any positive integer.
+    if (value.board === undefined &&
+        value.maxBoards !== undefined &&
+        value.maxBoards > HARD_CAP_MAX_BOARDS) {
+        ctx.addIssue({
+            code: 'custom',
+            message: `--max-boards exceeds the hard cap of ${String(HARD_CAP_MAX_BOARDS)} (wall-clock fan-out latency cap, not a complexity-budget cap; the cap protects against the 30s request timeout per Decision 5 \`3a2f1db\`); narrow the cross-board set with --workspace or --favorites`,
+            path: ['maxBoards'],
+        });
+    }
+});
 const buildColumnQueries = async (inputs) => {
     // Reject non-equality operators upfront — the endpoint doesn't
     // support them and validating before resolution avoids burning a
@@ -173,24 +279,204 @@ const extractNext = (r) => {
        next_items_page; same rationale as item/list.ts. */
     return { cursor: page?.cursor ?? null, items: page?.items ?? [] };
 };
+/**
+ * GraphQL document for cross-board enumeration (workspace + all-
+ * accessible scoping levers). Selects only `id` + `name` — the
+ * fan-out walker hydrates `name` again per-board, but we read it
+ * here too so a future debug path (e.g. printing the resolved set)
+ * has it without an extra round-trip.
+ *
+ * State filter pinned to `active` so archived/deleted boards don't
+ * pollute the cross-board set; agents wanting archived must use the
+ * v0.1 `--board <bid>` single-board path.
+ */
+const CROSS_BOARD_ENUM_QUERY = `
+  query CrossBoardEnumerate($workspaceIds: [ID], $limit: Int) {
+    boards(workspace_ids: $workspaceIds, limit: $limit, state: active) {
+      id
+      name
+    }
+  }
+`;
+/**
+ * Resolves the cross-board ID set per the scoping lever. Three
+ * paths:
+ *
+ *   - `--favorites`: call `fetchBoardFavorites`; project to board
+ *     IDs; truncate to `--max-boards`. Stale-favorites warning (if
+ *     surfaced by the favorites resolver) is folded into
+ *     `preWarnings`.
+ *   - `--workspace <wid>`: `boards(workspace_ids: [<wid>], limit:
+ *     <maxBoards>, state: active)`.
+ *   - neither: `boards(limit: <maxBoards>, state: active)`
+ *     (all-accessible mode).
+ *
+ * All three paths cap at `--max-boards` (parse boundary already
+ * rejected above-hard-cap values; here we honour the agent's
+ * resolved cap). `aggregator` records `'live'` for the enum call
+ * (the lever queries are pure-live; the cache-bearing source is
+ * the per-board metadata pre-pass which records separately).
+ */
+const resolveCrossBoardSet = async (inputs) => {
+    if (inputs.parsed.favorites === true) {
+        const result = await fetchBoardFavorites({ client: inputs.client });
+        inputs.aggregator.record(result.source, result.cacheAgeSeconds);
+        inputs.preWarnings.push(...result.warnings);
+        return result.boards
+            .slice(0, inputs.maxBoards)
+            .map((b) => BoardIdSchema.parse(b.id));
+    }
+    const variables = { limit: inputs.maxBoards };
+    if (inputs.parsed.workspace !== undefined) {
+        variables.workspaceIds = [inputs.parsed.workspace];
+    }
+    const response = await inputs.client.raw(CROSS_BOARD_ENUM_QUERY, variables, { operationName: 'CrossBoardEnumerate' });
+    inputs.aggregator.record('live', null);
+    const boards = response.data.boards ?? [];
+    return boards.map((b) => BoardIdSchema.parse(b.id));
+};
+/**
+ * Builds the `PerBoardScanPlan` for one board, or returns
+ * `undefined` if the board doesn't have all the columns the
+ * `--where` clauses need.
+ *
+ * Per-board steps:
+ *   1. Load metadata (cache-first; `--no-cache` bypasses).
+ *      Record per-leg source via `aggregator`.
+ *   2. Resolve every clause's column token against this board's
+ *      columns. On `column_not_found`, surface a
+ *      `column_not_found_on_board` warning (which board, which
+ *      token) and return `undefined` — the walker skips this
+ *      board.
+ *   3. Project the `--where` clauses into wire rules: resolve
+ *      `me` for people columns, group by resolved column ID, OR
+ *      multiple values within a column.
+ *   4. Validate clauses use only the `=` operator (cross-board
+ *      surface inherits the v0.1 single-board restriction).
+ */
+const buildPerBoardPlan = async (inputs) => {
+    // Validate operators upfront — same rule the v0.1 single-board
+    // path applies in buildColumnQueries. Cross-board uses the
+    // narrower `=`-only surface; richer filters live on
+    // `item list --where`.
+    for (const clause of inputs.clauses) {
+        if (clause.operator.kind !== 'equals') {
+            throw new UsageError(`item search supports only the = operator (got ${clause.operator.literal} ` +
+                `in ${JSON.stringify(clause.raw)}); use \`item list --where\` for richer filters`, { details: { clause: clause.raw, operator: clause.operator.literal } });
+        }
+    }
+    const meta = await loadBoardMetadata({
+        client: inputs.client,
+        boardId: inputs.boardId,
+        env: inputs.env,
+        noCache: inputs.noCache,
+    });
+    inputs.aggregator.record(meta.source, meta.cacheAgeSeconds);
+    // Per-board column resolution. We don't auto-refresh on
+    // column_not_found here — the cross-board path treats per-board
+    // column-not-found as a structured warning (the agent's intent
+    // is fan-out; a board missing a column is not a fatal error).
+    let resolved;
+    try {
+        resolved = await resolveColumnsAcrossClauses({
+            metadata: meta.metadata,
+            tokens: inputs.clauses.map((c) => c.token),
+        });
+    }
+    catch (err) {
+        if (err instanceof ApiError && err.code === 'column_not_found') {
+            const details = err.details;
+            const token = details?.token ?? '<unknown>';
+            inputs.preWarnings.push(buildColumnNotFoundOnBoardWarning(String(inputs.boardId), token));
+            return undefined;
+        }
+        // Defensive: resolveColumnsAcrossClauses throws either
+        // column_not_found (handled above) or `ambiguous_column` — the
+        // latter requires duplicate same-cased column titles within one
+        // board, which the cross-board action filters out at the
+        // BoardMetadata step (each board's resolution happens against
+        // its own metadata). Other ApiError codes here would indicate
+        // a contract bug in the resolver, not a recoverable path —
+        // re-throw so the action's error envelope surfaces it.
+        /* c8 ignore next */
+        throw err;
+    }
+    // Resolver collision / refresh warnings ResolverWarning widens
+    // cleanly to envelope.Warning per the v0.1 single-board path.
+    inputs.preWarnings.push(...resolved.warnings);
+    // Group clauses by resolved column ID, resolving `me` for people
+    // columns lazily.
+    let cachedMe;
+    const me = async () => {
+        cachedMe ??= await inputs.resolveMe();
+        return cachedMe;
+    };
+    const byColumn = new Map();
+    for (let i = 0; i < inputs.clauses.length; i++) {
+        const clause = inputs.clauses[i];
+        const match = resolved.matches[i];
+        /* c8 ignore next 4 — defensive: helper contract pins
+           matches.length === clauses.length. */
+        if (clause === undefined || match === undefined) {
+            throw new UsageError(`buildPerBoardPlan: lost clause/match alignment at index ${String(i)}`);
+        }
+        /* c8 ignore next 3 — defensive: parser guarantees binary ops
+           carry a value. */
+        if (clause.value === undefined) {
+            throw new UsageError(`internal: missing value for ${clause.raw}`);
+        }
+        let value = clause.value;
+        if (match.column.type === 'people' && isMeToken(value)) {
+            value = await me();
+        }
+        const existing = byColumn.get(match.column.id);
+        if (existing === undefined) {
+            byColumn.set(match.column.id, [value]);
+        }
+        else {
+            existing.push(value);
+        }
+    }
+    const rules = Array.from(byColumn, ([columnId, values]) => ({
+        column_id: columnId,
+        compare_values: values,
+    }));
+    return { board_id: inputs.boardId, rules };
+};
 export const itemSearchCommand = {
     name: 'item.search',
-    summary: 'Search items by column value (any_of) on one board',
+    summary: 'Search items by column value (any_of) on one board (v0.1) or across many boards (v0.3-M23 — --workspace / --favorites / no-scoping-lever)',
     examples: [
         "monday item search --board 12345 --where 'status=Done'",
         "monday item search --board 12345 --where 'status=Done' --where 'status=Backlog'",
         'monday item search --board 12345 --where owner=me --json',
         'monday item search --board 12345 --where status=Done --all --output ndjson',
+        'monday item search --favorites --where status=Done                     # v0.3-M23',
+        'monday item search --workspace 67890 --where status=Done --max-boards 50  # v0.3-M23',
     ],
     idempotent: true,
     inputSchema,
-    outputSchema: itemSearchOutputSchema,
+    // Codex round-2 P1-1: command-registry-facing schema is the union
+    // of single-board + cross-board projections. The action body emits
+    // via the branch-specific schema below (existing v0.1 path uses
+    // `itemSearchOutputSchema` directly; M23 cross-board impl uses
+    // `crossBoardSearchOutputSchema`). The union keeps
+    // `monday schema item.search` accurate across both branches.
+    outputSchema: itemSearchCommandOutputSchema,
     attach: (program, ctx) => {
         const noun = ensureSubcommand(program, 'item', 'Item commands');
         noun
             .command('search')
             .description(itemSearchCommand.summary)
-            .requiredOption('--board <bid>', 'board ID (required)')
+            // v0.3-M23: `--board` is now optional (v0.1 path still works when
+            // supplied; cross-board path runs when omitted). Mutual-exclusion
+            // with `--workspace` / `--favorites` enforced at the schema layer
+            // (`.superRefine`); missing-all-of-three is treated as
+            // "all-accessible-boards" cross-board mode at M23 implementation.
+            .option('--board <bid>', 'board ID (v0.1 single-board path; omit for v0.3 cross-board)')
+            .option('--workspace <wid>', 'workspace ID (v0.3-M23 cross-board scoping lever; mutually exclusive with --board / --favorites)')
+            .option('--favorites', "v0.3-M23 cross-board scoping lever — use the current user's `board favorites` set; mutually exclusive with --board / --workspace")
+            .option('--max-boards <n>', `v0.3-M23 cross-board fan-out cap (default ${String(DEFAULT_MAX_BOARDS)}, hard cap ${String(HARD_CAP_MAX_BOARDS)}; ignored on the single-board path)`)
             .requiredOption('--where <expr>', 'repeatable: <col>=<val> only (no <, ~=, :is_empty)', (value, prev) => [...prev, value], [])
             .option('--all', 'auto-paginate every page')
             .option('--limit <n>', 'cap total items returned across pages')
@@ -198,10 +484,164 @@ export const itemSearchCommand = {
             .addHelpText('after', ['', 'Examples:', ...itemSearchCommand.examples.map((e) => `  ${e}`), ''].join('\n'))
             .action(async (opts) => {
             const parsed = parseArgv(itemSearchCommand.inputSchema, opts);
+            // v0.3-M23 cross-board path. Dispatches on the scoping
+            // lever (`--favorites` / `--workspace <wid>` / neither =
+            // all-accessible-boards). Each branch resolves the board
+            // ID set, runs the per-board column-resolution pre-pass,
+            // and calls the fan-out walker.
+            if (parsed.board === undefined) {
+                const maxBoards = parsed.maxBoards ?? DEFAULT_MAX_BOARDS;
+                const { client, globalFlags, apiVersion, } = resolveClient(ctx, program.opts());
+                // Step 1: resolve the cross-board ID set per scoping lever.
+                // SourceAggregator tracks per-leg `meta.source`. The
+                // walker is always `'live'`; the pre-pass legs may be
+                // `'cache'` for cached board-metadata fetches.
+                const aggregator = new SourceAggregator();
+                const preWarnings = [];
+                const boardIds = await resolveCrossBoardSet({
+                    parsed,
+                    client,
+                    maxBoards,
+                    preWarnings,
+                    aggregator,
+                });
+                // Step 2: per-board column-resolution pre-pass. Each
+                // board's metadata is loaded (cache-first), the where
+                // clauses are resolved per-board, and any board where a
+                // clause's column doesn't resolve gets dropped with a
+                // `column_not_found_on_board` warning. The surviving
+                // boards become `PerBoardScanPlan` entries.
+                const clauses = parsed.where.map(parseWhereSyntax);
+                const resolveMe = resolveMeFactory(client);
+                const plans = [];
+                for (const boardId of boardIds) {
+                    const planResult = await buildPerBoardPlan({
+                        boardId,
+                        clauses,
+                        client,
+                        env: ctx.env,
+                        noCache: globalFlags.noCache,
+                        resolveMe,
+                        preWarnings,
+                        aggregator,
+                    });
+                    if (planResult !== undefined) {
+                        plans.push(planResult);
+                    }
+                }
+                const format = selectOutput({
+                    json: globalFlags.json,
+                    table: globalFlags.table,
+                    ...(globalFlags.output === undefined
+                        ? {}
+                        : { output: globalFlags.output }),
+                    env: ctx.env,
+                    isTTY: ctx.isTTY,
+                });
+                // Step 3: if no boards have viable plans, short-circuit
+                // with empty data + the per-board warnings. The walker
+                // requires non-empty boardIds (defensive); the empty
+                // case is the action's concern.
+                if (plans.length === 0) {
+                    aggregator.record('live', null);
+                    const aggregated = aggregator.result('live');
+                    emitSuccess({
+                        ctx,
+                        data: [],
+                        schema: crossBoardSearchOutputSchema,
+                        programOpts: program.opts(),
+                        kind: 'collection',
+                        source: aggregated.source,
+                        cacheAgeSeconds: aggregated.cacheAgeSeconds,
+                        warnings: preWarnings,
+                        apiVersion,
+                        hasMore: false,
+                        totalReturned: 0,
+                    });
+                    return;
+                }
+                // Step 4: streaming NDJSON path emits items as they
+                // arrive + writes the §6.3 trailer; bypasses
+                // emitSuccess because the streaming contract requires
+                // bytes hitting stdout before the walk completes.
+                if (format === 'ndjson') {
+                    const stream = startNdjsonStream({
+                        stream: ctx.stdout,
+                        secrets: collectSecrets(ctx.env, ctx.runtimeSecrets),
+                        project: (item) => ({ ...item }),
+                    });
+                    const walkResult = await crossBoardSearch({
+                        client,
+                        boardIds: plans.map((p) => p.board_id),
+                        plans,
+                        pageSize: parsed.pageSize ?? DEFAULT_PAGE_SIZE,
+                        ...(parsed.limit === undefined ? {} : { maxItems: parsed.limit }),
+                        onItem: stream.onItem,
+                    });
+                    aggregator.record(walkResult.source, null);
+                    const aggregated = aggregator.result('live');
+                    stream.writeTrailer(buildStreamingTrailerMeta({
+                        ctx: {
+                            cliVersion: ctx.cliVersion,
+                            requestId: ctx.requestId,
+                            clock: ctx.clock,
+                        },
+                        apiVersion,
+                        source: aggregated.source,
+                        cacheAgeSeconds: aggregated.cacheAgeSeconds,
+                        result: {
+                            hasMore: walkResult.hasMore,
+                            totalReturned: walkResult.totalReturned,
+                            complexity: walkResult.complexity,
+                            nextCursor: null,
+                        },
+                    }));
+                    return;
+                }
+                // Step 5: non-streaming path — full walker run, then
+                // emitSuccess with merged source aggregation + the union
+                // schema's cross-board branch.
+                const walkResult = await crossBoardSearch({
+                    client,
+                    boardIds: plans.map((p) => p.board_id),
+                    plans,
+                    pageSize: parsed.pageSize ?? DEFAULT_PAGE_SIZE,
+                    ...(parsed.limit === undefined ? {} : { maxItems: parsed.limit }),
+                });
+                aggregator.record(walkResult.source, null);
+                const aggregated = aggregator.result('live');
+                const warnings = [
+                    ...preWarnings,
+                    ...walkResult.warnings,
+                ];
+                emitSuccess({
+                    ctx,
+                    data: walkResult.items,
+                    schema: crossBoardSearchOutputSchema,
+                    programOpts: program.opts(),
+                    kind: 'collection',
+                    source: aggregated.source,
+                    cacheAgeSeconds: aggregated.cacheAgeSeconds,
+                    warnings,
+                    apiVersion,
+                    complexity: walkResult.complexity,
+                    hasMore: walkResult.hasMore,
+                    totalReturned: walkResult.totalReturned,
+                });
+                return;
+            }
+            // v0.3-M23: `parsed.board` is now `BoardId | undefined` at
+            // the schema layer; the cross-board branch above threw so
+            // `board` is `BoardId` here, but the narrowing doesn't carry
+            // through inner closures (the `onColumnNotFound` lambda
+            // captures the original optional type). Bind to a local
+            // before the single-board path so every reference inside
+            // closures sees the narrowed type.
+            const boardId = parsed.board;
             const { client, globalFlags, apiVersion, toEmit } = resolveClient(ctx, program.opts());
             const meta = await loadBoardMetadata({
                 client,
-                boardId: parsed.board,
+                boardId,
                 env: ctx.env,
                 noCache: globalFlags.noCache,
             });
@@ -210,7 +650,7 @@ export const itemSearchCommand = {
                 ? async () => {
                     const refreshed = await refreshBoardMetadata({
                         client,
-                        boardId: parsed.board,
+                        boardId,
                         env: ctx.env,
                     });
                     return refreshed.metadata;
@@ -244,11 +684,11 @@ export const itemSearchCommand = {
             if (format === 'ndjson') {
                 const stream = startNdjsonStream({
                     stream: ctx.stdout,
-                    secrets: collectSecrets(ctx.env),
+                    secrets: collectSecrets(ctx.env, ctx.runtimeSecrets),
                     project: (raw) => projectFromRaw(raw, titles, { omitColumnTitles: true }),
                 });
                 const result = await paginate({
-                    fetchInitial: initialFetcher(client, parsed.board, columns),
+                    fetchInitial: initialFetcher(client, boardId, columns),
                     fetchNext: nextFetcher(client),
                     now: ctx.clock,
                     extractPage: (r) => {
@@ -267,17 +707,21 @@ export const itemSearchCommand = {
                 // they're dropped per the §6.3 "no warnings in trailer"
                 // rule. Same shape as item list's streaming branch.
                 void filterWarnings;
-                stream.writeTrailer(buildMeta({
-                    api_version: apiVersion,
-                    cli_version: ctx.cliVersion,
-                    request_id: ctx.requestId,
+                stream.writeTrailer(buildStreamingTrailerMeta({
+                    ctx: {
+                        cliVersion: ctx.cliVersion,
+                        requestId: ctx.requestId,
+                        clock: ctx.clock,
+                    },
+                    apiVersion,
                     source: effectiveSource,
-                    retrieved_at: ctx.clock().toISOString(),
-                    cache_age_seconds: effectiveCacheAge,
-                    complexity: result.complexity,
-                    next_cursor: result.nextCursor,
-                    has_more: result.hasMore,
-                    total_returned: result.totalReturned,
+                    cacheAgeSeconds: effectiveCacheAge,
+                    result: {
+                        hasMore: result.hasMore,
+                        totalReturned: result.totalReturned,
+                        complexity: result.complexity,
+                        nextCursor: result.nextCursor,
+                    },
                     columns: columnHeads,
                 }));
                 return;