Skip to content

chore(ci): probe Windows runner for Copilot setup#183

Draft
JReinhold wants to merge 12 commits intomainfrom
copilot/windows-copilot-setup-probe
Draft

chore(ci): probe Windows runner for Copilot setup#183
JReinhold wants to merge 12 commits intomainfrom
copilot/windows-copilot-setup-probe

Conversation

@JReinhold
Copy link
Contributor

This PR temporarily switches the Copilot setup workflow to windows-latest so cloud Copilot runs can validate Windows-compatible CI commands.\n\nFollow-up expected in this same PR:\n- update to run on ubuntu + windows via matrix\n- keep non-test jobs ubuntu-only\n- if needed, guard Playwright install flags by OS\n- restore back to ubuntu-latest before merge\n

Temporarily switch the copilot setup workflow to windows-latest so cloud Copilot runs can validate Windows commands before final CI changes.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@JReinhold
Copy link
Contributor Author

@copilot Please implement the CI follow-up in this PR:\n\n1. Update so only the job uses a strategy matrix for and .\n2. Keep all non-test jobs Ubuntu-only.\n3. Make any required OS-specific adjustments for Playwright setup so Windows test runs pass.\n4. Revert back to in your final commit.\n5. Keep changes minimal and focused on CI YAML only.\n\nPlease run/validate via GitHub Actions and iterate until green.

Copy link
Contributor

Copilot AI commented Mar 3, 2026

@JReinhold I've opened a new pull request, #184, to work on those changes. Once the pull request is ready, I'll request review from you.

@changeset-bot
Copy link

changeset-bot bot commented Mar 3, 2026

⚠️ No Changeset found

Latest commit: 4eec4f7

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@pkg-pr-new
Copy link

pkg-pr-new bot commented Mar 3, 2026

npm i https://pkg.pr.new/storybookjs/mcp/@storybook/addon-mcp@183
npm i https://pkg.pr.new/storybookjs/mcp/@storybook/mcp@183

commit: 4eec4f7

@codecov
Copy link

codecov bot commented Mar 3, 2026

❌ 1 Tests Failed:

Tests completed Failed Passed Skipped
438 1 437 0
View the full list of 1 ❄️ flaky test(s)
apps/internal-storybook/tests/check-deps.e2e.test.ts > Storybook Dependencies > should be using latest versions from registry

Flake rate in main: 35.71% (Passed 45 times, Failed 25 times)

Stack Traces | 5.37s run time
Error: Storybook dependencies are outdated. Update the catalog in pnpm-workspace.yaml:

  sed -i '' 's/10\.3\.0-alpha\.15/10.3.0-alpha.16/g' pnpm-workspace.yaml && pnpm install

Outdated packages:
  - @storybook/addon-docs: 10.3.0-alpha.15 → 10.3.0-alpha.16
  - @storybook/react-vite: 10.3.0-alpha.15 → 10.3.0-alpha.16
  - storybook: 10.3.0-alpha.15 → 10.3.0-alpha.16
 ❯ tests/check-deps.e2e.test.ts:42:10

To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.

Copilot AI and others added 6 commits March 3, 2026 10:53
…ert copilot setup to ubuntu

Co-authored-by: JReinhold <5678122+JReinhold@users.noreply.github.com>
Handle CRLF checkouts on Windows runners before applying the catalog regex so Playwright version resolution succeeds.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
ci: run test job on ubuntu + windows matrix, fix Playwright setup for Windows
@codecov
Copy link

codecov bot commented Mar 10, 2026

Bundle Report

Changes will decrease total bundle size by 47.98kB (-100.0%) ⬇️. This is within the configured threshold ✅

Detailed changes
Bundle name Size Change
@storybook/mcp-esm (removed) -42.92kB (-100.0%) ⬇️
@storybook/addon-mcp-esm (removed) -5.05kB (-100.0%) ⬇️

@JReinhold JReinhold force-pushed the copilot/windows-copilot-setup-probe branch from 9bfc2b0 to 76067d7 Compare March 10, 2026 12:49
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