Skip to content

update README: orchestration modes#6

Merged
tuanknguyen merged 5 commits into
mainfrom
docs-cao-server
Oct 13, 2025
Merged

update README: orchestration modes#6
tuanknguyen merged 5 commits into
mainfrom
docs-cao-server

Conversation

@tuanknguyen

@tuanknguyen tuanknguyen commented Oct 13, 2025

Copy link
Copy Markdown
Contributor

Issue #, if available:

Description of changes:

  • update README: orchestration modes
  • Remove CORS. Will add back when frontend is available
  • Add output showing where server logs are stored

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Comment thread README.md

## MCP Server Tools and Orchestration Modes

CAO provides a local HTTP server that processes orchestration requests. CLI agents can interact with this server through MCP tools to coordinate multi-agent workflows.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

NIT: We can add a note here to explicitly allowlist MCP tools for orchestration, its not clear from examples.

Comment thread README.md
← send_message("E2E tests passed") ← Tester 3
```

**3. Send Message** - Communicate with an existing agent

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Do you want to note that SendMsg by default is handled by subagents through the use of MCP. (So need to explicitly add this in subagent prompt unless you need intermediate/continuous message transfer).

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

send_message is a tool that can be used by both orchestrators and subagents. The key difference is that the receiver agent must already be running and the sender agent must know the receiver's terminal_id

@gokulanv gokulanv left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Change looks good to merge. Nit doc comments, which I can also address later.

@gokulanv

Copy link
Copy Markdown
Contributor

Also just noticed that there are 5 commits in PR. You might want to squash them into one.

@tuanknguyen

Copy link
Copy Markdown
Contributor Author

Also just noticed that there are 5 commits in PR. You might want to squash them into one.

Yes, the default merge for this repo is Squash merge.

@tuanknguyen tuanknguyen enabled auto-merge (squash) October 13, 2025 19:24
@tuanknguyen tuanknguyen disabled auto-merge October 13, 2025 19:30
@tuanknguyen tuanknguyen merged commit 5fe220f into main Oct 13, 2025
2 checks passed
@tuanknguyen tuanknguyen deleted the docs-cao-server branch October 13, 2025 19:31
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.

4 participants