Skip to content

[codex] teach orchestrator status command guidance#2227

Open
cneuralnetwork wants to merge 2 commits into
AgentWrapper:mainfrom
cneuralnetwork:codex/orchestrator-status-help
Open

[codex] teach orchestrator status command guidance#2227
cneuralnetwork wants to merge 2 commits into
AgentWrapper:mainfrom
cneuralnetwork:codex/orchestrator-status-help

Conversation

@cneuralnetwork

@cneuralnetwork cneuralnetwork commented Jun 27, 2026

Copy link
Copy Markdown

Summary

Teach orchestrator sessions the existing worker status commands instead of leaving them to guess. The standing orchestrator prompt now points to ao session ls, ao session get <worker-session-id>, and ao --help as the fallback before inventing commands.

Why

The prompt previously listed ao spawn and ao send, then told the orchestrator to track worker progress without naming the real inspection commands. That made orchestrators try nonexistent commands such as ao list or ao status --session before eventually discovering help.

Changes

  • Added status-command guidance to orchestratorPrompt.
  • Extended TestSpawnOrchestrator_UsesCoordinatorPrompt to assert the status and help hints are present in the orchestrator system prompt.
  • Stabilized the native CLI E2E harness by reaping the directly-started daemon child while ao stop polls for shutdown, matching the production ao start behavior.
  • Applied goimports/gofmt cleanup to the files reported by CI.

Testing

  • env GOTOOLCHAIN=auto GOSUMDB=sum.golang.org GOCACHE=/tmp/ao-go-cache GOPATH=/tmp/ao-gopath go test ./internal/session_manager
  • env GOTOOLCHAIN=auto GOSUMDB=sum.golang.org GOCACHE=/tmp/ao-go-cache GOPATH=/tmp/ao-gopath go test -race ./internal/session_manager
  • env GOTOOLCHAIN=auto GOSUMDB=sum.golang.org GOCACHE=/tmp/ao-go-cache GOPATH=/tmp/ao-gopath SHELL=/bin/bash go test -tags e2e -run TestE2E_Lifecycle -v ./internal/cli
  • env GOTOOLCHAIN=auto GOSUMDB=sum.golang.org GOCACHE=/tmp/ao-go-cache GOPATH=/tmp/ao-gopath SHELL=/bin/bash go test -tags e2e -v ./internal/cli
  • env GOTOOLCHAIN=auto GOSUMDB=sum.golang.org GOCACHE=/tmp/ao-go-cache GOPATH=/tmp/ao-gopath SHELL=/bin/bash go test ./...
  • env GOTOOLCHAIN=auto GOSUMDB=sum.golang.org GOCACHE=/tmp/ao-go-cache GOPATH=/tmp/ao-gopath SHELL=/bin/bash npm run lint
  • env GOTOOLCHAIN=auto GOSUMDB=sum.golang.org GOCACHE=/tmp/ao-go-cache GOPATH=/tmp/ao-gopath SHELL=/bin/bash go test -race ./...

Fixes #2197

@cneuralnetwork cneuralnetwork marked this pull request as ready for review June 27, 2026 11:41
@Vaibhaav-Tiwari Vaibhaav-Tiwari self-requested a review June 27, 2026 13:09

@Vaibhaav-Tiwari Vaibhaav-Tiwari left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm but there are some failing checks so please fix those

Image

@cneuralnetwork cneuralnetwork marked this pull request as draft June 27, 2026 14:04
@cneuralnetwork cneuralnetwork marked this pull request as ready for review June 28, 2026 06:07
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.

Orchestrator has no status command and isn't told to check ao --help first

2 participants