@bleedingdev/modern-js-code-tools 3.9.0-ultramodern.23 → 3.9.0-ultramodern.25

package/dist/cjs/cli/microvertical-api-check.cjs

@@ -48,9 +48,9 @@ const flags = [
     '--baseline-package-directory',
     '--rules'
 ];
-async function loadContractRules(file) {
+async function loadSourceRules(file) {
     const rules = (await import((0, external_node_url_namespaceObject.pathToFileURL)(external_node_path_default().resolve(file)).href)).default;
-    if (!Array.isArray(rules) || !rules.every((rule)=>'function' == typeof rule)) throw new Error(`${file}: default export must be an array of contract rule functions, e.g. \`export default [myRule]\``);
+    if (!Array.isArray(rules) || !rules.every((rule)=>'function' == typeof rule)) throw new Error(`${file}: default export must be an array of source rule functions, e.g. \`export default [myRule]\``);
     return rules;
 }
 async function runMicroVerticalApiCheckCli(args = process.argv.slice(2), filesOnly = false) {
@@ -60,7 +60,7 @@ async function runMicroVerticalApiCheckCli(args = process.argv.slice(2), filesOn
     for(let index = 0; index < args.length; index += 1){
         const argument = args[index];
         if ('--help' === argument || '-h' === argument) {
-            console.log(`${filesOnly ? 'modern-api-check-files' : 'modern-api-check'} [--workspace-root <path>] [--baseline-package-directory <path>] [--rules <module>]...\n${filesOnly ? 'Checks consumer files, contracts, package exports and owner identity. Does not analyze runtime topology; run the strict Effect source phase separately.' : 'Checks consumer files, contracts, package exports, owner identity and runtime topology once per API entry.'}\n--rules loads a module whose default export is an array of contract rules; each rule receives the module graph of every API contract.\nWorkspace defaults to ULTRAMODERN_WORKSPACE_ROOT then cwd. Exit codes: 0 valid, 1 consumer violation, 2 tool/configuration failure.`);
+            console.log(`${filesOnly ? 'modern-api-check-files' : 'modern-api-check'} [--workspace-root <path>] [--baseline-package-directory <path>] [--rules <module>]...\n${filesOnly ? 'Checks consumer files, contracts, package exports and owner identity. Does not analyze runtime topology; run the strict Effect source phase separately.' : 'Checks consumer files, contracts, package exports, owner identity and runtime topology once per API entry.'}\n--rules loads a module whose default export is an array of source rules; each rule runs once per workspace source file with the shared module graph.\nWorkspace defaults to ULTRAMODERN_WORKSPACE_ROOT then cwd. Exit codes: 0 valid, 1 consumer violation, 2 tool/configuration failure.`);
             return 0;
         }
         if (!flags.includes(argument ?? '') || !args[index + 1] || args[index + 1]?.startsWith('--')) {
@@ -72,9 +72,9 @@ async function runMicroVerticalApiCheckCli(args = process.argv.slice(2), filesOn
         else if ('--rules' === argument) ruleFiles.push(value);
         else baselinePackageDirectory = value;
     }
-    const contractRules = [];
+    const sourceRules = [];
     for (const file of ruleFiles)try {
-        contractRules.push(...await loadContractRules(file));
+        sourceRules.push(...await loadSourceRules(file));
     } catch (error) {
         console.error(`API tool error: --rules ${error instanceof Error ? error.message : String(error)}`);
         return 2;
@@ -82,7 +82,7 @@ async function runMicroVerticalApiCheckCli(args = process.argv.slice(2), filesOn
     const result = (filesOnly ? external_microvertical_api_boundary_cjs_namespaceObject.checkMicroVerticalApiConsumerFiles : external_microvertical_api_boundary_cjs_namespaceObject.checkMicroVerticalApiBoundaries)({
         workspaceRoot,
         baselinePackageDirectory,
-        contractRules
+        sourceRules
     });
     for (const diagnostic of result.diagnostics)console.error(`API violation: ${diagnostic}`);
     for (const error of result.toolErrors)console.error(`API tool error: ${error}`);

package/dist/cjs/microvertical-api-baseline.cjs

@@ -486,7 +486,13 @@ const configuredMicroVerticalApiStem = (verticalPath, verticals)=>{
 };
 const microVerticalApiBaselineViolation = (stem, filePath, expectation)=>{
     try {
-        const graph = (0, external_module_graph_cjs_namespaceObject.createModuleGraph)(filePath);
+        const modules = (0, external_module_graph_cjs_namespaceObject.createModuleGraph)();
+        const graph = {
+            ...modules,
+            root: modules.module(filePath)
+        };
+        const [mutation] = graph.root.mutations;
+        if (mutation) return `MicroVertical root contract must be valid TypeScript syntax (contract bindings must be immutable at ${mutation.start})`;
         if (!(0, external_microvertical_api_owner_cjs_namespaceObject.baselinePublicIdentityIsExact)(filePath, expectation)) return 'MicroVertical baseline imports must resolve the exact framework owner public export and schema identity';
         return validateParsedContract(graph, stem, expectation);
     } catch (error) {

package/dist/cjs/microvertical-api-boundary.cjs

@@ -455,27 +455,6 @@ function check(options, topology) {
                 const violation = (0, external_microvertical_api_baseline_cjs_namespaceObject.microVerticalApiBaselineViolation)(stem, absolute(contract), expectation);
                 if (violation) diagnostics.push(`${contract}: ${violation}`);
             }
-            if (options.contractRules?.length && exists(contract)) {
-                let graph;
-                try {
-                    graph = (0, external_module_graph_cjs_namespaceObject.createModuleGraph)(absolute(contract));
-                } catch (error) {
-                    if (!(error instanceof external_source_analysis_cjs_namespaceObject.SourceSyntaxError)) throw error;
-                    diagnostics.push(`${contract}: contract rules need a parseable contract with immutable bindings (${error.message})`);
-                }
-                if (graph) {
-                    const context = {
-                        appPath,
-                        protocol: rpc ? 'rpc' : 'rest',
-                        graph
-                    };
-                    for (const rule of options.contractRules)guarded(`${contract} (rule ${rule.name || 'anonymous'})`, ()=>{
-                        const messages = rule(context);
-                        if (!Array.isArray(messages) || !messages.every((message)=>'string' == typeof message)) throw new Error('contract rule must return an array of violation messages (return [] when the contract passes)');
-                        for (const message of messages)diagnostics.push(`${contract}: ${message}`);
-                    });
-                }
-            }
             if (client) {
                 const relativeContract = external_node_path_default().posix.relative(external_node_path_default().posix.dirname(client), contract);
                 moduleShape(client, [
@@ -535,6 +514,27 @@ function check(options, topology) {
                 ]
             ]);
         });
+        const sourceRules = options.sourceRules ?? [];
+        const graph = (0, external_module_graph_cjs_namespaceObject.createModuleGraph)();
+        for (const file of sourceRules.length ? sourceFiles : [])if (/\.[cm]?[jt]sx?$/u.test(file)) guarded(file, ()=>{
+            let module;
+            try {
+                module = graph.module(absolute(file));
+            } catch (error) {
+                if (!(error instanceof external_source_analysis_cjs_namespaceObject.SourceSyntaxError)) throw error;
+                diagnostics.push(`${file}: source rules need parseable source (${error.message})`);
+                return;
+            }
+            for (const rule of sourceRules){
+                const messages = rule({
+                    file,
+                    module,
+                    graph
+                });
+                if (!Array.isArray(messages) || !messages.every((message)=>'string' == typeof message)) throw new Error(`rule ${rule.name || 'anonymous'} must return an array of violation messages (return [] when the file passes)`);
+                for (const message of messages)diagnostics.push(`${file}: ${message}`);
+            }
+        });
         const shell = 'apps/shell-super-app';
         const verticals = apps.filter((app)=>app.path.startsWith('verticals/') && apiApp(app));
         if (exists(shell) && verticals.length) {