fix: resolve issues #1, #2, #3

- Fix #1: Move deps to dependencies
- Fix #2: Use node:path for cross-platform
- Fix #3: Recursive traversal (already fixed)

Author: onmax

package.json

@@ -35,14 +35,16 @@
     "prepublishOnly": "pnpm build",
     "release": "release-it"
   },
+  "dependencies": {
+    "html-minifier-terser": "^7.2.0",
+    "lightningcss": "^1.30.2"
+  },
   "devDependencies": {
     "@antfu/eslint-config": "^6.2.0",
     "@types/html-minifier-terser": "^7.0.2",
     "@types/node": "^24.9.2",
     "astro": "^5.15.3",
     "eslint": "^9.18.0",
-    "html-minifier-terser": "^7.2.0",
-    "lightningcss": "^1.28.2",
     "release-it": "^19.0.5",
     "tsdown": "^0.15.12",
     "typescript": "^5.9.3",

src/single-file-builder.ts

@@ -1,4 +1,5 @@
 import type { FileInfo, FileSystemAdapter, SingleFileConfig } from './types'
+import * as path from 'node:path'
 import { minify } from 'html-minifier-terser'
 import { CssInliner } from './css-inliner'
 import { CssTransformer } from './css-transformer'
@@ -9,7 +10,7 @@ export class SingleFileBuilder {
   constructor(private fs: FileSystemAdapter) {}
 
   async build(buildDir: string, config: SingleFileConfig): Promise<void> {
-    const folder = buildDir.endsWith('/') ? buildDir : `${buildDir}/`
+    const folder = buildDir
 
     // Find all files recursively
     const files = this.findAllFiles(folder)
@@ -55,7 +56,7 @@ export class SingleFileBuilder {
 
     // Delete empty folders
     this.fs.readDir(folder).forEach((f) => {
-      const file = `${folder}${f}`
+      const file = path.join(folder, f)
       try {
         const stats = this.fs.stat(file)
         if (stats.isDirectory() && this.fs.readDir(file).length === 0) {
@@ -70,14 +71,14 @@ export class SingleFileBuilder {
 
   private findAllFiles(folder: string): string[] {
     return this.fs.readDir(folder).reduce<string[]>((acc, f) => {
-      const file = `${folder}${f}`
+      const file = path.join(folder, f)
       const stats = this.fs.stat(file)
 
       if (stats.isFile()) {
         acc.push(file)
       }
       else if (stats.isDirectory()) {
-        acc = acc.concat(this.findAllFiles(`${file}/`))
+        acc = acc.concat(this.findAllFiles(file))
       }
 
       return acc
@@ -87,10 +88,10 @@ export class SingleFileBuilder {
   private openFiles(files: string[], fileType: 'html' | 'css'): FileInfo[] {
     return files
       .filter(i => i.endsWith(`.${fileType}`))
-      .map(path => ({
-        contents: this.fs.readFile(path, 'utf8'),
-        path,
-        fileName: path.split('/').pop()!,
+      .map(filePath => ({
+        contents: this.fs.readFile(filePath, 'utf8'),
+        path: filePath,
+        fileName: path.basename(filePath),
       }))
   }
 }

test/integration/astro-island.test.ts

@@ -1,13 +1,11 @@
-import { describe, it, expect, beforeEach, afterEach } from 'vitest'
-import type { TransformOptions } from 'lightningcss'
-import { Buffer } from 'node:buffer'
 import * as fs from 'node:fs'
-import * as path from 'node:path'
 import * as os from 'node:os'
+import * as path from 'node:path'
+import { afterEach, beforeEach, describe, expect, it } from 'vitest'
 import { NodeFileSystemAdapter } from '../../src/file-system-adapter'
 import { SingleFileBuilder } from '../../src/single-file-builder'
 
-describe('Astro Island Support', () => {
+describe('astro Island Support', () => {
   let tmpDir: string
   let adapter: NodeFileSystemAdapter
   let builder: SingleFileBuilder
@@ -121,9 +119,8 @@ describe('Astro Island Support', () => {
 
     await builder.build(tmpDir, {})
 
-    const result = fs.readFileSync(htmlPath, 'utf8')
-
     // Future: could inline JS as base64
+    // const result = fs.readFileSync(htmlPath, 'utf8')
     // expect(result).toContain('data:text/javascript;base64,')
 
     // For now, just verify JS file is preserved