refactor: simplify redudant calls, polish code

.github/workflows/ci.yml

@@ -41,7 +41,9 @@ jobs:
         run: pnpm build
 
       - name: Test
-        run: pnpm test
+        run: |
+          pnpm test
+          pnpm test:post
 
   release:
     runs-on: ubuntu-latest

package.json

@@ -8,6 +8,7 @@
   "scripts": {
     "test": "jest --env node",
     "test:update": "TEST_UPDATE_SNAPSHOT=1 pnpm test",
+    "test:post": "POST_BUILD=1 pnpm jest test/compile.test.ts test/integration.test.ts",
     "clean": "rm -rf ./dist",
     "typecheck": "tsc --noEmit",
     "prepublishOnly": "pnpm clean && pnpm build && chmod +x ./dist/bin/cli.js && pnpm test",

src/build-config.ts

@@ -49,8 +49,6 @@ const swcMinifyOptions = {
   compress: true,
   format: {
     comments: 'some',
-    // wrapFuncArgs: false,
-    // preserveAnnotations: true,
   },
   mangle: {
     toplevel: true,
@@ -309,13 +307,12 @@ function buildOutputConfigs(
           '.d.ts',
       )
 
-  // If there's dts file, use `output.file`
-  const dtsPathConfig = dtsFile ? { dir: dirname(dtsFile) } : { dir: dtsDir }
+  const dtsPathConfig = { dir: dtsFile ? dirname(dtsFile) : dtsDir }
   const outputFile: string = (dtsFile || file)!
 
   return {
     name: pkg.name || name,
-    ...(dts ? dtsPathConfig : { dir: dirname(outputFile!) }),
+    ...(dts ? dtsPathConfig : { dir: dirname(outputFile) }),
     format,
     exports: 'named',
     esModule: useEsModuleMark || 'if-default-prop',
@@ -325,7 +322,7 @@ function buildOutputConfigs(
     sourcemap: options.sourcemap,
     manualChunks: splitChunks,
     chunkFileNames: '[name].js',
-    entryFileNames: basename(outputFile!),
+    entryFileNames: basename(outputFile),
   }
 }
 

src/plugins/prepend-directives.ts

@@ -9,7 +9,6 @@ export function prependDirectives(): Plugin {
         const moduleInfo = this.getModuleInfo(id)
         if (moduleInfo?.meta?.preserveDirectives) {
           const firstDirective = moduleInfo.meta.preserveDirectives.directives[0]
-          console.log('prepend', id, firstDirective)
           if (firstDirective) {
             const directive = firstDirective.value
             const directiveCode = `'${directive}';`

src/plugins/raw-plugin.ts

@@ -2,20 +2,19 @@ import { type FilterPattern, createFilter } from '@rollup/pluginutils'
 import type { Plugin } from 'rollup'
 
 export function rawContent({ exclude }: { exclude: FilterPattern }): Plugin {
-  /\.(data|txt)$/
   const filter = createFilter(['**/*.data', '**/*.txt'], exclude)
 
   return {
-    name: "string",
+    name: 'string',
 
     transform(code, id) {
       if (filter(id)) {
         return {
-          code: `export default ${JSON.stringify(code)}`,
-          map: undefined,
+          code: `const data = ${JSON.stringify(code)};\nexport default data;`,
+          map: null,
         }
       }
-      return undefined
+      return null
     }
   }
 }

test/compile.test.ts

@@ -2,8 +2,8 @@ import fs, { promises as fsp } from 'fs'
 import { resolve, dirname, extname } from 'path'
 import { existsFile, fullExtension } from './testing-utils'
 
-const assetPath = process.env.POST_BUILD ? '../dist' : '../src/bundle'
-const bundle = require(assetPath).default
+const assetPath = process.env.POST_BUILD ? '..' : '../src/index.ts'
+const bundle = require(assetPath).bundle
 
 jest.setTimeout(10 * 60 * 1000)