fix: replace backslashes with forward slashes in refs on Windows when splitting

Author: tatomyr

.changeset/eighty-dolls-sniff.md

@@ -0,0 +1,5 @@
+---
+"@redocly/cli": patch
+---
+
+Fixed a problem when the `split` command produced backslashes instead of forward slashes in $refs on Windows.

.github/workflows/smoke.yaml

@@ -283,6 +283,27 @@ jobs:
           node-version: 18
       - run: bash ./__tests__/smoke/run-smoke.sh "yarn add ./redocly-cli.tgz" "yarn"
 
+  run-smoke--split--windows:
+    needs: prepare-smoke
+    runs-on: windows-latest
+    steps:
+      - uses: actions/cache@v3
+        with:
+          path: __tests__/smoke/
+          key: cache-${{ github.run_id }}-${{ github.run_attempt }}
+          enableCrossOsArchive: true
+      - uses: actions/setup-node@v3
+        with:
+          node-version: 22
+      - name: Run split smoke
+        run: |
+          npm i -g redocly-cli.tgz
+          redocly --version
+          redocly split resources/pets.yaml --outDir __tests__/smoke/__split-pets
+          git diff
+          redocly split resources/museum.yaml --outDir __tests__/smoke/__split-museum
+          git diff
+
   run-smoke--docker-image:
     runs-on: ubuntu-latest
     steps:

packages/cli/src/commands/split/index.ts

@@ -176,9 +176,9 @@ function replace$Refs(obj: unknown, relativeFrom: string, componentFiles = {} as
     const groupName = splittedNode[2];
     const filesGroupName = componentFiles[groupName];
     if (!filesGroupName || !filesGroupName[name!]) return;
-    let filename = path.relative(relativeFrom, filesGroupName[name!].filename);
+    let filename = slash(path.relative(relativeFrom, filesGroupName[name!].filename));
     if (!filename.startsWith('.')) {
-      filename = '.' + path.sep + filename;
+      filename = './' + filename;
     }
     node[key] = filename;
   }
@@ -192,11 +192,11 @@ function implicitlyReferenceDiscriminator(
 ) {
   if (!obj.discriminator) return;
   const defPtr = `#/${COMPONENTS}/${OPENAPI3_COMPONENT.Schemas}/${defName}`;
-  const implicitMapping = {} as any;
+  const implicitMapping: Record<string, string> = {};
   for (const [name, { inherits, filename: parentFilename }] of Object.entries(schemaFiles) as any) {
     if (inherits.indexOf(defPtr) > -1) {
-      const res = path.relative(path.dirname(filename), parentFilename);
-      implicitMapping[name] = res.startsWith('.') ? res : '.' + path.sep + res;
+      const res = slash(path.relative(path.dirname(filename), parentFilename));
+      implicitMapping[name] = res.startsWith('.') ? res : './' + res;
     }
   }
 
@@ -258,7 +258,7 @@ function extractFileNameFromPath(filename: string) {
 }
 
 function getFileNamePath(componentDirPath: string, componentName: string, ext: string) {
-  return path.join(componentDirPath, componentName) + `.${ext}`;
+  return slash(path.join(componentDirPath, componentName) + `.${ext}`);
 }
 
 function gatherComponentsFiles(
@@ -273,7 +273,7 @@ function gatherComponentsFiles(
     inherits = (
       (components?.[componentType]?.[componentName] as Oas3Schema | Oas3_1Schema)?.allOf || []
     )
-      .map(({ $ref }) => $ref)
+      .map(({ $ref }) => $ref && slash($ref))
       .filter(isTruthy);
   }
   componentsFiles[componentType] = componentsFiles[componentType] || {};