@ottocode/sdk 0.1.188 → 0.1.189

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@ottocode/sdk",
-	"version": "0.1.188",
+	"version": "0.1.189",
 	"description": "AI agent SDK for building intelligent assistants - tree-shakable and comprehensive",
 	"author": "nitishxyz",
 	"license": "MIT",

package/src/core/src/tools/builtin/bash.ts

@@ -6,13 +6,18 @@ import { createToolError, type ToolResponse } from '../error.ts';
 import { getAugmentedPath } from '../bin-manager.ts';
 
 function normalizePath(p: string) {
-	const parts = p.replace(/\\/g, '/').split('/');
+	const normalized = p.replace(/\\/g, '/');
+	const driveMatch = normalized.match(/^([A-Za-z]):\//);
+	const drivePrefix = driveMatch ? `${driveMatch[1]}:` : '';
+	const rest = driveMatch ? normalized.slice(2) : normalized;
+	const parts = rest.split('/');
 	const stack: string[] = [];
 	for (const part of parts) {
 		if (!part || part === '.') continue;
 		if (part === '..') stack.pop();
 		else stack.push(part);
 	}
+	if (drivePrefix) return `${drivePrefix}/${stack.join('/')}`;
 	return `/${stack.join('/')}`;
 }
 

package/src/core/src/tools/builtin/fs/write.ts

@@ -1,6 +1,7 @@
 import { tool, type Tool } from 'ai';
 import { z } from 'zod/v3';
 import { readFile, writeFile, mkdir } from 'node:fs/promises';
+import { dirname } from 'node:path';
 import {
 	buildWriteArtifact,
 	resolveSafePath,
@@ -68,7 +69,7 @@ export function buildWriteTool(projectRoot: string): {
 
 			try {
 				if (createDirs) {
-					const dirPath = abs.slice(0, abs.lastIndexOf('/'));
+					const dirPath = dirname(abs);
 					await mkdir(dirPath, { recursive: true });
 				}
 				let existed = false;

package/src/core/src/tools/builtin/grep.ts

@@ -110,13 +110,11 @@ export function buildGrepTool(projectRoot: string): {
 
 			for (const line of lines) {
 				if (!line) continue;
-				const idx1 = line.indexOf(':');
-				const idx2 = idx1 === -1 ? -1 : line.indexOf(':', idx1 + 1);
-				if (idx1 === -1 || idx2 === -1) continue;
-				const filePath = line.slice(0, idx1);
-				const lineNumStr = line.slice(idx1 + 1, idx2);
-				const lineText = line.slice(idx2 + 1);
-				const lineNum = parseInt(lineNumStr, 10);
+				const m = line.match(/^(.+?):(\d+):(.*)$/s);
+				if (!m) continue;
+				const filePath = m[1];
+				const lineNum = parseInt(m[2], 10);
+				const lineText = m[3];
 				if (!filePath || !Number.isFinite(lineNum)) continue;
 				matches.push({ file: filePath, line: lineNum, text: lineText });
 			}

package/src/core/src/tools/builtin/ripgrep.ts

@@ -96,11 +96,11 @@ export function buildRipgrepTool(projectRoot: string): {
 							.filter(Boolean)
 							.slice(0, maxResults);
 						const matches = lines.map((l) => {
-							const parts = l.split(':');
-							if (parts.length < 3) return { file: '', line: 0, text: l };
-							const file = parts[0];
-							const line = Number.parseInt(parts[1], 10);
-							const text = parts.slice(2).join(':');
+							const m = l.match(/^(.+?):(\d+):(.*)$/s);
+							if (!m) return { file: '', line: 0, text: l };
+							const file = m[1];
+							const line = Number.parseInt(m[2], 10);
+							const text = m[3];
 							return { file, line, text };
 						});
 						resolve({ ok: true, count: matches.length, matches });

package/src/core/src/tools/builtin/terminal.ts

@@ -18,13 +18,18 @@ function formatShellCommand(parts: string[]): string {
 }
 
 function normalizePath(p: string) {
-	const parts = p.replace(/\\/g, '/').split('/');
+	const normalized = p.replace(/\\/g, '/');
+	const driveMatch = normalized.match(/^([A-Za-z]):\//);
+	const drivePrefix = driveMatch ? `${driveMatch[1]}:` : '';
+	const rest = driveMatch ? normalized.slice(2) : normalized;
+	const parts = rest.split('/');
 	const stack: string[] = [];
 	for (const part of parts) {
 		if (!part || part === '.') continue;
 		if (part === '..') stack.pop();
 		else stack.push(part);
 	}
+	if (drivePrefix) return `${drivePrefix}/${stack.join('/')}`;
 	return `/${stack.join('/')}`;
 }
 
@@ -116,7 +121,10 @@ export function buildTerminalTool(
 
 						const cwd = resolveSafePath(projectRoot, params.cwd);
 
-						const shellPath = process.env.SHELL || '/bin/sh';
+						const shellPath =
+							process.platform === 'win32'
+								? process.env.COMSPEC || 'cmd.exe'
+								: process.env.SHELL || '/bin/sh';
 
 						let command = params.command ?? shellPath;
 						let args = params.args ?? [];
@@ -124,7 +132,7 @@ export function buildTerminalTool(
 
 						if (runInShell) {
 							command = shellPath;
-							args = ['-i'];
+							args = process.platform === 'win32' ? [] : ['-i'];
 							const providedCommand = params.command;
 							const providedArgs = params.args ?? [];
 

package/src/skills/loader.ts

@@ -101,7 +101,7 @@ async function loadSkillsFromDir(
 			const content = await fs.readFile(filePath, 'utf-8');
 			const skill = parseSkillFile(content, filePath, scope);
 
-			const dirName = dirname(filePath).split('/').pop();
+			const dirName = dirname(filePath).split(/[\\/]/).pop();
 			if (dirName !== skill.metadata.name) {
 				if (process.env.OTTO_DEBUG === '1') {
 					console.warn(

package/src/tunnel/tunnel.ts

@@ -210,7 +210,11 @@ export async function killStaleTunnels(): Promise<void> {
 		const execAsync = promisify(exec);
 
 		// Kill any existing tunnel processes (but not the parent otto process)
-		await execAsync('pkill -f "tunnel tunnel --url" 2>/dev/null || true');
+		const killCmd =
+			process.platform === 'win32'
+				? 'taskkill /F /IM tunnel.exe 2>NUL || exit /b 0'
+				: 'pkill -f "tunnel tunnel --url" 2>/dev/null || true';
+		await execAsync(killCmd);
 		// Give processes time to die
 		await new Promise((resolve) => setTimeout(resolve, 500));
 	} catch {