docs(agents): clarify work-on-main applies to maintainers, not fork contributors - #7
Open
levonk wants to merge 1 commit into
Open
Conversation
musichen
requested changes
Sep 8, 2026
musichen
left a comment
musichen
left a comment
Member
There was a problem hiding this comment.
Thanks for identifying the fork-contributor gap. I agree with the requested AGENTS.md clarification.
Requesting changes before merge:
- Rebase onto current
main. - Limit this PR to the contributor-branch clarification and any directly related documentation.
- Remove the unrelated ACP/Devin changes from this branch.
The current CI failure is also material: the profile cannot resolve @deepseek-ai/dsh-subagent-acp while loading subagent-acp-devin. That belongs in its own scoped PR with its dependency and Loader-composition verification.
levonk
force-pushed
the
fix/agents-third-party-branch-exception
branch
from
September 9, 2026 23:30
19ff8ce to
d12fe85
Compare
levonk
marked this pull request as draft
September 9, 2026 23:31
…ontributors The "work directly on main" convention in AGENTS.md line 29 was set for internal small-team rapid development. Third-party contributors on forks cannot push to main upstream and must work on feature branches to open PRs. Add an explicit clause so agents reading AGENTS.md on a fork don't commit to main when a PR is the intended delivery path. Closes acryldev#6
levonk
force-pushed
the
fix/agents-third-party-branch-exception
branch
from
September 9, 2026 23:33
d12fe85 to
efd779e
Compare
Contributor
Author
|
Sorry for the mixed workstreams on this branch — and thanks for the clear review. Here's what I did to resolve each request:
The CI failure from the unresolved |
levonk
marked this pull request as ready for review
September 9, 2026 23:39
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
main" convention applies to maintainers with push access, not to third-party contributors on forks who must use feature branches and PRs.Context
The "work directly on
main" instruction was set during an internal small-team rapid-development phase (blame:27356dcc). AI agents and other third-party contributors working on forks follow AGENTS.md by default, and this line causes them to commit tomainon the fork instead of creating a feature branch — making it impossible to open a clean PR upstream without retroactive history surgery.See #6 for the full problem description.
Closes #6