Skip to content

fix(installer): normalize artifact paths on Windows - #30

Merged
tnunamak merged 1 commit into
mainfrom
fix/windows-artifact-paths
Jul 31, 2026
Merged

fix(installer): normalize artifact paths on Windows#30
tnunamak merged 1 commit into
mainfrom
fix/windows-artifact-paths

Conversation

@tnunamak

Copy link
Copy Markdown
Contributor

Summary

  • canonicalize extracted artifact paths to the portable POSIX contract
  • preserve POSIX behavior while converting Windows separators before contract matching
  • cover deterministic Windows and POSIX path fixtures

Verification

  • npm run installer:test (45 tests passed)
  • npm run connector-index:test (8 tests passed)
  • npm run chatgpt-pdpp:test (2 passed, 2 intentionally skipped)
  • npm run github-pdpp:test reached and passed the installer-core PDPP artifact test; one unrelated retained-index baseline assertion fails on current main
  • node --check for both changed modules
  • git diff --check

Assisted-by: AI

Use portable POSIX paths when matching extracted files to signed artifact contracts. Cover Windows and POSIX relative paths with deterministic fixtures.

Signed-off-by: Tim Nunamaker <tnunamak@gmail.com>

Assisted-by: AI
@tnunamak
tnunamak force-pushed the fix/windows-artifact-paths branch from abbc695 to 7144426 Compare July 31, 2026 19:49
@github-actions

Copy link
Copy Markdown

Schema Health Check — All Clear

All 63 scopes have consistent local schema files.

@tnunamak
tnunamak merged commit 48440fe into main Jul 31, 2026
3 checks passed
@tnunamak
tnunamak deleted the fix/windows-artifact-paths branch July 31, 2026 19:51
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.

1 participant