@theotherwillembotha/node-red-plugincore 0.0.55 → 0.2.0

package/src/core/NodeConstructor.ts

@@ -1,23 +1,25 @@
-import { PluginDefinition } from "@node-red/registry";
-import { Node, NodeAPI, NodeDef, NodeAPISettingsWithData} from "node-red";
+import { Node, NodeAPI, NodeAPISettingsWithData} from "node-red";
 import { INamedType, INodeClass, ITemplateClass } from "./NodeGenerator";
 
 import "reflect-metadata";
-import jsdom from "jsdom";
 import fs from 'fs';
-import beautify from 'js-beautify';
-import markdownit from 'markdown-it'
-import Handlebars, { template } from "handlebars";
-import { NodeTypeService } from "./tagging/service/NodeTypeService";
-import { NodeDescriptionConfig } from "./tagging/NodeDescriptionDecorator";
 
+import {
+    NODEMANAGER_API_VERSION,
+    POST_CONSTRUCT_KEY,
+    BaseNodeConfig,
+    runPostConstructInitializers,
+    NodeManager
+} from "./NodeManagerRuntime";
 
-
-export interface BaseNodeConfig {
-    id:string;
-    name:string;
-    type:string;
-}
+// Re-export so existing downstream code keeps working unchanged.
+export {
+    NODEMANAGER_API_VERSION,
+    POST_CONSTRUCT_KEY,
+    BaseNodeConfig,
+    runPostConstructInitializers,
+    NodeManager
+};
 
 export abstract class BaseNode<Config extends BaseNodeConfig> {
 
@@ -199,8 +201,6 @@ export abstract class Template{
 
 type VoidMethod = (...args: any[]) => void | Promise<void>;
 
-const POST_CONSTRUCT_KEY = Symbol('postConstructInitializers');
-
 // Builder class for post-construct decorators
 class PostConstructDecoratorBuilder<TMetadata = any> {
     private decoratorName: string;
@@ -342,150 +342,6 @@ export function createPostConstructDecorator<TMetadata = any>(decoratorName: str
     return new PostConstructDecoratorBuilder<TMetadata>(decoratorName);
 }
 
-// Single function to run all post-construct initializers
-let runPostConstructInitializers = function(instance: any) {
-    const initializers = instance[POST_CONSTRUCT_KEY];
-    if (initializers && initializers.length > 0) {
-        const className = instance.constructor.name;
-        //console.log(`Running ${initializers.length} post-construct initializers for ${className}`);
-        
-        initializers.forEach((init: Function, index: number) => {
-            try {
-                init(instance);
-            } catch (error) {
-                console.error(
-                    `Post-construct initialization failed:\n` +
-                    `  Class: ${className}\n` +
-                    `  Initializer: ${index + 1}/${initializers.length}\n` +
-                    `  Error: ${error instanceof Error ? error.message : String(error)}`
-                );
-                throw error;
-            }
-        });
-        
-        // Clear the initializers after running
-        delete instance[POST_CONSTRUCT_KEY];
-    }
-}
-
-// ************************************************
-
-export class NodeManager{
-    private static PLUGINID = "node-red-contrib-theotherwillembotha/services-plugin";
-    private static _RED: NodeAPI<NodeAPISettingsWithData>;
-    
-    private nodeTypeService:NodeTypeService;
-    private typeBacklog:INodeClass[] = []
-
-    public constructor(RED : NodeAPI<NodeAPISettingsWithData>){
-        NodeManager._RED = RED;
-
-        let nodeTypeServiceListener = async (pluginID:String) => {
-            if(pluginID === "@theotherwillembotha/nodetypeservice"){
-                // disable the listener- the plugin has been installed.
-                RED.events.off('plugin.instantiated', nodeTypeServiceListener)
-
-                // get a handle on the NodeTypeService
-                let plugin  = RED.plugins.get("@theotherwillembotha/nodetypeservice") as any;
-                this.nodeTypeService = (RED.plugins.get("@theotherwillembotha/nodetypeservice") as any).instance as NodeTypeService;
-                
-                // process the backlog.
-                for(let nodeType of this.typeBacklog){
-                    for(let tag of nodeType.getNodeDescriptor().tags()){
-                        this.nodeTypeService.registerNodeType(tag, nodeType);
-                    }
-                }
-            }
-        }
-
-        // check if the reverserproxyttypesewrvice is installed yet.
-        this.nodeTypeService  = (RED.plugins.get("@theotherwillembotha/nodetypeservice") as any).instance as NodeTypeService;
-        if(this.nodeTypeService) { 
-            console.log("NodeTypeService installed!")
-        }
-        else {
-            // Register a listener to trigger when the RegerseProxyTypService gets installed.
-            console.log("nodetypeservice not installed yet. waiting for it to be installed");
-            RED.events.on('plugin.instantiated', nodeTypeServiceListener);
-        }
-    }
-
-    public static get RED() {
-        return NodeManager._RED;
-    }
-    
-    public registerService(type:IServiceClass):NodeManager{
-        let servicePlugin:any = NodeManager.RED.plugins.get(NodeManager.PLUGINID) as PluginDefinition<any>;
-        if(servicePlugin){
-            servicePlugin.instance.install(NodeManager.RED, type);
-        }
-        else{
-            let serviceName = `${type.getServiceDescriptor().sourceFile()}.${type.getServiceDescriptor().name()}`;
-            console.log(`ERROR: plugin ${NodeManager.PLUGINID} does not seem to be loaded. Cannot start service: ${serviceName}`);
-        }
-        return this;
-    }
-
-    public registerNodeType(typeName: string, type: INodeClass):NodeManager{
-        if(!type){
-            console.error(`Type "${typeName}" has no associated type. Make sure that it was exported and included in the index.js file!`)
-        }
-
-        let nodeConstructor = function(this: any, config: BaseNodeConfig | NodeDef){
-            //console.log("constructing node", typeName);
-
-            try{
-                // instantiate the node.
-                NodeManager.RED.nodes.createNode(this, config as any as NodeDef);
-                // assign an instance of it to the config.
-                //(config as any).__node = this;
-
-                // instatiate the actual node.
-                let node = new (type as any)(this, config);
-                //this.__node = node;
-
-                // run the constructor initializers.
-                runPostConstructInitializers(node);
-
-
-                // after the node has been instantiated, attempt to invole the onInit method.
-                if(node.onInit){
-                    node.onInit();
-                }
-            }
-            catch(error){
-                console.log(error);
-                console.log("typeName:", typeName)
-                console.log("type:", type)
-                console.log("config:", config)
-            }
-        }
-
-        try{
-            console.log("registering node " + typeName);
-            NodeManager.RED.nodes.registerType(typeName, nodeConstructor, { settings: {}});
-
-            let nodeDescription = type.getNodeDescriptor();
-
-            // registger the node tags.
-            if(this.nodeTypeService){
-                for(let tag of nodeDescription.tags()){
-                    this.nodeTypeService.registerNodeType(tag, type);
-                }
-            }
-            else{
-                this.typeBacklog.push(type);
-            }
-
-            console.log("registered node " + typeName);
-        }
-        catch(error){
-            console.error("registering node: " + typeName + " failed", error);
-        }
-        return this;
-    }
-}
-
 export type FlowDeployment = {
     rev:string,
     flows:[FlowElement]
@@ -521,25 +377,27 @@ export abstract class BaseService {
     }
 }
 
-const { JSDOM } = jsdom;
-const md = markdownit({
-  html: true,
-  linkify: true,
-  typographer: true
-})
-
-// Custom renderer for headings
-md.renderer.rules.heading_open = function (tokens, idx, options, env, self) {
-  const token = tokens[idx];
-  const level = token.tag.substring(1); // 'h1', 'h2', etc. -> '1', '2'
-  return `<h${level} class="custom-heading">`;
-};
-
-md.renderer.rules.heading_close = function (tokens, idx, options, env, self) {
-  const token = tokens[idx];
-  const level = token.tag.substring(1);
-  return `</h${level}>`;
-};
+// Build-time deps - lazy loaded so they are never required at runtime in production bundles.
+// These are only invoked during `npm run build` (GenerateNodes), never in the deployed container.
+function getJSDOM(): any { return require('jsdom').JSDOM; }
+function getBeautify(): any { return require('js-beautify'); }
+function getHandlebars(): any { return require('handlebars'); }
+
+let _md: any = null;
+function getMd(): any {
+    if (!_md) {
+        _md = require('markdown-it')({ html: true, linkify: true, typographer: true });
+        _md.renderer.rules.heading_open = (tokens: any[], idx: number) => {
+            const level = tokens[idx].tag.substring(1);
+            return `<h${level} class="custom-heading">`;
+        };
+        _md.renderer.rules.heading_close = (tokens: any[], idx: number) => {
+            const level = tokens[idx].tag.substring(1);
+            return `</h${level}>`;
+        };
+    }
+    return _md;
+}
 
 let beautifyOptions = {
     "indent_size": 4,
@@ -569,6 +427,7 @@ export type DefaultTemplateType = {
     minInstances?:number;
     maxInstances?:number;
     validate?:Function;
+    list?:boolean;
 }
 
 export class NodeBuilder {
@@ -598,12 +457,12 @@ export class NodeBuilder {
     }
 
     public addDefault(name:string, template:DefaultTemplateType):NodeBuilder{
-        // if the template type ends with squeare brackets, its an array type.
-        if(template.type && template.type.endsWith("[]")) {
+        // if list:true, this is an array of config references - generate shadow defaults for each slot.
+        if(template.list === true) {
             template.maxInstances =  template.maxInstances ? template.maxInstances : 10;
-            this._defaults[name] = {type:template.type, value:"", required:true, maxInstances:template.maxInstances};
+            this._defaults[name] = {value:"", required:(template.required) ? template.required : false, maxInstances:template.maxInstances};
             for(let i = 0; i < template.maxInstances; i++){
-                this._defaults["_" + name + "_" + i] = {value:"", required:false};
+                this._defaults["_" + name + "_" + i] = {value:"", required:false, type: template.type};
             }
         }
         else{
@@ -648,9 +507,10 @@ export class NodeBuilder {
 
     public addTemplate(template: { template: ITemplateClass; config: TemplateConfig; }): NodeBuilder {
         let templateDescriptor = template.template.getTemplateDescriptor();
+        let templateConfig = template.config;
 
         // load the html source file
-        let uiDom = new JSDOM(fs.readFileSync(templateDescriptor.templateFile()!));
+        let uiDom = new (getJSDOM())(fs.readFileSync(templateDescriptor.templateFile()!));
 
         // apply the onCompose section if available.
         let onCompose = uiDom.window.document.querySelector("script[template-section='onCompose']")?.innerHTML;
@@ -662,7 +522,16 @@ export class NodeBuilder {
         let sections:string[] = ['IncludeOnce', 'IncludeEditPrepare', 'IncludeEditSave', 'IncludeEditCancel', 'IncludeEditDelete', 'IncludeEditForm', 'IncludeDocumentation'];
         Object.values(sections).forEach(section => {
             let sectionData = uiDom.window.document.querySelector(`[template-section='on${section}']`)?.innerHTML;
+
             if(sectionData){
+                if(section !== "IncludeOnce" && section !== "IncludeEditForm" && section !== "IncludeDocumentation") {
+                    // add the packageing around the sectionData
+                    sectionData = `{
+                        let node = this;
+                        let templateConfig = ${JSON.stringify(templateConfig)};
+                        ${sectionData}
+                    }`;
+                }
                 (this as any)[`add${section}`](templateDescriptor.name(), sectionData);
             }
         });
@@ -705,6 +574,69 @@ export class NodeBuilder {
         return this;
     }
 
+    public buildOnceHtml(): {source: string, html: string}[] {
+        return this._onIncludeOnce.map(entry => ({
+            source: entry.source,
+            html: this.serializeHTML(entry.source, entry.script)
+        }));
+    }
+
+    public buildDeferredType(): string {
+        // Apply shadow defaults (same side-effect as buildType)
+        let shaddowDefaults = Object.entries(this._defaults).filter(([name, template]) => template.type && template.type.endsWith("[]"));
+        if (shaddowDefaults.length > 0) {
+            this.addIncludeEditSave(this._name + "_shaddowDefaults",
+                `{ let node = this; ` +
+                    shaddowDefaults
+                    .map(([name, template]) => `
+                    for(let i = 0; i < ${template.maxInstances}; i++){
+                        node["_${name}_" + i] = (node.${name}[i] || {}).proxy || "";
+                    }`)
+                    .join("\n\n") +
+                `}`
+            );
+        }
+
+        const registerTypeBody = `{
+            category: '${this._category}',
+            ${(this._category !== "config" && this._icon) ? `icon: '${this._icon}',` : ""}
+            ${(this._category !== "config" && this._color) ? `color: '${this._color}',` : ""}
+            ${(this._category !== "config" && this._labelStyle) ? `labelStyle: '${this._labelStyle}',` : ""}
+            ${(this._label) ? `label: ${this._label},` : ""}
+            ${this._paletteLabel ? `paletteLabel: '${this._paletteLabel}',` : ""}
+            ${(this._category !== "config") ? `inputs:${this._inputs ? "1" : "0"},` : ""}
+            ${(this._category !== "config") ? `inputLabels:(i) => '${this._inputs}',` : ""}
+            ${(this._category !== "config") ? `outputs:${this._outputs.length},` : ""}
+            ${(this._category !== "config") ? `outputLabels:(i) => ${this.serializeProperty(this._outputs)}[i],` : ""}
+            defaults: {
+                ${Object.entries(this._defaults).map(([key, value]) => this.serializeDefault(key, value)).join("\n")}
+            },
+            oneditprepare: function() {
+                ${this._onIncludeEditPrepare.map(entry => this.serializeJavaScript(entry.source, entry.script)).join("")}
+            },
+            oneditsave: function() {
+                ${this._onIncludeEditSave.map(entry => this.serializeJavaScript(entry.source, entry.script)).join("")}
+            },
+            oneditcancel: function() {
+                ${this._onIncludeEditCancel.map(entry => this.serializeJavaScript(entry.source, entry.script)).join("")}
+            },
+            oneditdelete: function() {
+                ${this._onIncludeEditDelete.map(entry => this.serializeJavaScript(entry.source, entry.script)).join("")}
+            },
+        }`;
+
+        return getBeautify().html_beautify(`
+        <script type="text/javascript">
+        (function () {
+            RED.events.on('registry:node-set-added', function (ns) {
+                if (ns.types && ns.types.indexOf('${this._name}') !== -1) {
+                    RED.nodes.registerType('${this._name}', ${registerTypeBody});
+                }
+            });
+        }());
+        </script>`, beautifyOptions);
+    }
+
     private serializeProperty(value:any):string{
         switch(typeof value){
             case "boolean": {
@@ -737,18 +669,18 @@ export class NodeBuilder {
         }
 
         return `<!-- ${source} -->
-        ${Handlebars.compile(script)(context)}
+        ${getHandlebars().compile(script)(context)}
         `
     }
 
-    private serializeJavaScript(source:string, script:string):string{ 
+    private serializeJavaScript(source:string, script:string):string{
         let context = {
-            defaults: (this._category === "config") 
+            defaults: (this._category === "config")
                 ? Object.fromEntries(Object.keys(this._defaults).map(k => [k, `node-config-input-${k}`]))
                 : Object.fromEntries(Object.keys(this._defaults).map(k => [k, `node-input-${k}`]))
         }
 
-        let serialized = Handlebars.compile(script)(context).trim();
+        let serialized = getHandlebars().compile(script)(context).trim();
         if(!serialized.startsWith("{") && !serialized.endsWith("}")){
             serialized = "{\n" + serialized + "\n}";
         }
@@ -758,14 +690,14 @@ export class NodeBuilder {
         `
     }
 
-    private serializeFormEntry(source:string, script:string):string{ 
+    private serializeFormEntry(source:string, script:string):string{
         let context = {
-            defaults: (this._category === "config") 
+            defaults: (this._category === "config")
                 ? Object.fromEntries(Object.keys(this._defaults).map(k => [k, `node-config-input-${k}`]))
                 : Object.fromEntries(Object.keys(this._defaults).map(k => [k, `node-input-${k}`]))
         }
         return `
-            <div id='section_${source}'>${Handlebars.compile(script)(context)}
+            <div id='section_${source}'>${getHandlebars().compile(script)(context)}
             </div>
         `
     }
@@ -794,7 +726,7 @@ export class NodeBuilder {
             );
         }
 
-        return beautify.html_beautify(`
+        return getBeautify().html_beautify(`
         ${this._onIncludeOnce.map(entry => this.serializeHTML(entry.source, entry.script)).join("")}
         <script type="text/javascript">
             RED.nodes.registerType('${this._name}',{      
@@ -828,7 +760,7 @@ export class NodeBuilder {
     };
 
     public buildHtml():string{
-        return beautify.html_beautify(`
+        return getBeautify().html_beautify(`
         <script type="text/html" data-template-name='${this._name}'>
             ${this._onIncludeEditForm.map(entry => this.serializeFormEntry(entry.source, entry.script)).join("")}
         </script>
@@ -861,13 +793,13 @@ let evalInContext = (context: any, js: string) => {
     }
 };
 
-let buildNode = function(node:INodeClass){
+let buildNodeBuilder = function(node: INodeClass): NodeBuilder {
     // get the node descriptor and create the builder.
     let nodeDescriptor = node.getNodeDescriptor();
     let nodeBuilder = new NodeBuilder(nodeDescriptor.id(), nodeDescriptor.group());
 
     // load the html source file
-    let uiDom = new JSDOM(fs.readFileSync(nodeDescriptor.sourceFile()));
+    let uiDom = new (getJSDOM())(fs.readFileSync(nodeDescriptor.sourceFile()));
 
     // apply the onCompose section if available.
     let onCompose = uiDom.window.document.querySelector("script[template-section='onCompose']")?.innerHTML;
@@ -887,10 +819,15 @@ let buildNode = function(node:INodeClass){
     // add some of the dependencies here.
     nodeDescriptor.templates().forEach(template => nodeBuilder.addTemplate(template));
 
-    return [nodeBuilder.buildType(), nodeBuilder.buildHtml(), nodeBuilder.buildDocumentation()].join("\n\n");
+    return nodeBuilder;
+}
 
+let buildNode = function(node:INodeClass){
+    let nodeBuilder = buildNodeBuilder(node);
+    return [nodeBuilder.buildType(), nodeBuilder.buildHtml(), nodeBuilder.buildDocumentation()].join("\n\n");
 }
 
 export {
-    buildNode
+    buildNode,
+    buildNodeBuilder
 }

package/src/core/NodeGenerator.ts

@@ -1,6 +1,7 @@
 import * as fs from 'fs';
+import * as path from 'path';
 
-import { buildNode, Template, NodeDescriptor, BaseService, ServiceDescriptor, BaseNode, DependencyType, TemplateDescriptor } from "./NodeConstructor"
+import { buildNode, buildNodeBuilder, Template, NodeDescriptor, BaseService, ServiceDescriptor, BaseNode, DependencyType, TemplateDescriptor } from "./NodeConstructor"
 
 const templatePropertyKeys = ["onIncludeOnce", "onIncludeDefaults", "onIncludeEditPrepare", "onIncludeEditPrepare", "onIncludeEditSave", "onIncludeEditForm"];
 
@@ -105,31 +106,17 @@ class NodeGenerator {
     }
 
     public generate(nodesOutputFile:string, pluginsOutputFile:string){
-        // STEP 1. Compile the templates.
-        // let templateInclude =  this._templates
-        //     .map(template => new (template.clazz()))
-        //     .map(template => {
-        //         return template.onIncludeOnce();
-        //     })
-        //     .join("\n");
-        let templateInclude = ""
-        
-        // STEP 2. compose the HTMLStructure of each of the nodes.
-        let nodeHTMLOutut = Object.values(this._nodes)
-            .map(node => {
-                console.log(`Processing node: ${node.descriptor.id()}`);
-
-                return buildNode(node.nodeClass);
-            })
-            .join("\n")
-
-        fs.writeFileSync(nodesOutputFile + ".html",`
+        // STEP 1. Write an empty Nodes.html.
+        // Client-side node type definitions live in Plugins.html (generated below)
+        // to prevent HTML-scanning conflicts when multiple plugins bundle the same
+        // plugincore infrastructure nodes.
+        fs.writeFileSync(nodesOutputFile + ".html", `
 <!--
 ${NodeGenerator.warning}
+Client-side node type definitions have been moved to Plugins.html to prevent
+duplicate type registration conflicts when multiple plugins share the same
+plugincore infrastructure nodes.
 -->
-${templateInclude}
-${nodeHTMLOutut}
-
         `);
 
         // Step 3. compose the JS structure for each of the nodes.
@@ -153,15 +140,37 @@ ${nodeHTMLOutut}
 ${NodeGenerator.warning}
 */
 "use strict";
-const NodeManager = require("@theotherwillembotha/node-red-plugincore").NodeManager;
+const { NodeManager } = require("./runtime/NodeManagerRuntime");
 ${importList}
 module.exports = (RED) => {
 let manager = new NodeManager(RED);
 ${moduleExports}
-}        
-        
+}
+
         `);
 
+        // Copy NodeManagerRuntime.js into the plugin's build/runtime/ folder so that
+        // the generated Nodes.js require("./runtime/NodeManagerRuntime") resolves locally
+        // with no dependency on plugincore being installed in the user's environment.
+        const outputDir = path.dirname(nodesOutputFile);
+        const runtimeDestDir = path.join(outputDir, 'runtime');
+        const runtimeDestPath = path.join(runtimeDestDir, 'NodeManagerRuntime.js');
+
+        let runtimeSrcPath: string;
+        try {
+            // Resolves when this is a consumer plugin - plugincore is in node_modules.
+            runtimeSrcPath = require.resolve('@theotherwillembotha/node-red-plugincore/build/runtime/NodeManagerRuntime');
+        } catch {
+            // We ARE plugincore - NodeManagerRuntime.js is compiled alongside us in build/core/.
+            runtimeSrcPath = path.join(__dirname, 'NodeManagerRuntime.js');
+        }
+
+        if (path.resolve(runtimeSrcPath) !== path.resolve(runtimeDestPath)) {
+            fs.mkdirSync(runtimeDestDir, { recursive: true });
+            fs.copyFileSync(runtimeSrcPath, runtimeDestPath);
+        }
+        console.log(`NodeManagerRuntime.js → ${runtimeDestPath}`);
+
         console.log(`Done building ${Object.keys(this._nodes).length} nodes\n${Object.values(this._nodes).map(node => " - " + node.descriptor.id() + "\n").join("")}\n`);
 
         let serviceExports = this._services.map(service => {
@@ -195,31 +204,39 @@ module.exports = function (RED) {
             let plugin = RED.plugins.get(pluginID);
 
             if(addedPlugin && plugin.instantiate && !plugin.instance){
-                console.log("Deploying instance of: " + plugin.id);
                 plugin.instance = new plugin.class();
-                
+
                 // instantiate the plugin.
                 await plugin.instance.init(RED);
 
+                // Eagerly call onDeploy with any pre-existing saved flows so that
+                // service registries (MetricsService, WebhookServerService, etc.) are
+                // populated before config nodes are constructed during flow startup.
+                // Without this, MetricsConfigNode._metrics is undefined on first boot.
+                const existingFlows = await runtime.flows.getFlows({});
+                const hasExistingFlows = existingFlows && existingFlows.flows && existingFlows.flows.length > 0;
+                if (hasExistingFlows) {
+                    await plugin.instance.onDeploy(existingFlows);
+                }
+
                 // publish an event that it has ben deployed.
                 RED.events.emit("plugin.instantiated", pluginID);
-                                
+
                 pluginList.splice(pluginList.findIndex(current => current.id() === addedPlugin.id()), 1);
 
                 // register a flow deployment listener.
-                let startupDeployment = true;
+                // startupDeployment is only needed for a truly fresh Node-RED with no saved flows.
+                let startupDeployment = !hasExistingFlows;
                 RED.events.on('runtime-event', async (event) => {
-                    // startup deployment.
+                    // startup deployment (fresh Node-RED with no pre-existing flows).
                     if ("runtime-deploy" === event?.id && startupDeployment) {
-                        //console.log("STARTUP DEPLOYMENT");
                         startupDeployment = false;
                         const flows = await runtime.flows.getFlows({});
                         await plugin.instance.onDeploy(flows);
                     }
-                    
+
                     // stopping flows on redeployment.
                     if ("runtime-state" === event?.id && event?.payload?.state === "stop" && event?.payload?.deploy) {
-                        //console.log("REDEPLOY!");
                         const flows = await runtime.flows.getFlows({});
                         await plugin.instance.onDeploy(flows);
                     }
@@ -240,6 +257,43 @@ module.exports = function (RED) {
 `
         fs.writeFileSync(pluginsOutputFile + ".js", output);
         console.log(`Done building ${this._services.length} services\n${this._services.map(service => " - " + service.name() + "\n").join("")}\n`);
+
+        // STEP 4. Generate Plugins.html with deferred client-side node definitions.
+        // All nodes are registered via registry:node-set-added so that:
+        //   a) registerType is only called after typeToId/nodeSets are populated, and
+        //   b) data-template-name declarations are absent from Nodes.html, avoiding
+        //      the HTML-scanning conflict that causes Node-RED to reject an entire
+        //      plugin when a second package declares the same node type.
+        const seenOnce = new Set<string>();
+        const onceHtmlParts: string[] = [];
+        const nodePluginParts: string[] = [];
+
+        Object.values(this._nodes).forEach(node => {
+            console.log(`Building Plugins.html for: ${node.descriptor.id()}`);
+            const nb = buildNodeBuilder(node.nodeClass);
+
+            nb.buildOnceHtml().forEach(({source, html}) => {
+                if (!seenOnce.has(source)) {
+                    seenOnce.add(source);
+                    onceHtmlParts.push(html);
+                }
+            });
+
+            nodePluginParts.push([
+                nb.buildDeferredType(),
+                nb.buildHtml(),
+                nb.buildDocumentation()
+            ].join('\n\n'));
+        });
+
+        fs.writeFileSync(pluginsOutputFile + ".html", `
+<!--
+${NodeGenerator.warning}
+-->
+${onceHtmlParts.join('\n')}
+${nodePluginParts.join('\n\n')}
+        `);
+        console.log(`Done building Plugins.html\n`);
     }
 
     private static generateHeader(node:NodeDescriptor){