@ishlabs/cli 0.18.0 → 0.20.0

package/dist/commands/ask.js

@@ -1,7 +1,7 @@
 /**
  * ish ask — Create and run asks (multi-round surveys with variants).
  */
-import { withClient, getWebUrl, terminalLink, resolveWorkspace, resolveAsk, collectRepeatable, parseWaitTimeout, resolvePersonIds, addPersonFilterFlags, } from "../lib/command-helpers.js";
+import { withClient, getWebUrl, terminalLink, resolveWorkspace, resolveAsk, collectRepeatable, parseWaitTimeout, resolvePersonIds, addPersonFilterFlags, confirmDestructive, } from "../lib/command-helpers.js";
 import { resolveId, tagAlias, ALIAS_PREFIX } from "../lib/alias-store.js";
 import { loadConfig, saveConfig } from "../config.js";
 import { formatAskList, formatAskDetail, formatRoundDetail, formatAskResults, output, } from "../lib/output.js";
@@ -927,9 +927,33 @@ Examples:
         .description("Delete an ask and all its rounds, responses, and participants")
         .argument("[id]", "Ask alias or UUID (defaults to active ask)")
         .option("--ask <id>", "Ask ID; alternative to positional argument")
-        .addHelpText("after", "\nExamples:\n  $ ish ask delete a-6ec")
+        .option("-y, --yes", "Skip confirmation prompt (required in --json or non-TTY contexts)")
+        .addHelpText("after", `
+Examples:
+  $ ish ask delete a-6ec             # interactive — prompts for confirmation
+  $ ish ask delete a-6ec --yes       # non-interactive
+  $ ish ask delete a-6ec --json --yes`)
         .action(async (id, opts, cmd) => {
         await withClient(cmd, async (client, globals) => {
+            // Code-review #5: when neither positional id nor --ask is provided,
+            // we fall back to the active ask in config. If config has no active
+            // ask either, the prompt would say "Delete the active ask..." and
+            // then error AFTER confirmation with "No active ask" — confusing.
+            // Detect the empty-ref case BEFORE the prompt so the user sees
+            // the actionable error first.
+            const ref = id ?? opts.ask;
+            if (!ref) {
+                const config = loadConfig();
+                if (!config.ask) {
+                    throw new Error("No active ask to delete. Pass an ask id (positional or --ask <id>), or run `ish ask use <id>` to set one first.");
+                }
+            }
+            // Pattern G follow-up: confirm BEFORE resolving so a typo'd alias
+            // (e.g. `a-fff`) doesn't bypass the guard. Show the raw input in
+            // the prompt; the alias resolution happens after the user
+            // confirms.
+            const displayRef = ref ?? "active ask";
+            await confirmDestructive(`Delete ${displayRef === "active ask" ? "the active ask" : `ask ${displayRef}`} (and all its rounds, responses, participants)? This cannot be undone.`, { yes: opts.yes, json: globals.json });
             const aid = resolveAsk(pickAskRef(id, opts.ask));
             await client.del(`/asks/${aid}`);
             // If we just deleted the active ask, clear it from config.

package/dist/commands/config.js

@@ -12,7 +12,15 @@ export function registerConfigCommands(program) {
 A simulation config tunes how participants behave during a run (model, timing, retries, etc.).
 Pass \`--config <id>\` to \`ish study run\` to override the default for one dispatch.
 
-Configs are global — not scoped to a workspace. The --workspace flag is rejected. Use aliases (c-...) or UUIDs to identify configs.
+Configs are global — not scoped to a workspace. Passing \`--workspace\` to a
+\`config\` command is silently ignored (the global flag is parsed; the value
+has no effect on which configs are returned). Identify configs by alias
+(\`c-...\`) or UUID.
+
+Note: listing simulation configs requires admin privileges. Non-admins get a
+\`forbidden\` (403) envelope on \`ish config list\`. If you need a specific
+config, pass it directly via \`ish study run --config <id>\`; the run-time
+override doesn't require admin access.
 
 Run \`ish docs overview\` for the full mental model.`);
     config

package/dist/commands/docs.js

@@ -85,13 +85,12 @@ export function registerDocsCommands(program) {
     const docs = program
         .command("docs")
         .description("Offline docs for agents — mental model, concept pages, search")
-        .addHelpText("after", `\nSubcommands:
-  overview              Mental model + how to navigate (default)
-  list                  All pages with one-line descriptions
-  get-page <slug>       Full markdown for one page
-  search <query>        Keyword search across all pages
-
-Examples:
+        .addHelpText("after", 
+    // ISSUE-013: dropped the manual "Subcommands:" block — Commander's
+    // auto-generated "Commands:" section already lists the verbs. Keeping
+    // just Examples avoids the near-duplicate that was visible on
+    // `ish docs --help`.
+    `\nExamples:
   $ ish docs                                    # overview
   $ ish docs list
   $ ish docs get-page concepts/study

package/dist/commands/person.js

@@ -2,7 +2,7 @@
  * ish person — Manage people, generation, and source uploads.
  */
 import fs from "node:fs";
-import { withClient, readJsonFileOrStdin, resolveWorkspace } from "../lib/command-helpers.js";
+import { withClient, readJsonFileOrStdin, resolveWorkspace, confirmDestructive, normalizeVisibility } from "../lib/command-helpers.js";
 import { resolveId, tagAlias, ALIAS_PREFIX } from "../lib/alias-store.js";
 import { formatPersonList, output, } from "../lib/output.js";
 import { resolveTextContent } from "../lib/upload.js";
@@ -37,6 +37,12 @@ Concept pages: ish docs get-page concepts/person
         .option("--country <country>", "Filter by country code, e.g. US (repeatable)", collect, [])
         .option("--min-age <n>", "Minimum age")
         .option("--max-age <n>", "Maximum age")
+        // ISSUE-032: asymmetric — `ask create` and `study run` accept
+        // `--visibility workspace|shared|platform` but `person list` didn't,
+        // so a user looking for "people scoped to MY workspace" had to
+        // download all 13k+ platform people and grep client-side. Same
+        // normalizer + flag wording as the sibling commands.
+        .option("--visibility <v>", "Filter by visibility: workspace | shared | platform (legacy 'private'/'public' accepted as aliases)")
         .option("--limit <n>", "Max results (default 50)", "50")
         .option("--offset <n>", "Offset for pagination", "0")
         .addHelpText("after", `
@@ -47,6 +53,8 @@ Examples:
   $ ish person list --occupation founder --occupation designer
   $ ish person list --gender female --gender male --country US --country GB
   $ ish person list --type all --json
+  $ ish person list --visibility workspace            # only workspace-scoped people
+  $ ish person list --visibility platform --country US  # platform pool by country
 
   # Pagination: default --limit is 50, iterate with --offset.
   $ ish person list --limit 100
@@ -75,6 +83,9 @@ Examples:
                 params.min_age = opts.minAge;
             if (opts.maxAge)
                 params.max_age = opts.maxAge;
+            const normVis = normalizeVisibility(opts.visibility);
+            if (normVis)
+                params.visibility = normVis;
             const data = await client.get("/people", params);
             formatPersonList(data, globals.json, parseInt(opts.limit, 10));
             // Pattern H1: when there's more data, surface a stderr hint so agents
@@ -104,13 +115,110 @@ Examples:
     });
     person
         .command("create")
-        .description("Create a person from an exact JSON spec (no LLM)")
-        .requiredOption("--file <path>", "JSON file with person data")
+        .description("Create a person from an exact JSON spec (no LLM). Accepts inline flags or --file.")
+        // ISSUE-033: previously --file was the ONLY accepted input — asymmetric
+        // with `person update` which has a rich inline-flag surface. First-time
+        // users reading `person update --help` reasonably expected the same on
+        // create. Now mirrored: --file is optional, inline flags compose into
+        // the body, inline flags override --file values.
+        .option("--file <path>", "JSON file with person data (or '-' for stdin). Escape hatch for fields not covered by inline flags.")
+        .option("--name <text>", "Person name (required if --file omitted)")
+        .option("--type <value>", "Person type: ai | human (default: ai when --file omitted)")
+        .option("--description <text>", "Person description")
+        .option("--bio <text>", "Person bio")
+        .option("--occupation <text>", "Occupation")
+        .option("--country <code>", "Country code, e.g. US")
+        .option("--gender <g>", "Gender, e.g. female")
+        .option("--date-of-birth <YYYY-MM-DD>", "Date of birth")
+        .option("--education-level <value>", `Education level. One of: ${EDUCATION_LEVELS.join(", ")}`)
+        .option("--household <value>", `Household composition (MECE). One of: ${HOUSEHOLDS.join(", ")}. A couple raising children is couple_with_kids, not couple_no_kids; "single" means lives alone with no partner, roommates, parents, or children in the household.`)
+        .option("--locale-type <value>", `Self-described neighborhood type. One of: ${LOCALE_TYPES.join(", ")}`)
+        .option("--income-level <value>", `Self-identified relative socioeconomic position. One of: ${INCOME_LEVELS.join(", ")}`)
+        .option("--employment-status <value>", `Primary daytime activity / labor-force status. One of: ${EMPLOYMENT_STATUSES.join(", ")}`)
+        .option("--accessibility-profile <json-or-path>", "AccessibilityProfile v1.0 as an inline JSON string OR a path to a JSON file. Empty object {} is the canonical default. Validated client-side against the spec before submit.")
         .option("--workspace <id>", "Workspace (product) ID; falls back to active workspace")
-        .addHelpText("after", "\nExamples:\n  $ ish person create --file profile.json\n\n  Expected JSON: { \"name\": \"...\", \"type\": \"ai\", \"gender\": \"female\", \"country\": \"US\", \"occupation\": \"...\", \"bio\": \"...\" }")
+        .addHelpText("after", `
+Examples:
+  # Inline (mirrors person update):
+  $ ish person create --name "Alice" --type ai --country US
+  $ ish person create --name "Bob" --gender male --country GB --occupation founder \\
+      --household single --locale-type urban --income-level middle \\
+      --employment-status employed_full_time --bio "..."
+
+  # From file:
+  $ ish person create --file profile.json
+
+  # From stdin:
+  $ cat profile.json | ish person create --file -
+
+  # Compose file + inline (inline overrides):
+  $ ish person create --file profile.json --bio "Updated bio for this run"
+
+Inline flags compose into the create body. --file is an escape hatch for
+fields not covered by inline flags. When both are provided, inline flags
+override values from --file.
+
+Minimum required (when --file omitted): --name (--type defaults to "ai").
+
+Household MECE rule: a couple raising children is \`couple_with_kids\`, not
+\`couple_no_kids\`. \`single\` means lives alone with no partner, roommates,
+parents, or children sharing the household.
+
+\`--accessibility-profile\` accepts either an inline JSON string OR a path to
+a JSON file. Schema: https://ishlabs.io/spec/accessibility-profile-schema.v1.json`)
         .action(async (opts, cmd) => {
         await withClient(cmd, async (client, globals) => {
-            const body = await readJsonFileOrStdin(opts.file);
+            let body = {};
+            if (opts.file) {
+                body = (await readJsonFileOrStdin(opts.file));
+            }
+            // Inline flags compose / override the file body (mirrors person update).
+            if (opts.name !== undefined)
+                body.name = opts.name;
+            if (opts.type !== undefined)
+                body.type = opts.type;
+            if (opts.description !== undefined)
+                body.description = opts.description;
+            if (opts.bio !== undefined)
+                body.bio = opts.bio;
+            if (opts.occupation !== undefined)
+                body.occupation = opts.occupation;
+            if (opts.country !== undefined)
+                body.country = opts.country;
+            if (opts.gender !== undefined)
+                body.gender = opts.gender;
+            if (opts.dateOfBirth !== undefined)
+                body.date_of_birth = opts.dateOfBirth;
+            if (opts.educationLevel !== undefined) {
+                body.education_level = assertEnumValue(opts.educationLevel, EDUCATION_LEVELS, "--education-level");
+            }
+            if (opts.household !== undefined) {
+                body.household = assertEnumValue(opts.household, HOUSEHOLDS, "--household");
+            }
+            if (opts.localeType !== undefined) {
+                body.locale_type = assertEnumValue(opts.localeType, LOCALE_TYPES, "--locale-type");
+            }
+            if (opts.incomeLevel !== undefined) {
+                body.income_level = assertEnumValue(opts.incomeLevel, INCOME_LEVELS, "--income-level");
+            }
+            if (opts.employmentStatus !== undefined) {
+                body.employment_status = assertEnumValue(opts.employmentStatus, EMPLOYMENT_STATUSES, "--employment-status");
+            }
+            if (opts.accessibilityProfile !== undefined) {
+                body.accessibility_profile = parseAccessibilityProfileFlag(opts.accessibilityProfile);
+            }
+            // Type default — keeps the inline-only path ergonomic without
+            // requiring `--type ai` on every minimal create.
+            if (body.name && body.type === undefined) {
+                body.type = "ai";
+            }
+            // Without --file AND with no inline flags, the body has only
+            // product_id — that would 422 server-side with no useful guidance.
+            // Provide a clear local error first.
+            if (!opts.file && body.name === undefined) {
+                throw new Error("Nothing to create. Provide --file <path> or at least --name (plus optional inline flags). " +
+                    "Run `ish person create --help` for the full inline-flag set.");
+            }
             if (opts.workspace || body.product_id == null) {
                 body.product_id = resolveWorkspace(opts.workspace);
             }
@@ -135,7 +243,7 @@ Examples:
         .option("--no-scenarios", "Skip fetching the evidence-grounded scenarios for each generated person")
         .option("--no-wait", "Don't poll source-processing status. Only relevant when --source is a local path (paths get auto-uploaded and processed).")
         .option("--source-timeout <seconds>", "Source-processing poll timeout in seconds. Only relevant when --source is a local path.", "300")
-        .option("--timeout <seconds>", "Generation-job poll timeout in seconds (default 600). The job keeps running server-side past this; re-poll, don't re-enqueue.", "600")
+        .option("--timeout <seconds>", "Generation-job poll timeout in seconds (default 180; pass a larger value for long-running jobs). The job keeps running server-side past this; re-poll later or re-run with a longer --timeout — don't re-enqueue, that would duplicate the work.", "180")
         .addHelpText("after", `
 Generation is an async job: ish enqueues it, polls until the people plus
 their evidence-grounded scenarios are ready (~30-60s), then prints them. The
@@ -428,12 +536,18 @@ Schema: https://ishlabs.io/spec/accessibility-profile-schema.v1.json`)
         .description("Delete a person")
         .argument("<id>", "Person ID")
         .option("--workspace <id>", "Workspace ID; accepted for consistency (workspace is inferred from the person)")
-        .addHelpText("after", "\nExamples:\n  $ ish person delete <id>")
-        .action(async (id, _opts, cmd) => {
+        .option("-y, --yes", "Skip confirmation prompt (required in --json or non-TTY contexts)")
+        .addHelpText("after", `
+Examples:
+  $ ish person delete <id>             # interactive — prompts for confirmation
+  $ ish person delete <id> --yes       # non-interactive
+  $ ish person delete <id> --json --yes`)
+        .action(async (id, opts, cmd) => {
         await withClient(cmd, async (client, globals) => {
             const rid = resolveId(id);
+            await confirmDestructive(`Delete person ${tagAlias(ALIAS_PREFIX.person, rid)}? This cannot be undone.`, { yes: opts.yes, json: globals.json });
             await client.del(`/people/${rid}`);
-            output({ id: rid, alias: tagAlias(ALIAS_PREFIX.person, rid), message: "Profile deleted" }, globals.json, { writePath: true });
+            output({ id: rid, alias: tagAlias(ALIAS_PREFIX.person, rid), message: "Person deleted" }, globals.json, { writePath: true });
         });
     });
     person

package/dist/commands/secret.js

@@ -16,7 +16,7 @@
  *     of shell history.
  *   - No interactive prompts (CLI is for autonomous agents).
  */
-import { withClient, resolveWorkspace, readFileOrStdin } from "../lib/command-helpers.js";
+import { withClient, resolveWorkspace, readFileOrStdin, confirmDestructive } from "../lib/command-helpers.js";
 import { output } from "../lib/output.js";
 const RESERVED_SITE_ACCESS_KEYS = new Set([
     "BASIC_AUTH_USERNAME",
@@ -127,6 +127,8 @@ Examples:
         .option("--workspace <id>", "Workspace ID; defaults to active workspace")
         .addHelpText("after", `
 Pick exactly one of: positional <value>, --value-file <path>, --value-stdin.
+Bare \`-\` as the positional value is accepted as an alias for --value-stdin
+(matches the \`--value-file -\` convention).
 Stdin and --value-file are preferred for real keys so the value never lands
 in shell history.
 
@@ -138,6 +140,16 @@ Examples:
         .action(async (key, value, opts, cmd) => {
         await withClient(cmd, async (client, globals) => {
             validateKey(key);
+            // ISSUE-019: bare `-` as the positional value is a common Unix
+            // shorthand for "read from stdin" (matches `--value-file -`'s
+            // existing convention). Previously bare `-` was silently stored
+            // as the literal string "-" — undocumented + surprising. Treat
+            // it as an alias for --value-stdin so the convention matches
+            // user expectation. Documented in --help below.
+            if (value === "-") {
+                value = undefined;
+                opts.valueStdin = true;
+            }
             // Exactly one input source.
             const sources = [
                 value !== undefined,
@@ -212,7 +224,12 @@ Examples:
         .description("Delete a workspace secret by key")
         .argument("<key>", "Secret key to delete")
         .option("--workspace <id>", "Workspace ID; defaults to active workspace")
-        .addHelpText("after", "\nExamples:\n  $ ish secret delete GROQ_API_KEY")
+        .option("-y, --yes", "Skip confirmation prompt (required in --json or non-TTY contexts)")
+        .addHelpText("after", `
+Examples:
+  $ ish secret delete GROQ_API_KEY              # interactive — prompts for confirmation
+  $ ish secret delete GROQ_API_KEY --yes        # non-interactive
+  $ ish secret delete GROQ_API_KEY --json --yes`)
         .action(async (key, opts, cmd) => {
         await withClient(cmd, async (client, globals) => {
             if (!KEY_PATTERN.test(key)) {
@@ -220,6 +237,12 @@ Examples:
                 err.name = "ValidationError";
                 throw err;
             }
+            // Pattern G follow-up: the original Pattern G fix covered
+            // workspace/person/source delete (commit 891b22d), then a
+            // follow-up added ask delete. `secret delete` was the seventh
+            // destructive op and slipped through; rolling the guard in
+            // for full consistency.
+            await confirmDestructive(`Delete secret "${key}"? This cannot be undone.`, { yes: opts.yes, json: globals.json });
             const wid = resolveWorkspace(opts.workspace);
             const all = await client.get(`/products/${wid}/secrets`);
             const match = all.find((s) => s.key === key);

package/dist/commands/source.d.ts

@@ -2,7 +2,7 @@
  * ish source — Upload and inspect participant attachments used as generation inputs.
  *
  * Attachments (transcripts, audio, images, PDFs) are inputs to `ish person
- * generate`. For one-shot generation, `profile generate --source <path>`
+ * generate`. For one-shot generation, `ish person generate --source <path>`
  * auto-uploads. Use these commands to upload once and reuse across multiple
  * generation runs, or to inspect processing status.
  *

package/dist/commands/source.js

@@ -2,7 +2,7 @@
  * ish source — Upload and inspect participant attachments used as generation inputs.
  *
  * Attachments (transcripts, audio, images, PDFs) are inputs to `ish person
- * generate`. For one-shot generation, `profile generate --source <path>`
+ * generate`. For one-shot generation, `ish person generate --source <path>`
  * auto-uploads. Use these commands to upload once and reuse across multiple
  * generation runs, or to inspect processing status.
  *
@@ -10,7 +10,7 @@
  * existing scripts and agents; internally we call the unified
  * /people/attachments/* endpoint family (see ADR-0034).
  */
-import { withClient, resolveWorkspace } from "../lib/command-helpers.js";
+import { withClient, resolveWorkspace, confirmDestructive } from "../lib/command-helpers.js";
 import { resolveId, tagAlias, ALIAS_PREFIX } from "../lib/alias-store.js";
 import { normalizeEnumValue } from "../lib/enums.js";
 import { formatAttachment, output } from "../lib/output.js";
@@ -24,7 +24,7 @@ export function registerSourceCommands(program) {
 A source — internally a participant attachment — is an input to \`ish person generate\`:
 transcript, audio, image, or PDF. Use \`source upload\` when you want to reuse the
 same attachment across multiple generation runs; otherwise pass a local path directly
-to \`profile generate --source\` and it auto-uploads.
+to \`person generate --source\` and it auto-uploads.
 
 Concept pages: ish docs get-page concepts/source
                 ish docs get-page concepts/profile`);
@@ -86,21 +86,25 @@ Examples:
         .command("delete")
         .description("Delete a participant attachment plus its uploaded file")
         .argument("<id>", "Attachment ID or alias")
+        .option("-y, --yes", "Skip confirmation prompt (required in --json or non-TTY contexts)")
         .addHelpText("after", `
 The backend ref-counts attachment deletes: the file row + storage object are
 removed only when no profile mappings remain. Profiles already generated from
 this attachment keep their seed mapping until they themselves are deleted.
 
 Examples:
-  $ ish source delete ps-3a4`)
-        .action(async (id, _opts, cmd) => {
+  $ ish source delete ps-3a4             # interactive — prompts for confirmation
+  $ ish source delete ps-3a4 --yes       # non-interactive
+  $ ish source delete ps-3a4 --json --yes`)
+        .action(async (id, opts, cmd) => {
         await withClient(cmd, async (client, globals) => {
             const rid = resolveId(id);
+            await confirmDestructive(`Delete source ${tagAlias(ALIAS_PREFIX.personSource, rid)}? This cannot be undone.`, { yes: opts.yes, json: globals.json });
             await client.del(`/people/attachments/${rid}`);
             output({
                 id: rid,
                 alias: tagAlias(ALIAS_PREFIX.personSource, rid),
-                message: "Attachment deleted",
+                message: "Source deleted",
             }, globals.json, { writePath: true });
         });
     });

package/dist/commands/study-run.js

@@ -12,6 +12,7 @@ import * as readline from "node:readline/promises";
 import { withClient, getWebUrl, terminalLink, resolveWorkspace, resolveStudy, parseWaitTimeout, resolvePersonIds, addPersonFilterFlags, hasPersonFlags, readFileOrStdin, } from "../lib/command-helpers.js";
 import { resolveId, tagAlias, ALIAS_PREFIX } from "../lib/alias-store.js";
 import { output, formatSimulationPoll } from "../lib/output.js";
+import { fetchStudyParticipants } from "../lib/study-participants.js";
 import { streamStudyEvents } from "../lib/study-events.js";
 import { isMediaModality, isChatModality, iterationHasContent, describeRequiredContentFlag, readChatMode, readParticipantPairConfig, summarizeRoleCriteria, } from "../lib/modality.js";
 // NOTE: local-sim modules are loaded via dynamic import at the `--local`
@@ -107,24 +108,27 @@ const POLL_INTERVAL_MS = 5_000;
 // transparently reverts to POLL_INTERVAL_MS.
 const SSE_BACKSTOP_INTERVAL_MS = 30_000;
 const TERMINAL_STATUSES = new Set(["completed", "errored", "failed", "cancelled", "canceled"]);
-function flattenParticipantStatuses(iterations, only) {
+function flattenParticipantStatuses(participants, opts = {}) {
     const rows = [];
-    for (const iteration of iterations ?? []) {
-        for (const t of iteration.participants ?? []) {
-            if (only && !only.has(t.id))
-                continue;
-            // Pattern A (cli half): backend now reports per-participant crash detail at
-            // `error_message`. Keep `error` / `failure_reason` as legacy fallbacks
-            // until every backend deploy is on the new contract.
-            const errorMessage = t.error_message || t.error || t.failure_reason || null;
-            rows.push({
-                id: t.id,
-                status: t.status,
-                participant_name: t.person?.name || "Unknown",
-                interaction_count: Array.isArray(t.interactions) ? t.interactions.length : 0,
-                ...(errorMessage && { error_message: errorMessage }),
-            });
-        }
+    for (const t of participants ?? []) {
+        if (opts.iterationId && t.iteration_id !== opts.iterationId)
+            continue;
+        if (opts.only && !opts.only.has(t.id))
+            continue;
+        // Pattern A (cli half): backend now reports per-participant crash detail at
+        // `error_message`. Keep `error` / `failure_reason` as legacy fallbacks
+        // until every backend deploy is on the new contract.
+        const errorMessage = t.error_message ||
+            t.error ||
+            t.failure_reason ||
+            null;
+        rows.push({
+            id: t.id,
+            status: t.status,
+            participant_name: t.person?.name || "Unknown",
+            interaction_count: Array.isArray(t.interactions) ? t.interactions.length : 0,
+            ...(errorMessage && { error_message: String(errorMessage) }),
+        });
     }
     return rows;
 }
@@ -146,13 +150,15 @@ async function pollStudyUntilDone(client, opts) {
     let pendingEvent = eventIter.next();
     try {
         while (true) {
-            const study = await client.get(`/studies/${opts.studyId}`, undefined, { timeout: 60_000 });
+            const [study, participants] = await Promise.all([
+                client.get(`/studies/${opts.studyId}`, undefined, { timeout: 60_000 }),
+                fetchStudyParticipants(client, opts.studyId, { timeout: 60_000 }),
+            ]);
             const isMedia = isMediaModality(study.modality);
-            let iterations = study.iterations;
-            if (opts.iterationId) {
-                iterations = (iterations ?? []).filter((it) => it.id === opts.iterationId);
-            }
-            const rows = flattenParticipantStatuses(iterations, opts.participantIds);
+            const rows = flattenParticipantStatuses(participants, {
+                iterationId: opts.iterationId,
+                only: opts.participantIds,
+            });
             const total = rows.length;
             const done = rows.filter((r) => TERMINAL_STATUSES.has(r.status)).length;
             const errored = rows.filter((r) => r.status === "errored" || r.status === "failed").length;
@@ -366,7 +372,8 @@ Examples:
                 || parseInt(opts.dispatchTimeout, 10) < 1)) {
                 throw new Error(`--dispatch-timeout must be a positive integer (seconds), got "${opts.dispatchTimeout}".`);
             }
-            // Step 0: Fetch study (with its iterations + their existing participants)
+            // Step 0: Fetch study metadata (lite — participants live on a
+            // separate endpoint; we fetch them below only when reuse is possible).
             const study = await client.get(`/studies/${resolvedStudy}`);
             const modality = study.modality || "interactive";
             const isMedia = isMediaModality(modality);
@@ -455,8 +462,15 @@ Examples:
                 const resolved = await resolvePersonIds(client, resolvedWorkspace, opts, { requireSimulatable: false, allFlagName: "--all" });
                 personIds.push(...resolved);
             }
-            else if (iteration.participants && iteration.participants.length > 0) {
-                for (const t of iteration.participants) {
+            else if (!isPair) {
+                // Reuse-existing path: no person flags, non-pair iteration. Fetch
+                // participants from the dedicated endpoint and filter to the
+                // current iteration. (Pair iterations don't reuse participant rows
+                // — they reuse Conversation refs above.)
+                const studyParticipants = await fetchStudyParticipants(client, resolvedStudy);
+                for (const t of studyParticipants) {
+                    if (t.iteration_id !== iteration.id)
+                        continue;
                     const pid = t.person_id || t.person?.id;
                     const name = t.person?.name;
                     if (pid && !personNames.has(pid)) {
@@ -667,12 +681,22 @@ Examples:
                 //               language?: str }
                 //     reply : { conversations: [{ conversation_id, pair_index,
                 //                              participant_a_id, participant_b_id }] }
+                //
+                // On the LITE study response, the iteration's conversation refs
+                // use the storage-shape field names group_a_participant_id /
+                // group_b_participant_id. Map them back to the pair-batch reply
+                // shape (participant_a_id / participant_b_id) the rest of this
+                // function expects.
                 const existingConvs = iteration.conversations ?? [];
                 const reusable = [];
                 for (const c of existingConvs) {
                     const cid = c.conversation_id || c.id;
-                    if (cid && c.participant_a_id && c.participant_b_id) {
-                        reusable.push({ conversation_id: cid, participant_a_id: c.participant_a_id, participant_b_id: c.participant_b_id });
+                    if (cid && c.group_a_participant_id && c.group_b_participant_id) {
+                        reusable.push({
+                            conversation_id: cid,
+                            participant_a_id: c.group_a_participant_id,
+                            participant_b_id: c.group_b_participant_id,
+                        });
                     }
                 }
                 let pairRows;
@@ -1059,9 +1083,12 @@ Examples:
             // an agent that ran `study use s-...` then `study poll` would get a
             // confusing "Provide a participant_id argument or --study flag" error.
             const rid = resolveStudy(opts.study);
-            const study = await client.get(`/studies/${rid}`);
+            const [study, participants] = await Promise.all([
+                client.get(`/studies/${rid}`),
+                fetchStudyParticipants(client, rid),
+            ]);
             const isMedia = isMediaModality(study.modality);
-            const allParticipants = flattenParticipantStatuses(study.iterations);
+            const allParticipants = flattenParticipantStatuses(participants);
             formatSimulationPoll(allParticipants, globals.json, isMedia);
             if (!globals.json && study.product_id) {
                 const url = getWebUrl(globals, `/${study.product_id}/${rid}/timeline`);

package/dist/commands/study.js

@@ -8,6 +8,7 @@ import { resolveId, tagAlias, ALIAS_PREFIX } from "../lib/alias-store.js";
 import { loadConfig, saveConfig } from "../config.js";
 import { formatStudyList, formatStudyDetail, formatStudyResults, buildStudyResultsSummary, buildChatTranscript, output, ValidationError, } from "../lib/output.js";
 import { VALID_CONTENT_TYPES } from "../lib/types.js";
+import { fetchStudyParticipants } from "../lib/study-participants.js";
 import { parseAssignment, loadAssignmentsFile, validateAssignmentsArray, parseQuestion } from "../lib/study-inputs.js";
 import { loadQuestionsManifest } from "../lib/ask-questions.js";
 import { isLocalPath } from "../lib/upload.js";
@@ -543,8 +544,18 @@ Next: configure a run with \`ish iteration create --study <id>\`,
             const data = await client.post(`/products/${resolvedWs}/studies`, body);
             if (data.id) {
                 const config = loadConfig();
+                const prevStudy = config.study;
                 config.study = data.id;
                 saveConfig(config);
+                // Auto-activating the new study is intentional ergonomics (the
+                // common next step is `ish iteration create --study <new>`), but
+                // it was previously silent — surprised users who had set a
+                // different active study just before (ISSUE-030). Always
+                // surface the side-effect on stderr.
+                if (!globals.json) {
+                    const verb = prevStudy && prevStudy !== data.id ? "replaced" : "set";
+                    console.error(`Active study ${verb} to "${data.name || data.id}".`);
+                }
             }
             const result = data;
             if (result.id)
@@ -616,14 +627,17 @@ list table layout in human mode.`)
                 throw new Error("Provide at least one study id.");
             if (flat.length === 1) {
                 const rid = resolveId(flat[0]);
-                const data = await client.get(`/studies/${rid}`);
+                const [data, participants] = await Promise.all([
+                    client.get(`/studies/${rid}`),
+                    fetchStudyParticipants(client, rid),
+                ]);
                 const result = data;
                 if (result.id)
                     result.alias = tagAlias(ALIAS_PREFIX.study, String(result.id));
                 if (data.product_id) {
                     result.url = getWebUrl(globals, `/${data.product_id}/${rid}/overview`);
                 }
-                formatStudyDetail(result, globals.json);
+                formatStudyDetail(result, globals.json, {}, participants);
                 if (!globals.json && data.product_id) {
                     const url = getWebUrl(globals, `/${data.product_id}/${rid}/overview`);
                     console.error(`\n  ${terminalLink(url, "Open in browser ↗")}\n`);
@@ -632,13 +646,17 @@ list table layout in human mode.`)
             }
             const results = await Promise.all(flat.map(async (raw) => {
                 const rid = resolveId(raw);
-                const data = await client.get(`/studies/${rid}`);
+                const [data, participants] = await Promise.all([
+                    client.get(`/studies/${rid}`),
+                    fetchStudyParticipants(client, rid),
+                ]);
                 const r = data;
                 if (r.id)
                     r.alias = tagAlias(ALIAS_PREFIX.study, String(r.id));
                 if (data.product_id) {
                     r.url = getWebUrl(globals, `/${data.product_id}/${rid}/overview`);
                 }
+                r.participants = participants;
                 return r;
             }));
             if (globals.json) {
@@ -751,12 +769,15 @@ When no runs have completed, the default envelope is returned with zero counts a
                 output(buildChatTranscript(participant), globals.json, { preProjected: true });
                 return;
             }
-            const data = await client.get(`/studies/${rid}`);
+            const [data, participants] = await Promise.all([
+                client.get(`/studies/${rid}`),
+                fetchStudyParticipants(client, rid),
+            ]);
             if (wantsSummary) {
-                output(buildStudyResultsSummary(data), globals.json, { preProjected: true });
+                output(buildStudyResultsSummary(data, participants), globals.json, { preProjected: true });
             }
             else {
-                formatStudyResults(data, globals.json);
+                formatStudyResults(data, participants, globals.json);
             }
             if (!globals.json && data.product_id) {
                 const url = getWebUrl(globals, `/${data.product_id}/${rid}/overview`);
@@ -845,6 +866,15 @@ checklists ("steps") ride along when present in the JSON forms
                 json: globals.json,
             });
             await client.del(`/studies/${rid}`);
+            // If the deleted study was active, clear it (Pattern A — parallel to
+            // ask delete and chat endpoint delete which already do this).
+            const config = loadConfig();
+            if (config.study === rid) {
+                delete config.study;
+                saveConfig(config);
+                if (!globals.json)
+                    console.error("(Cleared active study.)");
+            }
             output({ id: rid, alias: tagAlias(ALIAS_PREFIX.study, rid), message: "Study deleted" }, globals.json, { writePath: true });
         });
     });