@armstrongnate/april 0.0.3 → 0.0.4

package/README.md

@@ -8,7 +8,7 @@ april watches for GitHub issues assigned to you with a specific label, then spin
 
 - [Node.js](https://nodejs.org/) >= 22
 - [gh](https://cli.github.com/) (authenticated)
-- The `gh-webhook` extension: `gh extension install cli/gh-webhook`
+- The [`gh-webhook` extension](https://github.com/cli/gh-webhook): `gh extension install cli/gh-webhook`
 - [tmux](https://github.com/tmux/tmux)
 - [Claude Code](https://claude.ai/claude-code) CLI
 

package/config.example.yaml

@@ -2,10 +2,7 @@ assignee: "your-github-username"
 label: "agent:todo"
 claudeSkill: "issue-worker"
 # claudeModel: "opus"              # optional, defaults to opus
-# claudeAllowedTools:              # optional, defaults to Edit, Write, Bash(*)
-#   - "Edit"
-#   - "Write"
-#   - "Bash(*)"
+# claudePermissionMode: "auto"     # optional, defaults to auto (others: default, acceptEdits, plan, bypassPermissions)
 port: 7890
 repos:
   - owner: "org"

package/dist/config.js

@@ -63,9 +63,7 @@ export function loadConfig() {
     const label = validateString(parsed, "label", "config");
     const claudeSkill = validateString(parsed, "claudeSkill", "config");
     const claudeModel = typeof parsed.claudeModel === "string" ? parsed.claudeModel.trim() : undefined;
-    const claudeAllowedTools = Array.isArray(parsed.claudeAllowedTools)
-        ? parsed.claudeAllowedTools.filter((t) => typeof t === "string" && t.trim().length > 0).map((t) => t.trim())
-        : undefined;
+    const claudePermissionMode = typeof parsed.claudePermissionMode === "string" ? parsed.claudePermissionMode.trim() : undefined;
     const port = Number(parsed.port);
     if (!Number.isInteger(port) || port < 1024 || port > 65535) {
         throw new Error(`config: "port" must be an integer between 1024 and 65535, got: ${parsed.port}`);
@@ -96,7 +94,7 @@ export function loadConfig() {
             : undefined;
         return { owner, name, path: resolvedPath, defaultBranch, slackChannel, postWorktreeHook };
     });
-    const config = { assignee, label, claudeSkill, claudeModel, claudeAllowedTools, port, repos };
+    const config = { assignee, label, claudeSkill, claudeModel, claudePermissionMode, port, repos };
     log.info(`Config loaded: assignee=${assignee}, label=${label}, repos=${repos.map((r) => `${r.owner}/${r.name}`).join(", ")}`);
     return config;
 }

package/dist/processes.js

@@ -30,7 +30,6 @@ function cleanupStaleWebhooks(repoKey) {
 const INITIAL_BACKOFF_MS = 1000;
 const MAX_BACKOFF_MS = 30_000;
 const UPTIME_RESET_MS = 60_000;
-const MAX_CONSECUTIVE_FAILURES = 5;
 const forwarders = [];
 function spawnForwarder(config, repoKey, url) {
     const state = {
@@ -82,12 +81,8 @@ function spawnForwarder(config, repoKey, url) {
                 state.consecutiveFailures++;
                 state.backoffMs = Math.min(state.backoffMs * 2, MAX_BACKOFF_MS);
             }
-            if (state.consecutiveFailures >= MAX_CONSECUTIVE_FAILURES) {
-                log.error(`Forwarder for ${repoKey} failed ${MAX_CONSECUTIVE_FAILURES} consecutive times, giving up`);
-                return;
-            }
             log.warn(`Forwarder for ${repoKey} exited (code=${code}, signal=${signal}), ` +
-                `restarting in ${state.backoffMs}ms (failure ${state.consecutiveFailures}/${MAX_CONSECUTIVE_FAILURES})`);
+                `restarting in ${state.backoffMs}ms (consecutive failures: ${state.consecutiveFailures})`);
             setTimeout(() => start(), state.backoffMs);
         });
     }

package/dist/spawner.js

@@ -149,15 +149,11 @@ export function spawnClaude(config, repo, issue, worktreePath, sessionName) {
         // Session does not exist, proceed
     }
     const model = config.claudeModel || "opus";
-    const allowedTools = [
-        ...(config.claudeAllowedTools ?? ["Read", "Search", "Edit", "Write", "Bash(*)"]),
-        ...(repo.slackChannel ? ["mcp__plugin_slack_slack__*"] : []),
-    ];
+    const permissionMode = config.claudePermissionMode || "auto";
     const slackPart = repo.slackChannel ? ` Post the PR to Slack channel #${repo.slackChannel}.` : "";
     const prompt = `/${config.claudeSkill} Read GitHub issue #${issue.number} on ${repo.owner}/${repo.name} using the gh CLI. Implement it and open a PR.${slackPart}`;
     log.debug(`Prompt: ${prompt}`);
-    const allowedToolsArgs = allowedTools.map((t) => `--allowedTools '${t}'`).join(" ");
-    const claudeCommand = `claude --model ${model} ${allowedToolsArgs}`;
+    const claudeCommand = `claude --model ${model} --permission-mode ${permissionMode}`;
     log.info(`Spawning tmux session "${sessionName}" with claude`);
     execSync(`tmux new-session -d -s ${JSON.stringify(sessionName)} -c ${JSON.stringify(worktreePath)} ${JSON.stringify(claudeCommand)}`);
     // Send the prompt via send-keys after Claude starts

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@armstrongnate/april",
-  "version": "0.0.3",
+  "version": "0.0.4",
   "description": "She does all the work so you don't have to. Watches GitHub issues and spawns Claude Code sessions to work them.",
   "type": "module",
   "bin": {

package/skills/issue-worker/SKILL.md

@@ -1,11 +1,11 @@
 ---
 name: issue-worker
-description: Autonomously work a GitHub issue end-to-end — read, implement, and open a PR with no human input required.
+description: Autonomously work a GitHub issue end-to-end — read, implement, open a PR, and monitor CI/review feedback.
 ---
 
 # issue-worker
 
-You have been assigned a GitHub issue. Work it to completion autonomously. Do not stop to ask for approval or confirmation — go straight from reading the issue to opening a PR.
+You have been assigned a GitHub issue. Work it to completion autonomously. Do not stop to ask for approval or confirmation — go straight from reading the issue to opening a PR, then monitor and respond to CI failures and review feedback.
 
 ## 1. Read the issue
 
@@ -25,16 +25,17 @@ gh issue view {issue_number} --repo {owner}/{repo} --comments
 - Write or update tests as appropriate
 - Ensure the code builds/lints/passes tests
 
-## 4. Review and simplify
+## 4. Review
 
-Run `git diff` and review your own changes. Fix any issues before committing.
+Run `/simplify` to review your changes for reuse, quality, and efficiency, and fix anything it surfaces.
 
-- **Correctness:** Does it fully address the issue? Missing edge cases?
-- **Simplify:** Can anything be combined, inlined, or removed? Prefer fewer files, less indirection, and no unnecessary abstractions.
-- **Reuse:** Are you duplicating logic that already exists in the codebase? Use existing helpers and patterns.
-- **Cleanup:** Remove leftover debug code, TODOs, unused imports, and dead code.
-- **Style:** Match the conventions of the surrounding code.
-- **Tests:** Are the tests meaningful, not just testing mocks?
+Then do a quick manual pass on things `/simplify` won't catch:
+
+- **Correctness:** Does the change fully address the issue? Any missing edge cases?
+- **Tests:** Are the tests meaningful, or are they just asserting on mocks?
+- **Cleanup:** Any leftover debug code, TODOs, or commented-out lines?
+
+Fix anything you find before moving on.
 
 ## 5. Commit, push, and open a PR
 
@@ -42,12 +43,28 @@ Run `git diff` and review your own changes. Fix any issues before committing.
 gh pr create --title "..." --body "..."
 ```
 
-Then update the issue labels:
+## 6. Post to Slack (if instructed)
+
+If the prompt specifies a Slack channel, use the Slack MCP tool to post a message with a link to the PR. Format: `<pr_url|PR> repo-name: title of the pr`
+
+## 7. Monitor CI and review feedback
+
+After creating the PR, monitor it until all checks pass and all review feedback is addressed.
+
+Loop:
+1. Sleep for 3 minutes (`sleep 180`)
+2. Check CI status: `gh pr checks {pr_number} --repo {owner}/{repo}`
+3. If any checks failed, read the failure logs, fix the issue, commit, and push
+4. Check for review comments: `gh pr view {pr_number} --repo {owner}/{repo} --comments`
+5. If there are new or unresolved comments, address them, commit, and push
+6. Repeat from step 1
+
+Stop when:
+- All CI checks pass AND
+- No unresolved review comments remain
+
+Once everything is green, update the issue labels:
 
 ```
 gh issue edit {issue_number} --repo {owner}/{repo} --add-label agent:review --remove-label agent:wip
 ```
-
-## 6. Post to Slack (if instructed)
-
-If the prompt specifies a Slack channel, use the Slack MCP tool to post a message with a link to the PR. Format: `<pr_url|PR> title of the pr`