docs: document automation repository boundaries - #369
Conversation
Clarify ownership across automation, Agent Server, client, and Canvas repositories. Fixes #368 Co-authored-by: openhands <openhands@all-hands.dev>
|
|
Keep the cross-repository ownership model consistent with OpenHands extensions. Co-authored-by: openhands <openhands@all-hands.dev>
|
🤖 OpenHands is reviewing this PR. Head commit: This comment was posted by an AI agent (OpenHands). |
all-hands-bot
left a comment
all-hands-bot
left a comment
There was a problem hiding this comment.
This review was created by an AI agent (OpenHands) on behalf of the repository maintainers.
Verdict: No material findings — approve to merge.
Risk assessment: None. This is a documentation-only change (+19/-0 across AGENTS.md and README.md) with no code, config, migration, or behavioral impact.
Accuracy check:
- All four cross-repository links resolve to existing repos (
software-agent-sdk,typescript-client,OpenHands,extensions), and their stated responsibilities match their actual repo descriptions. - The ownership boundary description is correct and consistent with the repository's architecture: this service owns the "when" (scheduling, dispatch, sandbox lifecycle), while the Agent Server/SDK owns the "what" (agent behavior, conversations, workspaces, events, API endpoints).
- The
AGENTS.mdguidance to close/move PRs opened in the wrong repository aligns with the cross-repo coordination policy already documented in this file.
Issue #368 acceptance criteria are all satisfied: responsibilities documented for all four repos, the when-vs-what distinction is clear, the SDK/Agent Server ownership of agent behavior/tools/conversations is stated, and the wrong-repo PR guidance plus contribution/code-review requirement are present in AGENTS.md.
Minor note (not blocking): the README.md section omits OpenHands/extensions while AGENTS.md covers it. Since AGENTS.md is the contributor-facing source of truth and fully enumerates all four repos, this is acceptable.
enyst
left a comment
enyst
left a comment
There was a problem hiding this comment.
Funnily enough, the reviewer agent didn’t approve this one, though this is fine. 😅
Slight redundancy on the first paragraph, since the little previous section says the same.
4 participants
HUMAN:
I reviewed the automation service architecture and documented its ownership boundaries and cross-repository workflow.
AGENT:
This pull request was created by an AI agent (OpenHands) on behalf of the user.
Why
Contributors need to know which behavior belongs in the Automation Service versus the Agent Server/SDK, TypeScript client, or Agent Canvas. This prevents duplicated logic and PRs being opened in the wrong repository.
Summary
AGENTS.mdandREADME.md.Issue Number
Fixes #368
How to Test
Review the two changed Markdown files for accurate ownership descriptions and repository links.
Video/Screenshots
Not applicable: documentation-only change.
Type