@solid-stack/mason 1.0.23 → 1.0.24

package/dist/core/lintEngine.d.ts

@@ -51,5 +51,15 @@ export declare function discoverAllFeatures(projectRoot: string): Array<{
     name: string;
     dir: string;
 }>;
+export interface ClassMethodInfo {
+    name: string;
+    isAsync: boolean;
+    isPublic: boolean;
+    line: number;
+    code: string;
+}
+export declare function extractClassMethods(content: string, className: string): ClassMethodInfo[] | null;
+export declare function checkUseCaseShape(featureName: string, filePath: string, relativePath: string, content: string, addViolation: (v: Violation) => void): void;
+export declare function checkServiceShape(featureName: string, filePath: string, relativePath: string, content: string, addViolation: (v: Violation) => void): void;
 export declare function lintCodebase(options?: LintOptions): LintSummary;
 //# sourceMappingURL=lintEngine.d.ts.map

package/dist/core/lintEngine.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"lintEngine.d.ts","sourceRoot":"","sources":["../../src/core/lintEngine.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EACV,sBAAsB,EAEtB,yBAAyB,EACzB,SAAS,EACT,WAAW,EACX,WAAW,EAEX,SAAS,EACV,MAAM,kBAAkB,CAAC;AAG1B,qBAAa,mBAAoB,SAAQ,KAAK;IAC5C,SAAgB,UAAU,EAAE,SAAS,EAAE,CAAC;IACxC,SAAgB,UAAU,EAAE,OAAO,CAAC;IAEpC,YAAY,OAAO,EAAE,MAAM,EAAE,UAAU,EAAE,SAAS,EAAE,EAAE,UAAU,GAAE,OAAe,EAKhF;CACF;AAED,MAAM,WAAW,UAAU;IACzB,UAAU,EAAE,OAAO,CAAC;IACpB,aAAa,EAAE,OAAO,CAAC;IACvB,aAAa,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IACnC,WAAW,CAAC,EAAE,SAAS,GAAG,SAAS,CAAC;IACpC,eAAe,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IACrC,cAAc,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IACpC,YAAY,CAAC,EAAE,OAAO,GAAG,SAAS,CAAC;IACnC,YAAY,CAAC,EAAE,OAAO,GAAG,SAAS,CAAC;IACnC,kBAAkB,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IACxC,YAAY,CAAC,EAAE,OAAO,GAAG,SAAS,CAAC;IACnC,aAAa,CAAC,EAAE,OAAO,GAAG,SAAS,CAAC;IACpC,aAAa,CAAC,EAAE,OAAO,GAAG,SAAS,CAAC;IACpC,cAAc,CAAC,EAAE,OAAO,GAAG,SAAS,CAAC;CACtC;AAED,wBAAgB,UAAU,CAAC,QAAQ,EAAE,MAAM,EAAE,WAAW,CAAC,EAAE,MAAM,GAAG,OAAO,CAa1E;AAED,wBAAgB,YAAY,CAAC,GAAG,EAAE,MAAM,EAAE,WAAW,CAAC,EAAE,MAAM,GAAG,MAAM,EAAE,CAyBxE;AAED,wBAAgB,YAAY,CAC1B,QAAQ,EAAE,MAAM,EAChB,WAAW,EAAE,MAAM,GAClB;IAAE,OAAO,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IAAC,KAAK,CAAC,EAAE,SAAS,GAAG,SAAS,CAAA;CAAE,CA2BjE;AAED,wBAAgB,cAAc,CAC5B,SAAS,EAAE,MAAM,EACjB,aAAa,EAAE,MAAM,EACrB,WAAW,EAAE,MAAM,GAClB,UAAU,CAmGZ;AAED,MAAM,WAAW,eAAe;IAC9B,SAAS,EAAE,MAAM,CAAC;IAClB,IAAI,EAAE,MAAM,CAAC;IACb,OAAO,EAAE,MAAM,CAAC;CACjB;AAED,wBAAgB,uBAAuB,CAAC,OAAO,EAAE,MAAM,GAAG,eAAe,EAAE,CAsB1E;AAED,qBAAa,sBAAsB;IACjC,OAAO,CAAC,KAAK,CAA6D;IAEnE,OAAO,CAAC,QAAQ,EAAE,yBAAyB,GAAG;QACnD,YAAY,EAAE,OAAO,CAAC;QACtB,aAAa,CAAC,EAAE,yBAAyB,GAAG,SAAS,CAAC;QACtD,UAAU,CAAC,EAAE,MAAM,EAAE,GAAG,SAAS,CAAC;KACnC,CAwBA;IAEM,QAAQ,IAAI,KAAK,CAAC;QAAE,IAAI,EAAE,MAAM,CAAC;QAAC,EAAE,EAAE,MAAM,CAAC;QAAC,QAAQ,EAAE,yBAAyB,CAAA;KAAE,CAAC,CAQ1F;IAEM,gBAAgB,IAAI,sBAAsB,EAAE,CAwBlD;IAED,OAAO,CAAC,aAAa;CAqCtB;AAED,wBAAgB,mBAAmB,CACjC,WAAW,EAAE,MAAM,GAClB,KAAK,CAAC;IAAE,IAAI,EAAE,MAAM,CAAC;IAAC,GAAG,EAAE,MAAM,CAAA;CAAE,CAAC,CAYtC;AAED,wBAAgB,YAAY,CAAC,OAAO,GAAE,WAAgB,GAAG,WAAW,CAgkBnE"}
+{"version":3,"file":"lintEngine.d.ts","sourceRoot":"","sources":["../../src/core/lintEngine.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EACV,sBAAsB,EAEtB,yBAAyB,EACzB,SAAS,EACT,WAAW,EACX,WAAW,EAEX,SAAS,EACV,MAAM,kBAAkB,CAAC;AAG1B,qBAAa,mBAAoB,SAAQ,KAAK;IAC5C,SAAgB,UAAU,EAAE,SAAS,EAAE,CAAC;IACxC,SAAgB,UAAU,EAAE,OAAO,CAAC;IAEpC,YAAY,OAAO,EAAE,MAAM,EAAE,UAAU,EAAE,SAAS,EAAE,EAAE,UAAU,GAAE,OAAe,EAKhF;CACF;AAED,MAAM,WAAW,UAAU;IACzB,UAAU,EAAE,OAAO,CAAC;IACpB,aAAa,EAAE,OAAO,CAAC;IACvB,aAAa,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IACnC,WAAW,CAAC,EAAE,SAAS,GAAG,SAAS,CAAC;IACpC,eAAe,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IACrC,cAAc,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IACpC,YAAY,CAAC,EAAE,OAAO,GAAG,SAAS,CAAC;IACnC,YAAY,CAAC,EAAE,OAAO,GAAG,SAAS,CAAC;IACnC,kBAAkB,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IACxC,YAAY,CAAC,EAAE,OAAO,GAAG,SAAS,CAAC;IACnC,aAAa,CAAC,EAAE,OAAO,GAAG,SAAS,CAAC;IACpC,aAAa,CAAC,EAAE,OAAO,GAAG,SAAS,CAAC;IACpC,cAAc,CAAC,EAAE,OAAO,GAAG,SAAS,CAAC;CACtC;AAED,wBAAgB,UAAU,CAAC,QAAQ,EAAE,MAAM,EAAE,WAAW,CAAC,EAAE,MAAM,GAAG,OAAO,CAa1E;AAED,wBAAgB,YAAY,CAAC,GAAG,EAAE,MAAM,EAAE,WAAW,CAAC,EAAE,MAAM,GAAG,MAAM,EAAE,CAyBxE;AAED,wBAAgB,YAAY,CAC1B,QAAQ,EAAE,MAAM,EAChB,WAAW,EAAE,MAAM,GAClB;IAAE,OAAO,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IAAC,KAAK,CAAC,EAAE,SAAS,GAAG,SAAS,CAAA;CAAE,CA2BjE;AAED,wBAAgB,cAAc,CAC5B,SAAS,EAAE,MAAM,EACjB,aAAa,EAAE,MAAM,EACrB,WAAW,EAAE,MAAM,GAClB,UAAU,CAmGZ;AAED,MAAM,WAAW,eAAe;IAC9B,SAAS,EAAE,MAAM,CAAC;IAClB,IAAI,EAAE,MAAM,CAAC;IACb,OAAO,EAAE,MAAM,CAAC;CACjB;AAED,wBAAgB,uBAAuB,CAAC,OAAO,EAAE,MAAM,GAAG,eAAe,EAAE,CAsB1E;AAED,qBAAa,sBAAsB;IACjC,OAAO,CAAC,KAAK,CAA6D;IAEnE,OAAO,CAAC,QAAQ,EAAE,yBAAyB,GAAG;QACnD,YAAY,EAAE,OAAO,CAAC;QACtB,aAAa,CAAC,EAAE,yBAAyB,GAAG,SAAS,CAAC;QACtD,UAAU,CAAC,EAAE,MAAM,EAAE,GAAG,SAAS,CAAC;KACnC,CAwBA;IAEM,QAAQ,IAAI,KAAK,CAAC;QAAE,IAAI,EAAE,MAAM,CAAC;QAAC,EAAE,EAAE,MAAM,CAAC;QAAC,QAAQ,EAAE,yBAAyB,CAAA;KAAE,CAAC,CAQ1F;IAEM,gBAAgB,IAAI,sBAAsB,EAAE,CAwBlD;IAED,OAAO,CAAC,aAAa;CAqCtB;AAED,wBAAgB,mBAAmB,CACjC,WAAW,EAAE,MAAM,GAClB,KAAK,CAAC;IAAE,IAAI,EAAE,MAAM,CAAC;IAAC,GAAG,EAAE,MAAM,CAAA;CAAE,CAAC,CAYtC;AAED,MAAM,WAAW,eAAe;IAC9B,IAAI,EAAE,MAAM,CAAC;IACb,OAAO,EAAE,OAAO,CAAC;IACjB,QAAQ,EAAE,OAAO,CAAC;IAClB,IAAI,EAAE,MAAM,CAAC;IACb,IAAI,EAAE,MAAM,CAAC;CACd;AAED,wBAAgB,mBAAmB,CAAC,OAAO,EAAE,MAAM,EAAE,SAAS,EAAE,MAAM,GAAG,eAAe,EAAE,GAAG,IAAI,CA2LhG;AAED,wBAAgB,iBAAiB,CAC/B,WAAW,EAAE,MAAM,EACnB,QAAQ,EAAE,MAAM,EAChB,YAAY,EAAE,MAAM,EACpB,OAAO,EAAE,MAAM,EACf,YAAY,EAAE,CAAC,CAAC,EAAE,SAAS,KAAK,IAAI,GACnC,IAAI,CAwGN;AAED,wBAAgB,iBAAiB,CAC/B,WAAW,EAAE,MAAM,EACnB,QAAQ,EAAE,MAAM,EAChB,YAAY,EAAE,MAAM,EACpB,OAAO,EAAE,MAAM,EACf,YAAY,EAAE,CAAC,CAAC,EAAE,SAAS,KAAK,IAAI,GACnC,IAAI,CAwGN;AAED,wBAAgB,YAAY,CAAC,OAAO,GAAE,WAAgB,GAAG,WAAW,CAkmBnE"}

package/dist/index.cjs

@@ -4439,6 +4439,350 @@ function discoverAllFeatures(projectRoot) {
     dir: path2__default.default.join(featuresBase, e.name)
   })).sort((a, b) => a.name.localeCompare(b.name));
 }
