@pikku/core 0.12.84 → 0.12.88

package/src/wirings/secret/validate-secret-definitions.ts

@@ -54,6 +54,7 @@ export function validateAndBuildSecretDefinitionsMeta(
           description: def.description,
           secretId: def.secretId,
           schema: def.schema,
+          optional: def.optional,
           oauth2: def.oauth2,
           rotationPeriod: def.rotationPeriod,
           allowedHosts: def.allowedHosts,
@@ -73,6 +74,7 @@ export function validateAndBuildSecretDefinitionsMeta(
         description: def.description,
         secretId: def.secretId,
         schema: def.schema,
+        optional: def.optional,
         oauth2: def.oauth2,
         rotationPeriod: def.rotationPeriod,
         allowedHosts: def.allowedHosts,

package/src/wirings/variable/validate-variable-definitions.test.ts

@@ -88,4 +88,28 @@ describe('validateAndBuildVariableDefinitionsMeta', () => {
     const result = validateAndBuildVariableDefinitionsMeta([], new Map())
     assert.deepStrictEqual(result, {})
   })
+
+  test('carries optional into the meta so only the opted-out ones skip the gate', () => {
+    const result = validateAndBuildVariableDefinitionsMeta(
+      [
+        {
+          name: 'corsOrigins',
+          displayName: 'Allowed Browser Origins',
+          variableId: 'CORS_ORIGINS',
+          optional: true,
+          sourceFile: 'a.ts',
+        },
+        {
+          name: 'consoleUrl',
+          displayName: 'Console URL',
+          variableId: 'CONSOLE_URL',
+          sourceFile: 'a.ts',
+        },
+      ] as any,
+      new Map()
+    )
+    assert.strictEqual(result['corsOrigins']!.optional, true)
+    // Undeclared means required, which is what makes the gate ask by default.
+    assert.strictEqual(result['consoleUrl']!.optional, undefined)
+  })
 })

package/src/wirings/variable/validate-variable-definitions.ts

@@ -43,6 +43,7 @@ export function validateAndBuildVariableDefinitionsMeta(
           description: def.description,
           variableId: def.variableId,
           schema: def.schema,
+          optional: def.optional,
           docsUrl: def.docsUrl,
           sourceFile: def.sourceFile,
         }
@@ -59,6 +60,7 @@ export function validateAndBuildVariableDefinitionsMeta(
         description: def.description,
         variableId: def.variableId,
         schema: def.schema,
+        optional: def.optional,
         docsUrl: def.docsUrl,
         sourceFile: def.sourceFile,
       }

package/src/wirings/variable/variable.types.ts

@@ -4,6 +4,20 @@ export type CoreVariable<T = unknown> = {
   description?: string
   variableId: string
   schema: T
+  /**
+   * A variable is REQUIRED by default, and marking it `optional` is how a
+   * declaration says its absence is a supported state. Same flag, same
+   * polarity and same meaning as `CoreSecret.optional` — one word to learn
+   * rather than two with opposite senses.
+   *
+   * Defaulting to required rather than following `variables.get`'s
+   * `T | undefined` return is deliberate. That signature describes what a
+   * caller must HANDLE, not whether a deployment is correct without the value:
+   * an undefined feature flag is fine, an undefined API base URL is an outage
+   * that the type system cannot tell apart. Declaring the difference is the
+   * point of the flag, and the safe default for an undeclared one is to ask.
+   */
+  optional?: boolean
   docsUrl?: string
 }
 
@@ -13,6 +27,7 @@ export type VariableDefinitionMeta = {
   description?: string
   variableId: string
   schema?: Record<string, unknown> | string
+  optional?: boolean
   docsUrl?: string
   sourceFile?: string
 }

package/src/wirings/virtual-user/index.ts

@@ -57,10 +57,7 @@ export {
   type AgentReachability,
   type ReachableAgent,
 } from './virtual-user-agents.js'
-export {
-  IntentStack,
-  intentsForPersona,
-} from './virtual-user-intents.js'
+export { IntentStack, intentsForPersona } from './virtual-user-intents.js'
 export {
   deriveCatalogue,
   deriveIntents,

package/src/wirings/virtual-user/prepare-virtual-user-run.ts

@@ -1,4 +1,4 @@
-import type { FunctionsMeta } from '../../types/core.types.js'
+import type { FunctionsMeta } from '../../function/function-meta.types.js'
 import type { WorkflowsMeta } from '../workflow/workflow.types.js'
 import { flattenSystemRoleDefinitions } from '../role/validate-role-definitions.js'
 import type {

package/src/wirings/virtual-user/virtual-user-derive.test.ts

@@ -3,7 +3,7 @@ import assert from 'node:assert/strict'
 
 import { deriveCatalogue, deriveIntents } from './virtual-user-derive.js'
 import { personaVirtualUserTarget } from './virtual-user-target.js'
-import type { FunctionsMeta } from '../../types/core.types.js'
+import type { FunctionsMeta } from '../../function/function-meta.types.js'
 import type { WorkflowsMeta } from '../workflow/workflow.types.js'
 import type {
   ScenarioPersona,

package/src/wirings/virtual-user/virtual-user-derive.ts

@@ -1,4 +1,4 @@
-import type { FunctionsMeta } from '../../types/core.types.js'
+import type { FunctionsMeta } from '../../function/function-meta.types.js'
 import type { WorkflowsMeta } from '../workflow/workflow.types.js'
 import type { ScenarioStepPhase } from '../workflow/scenario-step.types.js'
 import { composeStepProse } from '../workflow/scenario-prose.js'

package/src/wirings/virtual-user/virtual-user.types.ts

@@ -166,11 +166,7 @@ export type VirtualUserAction =
 
 /** How an intent ended, or that it has not. */
 export type IntentStatus =
-  | 'open'
-  | 'suspended'
-  | 'completed'
-  | 'abandoned'
-  | 'stuck'
+  'open' | 'suspended' | 'completed' | 'abandoned' | 'stuck'
 
 /** One intent's life within a run. */
 export interface IntentRecord {