fix(publish): retain Redis command assets

Author: chrisbbreuer

package.json

@@ -26,7 +26,7 @@
   ],
   "scripts": {
     "build": "bun --bun build.ts",
-    "verify:package": "bun --bun run build && bun scripts/verify-published-artifact.ts",
+    "verify:package": "bun run --cwd packages/bun-queue build.ts && bun scripts/verify-published-artifact.ts",
     "compile": "bun build ./bin/cli.ts --compile --minify --outfile bin/queue",
     "compile:all": "bun run compile:linux-x64 && bun run compile:linux-arm64 && bun run compile:windows-x64 && bun run compile:darwin-x64 && bun run compile:darwin-arm64",
     "compile:linux-x64": "bun build ./bin/cli.ts --compile --minify --target=bun-linux-x64 --outfile bin/queue-linux-x64",

packages/bun-queue/build.ts

@@ -1,13 +1,24 @@
 import { dts } from 'bun-plugin-dtsx'
+import { cpSync, rmSync, statSync } from 'node:fs'
 
 // Library build only. The CLI (bin/cli.ts) is shipped as a compiled
 // binary via the `compile` script, not through dist — including it here
 // shifted Bun's output root and emitted dist/src/index.js instead of the
 // dist/index.js that package exports resolve to.
+rmSync('./dist', { recursive: true, force: true })
+
 await Bun.build({
   entrypoints: ['src/index.ts'],
   outdir: './dist',
   target: 'bun',
   minify: true,
   plugins: [dts({ build: { config: { root: 'src' } } })],
 })
+
+// Redis commands are runtime assets loaded relative to the compiled module.
+// Keep the package-local build used by prepublishOnly complete on a clean
+// checkout instead of relying on a prior root build to populate dist.
+cpSync('src/commands', 'dist/commands', {
+  recursive: true,
+  filter: source => statSync(source).isDirectory() || source.endsWith('.lua'),
+})

scripts/verify-published-artifact.ts

@@ -3,6 +3,29 @@
 import { existsSync } from 'node:fs'
 import { resolve } from 'node:path'
 
+interface PackedFile { path: string }
+interface PackReport { files?: PackedFile[] }
+
+const pack = Bun.spawnSync([
+  'npm',
+  'pack',
+  '--dry-run',
+  '--json',
+  '--workspace',
+  'packages/bun-queue',
+], {
+  cwd: resolve(import.meta.dir, '..'),
+  stdout: 'pipe',
+  stderr: 'pipe',
+})
+if (!pack.success)
+  throw new Error(`npm pack verification failed: ${pack.stderr.toString()}`)
+
+const reports = JSON.parse(pack.stdout.toString()) as PackReport[]
+const packedFiles = new Set(reports[0]?.files?.map(file => file.path) || [])
+if (!packedFiles.has('dist/commands/addStandardJob-8.lua') || !packedFiles.has('dist/commands/includes/storeJob.lua'))
+  throw new Error('Publishable npm tarball is missing Redis command assets')
+
 const commandDir = resolve(import.meta.dir, '../packages/bun-queue/dist/commands')
 if (!existsSync(resolve(commandDir, 'addStandardJob-8.lua')) || !existsSync(resolve(commandDir, 'includes/storeJob.lua')))
   throw new Error('Published dist is missing Redis command assets')
@@ -22,7 +45,7 @@ try {
   if (stored?.data?.artifact !== 'published-dist') throw new Error('Published queue could not round-trip a job')
   await queue.removeJob(job.id)
   if (await queue.getJob(job.id)) throw new Error('Published queue could not remove a job')
-  console.log(`Published queue artifact loaded ${commandDir} and passed a live Redis round trip`)
+  console.log(`Publishable npm artifact includes ${packedFiles.size} files, loaded ${commandDir}, and passed a live Redis round trip`)
 }
 finally {
   await queue.empty()