@link-assistant/agent 0.16.13 → 0.16.16

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@link-assistant/agent",
-  "version": "0.16.13",
+  "version": "0.16.16",
   "description": "A minimal, public domain AI CLI agent compatible with OpenCode's JSON interface. Bun-only runtime.",
   "main": "src/index.js",
   "type": "module",

package/src/cli/defaults.ts

@@ -0,0 +1,15 @@
+/**
+ * Default CLI configuration values.
+ *
+ * Centralizing defaults here ensures all code references the same value (#208).
+ * When the default model changes, update this file only.
+ */
+
+/** Default model used when no `--model` CLI argument is provided. */
+export const DEFAULT_MODEL = 'opencode/minimax-m2.5-free';
+
+/** Default provider ID extracted from DEFAULT_MODEL. */
+export const DEFAULT_PROVIDER_ID = DEFAULT_MODEL.split('/')[0];
+
+/** Default model ID extracted from DEFAULT_MODEL. */
+export const DEFAULT_MODEL_ID = DEFAULT_MODEL.split('/').slice(1).join('/');

package/src/cli/model-config.js

@@ -1,5 +1,6 @@
 import { getModelFromProcessArgv } from './argv.ts';
 import { Log } from '../util/log.ts';
+import { DEFAULT_PROVIDER_ID, DEFAULT_MODEL_ID } from './defaults.ts';
 
 /**
  * Parse model config from argv. Supports "provider/model" or short "model" format.
@@ -16,7 +17,7 @@ export async function parseModelConfig(argv, outputError, outputStatus) {
   let modelArg = argv.model;
 
   // ALWAYS prefer the CLI value over yargs when available (#196)
-  // The yargs default 'opencode/kimi-k2.5-free' can silently override user's --model argument
+  // The yargs default (DEFAULT_MODEL) can silently override user's --model argument
   if (cliModelArg) {
     if (cliModelArg !== modelArg) {
       Log.Default.warn(() => ({
@@ -45,7 +46,7 @@ export async function parseModelConfig(argv, outputError, outputStatus) {
     // Do NOT fall back to defaults - if the user provided an invalid format, fail clearly (#196)
     if (!providerID || !modelID) {
       throw new Error(
-        `Invalid model format: "${modelArg}". Expected "provider/model" format (e.g., "opencode/kimi-k2.5-free"). ` +
+        `Invalid model format: "${modelArg}". Expected "provider/model" format (e.g., "${DEFAULT_PROVIDER_ID}/${DEFAULT_MODEL_ID}"). ` +
           `Provider: "${providerID || '(empty)'}", Model: "${modelID || '(empty)'}".`
       );
     }
@@ -124,9 +125,9 @@ export async function parseModelConfig(argv, outputError, outputStatus) {
     // Set environment variable for the provider to use
     process.env.CLAUDE_CODE_OAUTH_TOKEN = creds.accessToken;
 
-    // If user specified the default model (opencode/kimi-k2.5-free), switch to claude-oauth
+    // If user specified the default model (DEFAULT_MODEL), switch to claude-oauth
     // If user explicitly specified kilo or another provider, warn but respect their choice
-    if (providerID === 'opencode' && modelID === 'kimi-k2.5-free') {
+    if (providerID === DEFAULT_PROVIDER_ID && modelID === DEFAULT_MODEL_ID) {
       providerID = 'claude-oauth';
       modelID = 'claude-sonnet-4-5';
     } else if (!['claude-oauth', 'anthropic'].includes(providerID)) {

package/src/index.js

@@ -6,6 +6,7 @@ import { Server } from './server/server.ts';
 import { Instance } from './project/instance.ts';
 import { Log } from './util/log.ts';
 import { parseModelConfig } from './cli/model-config.js';
+import { DEFAULT_MODEL } from './cli/defaults.ts';
 // Bus is used via createBusEventSubscription in event-handler.js
 import { Session } from './session/index.ts';
 import { SessionPrompt } from './session/prompt.ts';
@@ -588,7 +589,7 @@ async function main() {
             .option('model', {
               type: 'string',
               description: 'Model to use in format providerID/modelID',
-              default: 'opencode/kimi-k2.5-free',
+              default: DEFAULT_MODEL,
             })
             .option('json-standard', {
               type: 'string',

package/src/provider/provider.ts

@@ -1272,22 +1272,26 @@ export namespace Provider {
             }
           }
 
-          log.info(() => ({
-            message: 'HTTP request',
+          // Use direct (non-lazy) logging for HTTP request/response to ensure output
+          // is not lost when piped through external process wrappers (e.g., solve.mjs).
+          // The verbose check is already done above, so lazy evaluation is not needed here.
+          // See: https://github.com/link-assistant/agent/issues/211
+          log.info('HTTP request', {
             providerID: provider.id,
             method,
             url,
             headers: sanitizedHeaders,
             bodyPreview,
-          }));
+          });
 
           const startMs = Date.now();
           try {
             const response = await innerFetch(input, init);
             const durationMs = Date.now() - startMs;
 
-            log.info(() => ({
-              message: 'HTTP response',
+            // Use direct (non-lazy) logging to ensure HTTP response details are captured
+            // See: https://github.com/link-assistant/agent/issues/211
+            log.info('HTTP response', {
               providerID: provider.id,
               method,
               url,
@@ -1295,7 +1299,7 @@ export namespace Provider {
               statusText: response.statusText,
               durationMs,
               responseHeaders: Object.fromEntries(response.headers.entries()),
-            }));
+            });
 
             // Log response body for debugging provider failures
             // For streaming responses (SSE/event-stream), tee() the stream so the AI SDK
@@ -1335,14 +1339,15 @@ export namespace Provider {
                         }
                       }
                     }
-                    log.info(() => ({
-                      message: 'HTTP response body (stream)',
+                    // Use direct (non-lazy) logging for stream body
+                    // See: https://github.com/link-assistant/agent/issues/211
+                    log.info('HTTP response body (stream)', {
                       providerID: provider.id,
                       url,
                       bodyPreview: truncated
                         ? bodyPreview + `... [truncated]`
                         : bodyPreview,
-                    }));
+                    });
                   } catch {
                     // Ignore logging errors — do not affect the SDK stream
                   }
@@ -1362,12 +1367,13 @@ export namespace Provider {
                     ? bodyText.slice(0, responseBodyMaxChars) +
                       `... [truncated, total ${bodyText.length} chars]`
                     : bodyText;
-                log.info(() => ({
-                  message: 'HTTP response body',
+                // Use direct (non-lazy) logging for non-streaming body
+                // See: https://github.com/link-assistant/agent/issues/211
+                log.info('HTTP response body', {
                   providerID: provider.id,
                   url,
                   bodyPreview,
-                }));
+                });
                 return new Response(bodyText, {
                   status: response.status,
                   statusText: response.statusText,
@@ -1379,8 +1385,9 @@ export namespace Provider {
             return response;
           } catch (error) {
             const durationMs = Date.now() - startMs;
-            log.error(() => ({
-              message: 'HTTP request failed',
+            // Use direct (non-lazy) logging for error path
+            // See: https://github.com/link-assistant/agent/issues/211
+            log.error('HTTP request failed', {
               providerID: provider.id,
               method,
               url,
@@ -1389,7 +1396,7 @@ export namespace Provider {
                 error instanceof Error
                   ? { name: error.name, message: error.message }
                   : String(error),
-            }));
+            });
             throw error;
           }
         };

package/src/session/index.ts

@@ -630,9 +630,14 @@ export namespace Session {
           }
         | undefined;
 
+      // Check if standard usage has valid data (inputTokens or outputTokens defined)
+      // Also check for zero-valued tokens (some providers set them to 0 instead of undefined)
       const standardUsageIsEmpty =
-        input.usage.inputTokens === undefined &&
-        input.usage.outputTokens === undefined;
+        (input.usage.inputTokens === undefined &&
+          input.usage.outputTokens === undefined) ||
+        (input.usage.inputTokens === 0 &&
+          input.usage.outputTokens === 0 &&
+          !input.usage.totalTokens);
 
       // If standard usage is empty but openrouter metadata has usage, use it as source
       let effectiveUsage = input.usage;
@@ -658,6 +663,41 @@ export namespace Session {
         };
       }
 
+      // If still empty, try providerMetadata.anthropic.usage as fallback
+      // Some providers (e.g., opencode using @ai-sdk/anthropic) return usage in
+      // Anthropic-specific metadata with snake_case keys (input_tokens, output_tokens)
+      // while the standard AI SDK usage object remains empty.
+      // See: https://github.com/link-assistant/agent/issues/211
+      if (standardUsageIsEmpty && !openrouterUsage) {
+        const anthropicUsage = input.metadata?.['anthropic']?.['usage'] as
+          | {
+              input_tokens?: number;
+              output_tokens?: number;
+              cache_creation_input_tokens?: number;
+              cache_read_input_tokens?: number;
+            }
+          | undefined;
+
+        if (
+          anthropicUsage &&
+          (anthropicUsage.input_tokens || anthropicUsage.output_tokens)
+        ) {
+          if (Flag.OPENCODE_VERBOSE) {
+            log.debug(() => ({
+              message:
+                'Standard usage empty, falling back to anthropic provider metadata',
+              anthropicUsage: JSON.stringify(anthropicUsage),
+            }));
+          }
+          effectiveUsage = {
+            ...input.usage,
+            inputTokens: anthropicUsage.input_tokens ?? 0,
+            outputTokens: anthropicUsage.output_tokens ?? 0,
+            cachedInputTokens: anthropicUsage.cache_read_input_tokens ?? 0,
+          };
+        }
+      }
+
       // Extract top-level cachedInputTokens
       const topLevelCachedInputTokens = safeNum(
         toNumber(effectiveUsage.cachedInputTokens, 'cachedInputTokens')

package/src/tool/task.ts

@@ -9,6 +9,7 @@ import { Agent } from '../agent/agent';
 import { SessionPrompt } from '../session/prompt';
 import { iife } from '../util/iife';
 import { defer } from '../util/defer';
+import { DEFAULT_PROVIDER_ID, DEFAULT_MODEL_ID } from '../cli/defaults';
 
 export const TaskTool = Tool.define('task', async () => {
   const agents = await Agent.list().then((x) =>
@@ -99,8 +100,8 @@ export const TaskTool = Tool.define('task', async () => {
 
       const model = agent.model ??
         parentModel ?? {
-          modelID: 'minimax-m2.5-free',
-          providerID: 'opencode',
+          modelID: DEFAULT_MODEL_ID,
+          providerID: DEFAULT_PROVIDER_ID,
         };
 
       function cancel() {