From 7ad4e13e8e9d00da5104d32c5fc07d9bc128b1f3 Mon Sep 17 00:00:00 2001
From: umbraci <umbraci@qq.com>
Date: Wed, 22 Oct 2025 14:26:11 +0800
Subject: [PATCH 1/2] feat: add multiple build output formats (ES/CJS/UMD)

- Configure Rollup to generate three output formats:
  - ES Module (es/) for modern bundlers with tree-shaking
  - CommonJS (lib/) for require/legacy toolchains
  - UMD (dist/) for direct script usage with global variables
- Update package.json exports to support all formats
- Add build outputs to .gitignore
- Support both unpkg and jsdelivr CDN usage
---
 .config/rollup.config.js | 149 +++++++++++++++++++++++++--------------
 .gitignore               |   8 +++
 package.json             |  20 +++---
 3 files changed, 117 insertions(+), 60 deletions(-)

diff --git a/.config/rollup.config.js b/.config/rollup.config.js
index fa1d3471..308aa2ef 100644
--- a/.config/rollup.config.js
+++ b/.config/rollup.config.js
@@ -1,59 +1,111 @@
 import cleanup from 'rollup-plugin-cleanup';
 import terser from '@rollup/plugin-terser';
 
-export default [
-    {
-        input: 'src/jsmind.js',
-        output: {
+const banner =
+    '/**\n* @license BSD-3-Clause\n* @copyright 2014-2025 hizzgdev@163.com\n*\n* Project Home:\n*   https://github.com/hizzgdev/jsmind/\n*/';
+
+const cleanupPlugin = cleanup({
+    comments: 'none',
+});
+
+const terserPlugin = terser({
+    output: {
+        comments: 'all',
+    },
+});
+
+// Main library configuration
+const mainConfig = {
+    input: 'src/jsmind.js',
+    output: [
+        // ES Module - for modern bundlers with tree-shaking support
+        {
+            file: 'es/jsmind.js',
+            format: 'es',
+            banner,
+            sourcemap: true,
+        },
+        // CommonJS - for require/legacy toolchains
+        {
+            file: 'lib/jsmind.js',
+            format: 'cjs',
+            banner,
+            sourcemap: true,
+            exports: 'auto',
+        },
+        // UMD - for direct <script> usage, exposes global jsMind
+        {
             name: 'jsMind',
-            file: 'es6/jsmind.js',
+            file: 'dist/jsmind.js',
             format: 'umd',
-            banner: '/**\n* @license BSD-3-Clause\n* @copyright 2014-2025 hizzgdev@163.com\n*\n* Project Home:\n*   https://github.com/hizzgdev/jsmind/\n*/',
+            banner,
             sourcemap: true,
         },
-        plugins: [
-            cleanup({
-                comments: 'none',
-            }),
-            terser({
-                output: {
-                    comments: 'all',
-                },
-            }),
-        ],
-    },
-    {
-        input: 'src/plugins/jsmind.draggable-node.js',
-        output: {
+    ],
+    plugins: [cleanupPlugin, terserPlugin],
+};
+
+// Draggable-node plugin configuration
+const draggableNodeConfig = {
+    input: 'src/plugins/jsmind.draggable-node.js',
+    output: [
+        // ES Module
+        {
+            file: 'es/jsmind.draggable-node.js',
+            format: 'es',
+            banner,
+            sourcemap: true,
+        },
+        // CommonJS
+        {
+            file: 'lib/jsmind.draggable-node.js',
+            format: 'cjs',
+            banner,
+            sourcemap: true,
+            exports: 'named',
+        },
+        // UMD
+        {
             name: 'jsMindDraggableNode',
-            file: 'es6/jsmind.draggable-node.js',
+            file: 'dist/jsmind.draggable-node.js',
             format: 'umd',
-            banner: '/**\n* @license BSD-3-Clause\n* @copyright 2014-2025 hizzgdev@163.com\n*\n* Project Home:\n*   https://github.com/hizzgdev/jsmind/\n*/',
+            banner,
             sourcemap: true,
             globals: {
                 jsmind: 'jsMind',
             },
             exports: 'named',
         },
-        external: ['jsmind'],
-        plugins: [
-            cleanup({
-                comments: 'none',
-            }),
-            terser({
-                output: {
-                    comments: 'all',
-                },
-            }),
-        ],
-    },
-    {
-        input: 'src/plugins/jsmind.screenshot.js',
-        output: {
+    ],
+    external: ['jsmind'],
+    plugins: [cleanupPlugin, terserPlugin],
+};
+
+// Screenshot plugin configuration
+const screenshotConfig = {
+    input: 'src/plugins/jsmind.screenshot.js',
+    output: [
+        // ES Module
+        {
+            file: 'es/jsmind.screenshot.js',
+            format: 'es',
+            banner,
+            sourcemap: true,
+        },
+        // CommonJS
+        {
+            file: 'lib/jsmind.screenshot.js',
+            format: 'cjs',
+            banner,
+            sourcemap: true,
+            exports: 'named',
+        },
+        // UMD
+        {
             name: 'jsMindScreenshot',
-            file: 'es6/jsmind.screenshot.js',
+            file: 'dist/jsmind.screenshot.js',
             format: 'umd',
-            banner: '/**\n* @license BSD-3-Clause\n* @copyright 2014-2025 hizzgdev@163.com\n*\n* Project Home:\n*   https://github.com/hizzgdev/jsmind/\n*/',
+            banner,
             sourcemap: true,
             globals: {
                 'jsmind': 'jsMind',
@@ -61,16 +113,9 @@ export default [
             },
             exports: 'named',
         },
-        external: ['jsmind', 'dom-to-image'],
-        plugins: [
-            cleanup({
-                comments: 'none',
-            }),
-            terser({
-                output: {
-                    comments: 'all',
-                },
-            }),
-        ],
-    },
-];
+    ],
+    external: ['jsmind', 'dom-to-image'],
+    plugins: [cleanupPlugin, terserPlugin],
+};
+
+export default [mainConfig, draggableNodeConfig, screenshotConfig];
diff --git a/.gitignore b/.gitignore
index f8f575a9..fe2a0819 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1,5 +1,13 @@
 .DS_Store
 node_modules/
+
+# Build outputs
+es/*.js
+es/*.js.map
+lib/*.js
+lib/*.js.map
+dist/*.js
+dist/*.js.map
 es6/*.js
 es6/*.js.map
 types/generated/
diff --git a/package.json b/package.json
index 2b218f15..8f2ea78d 100644
--- a/package.json
+++ b/package.json
@@ -2,22 +2,24 @@
     "name": "jsmind",
     "version": "0.9.0",
     "description": "jsMind is a pure javascript library for mindmap, it base on html5 canvas. jsMind was released under BSD license, you can embed it in any project, if only you observe the license.",
-    "main": "es6/jsmind.js",
+    "main": "lib/jsmind.js",
+    "module": "es/jsmind.js",
+    "unpkg": "dist/jsmind.js",
     "types": "types/generated/index.d.ts",
     "exports": {
         ".": {
-            "import": "./es6/jsmind.js",
-            "require": "./es6/jsmind.js",
+            "import": "./es/jsmind.js",
+            "require": "./lib/jsmind.js",
             "types": "./types/generated/index.d.ts"
         },
         "./draggable-node": {
-            "import": "./es6/jsmind.draggable-node.js",
-            "require": "./es6/jsmind.draggable-node.js",
+            "import": "./es/jsmind.draggable-node.js",
+            "require": "./lib/jsmind.draggable-node.js",
             "types": "./types/generated/plugins/jsmind.draggable-node.d.ts"
         },
         "./screenshot": {
-            "import": "./es6/jsmind.screenshot.js",
-            "require": "./es6/jsmind.screenshot.js",
+            "import": "./es/jsmind.screenshot.js",
+            "require": "./lib/jsmind.screenshot.js",
             "types": "./types/generated/plugins/jsmind.screenshot.d.ts"
         },
         "./style/jsmind.css": "./style/jsmind.css"
@@ -27,7 +29,9 @@
         "example": "example"
     },
     "files": [
-        "es6",
+        "es",
+        "lib",
+        "dist",
         "style",
         "types",
         "LICENSE",

From 714704f2f042cd5dd9aecea4f73d32c59a278939 Mon Sep 17 00:00:00 2001
From: umbraci <umbraci@qq.com>
Date: Wed, 22 Oct 2025 16:00:57 +0800
Subject: [PATCH 2/2] chore: del some unused code

---
 .../fixtures/optional-parameters-test.ts      | 157 ------------------
 1 file changed, 157 deletions(-)
 delete mode 100644 tests/types/fixtures/optional-parameters-test.ts

diff --git a/tests/types/fixtures/optional-parameters-test.ts b/tests/types/fixtures/optional-parameters-test.ts
deleted file mode 100644
index 47b6cb67..00000000
--- a/tests/types/fixtures/optional-parameters-test.ts
+++ /dev/null
@@ -1,157 +0,0 @@
-/**
- * TypeScript test for optional parameters based on official documentation
- * This file tests the corrections made to jsDoc annotations for optional parameters
- */
-
-import jsMind, { JsMindOptions, NodeTreeFormat } from './types/generated/index';
-
-// ============================================================================
-// Test 1: Optional parameters in styling methods
-// ============================================================================
-
-function testOptionalStylingParameters() {
-    const options: JsMindOptions = {
-        container: 'test_container'
-    };
-    
-    const jm = new jsMind(options);
-    const testData: NodeTreeFormat = {
-        meta: { name: 'test', author: 'test', version: '1.0' },
-        format: 'node_tree',
-        data: { id: 'root', topic: 'Test Root' }
-    };
-    
-    jm.show(testData);
-    const root = jm.get_root();
-    
-    if (root) {
-        const node = jm.add_node(root, 'test_node', 'Test Node');
-        
-        if (node) {
-            // Test set_node_color with optional parameters
-            jm.set_node_color(node.id); // All parameters optional
-            jm.set_node_color(node.id, '#ff0000'); // Only bg_color
-            jm.set_node_color(node.id, '#ff0000', '#ffffff'); // Both colors
-            jm.set_node_color(node.id, undefined, '#ffffff'); // Only fg_color
-            
-            // Test set_node_font_style with optional parameters
-            jm.set_node_font_style(node.id); // All parameters optional
-            jm.set_node_font_style(node.id, 16); // Only size
-            jm.set_node_font_style(node.id, 16, 'bold'); // Size and weight
-            jm.set_node_font_style(node.id, 16, 'bold', 'italic'); // All parameters
-            jm.set_node_font_style(node.id, undefined, 'bold'); // Only weight
-            jm.set_node_font_style(node.id, undefined, undefined, 'italic'); // Only style
-            
-            // Test set_node_background_image with optional parameters
-            jm.set_node_background_image(node.id); // All parameters optional
-            jm.set_node_background_image(node.id, 'test.png'); // Only image
-            jm.set_node_background_image(node.id, 'test.png', 100); // Image and width
-            jm.set_node_background_image(node.id, 'test.png', 100, 100); // Image, width, height
-            jm.set_node_background_image(node.id, 'test.png', 100, 100, 45); // All parameters
-            jm.set_node_background_image(node.id, undefined, 100); // Only width
-            jm.set_node_background_image(node.id, undefined, undefined, 100); // Only height
-            jm.set_node_background_image(node.id, undefined, undefined, undefined, 45); // Only rotation
-        }
-    }
-}
-
-// ============================================================================
-// Test 2: Optional parameters in expansion methods
-// ============================================================================
-
-function testOptionalExpansionParameters() {
-    const options: JsMindOptions = {
-        container: 'expansion_test'
-    };
-    
-    const jm = new jsMind(options);
-    const testData: NodeTreeFormat = {
-        meta: { name: 'expansion_test', author: 'test', version: '1.0' },
-        format: 'node_tree',
-        data: { 
-            id: 'root', 
-            topic: 'Root',
-            children: [
-                { id: 'child1', topic: 'Child 1' },
-                { id: 'child2', topic: 'Child 2' }
-            ]
-        }
-    };
-    
-    jm.show(testData);
-    
-    // Test expand_to_depth with required depth parameter
-    jm.expand_to_depth(1); // Minimum depth
-    jm.expand_to_depth(2); // With depth parameter
-    jm.expand_to_depth(3); // With different depth
-}
-
-// ============================================================================
-// Test 3: Optional parameters in show method
-// ============================================================================
-
-function testOptionalShowParameters() {
-    const options: JsMindOptions = {
-        container: 'show_test'
-    };
-    
-    const jm = new jsMind(options);
-    const testData: NodeTreeFormat = {
-        meta: { name: 'show_test', author: 'test', version: '1.0' },
-        format: 'node_tree',
-        data: { id: 'root', topic: 'Show Test Root' }
-    };
-    
-    // Test show method with optional skip_centering parameter
-    jm.show(testData); // Without skip_centering
-    jm.show(testData, false); // With skip_centering = false
-    jm.show(testData, true); // With skip_centering = true
-}
-
-// ============================================================================
-// Test 4: Optional parameters in node manipulation methods
-// ============================================================================
-
-function testOptionalNodeManipulationParameters() {
-    const options: JsMindOptions = {
-        container: 'manipulation_test'
-    };
-    
-    const jm = new jsMind(options);
-    const testData: NodeTreeFormat = {
-        meta: { name: 'manipulation_test', author: 'test', version: '1.0' },
-        format: 'node_tree',
-        data: { id: 'root', topic: 'Manipulation Test Root' }
-    };
-    
-    jm.show(testData);
-    const root = jm.get_root();
-    
-    if (root) {
-        // Test add_node with optional parameters
-        const node1 = jm.add_node(root, 'node1', 'Node 1'); // Without data and direction
-        const node2 = jm.add_node(root, 'node2', 'Node 2', { color: 'red' }); // With data, without direction
-        const node3 = jm.add_node(root, 'node3', 'Node 3', undefined, 'right'); // Without data, with direction
-        const node4 = jm.add_node(root, 'node4', 'Node 4', { color: 'blue' }, 'left'); // With both
-        
-        if (node1 && node2) {
-            // Test insert methods with optional parameters
-            jm.insert_node_before(node1, 'before1', 'Before 1'); // Without data and direction
-            jm.insert_node_after(node2, 'after1', 'After 1', { style: 'bold' }); // With data, without direction
-            
-            // Test move_node with optional parameters
-            jm.move_node(node1.id); // Only node_id
-            jm.move_node(node2.id, '_first_'); // With before_id
-            jm.move_node(node1.id, '_last_', root.id); // With before_id and parent_id
-            jm.move_node(node2.id, node1.id, root.id, 'right'); // All parameters
-        }
-    }
-}
-
-// Run all tests
-testOptionalStylingParameters();
-testOptionalExpansionParameters();
-testOptionalShowParameters();
-testOptionalNodeManipulationParameters();
-
-console.log('Optional parameters test completed successfully!');