jiek 2.1.11 → 2.1.13-alpha.1

package/src/commands/publish.ts

@@ -1,8 +1,10 @@
+/* eslint-disable ts/strict-boolean-expressions */
 import * as childProcess from 'node:child_process'
 import fs from 'node:fs'
 import path from 'node:path'
+import process from 'node:process'
 
-import { bump, type BumperType, TAGS } from '@jiek/utils/bumper'
+import { type BumperType, TAGS, bump } from '@jiek/utils/bumper'
 import { program } from 'commander'
 import detectIndent from 'detect-indent'
 import { applyEdits, modify } from 'jsonc-parser'
@@ -91,13 +93,17 @@ program
     })
   })
 
-async function prepublish() {
+async function prepublish({ bumper }: {
+  bumper?: boolean | BumperType
+} = {}) {
   const {
     JIEK_PUBLISH_OUTDIR: outdirEnv,
     JIEK_PUBLISH_BUMPER: bumperEnv
   } = process.env
-  const outdir = outdirEnv ? JSON.parse(outdirEnv) : 'dist'
-  const bumper = bumperEnv ? JSON.parse(bumperEnv) : false
+  const outdir = outdirEnv ?? 'dist'
+  bumper = bumper ?? (
+    bumperEnv ? JSON.parse(bumperEnv) as string | boolean : false
+  )
 
   const generateNewManifest = (dir: string, manifest: NonNullable<ProjectsGraph['value']>[string]) => {
     const { name, type, exports: entrypoints = {} } = manifest
@@ -188,6 +194,7 @@ async function prepublish() {
         const index = exports?.['.']
         const indexPublishConfig: Record<string, string> = {}
         if (index) {
+          // eslint-disable-next-line ts/switch-exhaustiveness-check
           switch (typeof index) {
             case 'string':
               indexPublishConfig[
@@ -196,8 +203,8 @@ async function prepublish() {
               break
             case 'object': {
               const indexExports = index as Record<string, string>
-              indexPublishConfig.main = indexExports['require'] ?? indexExports['default']
-              indexPublishConfig.module = indexExports['import'] ?? indexExports['module'] ?? indexExports['default']
+              indexPublishConfig.main = indexExports.require ?? indexExports.default
+              indexPublishConfig.module = indexExports.import ?? indexExports.module ?? indexExports.default
               break
             }
           }
@@ -216,8 +223,19 @@ async function prepublish() {
         }
       }
     }
-    if (oldJSON['peerDependencies']) {
-      const peerDependenciesMeta = Object.keys(oldJSON['peerDependencies']).reduce(
+    if (oldJSON.devDependencies) {
+      newJSONString = applyEdits(
+        newJSONString,
+        modify(
+          newJSONString,
+          ['devDependencies'],
+          undefined,
+          { formattingOptions }
+        )
+      )
+    }
+    if (oldJSON.peerDependencies) {
+      const peerDependenciesMeta = Object.keys(oldJSON.peerDependencies).reduce(
         (acc, key) => {
           acc[key] = { optional: true }
           return acc
@@ -234,7 +252,7 @@ async function prepublish() {
         )
       )
     }
-    if (oldJSON['files']) {
+    if (oldJSON.files) {
       newJSONString = applyEdits(
         newJSONString,
         modify(
@@ -253,9 +271,9 @@ async function prepublish() {
     const resolveByDir = (...paths: string[]) => path.resolve(dir, ...paths)
 
     const oldJSONString = fs.readFileSync(resolveByDir('package.json'), 'utf-8')
-    const oldJSON = JSON.parse(oldJSONString) ?? '0.0.0'
+    const oldJSON = JSON.parse(oldJSONString) as Record<string, unknown>
     if (typeof oldJSON.version !== 'string') {
-      throw new Error(`${dir}/package.json must have a version field with a string value`)
+      throw new TypeError(`${dir}/package.json must have a version field with a string value`)
     }
 
     // TODO detectIndent by editorconfig
@@ -265,7 +283,9 @@ async function prepublish() {
       insertSpaces: true
     }
 
-    const newVersion = bumper ? bump(oldJSON.version, bumper) : oldJSON.version
+    const newVersion = bumper
+      ? bump(oldJSON.version, bumper === true ? 'patch' : bumper)
+      : oldJSON.version
     const modifyVersionPackageJSON = applyEdits(
       oldJSONString,
       modify(oldJSONString, ['version'], newVersion, { formattingOptions })
@@ -346,17 +366,24 @@ async function prepublish() {
     }
 
     if (oldJSON.files) {
-      if (!Array.isArray(oldJSON.files)) {
-        throw new Error(`${dir}/package.json files field must be an array`)
-      }
-      if (Array.isArray(oldJSON.files) && oldJSON.files.every((file: unknown) => typeof file !== 'string')) {
-        throw new Error(`${dir}/package.json files field must be an array of string`)
+      if (Array.isArray(oldJSON.files)) {
+        if (oldJSON.files.every((file: unknown) => typeof file !== 'string')) {
+          throw new TypeError(`${dir}/package.json files field must be an array of string`)
+        }
+      } else {
+        throw new TypeError(`${dir}/package.json files field must be an array`)
       }
     }
     const resolvedOutdirAbs = resolveByDir(resolvedOutdir)
     const files = (
       (oldJSON.files as undefined | string[]) ?? fs.readdirSync(resolveByDir('.'))
-    ).filter(file => file === 'node_modules' || resolveByDir(file) !== resolvedOutdirAbs)
+    ).filter(file =>
+      ![
+        'node_modules',
+        'package.json',
+        'pnpm-lock.yaml'
+      ].includes(file) && resolveByDir(file) !== resolvedOutdirAbs
+    )
 
     for (const file of files) {
       const path = resolveByDir(file)
@@ -387,6 +414,7 @@ async function postpublish() {
     if (fs.existsSync(jiekTempPackageJSON)) {
       fs.copyFileSync(jiekTempPackageJSON, packageJSON)
       fs.rmSync(jiekTempPackageJSON)
+      // eslint-disable-next-line no-console
       console.log(`${dir}/package.json has been restored`)
     } else {
       throw new Error(
@@ -396,11 +424,13 @@ async function postpublish() {
   })
 }
 
+console.log(process.env)
 program
   .action(async () => {
     const {
       npm_lifecycle_event: NPM_LIFECYCLE_EVENT
     } = process.env
+    // eslint-disable-next-line ts/switch-exhaustiveness-check
     switch (NPM_LIFECYCLE_EVENT) {
       case 'prepublish':
         await prepublish()
@@ -425,6 +455,8 @@ Prepare package.json for publish, you can add \`jk\` to the \`prepublish\` scrip
 program
   .command('prepublish')
   .description(prepublishDescription)
+  .option('-b, --bumper <bumper>', 'bump version')
+  .option('-no-b, --no-bumper', 'no bump version')
   .action(prepublish)
 
 const postpublishDescription = `

package/src/parseArgv.ts

@@ -0,0 +1,26 @@
+import { program } from 'commander'
+import process from 'node:process'
+
+const { argv } = process
+const env: Record<string, string> = {}
+let isPassThrough = false
+const newArgv = argv.filter((arg) => {
+  if (isPassThrough) {
+    return true
+  }
+  if (arg === '--') {
+    isPassThrough = true
+    return false
+  }
+  const m = /^--env\.(\w+)=(.*)$/.exec(arg)
+  if (m) {
+    env[m[1]] = m[2]
+    return false
+  }
+  return true
+})
+for (const [key, value] of Object.entries(env)) {
+  process.env[key] = value
+}
+
+export default () => program.parse(newArgv)

package/src/rollup/base.ts

@@ -100,38 +100,3 @@ export interface TemplateOptions {
       dts: InputPluginOption
     }
 }
-
-export type RollupProgressEvent =
-  | {
-    type: 'init'
-    data: {
-      leafMap: Map<string, string[][]>
-      targetsLength: number
-    }
-  }
-  | {
-    type: 'watchChange'
-    data: {
-      id: string
-      name: string
-      path: string
-      input: string
-    }
-  }
-  | {
-    type: 'debug'
-    data: unknown
-  }
-  | {
-    type: 'progress'
-    data: {
-      // name, path, exportConditions, input
-      name: string
-      path: string
-      exportConditions: string[]
-      input: string
-      tags?: string[]
-      event?: string
-      message?: string
-    }
-  }

package/src/rollup/bundle-analyzer.ts

@@ -0,0 +1,62 @@
+import process from 'node:process'
+
+import type { InputPluginOption, OutputPlugin, Plugin } from 'rollup'
+import type { AnalyzerPluginInternalAPI } from 'vite-bundle-analyzer'
+
+export type Module = ReturnType<AnalyzerPluginInternalAPI['processModule']>[number]
+
+const {
+  JIEK_ANALYZER
+} = process.env
+
+const ANALYZER = (JIEK_ANALYZER != null) && JSON.parse(JIEK_ANALYZER) as {
+  dir?: string
+  mode?: string
+  size?: string
+  port?: number
+  open?: boolean
+}
+
+export function bundleAnalyzer(modulesResolved: (modules: Module[]) => void) {
+  // eslint-disable-next-line ts/strict-boolean-expressions
+  if (!ANALYZER) {
+    return []
+  }
+
+  const defaultSizes = ({
+    parsed: 'parsed',
+    stat: 'stat',
+    gzip: 'gzip'
+  } as const)[ANALYZER.size ?? 'stat'] ?? 'parsed'
+
+  let module: typeof import('vite-bundle-analyzer') | undefined
+  let ana: Plugin | undefined
+  async function initAna() {
+    const { adapter, analyzer } = module ?? await import('vite-bundle-analyzer')
+    ana = ana ?? adapter(analyzer({
+      defaultSizes,
+      analyzerMode: modulesResolved
+    }))
+  }
+
+  return [
+    (async () => {
+      await initAna()
+      return {
+        name: 'jiek:bundle-analyzer',
+        async closeBundle(...args) {
+          if (typeof ana!.closeBundle !== 'function') return
+
+          return ana!.closeBundle?.call(this, ...args)
+        }
+      } satisfies InputPluginOption
+    })(),
+    (async () => {
+      await initAna()
+      return {
+        ...ana,
+        name: 'jiek:bundle-analyzer-output'
+      } satisfies OutputPlugin
+    })()
+  ] as const
+}

package/src/rollup/index.ts

@@ -1,5 +1,7 @@
+/* eslint-disable ts/strict-boolean-expressions */
 import fs from 'node:fs'
 import { dirname, extname, relative, resolve } from 'node:path'
+import process from 'node:process'
 
 import type { RecursiveRecord } from '@jiek/pkger/entrypoints'
 import { getAllLeafs } from '@jiek/pkger/entrypoints'
@@ -8,17 +10,20 @@ import { getWorkspaceDir } from '@jiek/utils/getWorkspaceDir'
 import commonjs from '@rollup/plugin-commonjs'
 import json from '@rollup/plugin-json'
 import { nodeResolve } from '@rollup/plugin-node-resolve'
-import { sendMessage } from 'execa'
 import { isMatch } from 'micromatch'
-import type { InputPluginOption, OutputOptions, OutputPlugin, Plugin, RollupOptions } from 'rollup'
+import type { InputPluginOption, OutputOptions, OutputPlugin, OutputPluginOption, Plugin, RollupOptions } from 'rollup'
 import ts from 'typescript'
 
+import type { RollupBuildEntryCtx, RollupBuildEventMap } from '#~/bridge.ts'
+import { publish } from '#~/bridge.ts'
+import { bundleAnalyzer } from '#~/rollup/bundle-analyzer.ts'
+import { getExports, getOutDirs } from '#~/utils/getExports.ts'
+import { loadConfig } from '#~/utils/loadConfig.ts'
 import { recusiveListFiles } from '#~/utils/recusiveListFiles.ts'
+import { getCompilerOptionsByFilePath } from '#~/utils/ts.ts'
 
-import { getExports, getOutDirs } from '../utils/getExports'
-import { loadConfig } from '../utils/loadConfig'
-import { getCompilerOptionsByFilePath } from '../utils/ts'
-import type { ConfigGenerateContext, RollupProgressEvent, TemplateOptions } from './base'
+import type { ConfigGenerateContext, TemplateOptions } from './base'
+import createRequire, { isFormatEsm } from './plugins/create-require'
 import progress from './plugins/progress'
 import skip from './plugins/skip'
 import externalResolver from './utils/externalResolver'
@@ -119,6 +124,7 @@ const resolveBuildPlugins = (context: ConfigGenerateContext, plugins: TemplateOp
   }
   let js: InputPluginOption = []
   let dts: InputPluginOption = []
+  // eslint-disable-next-line ts/switch-exhaustiveness-check
   switch (typeof plugins) {
     case 'function':
       js = plugins('js', context)
@@ -157,7 +163,9 @@ const resolveWorkspacePath = (p: string) => resolve(WORKSPACE_ROOT, p)
 
 const pascalCase = (str: string) =>
   str
+    // eslint-disable-next-line ts/no-unsafe-member-access,ts/no-unsafe-return,ts/no-unsafe-call
     .replace(/[@|/-](\w)/g, (_, $1) => $1.toUpperCase())
+    // eslint-disable-next-line ts/no-unsafe-member-access,ts/no-unsafe-return,ts/no-unsafe-call
     .replace(/(?:^|-)(\w)/g, (_, $1) => $1.toUpperCase())
 
 const reveal = (obj: string | Record<string, unknown>, keys: string[]) =>
@@ -188,15 +196,24 @@ const withMinify = (
   output: OutputOptions & {
     plugins?: OutputPlugin[]
   },
-  minify = build?.output?.minify ?? MINIFY_DEFAULT_VALUE
+  onlyOncePlugins: OutputPluginOption[] = []
 ): OutputOptions[] => {
-  if (minify === false) return [output]
+  const minify = build?.output?.minify ?? MINIFY_DEFAULT_VALUE
+  output.plugins = output.plugins ?? []
+  if (minify === false) {
+    output.plugins.push(...onlyOncePlugins)
+    return [output]
+  }
 
   const minifyPlugin = resolvedMinifyOptions.type === 'esbuild'
+    // eslint-disable-next-line ts/no-unsafe-argument
     ? import('rollup-plugin-esbuild').then(({ minify }) => minify(noTypeResolvedMinifyOptions as any))
     : resolvedMinifyOptions.type === 'swc'
+    // eslint-disable-next-line ts/no-unsafe-argument
     ? import('rollup-plugin-swc3').then(({ minify }) => minify(noTypeResolvedMinifyOptions as any))
+    // eslint-disable-next-line ts/no-unsafe-argument
     : import('@rollup/plugin-terser').then(({ default: minify }) => minify(noTypeResolvedMinifyOptions as any))
+  const notOnlyOncePlugins = output.plugins
   return minify === 'only-minify'
     ? [{
       ...output,
@@ -209,7 +226,7 @@ const withMinify = (
             throw new Error('entryFileNames must be a function')
           })(),
       plugins: [
-        ...(output.plugins ?? []),
+        ...notOnlyOncePlugins,
         minifyPlugin
       ]
     }]
@@ -225,7 +242,7 @@ const withMinify = (
             })(),
         file: output.file?.replace(/(\.[cm]?js)$/, '.min$1'),
         plugins: [
-          ...(output.plugins ?? []),
+          ...notOnlyOncePlugins,
           minifyPlugin
         ]
       }
@@ -246,6 +263,14 @@ const generateConfigs = (context: ConfigGenerateContext, options: TemplateOption
   const isModule = conditionals.includes('import')
   const isCommonJS = conditionals.includes('require')
   const isBrowser = conditionals.includes('browser')
+  const format = isModule ? 'esm' : (
+    isCommonJS ? 'cjs' : (
+      isBrowser ? 'umd' : (
+        pkgIsModule ? 'esm' : 'cjs'
+      )
+    )
+  )
+
   const dtsTSConfigPaths = [
     resolveWorkspacePath('tsconfig.json'),
     resolveWorkspacePath('tsconfig.dts.json')
@@ -285,10 +310,22 @@ const generateConfigs = (context: ConfigGenerateContext, options: TemplateOption
     delete compilerOptions.composite
   }
   const exportConditions = [...conditionals, ...(compilerOptions.customConditions ?? [])]
-  const throughEventProps: RollupProgressEvent & { type: 'progress' } = {
-    type: 'progress',
-    data: { name, path, exportConditions, input }
-  }
+  const publishInEntry = <K extends keyof RollupBuildEventMap>(
+    type: K,
+    data: Omit<RollupBuildEventMap[K], keyof RollupBuildEntryCtx>
+  ) =>
+    // eslint-disable-next-line ts/no-unsafe-argument
+    void publish(type, {
+      ...{
+        type: format,
+        name,
+        path,
+        exportConditions,
+        input
+      } as RollupBuildEntryCtx,
+      ...data
+    } as any)
+
   const { js: jsPlugins, dts: dtsPlugins } = resolveBuildPlugins(context, build.plugins)
   if (input.includes('**')) {
     throw new Error(
@@ -347,6 +384,10 @@ const generateConfigs = (context: ConfigGenerateContext, options: TemplateOption
           ...noTypeResolvedBuilderOptions
         })
       )
+    const [ana, anaOutputPlugin] = bundleAnalyzer(modules => void publishInEntry('modulesAnalyze', { modules }))
+    const onlyOncePlugins = [
+      anaOutputPlugin
+    ]
     rollupOptions.push({
       input: inputObj,
       external,
@@ -363,17 +404,14 @@ const generateConfigs = (context: ConfigGenerateContext, options: TemplateOption
               : output.replace(`${jsOutdir}/`, '')
           ),
           sourcemap,
-          format: isModule ? 'esm' : (
-            isCommonJS ? 'cjs' : (
-              isBrowser ? 'umd' : (
-                pkgIsModule ? 'esm' : 'cjs'
-              )
-            )
-          ),
+          format,
           strict: typeof options?.output?.strict === 'object'
             ? options.output.strict.js
-            : options?.output?.strict
-        })
+            : options?.output?.strict,
+          plugins: [
+            isFormatEsm(format === 'esm')
+          ]
+        }, onlyOncePlugins)
       ],
       plugins: [
         ...commonPlugins,
@@ -385,16 +423,12 @@ const generateConfigs = (context: ConfigGenerateContext, options: TemplateOption
             })
           )
           .catch(() => void 0),
-        builder,
         commonjs(),
+        createRequire(),
+        builder,
+        ana,
         progress({
-          onEvent: (event, message) =>
-            sendMessage(
-              {
-                ...throughEventProps,
-                data: { ...throughEventProps.data, event, message, tags: ['js'] }
-              } satisfies RollupProgressEvent
-            )
+          onEvent: (event, message) => void publishInEntry('progress', { event, message, tags: ['js'] })
         }),
         jsPlugins
       ]
@@ -444,13 +478,7 @@ const generateConfigs = (context: ConfigGenerateContext, options: TemplateOption
           tsconfig: dtsTSConfigPath
         }),
         progress({
-          onEvent: (event, message) =>
-            sendMessage(
-              {
-                ...throughEventProps,
-                data: { ...throughEventProps.data, event, message, tags: ['dts'] }
-              } satisfies RollupProgressEvent
-            )
+          onEvent: (event, message) => void publishInEntry('progress', { event, message, tags: ['dts'] })
         }),
         dtsPlugins
       ]
@@ -461,15 +489,9 @@ const generateConfigs = (context: ConfigGenerateContext, options: TemplateOption
     rollupOptions[0].plugins = [
       {
         name: 'jiek-plugin-watcher',
-        watchChange: (id) =>
-          sendMessage(
-            {
-              type: 'watchChange',
-              data: { id, name: JIEK_NAME!, path, input }
-            } satisfies RollupProgressEvent
-          )
+        watchChange: id => void publishInEntry('watchChange', { id })
       },
-      ...(rollupOptions[0].plugins as any)
+      ...(rollupOptions[0].plugins as Plugin[])
     ]
   }
   return rollupOptions
@@ -519,6 +541,7 @@ export function template(packageJSON: PackageJSON): RollupOptions[] {
         pkgIsModule
       }
 
+      // eslint-disable-next-line ts/switch-exhaustiveness-check
       switch (typeof keyExports) {
         case 'string': {
           configs.push(...generateConfigs({
@@ -545,15 +568,7 @@ export function template(packageJSON: PackageJSON): RollupOptions[] {
       }
     })
   )
-  sendMessage(
-    {
-      type: 'init',
-      data: {
-        leafMap,
-        targetsLength: configs.length
-      }
-    } satisfies RollupProgressEvent
-  )
+  void publish('init', { leafMap, targetsLength: configs.length })
   return configs.map(c => ({
     ...COMMON_OPTIONS,
     ...c,

package/src/rollup/plugins/create-require.ts

@@ -0,0 +1,74 @@
+// https://github.com/privatenumber/pkgroll/blob/73559f8864203a50a5aa12c0a6503ceed3690aff/src/utils/rollup-plugins/create-require.ts#L1
+// Thanks to @privatenumber for the snippet
+
+import inject from '@rollup/plugin-inject'
+import replace from '@rollup/plugin-replace'
+import type { Plugin } from 'rollup'
+
+const virtualModuleName = 'jiek:create-require'
+
+/**
+ * Since rollup is bundled by rollup, it needs to add a run-time
+ * suffix so that this doesn't get replaced.
+ */
+const isEsmVariableName = `IS_ESM${Math.random().toString(36).slice(2)}`
+
+const INSERT_STR = `
+import { createRequire } from 'node:module'
+
+export default (
+  ${isEsmVariableName}
+    ? /* @__PURE__ */ createRequire(import.meta.url)
+    : require
+)
+`.trim()
+
+/**
+ * Plugin to seamlessly allow usage of `require`
+ * across CJS and ESM modules.
+ *
+ * This is usually nor a problem for CJS outputs,
+ * but for ESM outputs, it must be used via
+ * createRequire.
+ *
+ * This plugin automatically injects it for ESM.
+ */
+export default (): Plugin => ({
+  ...inject({
+    require: virtualModuleName
+  }),
+
+  name: 'create-require',
+
+  resolveId: source => (
+    (source === virtualModuleName)
+      ? source
+      : null
+  ),
+
+  load: (id) => {
+    if (id !== virtualModuleName) {
+      return null
+    }
+
+    return INSERT_STR
+  }
+})
+
+export const isFormatEsm = (
+  isEsm: boolean
+): Plugin => {
+  const handler = replace({
+    [isEsmVariableName]: isEsm
+  }).renderChunk!
+
+  return ({
+    name: 'create-require-insert-format',
+
+    // Pick out renderChunk because it's used as an output plugin
+    renderChunk: {
+      order: 'pre',
+      handler: typeof handler === 'function' ? handler : handler.handler
+    }
+  })
+}

package/src/server.ts

@@ -0,0 +1,22 @@
+import Koa from 'koa'
+
+export const createServer = (port: number, host: string) => {
+  const app = new Koa()
+  app.listen(port, host)
+  const streams = new Map<string, string>()
+  app.use(async (ctx) => {
+    const stream = streams.get(ctx.path)
+    if (stream != null) {
+      ctx.body = stream
+    }
+  })
+  // noinspection HttpUrlsUsage
+  return {
+    port,
+    host,
+    rootUrl: `http://${host}:${port}`,
+    renderTo: async (path: string, stream: string) => {
+      streams.set(path, stream)
+    }
+  }
+}

package/rollup/package.json

@@ -1 +0,0 @@
-{"type":"module","main":"../dist/rollup/index.cjs","module":"../dist/rollup/index.js"}