Skip to content

[Docs Freshness] Document delegated model roles - #470

Open
pandemicsyn wants to merge 1 commit into
mainfrom
agent/docs-freshness-20261024-010000
Open

pandemicsyn wants to merge 1 commit into
mainfrom
agent/docs-freshness-20261024-010000

Conversation

@pandemicsyn

Copy link
Copy Markdown
Owner

Summary

Document the configurable repository, CI, and release-readiness subagent model roles, including their fallback chain.

Review coverage

  • Getting Started (mandatory): complete guide reviewed against package metadata, onboarding, runtime-home resolution, CLI behavior, and GitHub token guidance. No new correction was needed; open PRs [Docs Freshness] Correct learning review agent identifier #439 and [Docs Freshness] Refresh fine-grained PAT guidance #448 already cover their respective learning and PAT corrections.
  • CLI: reviewed commands, options, authentication, database, repository, watch, handoff, scheduled-task, MCP, learning, and diagnostics claims. No correction needed.
  • Configuration: reviewed runtime home, provider/model schemas, model roles, secrets, learning, schedules, watches, and dashboard configuration. Corrected the model-role table to include the three supported delegated roles.
  • Watched-PR Autopilot: reviewed modes, confirmation, owner lifecycle, worktree handling, recovery, and delivery guards. No correction needed.

Evidence

  • src/modules/runtime/agent-config.ts resolves repoResearcher, ciInvestigator, and releaseReviewer through subagents.default before the display assistant model.
  • src/modules/runtime/subagents.ts registers those delegates and defines their responsibilities.
  • src/cli/onboarding.ts configures all three roles during setup.
  • src/cli/index.ts, scheduler and watch services, runtime-home paths, README.md, and .plans/ROADMAP.md informed the remaining review.

Files changed

  • docs/src/pages/docs/configuration.astro

Validation

  • fnm exec --using 26.4.0 npm run docs:check
  • fnm exec --using 26.4.0 npm run docs:build (passed on sequential rerun; initial parallel run collided with Astro Vite cache generation)
  • fnm exec --using 26.4.0 npx prettier --check docs/src/pages/docs/configuration.astro
  • git diff --check

@pandemicsyn pandemicsyn added the documentation Improvements or additions to documentation label Oct 24, 2026
Copilot AI balanced review requested due to automatic review settings October 24, 2026 16:35

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟡 Changes recommended

The fallback descriptions omit supported intermediate settings, making the PR’s central documentation claim inaccurate.

1 open issue
What changed in this PR

This documentation-only PR expands Neondeck’s configuration reference to cover its three delegated model roles.

Changes:

  • Adds repository research, CI investigation, and release-readiness roles to the model-role table.
  • Describes each role’s responsibilities and intended fallback behavior.
File Description
docs/​src/​pages/​docs/​configuration.astro Adds three delegated model roles and their fallback descriptions.

Review effort: Balanced

Comment on lines +284 to +301
Uses <code>subagents.default</code>, then
<code>displayAssistant</code>
</td>
</tr>
<tr>
<th scope="row"><code>subagents.ciInvestigator</code></th>
<td>CI, check, and validation-fact investigation</td>
<td>
Uses <code>subagents.default</code>, then
<code>displayAssistant</code>
</td>
</tr>
<tr>
<th scope="row"><code>subagents.releaseReviewer</code></th>
<td>Release, PR-watch, and scheduler readiness review</td>
<td>
Uses <code>subagents.default</code>, then
<code>displayAssistant</code>

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants