Copy package files with absolute paths

Author: codykaup

node-src/lib/turbosnap/findChangedDependencies.test.ts

@@ -1,5 +1,9 @@
 import snykGraph from '@snyk/dep-graph';
-import { mkdtempSync as unMockedMkdtempSync, statSync as unMockedStatSync } from 'fs';
+import {
+  copyFileSync as unMockedCopyFileSync,
+  mkdtempSync as unMockedMkdtempSync,
+  statSync as unMockedStatSync,
+} from 'fs';
 import { buildDepTreeFromFiles } from 'snyk-nodejs-lockfile-parser';
 import snyk from 'snyk-nodejs-plugin';
 import { afterEach, beforeEach, describe, expect, it, Mock, vi } from 'vitest';
@@ -21,6 +25,9 @@ const tmpdir = '/tmpdir';
 const statSync = unMockedStatSync as Mock;
 statSync.mockReturnValue({ size: 1 });
 
+const copyFileSync = unMockedCopyFileSync as Mock;
+copyFileSync.mockReturnValue(undefined);
+
 const mkdtempSync = unMockedMkdtempSync as Mock;
 mkdtempSync.mockReturnValue(tmpdir);
 
@@ -348,4 +355,46 @@ describe('findChangedDependencies', () => {
       }
     );
   });
+
+  it('handles relative paths correctly when copying files to temp directory', async () => {
+    // Mock the repository root to be different from current working directory
+    getRepositoryRoot.mockResolvedValue('/root/subdir');
+
+    // Mock findFilesFromRepositoryRoot to return relative paths
+    findFilesFromRepositoryRoot.mockImplementation((_, ...patterns) => {
+      const results: string[] = [];
+      for (const pattern of patterns) {
+        if (pattern === 'package.json') {
+          results.push('package.json');
+        } else if (
+          pattern === 'yarn.lock' ||
+          pattern === 'pnpm-lock.yaml' ||
+          pattern === 'package-lock.json'
+        ) {
+          results.push('package-lock.json');
+        }
+      }
+      return Promise.resolve(results);
+    });
+
+    mockInspect(/* HEAD */ ['[email protected]'], /* Baseline A */ ['[email protected]']);
+    mockChangedPackagesGraph(['[email protected]']);
+
+    const context = getContext({
+      git: {
+        packageMetadataChanges: [{ changedFiles: ['package.json'], commit: 'A' }],
+      },
+    });
+
+    await expect(findChangedDependencies(context)).resolves.toEqual(['react']);
+
+    expect(copyFileSync).toHaveBeenCalledWith(
+      '/root/subdir/package.json',
+      `${tmpdir}/package.json`
+    );
+    expect(copyFileSync).toHaveBeenCalledWith(
+      '/root/subdir/package-lock.json',
+      `${tmpdir}/package-lock.json`
+    );
+  });
 });

node-src/lib/turbosnap/findChangedDependencies.ts

@@ -112,11 +112,13 @@ export const findChangedDependencies = async (ctx: Context) => {
         const tmpdir = fs.mkdtempSync(path.join(os.tmpdir(), 'chromatic'));
         tmpdirsCreated.add(tmpdir);
 
+        const absoluteManifestPath = path.join(rootPath, manifestPath);
+        const absoluteLockfilePath = path.join(rootPath, lockfilePath);
         const temporaryManifestPath = path.join(tmpdir, path.basename(manifestPath));
         const temporaryLockfilePath = path.join(tmpdir, path.basename(lockfilePath));
 
-        fs.copyFileSync(manifestPath, temporaryManifestPath);
-        fs.copyFileSync(lockfilePath, temporaryLockfilePath);
+        fs.copyFileSync(absoluteManifestPath, temporaryManifestPath);
+        fs.copyFileSync(absoluteLockfilePath, temporaryLockfilePath);
 
         const headDependencies = await getDependencies(ctx, {
           rootPath: tmpdir,