its done?

Author: SpaceFoon

.claude/settings.local.json

@@ -22,7 +22,16 @@
       "Bash(node --experimental-vm-modules node_modules/jest/bin/jest.js src/__tests__/selfOverwrite.test.ts --runInBand --verbose 2>&1 | grep -E '\\(✓|✕|PASS|FAIL|Tests:|✓|×|√|●|RUNS|✗|passed|failed|  \\)')",
       "Bash(node --experimental-vm-modules node_modules/jest/bin/jest.js src/__tests__/selfOverwrite.test.ts --runInBand --verbose 2>&1 | grep -E '\\(PASS|FAIL|Tests:|✓|✕|✗|×|√\\)' | tail -5)",
       "Bash(node --experimental-vm-modules node_modules/jest/bin/jest.js \"src/__tests__/\" --runInBand 2>&1 | grep -E '\\(FAIL |● \\)' | head -20)",
-      "Bash(node --experimental-vm-modules node_modules/jest/bin/jest.js src/__tests__/metadataFormatting.integration.test.ts src/__tests__/converterWorker.paths.test.ts --runInBand 2>&1 | grep -B1 -A5 '●.*should escape\\\\|●.*uses runtime\\\\|●.*throws' | head -40)"
+      "Bash(node --experimental-vm-modules node_modules/jest/bin/jest.js src/__tests__/metadataFormatting.integration.test.ts src/__tests__/converterWorker.paths.test.ts --runInBand 2>&1 | grep -B1 -A5 '●.*should escape\\\\|●.*uses runtime\\\\|●.*throws' | head -40)",
+      "Bash(node -e \"const {version} = require\\('dotenv/package.json'\\); console.log\\(version\\)\" 2>/dev/null || grep '\"dotenv\"' package.json)",
+      "Bash(node -e \"const dotenv = require\\('dotenv'\\); console.log\\(Object.keys\\(dotenv\\)\\)\")",
+      "Bash(node -e \"require\\('dotenv'\\).config\\({ quiet: true }\\)\" 2>&1)",
+      "Bash(node -e \"const dotenv = require\\('dotenv'\\); const r = dotenv.config.__proto__; console.log\\(dotenv.config.toString\\(\\).slice\\(0, 500\\)\\)\")",
+      "Bash(node -e \"require\\('dotenv'\\).configDotenv\\(\\)\" 2>&1)",
+      "Bash(DOTENV_LOG_LEVEL=error node -e \"require\\('dotenv'\\).config\\(\\)\" 2>&1)",
+      "Bash(node -e \"require\\('dotenv'\\).config\\({ debug: false, silent: true, logLevel: 'error' }\\)\" 2>&1)",
+      "Bash(npm run build 2>&1 | tail -5)",
+      "Bash(wc -l /mnt/c/Users/Darren/Ez-Game-Audio-Conversion/node_modules/cfonts/fonts/*.json)"
     ],
     "deny": [],
     "ask": []

.github/workflows/RELEASE-GUIDE.md

@@ -67,6 +67,8 @@ Each platform creates a separate release archive:
 - `add_context_menu.bat` - Windows context menu integration
 - `remove_context_menu.bat` - Uninstall context menu
 
+Note: For Windows builds, the packaged `.exe` icon + version metadata is applied automatically during `npm run package`. To disable this (rare), set `POST_SEA_SKIP_ICON=1`.
+
 ### Linux (`EZ-Game-Audio-Linux.zip`)
 - `ez-game-audio` - Single executable application
 - `ffmpeg` - Audio conversion binary

.jest-latest.json

@@ -47,8 +47,8 @@ npm run clean            # Remove dist/ and release/ folders
 
 **Icon/Metadata (Optional):**
 - Icon exists at `media/ico/icon.ico`
-- Manual icon application script at `src/ico/icon.js` (Windows only, requires `resedit` package)
-- Not currently integrated into automated build process
+- Icon + version metadata is applied automatically for Windows builds during `npm run package` via `src/ico/icon.js` (uses `resedit`)
+- To skip icon injection: set `POST_SEA_SKIP_ICON=1`
 
 ### Testing
 

