Summarize completedTx in tool calll response (#1985)

* WIP tx-analysis

* Add real implementation

* Remove test data and scripts

* Refine tx summary

* Remove long pointless README

* Remove accidental `mise.toml`

* Remove JournalMock from tests

* Remove unused analysis

* Format and lint

* PR feedback

Author: bfollington

packages/runner/src/builtins/llm-dialog.ts

@@ -20,6 +20,10 @@ import { ID, NAME, type Recipe, TYPE } from "../builder/types.ts";
 import type { Action } from "../scheduler.ts";
 import type { IRuntime } from "../runtime.ts";
 import type { IExtendedStorageTransaction } from "../storage/interface.ts";
+import {
+  debugTransactionWrites,
+  formatTransactionSummary,
+} from "../storage/transaction-summary.ts";
 import { parseLink } from "../link-utils.ts";
 // Avoid importing from @commontools/charm to prevent circular deps in tests
 
@@ -811,8 +815,13 @@ async function invokeToolCall(
         ...toolCall.input,
         result, // doesn't HAVE to be used, but can be
       }, (completedTx: IExtendedStorageTransaction) => {
-        resolve(result.withTx(completedTx).get()); // withTx likely superfluous
-      }); // TODO(bf): why any needed?
+        logger.info("Handler tx:", debugTransactionWrites(completedTx));
+
+        const summary = formatTransactionSummary(completedTx, space);
+        const value = result.withTx(completedTx).get();
+
+        resolve({ value, summary });
+      });
     } else {
       throw new Error("Tool has neither pattern nor handler");
     }

packages/runner/src/index.ts

@@ -19,6 +19,12 @@ export type {
   IExtendedStorageTransaction,
   MemorySpace,
 } from "./storage/interface.ts";
+export {
+  debugTransactionWrites,
+  formatTransactionSummary,
+  summarizeTransaction,
+  type TransactionSummary,
+} from "./storage/transaction-summary.ts";
 export { convertCellsToLinks, isCell, isStream } from "./cell.ts";
 export {
   getCellOrThrow,