@ishlabs/cli 0.17.7 → 0.19.0

package/dist/lib/modality.js

@@ -44,28 +44,28 @@ export function iterationHasContent(details, modality) {
     }
     if (modality === "chat") {
         const mode = readChatMode(details);
-        if (mode === "tester_pair") {
-            const pair = readTesterPairConfig(details);
+        if (mode === "participant_pair") {
+            const pair = readParticipantPairConfig(details);
             if (!pair)
                 return false;
-            // Each side needs *either* an explicit audience or a role-criteria
+            // Each side needs *either* an explicit group or a role-criteria
             // filter. Scenarios are always required (they're how the persona
             // gets a role to inhabit).
-            const sideAHasAudience = pair.audience_a.length > 0 || !!pair.role_criteria_a;
-            const sideBHasAudience = pair.audience_b.length > 0 || !!pair.role_criteria_b;
-            if (!sideAHasAudience || !sideBHasAudience)
+            const sideAHasGroup = pair.group_a.length > 0 || !!pair.role_criteria_a;
+            const sideBHasGroup = pair.group_b.length > 0 || !!pair.role_criteria_b;
+            if (!sideAHasGroup || !sideBHasGroup)
                 return false;
             if (pair.scenario_a.length === 0 || pair.scenario_b.length === 0)
                 return false;
             // Equal-length pairing is only enforced when *both* sides ship an
-            // explicit audience. Criteria-driven sides are resolved server-side
-            // and persisted back to audience_*; until then the lengths may
+            // explicit group. Criteria-driven sides are resolved server-side
+            // and persisted back to group_*; until then the lengths may
             // legitimately differ (or be zero).
-            const bothAudiencesExplicit = pair.audience_a.length > 0
-                && pair.audience_b.length > 0
+            const bothGroupsExplicit = pair.group_a.length > 0
+                && pair.group_b.length > 0
                 && !pair.role_criteria_a
                 && !pair.role_criteria_b;
-            if (bothAudiencesExplicit && pair.audience_a.length !== pair.audience_b.length) {
+            if (bothGroupsExplicit && pair.group_a.length !== pair.group_b.length) {
                 return false;
             }
             return true;
@@ -89,8 +89,8 @@ export function describeRequiredContentFlag(modality, chatMode) {
     if (modality === "image")
         return "--image-urls <comma-separated>";
     if (modality === "chat") {
-        if (chatMode === "tester_pair") {
-            return "--chat-mode tester_pair (--audience-a <ids> | --role-criteria-a <json-or-@file>) (--audience-b <ids> | --role-criteria-b <json-or-@file>) --scenario-a <text-or-@file> --scenario-b <text-or-@file>";
+        if (chatMode === "participant_pair") {
+            return "--chat-mode participant_pair (--group-a <ids> | --role-criteria-a <json-or-@file>) (--group-b <ids> | --role-criteria-b <json-or-@file>) --scenario-a <text-or-@file> --scenario-b <text-or-@file>";
         }
         return "--endpoint <id> | --endpoint-config <file>";
     }
@@ -103,10 +103,10 @@ export function describeRequiredContentFlag(modality, chatMode) {
  * the read helpers below fall back to that legacy shape and treat it as
  * `external_chatbot`.
  */
-const CHAT_MODES = ["external_chatbot", "tester_pair"];
+const CHAT_MODES = ["external_chatbot", "participant_pair"];
 /**
- * Normalise user-supplied `--chat-mode` values. Accepts both `tester_pair`
- * (canonical) and `tester-pair` (hyphenated — matches CLI flag convention
+ * Normalise user-supplied `--chat-mode` values. Accepts both `participant_pair`
+ * (canonical) and `participant-pair` (hyphenated — matches CLI flag convention
  * elsewhere in `ish`); same for `external-chatbot`. Returns `null` for
  * anything unrecognised so callers can throw a clean ValidationError.
  */
@@ -120,8 +120,8 @@ export function readChatMode(details) {
     const md = details.mode_details;
     if (md && typeof md === "object") {
         const mode = md.mode;
-        if (mode === "tester_pair")
-            return "tester_pair";
+        if (mode === "participant_pair")
+            return "participant_pair";
         if (mode === "external_chatbot")
             return "external_chatbot";
     }
@@ -155,22 +155,22 @@ export function readExternalChatbotEndpoint(details) {
     };
 }
 /**
- * Extract the tester-pair payload from chat details. Returns `undefined` if
- * `mode_details.mode !== "tester_pair"`. Does not validate equal lengths or
+ * Extract the participant-pair payload from chat details. Returns `undefined` if
+ * `mode_details.mode !== "participant_pair"`. Does not validate equal lengths or
  * non-empty scenarios — that's the caller's job (see `iterationHasContent`
  * and `validateIterationDetails`).
  */
-export function readTesterPairConfig(details) {
+export function readParticipantPairConfig(details) {
     if (!details || typeof details !== "object")
         return undefined;
     const md = details.mode_details;
     if (!md || typeof md !== "object")
         return undefined;
     const m = md;
-    if (m.mode !== "tester_pair")
+    if (m.mode !== "participant_pair")
         return undefined;
-    const audA = Array.isArray(m.audience_a) ? m.audience_a.filter((x) => typeof x === "string") : [];
-    const audB = Array.isArray(m.audience_b) ? m.audience_b.filter((x) => typeof x === "string") : [];
+    const audA = Array.isArray(m.group_a) ? m.group_a.filter((x) => typeof x === "string") : [];
+    const audB = Array.isArray(m.group_b) ? m.group_b.filter((x) => typeof x === "string") : [];
     const scenA = typeof m.scenario_a === "string" ? m.scenario_a : "";
     const scenB = typeof m.scenario_b === "string" ? m.scenario_b : "";
     const init = m.initiator_side === "b" ? "b" : "a";
@@ -181,8 +181,8 @@ export function readTesterPairConfig(details) {
         ? m.role_criteria_b
         : undefined;
     return {
-        audience_a: audA,
-        audience_b: audB,
+        group_a: audA,
+        group_b: audB,
         scenario_a: scenA,
         scenario_b: scenB,
         initiator_side: init,
@@ -191,8 +191,8 @@ export function readTesterPairConfig(details) {
     };
 }
 /**
- * RoleCriteria — persona-first filter that resolves a tester-profile pool
- * for one side of a tester_pair iteration. Mirrors the backend Pydantic
+ * RoleCriteria — persona-first filter that resolves a person pool
+ * for one side of a participant_pair iteration. Mirrors the backend Pydantic
  * shape in `app/api/models/iterations.py:RoleCriteria` (swift-charm plan).
  * All fields optional; an empty object is treated as "no filter".
  */
@@ -328,7 +328,7 @@ export function validateIterationDetails(modality, details) {
     const hasEndpointObj = !!ep.endpoint;
     const hasEndpointId = !!ep.chatbot_endpoint_id;
     const hasModeDetails = !!d.mode_details && typeof d.mode_details === "object";
-    const isPair = readChatMode(details) === "tester_pair";
+    const isPair = readChatMode(details) === "participant_pair";
     const hasContentText = typeof d.content_text === "string" && d.content_text.length > 0;
     const hasContentUrl = typeof d.content_url === "string" && d.content_url.length > 0;
     const imageUrls = d.image_urls;
@@ -337,12 +337,12 @@ export function validateIterationDetails(modality, details) {
         : typeof imageUrls === "string" && imageUrls.length > 0;
     if (modality === "chat") {
         if (isPair) {
-            const pair = readTesterPairConfig(details);
+            const pair = readParticipantPairConfig(details);
             if (!pair) {
                 return {
                     ok: false,
-                    reason: "missing or malformed mode_details for tester_pair chat iteration",
-                    suggestion: "tester_pair iterations require mode_details: { mode: 'tester_pair', audience_a|role_criteria_a, audience_b|role_criteria_b, scenario_a, scenario_b }",
+                    reason: "missing or malformed mode_details for participant_pair chat iteration",
+                    suggestion: "participant_pair iterations require mode_details: { mode: 'participant_pair', group_a|role_criteria_a, group_b|role_criteria_b, scenario_a, scenario_b }",
                 };
             }
             // Validate criteria shape if present; surfaces client-side errors
@@ -358,33 +358,33 @@ export function validateIterationDetails(modality, details) {
                     suggestion: `allowed keys: ${ROLE_CRITERIA_KEYS.join(", ")}`,
                 };
             }
-            const sideAHasAudience = pair.audience_a.length > 0 || !!pair.role_criteria_a;
-            const sideBHasAudience = pair.audience_b.length > 0 || !!pair.role_criteria_b;
-            if (!sideAHasAudience || !sideBHasAudience) {
+            const sideAHasGroup = pair.group_a.length > 0 || !!pair.role_criteria_a;
+            const sideBHasGroup = pair.group_b.length > 0 || !!pair.role_criteria_b;
+            if (!sideAHasGroup || !sideBHasGroup) {
                 return {
                     ok: false,
-                    reason: "tester_pair iteration is missing audience input on at least one side",
-                    suggestion: "each side needs either an explicit audience (audience_a/audience_b) or a role-criteria filter (role_criteria_a/role_criteria_b)",
+                    reason: "participant_pair iteration is missing group input on at least one side",
+                    suggestion: "each side needs either an explicit group (group_a/group_b) or a role-criteria filter (role_criteria_a/role_criteria_b)",
                 };
             }
-            const bothAudiencesExplicit = pair.audience_a.length > 0
-                && pair.audience_b.length > 0
+            const bothGroupsExplicit = pair.group_a.length > 0
+                && pair.group_b.length > 0
                 && !pair.role_criteria_a
                 && !pair.role_criteria_b;
-            const lenA = pair.audience_a.length;
-            const lenB = pair.audience_b.length;
+            const lenA = pair.group_a.length;
+            const lenB = pair.group_b.length;
             const validPairing = lenA === lenB || lenA === 1 || lenB === 1;
-            if (bothAudiencesExplicit && !validPairing) {
+            if (bothGroupsExplicit && !validPairing) {
                 return {
                     ok: false,
-                    reason: `tester_pair audiences cannot be paired (audience_a=${lenA}, audience_b=${lenB})`,
+                    reason: `participant_pair groups cannot be paired (group_a=${lenA}, group_b=${lenB})`,
                     suggestion: "pick the same number on each side (zip 1:1 by index), or exactly 1 on one side to broadcast across the other; or use role_criteria_a/_b to let the backend resolve the pool",
                 };
             }
             if (pair.scenario_a.length === 0 || pair.scenario_b.length === 0) {
                 return {
                     ok: false,
-                    reason: "tester_pair iteration is missing scenario_a or scenario_b",
+                    reason: "participant_pair iteration is missing scenario_a or scenario_b",
                     suggestion: "both scenarios are required and asymmetric; pass --scenario-a and --scenario-b on `iteration create`",
                 };
             }
@@ -396,7 +396,7 @@ export function validateIterationDetails(modality, details) {
             return {
                 ok: false,
                 reason: "unrecognised mode_details for chat iteration",
-                suggestion: "mode_details must be either { mode:'external_chatbot', endpoint|chatbot_endpoint_id } or { mode:'tester_pair', audience_a, audience_b, scenario_a, scenario_b }",
+                suggestion: "mode_details must be either { mode:'external_chatbot', endpoint|chatbot_endpoint_id } or { mode:'participant_pair', group_a, group_b, scenario_a, scenario_b }",
             };
         }
         if (hasUrl) {
@@ -410,7 +410,7 @@ export function validateIterationDetails(modality, details) {
             return {
                 ok: false,
                 reason: "media-shape details on a chat iteration",
-                suggestion: "chat iterations require mode_details with endpoint/chatbot_endpoint_id (external_chatbot mode) or audience_a/_b + scenario_a/_b (tester_pair mode)",
+                suggestion: "chat iterations require mode_details with endpoint/chatbot_endpoint_id (external_chatbot mode) or group_a/_b + scenario_a/_b (participant_pair mode)",
             };
         }
         // No recognised content keys at all. Don't reject — agent might be

package/dist/lib/output.d.ts

@@ -47,36 +47,37 @@ export declare function formatWorkspaceList(workspaces: Record<string, unknown>[
 export declare function formatWorkspaceDetail(workspace: Record<string, unknown>, json: boolean, options?: OutputOptions): void;
 export declare function formatSiteAccessStatus(summary: import("./site-access.js").SiteAccessSummary, json: boolean): void;
 export declare function formatStudyList(studies: Record<string, unknown>[], json: boolean): void;
-export declare function formatStudyDetail(study: Record<string, unknown>, json: boolean, options?: OutputOptions): void;
-export declare function formatStudyResults(study: Record<string, unknown>, json: boolean): void;
+export declare function formatStudyDetail(study: Record<string, unknown>, json: boolean, options?: OutputOptions, participants?: ReadonlyArray<Record<string, unknown>>): void;
+export declare function formatStudyResults(study: Record<string, unknown>, participants: ReadonlyArray<Record<string, unknown>>, json: boolean): void;
 /**
- * `study results --summary` projection. Drops interview_answers + per-tester
+ * `study results --summary` projection. Drops interview_answers + per-participant
  * interaction breakdowns; keeps headline counters, sentiment histogram, and a
- * per-tester {alias, status, sentiment, comment} row. Useful for agents that
+ * per-participant {alias, status, sentiment, comment} row. Useful for agents that
  * need to branch on outcome without paying for the full envelope.
  */
-export declare function buildStudyResultsSummary(study: Record<string, unknown>): Record<string, unknown>;
+export declare function buildStudyResultsSummary(study: Record<string, unknown>, participants: ReadonlyArray<Record<string, unknown>>): Record<string, unknown>;
 /**
- * `study results --transcript <tester_id>` projection. Mirrors the schema
+ * `study results --transcript <participant_id>` projection. Mirrors the schema
  * MCP's `get_chat_transcript` returns (`src/ish_mcp/projections.py:
  * build_chat_transcript`) so callers see the same shape regardless of
- * surface. Tester turns whose action carries no text (e.g. select_option)
+ * surface. Participant turns whose action carries no text (e.g. select_option)
  * surface `text: null`; intent lives on `action_type` + `option_label`.
  * Bot turns with a `bot_reply.failure` block surface `failure` and
  * `text: null` and don't count toward `unique_bot_replies`.
  */
-export declare function buildChatTranscript(tester: Record<string, unknown>): Record<string, unknown>;
+export declare function buildChatTranscript(participant: Record<string, unknown>): Record<string, unknown>;
 /**
- * `study tester --summary` projection. Drops the action timeline; keeps the
+ * `study participant --summary` projection. Drops the action timeline; keeps the
  * headline (alias, status, sentiment, comment, error_message). Useful for
- * the common "did this tester finish, what did they say" check that's
+ * the common "did this participant finish, what did they say" check that's
  * currently buried under the full interactions array.
  */
-export declare function buildTesterSummary(tester: Record<string, unknown>): Record<string, unknown>;
+export declare function buildParticipantSummary(participant: Record<string, unknown>): Record<string, unknown>;
 export declare function formatIterationList(iterations: Record<string, unknown>[], json: boolean): void;
-export declare function formatTesterDetail(tester: Record<string, unknown>, json: boolean): void;
-export declare function formatTesterProfileList(profiles: unknown, json: boolean, limit?: number): void;
-export declare function formatAudienceSource(source: Record<string, unknown>, json: boolean): void;
+export declare function formatParticipantDetail(participant: Record<string, unknown>, json: boolean): void;
+export declare function formatPersonList(profiles: unknown, json: boolean, limit?: number): void;
+export declare function formatAttachment(attachment: Record<string, unknown>, json: boolean): void;
+export declare const formatPersonSource: typeof formatAttachment;
 export declare function formatGeneratedProfileList(profiles: unknown, json: boolean): void;
 export declare function formatSimulationPoll(results: Record<string, unknown>[], json: boolean, isMedia?: boolean): void;
 export declare function formatAskList(asks: Record<string, unknown>[], json: boolean): void;
@@ -90,7 +91,7 @@ export declare function formatRoundDetail(round: Record<string, unknown>, json:
  *   - medium: 3 <= n < 10 (small sample but clean)
  *   - high:   n >= 10 AND no errored responses AND not tied
  *
- * Tuned for the typical 5-tester ask: a clean 5/5 lands at "medium" (you
+ * Tuned for the typical 5-participant ask: a clean 5/5 lands at "medium" (you
  * can probably trust the lean), 1/5 with no errors lands at "low" (you
  * need more data), 5/5 with a tie lands at "low" (no winner to call).
  */