CLAUDE.md

@@ -64,8 +64,10 @@
     "markdown-to-html-cli": "^5.0.3",
     "md-to-pdf": "^5.2.4",
     "nodemon": "^3.1.10",
+    "pe-library": "^2.0.1",
     "postject": "^1.0.0-alpha.6",
     "prettier": "^3.6.2",
+    "resedit": "^3.0.2",
     "rimraf": "^6.0.1",
     "ts-jest": "^29.4.1",
     "ts-node": "^10.9.2",
@@ -87,15 +89,32 @@
       "<rootDir>/build/",
       "<rootDir>/release/"
     ],
-    "extensionsToTreatAsEsm": [".ts"],
+    "extensionsToTreatAsEsm": [
+      ".ts"
+    ],
     "moduleNameMapper": {
       "^(\\.{1,2}/.*)\\.js$": "$1"
     },
     "transform": {
-      "^.+\\.ts$": ["ts-jest", { "useESM": true }]
+      "^.+\\.ts$": [
+        "ts-jest",
+        {
+          "useESM": true
+        }
+      ]
     },
-    "moduleFileExtensions": ["ts", "js", "cjs", "jsx", "json", "node"],
-    "testMatch": ["**/__tests__/**/*.test.ts", "**/__tests__/**/*.test.js"],
+    "moduleFileExtensions": [
+      "ts",
+      "js",
+      "cjs",
+      "jsx",
+      "json",
+      "node"
+    ],
+    "testMatch": [
+      "**/__tests__/**/*.test.ts",
+      "**/__tests__/**/*.test.js"
+    ],
     "verbose": false,
     "forceExit": true,
     "testTimeout": 30000

media/ico/audio-x-generic_36263(3).ico media/ico/audio-x-generic_36263.icomedia/ico/audio-x-generic_36263(3).ico renamed to media/ico/audio-x-generic_36263.ico

@@ -8,7 +8,7 @@
  */
 
 import { execSync } from 'child_process';
-import { copyFileSync, writeFileSync, mkdirSync, existsSync, rmSync } from 'fs';
+import { writeFileSync, mkdirSync, existsSync, rmSync } from 'fs';
 import { join } from 'path';
 import { platform } from 'os';
 
@@ -20,6 +20,30 @@ const SEA_CONFIG_PATH = 'sea-config.json';
 const SEA_BLOB_PATH = join(RELEASE_DIR, 'sea-prep.blob');
 const OUTPUT_EXE = join(RELEASE_DIR, isWindows ? `${APP_NAME}.exe` : APP_NAME);
 
+function tryApplyWindowsIcon() {
+  if (!isWindows) return;
+  if (process.env.POST_SEA_SKIP_ICON === '1') {
+    console.log('[SEA] Skipping icon injection (POST_SEA_SKIP_ICON=1)');
+    return;
+  }
+
+  const iconScript = join('src', 'ico', 'icon.js');
+  if (!existsSync(iconScript)) {
+    console.warn('[SEA] Icon script not found; skipping icon injection');
+    return;
+  }
+
+  console.log('[SEA] Applying Windows icon + version metadata...');
+  try {
+    execSync(`node "${iconScript}" "${OUTPUT_EXE}"`, { stdio: 'inherit' });
+  } catch (error) {
+    console.warn(
+      '[SEA] Warning: icon injection failed (continuing):',
+      error.message
+    );
+  }
+}
+
 console.log('[SEA] Building Single Executable Application...\n');
 
 // Step 1: Ensure release directory exists
@@ -29,10 +53,18 @@ mkdirSync(RELEASE_DIR, { recursive: true });
 // Step 1.5: Bundle ESM to CJS for SEA compatibility
 // The code handles import.meta being empty by falling back to CJS __filename
 // Exclude cfonts entirely - it uses dynamic require for fonts that can't work in SEA
