@polderlabs/bizar 10.23.16 → 10.23.17

package/cli/commands/models.mjs

@@ -2325,6 +2325,7 @@ export async function run(name, args, isHelpRequest, deps = {}) {
 
   const wantJson = args.includes('--json');
   const wantList = args.includes('--list');
+  const wantAgentTypes = args.includes('--agent-types');
   const wantClear = args.includes('--clear');
   const wantRefresh = args.includes('--refresh');
   const setFlag = args.find((a) => a.startsWith('--set='));
@@ -2341,6 +2342,19 @@ export async function run(name, args, isHelpRequest, deps = {}) {
   const routerPath = resolveRouterPath(process.cwd());
   const { endpoint, authToken, source: endpointSource } = resolveEndpoint({ cwd: process.cwd() });
 
+  // This is the orchestration-safe bridge from an operator-selected gateway
+  // ID to the Claude Code definition that owns it. It is deliberately local:
+  // no gateway discovery is needed to dispatch a model the operator selected.
+  if (wantAgentTypes) {
+    const router = loadRouter(routerPath);
+    const models = configuredEnabledModels(router);
+    const agentTypes = Object.fromEntries(models.map((id) => [id, modelAgentName(id)]));
+    const payload = { routerPath, models, agentTypes, agentsDir: join(resolveClaudeConfigDir(), 'agents', 'bizar-models') };
+    if (wantJson) process.stdout.write(JSON.stringify(payload, null, 2) + '\n');
+    else for (const id of models) process.stdout.write(`${id}\t${agentTypes[id]}\n`);
+    return true;
+  }
+
   // F-185: `bizar models explain <role>` — non-interactive ranking.
   // Handled BEFORE the picker fetch path so it never touches the gateway.
   const explainIdx = args.findIndex((a) => a === 'explain');

package/cli/provision.mjs

@@ -36,6 +36,7 @@ import {
   buildClaudeModelOverrides,
   configuredEnabledModels,
   requiresGatewayModelDiscovery,
+  syncGeneratedModelAgents,
 } from './commands/models.mjs';
 import { validateNativeWorkflowDirectory } from '../config/workflows/lib/native-contract.mjs';
 
@@ -502,6 +503,20 @@ export async function syncModelRouter({ dryRun = false, force = false } = {}) {
   return { ok: true, message: `global model-router.json → ${dest}`, path: dest };
 }
 
+// Recreate the managed definition projection on every install/update. This is
+// needed after an npm upgrade or a clean Claude directory: the router is
+// operator-owned and preserved, while generated definitions are disposable
+// installation artifacts derived exclusively from its selected IDs.
+export function syncConfiguredModelAgents({ dryRun = false } = {}) {
+  const routerPath = resolveGlobalModelRouter();
+  let router = {};
+  try { if (existsSync(routerPath)) router = JSON.parse(readFileSync(routerPath, 'utf8')); } catch { return { ok: false, message: `cannot read ${routerPath}` }; }
+  const models = configuredEnabledModels(router);
+  if (dryRun) return { ok: true, message: `[dry-run] would sync ${models.length} generated model agent(s)`, models };
+  const generated = syncGeneratedModelAgents(models, { agentsDir: join(resolveClaudeDir(), 'agents', 'bizar-models') });
+  return { ok: true, message: `${generated.names.length} generated model agent(s) synced`, ...generated };
+}
+
 export async function syncSkillFiles({ dryRun = false, force = false } = {}) {
   const src = join(REPO_ROOT, 'config', 'skills');
   const dest = CLAUDE_SKILLS_DIR;
@@ -1187,6 +1202,10 @@ export async function runProvision(opts = {}) {
   if (routerStep.ok) logOk(routerStep.message); else logErr(routerStep.message);
   stepResults.push({ label: 'model-router', ...routerStep });
 
+  const modelAgentsStep = syncConfiguredModelAgents({ dryRun });
+  if (modelAgentsStep.ok) logOk(modelAgentsStep.message); else logErr(modelAgentsStep.message);
+  stepResults.push({ label: 'model-agents', ...modelAgentsStep });
+
   section('Writing settings.json');
   const settingsStep = writeClaudeSettings({ dryRun, force });
   if (settingsStep.ok) logOk(settingsStep.message); else logErr(settingsStep.message);

package/config/claude/agents/office-manager.md

@@ -60,14 +60,19 @@ implementation around a broken workflow installation.
 
 For every dispatch, select the cheapest sufficient enabled configured model
 from the global Bizar router. User-selected models take precedence over tier
-candidates; `disabledProviders` excludes both. Native aliases are transport
-labels bound by `bizar models`, not Anthropic selections. Use the full selected
-model ID directly for native Agents and teams; every enabled selection is
-eligible. Use `bizar worker` only when a separately launched process worktree
-is useful. If no enabled configured candidate exists, stop with the
-configuration error. Never let Claude choose an unconfigured default, inherit
-the session model, use an unmapped alias,
-or retry by cycling models, providers, or tiers.
+candidates; `disabledProviders` excludes both. Native aliases are compatibility
+transport labels bound by `bizar models`, not Anthropic selections. Native
+Agent/Task calls accept only those aliases, so do not pass a full gateway ID in
+`model`. Run `bizar models --agent-types --json`, look up the selected ID, pass
+its generated name as `subagent_type`, and omit `model`; the generated global
+definition's `model:` frontmatter selects the full gateway ID. This is the
+default for individual subagents, workflows, and agent-team teammates. For
+teams, do not name a competing model in the spawn prompt. Use `bizar worker`
+only when a separately launched process worktree is useful. If no enabled
+configured candidate or generated definition exists, stop with the
+configuration error and run `bizar models`. Never let Claude choose an
+unconfigured default, inherit the session model, use an unmapped alias, or
+retry by cycling models, providers, or tiers.
 
 ## Worktree Discipline and integration
 

package/config/claude/hooks/agent-model-guard.mjs

@@ -248,11 +248,11 @@ export async function guardAgentModel(input, options = {}) {
     return {};
   }
 
-  // Claude Code accepts full model IDs for native Agent calls. A model picked
-  // through `bizar models` is therefore valid directly; do not collapse every
-  // user selection into the four family aliases.
-  if (userPicks.has(requested) && !hasFailoverContract) {
-    return {};
+  // Native Agent/Task transport accepts only Claude's four aliases, even when
+  // a gateway accepts arbitrary IDs for the main conversation. Full IDs belong
+  // in Bizar's generated subagent frontmatter, not in this enum-shaped field.
+  if (userPicks.has(requested)) {
+    return deny(`Bizar Agent dispatch blocked: native Agent model accepts aliases only. Use generated subagent_type ${modelAgentName(requested)} and omit model; re-run \`bizar models\` if it is missing.`);
   }
 
   // F-185 contract: when the orchestrator passes both `routingDecisionId`

package/config/claude/hooks/worker-suggest.mjs

@@ -60,7 +60,7 @@ const ROUTE_POLICY = [
   'Adaptive Bizar routing policy:',
   '- If this is the primary session, you ARE @mike. First do only bounded read-only orientation. Then ask one concise clarification question describing the inferred outcome, the material choice/risk, and your proposed coordination mode. Wait for the answer before edits, branches, tests, or editor dispatch. If the user explicitly waives questions, continue autonomously.',
   '- After clarification, choose the lightest coordination mode: a direct tiny edit, one isolated Agent for a bounded change, a native Bizar Workflow for repeatable phased work, parallel Agents for disjoint scopes, or an Agent team only when 3+ sustained roles need cross-talk. Do not force a workflow or team when it adds no value.',
-  '- Every Agent or team member receives one exact raw model ID from the enabled `bizar models` user selection. Pass that ID directly as `model`; never substitute a Claude family alias, provider default, or inherited session model. The guard allows only IDs in the global user-selected pool. If a selected custom model is denied, inspect `~/.claude/model-router.json`; do not retry with another model. Every editing worker uses call-level `isolation: "worktree"`. For genuinely disjoint writable scopes, dispatch concurrently; otherwise use one owner.',
+  '- For every Agent, workflow worker, or agent-team teammate, choose one exact ID from the enabled `bizar models` user selection, then obtain its generated definition name from `bizar models --agent-types --json`. Pass that name as `subagent_type` and OMIT the native `model` field. The definition frontmatter owns the exact gateway ID. Never put a raw gateway ID or a Claude family alias in the native model field; aliases are compatibility-only. For teams, do not name a competing model in the spawn prompt. If the map or definition is missing, run `bizar models`; do not retry by cycling providers or tiers. Every editing worker uses call-level `isolation: "worktree"`. For genuinely disjoint writable scopes, dispatch concurrently; otherwise use one owner.',
   '- Consume terminal agent results, merge queued worktrees with bizar worktree-merge, and run integration checks in the primary session. A subagent may not recursively dispatch itself.',
   '- Do NOT execute any tool you do not have. If a tool you need is missing from your tools list, dispatch to a subagent that has it — do not pretend you have it.',
   '- If you are already running as a Bizar custom agent, follow your assigned role and do not recursively dispatch yourself.',

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@polderlabs/bizar",
-  "version": "10.23.16",
+  "version": "10.23.17",
   "description": "Autonomous, human-in-the-loop multi-agent harness for Claude Code with guarded workflows, typed SDK primitives, and MCP tools.",
   "type": "module",
   "bin": {