Skip to content

Fix Windows package filters and WIT path handling - #14

Open
IOHelpMe wants to merge 3 commits into
highcard-dev:mainfrom
IOHelpMe:codex/fix-windows-turbo-filter
Open

IOHelpMe wants to merge 3 commits into
highcard-dev:mainfrom
IOHelpMe:codex/fix-windows-turbo-filter

Conversation

@IOHelpMe

@IOHelpMe IOHelpMe commented Aug 20, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • use shell-portable double quotes for package build and watch filters
  • add regression coverage that executes both scripts through the host shell
  • normalize JCO-generated WIT paths on Windows while preserving POSIX paths
  • add regression coverage for Windows and POSIX WIT output paths

Testing

  • npm test (9 files, 16 tests passed)
  • npm run build -- --dry=json (6 packages selected after the current main merge)
  • git diff --check

Scope

This PR contains the two related Windows build fixes used by the native Docker VMM workflow: Turbo package filtering and JCO WIT path normalization.

@IOHelpMe IOHelpMe changed the title Fix Turbo package filters on Windows Fix Windows package filters and WIT path handling Aug 31, 2026
@IOHelpMe
IOHelpMe marked this pull request as ready for review August 31, 2026 21:47
@druid-infra

Copy link
Copy Markdown
Error: This repo is not allowlisted for Atlantis.

@IOHelpMe

Copy link
Copy Markdown
Contributor Author

@MarcStdt This is now rebased/merged onto current main, the title/body match both Windows fixes, and CI is green on Node 20, 22, and 24. Ready for review.

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