refactor(workflow): name live provider availability accurately

Waishnav · Waishnav · commit c1e4bee5d886 · 2026-07-27T12:05:44.000Z
diff --git a/skills/dynamic-workflows/SKILL.md b/skills/dynamic-workflows/SKILL.md
@@ -91,7 +91,7 @@ profile supplies instructions, provider, model, and effort defaults; per-call
 mutually exclusive.
 
 Without a profile, default provider resolution is `opts.provider` →
-`meta.defaultProvider` → first **enabled ∩ available** provider.
+`meta.defaultProvider` → first currently available provider.
 
 ### Resume
 
diff --git a/src/cli.ts b/src/cli.ts
@@ -70,6 +70,11 @@ async function main(argv: string[]): Promise<void> {
       runConfigCommand(args);
       return;
     case "agents":
+      if (!loadConfig().subagents) {
+        throw new Error(
+          "Subagents are disabled. Set DEVSPACE_SUBAGENTS=1 to enable the experimental feature.",
+        );
+      }
       await runAgentsCommand(args);
       return;
     case "workflow":
diff --git a/src/workflow-api.ts b/src/workflow-api.ts
@@ -158,9 +158,9 @@ export interface WorkflowApiDeps {
   signal: AbortSignal;
   workspaceRoot: string;
   baseSha?: string;
-  /** Already-filtered enabled ∩ live provider ids, preference order. */
-  enabledProviders: LocalAgentProvider[];
-  /** Loaded, enabled profiles exposed by open_workspace for this project. */
+  /** Currently available provider ids in stable preference order. */
+  availableProviders: LocalAgentProvider[];
+  /** Loaded profiles available to this project. */
   agentProfiles?: LocalAgentProfile[];
   runProvider: WorkflowRunProvider;
   createWorktree?: CreateAgentWorktree;
@@ -186,7 +186,6 @@ export class WorkflowEngineError extends Error {
   constructor(
     readonly kind:
       | "cancelled"
-      | "provider_disabled"
       | "provider_unavailable"
       | "no_provider"
       | "profile"
@@ -711,7 +710,7 @@ interface ResolvedAgentTarget {
 function resolveAgentTarget(
   prompt: string,
   opts: AgentOpts,
-  deps: Pick<WorkflowApiDeps, "agentProfiles" | "enabledProviders" | "meta">,
+  deps: Pick<WorkflowApiDeps, "agentProfiles" | "availableProviders" | "meta">,
 ): ResolvedAgentTarget {
   try {
     const resolved = resolveLocalAgentExecution({
@@ -722,7 +721,7 @@ function resolveAgentTarget(
       model: opts.model,
       effort: opts.effort,
       profiles: deps.agentProfiles ?? [],
-      availableProviders: deps.enabledProviders,
+      availableProviders: deps.availableProviders,
     });
     return {
       provider: resolved.provider,
diff --git a/src/workflow-cli.ts b/src/workflow-cli.ts
@@ -352,7 +352,7 @@ export async function runWorkflowWorker(
   try {
     const source = await readFile(claimed.scriptPath, "utf8");
     const parsed = parseWorkflowScript(source, { filename: claimed.scriptPath });
-    const enabledProviders = resolveEnabledProviders();
+    const availableProviders = resolveAvailableProviders();
     const agentProfiles = await loadLocalAgentProfiles(config, claimed.workspaceRoot);
     const concurrency = resolveWorkflowConcurrency(
       parsed.meta.concurrency,
@@ -385,7 +385,7 @@ export async function runWorkflowWorker(
       signal: abort.signal,
       workspaceRoot: claimed.workspaceRoot,
       baseSha: claimed.baseSha,
-      enabledProviders,
+      availableProviders,
       agentProfiles,
       createWorktree,
       replay,
@@ -600,7 +600,7 @@ function safeParseJson(text: string): unknown {
   }
 }
 
-function resolveEnabledProviders(): LocalAgentProvider[] {
+function resolveAvailableProviders(): LocalAgentProvider[] {
   const snapshot = getLocalAgentProviderAvailabilitySnapshot();
   const live = new Set(snapshot.filter((row) => row.available).map((row) => row.name));
   return LOCAL_AGENT_PROVIDERS.filter((id) => live.has(id));
diff --git a/src/workflow-contracts.ts b/src/workflow-contracts.ts
@@ -121,7 +121,6 @@ export const workflowErrorKindSchema = z.enum([
   "syntax",
   "meta",
   "determinism",
-  "provider_disabled",
   "provider_unavailable",
   "no_provider",
   "provider",
diff --git a/src/workflow-engine.test.ts b/src/workflow-engine.test.ts
@@ -57,7 +57,7 @@ import type { LocalAgentProfile } from "./local-agent-profiles.js";
     concurrency: 4,
     signal: new AbortController().signal,
     workspaceRoot: dir,
-    enabledProviders: ["codex"],
+    availableProviders: ["codex"],
     runProvider: async (input) => {
       order.push(`start:${input.prompt}`);
       await new Promise((r) => setTimeout(r, 10));
@@ -102,7 +102,7 @@ import type { LocalAgentProfile } from "./local-agent-profiles.js";
     concurrency: 4,
     signal: new AbortController().signal,
     workspaceRoot: dir,
-    enabledProviders: ["codex"],
+    availableProviders: ["codex"],
     runProvider: async () => ({ finalResponse: "x" }),
   });
 
@@ -165,7 +165,7 @@ import type { LocalAgentProfile } from "./local-agent-profiles.js";
     concurrency: 4,
     signal: new AbortController().signal,
     workspaceRoot: dir,
-    enabledProviders: ["codex"],
+    availableProviders: ["codex"],
     runProvider: async (input: WorkflowProviderRunInput) => {
       seen.push({ prompt: input.prompt, phase: input.phase });
       await new Promise((r) => setTimeout(r, 15));
@@ -226,7 +226,7 @@ import type { LocalAgentProfile } from "./local-agent-profiles.js";
     concurrency: 2,
     signal: new AbortController().signal,
     workspaceRoot: dir,
-    enabledProviders: ["codex"],
+    availableProviders: ["codex"],
     createWorktree,
     runProvider: async (input) => {
       assert.equal(input.workspace, worktrees[0]);
@@ -265,7 +265,7 @@ import type { LocalAgentProfile } from "./local-agent-profiles.js";
     concurrency: 1,
     signal: new AbortController().signal,
     workspaceRoot: dir,
-    enabledProviders: ["codex"],
+    availableProviders: ["codex"],
     createWorktree: async () => {
       throw new Error("expected worktree setup failure");
     },
@@ -312,7 +312,7 @@ import type { LocalAgentProfile } from "./local-agent-profiles.js";
     concurrency: 1,
     signal: new AbortController().signal,
     workspaceRoot: dir,
-    enabledProviders: ["codex", "claude"],
+    availableProviders: ["codex", "claude"],
     runProvider: async (input) => {
       used.push(input.provider);
       return { finalResponse: input.provider };
@@ -360,7 +360,7 @@ import type { LocalAgentProfile } from "./local-agent-profiles.js";
     concurrency: 1,
     signal: new AbortController().signal,
     workspaceRoot: dir,
-    enabledProviders: ["codex", "claude"],
+    availableProviders: ["codex", "claude"],
     agentProfiles: [profile],
     runProvider: async (input) => {
       calls.push(input);
@@ -401,7 +401,7 @@ import type { LocalAgentProfile } from "./local-agent-profiles.js";
     concurrency: 1,
     signal: new AbortController().signal,
     workspaceRoot: dir,
-    enabledProviders: ["codex"],
+    availableProviders: ["codex"],
     agentProfiles: [profile],
     runProvider: async () => ({ finalResponse: "unreachable" }),
   });
@@ -440,7 +440,7 @@ import type { LocalAgentProfile } from "./local-agent-profiles.js";
     concurrency: 1,
     signal: new AbortController().signal,
     workspaceRoot: dir,
-    enabledProviders: ["codex"],
+    availableProviders: ["codex"],
     runProvider: async (input) => {
       calls.push(input);
       if (calls.length === 1) {
@@ -494,7 +494,7 @@ import type { LocalAgentProfile } from "./local-agent-profiles.js";
     concurrency: 1,
     signal: new AbortController().signal,
     workspaceRoot: dir,
-    enabledProviders: ["codex"],
+    availableProviders: ["codex"],
     runProvider: async () => ({ finalResponse: response }),
   });
 
@@ -539,7 +539,7 @@ return { a, nested }
     runId: run.id,
     journal: store,
     workspaceRoot: dir,
-    enabledProviders: ["codex", "claude"],
+    availableProviders: ["codex", "claude"],
     runProvider: async (input) => {
       prompts.push(input.prompt);
       providers.push(input.provider);
@@ -576,7 +576,7 @@ return await workflow({ scriptPath: ${JSON.stringify(childPath)} }).then(async (
         runId: run.id,
         journal: store,
         workspaceRoot: dir,
-        enabledProviders: ["codex"],
+        availableProviders: ["codex"],
         runProvider: async () => ({ finalResponse: "x" }),
         resolveNestedSource: async () => `
 export const meta = { name: 'mid', description: 'm' }
@@ -613,7 +613,7 @@ return await workflow({ scriptPath: 'x' })
     concurrency: 1,
     signal: ac.signal,
     workspaceRoot: dir,
-    enabledProviders: ["codex"],
+    availableProviders: ["codex"],
     runProvider: async () => {
       ac.abort();
       return { finalResponse: "late" };
diff --git a/src/workflow-engine.ts b/src/workflow-engine.ts
@@ -40,7 +40,7 @@ export interface ExecuteWorkflowOptions {
   signal?: AbortSignal;
   workspaceRoot: string;
   baseSha?: string;
-  enabledProviders: LocalAgentProvider[];
+  availableProviders: LocalAgentProvider[];
   agentProfiles?: LocalAgentProfile[];
   runProvider: WorkflowRunProvider;
   createWorktree?: CreateAgentWorktree;
@@ -89,7 +89,7 @@ export async function executeWorkflow(
     signal,
     workspaceRoot: options.workspaceRoot,
     baseSha: options.baseSha,
-    enabledProviders: options.enabledProviders,
+    availableProviders: options.availableProviders,
     agentProfiles: options.agentProfiles,
     runProvider: options.runProvider,
     createWorktree: options.createWorktree,
