Skip to content

fix: route Op Admiral through connected agent and resume approvals#106

Merged
jmagly merged 1 commit into
elder-plinius:mainfrom
jmagly:fix/104-op-admiral-local-agent-approvals
Jul 20, 2026
Merged

fix: route Op Admiral through connected agent and resume approvals#106
jmagly merged 1 commit into
elder-plinius:mainfrom
jmagly:fix/104-op-admiral-local-agent-approvals

Conversation

@jmagly

@jmagly jmagly commented Jul 20, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • route keyless Op Admiral planning through the connected local agent (including Hermes)
  • preserve approval receipt IDs across Full Auto retries and resume automatically after the requested receipt is approved
  • add regression coverage for connected-agent selection and receipt-aware retry behavior

Root cause

The Op Admiral configuration path did not consult the connected local-agent selection and could fall back to Codex. Separately, Full Auto discarded the receipt returned by the approval gate, so later attempts could not present the approved receipt and minted a new one.

Impact

Hermes users can plan operations without a local Codex binary, and Full Auto now progresses through the existing scoped approval gates after operator approval. The backend continues to validate every receipt ID against its action and target.

Verification

  • npm test — 60 test files / 644 tests passed; 11/11 ops-preflight checks passed
  • npm run typecheck
  • git diff --check

Closes: #104

@jmagly jmagly self-assigned this Jul 20, 2026
@jmagly
jmagly marked this pull request as ready for review July 20, 2026 18:30
@jmagly
jmagly merged commit 58193d6 into elder-plinius:main Jul 20, 2026
1 check 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.

Can't "Plan Operation" or "Full Auto"

1 participant