@theokit/sdk 4.5.1 → 4.6.0

package/dist/eval.js

@@ -6702,22 +6702,84 @@ async function readProjectSubagents(cwd) {
   }
   return subagents;
 }
+var ACCEPTED_FIELDS = /* @__PURE__ */ new Set([
+  "name",
+  "description",
+  "model",
+  "tools",
+  "reasoning_effort",
+  "mcp",
+  "sandbox"
+]);
 function parseSubagentMarkdown(raw, filename) {
   const { frontmatter, body } = splitFrontmatter(raw, filename);
   const fields = parseFrontmatterFields(frontmatter);
-  const baseName = filename.replace(/\.md$/, "");
-  const name = fields.name ?? baseName;
+  rejectUnknownFields(fields, filename);
+  rejectMcp(fields, filename);
   const definition = {
-    description: fields.description ?? "",
+    description: asString(fields.description) ?? "",
     prompt: body
   };
-  if (fields.model !== void 0) {
-    definition.model = fields.model === "inherit" ? "inherit" : { id: fields.model };
-  }
-  const tools = fields.tools?.split(/[\s,]+/).map((t) => t.trim()).filter((t) => t.length > 0);
-  if (tools !== void 0 && tools.length > 0) definition.tools = tools;
+  const model = resolveModel(fields, filename);
+  if (model !== void 0) definition.model = model;
+  const tools = toStringList(fields.tools);
+  if (tools.length > 0) definition.tools = tools;
+  const sandbox = resolveSandbox(fields, filename);
+  if (sandbox !== void 0) definition.sandbox = sandbox;
+  const name = asString(fields.name) ?? filename.replace(/\.md$/, "");
   return { name, definition };
 }
+function rejectUnknownFields(fields, filename) {
+  for (const key of Object.keys(fields)) {
+    if (!ACCEPTED_FIELDS.has(key)) {
+      throw new ConfigurationError(
+        `Subagent ${filename}: unknown frontmatter field "${key}" (accepted: ${[...ACCEPTED_FIELDS].join(", ")})`,
+        { code: "subagent_unknown_field" }
+      );
+    }
+  }
+}
+function rejectMcp(fields, filename) {
+  if (fields.mcp !== void 0) {
+    throw new ConfigurationError(
+      `Subagent ${filename}: per-subagent "mcp" is not yet supported on the local delegation path; declare MCP servers in .theokit/mcp.json (or the parent) instead`,
+      { code: "subagent_mcp_unsupported_local" }
+    );
+  }
+}
+function resolveModel(fields, filename) {
+  const modelId = asString(fields.model);
+  const effort = asString(fields.reasoning_effort);
+  if (effort !== void 0 && modelId === void 0) {
+    throw new ConfigurationError(
+      `Subagent ${filename}: reasoning_effort requires a model (effort is a model parameter)`,
+      { code: "subagent_reasoning_effort_without_model" }
+    );
+  }
+  if (modelId === void 0) return void 0;
+  if (modelId === "inherit") return "inherit";
+  return effort !== void 0 ? { id: modelId, params: [{ id: "thinking", value: effort }] } : { id: modelId };
+}
+function resolveSandbox(fields, filename) {
+  if (fields.sandbox === void 0) return void 0;
+  if (typeof fields.sandbox !== "boolean") {
+    throw new ConfigurationError(
+      `Subagent ${filename}: sandbox must be a boolean (got "${String(fields.sandbox)}"); granular sandbox modes are not supported by the runtime`,
+      { code: "subagent_sandbox_not_boolean" }
+    );
+  }
+  return fields.sandbox;
+}
+function asString(v) {
+  return typeof v === "string" ? v : void 0;
+}
+function toStringList(v) {
+  if (Array.isArray(v)) return v.map((t) => t.trim()).filter((t) => t.length > 0);
+  if (typeof v === "string") {
+    return v.split(/[\s,]+/).map((t) => t.trim()).filter((t) => t.length > 0);
+  }
+  return [];
+}
 function splitFrontmatter(raw, filename) {
   const match = /^---\s*\n([\s\S]*?)\n---\s*\n([\s\S]*)$/.exec(raw);
   if (match === null) {
@@ -6728,12 +6790,7 @@ function splitFrontmatter(raw, filename) {
   return { frontmatter: match[1] ?? "", body: (match[2] ?? "").trim() };
 }
 function parseFrontmatterFields(frontmatter) {
-  const raw = parseSimpleYaml(frontmatter);
-  const out = {};
-  for (const [k, v] of Object.entries(raw)) {
-    out[k] = typeof v === "string" ? v : void 0;
-  }
-  return out;
+  return parseSimpleYaml(frontmatter);
 }
 
 // src/define-tool.ts
@@ -6931,12 +6988,12 @@ init_errors();
 
 // src/internal/runtime/skills/skill-frontmatter.ts
 init_errors();
-function asString(v) {
+function asString2(v) {
   return typeof v === "string" ? v : void 0;
 }
 function toStringFields(raw) {
   const out = {};
-  for (const [k, v] of Object.entries(raw)) out[k] = asString(v);
+  for (const [k, v] of Object.entries(raw)) out[k] = asString2(v);
   return out;
 }
 function parseSkillFrontmatter(raw, fallbackName) {
@@ -13800,10 +13857,11 @@ ${lines.join("\n")}
 </subagent-tool-results>`;
 }
 function buildChildCreateOptions(spec, inherited) {
-  const model = spec.model ? { id: spec.model } : inherited?.model;
+  const model = spec.model !== void 0 ? typeof spec.model === "string" ? { id: spec.model } : spec.model : inherited?.model;
   return {
     ...inherited?.apiKey !== void 0 ? { apiKey: inherited.apiKey } : {},
     ...model !== void 0 ? { model } : {},
+    ...spec.sandbox === true ? { local: { sandboxOptions: { enabled: true } } } : {},
     ...inherited?.plugins !== void 0 ? { plugins: inherited.plugins } : {},
     systemPrompt: spec.instructions,
     tools: spec.tools ?? []
@@ -13912,7 +13970,10 @@ function subAgentToolsFromDefinitions(agents2, parentTools) {
       name,
       description: def.description,
       instructions: def.prompt,
-      ...def.model !== void 0 && def.model !== "inherit" ? { model: def.model.id } : {},
+      // Carry the FULL ModelSelection (id + reasoning params), not just the id, so
+      // per-subagent reasoning effort survives to buildChildCreateOptions.
+      ...def.model !== void 0 && def.model !== "inherit" ? { model: def.model } : {},
+      ...def.sandbox !== void 0 ? { sandbox: def.sandbox } : {},
       tools: [...childTools]
     });
   });