mirror of
https://github.com/fleetbase/fleetbase.git
synced 2025-12-19 14:18:57 +00:00
refactor: Simplify extension generator to write directly to app directory
- Remove Broccoli tree complexity, use included() hook instead - Write files directly to app/ using Node.js fs operations - Generate extension shims in app/extensions/ - Generate extension loaders in app/utils/extension-loaders.generated.js - Generate router.js with proper engine mounts using recast AST - Generate extensions.json manifest in public/ - Add recast dependency for AST manipulation - Much simpler and more reliable than Broccoli trees
This commit is contained in:
@@ -1,11 +1,8 @@
|
||||
'use strict';
|
||||
|
||||
const ExtensionDiscoveryPlugin = require('./plugins/extension-discovery');
|
||||
const ExtensionShimGeneratorPlugin = require('./plugins/extension-shim-generator');
|
||||
const ExtensionLoadersGeneratorPlugin = require('./plugins/extension-loaders-generator');
|
||||
const RouterGeneratorPlugin = require('./plugins/router-generator');
|
||||
const Funnel = require('broccoli-funnel');
|
||||
const mergeTrees = require('broccoli-merge-trees');
|
||||
const path = require('path');
|
||||
const fs = require('fs');
|
||||
const fg = require('fast-glob');
|
||||
|
||||
console.log('[fleetbase-extensions-generator] Addon loaded at startup');
|
||||
|
||||
@@ -13,122 +10,449 @@ module.exports = {
|
||||
name: 'fleetbase-extensions-generator',
|
||||
|
||||
isDevelopingAddon() {
|
||||
return true; // useful for live reload when developing
|
||||
return true;
|
||||
},
|
||||
|
||||
treeForApp(appTree) {
|
||||
console.log('[fleetbase-extensions-generator] treeForApp() called');
|
||||
/**
|
||||
* The included hook runs once when the addon is loaded.
|
||||
* We use it to generate extension files directly to the app directory.
|
||||
*/
|
||||
included(app) {
|
||||
this._super.included.apply(this, arguments);
|
||||
|
||||
console.log('[fleetbase-extensions-generator] ========================================');
|
||||
console.log('[fleetbase-extensions-generator] included() hook called');
|
||||
console.log('[fleetbase-extensions-generator] Project root:', this.project.root);
|
||||
|
||||
// Step 1: Create Broccoli plugin instances
|
||||
console.log('[fleetbase-extensions-generator] Creating ExtensionDiscoveryPlugin...');
|
||||
let extensionDiscovery = new ExtensionDiscoveryPlugin([], {
|
||||
projectRoot: this.project.root,
|
||||
annotation: 'Discover Fleetbase Extensions',
|
||||
});
|
||||
|
||||
console.log('[fleetbase-extensions-generator] Creating ExtensionShimGeneratorPlugin...');
|
||||
let extensionShims = new ExtensionShimGeneratorPlugin([extensionDiscovery], {
|
||||
projectRoot: this.project.root,
|
||||
annotation: 'Generate extension shims',
|
||||
});
|
||||
|
||||
console.log('[fleetbase-extensions-generator] Creating ExtensionLoadersGeneratorPlugin...');
|
||||
let extensionLoaders = new ExtensionLoadersGeneratorPlugin([extensionDiscovery], {
|
||||
projectRoot: this.project.root,
|
||||
annotation: 'Generate extension loaders module',
|
||||
});
|
||||
|
||||
console.log('[fleetbase-extensions-generator] Creating RouterGeneratorPlugin...');
|
||||
let routerGen = new RouterGeneratorPlugin([extensionDiscovery], {
|
||||
projectRoot: this.project.root,
|
||||
annotation: 'Generate router with extension mounts',
|
||||
});
|
||||
|
||||
// Step 2: Funnel plugin outputs into proper app/ structure
|
||||
//
|
||||
// Plugin outputs:
|
||||
// - extensionShims: outputPath/extensions/*.js
|
||||
// - extensionLoaders: outputPath/utils/extension-loaders.generated.js
|
||||
// - routerGen: outputPath/router.js
|
||||
//
|
||||
// We want these in the app tree as:
|
||||
// - app/extensions/*.js
|
||||
// - app/utils/extension-loaders.generated.js
|
||||
// - app/router.js
|
||||
// Discover extensions
|
||||
const extensions = this.discoverExtensions();
|
||||
console.log('[fleetbase-extensions-generator] Found', extensions.length, 'extension(s)');
|
||||
|
||||
console.log('[fleetbase-extensions-generator] Funneling extension shims from extensions/ to app/extensions/');
|
||||
let shimsFunneled = new Funnel(extensionShims, {
|
||||
srcDir: 'extensions', // Files are in outputPath/extensions/
|
||||
destDir: 'extensions', // Place at app/extensions/ in final tree
|
||||
annotation: 'Funnel extension shims'
|
||||
});
|
||||
// Generate files directly to app directory
|
||||
this.generateExtensionShims(extensions);
|
||||
this.generateExtensionLoaders(extensions);
|
||||
this.generateRouter(extensions);
|
||||
this.generateExtensionsManifest(extensions);
|
||||
|
||||
console.log('[fleetbase-extensions-generator] ========================================');
|
||||
},
|
||||
|
||||
console.log('[fleetbase-extensions-generator] Funneling extension loaders from utils/ to app/utils/');
|
||||
let loadersFunneled = new Funnel(extensionLoaders, {
|
||||
srcDir: 'utils', // Files are in outputPath/utils/
|
||||
destDir: 'utils', // Place at app/utils/ in final tree
|
||||
annotation: 'Funnel extension loaders'
|
||||
/**
|
||||
* Discover Fleetbase extensions from node_modules
|
||||
*/
|
||||
discoverExtensions() {
|
||||
console.log('[fleetbase-extensions-generator] Discovering extensions...');
|
||||
|
||||
const extensions = [];
|
||||
const seenPackages = new Set();
|
||||
|
||||
const results = fg.sync([
|
||||
'node_modules/*/package.json',
|
||||
'node_modules/*/*/package.json'
|
||||
], {
|
||||
cwd: this.project.root,
|
||||
absolute: true
|
||||
});
|
||||
|
||||
console.log('[fleetbase-extensions-generator] Funneling router.js to app/router.js');
|
||||
let routerFunneled = new Funnel(routerGen, {
|
||||
srcDir: '/', // Files are at outputPath root
|
||||
destDir: '', // Place at app/ root in final tree
|
||||
annotation: 'Funnel router.js'
|
||||
});
|
||||
|
||||
// Step 3: Merge all generated files into a single tree
|
||||
console.log('[fleetbase-extensions-generator] Merging generated files into single tree');
|
||||
let generated = mergeTrees(
|
||||
[shimsFunneled, loadersFunneled, routerFunneled],
|
||||
{
|
||||
overwrite: true,
|
||||
annotation: 'Merge generated extension files',
|
||||
|
||||
for (const packagePath of results) {
|
||||
let packageData = null;
|
||||
|
||||
try {
|
||||
const packageJson = fs.readFileSync(packagePath, 'utf8');
|
||||
packageData = JSON.parse(packageJson);
|
||||
} catch (e) {
|
||||
continue;
|
||||
}
|
||||
);
|
||||
|
||||
// Step 4: Merge with existing appTree
|
||||
console.log('[fleetbase-extensions-generator] Merging generated tree with existing app tree');
|
||||
if (!appTree) {
|
||||
console.log('[fleetbase-extensions-generator] No existing appTree, returning generated tree only');
|
||||
return generated;
|
||||
|
||||
// Check if this is a Fleetbase extension
|
||||
if (!packageData ||
|
||||
!packageData.keywords ||
|
||||
!packageData.keywords.includes('fleetbase-extension') ||
|
||||
!packageData.keywords.includes('ember-engine')) {
|
||||
continue;
|
||||
}
|
||||
|
||||
// Skip duplicates
|
||||
if (seenPackages.has(packageData.name)) {
|
||||
continue;
|
||||
}
|
||||
|
||||
seenPackages.add(packageData.name);
|
||||
|
||||
const extension = {
|
||||
name: packageData.name,
|
||||
version: packageData.version,
|
||||
fleetbase: packageData.fleetbase || {}
|
||||
};
|
||||
|
||||
extensions.push(extension);
|
||||
console.log('[fleetbase-extensions-generator] -', extension.name + '@' + extension.version);
|
||||
}
|
||||
|
||||
return mergeTrees([appTree, generated], {
|
||||
overwrite: true,
|
||||
annotation: 'Merge app tree with generated extensions'
|
||||
});
|
||||
return extensions;
|
||||
},
|
||||
|
||||
treeForPublic(publicTree) {
|
||||
console.log('[fleetbase-extensions-generator] treeForPublic() called');
|
||||
/**
|
||||
* Generate extension shim files in app/extensions/
|
||||
*/
|
||||
generateExtensionShims(extensions) {
|
||||
console.log('[fleetbase-extensions-generator] Generating extension shims...');
|
||||
|
||||
// Expose the extensions manifest to dist/extensions.json
|
||||
console.log('[fleetbase-extensions-generator] Creating ExtensionDiscoveryPlugin for public tree...');
|
||||
let extensionDiscovery = new ExtensionDiscoveryPlugin([], {
|
||||
projectRoot: this.project.root,
|
||||
annotation: 'Discover Fleetbase Extensions (public)',
|
||||
});
|
||||
|
||||
// Funnel just extensions.json to dist root
|
||||
console.log('[fleetbase-extensions-generator] Funneling extensions.json to dist/');
|
||||
let manifest = new Funnel(extensionDiscovery, {
|
||||
srcDir: '/',
|
||||
files: ['extensions.json'],
|
||||
destDir: '/',
|
||||
annotation: 'Funnel extensions.json manifest'
|
||||
});
|
||||
|
||||
if (!publicTree) {
|
||||
console.log('[fleetbase-extensions-generator] No existing publicTree, returning manifest only');
|
||||
return manifest;
|
||||
const extensionsDir = path.join(this.project.root, 'app', 'extensions');
|
||||
|
||||
// Create directory
|
||||
if (!fs.existsSync(extensionsDir)) {
|
||||
fs.mkdirSync(extensionsDir, { recursive: true });
|
||||
}
|
||||
|
||||
let shimCount = 0;
|
||||
|
||||
for (const extension of extensions) {
|
||||
const pkgName = extension.name;
|
||||
const mountName = this.getExtensionMountPath(pkgName);
|
||||
|
||||
// Path to extension.js in the engine's addon directory
|
||||
const extensionPath = path.join(
|
||||
this.project.root,
|
||||
'node_modules',
|
||||
pkgName,
|
||||
'addon',
|
||||
'extension.js'
|
||||
);
|
||||
|
||||
// Check if extension.js exists
|
||||
if (!fs.existsSync(extensionPath)) {
|
||||
console.log('[fleetbase-extensions-generator] ! No extension.js found for', pkgName);
|
||||
continue;
|
||||
}
|
||||
|
||||
// Read the extension code
|
||||
let extensionCode;
|
||||
try {
|
||||
extensionCode = fs.readFileSync(extensionPath, 'utf8');
|
||||
} catch (error) {
|
||||
console.error('[fleetbase-extensions-generator] ! Failed to read extension.js for', pkgName, ':', error.message);
|
||||
continue;
|
||||
}
|
||||
|
||||
// Generate shim content
|
||||
const shimContent = `// GENERATED BY fleetbase-extensions-generator - DO NOT EDIT
|
||||
// Extension setup for ${pkgName}
|
||||
//
|
||||
// This file contains the inlined extension setup code from the engine.
|
||||
// It is generated at build time to enable dynamic import without loading
|
||||
// the entire engine bundle.
|
||||
|
||||
console.log('[fleetbase-extensions-generator] Merging manifest with existing public tree');
|
||||
return mergeTrees([publicTree, manifest], {
|
||||
overwrite: true,
|
||||
annotation: 'Merge public tree with extensions manifest'
|
||||
});
|
||||
${extensionCode}
|
||||
`;
|
||||
|
||||
// Write shim file
|
||||
const shimPath = path.join(extensionsDir, `${mountName}.js`);
|
||||
try {
|
||||
fs.writeFileSync(shimPath, shimContent, 'utf8');
|
||||
shimCount++;
|
||||
console.log('[fleetbase-extensions-generator] \u2713 Generated', `app/extensions/${mountName}.js`);
|
||||
} catch (error) {
|
||||
console.error('[fleetbase-extensions-generator] ! Failed to write shim for', pkgName, ':', error.message);
|
||||
}
|
||||
}
|
||||
|
||||
console.log('[fleetbase-extensions-generator] Generated', shimCount, 'extension shim(s)');
|
||||
},
|
||||
|
||||
/**
|
||||
* Generate extension loaders in app/utils/
|
||||
*/
|
||||
generateExtensionLoaders(extensions) {
|
||||
console.log('[fleetbase-extensions-generator] Generating extension loaders...');
|
||||
|
||||
const utilsDir = path.join(this.project.root, 'app', 'utils');
|
||||
|
||||
// Create directory
|
||||
if (!fs.existsSync(utilsDir)) {
|
||||
fs.mkdirSync(utilsDir, { recursive: true });
|
||||
}
|
||||
|
||||
const lines = [
|
||||
'// GENERATED BY fleetbase-extensions-generator - DO NOT EDIT',
|
||||
'// Extension loader map for dynamic imports',
|
||||
'',
|
||||
'export const EXTENSION_LOADERS = {'
|
||||
];
|
||||
|
||||
let loaderCount = 0;
|
||||
|
||||
for (const extension of extensions) {
|
||||
const pkgName = extension.name;
|
||||
|
||||
// Check if extension.js exists
|
||||
const extensionPath = path.join(
|
||||
this.project.root,
|
||||
'node_modules',
|
||||
pkgName,
|
||||
'addon',
|
||||
'extension.js'
|
||||
);
|
||||
|
||||
if (!fs.existsSync(extensionPath)) {
|
||||
continue;
|
||||
}
|
||||
|
||||
const mountName = this.getExtensionMountPath(pkgName);
|
||||
lines.push(` '${pkgName}': () => import('@fleetbase/console/extensions/${mountName}'),`);
|
||||
loaderCount++;
|
||||
}
|
||||
|
||||
lines.push('};');
|
||||
lines.push('');
|
||||
|
||||
const loadersContent = lines.join('\n');
|
||||
const loadersPath = path.join(utilsDir, 'extension-loaders.generated.js');
|
||||
|
||||
try {
|
||||
fs.writeFileSync(loadersPath, loadersContent, 'utf8');
|
||||
console.log('[fleetbase-extensions-generator] \u2713 Generated app/utils/extension-loaders.generated.js with', loaderCount, 'loader(s)');
|
||||
} catch (error) {
|
||||
console.error('[fleetbase-extensions-generator] ! Failed to write extension loaders:', error.message);
|
||||
}
|
||||
},
|
||||
|
||||
/**
|
||||
* Generate router.js with extension mounts
|
||||
*/
|
||||
generateRouter(extensions) {
|
||||
console.log('[fleetbase-extensions-generator] Generating router.js...');
|
||||
|
||||
const routerMapFile = path.join(this.project.root, 'app', 'router.map.js');
|
||||
const routerFile = path.join(this.project.root, 'app', 'router.js');
|
||||
|
||||
if (!fs.existsSync(routerMapFile)) {
|
||||
console.error('[fleetbase-extensions-generator] ! router.map.js not found at:', routerMapFile);
|
||||
return;
|
||||
}
|
||||
|
||||
// Read router.map.js
|
||||
const routerMapContent = fs.readFileSync(routerMapFile, 'utf8');
|
||||
|
||||
// Separate extensions by mount location
|
||||
const consoleExtensions = [];
|
||||
const rootExtensions = [];
|
||||
|
||||
for (const extension of extensions) {
|
||||
const mountLocation = extension.fleetbase?.route?.mountLocation || 'console';
|
||||
const route = extension.fleetbase?.route?.slug || this.getExtensionMountPath(extension.name);
|
||||
|
||||
if (mountLocation === 'console') {
|
||||
consoleExtensions.push({ name: extension.name, route });
|
||||
} else if (mountLocation === 'root') {
|
||||
rootExtensions.push({ name: extension.name, route });
|
||||
}
|
||||
}
|
||||
|
||||
console.log('[fleetbase-extensions-generator] Console extensions:', consoleExtensions.length);
|
||||
console.log('[fleetbase-extensions-generator] Root extensions:', rootExtensions.length);
|
||||
|
||||
// Parse and modify the router using simple string manipulation
|
||||
// (We'll use recast for proper AST manipulation)
|
||||
const recast = require('recast');
|
||||
const babelParser = require('recast/parsers/babel');
|
||||
|
||||
const ast = recast.parse(routerMapContent, { parser: babelParser });
|
||||
|
||||
let consoleAdded = 0;
|
||||
let rootAdded = 0;
|
||||
|
||||
// Add console extensions
|
||||
if (consoleExtensions.length > 0) {
|
||||
consoleAdded = this.addConsoleExtensions(ast, consoleExtensions);
|
||||
}
|
||||
|
||||
// Add root extensions
|
||||
if (rootExtensions.length > 0) {
|
||||
rootAdded = this.addRootExtensions(ast, rootExtensions);
|
||||
}
|
||||
|
||||
// Generate output
|
||||
const output = recast.print(ast, { quote: 'single' }).code;
|
||||
|
||||
try {
|
||||
fs.writeFileSync(routerFile, output, 'utf8');
|
||||
console.log('[fleetbase-extensions-generator] \u2713 Generated app/router.js');
|
||||
console.log('[fleetbase-extensions-generator] - Console mounts:', consoleAdded);
|
||||
console.log('[fleetbase-extensions-generator] - Root mounts:', rootAdded);
|
||||
} catch (error) {
|
||||
console.error('[fleetbase-extensions-generator] ! Failed to write router.js:', error.message);
|
||||
}
|
||||
},
|
||||
|
||||
/**
|
||||
* Add console extensions to the router AST
|
||||
*/
|
||||
addConsoleExtensions(ast, extensions) {
|
||||
const recast = require('recast');
|
||||
const types = recast.types;
|
||||
const n = types.namedTypes;
|
||||
const b = types.builders;
|
||||
|
||||
let addedCount = 0;
|
||||
|
||||
types.visit(ast, {
|
||||
visitCallExpression(path) {
|
||||
const node = path.node;
|
||||
|
||||
// Look for this.route('console', ...)
|
||||
if (n.MemberExpression.check(node.callee) &&
|
||||
n.ThisExpression.check(node.callee.object) &&
|
||||
node.callee.property.name === 'route' &&
|
||||
node.arguments.length > 0 &&
|
||||
n.Literal.check(node.arguments[0]) &&
|
||||
node.arguments[0].value === 'console') {
|
||||
|
||||
// Find the function expression in the second argument
|
||||
if (node.arguments.length > 1 && n.FunctionExpression.check(node.arguments[1])) {
|
||||
const functionExpression = node.arguments[1];
|
||||
|
||||
// Add mount statements for each extension
|
||||
extensions.forEach(extension => {
|
||||
// Check if already mounted
|
||||
if (!this.isEngineMounted(functionExpression.body.body, extension.name)) {
|
||||
const mountStatement = b.expressionStatement(
|
||||
b.callExpression(
|
||||
b.memberExpression(
|
||||
b.thisExpression(),
|
||||
b.identifier('mount')
|
||||
),
|
||||
[
|
||||
b.literal(extension.name),
|
||||
b.objectExpression([
|
||||
b.property('init', b.identifier('as'), b.literal(extension.route))
|
||||
])
|
||||
]
|
||||
)
|
||||
);
|
||||
|
||||
functionExpression.body.body.push(mountStatement);
|
||||
addedCount++;
|
||||
}
|
||||
});
|
||||
}
|
||||
|
||||
return false; // Don't traverse children
|
||||
}
|
||||
|
||||
this.traverse(path);
|
||||
}
|
||||
});
|
||||
|
||||
return addedCount;
|
||||
},
|
||||
|
||||
/**
|
||||
* Add root extensions to the router AST
|
||||
*/
|
||||
addRootExtensions(ast, extensions) {
|
||||
const recast = require('recast');
|
||||
const types = recast.types;
|
||||
const n = types.namedTypes;
|
||||
const b = types.builders;
|
||||
|
||||
let addedCount = 0;
|
||||
|
||||
types.visit(ast, {
|
||||
visitExportDefaultDeclaration(path) {
|
||||
const node = path.node;
|
||||
|
||||
// Look for export default function(router) { ... }
|
||||
if (n.FunctionExpression.check(node.declaration)) {
|
||||
const functionExpression = node.declaration;
|
||||
|
||||
// Add mount statements for each root extension
|
||||
extensions.forEach(extension => {
|
||||
// Check if already mounted
|
||||
if (!this.isEngineMounted(functionExpression.body.body, extension.name)) {
|
||||
const mountStatement = b.expressionStatement(
|
||||
b.callExpression(
|
||||
b.memberExpression(
|
||||
b.identifier('router'),
|
||||
b.identifier('mount')
|
||||
),
|
||||
[
|
||||
b.literal(extension.name),
|
||||
b.objectExpression([
|
||||
b.property('init', b.identifier('as'), b.literal(extension.route))
|
||||
])
|
||||
]
|
||||
)
|
||||
);
|
||||
|
||||
functionExpression.body.body.push(mountStatement);
|
||||
addedCount++;
|
||||
}
|
||||
});
|
||||
|
||||
return false; // Don't traverse children
|
||||
}
|
||||
|
||||
this.traverse(path);
|
||||
}
|
||||
});
|
||||
|
||||
return addedCount;
|
||||
},
|
||||
|
||||
/**
|
||||
* Check if an engine is already mounted in the AST
|
||||
*/
|
||||
isEngineMounted(statements, engineName) {
|
||||
const recast = require('recast');
|
||||
const types = recast.types;
|
||||
const n = types.namedTypes;
|
||||
|
||||
for (const statement of statements) {
|
||||
if (n.ExpressionStatement.check(statement) &&
|
||||
n.CallExpression.check(statement.expression) &&
|
||||
n.MemberExpression.check(statement.expression.callee) &&
|
||||
statement.expression.callee.property.name === 'mount' &&
|
||||
statement.expression.arguments.length > 0 &&
|
||||
n.Literal.check(statement.expression.arguments[0]) &&
|
||||
statement.expression.arguments[0].value === engineName) {
|
||||
return true;
|
||||
}
|
||||
}
|
||||
|
||||
return false;
|
||||
},
|
||||
|
||||
/**
|
||||
* Generate extensions.json manifest in public/
|
||||
*/
|
||||
generateExtensionsManifest(extensions) {
|
||||
console.log('[fleetbase-extensions-generator] Generating extensions manifest...');
|
||||
|
||||
const publicDir = path.join(this.project.root, 'public');
|
||||
|
||||
// Create directory
|
||||
if (!fs.existsSync(publicDir)) {
|
||||
fs.mkdirSync(publicDir, { recursive: true });
|
||||
}
|
||||
|
||||
const manifestPath = path.join(publicDir, 'extensions.json');
|
||||
const manifestContent = JSON.stringify(extensions, null, 2);
|
||||
|
||||
try {
|
||||
fs.writeFileSync(manifestPath, manifestContent, 'utf8');
|
||||
console.log('[fleetbase-extensions-generator] \u2713 Generated public/extensions.json');
|
||||
} catch (error) {
|
||||
console.error('[fleetbase-extensions-generator] ! Failed to write extensions.json:', error.message);
|
||||
}
|
||||
},
|
||||
|
||||
/**
|
||||
* Extract the mount path from an extension package name
|
||||
*/
|
||||
getExtensionMountPath(extensionName) {
|
||||
const segments = extensionName.split('/');
|
||||
let mountName = segments[1] || segments[0];
|
||||
return mountName.replace('-engine', '');
|
||||
}
|
||||
};
|
||||
|
||||
Reference in New Issue
Block a user