Fix Windows build: add zip pkg, portable embed (od replaces xxd)

1. Add zip to MSYS2 build packages (both dry-run and release).
2. embed-frontend.sh: replace xxd with od+sed for hex dump — xxd
   requires vim which isn't in MSYS2 CLANG64 by default.
3. embed-frontend.sh: flip IS_MACOS→IS_LINUX — Linux gets ld -r -b
   binary (ELF-only), everything else (macOS, Windows) gets the
   portable C-array approach via od.
4. release.yml: same Windows archive fix (msys2 shell + .exe detect).

Author: DeusData

.github/workflows/dry-run.yml

@@ -191,6 +191,7 @@ jobs:
             mingw-w64-clang-x86_64-clang
             mingw-w64-clang-x86_64-zlib
             make
+            zip
 
       - uses: actions/setup-node@v4
         with:
@@ -204,7 +205,6 @@ jobs:
         shell: msys2 {0}
         run: |
           BIN=build/c/codebase-memory-mcp
-          # MSYS2 Clang produces .exe extension
           [ -f "${BIN}.exe" ] && BIN="${BIN}.exe"
           cp "$BIN" codebase-memory-mcp.exe
           zip codebase-memory-mcp-windows-amd64.zip codebase-memory-mcp.exe

.github/workflows/release.yml

@@ -187,6 +187,7 @@ jobs:
             mingw-w64-clang-x86_64-clang
             mingw-w64-clang-x86_64-zlib
             make
+            zip
 
       - uses: actions/setup-node@v4
         with:

scripts/embed-frontend.sh

@@ -19,10 +19,10 @@ OUTPUT_DIR="${2:?Usage: embed-frontend.sh <dist_dir> <output_dir>}"
 rm -rf "$OUTPUT_DIR"
 mkdir -p "$OUTPUT_DIR"
 
-# Detect platform
-IS_MACOS=false
-if [[ "$(uname -s)" == "Darwin" ]]; then
-    IS_MACOS=true
+# Detect platform — Linux uses ld -r -b binary, everything else uses xxd+cc
+IS_LINUX=false
+if [[ "$(uname -s)" == "Linux" ]] && ! [[ "$(uname -s)" =~ MINGW|MSYS ]]; then
+    IS_LINUX=true
 fi
 
 # Content-type detection
@@ -68,25 +68,24 @@ for file in "${FILES[@]}"; do
     mangled=$(mangle "$rel")
     obj="$OUTPUT_DIR/embed_${mangled}.o"
 
-    if $IS_MACOS; then
-        # macOS: use custom section + assembly approach
-        # ld -r -b binary not available on macOS default ld
-        # Instead, use xxd to create a C array (for macOS compatibility)
+    if $IS_LINUX; then
+        # Linux: ld -r -b binary (zero bloat, ELF only)
+        abs_obj="$(cd "$(dirname "$0")/.." && pwd)/$obj"
+        (cd "$DIST_DIR" && ld -r -b binary -o "$abs_obj" "$rel")
+    else
+        # macOS/Windows/MSYS2: generate C byte array + cc (no xxd dependency)
         local_c="$OUTPUT_DIR/embed_${mangled}.c"
         local_sym="_binary_${mangled}"
 
         echo "/* Generated from $rel */" > "$local_c"
         echo "const unsigned char ${local_sym}_data[] = {" >> "$local_c"
-        xxd -i < "$file" >> "$local_c"
+        # Use od (POSIX) to generate hex bytes — works everywhere without xxd/vim
+        od -An -tx1 -v < "$file" | tr -s ' ' '\n' | grep -v '^$' | sed 's/^/0x/; s/$/,/' | paste -sd' ' - | fold -s -w 76 | sed 's/^/  /' >> "$local_c"
         echo "};" >> "$local_c"
         echo "const unsigned int ${local_sym}_size = sizeof(${local_sym}_data);" >> "$local_c"
 
-        cc -c -O2 -o "$obj" "$local_c"
+        ${CC:-cc} -c -O2 -o "$obj" "$local_c"
         rm -f "$local_c"
-    else
-        # Linux: ld -r -b binary (zero bloat)
-        abs_obj="$(cd "$(dirname "$0")/.." && pwd)/$obj"
-        (cd "$DIST_DIR" && ld -r -b binary -o "$abs_obj" "$rel")
     fi
 
     OBJ_FILES+=("$obj")
@@ -111,7 +110,7 @@ for file in "${FILES[@]}"; do
     mangled=$(mangle "$rel")
     sym="_binary_${mangled}"
 
-    if $IS_MACOS; then
+    if ! $IS_LINUX; then
         echo "extern const unsigned char ${sym}_data[];" >> "$ASSETS_C"
         echo "extern const unsigned int ${sym}_size;" >> "$ASSETS_C"
     else
@@ -132,7 +131,7 @@ for file in "${FILES[@]}"; do
     # URL path: /index.html for root, /assets/... for assets
     url_path="/$rel"
 
-    if $IS_MACOS; then
+    if ! $IS_LINUX; then
         echo "    {\"$url_path\", ${sym}_data, 0, \"$ct\"}," >> "$ASSETS_C"
     else
         echo "    {\"$url_path\", ${sym}_start, 0, \"$ct\"}," >> "$ASSETS_C"
@@ -142,11 +141,11 @@ done
 echo "};" >> "$ASSETS_C"
 echo "const int CBM_EMBEDDED_FILE_COUNT = ${#FILES[@]};" >> "$ASSETS_C"
 
-# Generate size fixup (for macOS where we have explicit size vars)
-if $IS_MACOS; then
+# Generate size fixup
+if $IS_LINUX; then
+    # Linux: compute size from start/end pointers (ld -r -b binary symbols)
     cat >> "$ASSETS_C" <<'SIZEINIT'
 
-/* Initialize sizes — called once or we use the _size vars directly */
 static void __attribute__((constructor)) init_embedded_sizes(void) {
     cbm_embedded_file_t *files = CBM_EMBEDDED_FILES;
 SIZEINIT
@@ -155,12 +154,12 @@ SIZEINIT
         rel="${FILES[$i]#$DIST_DIR/}"
         mangled=$(mangle "$rel")
         sym="_binary_${mangled}"
-        echo "    files[$i].size = ${sym}_size;" >> "$ASSETS_C"
+        echo "    files[$i].size = (unsigned int)(${sym}_end - ${sym}_start);" >> "$ASSETS_C"
     done
 
     echo "}" >> "$ASSETS_C"
 else
-    # Linux: compute size from start/end pointers
+    # macOS/Windows: use explicit _size vars from xxd-generated C arrays
     cat >> "$ASSETS_C" <<'SIZEINIT'
 
 static void __attribute__((constructor)) init_embedded_sizes(void) {
@@ -171,7 +170,7 @@ SIZEINIT
         rel="${FILES[$i]#$DIST_DIR/}"
         mangled=$(mangle "$rel")
         sym="_binary_${mangled}"
-        echo "    files[$i].size = (unsigned int)(${sym}_end - ${sym}_start);" >> "$ASSETS_C"
+        echo "    files[$i].size = ${sym}_size;" >> "$ASSETS_C"
     done
 
     echo "}" >> "$ASSETS_C"