feat: validate options

__test__/utils.spec.ts

@@ -1,6 +1,14 @@
 import path from 'path'
 import { describe, expect, it, test } from 'vitest'
-import { eq, extractHashFromFileName, getContentHash, isPublicTypescript, linebreak, setEol } from '../src/utils'
+import {
+  eq,
+  extractHashFromFileName,
+  getContentHash,
+  isPublicTypescript,
+  linebreak,
+  setEol,
+  validateOptions,
+} from '../src/utils'
 import { getGlobalConfig, setGlobalConfig } from '../src/utils/globalConfig'
 
 describe('vite-plugin-public-typescript', () => {
@@ -78,4 +86,32 @@ describe('vite-plugin-public-typescript', () => {
     expect(hash1).toBe('1234')
     expect(hash2).toBe('1234')
   })
+
+  test('should validate options success', () => {
+    const opts = {
+      inputDir: 'publicTypescript',
+      outputDir: '/',
+      manifestName: 'manifest',
+      hash: true,
+      ssrBuild: false,
+      esbuildOptions: {},
+      sideEffects: false,
+    }
+
+    expect(() => validateOptions(opts)).not.toThrowError()
+  })
+
+  test('should validate options throw error', () => {
+    const opts = {
+      inputDir: '/publicTypescript/foo',
+      outputDir: 'js',
+      manifestName: 'manifest',
+      hash: true,
+      ssrBuild: false,
+      esbuildOptions: {},
+      sideEffects: false,
+    }
+
+    expect(() => validateOptions(opts)).toThrowError('dir')
+  })
 })

playground/spa/publicTypescript/manifest.json

@@ -1,5 +1,5 @@
 {
-  "haha": "js/haha.bdaaba63.js",
-  "index": "js/index.93041bfa.js",
-  "test": "js/test.b7d6e86f.js"
+  "haha": "/haha.bdaaba63.js",
+  "index": "/index.93041bfa.js",
+  "test": "/test.b7d6e86f.js"
 }

playground/spa/vite.config.ts

@@ -18,7 +18,7 @@ export default defineConfig({
       inputDir: 'publicTypescript',
       manifestName: 'manifest',
       hash: true,
-      outputDir: 'js',
+      outputDir: '/',
     }),
 
     {

src/index.ts

@@ -5,7 +5,7 @@ import glob from 'tiny-glob'
 import type { BuildOptions } from 'esbuild'
 import Watcher from 'watcher'
 import fs from 'fs-extra'
-import { TS_EXT, debug, eq, isEmptyObject, isPublicTypescript, reloadPage } from './utils'
+import { TS_EXT, debug, eq, isEmptyObject, isPublicTypescript, reloadPage, validateOptions } from './utils'
 import { build, deleteOldJsFile, esbuildTypescript } from './utils/build'
 import { ManifestCache } from './utils/manifestCache'
 import { getGlobalConfig, setGlobalConfig } from './utils/globalConfig'
@@ -25,6 +25,7 @@ export interface VPPTPluginOptions {
   inputDir?: string
   /**
    * @description output public javascript dir, relative to `publicDir`
+   * @note outputDir should start with '/'
    * @default '/'
    */
   outputDir?: string
@@ -75,6 +76,8 @@ export function publicTypescript(options: VPPTPluginOptions = {}) {
     ...options,
   }
 
+  validateOptions(opts)
+
   debug('options:', opts)
 
   let config: ResolvedConfig

src/utils/index.ts

@@ -125,3 +125,27 @@ export function extractHashFromFileName(filename: string, hash: VPPTPluginOption
   }
   return ''
 }
+
+export function validateOptions(options: Required<VPPTPluginOptions>) {
+  const { outputDir } = options
+  // ensure outputDir is Dir
+  if (!/^\/([a-zA-Z0-9]+\/)*[a-zA-Z0-9]*$/.test(outputDir)) {
+    throw new Error(`outputDir must be a directory, but got ${outputDir}`)
+  } else {
+    if (outputDir.length > 1 && outputDir.endsWith('/')) {
+      // remove last slash
+      options.outputDir = outputDir.replace(/\/$/, '')
+    }
+  }
+
+  // ensure inputDir is Dir
+  const { inputDir } = options
+  if (!/^([a-zA-Z0-9]+\/)*[a-zA-Z0-9]*$/.test(inputDir)) {
+    throw new Error(`inputDir must be a directory, but got ${inputDir}`)
+  } else {
+    if (inputDir.endsWith('/')) {
+      // remove last slash
+      options.inputDir = inputDir.replace(/\/$/, '')
+    }
+  }
+}