wip v28

Author: geraldarthurdavis

packages/pipelines/asset-pack/src/__tests__/finish-delivery.test.ts

@@ -89,6 +89,18 @@ describe('finish pull-request delivery', () => {
       path: '.bitcode/asset-packs/run-123.md',
     });
     expect(exec.get('finish', 'deliveryPath')).toBe('.bitcode/asset-packs/run-123.md');
+    expect(exec.get('tools', 'vcs_create_branch:0')).toMatchObject({
+      tool: 'vcs_create_branch',
+      ok: true,
+    });
+    expect(exec.get('tools', 'vcs_create_or_update_file:1')).toMatchObject({
+      tool: 'vcs_create_or_update_file',
+      ok: true,
+    });
+    expect(exec.get('tools', 'vcs_create_pull_request:2')).toMatchObject({
+      tool: 'vcs_create_pull_request',
+      ok: true,
+    });
     expect(emitToolUsage).toHaveBeenCalledTimes(3);
     expect(emitToolUsage.mock.calls[1][3]).toMatchObject({
       path: '.bitcode/asset-packs/run-123.md',

packages/pipelines/asset-pack/src/agents/finish/deliver-asset-pack-to-destination-agent.ts

@@ -393,16 +393,30 @@ async function runDeliveryTool<T>(
   usedTools: Array<Record<string, unknown>>,
   additionalInputSummary: Record<string, unknown> = {},
 ): Promise<T> {
+  const toolIndex = usedTools.length;
+  const toolKey = `${toolName}:${toolIndex}`;
   const summarizedInput = summarizeToolInput(input, additionalInputSummary);
   try {
     const output = await run();
     const summarizedOutput = summarizeToolOutput(output);
     usedTools.push({ tool: toolName, input: summarizedInput, output: summarizedOutput });
+    execution.store?.('tools', toolKey, {
+      tool: toolName,
+      ok: true,
+      input: summarizedInput,
+      output: summarizedOutput,
+    });
     await emitToolUsage(execution, toolName, summarizedInput, summarizedOutput, null);
     return output;
   } catch (error) {
     const message = error instanceof Error ? error.message : String(error);
     usedTools.push({ tool: toolName, input: summarizedInput, error: { message } });
+    execution.store?.('tools', toolKey, {
+      tool: toolName,
+      ok: false,
+      input: summarizedInput,
+      error: { message },
+    });
     await emitToolUsage(execution, toolName, summarizedInput, null, message);
     throw error;
   }