Deployment plan view fixes - #1627

Open

Megan Mott (motm32) wants to merge 81 commits into

mainfrom

meganmott/average-kingfisher

Megan Mott (motm32) commented Jul 29, 2026

Contributor

Fixes: #1622

I also noticed the subscription and location had not been properly being shown on the view even if they have been chosen in the chat and show up in the plan.md. I tested this and it seemed to fix it.

Nathan Turinski and others added 30 commits

June 8, 2026 10:48


          Remove Copilot on Rails feature

Removes the Copilot on Rails webviews, commands, agents, tree contributions, and build wiring. The feature is preserved on the feat/CoR branch for future development.


          Revert "Remove Copilot on Rails feature"

70dd310

This reverts commit 8044301.


          Update Copilot on Rails views (#1449)

a59a761

Co-authored-by: Nathan <naturins@microsoft.com>
Co-authored-by: Nathan Turinski <naturins@microsoft.comm>
Co-authored-by: Megan Mott <meganmott@Megans-MacBook-Pro.local>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>


          Add azure-debug-plan custom agent and instructions (#1475)

b9eaca9


          Add azure-debug-generate custom agent and instructions (#1478)

be42ab1

Co-authored-by: Nathan Turinski <naturins@microsoft.comm>


          Move skills to resources/agents instruction files (#1482)

Co-authored-by: Nathan Turinski <naturins@microsoft.comm>


          Improvements to dependency instructions for azure-debug-generate (#…

bb92297

…1483)


          Allow executing debug configurations in the project view (#1485)

a593097


          Improvements to compound launch/tasks & make sure TypeScript source m…

12b3b12

…aps are generated (#1489)


          add ui preview to planning doc (#1484)

9da8efa

Co-authored-by: Nathan Turinski <naturins@microsoft.comm>


          Add command to download agent instruction files to workspace (#1488)

Co-authored-by: Nathan Turinski <naturins@microsoft.comm>
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>


          Add activation events for our extension when we detect CoR artifacts (#…

a151b0d

…1496)

Co-authored-by: Nathan Turinski <naturins@microsoft.comm>


          Change instructions to rename src folder to services for monorepos (#…

96eeffe

…1498)

Co-authored-by: Nathan Turinski <naturins@microsoft.comm>


          Give folder names more service specific names (#1499)

8e50ce2

Co-authored-by: Nathan Turinski <naturins@microsoft.comm>


          Improvements to prerequisite finding for azure-debug-plan (#1505)

f24654a


          Autopilot Mode (#1497)

501d137

Co-authored-by: Nathan Turinski <naturins@microsoft.comm>
Co-authored-by: MicroFish91 <40250218+MicroFish91@users.noreply.github.com>


          Make changes for accessibility and add loading view (#1494)

729299d

Co-authored-by: Megan Mott <meganmott@Megans-MacBook-Pro.local>


          Add fixes for next steps views to work properly (#1510)

3910d84

Co-authored-by: Megan Mott <meganmott@Megans-MacBook-Pro.local>


          Forbid creating anything that is not app code (#1500)

48a5800

Co-authored-by: Nathan Turinski <naturins@microsoft.comm>


          Add prerequisite refresh (#1512)

f271d0a

Co-authored-by: Megan Mott <meganmott@Megans-MacBook-Pro.local>


          Use a more service-centric planning paradigm (#1508)

205a334

Co-authored-by: Nathan Turinski <naturins@microsoft.comm>
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>


          Fix view issues (#1513)

f08ac6d

Co-authored-by: Megan Mott <meganmott@Megans-MacBook-Pro.local>


          Make prerequisites logic shared, add a prerequisites view to the scaf…

a432d9c

…folding plan webview (#1516)


          Improve loading responsiveness of preview UI and add ability to write…

feae13c

… targeted page feedback (#1509)


          Add azure-project-integrate agent instructions (#1514)

6ccc4af

Co-authored-by: Nathan Turinski <naturins@microsoft.comm>
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>


          Update instructions to properly capture multiple data stores on requi…

ed8500f

…rements page (#1519)


          Nat/integrate integration (#1517)

4f7a2f6

Co-authored-by: Nathan Turinski <naturins@microsoft.comm>
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: MicroFish91 <40250218+MicroFish91@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>


          Fix comments showing up in the debug plan webview (#1520)

b881197


          add fix (#1521)

61f52f3

Co-authored-by: Megan Mott <meganmott@Megans-MacBook-Pro.local>


          Add refresh to scaffold prerequisites view (#1522)

b0b4b51

Co-authored-by: Megan Mott <meganmott@Megans-MacBook-Pro.local>

Matthew Fisher (MicroFish91) and others added 15 commits

July 24, 2026 14:22


          Add telemetry and diagnostics for interactions with the deployment pl…

8179c86

…an (#1600)


          Add some missing plan telemetry points (#1602)

62e7700


          CoR: Record ISO timestamps (#1605)

e2ce62c


          CoR: Record basic system / model specs for telemetry and diagnostics (#…

74356cb

…1601)


          CoR: Standardize command and telemetry ids (#1603)

3621f2e


          CoR: Delete some dead code (#1612)

5cd76a9


          Add a warning for when the GHC4A extension/skills are not installed w…

bb9bb54

…hen trying to deploy (#1614)

Co-authored-by: Megan Mott <meganmott@Megans-MacBook-Pro.local>


          Record autopilot telemetry automatically for all CoR commands (#1613)

9cd0225


          CoR: Improve prerequisites detection to be more shell agnostic (#1558)

05b883d


          CoR: Add extra telemetry for openWithChatAgent family of functions (#…

f69cab0

…1618)

Co-authored-by: Megan Mott <meganmott@Megans-MacBook-Pro.local>


          Change agent to always opening the requirements view (#1621)

3e2d1e9

Co-authored-by: Megan Mott <meganmott@Megans-MacBook-Pro.local>


          CoR: Add a way to report issue to GitHub with attached diagnostics in…

d0574eb

…fo (#1604)


          Write the status for approved project plan (#1619)

Co-authored-by: Megan Mott <meganmott@Megans-MacBook-Pro.local>


          Fix #1624: guard azd hook schema in deploy agent to stop deploy retry…

14da9d0

… loop (#1626)

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 86527a40-2bb0-4365-b6d5-49d38840de7e


          initial

d600c89

Megan Mott (motm32) requested a review from a team as a code owner

July 29, 2026 20:47

Nathan (nturinski) reviewed

View reviewed changes

src/webviews/copilotOnRails/extension/openDeploymentPlanView.ts

               import { ext } from "../../../extensionVariables";
-              import { CopilotOnRailsContext } from "../../../utils/copilotOnRails/CopilotOnRailsContext";
               import { DEPLOYMENT_PLAN_FILE_GLOB } from "../../../tree/project/projectPlanFiles";
+              import { CopilotOnRailsContext } from "../../../utils/copilotOnRails/CopilotOnRailsContext";

Nathan (nturinski) Jul 29, 2026

Member

Any idea why this got moved? I don't see any obvious difference

Megan Mott (motm32) Jul 30, 2026

Contributor Author

Probably just formatting differences I'll move it back

Nathan (nturinski) previously approved these changes

View reviewed changes

src/webviews/copilotOnRails/views/utils/parseDeploymentPlanMarkdown.ts Outdated

                       { name: 'East US 2', code: 'eastus2' },
                       { name: 'West US', code: 'westus' },
                       { name: 'West US 2', code: 'westus2' },
+                      { name: 'West US 3', code: 'westus3' },

Nathan (nturinski) Jul 29, 2026

Member

I know that it's been like this for a while, but I feel like we should get this static list from the locations step or something similar in the tools package rather than managing it


          changes

a54439a

Matthew Fisher (MicroFish91) approved these changes

View reviewed changes

Matthew Fisher (MicroFish91) left a comment •

edited

Loading

Contributor

Can also wrap with corId('deploymentPlan.getLocations')


          requested change

101ec9b

This was referenced Jul 31, 2026

[CoR] Location field remains blank after deployment plan is generated with a selected region #1636

Open

[CoR] Deployment does not start after deployment artifacts are successfully validated #1637

Open

Matthew Fisher (MicroFish91) previously approved these changes

View reviewed changes

Megan Mott (motm32) commented Jul 31, 2026

Contributor Author

Making more changes so waiting to get those into this PR before merging


          more changes

feff6d2

Megan Mott (motm32) commented Aug 5, 2026

Contributor Author

The above changes include some updates to the view to better align with what the azure-prepare skill flow looks like. I also removed the already approved plan functionality as it was super buggy since the status could be different with each iteration which made it so users could not continue.

Base automatically changed from feat/CoR to main

September 16, 2026 05:22

Matthew Fisher (MicroFish91) dismissed stale reviews from Nathan (nturinski) and themself

September 16, 2026 05:22

The base branch was changed.

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

Labels

None yet