Skip to content

CoR: Migrate to "Copilot" harness (update MCP; fix autopilot, model & custom agent carryover) - #1939

Draft
Matthew Fisher (MicroFish91) wants to merge 34 commits into
mainfrom
mwf/copilot-harness
Draft

Matthew Fisher (MicroFish91) wants to merge 34 commits into
mainfrom
mwf/copilot-harness

Conversation

@MicroFish91

Copy link
Copy Markdown
Contributor

VS Code will be migrating to the "Copilot" harness as the default, this ensures Copilot on Rails doesn't break when that happens.

Copilot AI balanced review requested due to automatic review settings October 5, 2026 17:49
@MicroFish91 Matthew Fisher (MicroFish91) changed the title CoR: Migrate to "Copilot" harness (update MCP, fix model & custom agent carryover) CoR: Migrate to "Copilot" harness (update MCP; fix autopilot, model & custom agent carryover) Oct 5, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🟡 Changes recommended

The scaffold handoff remains inconsistent, and the new drift validation rejects every current agent manifest.

Review effort: Balanced
Findings: 1 High severity · 1 Medium severity

Open (2)
What changed in this PR

Migrates Copilot on Rails to VS Code’s Copilot harness while preserving agent, model, and Autopilot state across phase handoffs.

Changes:

  • Enables the Copilot harness and TCP MCP transport.
  • Adds Agent Host model resolution and Autopilot propagation.
  • Updates agent aliases, tests, contracts, dependencies, and documentation.

Screenshots: None; no visual UI changed.

File Description
test/​copilotOnRails/​openChatWithAgent.test.ts Tests agent launch options and Autopilot markers.
test/​copilotOnRails/​modelSelection.test.ts Tests Agent Host model mapping.
test/​copilotOnRails/​autopilot.test.ts Tests Agent Host Autopilot configuration.
src/​webviews/​copilotOnRails/​extension/​harnessSettings.ts Enables the Copilot harness.
src/​webviews/​copilotOnRails/​extension/​autopilot.ts Configures and restores Agent Host Autopilot.
src/​utils/​copilotOnRails/​modelSelection.ts Resolves harness-compatible models.
src/​extension.ts Enables eager TCP MCP startup.
src/​commands/​copilotOnRails/​openChatWithAgent.ts Updates custom-agent launch sequencing.
resources/​agents/​azure-project-scaffold/​instructions.md Revises unattended integration handoff.
resources/​agents/​azure-project-scaffold.agent.md Adds harness tool aliases.
resources/​agents/​azure-project-plan.agent.md Adds harness tool aliases.
resources/​agents/​azure-project-integrate.agent.md Adds harness tool aliases.
resources/​agents/​azure-deploy.agent.md Adds harness tool aliases.
resources/​agents/​azure-debug-plan.agent.md Adds harness tool aliases.
resources/​agents/​azure-debug-generate.agent.md Adds harness tool aliases.
package.json Updates MCP and test dependencies.
package-lock.json Locks updated dependencies.
evals/​check-agent-drift.ts Adds agent-tool and handoff contracts.
evals/​agent-assets.lock.json Refreshes agent asset hashes.
docs/​copilot-create-project.md Documents harness behavior and state.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread evals/check-agent-drift.ts Outdated
Comment thread resources/agents/azure-project-scaffold/instructions.md
@MicroFish91
Matthew Fisher (MicroFish91) marked this pull request as ready for review October 5, 2026 18:34
@MicroFish91
Matthew Fisher (MicroFish91) requested a review from a team as a code owner October 5, 2026 18:34

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

CoR debug generation review

These are general recommendations. Treat them as suggestions, not rules that
every change must follow.

Verdict: PASS
Reviewed head: b9d0998
Compared base: 8a381e8

The scoped change only expands the custom agent's tool declarations for harness-compatible names and does not alter instruction ownership or workflow composition.

Findings

None.

Coverage

Inspected scoped file: resources/agents/azure-debug-generate.agent.md. No files under resources/agents/azure-debug-generate/ changed; the unchanged internal instructions were inspected as composition context.

Criterion Result Reason
AR-01 Pass Tool availability remains owned by the custom-agent frontmatter; no workflow responsibility moved.
AR-02 Pass The added aliases do not redefine commands, artifacts, ports, readiness, or lifecycle behavior.
AR-03 Not applicable Internal workflow coordination is unchanged.
AR-04 Not applicable Reference selection, ordering, output ownership, and conflict handling are unchanged.
AR-05 Not applicable No shared or implementation-specific workflow behavior changed.
AR-06 Not applicable No shared module was added or changed.
AR-07 Pass The declaration change introduces no orchestrator-specific behavior or generated-file changes.
AR-08 Not applicable No category entry, template, section, or ownership structure changed.

This is a static instruction review, not a successful F5 or end-to-end test.
After pushing fixes, someone with write, maintain, or admin access to the base
repository can post a new PR comment starting with /cor-debug-generate-review
to request another review.

Generated by CoR Azure Debug Generate Review for #1939 · copilot · gpt56 · 32.7 AIC · ⌖ 12.2 AIC · ⊞ 24.4K
Comment /cor-debug-generate-review to run again

@MicroFish91
Matthew Fisher (MicroFish91) marked this pull request as draft October 5, 2026 18:58
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.

2 participants