Skip to content

refactor(skills): merge the autodev mode-protocol skill into develop - #1735

Merged
aarontrowbridge merged 1 commit into
mainfrom
opencode/issue-1732-merge-autodev-develop
Oct 21, 2026
Merged

aarontrowbridge merged 1 commit into
mainfrom
opencode/issue-1732-merge-autodev-develop

Conversation

@aarontrowbridge

Copy link
Copy Markdown
Member

Closes #1732 · Part of #1731

What this does — One skill per mode, mirroring research: the autodev mode-protocol skill is merged into develop (protocol-first: ledger, the loop bound to the dev gate pack, roles, handoffs, honest degradation, compaction honesty; the issue-DAG walk is the mode's primary workflow section, worktree/parallelism content deduplicated). The former id is named inside the skill (forward-only read-resolve); the mode-id alias machinery (MODE_ID_ALIASES, session_spawn, mode_block) is untouched by design — an agent resolved as autodev still binds the develop posture.

Supersession, on the record — this merge supersedes the three-mode rename's id-retention clause (spec-20260907-011500 D1 rev 3, documented at the #858/#868 rename). The structural suite that pinned the two-skill split is rewritten in the same commit — the pin never lies for even one commit.

Sweep surface (all live skill references, alias/history surfaces deliberately untouched):

  • skills/develop/SKILL.md — merged (revision 3, agents: union of both)
  • skills/autodev/ — removed
  • modes/develop/mode.toml — autodev out of protocol_skills; comment updated
  • modes/develop/card.md + agents/develop.md — first-actions directive binds develop
  • ops/server/hub-AGENTS.md — index rows: stale autoresearch row → research; develop row → merged description; director-core row → copilot/research/develop posture model; onset-router line un-drifted
  • test/workflow_skills_public.test.ts — rewritten for the merged shape (protocol half + walk half + read-resolve + self-containment; manifest asserts develop present and autodev absent)
  • test/naming_records.test.ts, test/mode_cards.test.ts — inventories updated

Verification (fresh, this commit):

  • targeted suites: 42/42 (workflow_skills_public, naming_records, mode_cards)
  • packages/schema (the mode-registry validator): 301/301
  • lint:skills: 0 errors (the develop companion-path note is the pre-existing known pattern)
  • tsc --noEmit: clean
  • boot smoke: PASS

Ops checklist (NOT done by this PR — re-stage is a hub-restart action):

AC from #1732: one develop skill ✓ · protocol+walk content ✓ · bindings reference develop/director-core only ✓ · structural test rewritten in-commit ✓ · naming-records row flipped ✓ · repo grep zero live refs ✓ · hub index rows fixed ✓ · lint/parity/smoke green ✓

@coderabbitai

coderabbitai Bot commented Oct 8, 2026 •

Copy link
Copy Markdown
Contributor

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: ad2013b2-5a5a-4bd9-8471-cabfc1373959
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

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.

Merge the autodev mode-protocol skill into develop — one skill per mode

1 participant