@benzotti/jedi 0.1.28 → 0.1.29

package/action/workflow-template.yml

@@ -30,6 +30,8 @@ on:
     types: [created]
   pull_request_review_comment:
     types: [created]
+  pull_request_review:
+    types: [submitted]
   push:
     branches: [main, master]
 
@@ -40,23 +42,30 @@ permissions:
   id-token: write
 
 jobs:
-  # ── Triggered by "Hey Jedi" comments on issues/PRs ──
+  # ── Triggered by "Hey Jedi" comments on issues/PRs or PR reviews ──
   jedi:
     if: >-
-      (github.event_name == 'issue_comment' || github.event_name == 'pull_request_review_comment')
-      && contains(github.event.comment.body, 'Hey Jedi')
+      (github.event_name == 'issue_comment' && contains(github.event.comment.body, 'Hey Jedi'))
+      || (github.event_name == 'pull_request_review_comment' && contains(github.event.comment.body, 'Hey Jedi'))
+      || (github.event_name == 'pull_request_review' && contains(github.event.review.body, 'Hey Jedi'))
     runs-on: ubuntu-latest
     steps:
-      # Resolve the PR head branch for issue_comment events
-      # (github.head_ref is only set on pull_request events, not issue_comment)
+      # Resolve the PR head branch
+      # - issue_comment: github.head_ref is empty, resolve via API
+      # - pull_request_review / pull_request_review_comment: github.event.pull_request.head.ref is available
       - name: Resolve PR branch
         id: pr
-        if: github.event.issue.pull_request
         run: |
-          PR_DATA=$(gh api repos/${{ github.repository }}/pulls/${{ github.event.issue.number }} --jq '{head_ref: .head.ref, head_sha: .head.sha}')
-          echo "branch=$(echo "$PR_DATA" | jq -r .head_ref)" >> "$GITHUB_OUTPUT"
+          if [ -n "$PR_HEAD_REF" ]; then
+            echo "branch=$PR_HEAD_REF" >> "$GITHUB_OUTPUT"
+          elif [ -n "$PR_NUMBER" ]; then
+            BRANCH=$(gh api repos/${{ github.repository }}/pulls/$PR_NUMBER --jq '.head.ref')
+            echo "branch=$BRANCH" >> "$GITHUB_OUTPUT"
+          fi
         env:
           GH_TOKEN: ${{ github.token }}
+          PR_HEAD_REF: ${{ github.event.pull_request.head.ref || '' }}
+          PR_NUMBER: ${{ github.event.issue.number || github.event.pull_request.number || '' }}
 
       - uses: actions/checkout@v4
         with:
@@ -118,8 +127,7 @@ jobs:
         run: bun install -g @anthropic-ai/claude-code
 
       # Run Jedi CLI directly — full pipeline with routing, agents, and comment posting
-      # IMPORTANT: Comment body is passed via env var to avoid shell injection
-      # (backticks, $(), etc. in comments would be executed as commands if interpolated inline)
+      # IMPORTANT: All user input is passed via env vars to avoid shell injection
       - name: Run Jedi
         run: |
           ARGS="--repo $REPO"
@@ -129,10 +137,11 @@ jobs:
 
           bunx @benzotti/jedi@latest action "$COMMENT_BODY" $ARGS
         env:
-          COMMENT_BODY: ${{ github.event.comment.body }}
-          COMMENT_ID: ${{ github.event.comment.id }}
-          PR_NUMBER: ${{ github.event.issue.pull_request && github.event.issue.number || '' }}
-          ISSUE_NUMBER: ${{ github.event.issue.number }}
+          # Resolve comment body from whichever event type triggered this
+          COMMENT_BODY: ${{ github.event.comment.body || github.event.review.body || '' }}
+          COMMENT_ID: ${{ github.event.comment.id || github.event.review.id || '' }}
+          PR_NUMBER: ${{ github.event.issue.number || github.event.pull_request.number || '' }}
+          ISSUE_NUMBER: ${{ github.event.issue.number || github.event.pull_request.number || '' }}
           REPO: ${{ github.repository }}
           ANTHROPIC_API_KEY: ${{ secrets.ANTHROPIC_API_KEY }}
           CLICKUP_API_TOKEN: ${{ secrets.CLICKUP_API_TOKEN }}

package/dist/index.js

