feat(agents): Pixel + Ops + Theo in-session (retire acpx_run) — increments 2-4#501
Merged
Conversation
Increments 2-3 of the in-session migration. Replaces the acpx_run delegation flow (Step 2.5 CI check + Step 4 Path A/B embedded codex scripts) in the frontend-engineer (Pixel) and devops-engineer (Ops) presets with single-tick in-session coding: claim → write code/config yourself with file+shell tools → test → gh pr create → complete. Bakes in the git hygiene proven on #497/#498: git checkout -B <branch> origin/main (clean base, discards dirty tree) + git add only-changed-files (never git add -A) + git status --short before commit. Decision-point acpx_run imperatives flipped to in-session for both. Descriptions updated (via codex → in-session). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Increment 4. Theo is the PM/router (routes tasks, reviews PRs, syncs GitHub issues — does NOT write feature code). His acpx_run calls wrapped gh operations (scan PRs for CI failures, fetch open PRs, review one PR). Replaced all three with direct in-session shell/gh invocations. With this, ZERO acpx_run imperatives remain in presets.ts — only prohibitions. Completes the acpx_run/sam-local-codex retirement across all engineer presets. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
samxu01
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Continues the in-session migration (Nova was #498). Migrates Pixel (frontend-engineer) and Ops (devops-engineer) from the acpx_run delegation flow to single-tick in-session coding, same validated pattern as Nova + the git hygiene proven on #497:
git checkout -B <branch> origin/main(clean base, discards dirty tree)git addonly changed files (nevergit add -A),git status --shortbefore commitgh pr create; completeStep 2.5 CI-check and Step 4 Path A/B embedded codex scripts replaced; decision-point
acpx_runimperatives flipped to in-session; descriptions updated. Remaining: Theo (dev-pm — a router/PM, separate shape). Validated live after deploy on the now-healthy fleet (#500 fixed the rate-limit storm).🤖 Generated with Claude Code