+function extractClassMethods(content, className) {
+  let clean = "";
+  let inLineComment = false;
+  let inBlockComment = false;
+  let inString = null;
+  let escaped = false;
+  for (let i = 0; i < content.length; i++) {
+    const c = content[i];
+    const next = content[i + 1];
+    if (inLineComment) {
+      if (c === "\n") {
+        inLineComment = false;
+        clean += "\n";
+      } else {
+        clean += " ";
+      }
+      continue;
+    }
+    if (inBlockComment) {
+      if (c === "*" && next === "/") {
+        inBlockComment = false;
+        clean += "  ";
+        i++;
+      } else {
+        clean += c === "\n" ? "\n" : " ";
+      }
+      continue;
+    }
+    if (inString) {
+      clean += c === "\n" ? "\n" : " ";
+      if (escaped) {
+        escaped = false;
+      } else if (c === "\\") {
+        escaped = true;
+      } else if (c === inString) {
+        inString = null;
+      }
+      continue;
+    }
+    if (c === "/" && next === "/") {
+      inLineComment = true;
+      clean += "  ";
+      i++;
+      continue;
+    }
+    if (c === "/" && next === "*") {
+      inBlockComment = true;
+      clean += "  ";
+      i++;
+      continue;
+    }
+    if (c === '"' || c === "'" || c === "`") {
+      inString = c;
+      clean += " ";
+      continue;
+    }
+    clean += c;
+  }
+  const classDeclRegex = new RegExp(`(?:export\\s+(?:default\\s+)?)?class\\s+${className}\\b[^{]*\\{`);
+  const match = classDeclRegex.exec(clean);
+  if (!match) return null;
+  const classBodyStart = match.index + match[0].length - 1;
+  const methods = [];
+  let depth = 1;
+  let parenDepth = 0;
+  let atMemberStart = true;
+  const methodRegex = /^\s*(?:@[a-zA-Z0-9_$]+(?:\([^)]*\))?\s+)*(?:(public|private|protected|static)\s+)*(?:(async)\s+)?(?:(get|set)\s+)?(#?[a-zA-Z0-9_$]+)\s*(?:<[^>]+>)?\s*\(/;
+  const arrowRegex = /^\s*(?:(public|private|protected|static)\s+)?(#?[a-zA-Z0-9_$]+)\s*=\s*(?:async\s*)?\(/;
+  for (let i = classBodyStart + 1; i < clean.length && depth > 0; i++) {
+    const c = clean[i];
+    if (c === "{") {
+      depth++;
+      atMemberStart = false;
+      continue;
+    }
+    if (c === "}") {
+      depth--;
+      if (depth === 1) {
+        atMemberStart = true;
+      }
+      continue;
+    }
+    if (depth === 1) {
+      if (c === ";") {
+        atMemberStart = true;
+        parenDepth = 0;
+        continue;
+      }
+      if (c === "(") {
+        parenDepth++;
+      } else if (c === ")") {
+        parenDepth = Math.max(0, parenDepth - 1);
+      }
+      if (atMemberStart && parenDepth === 0) {
+        if (/\s/.test(c)) continue;
+        const slice = clean.slice(i);
+        const methodMatch = slice.match(methodRegex);
+        if (methodMatch) {
+          const fullMatch = methodMatch[0];
+          const beforeParen = fullMatch.slice(0, fullMatch.lastIndexOf("(")).trim();
+          const tokens = beforeParen.split(/\s+/).filter(Boolean);
+          const rawName = tokens[tokens.length - 1] || "";
+          const methodName = rawName.replace(/<.*$/, "");
+          const controlKeywords = ["if", "for", "while", "switch", "catch", "with"];
+          if (!controlKeywords.includes(methodName)) {
+            const isPrivate = tokens.includes("private") || tokens.includes("protected") || methodName.startsWith("#");
+            const isAsync = tokens.includes("async");
+            const isPublic = !isPrivate;
+            const lineNum = content.slice(0, i).split("\n").length;
+            const lineCode = content.split("\n")[lineNum - 1]?.trim() || "";
+            methods.push({
+              name: methodName,
+              isAsync,
+              isPublic,
+              line: lineNum,
+              code: lineCode
+            });
+            i += fullMatch.length - 1;
+            parenDepth = 1;
+            atMemberStart = false;
+            continue;
+          }
+        }
+        const arrowMatch = slice.match(arrowRegex);
+        if (arrowMatch) {
+          const fullMatch = arrowMatch[0];
+          const beforeEquals = fullMatch.split("=")[0].trim();
+          const tokens = beforeEquals.split(/\s+/).filter(Boolean);
+          const rawName = tokens[tokens.length - 1] || "";
+          const isPrivate = tokens.includes("private") || tokens.includes("protected") || rawName.startsWith("#");
+          const isAsync = fullMatch.includes("async");
+          const isPublic = !isPrivate;
+          const lineNum = content.slice(0, i).split("\n").length;
+          const lineCode = content.split("\n")[lineNum - 1]?.trim() || "";
+          methods.push({
+            name: rawName,
+            isAsync,
+            isPublic,
+            line: lineNum,
+            code: lineCode
+          });
+          i += fullMatch.length - 1;
+          parenDepth = 1;
+          atMemberStart = false;
+          continue;
+        }
+        atMemberStart = false;
+      }
+    }
+  }
+  return methods;
+}
+function checkUseCaseShape(featureName, filePath, relativePath, content, addViolation) {
+  const fileName = path2__default.default.basename(filePath);
+  const lines = content.split(/\r?\n/);
+  const classMatch = content.match(/(?:export\s+(?:default\s+)?)?class\s+([A-Za-z0-9_]+)/);
+  if (!classMatch) {
+    addViolation({
+      feature: featureName,
+      layer: "useCases",
+      file: filePath,
+      relativePath,
+      line: 1,
+      category: "Use Case Shape Violation",
+      code: lines[0]?.trim() || "",
+      ruleId: "U_SHAPE_CLASS",
+      severity: "error",
+      advice: `Use case file '${fileName}' must define and export a class.`,
+      fixExample: {
+        bad: "// Missing class definition",
+        good: `export class ${fileName.replace(/\.(ts|js)$/, "")} { ... }`
+      }
+    });
+    return;
+  }
+  const className = classMatch[1];
+  let classLine = 1;
+  for (let i = 0; i < lines.length; i++) {
+    if (new RegExp(`(?:export\\s+(?:default\\s+)?)?class\\s+${className}\\b`).test(lines[i])) {
+      classLine = i + 1;
+      break;
+    }
+  }
+  if (!/@MakeInjectable\b/.test(content)) {
+    addViolation({
+      feature: featureName,
+      layer: "useCases",
+      file: filePath,
+      relativePath,
+      line: classLine,
+      category: "Use Case Shape Violation",
+      code: lines[classLine - 1]?.trim() || `class ${className}`,
+      ruleId: "U_SHAPE_DECORATOR",
+      severity: "error",
+      advice: `Use case class '${className}' in '${fileName}' must be decorated with @MakeInjectable.`,
+      fixExample: {
+        bad: lines[classLine - 1]?.trim() || `class ${className}`,
+        good: `@MakeInjectable
+export class ${className} { ... }`
+      }
+    });
+  }
+  const hasAsyncExecute = /(?:public\s+)?async\s+execute\s*(?:<[^>]+>)?\s*\(/.test(content);
+  if (!hasAsyncExecute) {
+    const hasSyncExecute = /(?:public\s+)?execute\s*(?:<[^>]+>)?\s*\(/.test(content);
+    addViolation({
+      feature: featureName,
+      layer: "useCases",
+      file: filePath,
+      relativePath,
+      line: classLine,
+      category: "Use Case Shape Violation",
+      code: lines[classLine - 1]?.trim() || `class ${className}`,
+      ruleId: "U_SHAPE_EXECUTE",
+      severity: "error",
+      advice: hasSyncExecute ? `Use case class '${className}' in '${fileName}' must have an 'async execute' method (found synchronous 'execute').` : `Use case class '${className}' in '${fileName}' must implement an 'async execute(...)' method.`,
+      fixExample: {
+        bad: hasSyncExecute ? "execute(props: ...) { ... }" : `class ${className} { ... }`,
+        good: "async execute(props: ...): Promise<...> { ... }"
+      }
+    });
+  }
+  const classMethods = extractClassMethods(content, className);
+  if (classMethods) {
+    const extraPublicMethods = classMethods.filter(
+      (m) => m.isPublic && m.name !== "constructor" && m.name !== "execute"
+    );
+    for (const m of extraPublicMethods) {
+      addViolation({
+        feature: featureName,
+        layer: "useCases",
+        file: filePath,
+        relativePath,
+        line: m.line,
+        category: "Use Case Shape Violation",
+        code: m.code || `public ${m.name}(...)`,
+        ruleId: "U_SHAPE_ONLY_EXECUTE",
+        severity: "error",
+        advice: `Use case class '${className}' in '${fileName}' has disallowed public method '${m.name}'. The only allowed public methods are constructor and async execute. Extract extra methods into their own use cases, or make them private.`,
+        fixExample: {
+          bad: m.code || `${m.name}(...) { ... }`,
+          good: `// Extract '${m.name}' into its own dedicated Use Case, or make it private if it is internal helper logic:
+private ${m.name}(...) { ... }`
+        }
+      });
+    }
+  }
+}
+function checkServiceShape(featureName, filePath, relativePath, content, addViolation) {
+  const fileName = path2__default.default.basename(filePath);
+  const lines = content.split(/\r?\n/);
+  const classMatch = content.match(/(?:export\s+(?:default\s+)?)?class\s+([A-Za-z0-9_]+)/);
+  if (!classMatch) {
+    addViolation({
+      feature: featureName,
+      layer: "services",
+      file: filePath,
+      relativePath,
+      line: 1,
+      category: "Service Shape Violation",
+      code: lines[0]?.trim() || "",
+      ruleId: "S_SHAPE_CLASS",
+      severity: "error",
+      advice: `Service file '${fileName}' must define and export a class.`,
+      fixExample: {
+        bad: "// Missing class definition",
+        good: `export class ${fileName.replace(/\.(ts|js)$/, "")} { ... }`
+      }
+    });
+    return;
+  }
+  const className = classMatch[1];
+  let classLine = 1;
+  for (let i = 0; i < lines.length; i++) {
+    if (new RegExp(`(?:export\\s+(?:default\\s+)?)?class\\s+${className}\\b`).test(lines[i])) {
+      classLine = i + 1;
+      break;
+    }
+  }
+  if (!/@MakeInjectable\b/.test(content)) {
+    addViolation({
+      feature: featureName,
+      layer: "services",
+      file: filePath,
+      relativePath,
+      line: classLine,
+      category: "Service Shape Violation",
+      code: lines[classLine - 1]?.trim() || `class ${className}`,
+      ruleId: "S_SHAPE_DECORATOR",
+      severity: "error",
+      advice: `Service class '${className}' in '${fileName}' must be decorated with @MakeInjectable.`,
+      fixExample: {
+        bad: lines[classLine - 1]?.trim() || `class ${className}`,
+        good: `@MakeInjectable
+export class ${className} { ... }`
+      }
+    });
+  }
+  const hasAsyncExecute = /(?:public\s+)?async\s+execute\s*(?:<[^>]+>)?\s*\(/.test(content);
+  if (!hasAsyncExecute) {
+    const hasSyncExecute = /(?:public\s+)?execute\s*(?:<[^>]+>)?\s*\(/.test(content);
+    addViolation({
+      feature: featureName,
+      layer: "services",
+      file: filePath,
+      relativePath,
+      line: classLine,
+      category: "Service Shape Violation",
+      code: lines[classLine - 1]?.trim() || `class ${className}`,
+      ruleId: "S_SHAPE_EXECUTE",
+      severity: "error",
+      advice: hasSyncExecute ? `Service class '${className}' in '${fileName}' must have an 'async execute' method (found synchronous 'execute').` : `Service class '${className}' in '${fileName}' must implement an 'async execute(...)' method.`,
+      fixExample: {
+        bad: hasSyncExecute ? "execute(props: ...) { ... }" : `class ${className} { ... }`,
+        good: "async execute(props: ...): Promise<...> { ... }"
+      }
+    });
+  }
+  const classMethods = extractClassMethods(content, className);
+  if (classMethods) {
+    const extraPublicMethods = classMethods.filter(
+      (m) => m.isPublic && m.name !== "constructor" && m.name !== "execute"
+    );
+    for (const m of extraPublicMethods) {
+      addViolation({
+        feature: featureName,
+        layer: "services",
+        file: filePath,
+        relativePath,
+        line: m.line,
+        category: "Service Shape Violation",
+        code: m.code || `public ${m.name}(...)`,
+        ruleId: "S_SHAPE_ONLY_EXECUTE",
+        severity: "error",
+        advice: `Service class '${className}' in '${fileName}' has disallowed public method '${m.name}'. The only allowed public methods are constructor and async execute. If a service has multiple public methods, those should be on their own services instead of being cramped in a single service.`,
+        fixExample: {
+          bad: m.code || `${m.name}(...) { ... }`,
+          good: `// Extract '${m.name}' into its own dedicated Service, or make it private if it is internal helper logic:
+private ${m.name}(...) { ... }`
+        }
+      });
+    }
+  }
+}
 function lintCodebase(options = {}) {
   const startTime = Date.now();
   const projectRoot = options.projectRoot || findProjectRoot(options.targetDir || process.cwd());
@@ -4482,6 +4826,14 @@ Code: ${v.code}`;
       if (!layer) continue;
       const content = fs4__default.default.readFileSync(file, "utf-8");
       const statements = extractImportStatements(content);
+      const baseName = path2__default.default.basename(file);
+      const isBarrel = baseName === "index.ts" || baseName === "index.js";
+      if (layer === "useCases" && !isBarrel) {
+        checkUseCaseShape(feat.name, file, relPath, content, addViolation);
+      }
+      if (layer === "services" && !isBarrel) {
+        checkServiceShape(feat.name, file, relPath, content, addViolation);
+      }
       if (layer === "domain") {
         const lines = content.split(/\r?\n/);
         for (let i = 0; i < lines.length; i++) {
@@ -4892,6 +5244,24 @@ Code: ${v.code}`;
               }
             });
           }
+          if (target.targetLayer === "services") {
+            addViolation({
+              feature: feat.name,
+              layer,
+              file,
+              relativePath: relPath,
+              line: stmt.line,
+              category: "Presentation Layer Violation",
+              code: stmt.rawCode,
+              ruleId: "P4_DIRECT_SERVICE",
+              severity: "error",
+              advice: "Presentation handlers must not depend on services directly. They must use use cases, and use cases will then depend on the service.",
+              fixExample: {
+                bad: stmt.rawCode,
+                good: "// Inject a Use Case via static deps instead of a Service."
+              }
+            });
+          }
         }
       }
     }