+console.log('[SEA] Injecting banner snapshot...');
+try {
+  execSync('node scripts/gen-banner.js', { stdio: 'inherit' });
+} catch (error) {
+  console.error('[SEA] Failed to generate banner snapshot:', error.message);
+  process.exit(1);
+}
+
 console.log('[SEA] Bundling application with esbuild...');
 try {
   execSync(
-    `npx esbuild dist/app.js --bundle --platform=node --format=cjs --outfile=${BUNDLED_APP} --external:worker_threads --external:cfonts --log-override:empty-import-meta=silent`,
+    `npx esbuild dist/app.js --bundle --platform=node --format=cjs --minify --tree-shaking=true --legal-comments=none --outfile=${BUNDLED_APP} --external:worker_threads --external:cfonts --define:__SEA_BUILD__=true --log-override:empty-import-meta=silent`,
     {
       stdio: 'inherit',
     }
@@ -61,7 +93,7 @@ const WORKER_BUNDLE = join(RELEASE_DIR, 'dist', 'converterWorker.cjs');
 mkdirSync(join(RELEASE_DIR, 'dist'), { recursive: true });
 try {
   execSync(
-    `npx esbuild dist/converterWorker.js --bundle --platform=node --format=cjs --outfile=${WORKER_BUNDLE} --log-override:empty-import-meta=silent`,
+    `npx esbuild dist/converterWorker.js --bundle --platform=node --format=cjs --minify --tree-shaking=true --legal-comments=none --outfile=${WORKER_BUNDLE} --log-override:empty-import-meta=silent`,
     {
       stdio: 'inherit',
     }
@@ -85,7 +117,12 @@ try {
 // Step 4: Copy Node.js executable
 console.log('[SEA] Copying Node.js executable...');
 try {
-  copyFileSync(process.execPath, OUTPUT_EXE);
+  // Use system copy command instead of Node API to avoid file lock issues
+  // when npm itself holds the Node.exe lock during package builds
+  const copyCmd = isWindows
+    ? `cmd /c copy /Y "${process.execPath}" "${OUTPUT_EXE}"`
+    : `cp "${process.execPath}" "${OUTPUT_EXE}"`;
+  execSync(copyCmd, { stdio: 'inherit' });
 } catch (error) {
   console.error('[SEA] Failed to copy Node.js executable:', error.message);
   process.exit(1);
@@ -116,6 +153,9 @@ try {
   process.exit(1);
 }
 
+// Step 5.5: Inject custom icon/version info (best-effort)
+tryApplyWindowsIcon();
+
 // Step 6: Cleanup temporary files
 console.log('[SEA] Cleaning up temporary files...');
 try {

package-lock.json

@@ -0,0 +1,47 @@
+#!/usr/bin/env node
+/**
+ * Captures the exact cfonts ANSI output and injects it into dist/banner.js,
+ * replacing the '__SEA_BANNER__' sentinel before esbuild bundles the app.
+ * Must run after `tsc` and before `esbuild`.
+ */
+
+import { readFileSync, writeFileSync } from 'fs';
+
+// Force chalk to output full ANSI colour codes even when stdout is not a TTY.
+process.env.FORCE_COLOR = '3';
+// Render left-aligned; runtime centering will adapt to actual terminal width.
+
+const { default: cfonts } = await import('cfonts');
+const { render } = cfonts;
+
+const result = render('|||EZ Game|Audio', {
+  font: 'huge',
+  align: 'left',
+  gradient: ['green', '#f80'],
+  background: 'black',
+  independentGradient: true,
+  transitionGradient: false,
+  env: 'node',
+});
+
+if (!result) {
+  console.error('[gen-banner] cfonts render() returned false — aborting.');
+  process.exit(1);
+}
+
+const bannerPath = 'dist/banner.js';
+let src = readFileSync(bannerPath, 'utf8');
+
+const sentinel = "const BANNER_SNAPSHOT = '__SEA_BANNER__'";
+if (!src.includes(sentinel)) {
+  console.error(`[gen-banner] Sentinel ${sentinel} not found in ${bannerPath}`);
+  process.exit(1);
+}
+
+src = src.replace(
+  "const BANNER_SNAPSHOT = '__SEA_BANNER__'",
+  `const BANNER_SNAPSHOT = ${JSON.stringify(result.string)}`
+);
+writeFileSync(bannerPath, src, 'utf8');
+
+console.log('[gen-banner] Banner snapshot injected into dist/banner.js');

package.json

@@ -195,6 +195,34 @@ function ensureFile(filePath, message) {
   }
 }
 
+function tryApplyWindowsIcon({ exePath }) {
+  // Best-effort: icon injection should never fail a release build.
+  if (!isTargetWindows) return;
+  if (process.env.POST_SEA_SKIP_ICON === '1') {
+    console.log('[post-sea] Skipping icon injection (POST_SEA_SKIP_ICON=1)');
+    return;
+  }
+
+  const iconScript = join(rootDir, 'src', 'ico', 'icon.js');
+  if (!existsSync(iconScript)) {
+    console.warn('[post-sea] Icon script missing; skipping:', iconScript);
+    return;
+  }
+
+  console.log('[post-sea] Applying Windows icon + version info...');
+  try {
+    execSync(`node "${iconScript}" "${exePath}"`, {
+      cwd: rootDir,
+      stdio: 'inherit',
+    });
+  } catch (error) {
+    console.warn(
+      '[post-sea] Icon injection failed (continuing):',
+      error instanceof Error ? error.message : String(error)
+    );
+  }
+}
+
 function cleanupIntermediateArtifacts() {
   console.log('[post-sea] Cleaning up intermediate files...');
   // Remove staging/temp artifacts.
@@ -329,6 +357,9 @@ const manifestArtifactPaths = [];
 let releaseReady = false;
 
 try {
+  // Apply Windows icon/version metadata before copying into stage.
+  tryApplyWindowsIcon({ exePath: builtExePath });
+
   // Stage directory structure
   console.log('[post-sea] Staging files...');
   emptyDir(stageDir);
@@ -426,9 +457,33 @@ try {
   }
 
   // Ensure README assets are available for both ZIP and PDF rendering.
+  // Only copy the media files that README.md actually references — not the entire media/ folder.
   const repoMediaDir = join(rootDir, 'media');
   if (existsSync(repoMediaDir)) {
-    copyDir(repoMediaDir, join(readmesDir, 'media'));
+    const readmeSrc = join(rootDir, 'README.md');
+    const referencedMediaFiles = [];
+    if (existsSync(readmeSrc)) {
+      const readmeText = readFileSync(readmeSrc, 'utf8');
+      // Match Markdown image/link syntax and HTML src/href attributes pointing into media/
+      const mdRe = /\]\((media\/[^)\s"]+)\)/g;
+      const htmlRe = /(?:src|href)="(media\/[^"]+)"/g;
+      for (const re of [mdRe, htmlRe]) {
+        let m;
+        while ((m = re.exec(readmeText)) !== null) {
+          referencedMediaFiles.push(m[1]);
+        }
+      }
+    }
+    for (const relPath of [...new Set(referencedMediaFiles)]) {
+      const srcFile = join(rootDir, relPath);
+      if (!existsSync(srcFile)) {
+        console.warn(`[post-sea] Referenced media file not found: ${srcFile}`);
+        continue;
+      }
+      const destFile = join(readmesDir, relPath);
+      mkdirSync(dirname(destFile), { recursive: true });
+      copyFileSync(srcFile, destFile);
+    }
   }
 
   // Copy existing docs/readmes if present
@@ -440,10 +495,11 @@ try {
     for (const entry of entries) {
       if (!entry.isFile()) continue;
       if (/^README\.(html|pdf)$/i.test(entry.name)) continue;
-      copyFileSync(
-        join(legacyDocsDir, entry.name),
-        join(readmesDir, entry.name)
-      );
+      // Place How-to-start.txt at the archive root for easy first-run discovery.
+      const dest = /^how-to-start\.txt$/i.test(entry.name)
+        ? join(stageDir, entry.name)
+        : join(readmesDir, entry.name);
+      copyFileSync(join(legacyDocsDir, entry.name), dest);
     }
   }