@link-assistant/agent 0.5.3 → 0.6.1

package/src/project/state.ts

@@ -36,16 +36,20 @@ export namespace State {
     const entries = recordsByKey.get(key);
     if (!entries) return;
 
-    log.info('waiting for state disposal to complete', { key });
+    log.info(() => ({
+      message: 'waiting for state disposal to complete',
+      key,
+    }));
 
     let disposalFinished = false;
 
     setTimeout(() => {
       if (!disposalFinished) {
-        log.warn(
-          'state disposal is taking an unusually long time - if it does not complete in a reasonable time, please report this as a bug',
-          { key }
-        );
+        log.warn(() => ({
+          message:
+            'state disposal is taking an unusually long time - if it does not complete in a reasonable time, please report this as a bug',
+          key,
+        }));
       }
     }, 10000).unref();
 
@@ -56,7 +60,11 @@ export namespace State {
       const task = Promise.resolve(entry.state)
         .then((state) => entry.dispose!(state))
         .catch((error) => {
-          log.error('Error while disposing state:', { error, key });
+          log.error(() => ({
+            message: 'Error while disposing state',
+            error,
+            key,
+          }));
         });
 
       tasks.push(task);
@@ -64,6 +72,6 @@ export namespace State {
     await Promise.all(tasks);
     recordsByKey.delete(key);
     disposalFinished = true;
-    log.info('state disposal completed', { key });
+    log.info(() => ({ message: 'state disposal completed', key }));
   }
 }

package/src/provider/models.ts

@@ -78,18 +78,17 @@ export namespace ModelsDev {
 
   export async function refresh() {
     const file = Bun.file(filepath);
-    log.info('refreshing', {
-      file,
-    });
+    log.info(() => ({ message: 'refreshing', file }));
     const result = await fetch('https://models.dev/api.json', {
       headers: {
         'User-Agent': 'agent-cli/1.0.0',
       },
       signal: AbortSignal.timeout(10 * 1000),
     }).catch((e) => {
-      log.error('Failed to fetch models.dev', {
+      log.error(() => ({
+        message: 'Failed to fetch models.dev',
         error: e,
-      });
+      }));
     });
     if (result && result.ok) await Bun.write(file, await result.text());
   }

package/src/provider/provider.ts

@@ -37,7 +37,7 @@ export namespace Provider {
       // Check if OAuth credentials are available via the auth plugin
       const auth = await Auth.get('anthropic');
       if (auth?.type === 'oauth') {
-        log.info('using anthropic oauth credentials');
+        log.info(() => ({ message: 'using anthropic oauth credentials' }));
         const loaderFn = await AuthPlugins.getLoader('anthropic');
         if (loaderFn) {
           const result = await loaderFn(() => Auth.get('anthropic'), input);
@@ -330,7 +330,7 @@ export namespace Provider {
     google: async (input) => {
       const auth = await Auth.get('google');
       if (auth?.type === 'oauth') {
-        log.info('using google oauth credentials');
+        log.info(() => ({ message: 'using google oauth credentials' }));
         const loaderFn = await AuthPlugins.getLoader('google');
         if (loaderFn) {
           const result = await loaderFn(() => Auth.get('google'), input);
@@ -355,7 +355,9 @@ export namespace Provider {
     'github-copilot': async (input) => {
       const auth = await Auth.get('github-copilot');
       if (auth?.type === 'oauth') {
-        log.info('using github copilot oauth credentials');
+        log.info(() => ({
+          message: 'using github copilot oauth credentials',
+        }));
         const loaderFn = await AuthPlugins.getLoader('github-copilot');
         if (loaderFn) {
           const result = await loaderFn(
@@ -383,7 +385,9 @@ export namespace Provider {
     'github-copilot-enterprise': async (input) => {
       const auth = await Auth.get('github-copilot-enterprise');
       if (auth?.type === 'oauth') {
-        log.info('using github copilot enterprise oauth credentials');
+        log.info(() => ({
+          message: 'using github copilot enterprise oauth credentials',
+        }));
         const loaderFn = await AuthPlugins.getLoader('github-copilot');
         if (loaderFn) {
           const result = await loaderFn(
@@ -435,7 +439,10 @@ export namespace Provider {
         return { autoload: false };
       }
 
-      log.info('using claude oauth credentials', { source: tokenSource });
+      log.info(() => ({
+        message: 'using claude oauth credentials',
+        source: tokenSource,
+      }));
 
       // Create authenticated fetch with Bearer token and OAuth beta header
       const customFetch = ClaudeOAuth.createAuthenticatedFetch(oauthToken);
@@ -537,7 +544,7 @@ export namespace Provider {
     // Maps `${provider}/${key}` to the provider’s actual model ID for custom aliases.
     const realIdByKey = new Map<string, string>();
 
-    log.info('init');
+    log.info(() => ({ message: 'init' }));
 
     function mergeProvider(
       id: string,
@@ -783,7 +790,7 @@ export namespace Provider {
         delete providers[providerID];
         continue;
       }
-      log.info('found', { providerID });
+      log.info(() => ({ message: 'found', providerID }));
     }
 
     return {
@@ -818,19 +825,21 @@ export namespace Provider {
 
       let installedPath: string;
       if (!pkg.startsWith('file://')) {
-        log.info('installing provider package', {
+        log.info(() => ({
+          message: 'installing provider package',
           providerID: provider.id,
           pkg,
           version: 'latest',
-        });
+        }));
         installedPath = await BunProc.install(pkg, 'latest');
-        log.info('provider package installed successfully', {
+        log.info(() => ({
+          message: 'provider package installed successfully',
           providerID: provider.id,
           pkg,
           installedPath,
-        });
+        }));
       } else {
-        log.info('loading local provider', { pkg });
+        log.info(() => ({ message: 'loading local provider', pkg }));
         installedPath = pkg;
       }
 
@@ -876,13 +885,14 @@ export namespace Provider {
       s.sdk.set(key, loaded);
       return loaded as SDK;
     })().catch((e) => {
-      log.error('provider initialization failed', {
+      log.error(() => ({
+        message: 'provider initialization failed',
         providerID: provider.id,
         pkg: model.provider?.npm ?? provider.npm ?? provider.id,
         error: e instanceof Error ? e.message : String(e),
         stack: e instanceof Error ? e.stack : undefined,
         cause: e instanceof Error && e.cause ? String(e.cause) : undefined,
-      });
+      }));
       throw new InitError({ providerID: provider.id }, { cause: e });
     });
   }
@@ -896,10 +906,7 @@ export namespace Provider {
     const s = await state();
     if (s.models.has(key)) return s.models.get(key)!;
 
-    log.info('getModel', {
-      providerID,
-      modelID,
-    });
+    log.info(() => ({ message: 'getModel', providerID, modelID }));
 
     const provider = s.providers[providerID];
     if (!provider) throw new ModelNotFoundError({ providerID, modelID });
@@ -929,7 +936,7 @@ export namespace Provider {
           ? await provider.getModel(sdk, realID, provider.options)
           : sdk.languageModel(realID);
       }
-      log.info('found', { providerID, modelID });
+      log.info(() => ({ message: 'found', providerID, modelID }));
       s.models.set(key, {
         providerID,
         modelID,
@@ -1032,10 +1039,11 @@ export namespace Provider {
     if (opencodeProvider) {
       const [model] = sort(Object.values(opencodeProvider.info.models));
       if (model) {
-        log.info('using opencode provider as default', {
+        log.info(() => ({
+          message: 'using opencode provider as default',
           provider: opencodeProvider.info.id,
           model: model.id,
-        });
+        }));
         return {
           providerID: opencodeProvider.info.id,
           modelID: model.id,

package/src/server/server.ts

@@ -55,9 +55,7 @@ export namespace Server {
   export const App = lazy(() =>
     app
       .onError((err, c) => {
-        log.error('failed', {
-          error: err,
-        });
+        log.error(() => ({ message: 'failed', error: err }));
         if (err instanceof NamedError) {
           let status: ContentfulStatusCode;
           if (err instanceof Storage.NotFoundError) status = 404;
@@ -71,10 +69,11 @@ export namespace Server {
         });
       })
       .use(async (c, next) => {
-        log.info('request', {
+        log.info(() => ({
+          message: 'request',
           method: c.req.method,
           path: c.req.path,
-        });
+        }));
         const timer = log.time('request', {
           method: c.req.method,
           path: c.req.path,

package/src/session/agent.js

@@ -96,8 +96,22 @@ export class Agent {
             const errorTime = Date.now();
             const callID = `call_${Math.floor(Math.random() * 100000000)}`;
 
-            // Log full error to stderr for debugging
-            console.error('Tool execution error:', error);
+            // Log full error to stderr for debugging in JSON format
+            console.error(
+              JSON.stringify({
+                log: {
+                  level: 'error',
+                  timestamp: new Date().toISOString(),
+                  message: 'Tool execution error',
+                  tool: tool.name,
+                  error: {
+                    name: error.name,
+                    message: error.message,
+                    stack: error.stack,
+                  },
+                },
+              })
+            );
 
             // Emit tool_use event with error
             this.emitEvent('tool_use', {

package/src/session/compaction.ts

@@ -52,7 +52,7 @@ export namespace SessionCompaction {
   // tool calls that are no longer relevant.
   export async function prune(input: { sessionID: string }) {
     if (Flag.OPENCODE_DISABLE_PRUNE) return;
-    log.info('pruning');
+    log.info(() => ({ message: 'pruning' }));
     const msgs = await Session.messages({ sessionID: input.sessionID });
     let total = 0;
     let pruned = 0;
@@ -78,7 +78,7 @@ export namespace SessionCompaction {
           }
       }
     }
-    log.info('found', { pruned, total });
+    log.info(() => ({ message: 'found', pruned, total }));
     if (pruned > PRUNE_MINIMUM) {
       for (const part of toPrune) {
         if (part.state.status === 'completed') {
@@ -86,7 +86,7 @@ export namespace SessionCompaction {
           await Session.updatePart(part);
         }
       }
-      log.info('pruned', { count: toPrune.length });
+      log.info(() => ({ message: 'pruned', count: toPrune.length }));
     }
   }
 
@@ -139,9 +139,7 @@ export namespace SessionCompaction {
     const result = await processor.process(() =>
       streamText({
         onError(error) {
-          log.error('stream error', {
-            error,
-          });
+          log.error(() => ({ message: 'stream error', error }));
         },
         // set to 0, we handle loop
         maxRetries: 0,

package/src/session/index.ts

@@ -178,7 +178,7 @@ export namespace Session {
         updated: Date.now(),
       },
     };
-    log.info('created', result);
+    log.info(() => ({ message: 'created', ...result }));
     await Storage.write(['session', Instance.project.id, result.id], result);
     Bus.publish(Event.Created, {
       info: result,
@@ -273,7 +273,7 @@ export namespace Session {
         info: session,
       });
     } catch (e) {
-      log.error(e);
+      log.error(() => ({ error: e }));
     }
   });
 

package/src/session/processor.ts

@@ -39,7 +39,7 @@ export namespace SessionProcessor {
         return toolcalls[toolCallID];
       },
       async process(fn: () => StreamTextResult<Record<string, AITool>, never>) {
-        log.info('process');
+        log.info(() => ({ message: 'process' }));
         while (true) {
           try {
             let currentText: MessageV2.TextPart | undefined;
@@ -305,16 +305,12 @@ export namespace SessionProcessor {
                   break;
 
                 default:
-                  log.info('unhandled', {
-                    ...value,
-                  });
+                  log.info(() => ({ message: 'unhandled', ...value }));
                   continue;
               }
             }
           } catch (e) {
-            log.error('process', {
-              error: e,
-            });
+            log.error(() => ({ message: 'process', error: e }));
             const error = MessageV2.fromError(e, {
               providerID: input.providerID,
             });

package/src/session/prompt.ts

@@ -216,7 +216,7 @@ export namespace SessionPrompt {
   }
 
   export function cancel(sessionID: string) {
-    log.info('cancel', { sessionID });
+    log.info(() => ({ message: 'cancel', sessionID }));
     const s = state();
     const match = s[sessionID];
     if (!match) return;
@@ -242,7 +242,7 @@ export namespace SessionPrompt {
 
     let step = 0;
     while (true) {
-      log.info('loop', { step, sessionID });
+      log.info(() => ({ message: 'loop', step, sessionID }));
       if (abort.aborted) break;
       let msgs = await MessageV2.filterCompacted(MessageV2.stream(sessionID));
 
@@ -276,7 +276,7 @@ export namespace SessionPrompt {
         lastAssistant.finish !== 'tool-calls' &&
         lastUser.id < lastAssistant.id
       ) {
-        log.info('exiting loop', { sessionID });
+        log.info(() => ({ message: 'exiting loop', sessionID }));
         break;
       }
 
@@ -297,14 +297,13 @@ export namespace SessionPrompt {
           lastUser.model.modelID
         );
       } catch (error) {
-        log.warn(
-          'Failed to initialize specified model, falling back to default model',
-          {
-            providerID: lastUser.model.providerID,
-            modelID: lastUser.model.modelID,
-            error: error instanceof Error ? error.message : String(error),
-          }
-        );
+        log.warn(() => ({
+          message:
+            'Failed to initialize specified model, falling back to default model',
+          providerID: lastUser.model.providerID,
+          modelID: lastUser.model.modelID,
+          error: error instanceof Error ? error.message : String(error),
+        }));
         const defaultModel = await Provider.defaultModel();
         model = await Provider.getModel(
           defaultModel.providerID,
@@ -551,53 +550,79 @@ export namespace SessionPrompt {
         );
         const totalEstimatedTokens = systemTokens + userTokens;
 
-        log.info('=== VERBOSE: API Request Details ===');
-        log.info(`Model: ${model.providerID}/${model.modelID}`);
-        log.info(`Session ID: ${sessionID}`);
-        log.info(`Agent: ${agent.name}`);
-        log.info(`Temperature: ${params.temperature ?? 'default'}`);
-        log.info(`Top P: ${params.topP ?? 'default'}`);
-        log.info(
-          `Active Tools: ${Object.keys(tools)
-            .filter((x) => x !== 'invalid')
-            .join(', ')}`
-        );
-        log.info('--- System Prompt ---');
+        log.info(() => ({
+          message: '=== VERBOSE: API Request Details ===',
+        }));
+        log.info(() => ({
+          message: 'Model',
+          model: `${model.providerID}/${model.modelID}`,
+        }));
+        log.info(() => ({ message: 'Session ID', sessionID }));
+        log.info(() => ({ message: 'Agent', agent: agent.name }));
+        log.info(() => ({
+          message: 'Temperature',
+          temperature: params.temperature ?? 'default',
+        }));
+        log.info(() => ({
+          message: 'Top P',
+          topP: params.topP ?? 'default',
+        }));
+        log.info(() => ({
+          message: 'Active Tools',
+          tools: Object.keys(tools).filter((x) => x !== 'invalid'),
+        }));
+        log.info(() => ({ message: '--- System Prompt ---' }));
         for (let i = 0; i < system.length; i++) {
           const tokens = Token.estimate(system[i]);
-          log.info(`System Message ${i + 1} (${tokens} tokens estimated):`);
-          log.info(
-            system[i].slice(0, 2000) +
-              (system[i].length > 2000 ? '... [truncated]' : '')
-          );
+          log.info(() => ({
+            message: 'System Message',
+            index: i + 1,
+            tokens,
+          }));
+          log.info(() => ({
+            message: 'System Message Content',
+            content:
+              system[i].slice(0, 2000) +
+              (system[i].length > 2000 ? '... [truncated]' : ''),
+          }));
         }
-        log.info('--- Token Summary ---');
-        log.info(`System prompt tokens (estimated): ${systemTokens}`);
-        log.info(`User message tokens (estimated): ${userTokens}`);
-        log.info(`Total estimated tokens: ${totalEstimatedTokens}`);
-        log.info(
-          `Model context limit: ${model.info?.limit?.context || 'unknown'}`
-        );
-        log.info(
-          `Model output limit: ${model.info?.limit?.output || 'unknown'}`
-        );
-        log.info('=== END VERBOSE ===');
+        log.info(() => ({ message: '--- Token Summary ---' }));
+        log.info(() => ({
+          message: 'System prompt tokens (estimated)',
+          systemTokens,
+        }));
+        log.info(() => ({
+          message: 'User message tokens (estimated)',
+          userTokens,
+        }));
+        log.info(() => ({
+          message: 'Total estimated tokens',
+          totalEstimatedTokens,
+        }));
+        log.info(() => ({
+          message: 'Model context limit',
+          contextLimit: model.info?.limit?.context || 'unknown',
+        }));
+        log.info(() => ({
+          message: 'Model output limit',
+          outputLimit: model.info?.limit?.output || 'unknown',
+        }));
+        log.info(() => ({ message: '=== END VERBOSE ===' }));
       }
 
       const result = await processor.process(() =>
         streamText({
           onError(error) {
-            log.error('stream error', {
-              error,
-            });
+            log.error(() => ({ message: 'stream error', error }));
           },
           async experimental_repairToolCall(input) {
             const lower = input.toolCall.toolName.toLowerCase();
             if (lower !== input.toolCall.toolName && tools[lower]) {
-              log.info('repairing tool call', {
+              log.info(() => ({
+                message: 'repairing tool call',
                 tool: input.toolCall.toolName,
                 repaired: lower,
-              });
+              }));
               return {
                 ...input.toolCall,
                 toolName: lower,
@@ -945,7 +970,7 @@ export namespace SessionPrompt {
               }
               break;
             case 'file:':
-              log.info('file', { mime: part.mime });
+              log.info(() => ({ message: 'file', mime: part.mime }));
               // have to normalize, symbol search returns absolute paths
               // Decode the pathname since URL constructor doesn't automatically decode it
               const filepath = fileURLToPath(part.url);
@@ -1012,7 +1037,10 @@ export namespace SessionPrompt {
                     );
                   })
                   .catch((error) => {
-                    log.error('failed to read file', { error });
+                    log.error(() => ({
+                      message: 'failed to read file',
+                      error,
+                    }));
                     const message =
                       error instanceof Error ? error.message : error.toString();
                     Bus.publish(Session.Event.Error, {
@@ -1376,7 +1404,7 @@ export namespace SessionPrompt {
    */
 
   export async function command(input: CommandInput) {
-    log.info('command', input);
+    log.info(() => ({ message: 'command', ...input }));
     const command = await Command.get(input.command);
     const agentName = command.agent ?? input.agent ?? 'build';
 
@@ -1572,10 +1600,11 @@ export namespace SessionPrompt {
           });
       })
       .catch((error) => {
-        log.error('failed to generate title', {
+        log.error(() => ({
+          message: 'failed to generate title',
           error,
           model: small.info?.id ?? small.modelID,
-        });
+        }));
       });
   }
 }

package/src/session/revert.ts

@@ -72,7 +72,7 @@ export namespace SessionRevert {
   }
 
   export async function unrevert(input: { sessionID: string }) {
-    log.info('unreverting', input);
+    log.info(() => ({ message: 'unreverting', ...input }));
     SessionPrompt.assertNotBusy(input.sessionID);
     const session = await Session.get(input.sessionID);
     if (!session.revert) return session;

package/src/session/summary.ts

@@ -115,7 +115,7 @@ export namespace SessionSummary {
         headers: small.info.headers,
         model: small.language,
       });
-      log.info('title', { title: result.text });
+      log.info(() => ({ message: 'title', title: result.text }));
       userMsg.summary.title = result.text;
       await Session.updateMessage(userMsg);
     }
@@ -152,7 +152,7 @@ export namespace SessionSummary {
         if (result) summary = result.text;
       }
       userMsg.summary.body = summary;
-      log.info('body', { body: summary });
+      log.info(() => ({ message: 'body', body: summary }));
       await Session.updateMessage(userMsg);
     }
   }