aegiscode 6.5.2 → 6.5.4

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "aegiscode",
   "productName": "AEGIS Code",
-  "version": "6.5.2",
+  "version": "6.5.4",
   "description": "aegiscode \u2014 the command-line version of AEGIS Desktop. The shared tool surface in your shell, over the same thin transport and tool registry as the MCP plugin and the desktop app. Ships transport + UI only; no brain.",
   "author": {
     "name": "AEGIS Code",

package/src/app.js

@@ -21,7 +21,7 @@
 const readline = require('node:readline');
 const os = require('node:os');
 const { randomUUID } = require('node:crypto');
-const { createTools, createClient, usageTokens } = require('./deps.js');
+const { createTools, createClient, usageTokens, buildSystemPrompt } = require('./deps.js');
 const { createEngine } = require('./engine.js');
 const { GLYPH, VERBS, themeOf, RESET, THEME_TABLE } = require('./theme.js');
 const { LiveRegion, termWidth, w } = require('./screen.js');
@@ -41,6 +41,30 @@ const { fmtTokens, fmtEur, maskKey } = require('./format.js');
 
 const VERSION = require('../package.json').version;
 
+/**
+ * The persona for a turn this host did not get an explicit system prompt for.
+ *
+ * Built once and cached: it is the stable head of every request, and a prompt
+ * that changed between turns would defeat the provider's prompt cache for the
+ * whole conversation behind it.
+ */
+let _defaultSystem = null;
+function defaultSystemPrompt() {
+  if (_defaultSystem === null) {
+    try {
+      _defaultSystem = buildSystemPrompt({
+        platform: process.platform,
+        arch: process.arch,
+        homedir: os.homedir(),
+        cwd: process.cwd(),
+      });
+    } catch {
+      _defaultSystem = ''; // never let persona construction break a turn
+    }
+  }
+  return _defaultSystem || undefined;
+}
+
 function createApp(options = {}) {
   const opts = {
     model: null,
@@ -458,7 +482,12 @@ function createApp(options = {}) {
           prompt,
           messages: history,
           model: commandCtx.model || undefined,
-          system: opts.system,
+          // A caller-supplied system wins; otherwise the shared persona. Sent
+          // explicitly rather than left to the server, because the pooled
+          // route takes the system prompt from the request and the CLI was
+          // sending none — the model got tool schemas and no rule about when
+          // to use them.
+          system: opts.system || defaultSystemPrompt(),
           maxTokens: opts.maxTokens,
           // `effort` is the pooled budget control, and it travels on every turn
           // — this host only ever runs the 'aegis' class, whose calls are sized

package/src/deps.js

@@ -59,12 +59,19 @@ const enginePath = resolveShared(path.join('desktop', 'lib', 'local', 'engine.js
 // Subagent role presets (/agents lists these; the model's task tool delegates
 // to them by name) — lives beside engine.js, staged into the same vendor dir.
 const agentsPath = resolveShared(path.join('desktop', 'lib', 'local', 'agents.js'));
+// The persona the tool loop runs under. The CLI sent NO system prompt at all,
+// so a pooled turn reached the model as a bare user message plus tool schemas
+// with nothing saying when a tool is appropriate — which is how "hey" started
+// running shell commands. Same file as the GUI, for the same reason engine.js
+// is: two personas would drift, and only one of them would get fixed.
+const promptPath = resolveShared(path.join('desktop', 'lib', 'local', 'prompt.js'));
 
 const { createClient } = require(clientPath);
 const { createTools } = require(toolsPath);
 const { usageTokens } = require(usagePath);
 const { createLocalEngine } = require(enginePath);
 const { agentRoles, agentRoleLabel } = require(agentsPath);
+const { buildSystemPrompt } = require(promptPath);
 
 module.exports = {
   createClient,
@@ -73,6 +80,7 @@ module.exports = {
   createLocalEngine,
   agentRoles,
   agentRoleLabel,
-  paths: { client: clientPath, tools: toolsPath, usage: usagePath, engine: enginePath, agents: agentsPath },
+  buildSystemPrompt,
+  paths: { client: clientPath, tools: toolsPath, usage: usagePath, engine: enginePath, agents: agentsPath, prompt: promptPath },
   roots,
 };

package/vendor/desktop/lib/local/prompt.js

@@ -31,8 +31,15 @@ const MAIN_CHAT_PROMPT =
   `- Use tools silently. A one-line reason is enough; do not narrate your plan as a story. ` +
   `- Never claim what a tool found or what a command returned before the tool actually runs. ` +
   `  Report only the results you really received. ` +
-  `- Act, don't just inspect. After at most 2 rounds of reading or exploration, start making ` +
-  `  changes with writeFile or editFile. Reconnaissance is not progress — implement, then verify.\n` +
+  `- Match the response to the request. A greeting, a question about something you already ` +
+  `  know, or a request for an opinion is answered in plain words with NO tools. Only reach ` +
+  `  for a tool when the answer genuinely depends on something in this repo or on this ` +
+  `  machine. "hey" is not a task.\n` +
+  `- WHEN THE USER HAS ASKED FOR WORK: act, don't just inspect. After at most 2 rounds of ` +
+  `  reading or exploration, start making changes with writeFile or editFile. Reconnaissance ` +
+  `  is not progress — implement, then verify. This rule is about HOW to carry out a task you ` +
+  `  were given; it is never a reason to invent one. Never write or modify a file the user ` +
+  `  did not ask you to touch.\n` +
   `- When you have what you need, stop using tools and give a concise, direct answer to the ` +
   `  user's question. Never end your turn with an intention like "Let me check…" or "I'll now…" ` +
   `  — that is not an answer. ` +

package/vendor/desktop/lib/local/tools.js

@@ -81,14 +81,12 @@ const fail = (error) => ({ ok: false, error: cap(error) });
 const SCHEMAS = {
   readFile: {
     name: 'readFile',
-    description:
-      'Reads a file from the local filesystem. The file_path parameter must be an absolute path. ' +
-      'Returns line-numbered content.',
+    description: 'Read a file. Absolute path. Returns line-numbered content.',
     parameters: {
       type: 'object',
       properties: {
-        file_path: { type: 'string', description: 'The absolute path to the file to read' },
-        offset: { type: 'number', description: 'The line number to start reading from (0-based)' },
+        file_path: { type: 'string', description: 'Absolute path' },
+        offset: { type: 'number', description: 'Start line (0-based)' },
         limit: { type: 'number', description: `The number of lines to read (max 10000, default ${READ_LINE_CAP})` },
       },
       required: ['file_path'],
@@ -98,13 +96,12 @@ const SCHEMAS = {
   writeFile: {
     name: 'writeFile',
     description:
-      'Writes a file to the local filesystem. Parent directories are created automatically. ' +
-      'Use it to create or replace a whole file; it overwrites whatever was there.',
+      'Write a whole file, overwriting it. Parent dirs are created. Use editFile to change part of one.',
     parameters: {
       type: 'object',
       properties: {
-        file_path: { type: 'string', description: 'The absolute path to the file to write' },
-        content: { type: 'string', description: 'The full contents of the file' },
+        file_path: { type: 'string', description: 'Absolute path' },
+        content: { type: 'string', description: 'Full file contents' },
       },
       required: ['file_path', 'content'],
       additionalProperties: false,
@@ -113,16 +110,15 @@ const SCHEMAS = {
   editFile: {
     name: 'editFile',
     description:
-      'Performs an exact string replacement in a file. old_string must be unique in the file ' +
-      'unless replace_all is true. Use this instead of writeFile when changing part of an ' +
-      'existing file — it fails loudly on a non-unique or missing match instead of guessing.',
+      'Exact string replacement. old_string must be unique unless replace_all. Fails loudly on a ' +
+      'missing or ambiguous match rather than guessing.',
     parameters: {
       type: 'object',
       properties: {
-        file_path: { type: 'string', description: 'The absolute path to the file to modify' },
-        old_string: { type: 'string', description: 'The text to replace (must be unique unless replace_all is true)' },
-        new_string: { type: 'string', description: 'The text to replace it with' },
-        replace_all: { type: 'boolean', description: 'If true, replace all occurrences of old_string' },
+        file_path: { type: 'string', description: 'Absolute path' },
+        old_string: { type: 'string', description: 'Text to replace; must be unique unless replace_all' },
+        new_string: { type: 'string', description: 'Replacement text' },
+        replace_all: { type: 'boolean', description: 'Replace every occurrence' },
       },
       required: ['file_path', 'old_string', 'new_string'],
       additionalProperties: false,
@@ -130,9 +126,7 @@ const SCHEMAS = {
   },
   listDir: {
     name: 'listDir',
-    description:
-      'Lists one directory (non-recursive). Directories are marked with a trailing slash. ' +
-      'Skips node_modules, .git and dist.',
+    description: 'List one directory (non-recursive). Skips node_modules, .git, dist.',
     parameters: {
       type: 'object',
       properties: {
@@ -145,12 +139,12 @@ const SCHEMAS = {
   glob: {
     name: 'glob',
     description:
-      'Find files matching a glob pattern. Supports **, * and ?. Skips node_modules, .git and dist by default.',
+      'Find files by glob (**, *, ?). Skips node_modules, .git, dist.',
     parameters: {
       type: 'object',
       properties: {
-        pattern: { type: 'string', description: 'The glob pattern to match, e.g. "**/*.test.js"' },
-        path: { type: 'string', description: 'The directory to search from (default: the working directory)' },
+        pattern: { type: 'string', description: 'Glob, e.g. "**/*.test.js"' },
+        path: { type: 'string', description: 'Search root (default: cwd)' },
       },
       required: ['pattern'],
       additionalProperties: false,
@@ -158,9 +152,7 @@ const SCHEMAS = {
   },
   grep: {
     name: 'grep',
-    description:
-      'Search file contents using a regular expression. Returns file:line matches. ' +
-      'Skips node_modules, .git and dist by default.',
+    description: 'Search file contents by regex. Returns file:line matches. Skips node_modules, .git, dist.',
     parameters: {
       type: 'object',
       properties: {
@@ -174,15 +166,13 @@ const SCHEMAS = {
   exec: {
     name: 'exec',
     description:
-      'Executes a shell command in a persistent shell session and returns its combined ' +
-      'stdout+stderr plus the exit code. State (cd, exported env vars) carries across calls ' +
-      'within the same turn — it is a real session, not a fresh process each time. ' +
-      'Use for system operations, git commands and package management.',
+      'Run a shell command in a PERSISTENT session: cd and exported env carry across calls in ' +
+      'this turn. Returns stdout+stderr and the exit code. Use for git, packages, system ops.',
     parameters: {
       type: 'object',
       properties: {
-        command: { type: 'string', description: 'The shell command to execute' },
-        cwd: { type: 'string', description: 'Run this one command in a different directory without moving the session (the session cwd is unchanged for later calls)' },
+        command: { type: 'string', description: 'Command to run' },
+        cwd: { type: 'string', description: 'Run this one command elsewhere; session cwd unchanged' },
         timeout: { type: 'number', description: `Timeout in milliseconds (max ${EXEC_TIMEOUT_CAP}, default ${EXEC_TIMEOUT_DEFAULT})` },
         description: { type: 'string', description: 'A brief description of what the command does (for display)' },
       },
@@ -193,17 +183,12 @@ const SCHEMAS = {
   task: {
     name: 'task',
     description:
-      'Spawn a specialized subagent to autonomously handle a focused, multi-step sub-task. ' +
-      'The subagent runs its own tool loop (readFile/writeFile/editFile/listDir/glob/grep/exec) ' +
-      'on the same model and returns a final report as the tool result. Use it to delegate work ' +
-      'like scanning for vulnerabilities, reviewing code, planning a refactor, or scaffolding a ' +
-      'component — give it a complete, self-contained prompt since it cannot ask follow-up ' +
-      'questions. Subagents can delegate further with task, so a large job can be split ' +
-      'hierarchically as deep as useful.',
+      'Delegate a focused multi-step sub-task to a subagent with its own tool loop; it returns a ' +
+      'final report. Give it a complete, self-contained prompt — it cannot ask follow-ups.',
     parameters: {
       type: 'object',
       properties: {
-        description: { type: 'string', description: 'A short (3-5 word) description of the sub-task' },
+        description: { type: 'string', description: 'Short label (3-5 words)' },
         subagent_type: {
           type: 'string',
           enum: [...agentRoles(), 'general'],