@ravshansbox/pi-github 1.1.0 → 1.3.0

package/README.md

@@ -7,7 +7,8 @@ Watches open issues in a GitHub repo and answers new issues and comments with [p
 - Polls the repo of the current directory for new issues and comments every 30 seconds. Pull requests are ignored.
 - Only handles issues and comments created after the first start. After a restart it picks up where it stopped, including runs a crash cut off. A crash right after posting a reply can lead to that reply being posted twice.
 - Each issue gets its own git worktree in `.worktrees/issue-<number>` on branch `issue-<number>`, and its own pi session that continues across comments.
-- New issues and comments get an 👀 reaction as soon as they are picked up.
+- The terminal shows each issue's progress: every tool call the agent makes, failed tool calls, and when a branch is pushed or a pull request opened.
+- New issues and comments get an 👀 reaction as soon as they are picked up, then 🚀 once answered or 😕 if the run failed.
 - Issues are handled in parallel, up to 3 at once. Comments on the same issue are handled one after another.
 - The agent's final message is posted as a comment that starts with `agent:`. Comments that start with `agent:` are ignored, so it does not answer itself.
 - If the agent made commits, the branch is pushed and a pull request is opened once. Later commits update the same pull request.

package/dist/handler.js

@@ -1,12 +1,13 @@
 import { branchFor, ensureWorktree, hasUnpushedCommits, pushBranch } from "./git.js";
 import { agentPrefix } from "./poller.js";
-export async function handleTrigger(trigger, { github, agent, repoRoot }) {
+export async function handleTrigger(trigger, { github, agent, repoRoot, log = () => { }, }) {
     const base = await github.getDefaultBranch();
     const worktree = await ensureWorktree(repoRoot, trigger.issueNumber, base);
-    const answer = await agent.run(trigger, { cwd: worktree });
+    const answer = await agent.run(trigger, { cwd: worktree, progress: log });
     if (await hasUnpushedCommits(worktree, base)) {
-        await pushBranch(worktree);
         const branch = branchFor(trigger.issueNumber);
+        await pushBranch(worktree);
+        log(`pushed ${branch}`);
         if (!(await github.hasOpenPullRequest(branch))) {
             const issue = await github.getIssue(trigger.issueNumber);
             await github.createPullRequest({
@@ -15,6 +16,7 @@ export async function handleTrigger(trigger, { github, agent, repoRoot }) {
                 title: issue.title,
                 body: `${agentPrefix} Closes #${trigger.issueNumber}`,
             });
+            log("opened a pull request");
         }
     }
     await github.postComment(trigger.issueNumber, `${agentPrefix} ${answer}`);

package/dist/pi-agent.js

@@ -1,11 +1,18 @@
 import { join } from "node:path";
 import { createAgentSession, SessionManager } from "@earendil-works/pi-coding-agent";
+import { describeToolCall } from "./progress.js";
 import { buildPrompt } from "./prompt.js";
 export function createPiAgent({ github, repo, sessionsDir }) {
     return {
-        async run(trigger, { cwd }) {
+        async run(trigger, { cwd, progress }) {
             const sessionManager = SessionManager.continueRecent(cwd, join(sessionsDir, `issue-${trigger.issueNumber}`));
             const { session } = await createAgentSession({ cwd, sessionManager });
+            const unsubscribe = session.subscribe((event) => {
+                if (event.type === "tool_execution_start" && !event.parentToolCallId)
+                    progress(describeToolCall(event.toolName, event.args));
+                if (event.type === "tool_execution_end" && event.isError && !event.parentToolCallId)
+                    progress(`${event.toolName} failed`);
+            });
             try {
                 const issue = await github.getIssue(trigger.issueNumber);
                 const firstInSession = session.messages.length === 0;
@@ -13,6 +20,7 @@ export function createPiAgent({ github, repo, sessionsDir }) {
                 return session.getLastAssistantText() ?? "";
             }
             finally {
+                unsubscribe();
                 session.dispose();
             }
         },

package/dist/progress.js

@@ -0,0 +1,8 @@
+const maximumLength = 100;
+export function describeToolCall(toolName, args) {
+    const values = args !== null && typeof args === "object" ? args : {};
+    const detail = ["command", "path", "pattern", "query"].map((key) => values[key]).find((value) => typeof value === "string") ??
+        JSON.stringify(args ?? {});
+    const line = detail.split("\n")[0] ?? "";
+    return `${toolName}: ${line.length > maximumLength ? `${line.slice(0, maximumLength - 1)}…` : line}`;
+}

package/dist/watch.js

@@ -30,12 +30,20 @@ export async function watch({ github, agent, repoRoot, stateFile, intervalMs, co
     let { pending = [], ...pollState } = await loadState(stateFile);
     let saving = Promise.resolve();
     const save = () => (saving = saving.then(() => saveState(stateFile, { ...pollState, pending })));
+    const react = (trigger, content) => github
+        .addReaction(trigger, content)
+        .catch((error) => log(`#${trigger.issueNumber}: reaction failed: ${error instanceof Error ? error.message : error}`));
     const dispatcher = createDispatcher({
         concurrency,
         handle: async (trigger) => {
             try {
-                await handleTrigger(trigger, { github, agent, repoRoot });
+                await handleTrigger(trigger, { github, agent, repoRoot, log: (message) => log(`#${trigger.issueNumber}: ${message}`) });
                 log(`#${trigger.issueNumber}: answered @${trigger.author}`);
+                await react(trigger, "rocket");
+            }
+            catch (error) {
+                await react(trigger, "confused");
+                throw error;
             }
             finally {
                 pending = pending.filter((candidate) => candidate.key !== trigger.key);
@@ -57,7 +65,7 @@ export async function watch({ github, agent, repoRoot, stateFile, intervalMs, co
             for (const trigger of result.triggers) {
                 log(`#${trigger.issueNumber}: new ${trigger.kind} from @${trigger.author}`);
                 dispatcher.dispatch(trigger);
-                await github.addReaction(trigger, "eyes").catch((error) => log(`#${trigger.issueNumber}: reaction failed: ${error instanceof Error ? error.message : error}`));
+                await react(trigger, "eyes");
             }
         }
         catch (error) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ravshansbox/pi-github",
-  "version": "1.1.0",
+  "version": "1.3.0",
   "description": "Watch GitHub issues and answer them with pi agents",
   "type": "module",
   "license": "MIT",