@namzu/sdk 45.1.0 → 48.0.0

package/src/tools/builtins/edit.ts

@@ -8,6 +8,7 @@ import { atomicWriteFile } from './atomic-write-file.js'
 import { fingerprintContent, staleFileError } from './content-fingerprint.js'
 import { applyEdit, normalizeEditInput } from './edit-apply.js'
 import { withFileMutationLock } from './file-mutation-lock.js'
+import { jsonStringEscapes } from './json-string-hint.js'
 
 /**
  * Two schemas, on purpose.
@@ -278,6 +279,9 @@ function recordDrift(context: ToolContext, key: string): void {
 	context.fileReadTracker?.recordDriftObserved?.(key)
 }
 
+const EDIT_SHAPES =
+	'Three shapes. Replace: {"path":"file.md","old_string":"exact unique text","new_string":"replacement text"} (optional "replace_all": true). Insert: {"path":"file.md","insertLine":"end","new_string":"text to add"} where insertLine is a non-negative line number or "end". Batch: {"path":"file.md","edits":[{"old_string":"a","new_string":"b"},{"old_string":"c","new_string":"d"}]} applied in order, all or nothing. Exactly one of old_string, insertLine or edits — a call carrying more than one of them is refused rather than resolved.'
+
 export const EditTool = defineTool({
 	name: 'edit',
 	description:
@@ -285,8 +289,14 @@ export const EditTool = defineTool({
 	inputSchema,
 	modelInputSchema,
 	enforceModelInput: true,
-	validationErrorHint:
-		'Three shapes. Replace: {"path":"file.md","old_string":"exact unique text","new_string":"replacement text"} (optional "replace_all": true). Insert: {"path":"file.md","insertLine":"end","new_string":"text to add"} where insertLine is a non-negative line number or "end". Batch: {"path":"file.md","edits":[{"old_string":"a","new_string":"b"},{"old_string":"c","new_string":"d"}]} applied in order, all or nothing. Exactly one of old_string, insertLine or edits — a call carrying more than one of them is refused rather than resolved.',
+	validationErrorHint: EDIT_SHAPES,
+	// Replacement text is where arguments stop being JSON: a raw newline,
+	// quote or backslash copied into the string. The shapes follow, since a
+	// malformed call may equally have got those wrong.
+	malformedInputHint: `${jsonStringEscapes('"old_string" and "new_string", in "edits" too,')} ${EDIT_SHAPES}`,
+	largeStringArguments: { old_string: 12_000, new_string: 12_000 },
+	truncatedInputHint:
+		'Split a large change into several edit calls, each replacing a shorter unique span; to add a long section, append it in parts with insertLine: "end".',
 	category: 'filesystem',
 	// Declared so a path outside the turn's roots can be reviewed before the
 	// call runs, rather than refused after. See `ToolDefinition.pathArgument`.

package/src/tools/builtins/index.ts

@@ -7,7 +7,13 @@ export { LSP_TOOL_NAME, LspTool, getCodeNavigationTools } from './lsp.js'
 export { GrepTool } from './grep.js'
 export { JobTool } from './job.js'
 export { WaitForJobTool } from './wait-for-job.js'
-export { SKILL_TOOL_NAME, SkillTool, parseAllowedTools } from './skill.js'
+export { SKILL_TOOL_NAME, SkillTool, createSkillTool, parseAllowedTools } from './skill.js'
+export type {
+	SkillDirectoryContext,
+	SkillDirectoryRequest,
+	SkillDirectoryResolver,
+	SkillToolOptions,
+} from './skill.js'
 export {
 	WEB_FETCH_TOOL_NAME,
 	WEB_SEARCH_TOOL_NAME,

package/src/tools/builtins/json-string-hint.ts

@@ -0,0 +1,16 @@
+/**
+ * The advice a tool whose arguments carry raw text gives a model whose call
+ * was malformed.
+ *
+ * A file body, a replacement or a shell command is copied into a JSON string
+ * as it stands, and the characters JSON does not allow there unescaped (a
+ * newline, a tab, a double quote, a backslash) are what make such a call
+ * unreadable. The tool's shape, which is all a validation hint gives, does
+ * not say that.
+ *
+ * @param where - The arguments the text goes into, as the model sees them,
+ * e.g. `"content"`.
+ */
+export function jsonStringEscapes(where: string): string {
+	return `Every character of ${where} goes inside a JSON string: write a newline as \\n, a tab as \\t, a double quote as \\" and a backslash as \\\\.`
+}

package/src/tools/builtins/search-tools.ts

@@ -40,39 +40,21 @@ export const SearchToolsTool = defineTool({
 
 		const allowed = context.allowedTools !== undefined ? new Set(context.allowedTools) : null
 		// `searchDeferred` returns a ranked list (score-descending), so slicing
-		// the head is a true top-k activation, not an arbitrary subset.
+		// the head is a true top-k reveal, not an arbitrary subset.
 		const ranked = context.toolRegistry
 			.searchDeferred(input.query)
 			.filter((tool) => !allowed || allowed.has(tool.name))
 
 		if (ranked.length === 0) {
-			const active = context.toolRegistry
-				.searchActive?.(input.query)
-				.filter(
-					(tool) =>
-						(!allowed || allowed.has(tool.name)) &&
-						context.toolRegistry?.getAvailability(tool.name) === 'active',
-				)
-			const activeReceipt =
-				active === undefined
-					? 'This registry cannot search active tools.'
-					: active.length === 0
-						? 'No matching active tools were found.'
-						: `Already active matching tools (up to ${ACTIVATION_TOP_K}):\n${active
-								.slice(0, ACTIVATION_TOP_K)
-								.map((tool) => `- ${tool.name}`)
-								.join('\n')}`
 			return {
 				success: true,
-				output: `No deferred tools matching "${input.query}". ${activeReceipt}`,
+				output: `No deferred tools matching "${input.query}".`,
 			}
 		}
 
 		const activated = ranked.slice(0, ACTIVATION_TOP_K)
 		const nearMisses = ranked.slice(ACTIVATION_TOP_K, ACTIVATION_TOP_K + NEAR_MISS_LIMIT)
 
-		context.toolRegistry.activate(activated.map((t) => t.name))
-
 		const descriptions = activated.map((t) => `- ${t.name}: ${t.description}`).join('\n')
 		const sections = [`Activated ${activated.length} tool(s):\n${descriptions}`]
 
@@ -91,6 +73,11 @@ export const SearchToolsTool = defineTool({
 		return {
 			success: true,
 			output: sections.join('\n\n'),
+			// No `activate()` call: the manager derives availability from what a
+			// tool message REVEALS (`ToolResult.reveals`, persisted as
+			// `ToolMessage.revealedTools`), not from a mutation made here. See
+			// `toolsets/manager.ts`'s `availability()`.
+			reveals: activated.map((t) => t.name),
 			data: {
 				activated: activated.map((t) => t.name),
 				count: activated.length,