Skip to content

fix(cli): do not fall through open/attach args into session#40176

Merged
pavelfeldman merged 1 commit intomicrosoft:mainfrom
pavelfeldman:fix-cli-attach-args
Apr 13, 2026
Merged

fix(cli): do not fall through open/attach args into session#40176
pavelfeldman merged 1 commit intomicrosoft:mainfrom
pavelfeldman:fix-cli-attach-args

Conversation

@pavelfeldman
Copy link
Copy Markdown
Member

Summary

  • Separate open/attach command handling so args don't fall through into session
  • Track browser ownership (attached vs own) in session config
  • Fix attach via CDP to work correctly with snapshot

ownership = 'own';
} else if (config.extension) {
browser = await createExtensionBrowser(config, clientInfo.clientName);
ownership = 'own';
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

'attached'? Or perhaps 'extension' to show (extension) in the list?

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions
Copy link
Copy Markdown
Contributor

Test results for "tests 1"

1 failed
❌ [playwright-test] › reporter-html.spec.ts:337 › merged › should include screenshot on failure @macos-latest-node20

5 flaky ⚠️ [chromium-library] › library/video.spec.ts:724 › screencast › should work with video+trace `@chromium-ubuntu-22.04-arm-node20`
⚠️ [chromium-library] › library/video.spec.ts:687 › screencast › should capture full viewport on hidpi `@chromium-ubuntu-22.04-node20`
⚠️ [firefox-library] › library/inspector/cli-codegen-1.spec.ts:1080 › cli codegen › should not throw csp directive violation errors `@firefox-ubuntu-22.04-node20`
⚠️ [playwright-test] › ui-mode-trace.spec.ts:700 › should indicate current test status `@macos-latest-node20`
⚠️ [playwright-test] › ui-mode-trace.spec.ts:700 › should indicate current test status `@windows-latest-node20`

39154 passed, 846 skipped


Merge workflow run.

@github-actions
Copy link
Copy Markdown
Contributor

Test results for "MCP"

6521 passed, 382 skipped


Merge workflow run.

@pavelfeldman pavelfeldman merged commit 0d89945 into microsoft:main Apr 13, 2026
37 of 38 checks passed
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.

3 participants