@link-assistant/agent 0.5.3 → 0.6.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);
     }
   }

package/src/snapshot/index.ts

@@ -28,7 +28,7 @@ export namespace Snapshot {
       await $`git --git-dir ${git} config core.autocrlf false`
         .quiet()
         .nothrow();
-      log.info('initialized');
+      log.info(() => ({ message: 'initialized' }));
     }
     await $`git --git-dir ${git} --work-tree ${Instance.worktree} add .`
       .quiet()
@@ -40,7 +40,12 @@ export namespace Snapshot {
         .cwd(Instance.directory)
         .nothrow()
         .text();
-    log.info('tracking', { hash, cwd: Instance.directory, git });
+    log.info(() => ({
+      message: 'tracking',
+      hash,
+      cwd: Instance.directory,
+      git,
+    }));
     return hash.trim();
   }
 
@@ -64,7 +69,11 @@ export namespace Snapshot {
 
     // If git diff fails, return empty patch
     if (result.exitCode !== 0) {
-      log.warn('failed to get diff', { hash, exitCode: result.exitCode });
+      log.warn(() => ({
+        message: 'failed to get diff',
+        hash,
+        exitCode: result.exitCode,
+      }));
       return { hash, files: [] };
     }
 
@@ -81,7 +90,7 @@ export namespace Snapshot {
   }
 
   export async function restore(snapshot: string) {
-    log.info('restore', { commit: snapshot });
+    log.info(() => ({ message: 'restore', commit: snapshot }));
     const git = gitdir();
     const result =
       await $`git --git-dir ${git} --work-tree ${Instance.worktree} read-tree ${snapshot} && git --git-dir ${git} --work-tree ${Instance.worktree} checkout-index -a -f`
@@ -90,12 +99,13 @@ export namespace Snapshot {
         .nothrow();
 
     if (result.exitCode !== 0) {
-      log.error('failed to restore snapshot', {
+      log.error(() => ({
+        message: 'failed to restore snapshot',
         snapshot,
         exitCode: result.exitCode,
         stderr: result.stderr.toString(),
         stdout: result.stdout.toString(),
-      });
+      }));
     }
   }
 
@@ -105,7 +115,7 @@ export namespace Snapshot {
     for (const item of patches) {
       for (const file of item.files) {
         if (files.has(file)) continue;
-        log.info('reverting', { file, hash: item.hash });
+        log.info(() => ({ message: 'reverting', file, hash: item.hash }));
         const result =
           await $`git --git-dir ${git} --work-tree ${Instance.worktree} checkout ${item.hash} -- ${file}`
             .quiet()
@@ -119,11 +129,15 @@ export namespace Snapshot {
               .cwd(Instance.worktree)
               .nothrow();
           if (checkTree.exitCode === 0 && checkTree.text().trim()) {
-            log.info('file existed in snapshot but checkout failed, keeping', {
+            log.info(() => ({
+              message: 'file existed in snapshot but checkout failed, keeping',
               file,
-            });
+            }));
           } else {
-            log.info('file did not exist in snapshot, deleting', { file });
+            log.info(() => ({
+              message: 'file did not exist in snapshot, deleting',
+              file,
+            }));
             await fs.unlink(file).catch(() => {});
           }
         }
@@ -145,12 +159,13 @@ export namespace Snapshot {
         .nothrow();
 
     if (result.exitCode !== 0) {
-      log.warn('failed to get diff', {
+      log.warn(() => ({
+        message: 'failed to get diff',
         hash,
         exitCode: result.exitCode,
         stderr: result.stderr.toString(),
         stdout: result.stdout.toString(),
-      });
+      }));
       return '';
     }
 

package/src/storage/storage.ts

@@ -28,7 +28,7 @@ export namespace Storage {
         cwd: project,
         onlyFiles: false,
       })) {
-        log.info(`migrating project ${projectDir}`);
+        log.info(() => ({ message: 'migrating project', projectDir }));
         let projectID = projectDir;
         const fullProjectDir = path.join(project, projectDir);
         let worktree = '/';
@@ -74,7 +74,10 @@ export namespace Storage {
             })
           );
 
-          log.info(`migrating sessions for project ${projectID}`);
+          log.info(() => ({
+            message: 'migrating sessions for project',
+            projectID,
+          }));
           for await (const sessionFile of new Bun.Glob(
             'storage/session/info/*.json'
           ).scan({
@@ -87,13 +90,13 @@ export namespace Storage {
               projectID,
               path.basename(sessionFile)
             );
-            log.info('copying', {
-              sessionFile,
-              dest,
-            });
+            log.info(() => ({ message: 'copying', sessionFile, dest }));
             const session = await Bun.file(sessionFile).json();
             await Bun.write(dest, JSON.stringify(session));
-            log.info(`migrating messages for session ${session.id}`);
+            log.info(() => ({
+              message: 'migrating messages for session',
+              sessionID: session.id,
+            }));
             for await (const msgFile of new Bun.Glob(
               `storage/session/message/${session.id}/*.json`
             ).scan({
@@ -106,14 +109,14 @@ export namespace Storage {
                 session.id,
                 path.basename(msgFile)
               );
-              log.info('copying', {
-                msgFile,
-                dest,
-              });
+              log.info(() => ({ message: 'copying', msgFile, dest }));
               const message = await Bun.file(msgFile).json();
               await Bun.write(dest, JSON.stringify(message));
 
-              log.info(`migrating parts for message ${message.id}`);
+              log.info(() => ({
+                message: 'migrating parts for message',
+                messageID: message.id,
+              }));
               for await (const partFile of new Bun.Glob(
                 `storage/session/part/${session.id}/${message.id}/*.json`
               ).scan({
@@ -127,10 +130,7 @@ export namespace Storage {
                   path.basename(partFile)
                 );
                 const part = await Bun.file(partFile).json();
-                log.info('copying', {
-                  partFile,
-                  dest,
-                });
+                log.info(() => ({ message: 'copying', partFile, dest }));
                 await Bun.write(dest, JSON.stringify(part));
               }
             }
@@ -178,10 +178,10 @@ export namespace Storage {
       .then((x) => parseInt(x))
       .catch(() => 0);
     for (let index = migration; index < MIGRATIONS.length; index++) {
-      log.info('running migration', { index });
+      log.info(() => ({ message: 'running migration', index }));
       const migration = MIGRATIONS[index];
       await migration(dir).catch(() =>
-        log.error('failed to run migration', { index })
+        log.error(() => ({ message: 'failed to run migration', index }))
       );
       await Bun.write(path.join(dir, 'migration'), (index + 1).toString());
     }