@link-assistant/agent 0.18.3 → 0.19.2

package/src/session/index.ts

@@ -2,8 +2,8 @@ import { Decimal } from 'decimal.js';
 import z from 'zod';
 import { type LanguageModelUsage, type ProviderMetadata } from 'ai';
 import { Bus } from '../bus';
-import { Config } from '../config/config';
-import { Flag } from '../flag/flag';
+import { Config } from '../config/file-config';
+import { isVerbose } from '../config/config';
 import { Identifier } from '../id/id';
 import type { ModelsDev } from '../provider/models';
 import { Storage } from '../storage/storage';
@@ -35,7 +35,7 @@ export namespace Session {
    */
   export const toDecimal = (value: unknown, context?: string): Decimal => {
     // Log input data in verbose mode to help identify issues
-    if (Flag.OPENCODE_VERBOSE) {
+    if (isVerbose()) {
       log.debug(() => ({
         message: 'toDecimal input',
         context,
@@ -51,7 +51,7 @@ export namespace Session {
       const result = new Decimal(value as any);
 
       // Log successful conversion in verbose mode
-      if (Flag.OPENCODE_VERBOSE) {
+      if (isVerbose()) {
         log.debug(() => ({
           message: 'toDecimal success',
           context,
@@ -62,7 +62,7 @@ export namespace Session {
       return result;
     } catch (error) {
       // Log the error and return Decimal(NaN)
-      if (Flag.OPENCODE_VERBOSE) {
+      if (isVerbose()) {
         log.debug(() => ({
           message: 'toDecimal error - returning Decimal(NaN)',
           context,
@@ -397,7 +397,7 @@ export namespace Session {
    */
   export const toNumber = (value: unknown, context?: string): number => {
     // Log input data in verbose mode to help identify issues
-    if (Flag.OPENCODE_VERBOSE) {
+    if (isVerbose()) {
       log.debug(() => ({
         message: 'toNumber input',
         context,
@@ -412,7 +412,7 @@ export namespace Session {
       // These are expected for optional fields like cachedInputTokens, reasoningTokens
       // See: https://github.com/link-assistant/agent/issues/127
       if (value === undefined || value === null) {
-        if (Flag.OPENCODE_VERBOSE) {
+        if (isVerbose()) {
           log.debug(() => ({
             message: 'toNumber received undefined/null, returning 0',
             context,
@@ -432,7 +432,7 @@ export namespace Session {
         typeof (value as { total: unknown }).total === 'number'
       ) {
         const result = (value as { total: number }).total;
-        if (Flag.OPENCODE_VERBOSE) {
+        if (isVerbose()) {
           log.debug(() => ({
             message: 'toNumber extracted total from object',
             context,
@@ -452,7 +452,7 @@ export namespace Session {
       }
 
       // Log successful conversion in verbose mode
-      if (Flag.OPENCODE_VERBOSE) {
+      if (isVerbose()) {
         log.debug(() => ({
           message: 'toNumber success',
           context,
@@ -463,7 +463,7 @@ export namespace Session {
       return result;
     } catch (error) {
       // Log the error and return NaN
-      if (Flag.OPENCODE_VERBOSE) {
+      if (isVerbose()) {
         log.debug(() => ({
           message: 'toNumber error - returning NaN',
           context,
@@ -493,7 +493,7 @@ export namespace Session {
    */
   export const toFinishReason = (value: unknown): string => {
     // Log input data in verbose mode to help identify issues
-    if (Flag.OPENCODE_VERBOSE) {
+    if (isVerbose()) {
       log.debug(() => ({
         message: 'toFinishReason input',
         valueType: typeof value,
@@ -518,7 +518,7 @@ export namespace Session {
 
       // Try common field names that might contain the reason
       if (typeof obj.type === 'string') {
-        if (Flag.OPENCODE_VERBOSE) {
+        if (isVerbose()) {
           log.debug(() => ({
             message: 'toFinishReason extracted type from object',
             result: obj.type,
@@ -528,7 +528,7 @@ export namespace Session {
       }
 
       if (typeof obj.finishReason === 'string') {
-        if (Flag.OPENCODE_VERBOSE) {
+        if (isVerbose()) {
           log.debug(() => ({
             message: 'toFinishReason extracted finishReason from object',
             result: obj.finishReason,
@@ -538,7 +538,7 @@ export namespace Session {
       }
 
       if (typeof obj.reason === 'string') {
-        if (Flag.OPENCODE_VERBOSE) {
+        if (isVerbose()) {
           log.debug(() => ({
             message: 'toFinishReason extracted reason from object',
             result: obj.reason,
@@ -550,7 +550,7 @@ export namespace Session {
       // Handle AI SDK unified/raw format: {unified: "tool-calls", raw: "tool_calls"}
       // See: https://github.com/link-assistant/agent/issues/129
       if (typeof obj.unified === 'string') {
-        if (Flag.OPENCODE_VERBOSE) {
+        if (isVerbose()) {
           log.debug(() => ({
             message: 'toFinishReason extracted unified from object',
             result: obj.unified,
@@ -560,7 +560,7 @@ export namespace Session {
       }
 
       // If we can't extract a specific field, return JSON representation
-      if (Flag.OPENCODE_VERBOSE) {
+      if (isVerbose()) {
         log.debug(() => ({
           message: 'toFinishReason could not extract string, using JSON',
           result: JSON.stringify(value),
@@ -602,7 +602,7 @@ export namespace Session {
       }
 
       // Log raw usage data in verbose mode for debugging
-      if (Flag.OPENCODE_VERBOSE) {
+      if (isVerbose()) {
         log.debug(() => ({
           message: 'getUsage called with raw data',
           rawUsage: JSON.stringify(input.usage),
@@ -642,7 +642,7 @@ export namespace Session {
       // If standard usage is empty but openrouter metadata has usage, use it as source
       let effectiveUsage = input.usage;
       if (standardUsageIsEmpty && openrouterUsage) {
-        if (Flag.OPENCODE_VERBOSE) {
+        if (isVerbose()) {
           log.debug(() => ({
             message:
               'Standard usage empty, falling back to openrouter metadata',
@@ -682,7 +682,7 @@ export namespace Session {
           anthropicUsage &&
           (anthropicUsage.input_tokens || anthropicUsage.output_tokens)
         ) {
-          if (Flag.OPENCODE_VERBOSE) {
+          if (isVerbose()) {
             log.debug(() => ({
               message:
                 'Standard usage empty, falling back to anthropic provider metadata',

package/src/session/processor.ts

@@ -16,7 +16,7 @@ import { SessionSummary } from './summary';
 import { Bus } from '../bus';
 import { SessionRetry } from './retry';
 import { SessionStatus } from './status';
-import { Flag } from '../flag/flag';
+import { config, isVerbose } from '../config/config';
 import { SessionCompaction } from './compaction';
 
 export namespace SessionProcessor {
@@ -356,7 +356,7 @@ export namespace SessionProcessor {
                   // Build model info if --output-response-model flag is enabled
                   // @see https://github.com/link-assistant/agent/issues/179
                   const modelInfo: MessageV2.ModelInfo | undefined =
-                    Flag.OUTPUT_RESPONSE_MODEL
+                    config.outputResponseModel
                       ? {
                           providerID: input.providerID,
                           requestedModelID: input.model.id,
@@ -374,7 +374,7 @@ export namespace SessionProcessor {
                     model: input.model,
                   });
 
-                  if (Flag.OPENCODE_VERBOSE && contextDiag) {
+                  if (isVerbose() && contextDiag) {
                     log.info(() => ({
                       message: 'step-finish context diagnostics',
                       providerID: input.providerID,
@@ -547,7 +547,7 @@ export namespace SessionProcessor {
                       ? SessionRetry.timeoutDelay(attempt)
                       : SessionRetry.delay(error, attempt);
               } catch (delayError) {
-                // If retry-after exceeds AGENT_RETRY_TIMEOUT, fail immediately
+                // If retry-after exceeds LINK_ASSISTANT_AGENT_RETRY_TIMEOUT, fail immediately
                 if (
                   delayError instanceof SessionRetry.RetryTimeoutExceededError
                 ) {

package/src/session/prompt.ts

@@ -24,7 +24,7 @@ import { Instance } from '../project/instance';
 import { Bus } from '../bus';
 import { ProviderTransform } from '../provider/transform';
 import { SystemPrompt } from './system';
-import { Flag } from '../flag/flag';
+import { config, isVerbose } from '../config/config';
 import { Token } from '../util/token';
 
 import PROMPT_PLAN from '../session/prompt/plan.txt';
@@ -684,7 +684,7 @@ export namespace SessionPrompt {
         : [];
 
       // Verbose logging: output request details for debugging
-      if (Flag.OPENCODE_VERBOSE) {
+      if (isVerbose()) {
         const systemTokens = system.reduce(
           (acc, s) => acc + Token.estimate(s),
           0
@@ -766,8 +766,8 @@ export namespace SessionPrompt {
       const result = await processor.process(() =>
         streamText({
           timeout: {
-            chunkMs: Flag.STREAM_CHUNK_TIMEOUT_MS(),
-            stepMs: Flag.STREAM_STEP_TIMEOUT_MS(),
+            chunkMs: config.streamChunkTimeoutMs,
+            stepMs: config.streamStepTimeoutMs,
           },
           onError(error) {
             log.error(() => ({ message: 'stream error', error }));
@@ -1671,7 +1671,7 @@ export namespace SessionPrompt {
     modelID: string;
   }) {
     // Skip title generation if disabled (default)
-    if (!Flag.GENERATE_TITLE) {
+    if (!config.generateTitle) {
       log.info(() => ({
         message: 'title generation disabled',
         hint: 'Enable with --generate-title flag or AGENT_GENERATE_TITLE=true',

package/src/session/retry.ts

@@ -1,5 +1,5 @@
 import { MessageV2 } from './message-v2';
-import { Flag } from '../flag/flag';
+import { config } from '../config/config';
 import { Log } from '../util/log';
 
 export namespace SessionRetry {
@@ -13,16 +13,16 @@ export namespace SessionRetry {
   // This caps exponential backoff when headers are not available
   // Can be configured via AGENT_MAX_RETRY_DELAY env var
   export function getMaxRetryDelay(): number {
-    return Flag.MAX_RETRY_DELAY();
+    return config.maxRetryDelay * 1000;
   }
 
   // Get retry timeout in milliseconds
   export function getRetryTimeout(): number {
-    return Flag.RETRY_TIMEOUT() * 1000;
+    return config.retryTimeout * 1000;
   }
 
   /**
-   * Error thrown when retry-after exceeds AGENT_RETRY_TIMEOUT
+   * Error thrown when retry-after exceeds LINK_ASSISTANT_AGENT_RETRY_TIMEOUT
    * This indicates the wait time is too long and we should fail immediately
    */
   export class RetryTimeoutExceededError extends Error {
@@ -34,7 +34,7 @@ export namespace SessionRetry {
       const maxTimeoutHours = (maxTimeoutMs / 1000 / 3600).toFixed(2);
       super(
         `API returned retry-after of ${retryAfterHours} hours, which exceeds the maximum retry timeout of ${maxTimeoutHours} hours. ` +
-          `Failing immediately instead of waiting. You can adjust AGENT_RETRY_TIMEOUT env var to increase the limit.`
+          `Failing immediately instead of waiting. You can adjust LINK_ASSISTANT_AGENT_RETRY_TIMEOUT env var to increase the limit.`
       );
       this.name = 'RetryTimeoutExceededError';
       this.retryAfterMs = retryAfterMs;
@@ -75,7 +75,7 @@ export namespace SessionRetry {
     sessionID: string,
     errorType: string
   ): { shouldRetry: boolean; elapsedTime: number; maxTime: number } {
-    const maxTime = Flag.RETRY_TIMEOUT() * 1000; // Convert to ms
+    const maxTime = config.retryTimeout * 1000; // Convert to ms
     const state = retryStates.get(sessionID);
 
     if (!state || state.errorType !== errorType) {
@@ -198,15 +198,15 @@ export namespace SessionRetry {
    *
    * RETRY LOGIC (per issue #157 requirements):
    * 1. If retry-after header is available:
-   *    - If retry-after <= AGENT_RETRY_TIMEOUT: use it directly (exact time)
-   *    - If retry-after > AGENT_RETRY_TIMEOUT: throw RetryTimeoutExceededError (fail immediately)
+   *    - If retry-after <= LINK_ASSISTANT_AGENT_RETRY_TIMEOUT: use it directly (exact time)
+   *    - If retry-after > LINK_ASSISTANT_AGENT_RETRY_TIMEOUT: throw RetryTimeoutExceededError (fail immediately)
    * 2. If no retry-after header:
    *    - Use exponential backoff up to AGENT_MAX_RETRY_DELAY
    *
    * Adds jitter to prevent thundering herd when multiple requests retry.
    * See: https://github.com/link-assistant/agent/issues/157
    *
-   * @throws {RetryTimeoutExceededError} When retry-after exceeds AGENT_RETRY_TIMEOUT
+   * @throws {RetryTimeoutExceededError} When retry-after exceeds LINK_ASSISTANT_AGENT_RETRY_TIMEOUT
    */
   export function delay(error: MessageV2.APIError, attempt: number): number {
     const maxRetryTimeout = getRetryTimeout();

package/src/session/summary.ts

@@ -13,7 +13,7 @@ import path from 'path';
 import { Instance } from '../project/instance';
 import { Storage } from '../storage/storage';
 import { Bus } from '../bus';
-import { Flag } from '../flag/flag';
+import { config, isVerbose } from '../config/config';
 import { Token } from '../util/token';
 
 export namespace SessionSummary {
@@ -83,7 +83,7 @@ export namespace SessionSummary {
 
     // Skip AI-powered summarization if disabled
     // See: https://github.com/link-assistant/agent/issues/217
-    if (!Flag.SUMMARIZE_SESSION) {
+    if (!config.summarizeSession) {
       log.info(() => ({
         message: 'session summarization disabled',
         hint: 'Enable with --summarize-session flag (enabled by default) or AGENT_SUMMARIZE_SESSION=true',
@@ -141,7 +141,7 @@ export namespace SessionSummary {
       return;
     }
 
-    if (Flag.OPENCODE_VERBOSE) {
+    if (isVerbose()) {
       log.info(() => ({
         message: 'summarization model loaded',
         providerID: model.providerID,
@@ -167,7 +167,7 @@ export namespace SessionSummary {
               </text>
             `;
 
-      if (Flag.OPENCODE_VERBOSE) {
+      if (isVerbose()) {
         log.info(() => ({
           message: 'generating title via API',
           providerID: model.providerID,
@@ -203,7 +203,7 @@ export namespace SessionSummary {
         model: model.language,
       });
 
-      if (Flag.OPENCODE_VERBOSE) {
+      if (isVerbose()) {
         log.info(() => ({
           message: 'title API response received',
           providerID: model.providerID,
@@ -234,7 +234,7 @@ export namespace SessionSummary {
         const modelMessages = await MessageV2.toModelMessage(messages);
         const conversationContent = JSON.stringify(modelMessages);
 
-        if (Flag.OPENCODE_VERBOSE) {
+        if (isVerbose()) {
           log.info(() => ({
             message: 'generating body summary via API',
             providerID: model.providerID,
@@ -264,7 +264,7 @@ export namespace SessionSummary {
           ],
           headers: model.info.headers,
         }).catch((err) => {
-          if (Flag.OPENCODE_VERBOSE) {
+          if (isVerbose()) {
             log.warn(() => ({
               message: 'body summary API call failed',
               providerID: model.providerID,
@@ -276,7 +276,7 @@ export namespace SessionSummary {
           return undefined;
         });
         if (result) {
-          if (Flag.OPENCODE_VERBOSE) {
+          if (isVerbose()) {
             log.info(() => ({
               message: 'body summary API response received',
               providerID: model.providerID,

package/src/session/system.ts

@@ -1,7 +1,7 @@
 import { Ripgrep } from '../file/ripgrep';
 import { Global } from '../global';
 import { Filesystem } from '../util/filesystem';
-import { Config } from '../config/config';
+import { Config } from '../config/file-config';
 
 import { Instance } from '../project/instance';
 import path from 'path';

package/src/snapshot/index.ts

@@ -4,7 +4,7 @@ import fs from 'fs/promises';
 import { Log } from '../util/log';
 import { Global } from '../global';
 import z from 'zod';
-import { Config } from '../config/config';
+import { Config } from '../config/file-config';
 import { Instance } from '../project/instance';
 
 export namespace Snapshot {

package/src/storage/storage.ts

@@ -180,8 +180,19 @@ export namespace Storage {
     for (let index = migration; index < MIGRATIONS.length; index++) {
       log.info(() => ({ message: 'running migration', index }));
       const migration = MIGRATIONS[index];
-      await migration(dir).catch(() =>
-        log.error(() => ({ message: 'failed to run migration', index }))
+      await migration(dir).catch((migrationError) =>
+        log.error(() => ({
+          message: 'failed to run migration',
+          index,
+          error:
+            migrationError instanceof Error
+              ? {
+                  name: migrationError.name,
+                  message: migrationError.message,
+                  stack: migrationError.stack,
+                }
+              : String(migrationError),
+        }))
       );
       await Bun.write(path.join(dir, 'migration'), (index + 1).toString());
     }

package/src/tool/read.ts

@@ -4,6 +4,7 @@ import * as path from 'path';
 import { Tool } from './tool';
 import { FileTime } from '../file/time';
 import DESCRIPTION from './read.txt';
+import { config } from '../config/config';
 import { Filesystem } from '../util/filesystem';
 import { Instance } from '../project/instance';
 import { Provider } from '../provider/provider';
@@ -70,8 +71,8 @@ export const ReadTool = Tool.define('read', {
       return model.info.modalities?.input?.includes('image') ?? false;
     })();
     if (isImage) {
-      // Image format validation (can be disabled via environment variable)
-      const verifyImages = process.env.VERIFY_IMAGES_AT_READ_TOOL !== 'false';
+      // Image format validation (can be disabled via LINK_ASSISTANT_AGENT_VERIFY_IMAGES_AT_READ_TOOL)
+      const verifyImages = config.verifyImagesAtReadTool;
       if (verifyImages && !supportsImages) {
         throw new Error(
           `Failed to read image: ${filepath}, model may not be able to read images`
@@ -90,7 +91,7 @@ export const ReadTool = Tool.define('read', {
               )
                 .map((b) => b.toString(16).padStart(2, '0'))
                 .join(' ')}\n` +
-              `To disable image validation, set environment variable: VERIFY_IMAGES_AT_READ_TOOL=false`
+              `To disable image validation, set environment variable: LINK_ASSISTANT_AGENT_VERIFY_IMAGES_AT_READ_TOOL=false`
           );
         }
       }

package/src/tool/registry.ts

@@ -13,10 +13,9 @@ import { InvalidTool } from './invalid';
 import type { Agent } from '../agent/agent';
 import { Tool } from './tool';
 import { Instance } from '../project/instance';
-import { Config } from '../config/config';
+import { Config } from '../config/file-config';
 import { WebSearchTool } from './websearch';
 import { CodeSearchTool } from './codesearch';
-import { Flag } from '../flag/flag';
 
 export namespace ToolRegistry {
   export const state = Instance.state(async () => {

package/src/tool/websearch.ts

@@ -1,7 +1,7 @@
 import z from 'zod';
 import { Tool } from './tool';
 import DESCRIPTION from './websearch.txt';
-import { Config } from '../config/config';
+import { Config } from '../config/file-config';
 import { createVerboseFetch } from '../util/verbose-fetch';
 
 const verboseFetch = createVerboseFetch(fetch, { caller: 'websearch' });

package/src/util/log-lazy.ts

@@ -1,5 +1,5 @@
 import makeLog, { levels, LogLevel } from 'log-lazy';
-import { Flag } from '../flag/flag';
+import { config, isVerbose } from '../config/config';
 
 /**
  * JSON Lazy Logger
@@ -53,10 +53,8 @@ const LEVEL_PRESETS = {
 
 type LevelPreset = keyof typeof LEVEL_PRESETS;
 
-import { Flag } from '../flag/flag';
-
-// Compact JSON mode (can be set at runtime, initialized from Flag)
-let compactJsonMode = Flag.COMPACT_JSON();
+// Compact JSON mode (can be set at runtime, initialized from config)
+let compactJsonMode = false; // Deferred to avoid circular init; config.compactJson checked at runtime
 
 /**
  * Set compact JSON output mode
@@ -99,8 +97,8 @@ function formatLogEntry(
     logEntry.message = String(data);
   }
 
-  // Check both local setting and global Flag
-  const useCompact = compactJsonMode || Flag.COMPACT_JSON();
+  // Check both local setting and global config
+  const useCompact = compactJsonMode || config.compactJson;
   return useCompact
     ? JSON.stringify(logEntry)
     : JSON.stringify(logEntry, null, 2);
@@ -158,7 +156,7 @@ export function createLazyLogger(
   initialTags?: Record<string, unknown>
 ): LazyLogger {
   // Determine initial log level based on verbose flag
-  const initialLevel = Flag.OPENCODE_VERBOSE ? LEVEL_VERBOSE : LEVEL_DISABLED;
+  const initialLevel = isVerbose() ? LEVEL_VERBOSE : LEVEL_DISABLED;
 
   // Create base log-lazy instance
   const baseLog = makeLog({ level: initialLevel });
@@ -281,7 +279,7 @@ export function createLazyLogger(
 
   // Configuration
   Object.defineProperty(wrappedLog, 'enabled', {
-    get: () => Flag.OPENCODE_VERBOSE,
+    get: () => isVerbose(),
     enumerable: true,
   });
 
@@ -296,10 +294,10 @@ export const lazyLog = createLazyLogger({ service: 'agent' });
 
 /**
  * Utility to update the global logger level at runtime
- * Call this after Flag.setVerbose() to sync the logger state
+ * Call this after setVerbose() to sync the logger state
  */
 export function syncLoggerWithVerboseFlag(): void {
-  if (Flag.OPENCODE_VERBOSE) {
+  if (isVerbose()) {
     lazyLog.setLevel('verbose');
   } else {
     lazyLog.setLevel('disabled');

package/src/util/log.ts

@@ -3,7 +3,7 @@ import fs from 'fs/promises';
 import { Global } from '../global';
 import z from 'zod';
 import makeLog, { levels } from 'log-lazy';
-import { Flag } from '../flag/flag';
+import { config, isVerbose } from '../config/config';
 
 /**
  * Logging module with JSON output and lazy evaluation support.
@@ -32,7 +32,7 @@ export namespace Log {
 
   let level: Level = 'INFO';
   let jsonOutput = false; // Whether to output JSON format (enabled in verbose mode)
-  let compactJsonOutput = Flag.COMPACT_JSON(); // Whether to use compact JSON (single line)
+  let compactJsonOutput = false; // Whether to use compact JSON (single line); deferred to avoid circular init
 
   function shouldLog(input: Level): boolean {
     return levelPriority[input] >= levelPriority[level];
@@ -109,8 +109,9 @@ export namespace Log {
     // Always use JSON output format for logs
     jsonOutput = true;
 
-    // Configure lazy logging level based on verbose flag
-    if (Flag.OPENCODE_VERBOSE || options.print) {
+    // Configure lazy logging level based on verbose flag (with env var fallback)
+    // See: https://github.com/link-assistant/agent/issues/227
+    if (isVerbose() || options.print) {
       // Enable all levels for lazy logging when verbose
       lazyLogInstance = makeLog({
         level: levels.debug | levels.info | levels.warn | levels.error,
@@ -208,8 +209,8 @@ export namespace Log {
     }
 
     // Use compact or pretty format based on configuration
-    // Check both local setting and global Flag
-    const useCompact = compactJsonOutput || Flag.COMPACT_JSON();
+    // Check both local setting and global config
+    const useCompact = compactJsonOutput || config.compactJson;
     return useCompact
       ? JSON.stringify(logEntry)
       : JSON.stringify(logEntry, null, 2);
@@ -371,10 +372,10 @@ export namespace Log {
 
   /**
    * Sync lazy logging with verbose flag at runtime
-   * Call after Flag.setVerbose() to update lazy logging state
+   * Call after setVerbose() to update lazy logging state
    */
   export function syncWithVerboseFlag(): void {
-    if (Flag.OPENCODE_VERBOSE) {
+    if (isVerbose()) {
       jsonOutput = true;
       // Use stdout for verbose output (following Unix conventions)
       write = (msg: any) => process.stdout.write(msg);