@benzotti/jedi 0.1.26 → 0.1.28

package/action/workflow-template.yml

@@ -47,9 +47,20 @@ jobs:
       && contains(github.event.comment.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)
+      - 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"
+        env:
+          GH_TOKEN: ${{ github.token }}
+
       - uses: actions/checkout@v4
         with:
-          ref: ${{ github.head_ref || github.ref }}
+          ref: ${{ steps.pr.outputs.branch || github.head_ref || github.ref }}
 
       # Restore persisted Jedi state (learnings + codebase-index)
       # 1. Exact match for this branch
@@ -63,7 +74,7 @@ jobs:
             .jdi/framework/
             .jdi/config/
             .claude/
-          key: jedi-state-${{ github.repository }}-${{ github.head_ref || github.ref_name }}
+          key: jedi-state-${{ github.repository }}-${{ steps.pr.outputs.branch || github.head_ref || github.ref_name }}
           restore-keys: |
             jedi-state-${{ github.repository }}-main
 
@@ -107,20 +118,22 @@ 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)
       - name: Run Jedi
         run: |
-          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 }}"
-
-          ARGS="--repo ${{ github.repository }}"
+          ARGS="--repo $REPO"
           [ -n "$COMMENT_ID" ] && ARGS="$ARGS --comment-id $COMMENT_ID"
           [ -n "$PR_NUMBER" ] && ARGS="$ARGS --pr-number $PR_NUMBER"
           [ -n "$ISSUE_NUMBER" ] && ARGS="$ARGS --issue-number $ISSUE_NUMBER"
 
           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 }}
+          REPO: ${{ github.repository }}
           ANTHROPIC_API_KEY: ${{ secrets.ANTHROPIC_API_KEY }}
           CLICKUP_API_TOKEN: ${{ secrets.CLICKUP_API_TOKEN }}
           GH_TOKEN: ${{ github.token }}
@@ -135,7 +148,7 @@ jobs:
             .jdi/framework/
             .jdi/config/
             .claude/
-          key: jedi-state-${{ github.repository }}-${{ github.head_ref || github.ref_name }}-${{ github.run_id }}
+          key: jedi-state-${{ github.repository }}-${{ steps.pr.outputs.branch || github.head_ref || github.ref_name }}-${{ github.run_id }}
 
   # ── Promote learnings to main baseline when PRs merge ──
   promote-learnings:

package/dist/index.js

@@ -11169,8 +11169,9 @@ function buildConversationContext(thread, currentCommentId) {
   }
   const previousJediRuns = jediSegments.filter((c3) => c3.isJedi).length;
   const isFollowUp = previousJediRuns > 0;
+  const isPostImplementation = jediSegments.some((c3) => c3.isJedi && c3.body.includes("<sup>implement</sup>"));
   if (jediSegments.length === 0) {
-    return { history: "", previousJediRuns: 0, isFollowUp: false };
+    return { history: "", previousJediRuns: 0, isFollowUp: false, isPostImplementation: false };
   }
   const lines = ["## Previous Conversation", ""];
   for (const comment of jediSegments) {
@@ -11186,7 +11187,7 @@ function buildConversationContext(thread, currentCommentId) {
     lines.push("");
   }
   return { history: lines.join(`
-`), previousJediRuns, isFollowUp };
+`), previousJediRuns, isFollowUp, isPostImplementation };
 }
 var COMMAND_EMOJI = {
   plan: "\uD83D\uDD2E",
@@ -11321,13 +11322,15 @@ var actionCommand = defineCommand({
     const issueNumber = Number(args["pr-number"] ?? args["issue-number"] ?? 0);
     let conversationHistory = "";
     let isFollowUp = false;
+    let isPostImplementation = false;
     if (repo && issueNumber) {
       const thread = await fetchCommentThread(repo, issueNumber);
       const context = buildConversationContext(thread, commentId ?? 0);
       conversationHistory = context.history;
       isFollowUp = context.isFollowUp;
+      isPostImplementation = context.isPostImplementation;
       if (isFollowUp) {
-        consola.info(`Continuing conversation (${context.previousJediRuns} previous Jedi run(s))`);
+        consola.info(`Continuing conversation (${context.previousJediRuns} previous Jedi run(s))${isPostImplementation ? " [post-implementation]" : ""}`);
       }
     }
     const intent = parseComment(args.comment, isFollowUp);
@@ -11339,7 +11342,7 @@ var actionCommand = defineCommand({
     if (repo && commentId) {
       await reactToComment(repo, commentId, "eyes").catch(() => {});
     }
-    const commandLabel = intent.isApproval ? "plan" : intent.isFeedback ? "feedback" : intent.command;
+    const commandLabel = intent.isApproval ? "plan" : intent.isFeedback && isPostImplementation ? "implement" : intent.isFeedback ? "feedback" : intent.command;
     let placeholderCommentId = null;
     if (repo && issueNumber) {
       const thinkingBody = `<h3>\uD83E\uDDE0 Jedi <sup>thinking</sup></h3>
@@ -11445,6 +11448,31 @@ _Working on it..._`;
         `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) {
+      prompt2 = [
+        `Read ${baseProtocol} for the base agent protocol.`,
+        ``,
+        ...contextLines,
+        ``,
+        conversationHistory,
+        ``,
+        `## Feedback on Implementation`,
+        `> ${intent.description}`,
+        ``,
+        `## Instructions`,
+        `The user is iterating on code that Jedi already implemented. Review the conversation above to understand what was built.`,
+        `Be conversational \u2014 if the user asks a question, answer it first. Then make changes if needed.`,
+        `Apply changes incrementally to the existing code \u2014 do not rewrite from scratch.`,
+        ``,
+        `## Auto-Commit`,
+        `You are already on the correct PR branch. Do NOT create new branches or switch branches.`,
+        `After making changes:`,
+        `1. \`git add\` only source files you changed (NOT .jdi/ or .claude/)`,
+        `2. \`git commit -m "fix: ..."\` or \`git commit -m "refactor: ..."\` with a conventional commit message`,
+        `3. \`git push\` (no -u, no origin, no branch name \u2014 just \`git push\`)`,
+        `Present a summary of what you changed.`
+      ].join(`
 `);
     } else if (intent.isFeedback) {
       const agentSpec = resolve9(cwd, `.jdi/framework/agents/jdi-planner.md`);
@@ -11770,7 +11798,7 @@ var setupActionCommand = defineCommand({
 // package.json
 var package_default = {
   name: "@benzotti/jedi",
-  version: "0.1.26",
+  version: "0.1.28",
   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.26",
+  "version": "0.1.28",
   "description": "JDI - Context-efficient AI development framework for Claude Code",
   "type": "module",
   "bin": {