Skip to content

fix(vscuse): Auto-fix da-mcp-server--da-mcp-dcr-copilot - #16955

Open
Tian Yuan (xiaolang124) wants to merge 3 commits into
devfrom
auto-fix/da-mcp-server--da-mcp-dcr-copilot/2026-11-09-2
Open

Tian Yuan (xiaolang124) wants to merge 3 commits into
devfrom
auto-fix/da-mcp-server--da-mcp-dcr-copilot/2026-11-09-2

Conversation

@xiaolang124

Copy link
Copy Markdown
Contributor

VscUse Auto-Fix Results

Plan: da-mcp-server--da-mcp-dcr-copilot
Status: ✅ FIXED (attempt 3/8)
Initial Report: Link
Final Report: Link

Iteration Summary

# Fix Applied Result Report
1 Updated compiler/component sources to verify the current OAuth tool result inste ❌ Failed Report
2 Added a screenshot-backed scroll-to-latest click after DCR consent to reveal the ❌ Failed Report
3 Updated the compiler to scroll to the latest DCR response before asserting conse ✅ Passed Report

How to Review

  • Each commit represents one fix attempt for plans/da-mcp-server--da-mcp-dcr-copilot.json
  • Compare the initial report with the final report to verify the fix
  • Check that coordinate/dhash changes are reasonable for the current VS Code layout

Changes Made to Test Plan

.../plans/da-mcp-server--da-mcp-dcr-copilot.json   | 65 ++++++++++++++++------
 1 file changed, 47 insertions(+), 18 deletions(-)
View diff
diff --git a/packages/tests/vscuse/vscode-test-cases/plans/da-mcp-server--da-mcp-dcr-copilot.json b/packages/tests/vscuse/vscode-test-cases/plans/da-mcp-server--da-mcp-dcr-copilot.json
index 84665b7..1648dc6 100644
--- a/packages/tests/vscuse/vscode-test-cases/plans/da-mcp-server--da-mcp-dcr-copilot.json
+++ b/packages/tests/vscuse/vscode-test-cases/plans/da-mcp-server--da-mcp-dcr-copilot.json
@@ -8,7 +8,7 @@
       "precondition_wait_timeout": 30,
       "precondition_retry_interval": 1
     },
-    "total_steps": 155,
+    "total_steps": 156,
     "name": "da-mcp-dcr-copilot",
     "description": {
       "owner": "",
@@ -148,12 +148,13 @@
       "step_sendCopilotMessage_send_c81a54377_7_6",
       "step_allowCopilotAction_assert_c81a54377_7_7",
       "step_allowCopilotAction_click_c81a54377_7_7",
+      "step_revealLatestCopilotResponse_c81a54377_7_8",
       "step_allowCopilotAction_assertDismissed_c81a54377_7_7",
-      "step_assertChatReplied_assert_c81a54377_7_8",
-      "step_assertToolCalled_c81a54377_7_9",
-      "step_assertChatNonEmptyField_c81a54377_7_10",
-      "step_assertChatContains_assert_c81a54377_7_11",
+      "step_assertChatReplied_assert_c81a54377_7_9",
+      "step_assertToolCalled_c81a54377_7_10",
+      "step_assertChatNonEmptyField_c81a54377_7_11",
       "step_assertChatContains_assert_c81a54377_7_12",
+      "step_assertChatContains_assert_c81a54377_7_13",
       "step_sendCopilotMessage_assertInput_c81a54377_8_1",
       "step_sendCopilotMessage_focusInput_c81a54377_8_1",
       "step_sendCopilotMessage_type_c81a54377_8_1",
@@ -3113,30 +3114,58 @@
         "ocr:true"
       ]
     },
