fix browser for zips

Author: willwade

.github/workflows/publish.yml

@@ -47,4 +47,5 @@ jobs:
           npm version "${{ steps.release_version.outputs.version }}" --no-git-tag-version
 
       - run: npm run build:all --if-present
+      - run: npm run verify:browser-build --if-present
       - run: npm publish --access public

package.json

@@ -94,6 +94,7 @@
     "format": "prettier --write \"src/**/*.{js,ts}\" \"test/**/*.{js,ts}\" \"*.{js,ts,json,md}\"",
     "format:check": "prettier --check \"src/**/*.{js,ts}\" \"test/**/*.{js,ts}\" \"*.{js,ts,json,md}\"",
     "smoke:browser": "node scripts/smoke-browser-bundle.js",
+    "verify:browser-build": "node scripts/verify-browser-build.js",
     "test": "npm run build && jest",
     "test:watch": "npm run build && jest --watch",
     "test:coverage": "npm run build && jest --coverage",
@@ -102,8 +103,8 @@
     "docs": "typedoc",
     "coverage:report": "node scripts/coverage-analysis.js",
     "type-check": "tsc --noEmit",
-    "prepublishOnly": "npm run build",
-    "prepack": "npm run build"
+    "prepublishOnly": "npm run build:all && npm run verify:browser-build",
+    "prepack": "npm run build:all && npm run verify:browser-build"
   },
   "keywords": [
     "aac",

scripts/verify-browser-build.js

@@ -0,0 +1,28 @@
+const fs = require('fs');
+const path = require('path');
+
+const requiredFiles = [
+  path.join('dist', 'browser', 'index.browser.js'),
+  path.join('dist', 'index.browser.d.ts'),
+];
+
+const missing = requiredFiles.filter((file) => !fs.existsSync(file));
+
+if (missing.length > 0) {
+  console.error('Browser build verification failed. Missing files:');
+  missing.forEach((file) => {
+    console.error(`- ${file}`);
+  });
+  process.exit(1);
+}
+
+const empty = requiredFiles.filter((file) => fs.statSync(file).size === 0);
+if (empty.length > 0) {
+  console.error('Browser build verification failed. Empty files:');
+  empty.forEach((file) => {
+    console.error(`- ${file}`);
+  });
+  process.exit(1);
+}
+
+console.log('Browser build verification passed.');

src/processors/gridsetProcessor.ts

@@ -553,9 +553,14 @@ class GridsetProcessor extends BaseProcessor {
 
     // Debug: log all entry names
     console.log('[Gridset] Total zip entries:', entries.length);
-    const gridEntries = entries.filter(
-      (e) => e.entryName.startsWith('Grids/') && e.entryName.endsWith('grid.xml')
-    );
+    const normalizeEntryName = (entryName: string): string =>
+      entryName.replace(/\\/g, '/').toLowerCase();
+    const isGridXmlEntry = (entryName: string): boolean => {
+      const normalized = normalizeEntryName(entryName);
+      if (!normalized.endsWith('grid.xml')) return false;
+      return normalized.startsWith('grids/') || normalized.includes('/grids/');
+    };
+    const gridEntries = entries.filter((e) => isGridXmlEntry(e.entryName));
     console.log('[Gridset] Grid XML entries found:', gridEntries.length);
     if (gridEntries.length > 0) {
       console.log(
@@ -569,7 +574,7 @@ class GridsetProcessor extends BaseProcessor {
     const gridIdToNameMap = new Map<string, string>();
 
     for (const entry of entries) {
-      if (entry.entryName.startsWith('Grids/') && entry.entryName.endsWith('grid.xml')) {
+      if (isGridXmlEntry(entry.entryName)) {
         try {
           const xmlContent = decodeText(await readEntryBuffer(entry));
           const data = parser.parse(xmlContent);
@@ -605,7 +610,7 @@ class GridsetProcessor extends BaseProcessor {
     // Second pass: process each grid file in the gridset
     for (const entry of entries) {
       // Only process files named grid.xml under Grids/ (any subdir)
-      if (entry.entryName.startsWith('Grids/') && entry.entryName.endsWith('grid.xml')) {
+      if (isGridXmlEntry(entry.entryName)) {
         let xmlContent: string;
         try {
           const buffer = await readEntryBuffer(entry);

src/processors/obfProcessor.ts

@@ -449,7 +449,7 @@ class ObfProcessor extends BaseProcessor {
     }
 
     // If input is a string path and ends with .obf, treat as JSON
-    if (typeof filePathOrBuffer === 'string' && filePathOrBuffer.endsWith('.obf')) {
+    if (typeof filePathOrBuffer === 'string' && filePathOrBuffer.toLowerCase().endsWith('.obf')) {
       try {
         const content = readTextFromInput(filePathOrBuffer);
         const boardData = tryParseObfJson(content);
@@ -502,7 +502,10 @@ class ObfProcessor extends BaseProcessor {
 
     // Otherwise, try as ZIP (.obz). Detect likely zip signature first; throw if neither JSON nor ZIP
     function isLikelyZip(input: ProcessorInput): boolean {
-      if (typeof input === 'string') return input.endsWith('.zip') || input.endsWith('.obz');
+      if (typeof input === 'string') {
+        const lowered = input.toLowerCase();
+        return lowered.endsWith('.zip') || lowered.endsWith('.obz');
+      }
       const bytes = readBinaryFromInput(input);
       return bytes.length >= 2 && bytes[0] === 0x50 && bytes[1] === 0x4b;
     }
@@ -531,7 +534,7 @@ class ObfProcessor extends BaseProcessor {
     const obfEntries: Array<{ name: string; file: JSZip.JSZipObject }> = [];
     zip.forEach((relativePath: string, file: JSZip.JSZipObject) => {
       if (file.dir) return;
-      if (relativePath.endsWith('.obf')) {
+      if (relativePath.toLowerCase().endsWith('.obf')) {
         obfEntries.push({ name: relativePath, file });
       }
     });

tsconfig.browser.json

@@ -5,7 +5,8 @@
     "module": "ES2020",
     "moduleResolution": "node",
     "lib": ["ES2020", "DOM"],
-    "declaration": false
+    "declaration": true,
+    "declarationDir": "./dist"
   },
   "include": ["src/index.browser.ts"]
 }