@@ -11352,6 +11352,49 @@ var actionCommand = defineCommand({
 _Working on it..._`;
       placeholderCommentId = await postGitHubComment(repo, issueNumber, thinkingBody).catch(() => null);
     }
+    if (intent.isFeedback && intent.isApproval) {
+      const { existsSync: existsSync13 } = await import("fs");
+      const { join: join12 } = await import("path");
+      const statePath = join12(cwd, ".jdi/config/state.yaml");
+      if (existsSync13(statePath)) {
+        const stateContent = await Bun.file(statePath).text();
+        const now = new Date().toISOString();
+        let updated = stateContent;
+        if (/review\.status:/.test(updated)) {
+          updated = updated.replace(/review\.status:\s*.+/, `review.status: approved`);
+        } else {
+          updated += `
+review.status: approved
+`;
+        }
+        if (/review\.approved_at:/.test(updated)) {
+          updated = updated.replace(/review\.approved_at:\s*.+/, `review.approved_at: ${now}`);
+        } else {
+          updated += `review.approved_at: ${now}
+`;
+        }
+        await Bun.write(statePath, updated);
+      }
+      const approvalBody = `Plan approved and locked in.
+
+Say **\`Hey Jedi implement\`** when you're ready to go.`;
+      const finalBody = formatJediComment("plan", approvalBody);
+      if (repo && placeholderCommentId) {
+        await updateGitHubComment(repo, placeholderCommentId, finalBody).catch((err) => {
+          consola.error("Failed to update approval comment:", err);
+        });
+      } else if (repo && issueNumber) {
+        await postGitHubComment(repo, issueNumber, finalBody).catch((err) => {
+          consola.error("Failed to post approval comment:", err);
+        });
+      } else {
+        console.log(finalBody);
+      }
+      if (repo && commentId) {
+        await reactToComment(repo, commentId, "+1").catch(() => {});
+      }
+      return;
+    }
     if (intent.command === "ping") {
       const { existsSync: existsSync13 } = await import("fs");
       const { join: join12 } = await import("path");
@@ -11430,26 +11473,7 @@ _Working on it..._`;
     }
     const baseProtocol = resolve9(cwd, ".jdi/framework/components/meta/AgentBase.md");
     let prompt2;
-    if (intent.isFeedback && intent.isApproval) {
-      prompt2 = [
-        `Read ${baseProtocol} for the base agent protocol.`,
-        ``,
-        ...contextLines,
-        ``,
-        conversationHistory,
-        ``,
-        `## Plan Approved`,
-        `> ${intent.description}`,
-        ``,
-        `The user has approved the plan. Finalise it:`,
-        `1. Read \`.jdi/config/state.yaml\` and update \`review.status\` to "approved" and \`review.approved_at\` to now`,
-        `2. Do NOT implement any code \u2014 approval and implementation are separate gates`,
-        ``,
-        `Respond with a short confirmation that the plan is approved and ready, then ask:`,
-        `"Say **implement** when you're ready to go."`
-      ].join(`
-`);
-    } else if (intent.isFeedback && isPostImplementation) {
+    if (intent.isFeedback && isPostImplementation) {
       prompt2 = [
         `Read ${baseProtocol} for the base agent protocol.`,
         ``,
@@ -11709,7 +11733,7 @@ Use the ClickUp ticket above as the primary requirements source.` : ``,
     if (saved.codebaseIndexSaved)
       consola.info("Codebase index persisted to storage");
     if (repo && issueNumber) {
-      const actionLabel = intent.isApproval ? "implement" : intent.isFeedback ? "feedback" : intent.command;
+      const actionLabel = intent.isFeedback ? "feedback" : intent.command;
       let commentBody;
       if (success && fullResponse) {
         commentBody = formatJediComment(actionLabel, fullResponse);
@@ -11798,7 +11822,7 @@ var setupActionCommand = defineCommand({
 // package.json
 var package_default = {
   name: "@benzotti/jedi",
-  version: "0.1.28",
+  version: "0.1.29",
   description: "JDI - Context-efficient AI development framework for Claude Code",
   type: "module",
   bin: {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@benzotti/jedi",
-  "version": "0.1.28",
+  "version": "0.1.29",
   "description": "JDI - Context-efficient AI development framework for Claude Code",
   "type": "module",
   "bin": {