+    {
+      "step_id": "step_revealLatestCopilotResponse_c81a54377_7_8",
+      "agent": "interaction",
+      "tool": "click",
+      "parameters": {
+        "button": "left",
+        "x": 627,
+        "y": 591
+      },
+      "description": "Click the circular down-arrow button above the message composer to scroll to the latest Copilot response and reveal the result hidden behind the composer.",
+      "content_refs": [],
+      "timeout": 30,
+      "retry_count": 0,
+      "continue_on_error": "false",
+      "depends_on": [
+        "step_allowCopilotAction_click_c81a54377_7_7"
+      ],
+      "preconditions": [
+        "dhash:627:591:16:5:2c2d2d0e6d29934e"
+      ],
+      "postconditions": [],
+      "tags": [
+        "component:browser",
+        "host_surface:copilot",
+        "action:reveal-latest-response",
+        "evidence:screenshot_20261109_062712_187"
+      ]
+    },
     {
       "step_id": "step_allowCopilotAction_assertDismissed_c81a54377_7_7",
       "agent": "assertion",
       "tool": "",
       "parameters": {},
-      "description": "@assertion the Copilot action-consent Allow button is no longer visible.",
+      "description": "@assertion the current assistant response is complete and contains returned result data for the requested remote tool after granting access. An Allow/Cancel card in an earli
... (truncated, see commit diff for full changes)

Generated by vscuse-auto-fix.yml | model: gpt-6.1-sol

… the current OAuth tool result instead of requiring a historical Allow card to disappear; preserved ordinary consent checks and regenerated plans.
…lick after DCR consent to reveal the obscured `client_id` value before validation. Preserved all result assertions and regenerated the plans.
… DCR response before asserting consent completion, correcting dependency order while preserving all result checks and sibling behavior. Regenerated plans.
@github-actions

github-actions Bot commented Nov 9, 2026 •

Copy link
Copy Markdown
Contributor

❌ VscUse Test Plan — Tests failure

Why these tests: Selected 3 impacted test plan(s) changed in this PR (no product code modified).

Branch diff: auto-fix/da-mcp-server--da-mcp-dcr-copilot/2026-11-09-2 → dev

Results: ✅ 2 passed · ❌ 1 failed (of 3 plans)

Plans run:

  • da-mcp-server--da-mcp-dcr-copilot
  • da-mcp-server--da-mcp-dcr-update-copilot
  • da-mcp-server--da-mcp-static-entra-copilot

Step Status
1️⃣ Build VSIX (CD) ✅ Done
2️⃣ Build Docker image ✅ Done
3️⃣ Run UI tests ❌ Tests failure (2/3 passed)

🎯 Actual UI test run
🔗 Full pipeline results
📊 Detailed test report

ℹ️ How were these tests selected?

GitHub Copilot (GPT-5.6-sol, high reasoning) analysed the PR title, description, and the diff between
auto-fix/da-mcp-server--da-mcp-dcr-copilot/2026-11-09-2 and dev
to pick the most relevant test plans from packages/tests/vscuse/Index.md.

@github-actions

github-actions Bot commented Nov 9, 2026 •

Copy link
Copy Markdown
Contributor

E2E Test Selection — AI Selected

Why these tests: AI selection failed — will fall back to all cases

Cases selected (1):

  • ./vs/FoundryProxyAgent.dotnet.tests.ts

View pipeline run

Need to run more tests?

Comment on this PR:

  • /e2e-run ./path/to/test.tests.ts — add specific cases to AI selection
  • /e2e-run-all — run all e2e cases

Then re-run the workflow.

@codecov

codecov Bot commented Nov 9, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.88%. Comparing base (996cd05) to head (422b1c6).
⚠️ Report is 2 commits behind head on dev.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##              dev   #16955   +/-   ##
=======================================
  Coverage   88.88%   88.88%           
=======================================
  Files         689      689           
  Lines       37101    37153   +52     
  Branches     9302     9328   +26     
=======================================
+ Hits        32977    33024   +47     
+ Misses       2206     2205    -1     
- Partials     1918     1924    +6     

see 5 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

This branch was previously deployed

1 inactive deployment
engineering — 422b1c6e Deployed Nov 9, 2026 by teamstoolkitworkflowapp[bot] via run / report #1711
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant