primitive-admin 1.1.0-alpha.75 → 1.1.0-alpha.76

package/dist/src/commands/workflows.js

@@ -6,6 +6,7 @@ import { generateWorkflowTypes, } from "../lib/workflow-codegen/generator.js";
 import { generateWorkflowSwiftTypes } from "../lib/swift-codegen/generator.js";
 import { ApiClient } from "../lib/api-client.js";
 import { resolveAppId } from "../lib/config.js";
+import { resolveWorkflowSelector } from "../lib/block-selector.js";
 import { formatAccessRuleForDisplay } from "../lib/access-rule-display.js";
 import { resolveCodegenSourceDir } from "../lib/codegen-shared/resolveCodegenSourceDir.js";
 import { resolveSyncDirOverride } from "../lib/sync-dir-selector.js";
@@ -254,6 +255,17 @@ export function formatConfigSlotLabel(slot, configName, configId, prefixLength =
     }
     return slot;
 }
+/**
+ * Resolve a workflow reference — its server id, or the author-chosen key its
+ * TOML file is named for — for the `tests` verbs (#3249).
+ *
+ * The key is what an operator has: `workflows/<key>.toml` is the object and
+ * `workflows/<key>.tests/` its cases. The resolution lives in
+ * `block-selector.ts`, shared with `prompts`.
+ */
+async function resolveWorkflowId(client, appId, ref) {
+    return (await resolveWorkflowSelector(client, appId, ref)).blockId;
+}
 export function registerWorkflowsCommands(program) {
     const workflows = program
         .command("workflows")
@@ -1810,22 +1822,30 @@ Test cases are authored in TOML at workflows/<key>.tests/<name>.toml and applied
 with \`primitive config push\` (issue #2769); \`config fields workflow\` lists the
 fields. Deleting a case is removing its file and running \`config push --prune\`.
 
+Every verb here takes the workflow's KEY or its ID (#3249). An identifier
+that names no workflow in the app fails; "No test cases found." only ever
+means the workflow exists and has no cases.
+
 Examples:
-  $ primitive workflows tests list <workflow-id>
-  $ primitive workflows tests run <workflow-id> <test-case-id>
-  $ primitive workflows tests run-all <workflow-id>
-  $ primitive workflows tests run-all <workflow-id> --test-cases 01ABC,01DEF,01GHI
+  $ primitive workflows tests list <workflow>
+  $ primitive workflows tests run <workflow> <test-case-id>
+  $ primitive workflows tests run-all <workflow>
+  $ primitive workflows tests run-all <workflow> --test-cases 01ABC,01DEF,01GHI
 `);
     tests
         .command("list")
         .description("List test cases for a workflow")
-        .argument("<workflow-id>", "Workflow ID")
+        .argument("<workflow>", "Workflow key or ID")
         .option("--app <app-id>", "App ID (uses current app if not specified)")
         .option("--json", "Output as JSON")
