feat(factory): add --yes confirmation guard for destructive commands

.github/workflows/ci.yml

@@ -22,6 +22,9 @@ jobs:
         os: [ubuntu-latest, macos-latest, windows-latest]
         node-version: [22.x, 24.x, 25.x]
     runs-on: ${{ matrix.os }}
+    # Baseline job time is 1-3 min; cap at 15 to fail fast on a hang instead
+    # of burning hours and losing logs to blob storage expiry.
+    timeout-minutes: 15
     # Per-endpoint Zod schemas (#171) that once bloated .d.ts emit are gone,
     # but tsc build still peaks around 3.7 GB locally; keep headroom above
     # the 2 GB Node default for slower/smaller CI runners.

codegen/functional/mapper.ts

@@ -6,6 +6,7 @@
 import type { EsApiDefinition } from '../../src/es/types.ts'
 import { extractSchemaArgs } from '../../src/lib/json-schema-args.ts'
 import type { SchemaArgDefinition } from '../../src/lib/json-schema-args.ts'
+import { inferIntentFromHttp } from '@cli-schema/spec'
 
 /**
  * Result of mapping a YAML dot-notation action to a CLI command.
@@ -62,6 +63,12 @@ export function mapAction (
   if (def.namespace != null) args.push(def.namespace)
   args.push(def.name)
 
+  // Destructive commands prompt for confirmation; test scripts run non-interactively.
+  const intent = def.intent ?? inferIntentFromHttp(def.method)
+  if (intent?.destructive === true || intent?.requiresConfirmation === true) {
+    args.push('--yes')
+  }
+
   const schemaArgs = def.input != null ? extractSchemaArgs(def.input) : []
 
   const bodyFields = new Set(

codegen/functional/test/mapper.test.ts

@@ -45,6 +45,20 @@ const testDefs: EsApiDefinition[] = [
     description: 'Get cluster info',
     method: 'GET',
     path: '/'
+  },
+  {
+    name: 'delete',
+    namespace: 'indices',
+    description: 'Delete an index',
+    method: 'DELETE',
+    path: '/{index}',
+    input: {
+      type: 'object',
+      properties: {
+        index: { type: 'string', 'x-found-in': 'path' },
+      },
+      required: ['index'],
+    }
   }
 ]
 
@@ -107,4 +121,16 @@ describe('mapAction', () => {
     assert.ok(result)
     assert.ok(result.cliArgs.includes('--wait-for-active-shards'))
   })
+
+  it('appends --yes for a DELETE action so non-interactive test runs do not prompt', () => {
+    const result = mapAction('indices.delete', { index: 'test' }, actionMap)
+    assert.ok(result)
+    assert.deepStrictEqual(result.cliArgs, ['stack', 'es', 'indices', 'delete', '--yes', '--index', 'test'])
+  })
+
+  it('does not append --yes for a non-destructive action', () => {
+    const result = mapAction('indices.create', { index: 'test' }, actionMap)
+    assert.ok(result)
+    assert.ok(!result.cliArgs.includes('--yes'))
+  })
 })