AGENTS.md: carve out autonomous-runner exception to the no-push rule#1598
AGENTS.md: carve out autonomous-runner exception to the no-push rule#1598ChenhanYu wants to merge 1 commit into
Conversation
The "Never `git push` without explicit approval" rule was written for interactive Claude Code / codex sessions where a human is in the loop. It's been blocking pensieve-intern's autonomous AgentLoop runs: the runner SPEC instructs the cell agent to commit, push, open a draft PR, and submit a slurm job — but the agent reads AGENTS.md, sees the no-push rule, and refuses to take any of those terminal actions. Surfaced on OMNIML-4886 retries under pensieve-intern v0.55.0+: "Slurm job status: NONE — runner instructions say 'Do NOT commit, push, or create PRs — the runner handles that,' and no slurm submission was invoked in this run." The agent is correctly conservative; the rule didn't anticipate autonomous use. This change adds an env-var-gated exception: `$PENSIEVE_INTERN_RUNNER_PROMPT_PATH` or `$INTERN_TICKET` being set means "you are the autonomous runner; the SPEC IS the human approval." Interactive sessions are unaffected (neither env var is set there). Pairs with pensieve-intern's engine-level guidance — the engine preamble (v0.55.0+) tells autonomous agents to report what they did + what slurm submission state they reached, which surfaced this conflict in the first place by making "slurm: NONE — <reason>" explicit instead of silently skipped. CLAUDE.md is symlinked to AGENTS.md so this single edit covers both. Signed-off-by: chenhany <chenhany@nvidia.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Enterprise Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughUpdated ChangesAutonomous Runner Exception Documentation
🎯 2 (Simple) | ⏱️ ~10 minutes 🚥 Pre-merge checks | ✅ 6✅ Passed checks (6 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
Surfaced on OMNIML-4886 — the pensieve-intern AgentLoop cell agent reads AGENTS.md, sees the no-push rule, and refuses to commit/push/PR/submit slurm even though the SPEC explicitly instructs it to do all four.
Adds an env-var-gated carve-out:
$PENSIEVE_INTERN_RUNNER_PROMPT_PATHor$INTERN_TICKETbeing set means "you are the autonomous runner; the SPEC IS the human approval." Interactive sessions (neither env set) are unaffected.CLAUDE.md is symlinked to AGENTS.md — single-file edit covers both.
Full rationale in the commit message.
Maintainers: this is unblocking the inaugural specdec_bench workflow (https://jirasw.nvidia.com/browse/OMNIML-4885).
Summary by CodeRabbit