feat: enhance CI configuration for cross-platform support and improve path normalization in scanning functions

Author: CasperKristiansson

.github/workflows/ci.yml

@@ -8,7 +8,11 @@ on:
 
 jobs:
   build:
-    runs-on: ubuntu-latest
+    runs-on: ${{ matrix.os }}
+    strategy:
+      fail-fast: false
+      matrix:
+        os: [ubuntu-latest, windows-latest]
     steps:
       - uses: actions/checkout@v4
       - name: Use Node.js 20

docs/tasks.md

@@ -259,7 +259,7 @@ Use Context7 MCP for up to date documentation.
         Output JSON of changed files and new version.
         Verify: Example consumes output.
 
-44. [ ] **Windows path handling**
+44. [x] **Windows path handling**
         Use `node:path`. Add Windows CI job.
         Verify: Windows runner green.
 

src/scanning/patterns/python-version.ts

@@ -1,3 +1,5 @@
+import path from 'node:path';
+
 export interface VersionMatch {
   file: string;
   line: number;
@@ -103,13 +105,12 @@ export const pythonVersionPatterns: PatternDefinition[] = [
 ];
 
 function normalizePath(filePath: string): string {
-  return filePath.replace(/\\\\/g, '/');
+  return filePath.split(path.win32.sep).join(path.posix.sep);
 }
 
 function getBasename(filePath: string): string {
   const normalized = normalizePath(filePath);
-  const index = normalized.lastIndexOf('/');
-  const base = index === -1 ? normalized : normalized.slice(index + 1);
+  const base = path.posix.basename(normalized);
   return base.toLowerCase();
 }
 

src/scanning/scanner.ts

@@ -46,17 +46,18 @@ export async function scanForPythonVersions(options: ScanOptions): Promise<ScanR
 
   for (const relative of relativeFiles) {
     const absolute = path.join(root, relative);
+    const relativePosix = relative.split(path.win32.sep).join(path.posix.sep);
     const content = await readFileSafe(absolute);
     if (content === null) {
       continue;
     }
 
     filesScanned += 1;
-    const fileMatches = findPythonVersionMatches(relative, content);
+    const fileMatches = findPythonVersionMatches(relativePosix, content);
     matches.push(
       ...fileMatches.map((match) => ({
         ...match,
-        file: relative,
+        file: relativePosix,
       })),
     );
   }

src/versioning/release-notes.ts

@@ -53,7 +53,9 @@ export async function fetchReleaseNotes(
   }
 
   if (!response.ok) {
-    throw new Error(`Failed to fetch release notes for ${normalizedTag} (status ${response.status}).`);
+    throw new Error(
+      `Failed to fetch release notes for ${normalizedTag} (status ${response.status}).`,
+    );
   }
 
   const payload = (await response.json()) as GitHubReleasePayload;

tests/python-version-patterns.test.ts

@@ -12,6 +12,15 @@ describe('findPythonVersionMatches', () => {
     expect(matches[0]?.matched).toBe('3.13.2');
   });
 
+  it('normalizes Windows-style workflow paths', () => {
+    const content = `jobs:\n  build:\n    steps:\n      - uses: actions/setup-python@v4\n        with:\n          python-version: "3.13.5"`;
+
+    const matches = findPythonVersionMatches('.github\\workflows\\ci.yml', content);
+
+    expect(matches).toHaveLength(1);
+    expect(matches[0]?.matched).toBe('3.13.5');
+  });
+
   it('captures versions within Dockerfiles', () => {
     const content = `FROM python:3.13.2-slim\nARG PYTHON_VERSION="3.12.8"\nENV PYTHON_VERSION=3.12.8`;
 

tests/release-notes.test.ts

@@ -3,14 +3,15 @@ import { describe, expect, it } from 'vitest';
 import { fetchReleaseNotes } from '../src/versioning';
 
 describe('fetchReleaseNotes', () => {
-  const createFetch = (response: { status: number; body?: unknown }) =>
-    async () => ({
+  function createFetch(response: { status: number; body?: unknown }) {
+    return async () => ({
       status: response.status,
       ok: response.status >= 200 && response.status < 300,
       async json() {
         return response.body ?? {};
       },
     });
+  }
 
   it('returns release note body when available', async () => {
     const fetchImpl = createFetch({ status: 200, body: { body: 'Security fix included.' } });

tests/scanner-errors.test.ts

@@ -5,11 +5,11 @@ const missingError = Object.assign(new Error('missing'), { code: 'ENOENT' });
 describe('scanForPythonVersions error handling', () => {
   it('ignores files that cannot be read', async () => {
     vi.resetModules();
-    vi.mock('../src/scanning/glob-discovery', () => ({
+    await vi.doMock('../src/scanning/glob-discovery', () => ({
       __esModule: true,
       discoverFiles: vi.fn(async () => ['missing.txt']),
     }));
-    vi.mock('node:fs/promises', () => ({
+    await vi.doMock('node:fs/promises', () => ({
       __esModule: true,
       readFile: vi.fn(async () => {
         throw missingError;
@@ -25,4 +25,32 @@ describe('scanForPythonVersions error handling', () => {
     vi.unmock('../src/scanning/glob-discovery');
     vi.unmock('node:fs/promises');
   });
+
+  it('normalizes Windows-style separators from glob discovery', async () => {
+    vi.resetModules();
+    const discoverFilesMock = vi.fn(async () => ['.github\\workflows\\ci.yml']);
+    await vi.doMock('../src/scanning/glob-discovery', () => ({
+      __esModule: true,
+      discoverFiles: discoverFilesMock,
+    }));
+    await vi.doMock('node:fs/promises', () => ({
+      __esModule: true,
+      readFile: vi.fn(async () => 'python-version: "3.13.2"'),
+    }));
+
+    const { scanForPythonVersions } = await import('../src/scanning/scanner');
+    const result = await scanForPythonVersions({
+      root: '.',
+      patterns: ['**/*.yml'],
+    });
+
+    expect(discoverFilesMock).toHaveBeenCalled();
+    expect(result.filesScanned).toBe(1);
+    expect(result.matches).toEqual([
+      expect.objectContaining({ file: '.github/workflows/ci.yml', matched: '3.13.2' }),
+    ]);
+
+    vi.unmock('../src/scanning/glob-discovery');
+    vi.unmock('node:fs/promises');
+  });
 });