@ottocode/server 0.1.242 → 0.1.244

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@ottocode/server",
-	"version": "0.1.242",
+	"version": "0.1.244",
 	"description": "HTTP API server for ottocode",
 	"type": "module",
 	"main": "./src/index.ts",
@@ -49,8 +49,8 @@
 		"typecheck": "tsc --noEmit"
 	},
 	"dependencies": {
-		"@ottocode/sdk": "0.1.242",
-		"@ottocode/database": "0.1.242",
+		"@ottocode/sdk": "0.1.244",
+		"@ottocode/database": "0.1.244",
 		"drizzle-orm": "^0.44.5",
 		"hono": "^4.9.9",
 		"zod": "^4.3.6"

package/src/routes/skills.ts

@@ -17,8 +17,18 @@ export function registerSkillsRoutes(app: Hono) {
 			const projectRoot = c.req.query('project') || process.cwd();
 			const repoRoot = (await findGitRoot(projectRoot)) ?? projectRoot;
 			const skills = await discoverSkills(projectRoot, repoRoot);
+			// Dedupe by name (same skill may exist in multiple source dirs like
+			// ~/.claude/skills and ~/.codex/skills). `discoverSkills` already
+			// dedupes via its internal Map, but be defensive here for UI consistency.
+			const seen = new Set<string>();
+			const unique = skills.filter((s) => {
+				const key = s.name.trim();
+				if (!key || seen.has(key)) return false;
+				seen.add(key);
+				return true;
+			});
 			return c.json({
-				skills: skills.map((s) => ({
+				skills: unique.map((s) => ({
 					name: s.name,
 					description: s.description,
 					scope: s.scope,

package/src/runtime/agent/runner-setup.ts

@@ -153,7 +153,7 @@ export async function setupRunner(opts: RunOpts): Promise<SetupResult> {
 		oneShot: opts.oneShot,
 		guidedMode: cfg.defaults.guidedMode,
 		spoofPrompt: undefined,
-		includeProjectTree: isFirstMessage,
+		includeProjectTree: false,
 		userContext: opts.userContext,
 		contextSummary,
 		isOpenAIOAuth: oauth.isOpenAIOAuth,
@@ -347,24 +347,3 @@ export async function setupRunner(opts: RunOpts): Promise<SetupResult> {
 		mcpToolsRecord,
 	};
 }
-
-export function buildMessages(
-	additionalSystemMessages: Array<{ role: string; content: string }>,
-	history: Array<{ role: string; content: string | Array<unknown> }>,
-	isFirstMessage: boolean,
-): Array<{ role: string; content: string | Array<unknown> }> {
-	const messagesWithSystemInstructions: Array<{
-		role: string;
-		content: string | Array<unknown>;
-	}> = [...additionalSystemMessages, ...history];
-
-	if (!isFirstMessage) {
-		messagesWithSystemInstructions.push({
-			role: 'user',
-			content:
-				'SYSTEM REMINDER: You are continuing an existing session. When you have completed the task, you MUST stream a text summary of what you did to the user, and THEN call the `finish` tool. Do not call `finish` without a summary.',
-		});
-	}
-
-	return messagesWithSystemInstructions;
-}

package/src/runtime/agent/runner.ts

@@ -189,13 +189,13 @@ async function runAssistant(opts: RunOpts) {
 			messagesWithSystemInstructions.push({
 				role: 'system',
 				content:
-					'SYSTEM REMINDER: You are continuing an existing session. Continue executing directly, use tools as needed, and provide a concise final summary when complete.',
+					'[system-reminder] Continuing an existing session. Execute directly, use tools as needed, and call `finish` at the end. For simple questions, your answer IS the response — do not add a "Summary:" recap.',
 			});
 		} else {
 			messagesWithSystemInstructions.push({
 				role: 'user',
 				content:
-					'SYSTEM REMINDER: You are continuing an existing session. When you have completed the task, you MUST stream a text summary of what you did to the user, and THEN call the `finish` tool. Do not call `finish` without a summary.',
+					'<system-reminder>Continuing an existing session. Answer or complete the work directly, then call `finish`. For simple questions, your answer IS the response — do NOT add a labeled "Summary:" line or recap trivial replies.</system-reminder>',
 			});
 		}
 	}
@@ -204,13 +204,13 @@ async function runAssistant(opts: RunOpts) {
 			messagesWithSystemInstructions.push({
 				role: 'system',
 				content:
-					'SYSTEM REMINDER: Your previous response stopped mid-task. Continue immediately from where you left off and finish the actual implementation, not just a plan update.',
+					'[system-reminder] Your previous response stopped mid-task. Resume from where you left off and complete the actual work — not a plan-only update.',
 			});
 		} else {
 			messagesWithSystemInstructions.push({
 				role: 'user',
 				content:
-					'SYSTEM REMINDER: Your previous response stopped before calling `finish`. Continue executing immediately from where you left off, avoid plan-only updates, and call `finish` only after streaming the final user summary.',
+					'<system-reminder>Your previous response stopped before calling `finish`. Resume from where you left off, do the actual work (no plan-only updates), then stream a summary and call `finish`.</system-reminder>',
 			});
 		}
 	}