@link-assistant/agent 0.25.3 → 0.25.5

package/LICENSE

@@ -0,0 +1,24 @@
+This is free and unencumbered software released into the public domain.
+
+Anyone is free to copy, modify, publish, use, compile, sell, or
+distribute this software, either in source code form or as a compiled
+binary, for any purpose, commercial or non-commercial, and by any
+means.
+
+In jurisdictions that recognize copyright laws, the author or authors
+of this software dedicate any and all copyright interest in the
+software to the public domain. We make this dedication for the benefit
+of the public at large and to the detriment of our heirs and
+successors. We intend this dedication to be an overt act of
+relinquishment in perpetuity of all present and future rights to this
+software under copyright law.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
+EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
+MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.
+IN NO EVENT SHALL THE AUTHORS BE LIABLE FOR ANY CLAIM, DAMAGES OR
+OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE,
+ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
+OTHER DEALINGS IN THE SOFTWARE.
+
+For more information, please refer to <https://unlicense.org>

package/package.json

@@ -1,20 +1,21 @@
 {
   "name": "@link-assistant/agent",
-  "version": "0.25.3",
+  "version": "0.25.5",
   "description": "A minimal, public domain AI CLI agent compatible with OpenCode's JSON interface. Bun-only runtime.",
   "main": "src/index.js",
   "type": "module",
   "bin": {
-    "agent": "./src/index.js"
+    "agent": "src/index.js"
   },
   "scripts": {
     "dev": "bun run src/index.js",
-    "test": "bun test ./tests/*.js ./tests/*.ts",
+    "test": "bun test --timeout 30000 ./tests/*.js ./tests/*.ts",
     "test:integration": "bun test ./tests/integration/basic.js",
-    "lint": "eslint .",
-    "lint:fix": "eslint . --fix",
-    "format": "prettier --write .",
-    "format:check": "prettier --check .",
+    "lint": "eslint . && npm run lint:scripts",
+    "lint:scripts": "cd ../scripts && eslint . --config ./eslint.config.mjs",
+    "lint:fix": "eslint . --fix && cd ../scripts && eslint . --config ./eslint.config.mjs --fix",
+    "format": "prettier --write . ../scripts",
+    "format:check": "prettier --check . ../scripts",
     "check:file-size": "node ../scripts/check-file-size.mjs",
     "check": "npm run lint && npm run format:check && npm run check:file-size",
     "prepare": "cd .. && husky || true",
@@ -28,7 +29,7 @@
   },
   "repository": {
     "type": "git",
-    "url": "https://github.com/link-assistant/agent"
+    "url": "git+https://github.com/link-assistant/agent.git"
   },
   "keywords": [
     "ai",
@@ -46,10 +47,7 @@
     "src/",
     "package.json",
     "README.md",
-    "../MODELS.md",
-    "../TOOLS.md",
-    "../EXAMPLES.md",
-    "../LICENSE"
+    "LICENSE"
   ],
   "dependencies": {
     "@actions/core": "^1.11.1",

package/src/branding.ts

@@ -0,0 +1,47 @@
+/**
+ * Single source of truth for the product identity that is exposed to models
+ * and to users.
+ *
+ * Several system prompts in `src/session/prompt/*.txt` are kept byte-identical
+ * to their upstream (opencode) originals so they can be re-synced without
+ * conflicts. Instead of editing those files, the identity is substituted at
+ * render time by {@link applyBranding}, so a rename is one edit here rather
+ * than one edit per prompt file. See issue #285.
+ */
+export namespace Branding {
+  /** Product name as it should appear in prose. */
+  export const NAME = 'Agent';
+  /** Product name as it appears in a shell (the binary name). */
+  export const BINARY = 'agent';
+  /** Canonical source repository. */
+  export const REPO_URL = 'https://github.com/link-assistant/agent';
+  /** Where users should report issues with this tool. */
+  export const ISSUES_URL = 'https://github.com/link-assistant/agent/issues';
+  /** Where documentation about this tool lives. */
+  export const DOCS_URL = 'https://github.com/link-assistant/agent#readme';
+
+  /**
+   * Ordered replacements. URLs must be replaced before the bare product name,
+   * otherwise the name substitution would corrupt the URLs.
+   */
+  const REPLACEMENTS: [RegExp, string][] = [
+    [/https:\/\/github\.com\/sst\/opencode\/issues/gi, ISSUES_URL],
+    [/https:\/\/github\.com\/sst\/opencode/gi, REPO_URL],
+    [/https:\/\/opencode\.ai\/docs/gi, DOCS_URL],
+    [/https:\/\/opencode\.ai/gi, DOCS_URL],
+    [/sst\/opencode/gi, 'link-assistant/agent'],
+    [/opencode/gi, NAME],
+  ];
+
+  /**
+   * Replace every upstream product reference in `text` with this product's
+   * identity. Safe to call on text that contains no references.
+   */
+  export function apply(text: string): string {
+    let result = text;
+    for (const [pattern, replacement] of REPLACEMENTS) {
+      result = result.replace(pattern, replacement);
+    }
+    return result;
+  }
+}

package/src/index.js

@@ -46,22 +46,12 @@ import {
   outputInput,
 } from './cli/output.ts';
 import stripAnsi from 'strip-ansi';
-import { createRequire } from 'module';
-import { readFileSync } from 'fs';
-import { dirname, join } from 'path';
-import { fileURLToPath } from 'url';
-
-const require = createRequire(import.meta.url);
-let pkg;
-try {
-  pkg = require('../package.json');
-} catch (_e) {
-  // Fallback: read package.json directly
-  const __dirname = dirname(fileURLToPath(import.meta.url));
-  const pkgPath = join(__dirname, '../package.json');
-  const pkgContent = readFileSync(pkgPath, 'utf8');
-  pkg = JSON.parse(pkgContent);
-}
+import { resolve as resolvePath } from 'path';
+import { VERSION } from './version.ts';
+
+// Version comes from the shared module so that `--version`, the process log
+// and stored session records can never disagree (#285).
+const pkg = { version: VERSION };
 
 // Track if any errors occurred during execution
 let hasError = false;
@@ -226,7 +216,7 @@ async function readSystemMessages(argv) {
   let appendSystemMessage = argv['append-system-message'];
 
   if (argv['system-message-file']) {
-    const resolvedPath = require('path').resolve(
+    const resolvedPath = resolvePath(
       process.cwd(),
       argv['system-message-file']
     );
@@ -242,7 +232,7 @@ async function readSystemMessages(argv) {
   }
 
   if (argv['append-system-message-file']) {
-    const resolvedPath = require('path').resolve(
+    const resolvedPath = resolvePath(
       process.cwd(),
       argv['append-system-message-file']
     );

package/src/provider/retry-fetch.ts

@@ -183,9 +183,16 @@ export namespace RetryFetch {
         return;
       }
 
+      // The timer is deliberately NOT unref'd. Every caller awaits this
+      // promise, so an unref'd timer is the only pending work while a retry
+      // wait is in flight: the runtime is then free to stop waiting on it,
+      // which drops the retry instead of performing it. On Bun for Windows
+      // this showed up as `tests/retry-fetch.ts > retries on 429 and succeeds
+      // on second attempt` never resuming, burning the whole job timeout.
+      // See https://github.com/link-assistant/agent/issues/287.
+      // The abort path below clears the timer, so a cancelled wait still lets
+      // the process exit immediately (the concern behind #213).
       const timeout = setTimeout(resolve, ms);
-      // Prevent sleep timer from keeping event loop alive (#213)
-      if (timeout.unref) timeout.unref();
 
       if (signal) {
         const abortHandler = () => {

package/src/session/index.ts

@@ -14,6 +14,7 @@ import { SessionPrompt } from './prompt';
 import { fn } from '../util/fn';
 import { Command } from '../command';
 import { Snapshot } from '../snapshot';
+import { VERSION } from '../version';
 
 export namespace Session {
   const log = Log.create({ service: 'session' });
@@ -226,7 +227,7 @@ export namespace Session {
   }) {
     const result: Info = {
       id: Identifier.descending('session', input.id),
-      version: 'agent-cli-1.0.0',
+      version: VERSION,
       projectID: Instance.project.id,
       directory: input.directory,
       parentID: input.parentID,

package/src/session/system.ts

@@ -18,27 +18,105 @@ import PROMPT_SUMMARIZE from './prompt/summarize.txt';
 import PROMPT_TITLE from './prompt/title.txt';
 import PROMPT_CODEX from './prompt/codex.txt';
 import PROMPT_GROK_CODE from './prompt/grok-code.txt';
+import { Branding } from '../branding';
+import { Log } from '../util/log';
 
 export namespace SystemPrompt {
+  const log = Log.create({ service: 'system-prompt' });
+
   export function header(providerID: string) {
     if (providerID.includes('anthropic'))
       return [PROMPT_ANTHROPIC_SPOOF.trim()];
     return [];
   }
 
+  /** Identifiers of the selectable system prompts. */
+  export type PromptID =
+    | 'anthropic'
+    | 'anthropic-without-todo'
+    | 'beast'
+    | 'codex'
+    | 'gemini'
+    | 'grok-code'
+    | 'polaris';
+
+  const RAW: Record<PromptID, string> = {
+    anthropic: PROMPT_ANTHROPIC,
+    'anthropic-without-todo': PROMPT_ANTHROPIC_WITHOUT_TODO,
+    beast: PROMPT_BEAST,
+    codex: PROMPT_CODEX,
+    gemini: PROMPT_GEMINI,
+    'grok-code': PROMPT_GROK_CODE,
+    polaris: PROMPT_POLARIS,
+  };
+
+  /**
+   * The prompt used for any model that is not matched by an explicit rule.
+   * It is the full prompt (with todo/task-tracking discipline); models that
+   * genuinely break on todo tools must opt out explicitly via the
+   * `AGENT_SYSTEM_PROMPT` environment variable. See issue #285.
+   */
+  export const DEFAULT_PROMPT_ID: PromptID = 'anthropic';
+
+  const RULES: { matches: (modelID: string) => boolean; id: PromptID }[] = [
+    { matches: (m) => m.includes('gpt-5'), id: 'codex' },
+    {
+      matches: (m) =>
+        m.includes('gpt-') || m.includes('o1') || m.includes('o3'),
+      id: 'beast',
+    },
+    { matches: (m) => m.includes('gemini-'), id: 'gemini' },
+    { matches: (m) => m.includes('claude'), id: 'anthropic' },
+    { matches: (m) => m.includes('polaris-alpha'), id: 'polaris' },
+    { matches: (m) => m.includes('grok-code'), id: 'grok-code' },
+  ];
+
+  export function isPromptID(value: string): value is PromptID {
+    return value in RAW;
+  }
+
+  /**
+   * Resolve which prompt a model gets, and why.
+   *
+   * Resolution order:
+   * 1. explicit override via the `AGENT_SYSTEM_PROMPT` environment variable;
+   * 2. an explicit rule matching the model id;
+   * 3. {@link DEFAULT_PROMPT_ID}.
+   */
+  export function resolve(modelID: string): { id: PromptID; reason: string } {
+    const override = process.env['AGENT_SYSTEM_PROMPT']?.trim();
+    if (override) {
+      if (isPromptID(override))
+        return { id: override, reason: 'AGENT_SYSTEM_PROMPT override' };
+      log.warn(() => ({
+        message: 'unknown AGENT_SYSTEM_PROMPT value, ignoring',
+        value: override,
+        known: Object.keys(RAW),
+      }));
+    }
+    for (const rule of RULES) {
+      if (rule.matches(modelID))
+        return { id: rule.id, reason: `matched model id ${modelID}` };
+    }
+    return {
+      id: DEFAULT_PROMPT_ID,
+      reason: `default for unknown model ${modelID}`,
+    };
+  }
+
+  /** Get a prompt by id, with the product identity substituted in (#285). */
+  export function text(id: PromptID): string {
+    return Branding.apply(RAW[id]);
+  }
+
   export function provider(modelID: string) {
-    if (modelID.includes('gpt-5')) return [PROMPT_CODEX];
-    if (
-      modelID.includes('gpt-') ||
-      modelID.includes('o1') ||
-      modelID.includes('o3')
-    )
-      return [PROMPT_BEAST];
-    if (modelID.includes('gemini-')) return [PROMPT_GEMINI];
-    if (modelID.includes('claude')) return [PROMPT_ANTHROPIC];
-    if (modelID.includes('polaris-alpha')) return [PROMPT_POLARIS];
-    if (modelID.includes('grok-code')) return [PROMPT_GROK_CODE];
-    return [PROMPT_ANTHROPIC_WITHOUT_TODO];
+    const resolved = resolve(modelID);
+    log.info(() => ({
+      message: `system prompt: ${resolved.id} (${resolved.reason})`,
+      prompt: resolved.id,
+      model: modelID,
+    }));
+    return [text(resolved.id)];
   }
 
   export async function environment() {
@@ -149,18 +227,21 @@ export namespace SystemPrompt {
   export function summarize(providerID: string) {
     switch (providerID) {
       case 'anthropic':
-        return [PROMPT_ANTHROPIC_SPOOF.trim(), PROMPT_SUMMARIZE];
+        return [
+          PROMPT_ANTHROPIC_SPOOF.trim(),
+          Branding.apply(PROMPT_SUMMARIZE),
+        ];
       default:
-        return [PROMPT_SUMMARIZE];
+        return [Branding.apply(PROMPT_SUMMARIZE)];
     }
   }
 
   export function title(providerID: string) {
     switch (providerID) {
       case 'anthropic':
-        return [PROMPT_ANTHROPIC_SPOOF.trim(), PROMPT_TITLE];
+        return [PROMPT_ANTHROPIC_SPOOF.trim(), Branding.apply(PROMPT_TITLE)];
       default:
-        return [PROMPT_TITLE];
+        return [Branding.apply(PROMPT_TITLE)];
     }
   }
 }

package/src/tool/bash.ts

@@ -2,6 +2,7 @@ import z from 'zod';
 import { spawn } from 'child_process';
 import { Tool } from './tool';
 import DESCRIPTION from './bash.txt';
+import { Branding } from '../branding';
 import { Log } from '../util/log';
 import { Instance } from '../project/instance';
 import { lazy } from '../util/lazy';
@@ -53,7 +54,7 @@ const parser = lazy(async () => {
 });
 
 export const BashTool = Tool.define('bash', {
-  description: DESCRIPTION,
+  description: Branding.apply(DESCRIPTION),
   parameters: z.object({
     command: z.string().describe('The command to execute'),
     timeout: z.number().describe('Optional timeout in milliseconds').optional(),

package/src/version.ts

@@ -0,0 +1,25 @@
+/**
+ * Single source of truth for the running version.
+ *
+ * The version is read from the package manifest, so `--version`, the process
+ * log and stored session records can never disagree. See issue #285.
+ */
+import { createRequire } from 'module';
+import { readFileSync } from 'fs';
+import { dirname, join } from 'path';
+import { fileURLToPath } from 'url';
+
+function read(): string {
+  try {
+    const require = createRequire(import.meta.url);
+    return require('../package.json').version;
+  } catch {
+    // Fallback: read package.json directly (e.g. when `require` of JSON is
+    // unavailable in the current runtime configuration).
+    const here = dirname(fileURLToPath(import.meta.url));
+    return JSON.parse(readFileSync(join(here, '../package.json'), 'utf8'))
+      .version;
+  }
+}
+
+export const VERSION: string = read();