fix(test): rebase onto v0.2.0 and harden empty-code --out cleanup

- Resolve rebase conflicts: keep atomic temp-file --out writes from main
  while rejecting empty inline code with VALIDATION_ERROR (exit 5)
- abortOutputFile: wait for stream close before unlinking tmpPath
- BOM test: use .py code-file (assertPythonCodeFile from v0.2.0)
- CI: build before test + fileParallelism false (dist/ race flake)

Author: SahilRakhaiya05

.github/workflows/ci.yml

@@ -60,6 +60,7 @@ jobs:
           cache: 'npm'
 
       - run: npm ci
+      - run: npm run build
       - run: npm test
         env:
           CI: true

src/commands/test.test.ts

@@ -1708,7 +1708,7 @@ describe('runCodePut', () => {
   it('strips a UTF-8 BOM from --code-file before uploading (Windows PowerShell 5.1 default)', async () => {
     const { credentialsPath } = makeCreds();
     const dir = mkdtempSync(join(tmpdir(), 'cli-p4-bom-'));
-    const codeFile = join(dir, 'updated.spec.ts');
+    const codeFile = join(dir, 'updated.py');
     writeFileSync(codeFile, '\uFEFF' + 'updated body', 'utf8');
     let seenBody: unknown;
     const fetchImpl = makeFetch((_url, init) => {

src/commands/test.ts

@@ -8011,7 +8011,14 @@ async function closeOutputFile(sink: FileSink, commit: boolean): Promise<void> {
 
 /** Tear down an opened `--out` sink without leaving a zero-byte artifact. */
 async function abortOutputFile(sink: FileSink): Promise<void> {
-  sink.stream.destroy();
+  await new Promise<void>(resolve => {
+    if (sink.stream.destroyed) {
+      resolve();
+      return;
+    }
+    sink.stream.once('close', () => resolve());
+    sink.stream.destroy();
+  });
   await unlink(sink.tmpPath).catch(() => undefined);
 }
 

vitest.config.ts

@@ -4,6 +4,9 @@ export default defineConfig({
   test: {
     include: ['src/**/*.{test,spec}.ts', 'test/**/*.{test,spec}.ts'],
     exclude: ['test/dev-e2e/**', 'test/e2e/**', 'node_modules/**', 'dist/**'],
+    // Subprocess/snapshot suites each run `npm run build` in beforeAll; parallel
+    // file workers can race on dist/ and produce a stale binary (exit 1 vs 5 flakes).
+    fileParallelism: false,
     coverage: {
       provider: 'v8',
       reporter: ['text', 'text-summary', 'json-summary', 'html'],