fix: fix: install_project_crons does not set PATH — claude not found in cron jobs (#366) #368

Merged

dev-bot merged 1 commit from fix/issue-366 into main

2026-04-07 17:44:02 +00:00

dev-bot commented

2026-04-07 17:39:37 +00:00

Collaborator

Fixes #366

Changes

Fixes #366 ## Changes

dev-bot added 1 commit 2026-04-07 17:39:37 +00:00

fix: fix: install_project_crons does not set PATH — claude not found in cron jobs (#366 )

ci/woodpecker/push/ci Pipeline was successful

Details

ci/woodpecker/pr/ci Pipeline was successful

Details

7e7fafd234

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

review-bot commented

2026-04-07 17:42:28 +00:00

Collaborator

AI Review

Summary\n\nAdds `PATH=/usr/local/bin:/usr/bin:/bin` as the first variable in the crontab environment block inside `install_project_crons`.\n\n### Assessment\n\n- Correct: Cron does not inherit the user shell PATH. Without this line, `claude` (volume-mounted at `/usr/local/bin/claude`) is not on cron's search path, causing all cron-launched agent scripts to fail.\n- Safe PATH value: `/usr/local/bin:/usr/bin:/bin` is a standard minimal PATH with no user-writable directories — no PATH-hijacking risk.\n- Correct placement: crontab environment variable lines must precede job lines; placing it first in the block is correct.\n- No regressions: The change only prepends one env line; all other variables and job lines are unchanged.\n\nChange is minimal, targeted, and correct.

Verdict

APPROVE — Correct minimal fix: adds PATH to cron environment so claude CLI at /usr/local/bin is found by cron jobs.

Reviewed at 7e7fafd | AGENTS.md

## AI Review  ### Summary\n\nAdds `PATH=/usr/local/bin:/usr/bin:/bin` as the first variable in the crontab environment block inside `install_project_crons`.\n\n### Assessment\n\n- **Correct**: Cron does not inherit the user shell PATH. Without this line, `claude` (volume-mounted at `/usr/local/bin/claude`) is not on cron's search path, causing all cron-launched agent scripts to fail.\n- **Safe PATH value**: `/usr/local/bin:/usr/bin:/bin` is a standard minimal PATH with no user-writable directories — no PATH-hijacking risk.\n- **Correct placement**: crontab environment variable lines must precede job lines; placing it first in the block is correct.\n- **No regressions**: The change only prepends one env line; all other variables and job lines are unchanged.\n\nChange is minimal, targeted, and correct. ### Verdict **APPROVE** — Correct minimal fix: adds PATH to cron environment so claude CLI at /usr/local/bin is found by cron jobs. --- *Reviewed at `7e7fafd` | [AGENTS.md](AGENTS.md)*

review-bot approved these changes 2026-04-07 17:42:28 +00:00

review-bot left a comment

Collaborator

AI Review: APPROVE — Correct minimal fix: adds PATH to cron environment so claude CLI at /usr/local/bin is found by cron jobs.

AI Review: **APPROVE** — Correct minimal fix: adds PATH to cron environment so claude CLI at /usr/local/bin is found by cron jobs.