-        .action(async (workflowId, options) => {
+        .action(async (workflow, options) => {
         const resolvedAppId = resolveAppId(undefined, options);
         const client = new ApiClient();
         try {
+            // #3249 — resolve BEFORE listing: the test-case endpoint answers an
+            // unknown block with an empty page, so an unresolved identifier would
+            // print "No test cases found." and exit 0.
+            const workflowId = await resolveWorkflowId(client, resolvedAppId, workflow);
             const { items } = await client.listTestCases(resolvedAppId, "workflow", workflowId);
             if (options.json) {
                 json(items);
@@ -1850,14 +1870,15 @@ Examples:
     tests
         .command("get")
         .description("Get test case details")
-        .argument("<workflow-id>", "Workflow ID")
+        .argument("<workflow>", "Workflow key or ID")
         .argument("<test-case-id>", "Test Case ID")
         .option("--app <app-id>", "App ID (uses current app if not specified)")
         .option("--json", "Output as JSON")
-        .action(async (workflowId, testCaseId, options) => {
+        .action(async (workflow, testCaseId, options) => {
         const resolvedAppId = resolveAppId(undefined, options);
         const client = new ApiClient();
         try {
+            const workflowId = await resolveWorkflowId(client, resolvedAppId, workflow);
             const result = await client.getTestCase(resolvedAppId, "workflow", workflowId, testCaseId);
             if (options.json) {
                 json(result);
@@ -1917,14 +1938,15 @@ Examples:
     tests
         .command("run")
         .description("Run a single test case")
-        .argument("<workflow-id>", "Workflow ID")
+        .argument("<workflow>", "Workflow key or ID")
         .argument("<test-case-id>", "Test Case ID")
         .option("--app <app-id>", "App ID (uses current app if not specified)")
         .option("--json", "Output as JSON")
-        .action(async (workflowId, testCaseId, options) => {
+        .action(async (workflow, testCaseId, options) => {
         const resolvedAppId = resolveAppId(undefined, options);
         const client = new ApiClient();
         try {
+            const workflowId = await resolveWorkflowId(client, resolvedAppId, workflow);
             // Get test case to retrieve its variables
             const testCase = await client.getTestCase(resolvedAppId, "workflow", workflowId, testCaseId);
             // #3158 — the case's stored input IS the run's input, and the server
@@ -1983,14 +2005,17 @@ Examples:
     tests
         .command("run-all")
         .description("Run all test cases for a workflow, or a specific subset")
-        .argument("<workflow-id>", "Workflow ID")
+        .argument("<workflow>", "Workflow key or ID")
         .option("--app <app-id>", "App ID (uses current app if not specified)")
         .option("--test-cases <ids>", "Comma-separated list of test case IDs to run (runs only these)")
         .option("--json", "Output as JSON")
-        .action(async (workflowId, options) => {
+        .action(async (workflow, options) => {
         const resolvedAppId = resolveAppId(undefined, options);
         const client = new ApiClient();
         try {
+            // #3249 — resolve first, so `Total Tests: 0` can only ever mean the
+            // workflow exists and has no cases, never that the identifier was wrong.
+            const workflowId = await resolveWorkflowId(client, resolvedAppId, workflow);
             // Parse test case IDs if provided
             let testCaseIds;
             if (options.testCases) {
@@ -2047,15 +2072,16 @@ Examples:
     tests
         .command("runs")
         .description("List recent test runs for a workflow")
-        .argument("<workflow-id>", "Workflow ID")
+        .argument("<workflow>", "Workflow key or ID")
         .option("--app <app-id>", "App ID (uses current app if not specified)")
         .option("--limit <n>", "Maximum number of runs to show", "20")
         .option("--group <group>", "Filter by comparison group")
         .option("--json", "Output as JSON")
-        .action(async (workflowId, options) => {
+        .action(async (workflow, options) => {
         const resolvedAppId = resolveAppId(undefined, options);
         const client = new ApiClient();
         try {
+            const workflowId = await resolveWorkflowId(client, resolvedAppId, workflow);
             const { items } = await client.listTestRuns(resolvedAppId, "workflow", workflowId, {
                 limit: parseInt(options.limit),
                 comparisonGroup: options.group,
@@ -2097,23 +2123,24 @@ Examples:
         .description("Run tests in parallel using workflow-based execution")
         .addHelpText("after", `
 Examples:
-  $ primitive workflows tests batch start <workflow-id>
-  $ primitive workflows tests batch start <workflow-id> --test-cases 01ABC,01DEF
-  $ primitive workflows tests batch status <workflow-id> <batch-id>
-  $ primitive workflows tests batch cancel <workflow-id> <batch-id>
+  $ primitive workflows tests batch start <workflow>
+  $ primitive workflows tests batch start <workflow> --test-cases 01ABC,01DEF
+  $ primitive workflows tests batch status <workflow> <batch-id>
+  $ primitive workflows tests batch cancel <workflow> <batch-id>
 `);
     // Start batch tests
     batch
         .command("start")
         .description("Start parallel batch test execution")
-        .argument("<workflow-id>", "Workflow ID")
+        .argument("<workflow>", "Workflow key or ID")
         .option("--app <app-id>", "App ID (uses current app if not specified)")
         .option("--test-cases <ids>", "Comma-separated list of test case IDs to run")
         .option("--json", "Output as JSON")
-        .action(async (workflowId, options) => {
+        .action(async (workflow, options) => {
         const resolvedAppId = resolveAppId(undefined, options);
         const client = new ApiClient();
         try {
+            const workflowId = await resolveWorkflowId(client, resolvedAppId, workflow);
             let testCaseIds;
             if (options.testCases) {
                 testCaseIds = options.testCases.split(",").map((id) => id.trim()).filter(Boolean);
@@ -2149,15 +2176,16 @@ Examples:
     batch
         .command("status")
         .description("Get status of a batch test execution")
-        .argument("<workflow-id>", "Workflow ID")
+        .argument("<workflow>", "Workflow key or ID")
         .argument("<batch-id>", "Batch ID")
         .option("--app <app-id>", "App ID (uses current app if not specified)")
         .option("--wait", "Wait for completion (polls every 2 seconds)")
         .option("--json", "Output as JSON")
-        .action(async (workflowId, batchId, options) => {
+        .action(async (workflow, batchId, options) => {
         const resolvedAppId = resolveAppId(undefined, options);
         const client = new ApiClient();
         try {
+            const workflowId = await resolveWorkflowId(client, resolvedAppId, workflow);
             const fetchStatus = async () => {
                 return client.getBatchTestStatus(resolvedAppId, "workflow", workflowId, batchId);
             };
@@ -2214,12 +2242,12 @@ Examples:
     batch
         .command("cancel")
         .description("Cancel a running batch test execution")
-        .argument("<workflow-id>", "Workflow ID")
+        .argument("<workflow>", "Workflow key or ID")
         .argument("<batch-id>", "Batch ID")
         .option("--app <app-id>", "App ID (uses current app if not specified)")
         .option("-y, --yes", "Skip confirmation prompt")
         .option("--json", "Output as JSON")
-        .action(async (workflowId, batchId, options) => {
+        .action(async (workflow, batchId, options) => {
         const resolvedAppId = resolveAppId(undefined, options);
         if (!options.yes) {
             let confirm;
@@ -2237,6 +2265,7 @@ Examples:
         }
         const client = new ApiClient();
         try {
+            const workflowId = await resolveWorkflowId(client, resolvedAppId, workflow);
             // First get current status to get instance IDs
             const status = await client.getBatchTestStatus(resolvedAppId, "workflow", workflowId, batchId);
             // Get instance IDs that are still running
@@ -2276,21 +2305,22 @@ with \`primitive config push\` (#2769); removing one and running
 \`config push --prune\` deletes it server-side. These verbs read server state.
 
 Examples:
-  $ primitive workflows tests attachments list <workflow-id> <test-case-id>
-  $ primitive workflows tests attachments download <workflow-id> <test-case-id> document.pdf
+  $ primitive workflows tests attachments list <workflow> <test-case-id>
+  $ primitive workflows tests attachments download <workflow> <test-case-id> document.pdf
 `);
     // List attachments
     attachments
         .command("list")
         .description("List attachments for a test case")
-        .argument("<workflow-id>", "Workflow ID")
+        .argument("<workflow>", "Workflow key or ID")
         .argument("<test-case-id>", "Test Case ID")
         .option("--app <app-id>", "App ID (uses current app if not specified)")
         .option("--json", "Output as JSON")
-        .action(async (workflowId, testCaseId, options) => {
+        .action(async (workflow, testCaseId, options) => {
         const resolvedAppId = resolveAppId(undefined, options);
         const client = new ApiClient();
         try {
+            const workflowId = await resolveWorkflowId(client, resolvedAppId, workflow);
             const result = await client.listTestCaseAttachments(resolvedAppId, "workflow", workflowId, testCaseId);
             if (options.json) {
                 json(result.attachments);
@@ -2319,15 +2349,16 @@ Examples:
     attachments
         .command("download")
         .description("Download a file attachment from a test case")
-        .argument("<workflow-id>", "Workflow ID")
+        .argument("<workflow>", "Workflow key or ID")
         .argument("<test-case-id>", "Test Case ID")
         .argument("<filename>", "Name of the file to download")
         .argument("[output-path]", "Output path (defaults to current directory)")
         .option("--app <app-id>", "App ID (uses current app if not specified)")
-        .action(async (workflowId, testCaseId, filename, outputPath, options) => {
+        .action(async (workflow, testCaseId, filename, outputPath, options) => {
         const resolvedAppId = resolveAppId(undefined, options);
         const client = new ApiClient();
         try {
+            const workflowId = await resolveWorkflowId(client, resolvedAppId, workflow);
             const result = await client.downloadTestCaseAttachment(resolvedAppId, "workflow", workflowId, testCaseId, filename);
             const finalPath = outputPath || filename;
             writeFileSync(finalPath, result.data);