zemdomu 1.3.8 → 1.3.10

package/out/component-analyzer.js

@@ -318,7 +318,7 @@ class ComponentAnalyzer {
         return results;
     }
     findCrossComponentH1Issues(results) {
-        var _a;
+        var _a, _b;
         const entryPoints = this.findEntryPoints();
         const emitted = new Set();
         const getDisplayName = (component) => {
@@ -326,35 +326,92 @@ class ComponentAnalyzer {
                 return component.name;
             return path.basename(component.filePath, path.extname(component.filePath));
         };
+        const addResult = (result) => {
+            const key = `${result.rule}|${result.filePath}|${result.line}|${result.column}|${result.message}`;
+            if (emitted.has(key))
+                return;
+            emitted.add(key);
+            results.push(result);
+        };
         for (const entry of entryPoints) {
             const comps = this.findComponentsWithRule(entry, 'singleH1', 0);
             if (comps.length <= 1)
                 continue;
+            const conflictMap = new Map();
             for (const comp of comps) {
-                if (!comp)
+                const conflicts = comps
+                    .filter(other => other.filePath !== comp.filePath)
+                    .map(getDisplayName);
+                if (conflicts.length)
+                    conflictMap.set(comp.filePath, conflicts);
+            }
+            const usageMap = new Map();
+            const usageStack = new Set();
+            const collectUsage = (component, depth = 0) => {
+                if (this.maxDepth !== undefined && depth > this.maxDepth)
+                    return;
+                if (usageStack.has(component.filePath))
+                    return;
+                usageStack.add(component.filePath);
+                for (const ref of component.usesComponents) {
+                    if (!ref.path || !this.componentRegistry.has(ref.path))
+                        continue;
+                    const child = this.componentRegistry.get(ref.path);
+                    if (!usageMap.has(child.filePath))
+                        usageMap.set(child.filePath, []);
+                    const locations = ref.usageLocations.length > 0 ? ref.usageLocations : [ref.sourceLocation];
+                    for (const loc of locations) {
+                        usageMap.get(child.filePath).push({
+                            parent: component,
+                            location: { filePath: component.filePath, line: loc.line, column: loc.column },
+                        });
+                    }
+                    collectUsage(child, depth + 1);
+                }
+                usageStack.delete(component.filePath);
+            };
+            collectUsage(entry, 0);
+            for (const comp of comps) {
+                const conflicts = conflictMap.get(comp.filePath);
+                if (!conflicts || !conflicts.length)
                     continue;
                 const compName = getDisplayName(comp);
                 const issues = (_a = comp.issues.get('singleH1')) !== null && _a !== void 0 ? _a : [];
                 if (!issues.length)
                     continue;
-                const conflictingNames = comps
-                    .filter(other => other.filePath !== comp.filePath)
-                    .map(getDisplayName);
-                if (!conflictingNames.length)
-                    continue;
-                const conflicts = conflictingNames.map(name => `'${name}'`).join(', ');
-                const context = `This <h1> in '${compName}' conflicts with ${conflicts}.`;
+                const conflictText = conflicts.map(name => `'${name}'`).join(', ');
+                const usageEntries = (_b = usageMap.get(comp.filePath)) !== null && _b !== void 0 ? _b : [];
+                const usageRelated = usageEntries.map(u => ({
+                    filePath: u.location.filePath,
+                    line: u.location.line,
+                    column: u.location.column,
+                    message: `Rendered via '${getDisplayName(u.parent)}'`,
+                }));
                 for (const issue of issues) {
-                    const key = `${entry.filePath}|${comp.filePath}|${issue.line}|${issue.column}`;
-                    if (emitted.has(key))
-                        continue;
-                    emitted.add(key);
-                    results.push({
+                    addResult({
                         filePath: comp.filePath,
                         line: issue.line,
                         column: issue.column,
-                        message: `Multiple <h1> tags across components. ${context}`,
-                        rule: 'singleH1'
+                        message: `Multiple <h1> tags across components. This <h1> in '${compName}' conflicts with ${conflictText}.`,
+                        rule: 'singleH1',
+                        related: usageRelated,
+                    });
+                }
+                const childIssueLocations = issues.map(issue => ({
+                    filePath: comp.filePath,
+                    line: issue.line,
+                    column: issue.column,
+                    message: `Defined in '${compName}'`,
+                }));
+                for (const usage of usageEntries) {
+                    const parentName = getDisplayName(usage.parent);
+                    addResult({
+                        filePath: usage.location.filePath,
+                        line: usage.location.line,
+                        column: usage.location.column,
+                        message: `Component '${compName}' renders an extra <h1> that conflicts with ${conflictText}.`,
+                        rule: 'singleH1',
+                        related: childIssueLocations.length ? childIssueLocations : undefined,
                     });
                 }
             }

package/out/linter.d.ts

@@ -18,6 +18,12 @@ export interface LintResult {
     rule: string;
     severity?: RuleSeverity;
     filePath?: string;
+    related?: Array<{
+        filePath: string;
+        line: number;
+        column: number;
+        message?: string;
+    }>;
 }
 export interface Rule {
     name: string;

package/out/src/component-analyzer.js

@@ -318,7 +318,7 @@ class ComponentAnalyzer {
         return results;
     }
     findCrossComponentH1Issues(results) {
-        var _a;
+        var _a, _b;
         const entryPoints = this.findEntryPoints();
         const emitted = new Set();
         const getDisplayName = (component) => {
@@ -326,35 +326,92 @@ class ComponentAnalyzer {
                 return component.name;
             return path.basename(component.filePath, path.extname(component.filePath));
         };
+        const addResult = (result) => {
+            const key = `${result.rule}|${result.filePath}|${result.line}|${result.column}|${result.message}`;
+            if (emitted.has(key))
+                return;
+            emitted.add(key);
+            results.push(result);
+        };
         for (const entry of entryPoints) {
             const comps = this.findComponentsWithRule(entry, 'singleH1', 0);
             if (comps.length <= 1)
                 continue;
+            const conflictMap = new Map();
             for (const comp of comps) {
-                if (!comp)
+                const conflicts = comps
+                    .filter(other => other.filePath !== comp.filePath)
+                    .map(getDisplayName);
+                if (conflicts.length)
+                    conflictMap.set(comp.filePath, conflicts);
+            }
+            const usageMap = new Map();
+            const usageStack = new Set();
+            const collectUsage = (component, depth = 0) => {
+                if (this.maxDepth !== undefined && depth > this.maxDepth)
+                    return;
+                if (usageStack.has(component.filePath))
+                    return;
+                usageStack.add(component.filePath);
+                for (const ref of component.usesComponents) {
+                    if (!ref.path || !this.componentRegistry.has(ref.path))
+                        continue;
+                    const child = this.componentRegistry.get(ref.path);
+                    if (!usageMap.has(child.filePath))
+                        usageMap.set(child.filePath, []);
+                    const locations = ref.usageLocations.length > 0 ? ref.usageLocations : [ref.sourceLocation];
+                    for (const loc of locations) {
+                        usageMap.get(child.filePath).push({
+                            parent: component,
+                            location: { filePath: component.filePath, line: loc.line, column: loc.column },
+                        });
+                    }
+                    collectUsage(child, depth + 1);
+                }
+                usageStack.delete(component.filePath);
+            };
+            collectUsage(entry, 0);
+            for (const comp of comps) {
+                const conflicts = conflictMap.get(comp.filePath);
+                if (!conflicts || !conflicts.length)
                     continue;
                 const compName = getDisplayName(comp);
                 const issues = (_a = comp.issues.get('singleH1')) !== null && _a !== void 0 ? _a : [];
                 if (!issues.length)
                     continue;
-                const conflictingNames = comps
-                    .filter(other => other.filePath !== comp.filePath)
-                    .map(getDisplayName);
-                if (!conflictingNames.length)
-                    continue;
-                const conflicts = conflictingNames.map(name => `'${name}'`).join(', ');
-                const context = `This <h1> in '${compName}' conflicts with ${conflicts}.`;
+                const conflictText = conflicts.map(name => `'${name}'`).join(', ');
+                const usageEntries = (_b = usageMap.get(comp.filePath)) !== null && _b !== void 0 ? _b : [];
+                const usageRelated = usageEntries.map(u => ({
+                    filePath: u.location.filePath,
+                    line: u.location.line,
+                    column: u.location.column,
+                    message: `Rendered via '${getDisplayName(u.parent)}'`,
+                }));
                 for (const issue of issues) {
-                    const key = `${entry.filePath}|${comp.filePath}|${issue.line}|${issue.column}`;
-                    if (emitted.has(key))
-                        continue;
-                    emitted.add(key);
-                    results.push({
+                    addResult({
                         filePath: comp.filePath,
                         line: issue.line,
                         column: issue.column,
-                        message: `Multiple <h1> tags across components. ${context}`,
-                        rule: 'singleH1'
+                        message: `Multiple <h1> tags across components. This <h1> in '${compName}' conflicts with ${conflictText}.`,
+                        rule: 'singleH1',
+                        related: usageRelated,
+                    });
+                }
+                const childIssueLocations = issues.map(issue => ({
+                    filePath: comp.filePath,
+                    line: issue.line,
+                    column: issue.column,
+                    message: `Defined in '${compName}'`,
+                }));
+                for (const usage of usageEntries) {
+                    const parentName = getDisplayName(usage.parent);
+                    addResult({
+                        filePath: usage.location.filePath,
+                        line: usage.location.line,
+                        column: usage.location.column,
+                        message: `Component '${compName}' renders an extra <h1> that conflicts with ${conflictText}.`,
+                        rule: 'singleH1',
+                        related: childIssueLocations.length ? childIssueLocations : undefined,
                     });
                 }
             }

package/out/tests/crossComponent/cross-heading-order.test.js

@@ -46,6 +46,7 @@ describe("cross component heading order", () => {
             .map((r) => path_1.default.basename(r.filePath)));
         assert_1.default.ok(singleH1Files.has("Page.tsx"), "Expected cross-component singleH1 to surface on Page.tsx");
         assert_1.default.ok(singleH1Files.has("Button.tsx"), "Expected cross-component singleH1 to surface on Button.tsx");
+        assert_1.default.ok(singleH1Files.has("SubSection.tsx"), "Expected cross-component singleH1 to surface on SubSection.tsx usage");
         const headingLocations = results
             .filter((r) => r.rule === "enforceHeadingOrder" && r.filePath)
             .map((r) => path_1.default.basename(r.filePath));

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "zemdomu",
-  "version": "1.3.8",
+  "version": "1.3.10",
   "description": "Semantic HTML linter for HTML, JSX, and TSX. Detects accessibility, SEO, and structure issues before deployment.",
   "main": "./out/index.js",
   "types": "./out/index.d.ts",