fix: address second round of bot review comments

- Replace parsed.type ternaries with Extract<> casts in issue/list.ts and project/list.ts override closures
- Fix handleProjectSearch to throw ContextError("Project", ...) instead of ContextError(config.entityName, ...)
- Add List Command Infrastructure section to AGENTS.md documenting list-command.ts and org-list.ts helpers

Author: BYK

AGENTS.md

@@ -319,6 +319,19 @@ import type { SentryContext } from "../../context.js";
 import { getAuthToken } from "../../lib/config.js";
 ```
 
+### List Command Infrastructure
+
+Two abstraction levels exist for list commands:
+
+1. **`src/lib/list-command.ts`** — `buildOrgListCommand` factory + shared Stricli parameter constants (`LIST_TARGET_POSITIONAL`, `LIST_JSON_FLAG`, `LIST_CURSOR_FLAG`, `buildListLimitFlag`). Use this for simple entity lists like `team list` and `repo list`.
+
+2. **`src/lib/org-list.ts`** — `dispatchOrgScopedList` with `OrgListConfig` and a 4-mode handler map: `auto-detect`, `explicit`, `org-all`, `project-search`. Complex commands (`project list`, `issue list`) call `dispatchOrgScopedList` with an `overrides` map directly instead of using `buildOrgListCommand`.
+
+Key rules when writing overrides:
+- Use `Extract<ParsedOrgProject, { type: "..." }>` casts to access variant-specific fields — TypeScript cannot narrow discriminated unions across closure boundaries.
+- `resolveCursor()` must be called **inside** the `org-all` override closure, not before `dispatchOrgScopedList`, so that `--cursor` validation errors fire correctly for non-org-all modes.
+- `handleProjectSearch` errors must use `"Project"` as the `ContextError` resource, not `config.entityName`.
+
 ## Commenting & Documentation (JSDoc-first)
 
 ### Default Rule

src/commands/issue/list.ts

@@ -13,7 +13,10 @@ import {
   listIssuesPaginated,
   listProjects,
 } from "../../lib/api-client.js";
-import { parseOrgProjectArg } from "../../lib/arg-parsing.js";
+import {
+  type ParsedOrgProject,
+  parseOrgProjectArg,
+} from "../../lib/arg-parsing.js";
 import { buildCommand } from "../../lib/command.js";
 import {
   clearPaginationCursor,
@@ -699,7 +702,7 @@ export const listCommand = buildCommand({
         "org-all": () =>
           handleOrgAllIssues({
             stdout,
-            org: parsed.type === "org-all" ? parsed.org : "",
+            org: (parsed as Extract<ParsedOrgProject, { type: "org-all" }>).org,
             flags,
             setContext,
           }),

src/commands/project/list.ts

@@ -673,27 +673,31 @@ export const listCommand = buildCommand({
       parsed,
       overrides: {
         "auto-detect": () => handleAutoDetect(stdout, cwd, flags),
-        explicit: () =>
-          handleExplicit(
-            stdout,
-            parsed.type === "explicit" ? parsed.org : "",
-            parsed.type === "explicit" ? parsed.project : "",
-            flags
-          ),
+        explicit: () => {
+          const p = parsed as Extract<ParsedOrgProject, { type: "explicit" }>;
+          return handleExplicit(stdout, p.org, p.project, flags);
+        },
         "org-all": () => {
           // Build context key and resolve cursor only in org-all mode, after
           // dispatchOrgScopedList has already validated --cursor is allowed here.
-          const org = parsed.type === "org-all" ? parsed.org : "";
+          const p = parsed as Extract<ParsedOrgProject, { type: "org-all" }>;
           const contextKey = buildContextKey(parsed, flags, getApiBaseUrl());
           const cursor = resolveCursor(flags.cursor, contextKey);
-          return handleOrgAll({ stdout, org, flags, contextKey, cursor });
-        },
-        "project-search": () =>
-          handleProjectSearch(
+          return handleOrgAll({
             stdout,
-            parsed.type === "project-search" ? parsed.projectSlug : "",
-            flags
-          ),
+            org: p.org,
+            flags,
+            contextKey,
+            cursor,
+          });
+        },
+        "project-search": () => {
+          const p = parsed as Extract<
+            ParsedOrgProject,
+            { type: "project-search" }
+          >;
+          return handleProjectSearch(stdout, p.projectSlug, flags);
+        },
       },
     });
   },

src/lib/org-list.ts

@@ -505,7 +505,7 @@ export async function handleProjectSearch<TEntity, TWithOrg>(
       return;
     }
     throw new ContextError(
-      config.entityName,
+      "Project",
       `No project '${projectSlug}' found in any accessible organization.\n\n` +
         `Try: ${config.commandPrefix} <org>/${projectSlug}`
     );