docs(agents): clarify work-on-main applies to maintainers, not fork contributors - #7
Open
levonk wants to merge 6 commits into
Open
docs(agents): clarify work-on-main applies to maintainers, not fork contributors#7levonk wants to merge 6 commits into
levonk wants to merge 6 commits into
Conversation
…skills-lock.json)
Add `acryl acp` subcommand that boots DSH's ACP profile with acryl coding capabilities, allowing acryl to run as an ACP agent inside Devin Desktop's Agent Command Center. - Add 'acp' surface to coding-capabilities and bootAcrylAcpProfile to acryl-harness-runtime, composing dsh-base + dsh-acp with an acryl-owned startup latch (replacing the upstream dsh-acp-app bundle, which is not published at the 0.1.1-rc.2 version line acryl pins) - Add 'acp' subcommand to acryl-tui CLI - Add @deepseek-ai/dsh-acp dependency to acryl-harness-runtime and acryl-tui - Document ACP registry config for Devin Desktop at docs/acryl/ACP_REGISTRY.md
Add a `devin` agent preset that extends the standard coding agent with a `subagent_devin` tool, letting the model delegate tasks to Devin (running as `devin acp`) as an out-of-process ACP subagent. - Create acryl-owned `devin` preset under `acryl-harness-runtime/presets/devin/` mirroring the shipped standard preset plus a `tool-subagent-devin` row - Add `subagent-acp` provider to coding-capabilities patches for the TUI surface, registering the `devin` provider on `ctx.subagents` with `command: 'devin', args: ['acp']` - Add the acryl preset root to the agent-presets config so the `devin` preset appears alongside the shipped standard/cordis/ptc presets - Add `@deepseek-ai/dsh-subagent-acp` and `@deepseek-ai/dsh-tool-subagent` dependencies to acryl-harness-runtime - Update coding-capabilities tests for the new `devin-subagent` capability and `acp` surface
…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
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
1 participant
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