Skip to content

feat: add supervisor dispatch-focus entrypoint - #62

Merged
CondorCommodore merged 1 commit into
mainfrom
agent/cos-supervisor-dispatch-focus
Aug 4, 2026
Merged

feat: add supervisor dispatch-focus entrypoint#62
CondorCommodore merged 1 commit into
mainfrom
agent/cos-supervisor-dispatch-focus

Conversation

@CondorCommodore

Copy link
Copy Markdown
Owner

Summary

  • add a bounded supervisor dispatch-focus entrypoint that delegates to the durable focused-dispatch orchestrator
  • require live supervisor authority, current focus/decision projections, and a worker receipt adapter before dispatch
  • reject dispatch when external-state divergence is blocking and cover the new path with focused supervisor tests

Testing

  • python3 -m pytest -q tests/test_cos_bootstrap_supervisor.py tests/test_cos_dispatch_orchestrator.py tests/test_cos_assignment_coordinator.py
  • python3 -m pytest -q tests/test_cos_bootstrap_watchdog.py

@CondorCommodore
CondorCommodore force-pushed the agent/cos-supervisor-dispatch-focus branch from c015730 to 13506c4 Compare August 4, 2026 00:24
@CondorCommodore
CondorCommodore force-pushed the agent/cos-supervisor-dispatch-focus branch from 13506c4 to 6c76d18 Compare August 4, 2026 00:25

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: c0157307d9

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +1744 to +1747
_plan, envelope, parsed_manifest = cos_dispatch_orchestrator.build_focus_dispatch_plan(
manifest_path=manifest_path,
current_focus_path=paths["current_focus"],
decision_path=paths["decision"],

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Reject a focus projection from a different decision

When run_tick is concurrently reconciling—or is interrupted after replacing decision-current.json but before replacing current-focus.md—this call can pair the previous focus with the newer decision. If that old task remains in the actionable feed, the planner successfully dispatches it even though the current focus has moved; validate the projection's decision_digest and controller_epoch against the loaded decision and verified epoch before dispatching.

Useful? React with 👍 / 👎.

Comment on lines +1933 to +1934
elif args.command == "dispatch-focus":
result = dispatch_current_focus(

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Return a failure status when focused dispatch fails

When the dispatch completes normally but returns {"ok": false}—for example, the terminal edge reports unsuccessful delivery—this new branch falls through to the unconditional return 0 at the end of main. Consequently scripts/cosctl dispatch-focus signals success to shell automation despite reporting a failed dispatch; this command should derive its exit status from result["ok"].

Useful? React with 👍 / 👎.

@CondorCommodore
CondorCommodore merged commit 2a81bf1 into main Aug 4, 2026
5 checks passed
@CondorCommodore
CondorCommodore deleted the agent/cos-supervisor-dispatch-focus branch August 4, 2026 00:27
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