Skip to content

feat(agents): engineer agents write code in-session (retire acpx_run/sam-local-codex) — Nova first#498

Merged
samxu01 merged 2 commits into
mainfrom
feat/in-session-coding-presets
Jun 27, 2026
Merged

feat(agents): engineer agents write code in-session (retire acpx_run/sam-local-codex) — Nova first#498
samxu01 merged 2 commits into
mainfrom
feat/in-session-coding-presets

Conversation

@samxu01

@samxu01 samxu01 commented Jun 27, 2026

Copy link
Copy Markdown
Contributor

Why

Agents were delegating coding (acpx_run codex sub-agent, or sam-local-codex laptop wrapper) instead of writing code in their own session. The acpx_run path 401'd on codex auth today, blocking a real PR. Validated the fix: Theo wrote real issue #449 code in-session on gpt-5.4-mini and opened clean PR #497 (+31, 1 file) with proper git hygiene.

This PR (increment 1 of the migration)

Migrates Nova (backend-engineer) from the sam-local-codex delegation state-machine (PendingDelegation lifecycle, Branches A–E, multi-tick wait) to a single-tick in-session flow: claim → write code with own file+shell tools → test → gh pr create → complete. Bakes in the git hygiene proven on #497:

  • always branch off origin/main (git checkout -B <branch> origin/main — discards dirty tree)
  • stage ONLY changed files (never git add -A), git status --short before commit

Pixel / Theo / Ops follow in subsequent increments (each validated live).

🤖 Generated with Claude Code

samxu01 and others added 2 commits June 27, 2026 04:51
… delegation

Replaces Nova's sam-local-codex delegation state-machine (PendingDelegation
lifecycle, Branches A-E, multi-tick wait) with a single-tick in-session coding
flow: claim → write the code yourself with file+shell tools → test → gh pr
create → complete. Bakes in the git hygiene proven on #497: always branch off
origin/main (git checkout -B <branch> origin/main, discards dirty tree) and
stage ONLY changed files (never git add -A). First preset of the acpx_run /
sam-local-codex retirement; Pixel/Theo/Ops follow.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
The test locked the old sam-local-codex delegation invariants (SamCodexDmPodId,
PendingDelegation, Branch A-E). Rewrite it to lock the new in-session invariants:
delegation primitives gone; acpx_run/sam-local-codex only in prohibition lines;
in-session + git-hygiene markers (git checkout -B origin/main, never git add -A,
gh pr create) present.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@samxu01 samxu01 merged commit 5a7980d into main Jun 27, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant