@theokit/sdk 5.2.0 → 5.2.1

package/CHANGELOG.md

@@ -1,5 +1,39 @@
 # Changelog
 
+## 5.2.1
+
+### Patch Changes
+
+- [#594](https://github.com/usetheokit/theokit-sdk/pull/594) [`9dc8cf5`](https://github.com/usetheokit/theokit-sdk/commit/9dc8cf52bee690bfb9cf55bee63bcda71d3132b3) Thanks [@usetheodev](https://github.com/usetheodev)! - `effectiveToolNames` refuses a created tool instead of answering confidently about it ([#583](https://github.com/usetheokit/theokit-sdk/issues/583) follow-up)
+  
+  Every field on `AgentOptions` is optional, so the `CustomTool` that `SubAgent.create()` returns —
+  `{ name, description, inputSchema, handler }` — satisfies the type **by vacuity, with no cast**.
+  Measured: a `@ts-expect-error` on that call is reported unused (`TS2578`), so the compiler genuinely
+  does not refuse it.
+  
+  What came back was worse than a wrong number:
+  
+  ```ts
+  effectiveToolNames(SubAgent.create({ name: "analyst", … }))
+  // { names: ["shell"], unresolved: [] }
+  ```
+  
+  The empty `unresolved` claims **completeness** about an object the function never understood. A
+  caller reads that as *"this subagent still announces a shell"* and either disbelieves a fix that
+  worked, or "fixes" something on the strength of it. That is the defect [#583](https://github.com/usetheokit/theokit-sdk/issues/583) exists to eliminate, one
+  function further on — and the same argument that made this return `{ names, unresolved }` rather than
+  a bare array forbids it.
+  
+  It now throws a `ConfigurationError` naming what to pass instead. The detection is exact rather than
+  heuristic: `AgentOptions` declares neither `handler` nor `inputSchema`, so an object carrying **both**
+  is a tool and not options.
+  
+  **The workaround is still needed, and the 5.2.0 notes should not have implied otherwise.**
+  `SubAgent.create()` closes its spec inside the handler, so the spec cannot be recovered from the
+  returned tool — keep it in a variable, or extract it into a function a test can call, and pass that.
+  
+  Reported by the `theocode` session, against advice of mine that was wrong.
+
 ## 5.2.0
 
 ### Minor Changes

package/dist/index.cjs

@@ -919,8 +919,17 @@ var NoopMemoryProvider = class {
 var ALWAYS_REGISTERED = ["shell"];
 var MEMORY_BUILTINS = ["memory_search", "memory_get"];
 function effectiveToolNames(options) {
+  refuseCreatedTool(options);
   return { names: resolvableNames(options), unresolved: unresolvableSources(options) };
 }
+function refuseCreatedTool(options) {
+  const candidate = options;
+  if (candidate.handler === void 0 || candidate.inputSchema === void 0) return;
+  throw new chunkJ7J7J2GN_cjs.ConfigurationError(
+    `effectiveToolNames received a created tool${typeof candidate.name === "string" ? ` ("${candidate.name}")` : ""}, not AgentOptions. \`SubAgent.create()\` returns a CustomTool and closes its spec inside the handler, so the spec cannot be recovered from it. Pass the spec you built it from \u2014 keep it in a variable, or extract it into a function the test can call.`,
+    { code: "effective_tools_expected_options" }
+  );
+}
 function resolvableNames(options) {
   const withheld = new Set(options.withheldBuiltinTools ?? []);
   const builtins = [