@clickview/online 0.81.0 → 0.82.0-rc.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@clickview/online",
-  "version": "0.81.0",
+  "version": "0.82.0-rc.0",
   "description": "Online",
   "main": "dist/online-app.js",
   "types": "dist/index.d.ts",
@@ -8,8 +8,8 @@
   "scripts": {
     "start": "npm run build",
     "type-check": "tsc --noEmit",
-    "build-app": "vite build --config vite.config.app.ts",
-    "build-styles": "vite build --config vite.config.styles.ts",
+    "build-app": "node ../../../node_modules/vite-classic/bin/vite.js build --config vite.config.app.ts",
+    "build-styles": "node ../../../node_modules/vite/bin/vite.js build --config vite.config.styles.ts",
     "build": "npm run type-check && npm run build-app && npm run build-styles"
   },
   "repository": {
@@ -21,7 +21,8 @@
   "devDependencies": {
     "@types/linkifyjs": "2.1.3",
     "@types/react-transition-group": "4.2.3",
-    "@types/yup": "0.26.24"
+    "@types/yup": "0.26.24",
+    "vite-classic": "npm:vite@7.3.3"
   },
   "dependencies": {
     "intersection-observer": "0.11.0",

package/vite.config.app.ts

@@ -1,6 +1,28 @@
-import config from '../../libs/tooling/vite/vite.base.config';
+// IMPORTANT: This build deliberately uses classic Rollup-based Vite 7
+// (aliased as `vite-classic` in package.json), not Vite 8 (Rolldown).
+//
+// Why: Rolldown's UMD output leaves bare `require("<external>")` calls inside
+// its prebundled CommonJS module shims (`__commonJSMin`) instead of rewriting
+// them to use the wrapper's factory parameters. The `online` bundle is loaded
+// via a <script> tag in the legacy host, which doesn't provide a working
+// `require`, so those bare calls blow up at runtime. Rollup handles UMD
+// externals correctly — verified empirically. See the migration history if
+// Rolldown ever gains proper UMD external rewriting; we can rejoin then.
+//
+// This config is intentionally self-contained (not extending the shared
+// `vite.base.config.ts`) because the shared base imports from `vite` (=v8/
+// Rolldown). Plugins are version-agnostic Rollup plugins and work fine on v7.
+import inject from '@rollup/plugin-inject';
+import react from '@vitejs/plugin-react';
+import handlebarsPlugin from '@yoichiro/vite-plugin-handlebars';
+import * as fs from 'fs';
+import * as path from 'path';
+import { defineConfig } from 'vite-classic';
+import tsconfigPaths from 'vite-tsconfig-paths';
 
-const externalGlobals = {
+import i18n from '../../libs/tooling/vite/vite-i18n-plugin';
+
+const externalGlobals: Record<string, string> = {
   // Cropper
   'cropperjs': 'Cropper',
   'cropper': 'Cropper',
@@ -45,24 +67,81 @@ const externalGlobals = {
   'gapi': 'gapi'
 };
 
-export default config({
+const sourcePaths = [
+  path.resolve(__dirname, '../../libs/common/src'),
+  path.resolve(__dirname, '../../libs/shared/src'),
+  path.resolve(__dirname, '../../projects/player/src'),
+  path.resolve(__dirname, './src')
+].map(p => p.replace(/\\/g, '/'));
+
+export default defineConfig({
   root: __dirname,
-  // Use a function for externals to ensure proper matching
-  externals: id => Object.keys(externalGlobals).includes(id),
-  globals: externalGlobals,
+  plugins: [
+    inject({ $: 'jquery' }) as any,
+    handlebarsPlugin() as any,
+    i18n(sourcePaths.map(p => `${p}/**/*.lang.json`)) as any,
+    react(),
+    tsconfigPaths(),
+    // The legacy host reads bundles.json to discover the entry filenames.
+    // In lib mode Vite's native manifest is disabled, so we emit it ourselves.
+    {
+      name: 'write-bundles-json',
+      apply: 'build' as const,
+      closeBundle() {
+        fs.writeFileSync(
+          path.resolve(__dirname, 'dist/online/bundles.json'),
+          JSON.stringify({ appBundle: 'online-app.js', cssBundle: 'online-app.css' }, null, '\t')
+        );
+      }
+    }
+  ],
 
-  lib: {
-    entry: './src/index.ts',
-    name: '@clickview/online',
-    formats: ['umd']
+  build: {
+    outDir: 'dist/online',
+    cssCodeSplit: false,
+    lib: {
+      entry: './src/index.ts',
+      name: '@clickview/online',
+      formats: ['umd']
+    },
+    rollupOptions: {
+      external: (id: string) => Object.keys(externalGlobals).includes(id),
+      output: {
+        entryFileNames: 'online-app.js',
+        chunkFileNames: 'scripts/[hash].chunk.js',
+        assetFileNames: assetInfo => {
+          if (assetInfo.name && assetInfo.name.endsWith('.css')) return 'online-app.css';
+          return 'assets/[name]-[hash].[ext]';
+        },
+        globals: externalGlobals
+      }
+    },
+    sourcemap: true,
+    chunkSizeWarningLimit: 1000
   },
 
-  outDir: 'dist/online',
-  outputScriptName: 'online-app.js',
-  outputCssName: 'online-app.css',
+  css: {
+    preprocessorOptions: {
+      scss: {
+        additionalData: `@import "${path.resolve(__dirname, 'src/styles/theme/_variables.scss').replace(/\\/g, '/')}";`,
+        silenceDeprecations: [ 'import', 'global-builtin', 'color-functions', 'if-function' ]
+      }
+    }
+  },
 
   define: {
+    '__ENVIRONMENT__': JSON.stringify(process.env.NODE_ENV),
     'process.env.NODE_ENV': JSON.stringify(process.env.NODE_ENV ?? 'development'),
-    'process.env': {}
+    'global': 'globalThis'
+  },
+
+  resolve: {
+    extensions: [ '.ts', '.tsx', '.js' ],
+    alias: {
+      '~styles/globals': path.resolve(__dirname, '../../libs/styles/src/globals.scss'),
+      '~styles/core': path.resolve(__dirname, '../../libs/styles/src/core.scss'),
+      '~styles/utils': path.resolve(__dirname, '../../libs/styles/src/utils'),
+      '~styles/player': path.resolve(__dirname, '../../projects/player')
+    }
   }
 });

package/.eslintrc.cjs

@@ -1,5 +0,0 @@
-module.exports = {
-  rules: {
-    'rulesdir/require-view-title': 'off'
-  }
-};

package/dist/styles/scripts/core-DfLwBfdK.js

@@ -1,2 +0,0 @@
-
-//# sourceMappingURL=core-DfLwBfdK.js.map

package/dist/styles/scripts/core-DfLwBfdK.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"core-DfLwBfdK.js","sources":[],"sourcesContent":[],"names":[],"mappings":""}