angular-doctor 1.1.3 → 1.3.0

package/dist/cli.mjs

@@ -130,12 +130,26 @@ const detectFramework = (dependencies) => {
 	return "unknown";
 };
 const detectAngularVersion = (dependencies) => dependencies["@angular/core"] ?? null;
+const detectAngularMajorVersion = (version) => {
+	if (!version) return null;
+	const major = parseInt(version.match(/\d+/)?.[0] ?? "", 10);
+	return isNaN(major) ? null : major;
+};
 const detectStandaloneComponents = (packageJson) => {
 	const angularVersion = collectAllDependencies(packageJson)["@angular/core"];
 	if (!angularVersion) return false;
 	const majorVersion = parseInt(angularVersion.match(/\d+/)?.[0] ?? "", 10);
 	return !isNaN(majorVersion) && majorVersion >= 14;
 };
+const detectNgRxPackages = (dependencies) => {
+	return Object.keys(dependencies).some((dep) => dep.startsWith("@ngrx/") || dep === "@ngrx/store");
+};
+const detectAngularMaterial = (dependencies) => {
+	return Object.keys(dependencies).includes("@angular/material");
+};
+const detectSignals = (angularMajorVersion) => {
+	return angularMajorVersion !== null && angularMajorVersion >= 17;
+};
 const countSourceFiles = (rootDirectory) => {
 	const result = spawnSync("git", [
 		"ls-files",
@@ -173,9 +187,13 @@ const discoverProject = (directory) => {
 	const packageJson = readPackageJson(path.join(packageJsonDir, "package.json"));
 	const allDeps = collectAllDependencies(packageJson);
 	const angularVersion = detectAngularVersion(allDeps);
+	const angularMajorVersion = detectAngularMajorVersion(angularVersion);
 	const framework = detectFramework(allDeps);
 	const hasTypeScript = fs.existsSync(path.join(directory, "tsconfig.json")) || fs.existsSync(path.join(packageJsonDir, "tsconfig.json"));
 	const hasStandaloneComponents = detectStandaloneComponents(packageJson);
+	const hasNgRx = detectNgRxPackages(allDeps);
+	const hasAngularMaterial = detectAngularMaterial(allDeps);
+	const hasSignals = detectSignals(angularMajorVersion);
 	const sourceFileCount = countSourceFiles(directory);
 	const angularJsonPath = path.join(packageJsonDir, "angular.json");
 	let projectName = packageJson.name ?? path.basename(directory);
@@ -184,9 +202,13 @@ const discoverProject = (directory) => {
 		rootDirectory: directory,
 		projectName,
 		angularVersion,
+		angularMajorVersion,
 		framework,
 		hasTypeScript,
 		hasStandaloneComponents,
+		hasNgRx,
+		hasAngularMaterial,
+		hasSignals,
 		sourceFileCount
 	};
 };
@@ -369,30 +391,88 @@ const loadConfig = (rootDirectory) => {
 //#region src/utils/run-eslint.ts
 const RULE_CATEGORY_MAP = {
 	"@angular-eslint/component-class-suffix": "Components",
+	"@angular-eslint/component-max-inline-declarations": "Performance",
+	"@angular-eslint/component-selector": "Components",
 	"@angular-eslint/directive-class-suffix": "Components",
+	"@angular-eslint/directive-selector": "Components",
 	"@angular-eslint/pipe-prefix": "Components",
 	"@angular-eslint/use-pipe-transform-interface": "Components",
 	"@angular-eslint/no-empty-lifecycle-method": "Components",
 	"@angular-eslint/use-lifecycle-interface": "Components",
 	"@angular-eslint/consistent-component-styles": "Components",
+	"@angular-eslint/sort-lifecycle-methods": "Components",
+	"@angular-eslint/use-component-selector": "Components",
 	"@angular-eslint/prefer-on-push-component-change-detection": "Performance",
 	"@angular-eslint/no-output-native": "Performance",
+	"@angular-eslint/no-pipe-impure": "Performance",
 	"@angular-eslint/no-conflicting-lifecycle": "Correctness",
 	"@angular-eslint/contextual-lifecycle": "Correctness",
+	"@angular-eslint/contextual-decorator": "Correctness",
+	"@angular-eslint/no-async-lifecycle-method": "Correctness",
+	"@angular-eslint/no-duplicates-in-metadata-arrays": "Correctness",
+	"@angular-eslint/no-lifecycle-call": "Correctness",
+	"@angular-eslint/require-lifecycle-on-prototype": "Correctness",
+	"@angular-eslint/no-attribute-decorator": "Architecture",
 	"@angular-eslint/no-forward-ref": "Architecture",
 	"@angular-eslint/no-input-rename": "Architecture",
 	"@angular-eslint/no-output-rename": "Architecture",
 	"@angular-eslint/no-inputs-metadata-property": "Architecture",
 	"@angular-eslint/no-outputs-metadata-property": "Architecture",
+	"@angular-eslint/no-queries-metadata-property": "Architecture",
 	"@angular-eslint/prefer-standalone": "Architecture",
+	"@angular-eslint/prefer-host-metadata-property": "Architecture",
+	"@angular-eslint/prefer-inject": "Architecture",
+	"@angular-eslint/prefer-output-emitter-ref": "Architecture",
+	"@angular-eslint/prefer-output-readonly": "Architecture",
+	"@angular-eslint/use-component-view-encapsulation": "Architecture",
+	"@angular-eslint/use-injectable-provided-in": "Architecture",
+	"@angular-eslint/no-input-prefix": "Architecture",
+	"@angular-eslint/no-output-on-prefix": "Architecture",
+	"@angular-eslint/relative-url-prefix": "Security",
+	"@angular-eslint/prefer-signals": "Signals",
+	"@angular-eslint/prefer-signal-model": "Signals",
+	"@angular-eslint/no-uncalled-signals": "Signals",
+	"@angular-eslint/template/accessibility": "Accessibility",
+	"@angular-eslint/template/alt-text": "Accessibility",
+	"@angular-eslint/template/click-events-have-key-events": "Accessibility",
+	"@angular-eslint/template/control-events-have-key-events": "Accessibility",
+	"@angular-eslint/template/elements-have-content": "Accessibility",
+	"@angular-eslint/template/interactive-supports-focus": "Accessibility",
+	"@angular-eslint/template/mouse-events-have-key-events": "Accessibility",
+	"@angular-eslint/template/no-any": "Accessibility",
+	"@angular-eslint/template/table-scope": "Accessibility",
+	"@angular-eslint/template/valid-aria": "Accessibility",
+	"@ngrx/contextual-action-creator": "NgRx",
+	"@ngrx/no-cyclic-action-creators": "NgRx",
+	"@ngrx/no-discrete-actions": "NgRx",
+	"@ngrx/no-effect-decorator": "NgRx",
+	"@ngrx/no-effect-decorator-and-creator": "NgRx",
+	"@ngrx/no-multiple-actions-in-effects": "NgRx",
+	"@ngrx/no-reordering-in-effect-reducers": "NgRx",
+	"@ngrx/no-typed-global-store": "NgRx",
+	"@ngrx/on-function-explicit-return-type": "NgRx",
+	"@ngrx/prefix-selectors-with-namespace": "NgRx",
+	"@ngrx/require-middleware-selector": "NgRx",
+	"@ngrx/select-style": "NgRx",
+	"@ngrx/use-consumer-selector": "NgRx",
+	"@angular/material/prefix-selector": "Material",
+	"@angular/material/no-conflicting-mixins": "Material",
 	"@typescript-eslint/no-explicit-any": "TypeScript",
-	"@typescript-eslint/no-unused-vars": "Dead Code"
+	"@typescript-eslint/no-unused-vars": "Dead Code",
+	"@typescript-eslint/sort-keys": "TypeScript"
 };
 const RULE_SEVERITY_MAP = {
 	"@angular-eslint/no-conflicting-lifecycle": "error",
 	"@angular-eslint/contextual-lifecycle": "error",
 	"@angular-eslint/use-pipe-transform-interface": "error",
 	"@angular-eslint/no-output-native": "error",
+	"@angular-eslint/no-async-lifecycle-method": "error",
+	"@angular-eslint/no-lifecycle-call": "error",
+	"@angular-eslint/no-duplicates-in-metadata-arrays": "error",
+	"@angular-eslint/require-lifecycle-on-prototype": "error",
+	"@angular-eslint/relative-url-prefix": "error",
+	"@angular-eslint/contextual-decorator": "error",
+	"@angular-eslint/no-uncalled-signals": "error",
 	"@angular-eslint/component-class-suffix": "warning",
 	"@angular-eslint/directive-class-suffix": "warning",
 	"@angular-eslint/pipe-prefix": "warning",
@@ -406,8 +486,42 @@ const RULE_SEVERITY_MAP = {
 	"@angular-eslint/no-inputs-metadata-property": "warning",
 	"@angular-eslint/no-outputs-metadata-property": "warning",
 	"@angular-eslint/prefer-standalone": "warning",
+	"@angular-eslint/component-selector": "warning",
+	"@angular-eslint/directive-selector": "warning",
+	"@angular-eslint/no-pipe-impure": "warning",
+	"@angular-eslint/no-attribute-decorator": "warning",
+	"@angular-eslint/no-queries-metadata-property": "warning",
+	"@angular-eslint/prefer-host-metadata-property": "warning",
+	"@angular-eslint/prefer-inject": "warning",
+	"@angular-eslint/prefer-output-emitter-ref": "warning",
+	"@angular-eslint/prefer-output-readonly": "warning",
+	"@angular-eslint/use-component-selector": "warning",
+	"@angular-eslint/use-component-view-encapsulation": "warning",
+	"@angular-eslint/use-injectable-provided-in": "warning",
+	"@angular-eslint/component-max-inline-declarations": "warning",
+	"@angular-eslint/sort-lifecycle-methods": "warning",
+	"@angular-eslint/no-input-prefix": "warning",
+	"@angular-eslint/no-output-on-prefix": "warning",
+	"@angular-eslint/prefer-signals": "warning",
+	"@angular-eslint/prefer-signal-model": "warning",
 	"@typescript-eslint/no-explicit-any": "warning",
-	"@typescript-eslint/no-unused-vars": "warning"
+	"@typescript-eslint/no-unused-vars": "warning",
+	"@typescript-eslint/sort-keys": "warning",
+	"@ngrx/contextual-action-creator": "warning",
+	"@ngrx/no-cyclic-action-creators": "error",
+	"@ngrx/no-discrete-actions": "warning",
+	"@ngrx/no-effect-decorator": "warning",
+	"@ngrx/no-effect-decorator-and-creator": "error",
+	"@ngrx/no-multiple-actions-in-effects": "error",
+	"@ngrx/no-reordering-in-effect-reducers": "error",
+	"@ngrx/no-typed-global-store": "error",
+	"@ngrx/on-function-explicit-return-type": "warning",
+	"@ngrx/prefix-selectors-with-namespace": "warning",
+	"@ngrx/require-middleware-selector": "error",
+	"@ngrx/select-style": "warning",
+	"@ngrx/use-consumer-selector": "warning",
+	"@angular/material/prefix-selector": "warning",
+	"@angular/material/no-conflicting-mixins": "error"
 };
 const RULE_MESSAGE_MAP = {
 	"@angular-eslint/component-class-suffix": "Component class should end with 'Component'",
@@ -421,14 +535,55 @@ const RULE_MESSAGE_MAP = {
 	"@angular-eslint/no-output-native": "Avoid shadowing native DOM events in output names",
 	"@angular-eslint/no-conflicting-lifecycle": "Lifecycle hooks DoCheck and OnChanges cannot be used together",
 	"@angular-eslint/contextual-lifecycle": "Lifecycle hook is not available in this context",
+	"@angular-eslint/contextual-decorator": "Use contextual decorator to specify injection context",
 	"@angular-eslint/no-forward-ref": "Avoid using forwardRef — restructure to avoid circular dependency",
 	"@angular-eslint/no-input-rename": "Avoid renaming directive inputs — use the property name as the binding name",
 	"@angular-eslint/no-output-rename": "Avoid renaming directive outputs — use the property name as the binding name",
 	"@angular-eslint/no-inputs-metadata-property": "Use @Input() decorator instead of inputs metadata property",
 	"@angular-eslint/no-outputs-metadata-property": "Use @Output() decorator instead of outputs metadata property",
 	"@angular-eslint/prefer-standalone": "Prefer standalone components over NgModule-based components",
+	"@angular-eslint/component-selector": "Component selector should follow naming convention",
+	"@angular-eslint/directive-selector": "Directive selector should follow naming convention",
+	"@angular-eslint/no-pipe-impure": "Avoid impure pipes — they run on every change detection cycle",
+	"@angular-eslint/no-async-lifecycle-method": "Avoid async lifecycle methods — use signals instead",
+	"@angular-eslint/no-duplicates-in-metadata-arrays": "Remove duplicate entries in decorator metadata arrays",
+	"@angular-eslint/no-lifecycle-call": "Don't call lifecycle method directly — let Angular call them",
+	"@angular-eslint/require-lifecycle-on-prototype": "Lifecycle methods should be declared on prototype",
+	"@angular-eslint/no-attribute-decorator": "Avoid @Attribute() — use @Input() for property bindings",
+	"@angular-eslint/no-queries-metadata-property": "Use decorator-based queries instead of metadata properties",
+	"@angular-eslint/prefer-host-metadata-property": "Prefer host metadata property over @Host decorator",
+	"@angular-eslint/prefer-inject": "Prefer inject() function over constructor dependency injection",
+	"@angular-eslint/prefer-output-emitter-ref": "Use EventEmitter with Output instead of Subject",
+	"@angular-eslint/prefer-output-readonly": "Mark outputs as readonly when possible",
+	"@angular-eslint/use-component-selector": "Components must have selector for proper encapsulation",
+	"@angular-eslint/use-component-view-encapsulation": "Specify view encapsulation strategy explicitly",
+	"@angular-eslint/use-injectable-provided-in": "Specify providedIn scope for @Injectable()",
+	"@angular-eslint/component-max-inline-declarations": "Too many inline declarations in component — extract to separate files",
+	"@angular-eslint/sort-lifecycle-methods": "Lifecycle methods should be declared in correct order",
+	"@angular-eslint/no-input-prefix": "Avoid prefix for input property names",
+	"@angular-eslint/no-output-on-prefix": "Avoid 'on' prefix for output event names",
+	"@angular-eslint/relative-url-prefix": "Use relative URL prefixes for better security",
+	"@angular-eslint/prefer-signals": "Prefer Angular signals over other reactive patterns",
+	"@angular-eslint/prefer-signal-model": "Prefer signal-based model for component state",
+	"@angular-eslint/no-uncalled-signals": "Signal getters must be called to access value",
 	"@typescript-eslint/no-explicit-any": "Avoid 'any' type — use specific types for better type safety",
-	"@typescript-eslint/no-unused-vars": "Remove unused variable declaration"
+	"@typescript-eslint/no-unused-vars": "Remove unused variable declaration",
+	"@typescript-eslint/sort-keys": "Sort object keys consistently",
+	"@ngrx/contextual-action-creator": "Use contextual action creators for typed actions",
+	"@ngrx/no-cyclic-action-creators": "Action creators should not reference each other cyclically",
+	"@ngrx/no-discrete-actions": "Use discrete actions instead of broad action types",
+	"@ngrx/no-effect-decorator": "Consider using functional effects instead of @Effect decorator",
+	"@ngrx/no-effect-decorator-and-creator": "Don't use both @Effect decorator and createEffect function",
+	"@ngrx/no-multiple-actions-in-effects": "Effects should dispatch a single action or none",
+	"@ngrx/no-reordering-in-effect-reducers": "Don't reorder actions in effect reducers",
+	"@ngrx/no-typed-global-store": "Use typed GlobalStore for better type safety",
+	"@ngrx/on-function-explicit-return-type": "Specify explicit return type for on() reducer functions",
+	"@ngrx/prefix-selectors-with-namespace": "Prefix selectors with feature namespace",
+	"@ngrx/require-middleware-selector": "Middleware must have selector for proper scoping",
+	"@ngrx/select-style": "Prefer selector functions over props in select",
+	"@ngrx/use-consumer-selector": "Use useSelector with selector function for proper memoization",
+	"@angular/material/prefix-selector": "Material components should use proper selector prefix",
+	"@angular/material/no-conflicting-mixins": "Avoid conflicting mixins in Material components"
 };
 const RULE_HELP_MAP = {
 	"@angular-eslint/component-class-suffix": "Add 'Component' suffix: `export class UserProfileComponent { }`",
@@ -445,9 +600,31 @@ const RULE_HELP_MAP = {
 	"@angular-eslint/no-outputs-metadata-property": "Use `@Output() myEvent = new EventEmitter()` instead of `outputs: ['myEvent']` in the decorator metadata",
 	"@angular-eslint/prefer-standalone": "Add `standalone: true` to component: `@Component({ standalone: true, ... })`",
 	"@typescript-eslint/no-explicit-any": "Replace `any` with a specific type or `unknown` if the type is truly unknown",
-	"@typescript-eslint/no-unused-vars": "Remove the unused variable or prefix with `_` to indicate it's intentionally unused"
+	"@typescript-eslint/no-unused-vars": "Remove the unused variable or prefix with `_` to indicate it's intentionally unused",
+	"@typescript-eslint/sort-keys": "Sort object keys alphabetically or by a consistent pattern",
+	"@angular-eslint/prefer-inject": "Use `inject(MyService)` instead of constructor injection: `constructor(private myService: MyService) {}`",
+	"@angular-eslint/no-pipe-impure": "Remove `pure: false` from pipe decorator or refactor to use a service",
+	"@angular-eslint/prefer-signals": "Replace observables with signals for simpler reactive state: `count = signal(0)`",
+	"@angular-eslint/prefer-signal-model": "Use signal-based input/output model: `count = model(0)` instead of `@Input() count: number`",
+	"@angular-eslint/no-uncalled-signals": "Call the signal getter to access its value: `this.count()` not `this.count`",
+	"@angular-eslint/component-max-inline-declarations": "Move templates/styles to separate files or use inline with caution — consider extracting when > 3 inline declarations",
+	"@angular-eslint/relative-url-prefix": "Use relative URLs (no leading slash) or ensure absolute URLs are intentional for security",
+	"@ngrx/contextual-action-creator": "Use `createActionGroup` or `createAction` with props for type-safe actions",
+	"@ngrx/no-multiple-actions-in-effects": "Split effect into multiple effects or use `mergeMap` with individual actions"
+};
+const detectPackagePresence = (packageJson) => {
+	const deps = {
+		...packageJson.dependencies,
+		...packageJson.devDependencies,
+		...packageJson.peerDependencies
+	};
+	return {
+		hasNgRx: Object.keys(deps).some((dep) => dep.startsWith("@ngrx/") && !dep.includes("store")) || Object.keys(deps).includes("@ngrx/store"),
+		hasAngularMaterial: Object.keys(deps).includes("@angular/material"),
+		hasSignals: true
+	};
 };
-const buildEslintConfig = (hasTypeScript, tsconfigPath, useTypeAware) => {
+const buildEslintConfig = (hasTypeScript, tsconfigPath, useTypeAware, packagePresence) => {
 	const languageOptions = {
 		parser: tsEslint.parser,
 		parserOptions: {
@@ -459,20 +636,73 @@ const buildEslintConfig = (hasTypeScript, tsconfigPath, useTypeAware) => {
 	const angularRules = {
 		"@angular-eslint/component-class-suffix": "warn",
 		"@angular-eslint/directive-class-suffix": "warn",
+		"@angular-eslint/pipe-prefix": "warn",
 		"@angular-eslint/no-empty-lifecycle-method": "warn",
 		"@angular-eslint/use-lifecycle-interface": "warn",
-		"@angular-eslint/use-pipe-transform-interface": "error",
+		"@angular-eslint/consistent-component-styles": "warn",
+		"@angular-eslint/sort-lifecycle-methods": "warn",
+		"@angular-eslint/use-component-selector": "warn",
+		"@angular-eslint/component-selector": "warn",
+		"@angular-eslint/directive-selector": "warn",
 		"@angular-eslint/prefer-on-push-component-change-detection": "warn",
 		"@angular-eslint/no-output-native": "error",
+		"@angular-eslint/no-pipe-impure": "warn",
+		"@angular-eslint/component-max-inline-declarations": "warn",
+		"@angular-eslint/use-pipe-transform-interface": "error",
 		"@angular-eslint/no-conflicting-lifecycle": "error",
 		"@angular-eslint/contextual-lifecycle": "error",
+		"@angular-eslint/contextual-decorator": "error",
+		"@angular-eslint/no-async-lifecycle-method": "error",
+		"@angular-eslint/no-duplicates-in-metadata-arrays": "error",
+		"@angular-eslint/no-lifecycle-call": "error",
+		"@angular-eslint/require-lifecycle-on-prototype": "error",
 		"@angular-eslint/no-forward-ref": "warn",
 		"@angular-eslint/no-input-rename": "warn",
 		"@angular-eslint/no-output-rename": "warn",
 		"@angular-eslint/no-inputs-metadata-property": "warn",
-		"@angular-eslint/no-outputs-metadata-property": "warn"
+		"@angular-eslint/no-outputs-metadata-property": "warn",
+		"@angular-eslint/no-queries-metadata-property": "warn",
+		"@angular-eslint/prefer-standalone": "warn",
+		"@angular-eslint/prefer-host-metadata-property": "warn",
+		"@angular-eslint/prefer-inject": "warn",
+		"@angular-eslint/prefer-output-emitter-ref": "warn",
+		"@angular-eslint/prefer-output-readonly": "warn",
+		"@angular-eslint/use-component-view-encapsulation": "warn",
+		"@angular-eslint/use-injectable-provided-in": "warn",
+		"@angular-eslint/no-attribute-decorator": "warn",
+		"@angular-eslint/no-input-prefix": "warn",
+		"@angular-eslint/no-output-on-prefix": "warn",
+		"@angular-eslint/relative-url-prefix": "error"
+	};
+	const signalsRules = packagePresence.hasSignals ? {
+		"@angular-eslint/prefer-signals": "warn",
+		"@angular-eslint/prefer-signal-model": "warn",
+		"@angular-eslint/no-uncalled-signals": "error"
+	} : {};
+	const tsRules = {
+		"@typescript-eslint/no-explicit-any": "warn",
+		"@typescript-eslint/no-unused-vars": "warn",
+		"@typescript-eslint/sort-keys": "warn"
 	};
-	const tsRules = { "@typescript-eslint/no-explicit-any": "warn" };
+	const ngrxRules = packagePresence.hasNgRx ? {
+		"@ngrx/contextual-action-creator": "warn",
+		"@ngrx/no-cyclic-action-creators": "error",
+		"@ngrx/no-discrete-actions": "warn",
+		"@ngrx/no-effect-decorator": "warn",
+		"@ngrx/no-effect-decorator-and-creator": "error",
+		"@ngrx/no-multiple-actions-in-effects": "error",
+		"@ngrx/no-reordering-in-effect-reducers": "error",
+		"@ngrx/no-typed-global-store": "error",
+		"@ngrx/on-function-explicit-return-type": "warn",
+		"@ngrx/prefix-selectors-with-namespace": "warn",
+		"@ngrx/require-middleware-selector": "error",
+		"@ngrx/select-style": "warn",
+		"@ngrx/use-consumer-selector": "warn"
+	} : {};
+	const materialRules = packagePresence.hasAngularMaterial ? {
+		"@angular/material/prefix-selector": "warn",
+		"@angular/material/no-conflicting-mixins": "error"
+	} : {};
 	return [{
 		files: ["**/*.ts"],
 		plugins: {
@@ -482,7 +712,10 @@ const buildEslintConfig = (hasTypeScript, tsconfigPath, useTypeAware) => {
 		languageOptions,
 		rules: {
 			...angularRules,
-			...tsRules
+			...tsRules,
+			...signalsRules,
+			...ngrxRules,
+			...materialRules
 		}
 	}];
 };
@@ -505,24 +738,70 @@ const parsePluginAndRule = (ruleId) => {
 	};
 };
 const runEslint = async (rootDirectory, hasTypeScript, includePaths, options) => {
-	if (includePaths !== void 0 && includePaths.length === 0) return [];
+	if (includePaths !== void 0 && includePaths.length === 0) return {
+		diagnostics: [],
+		errors: []
+	};
 	const tsconfigPath = hasTypeScript ? path.join(rootDirectory, "tsconfig.json") : null;
+	let packagePresence;
+	if (options?.frameworkInfo) packagePresence = {
+		hasNgRx: options.frameworkInfo.hasNgRx,
+		hasAngularMaterial: options.frameworkInfo.hasAngularMaterial,
+		hasSignals: options.frameworkInfo.hasSignals
+	};
+	else {
+		packagePresence = {
+			hasNgRx: false,
+			hasAngularMaterial: false,
+			hasSignals: true
+		};
+		try {
+			const packageJsonPath = path.join(rootDirectory, "package.json");
+			if (fs.existsSync(packageJsonPath)) {
+				const packageJsonContent = fs.readFileSync(packageJsonPath, "utf-8");
+				packagePresence = detectPackagePresence(JSON.parse(packageJsonContent));
+			}
+		} catch {}
+	}
 	const cacheRoot = path.join(rootDirectory, "node_modules", ".cache", "angular-doctor");
 	fs.mkdirSync(cacheRoot, { recursive: true });
 	const eslint = new ESLint({
 		cwd: rootDirectory,
 		overrideConfigFile: null,
-		overrideConfig: buildEslintConfig(hasTypeScript, tsconfigPath && fs.existsSync(tsconfigPath) ? tsconfigPath : null, options?.useTypeAware ?? true),
+		overrideConfig: buildEslintConfig(hasTypeScript, tsconfigPath && fs.existsSync(tsconfigPath) ? tsconfigPath : null, options?.useTypeAware ?? true, packagePresence),
 		ignore: true,
 		cache: true,
 		cacheLocation: path.join(cacheRoot, ".eslintcache")
 	});
 	const patterns = includePaths ?? ["**/*.ts"];
 	let results;
+	const errors = [];
 	try {
 		results = await eslint.lintFiles(patterns);
-	} catch {
-		return [];
+	} catch (error) {
+		const errorMessage = error instanceof Error ? error.message : String(error);
+		const errorStack = error instanceof Error ? error.stack : void 0;
+		const filePathMatch = errorMessage.match(/^(.+?):\s* /);
+		const errorFilePath = filePathMatch ? path.relative(rootDirectory, filePathMatch[1]) : void 0;
+		errors.push({
+			message: errorMessage,
+			stack: errorStack,
+			filePath: errorFilePath
+		});
+		return {
+			diagnostics: [{
+				filePath: errorFilePath ?? "unknown",
+				plugin: "eslint",
+				rule: "parse-error",
+				severity: "error",
+				message: errorMessage,
+				help: "Fix the syntax error in this file. ESLint could not parse the file.",
+				line: 0,
+				column: 0,
+				category: "Parse Error"
+			}],
+			errors
+		};
 	}
 	const diagnostics = [];
 	for (const result of results) for (const message of result.messages) {
@@ -541,7 +820,10 @@ const runEslint = async (rootDirectory, hasTypeScript, includePaths, options) =>
 			category: resolveDiagnosticCategory(ruleKey)
 		});
 	}
-	return diagnostics;
+	return {
+		diagnostics,
+		errors
+	};
 };
 
 //#endregion
@@ -696,24 +978,106 @@ const buildFileLineMap = (diagnostics) => {
 	}
 	return fileLines;
 };
+/**
+* Format file:line:column location string
+*/
+const formatLocation = (diagnostic) => {
+	const { filePath, line, column } = diagnostic;
+	return `${filePath}:${line}${column > 0 ? `:${column}` : ""}`;
+};
+/**
+* Print summary breakdown showing:
+* - Count by severity
+* - Count by category
+* - Top rules by frequency
+*/
+const printSummaryBreakdown = (diagnostics) => {
+	const errorCount = diagnostics.filter((d) => d.severity === "error").length;
+	const warningCount = diagnostics.filter((d) => d.severity === "warning").length;
+	const sortedCategories = [...groupBy(diagnostics, (d) => d.category).entries()].sort(([, a], [, b]) => b.length - a.length);
+	const sortedRules = [...groupBy(diagnostics, (d) => `${d.plugin}/${d.rule}`).entries()].sort(([, a], [, b]) => b.length - a.length).slice(0, 5);
+	logger.break();
+	logger.log("  Summary Breakdown");
+	logger.log("  ─────────────────");
+	const severityParts = [];
+	if (errorCount > 0) severityParts.push(highlighter.error(`✗ ${errorCount} error${errorCount === 1 ? "" : "s"}`));
+	if (warningCount > 0) severityParts.push(highlighter.warn(`⚠ ${warningCount} warning${warningCount === 1 ? "" : "s"}`));
+	if (severityParts.length > 0) {
+		logger.log(`  ${severityParts.join(", ")}`);
+		logger.break();
+	}
+	if (sortedCategories.length > 0) {
+		logger.dim("  Categories:");
+		for (const [category, categoryDiags] of sortedCategories) {
+			const count = categoryDiags.length;
+			logger.dim(`    ${category}: ${count}`);
+		}
+		logger.break();
+	}
+	if (sortedRules.length > 0) {
+		logger.dim("  Top rules:");
+		for (const [rule, ruleDiags] of sortedRules) {
+			const count = ruleDiags.length;
+			const severity = ruleDiags[0].severity === "error" ? highlighter.error("✗") : highlighter.warn("⚠");
+			logger.dim(`    ${severity} ${rule}: ${count}`);
+		}
+		logger.break();
+	}
+};
 const printDiagnostics = (diagnostics, isVerbose) => {
 	const sortedRuleGroups = sortBySeverity([...groupBy(diagnostics, (diagnostic) => `${diagnostic.plugin}/${diagnostic.rule}`).entries()]);
-	for (const [, ruleDiagnostics] of sortedRuleGroups) {
+	const errorGroups = sortedRuleGroups.filter(([, diags]) => diags[0]?.severity === "error");
+	const warningGroups = sortedRuleGroups.filter(([, diags]) => diags[0]?.severity === "warning");
+	if (errorGroups.length > 0) {
+		logger.error(`  ${errorGroups.length} error${errorGroups.length === 1 ? "" : "s"}:`);
+		logger.log(highlighter.error("  ─────────────────────────────────────────"));
+	}
+	for (const [ruleKey, ruleDiagnostics] of errorGroups) {
 		const firstDiagnostic = ruleDiagnostics[0];
-		const icon = colorizeBySeverity(firstDiagnostic.severity === "error" ? "✗" : "⚠", firstDiagnostic.severity);
+		const icon = colorizeBySeverity("✗", firstDiagnostic.severity);
 		const count = ruleDiagnostics.length;
 		const countLabel = count > 1 ? colorizeBySeverity(` (${count})`, firstDiagnostic.severity) : "";
-		logger.log(`  ${icon} ${firstDiagnostic.message}${countLabel}`);
+		logger.log("  ┌────────────────────────────────────────────────");
+		logger.log(`  │ ${icon} ${firstDiagnostic.message}${countLabel}`);
 		if (firstDiagnostic.help) logger.dim(indentMultilineText(firstDiagnostic.help, "    "));
 		if (isVerbose) {
-			const fileLines = buildFileLineMap(ruleDiagnostics);
-			for (const [filePath, lines] of fileLines) {
-				const lineLabel = lines.length > 0 ? `: ${lines.join(", ")}` : "";
-				logger.dim(`    ${filePath}${lineLabel}`);
+			logger.dim(`    ${highlighter.info("category:")} ${firstDiagnostic.category}`);
+			logger.dim(`    ${highlighter.info("rule:")} ${ruleKey}`);
+			for (const diagnostic of ruleDiagnostics) {
+				const location = formatLocation(diagnostic);
+				logger.dim(`    at ${location}`);
 			}
+			if (ruleDiagnostics.length > 5) logger.dim(`    ... and ${ruleDiagnostics.length - 5} more occurrences`);
 		}
+		logger.log("  └────────────────────────────────────────────────");
 		logger.break();
 	}
+	if (warningGroups.length > 0) {
+		if (errorGroups.length > 0) logger.break();
+		logger.warn(`  ${warningGroups.length} warning${warningGroups.length === 1 ? "" : "s"}:`);
+		logger.log(highlighter.warn("  ─────────────────────────────────────────"));
+	}
+	for (const [ruleKey, ruleDiagnostics] of warningGroups) {
+		const firstDiagnostic = ruleDiagnostics[0];
+		const icon = colorizeBySeverity("⚠", firstDiagnostic.severity);
+		const count = ruleDiagnostics.length;
+		const countLabel = count > 1 ? colorizeBySeverity(` (${count})`, firstDiagnostic.severity) : "";
+		logger.log("  ┌────────────────────────────────────────────────");
+		logger.log(`  │ ${icon} ${firstDiagnostic.message}${countLabel}`);
+		if (firstDiagnostic.help) logger.dim(indentMultilineText(firstDiagnostic.help, "    "));
+		if (isVerbose) {
+			logger.dim(`    ${highlighter.info("category:")} ${firstDiagnostic.category}`);
+			logger.dim(`    ${highlighter.info("rule:")} ${ruleKey}`);
+			for (const diagnostic of ruleDiagnostics) {
+				const location = formatLocation(diagnostic);
+				logger.dim(`    at ${location}`);
+			}
+			if (ruleDiagnostics.length > 5) logger.dim(`    ... and ${ruleDiagnostics.length - 5} more occurrences`);
+		}
+		logger.log("  └────────────────────────────────────────────────");
+		logger.break();
+	}
+	if (isVerbose && diagnostics.length > 0) printSummaryBreakdown(diagnostics);
 };
 const formatElapsedTime = (elapsedMilliseconds) => {
 	if (elapsedMilliseconds < MILLISECONDS_PER_SECOND) return `${Math.round(elapsedMilliseconds)}ms`;
@@ -738,7 +1102,7 @@ const formatRuleSummary = (ruleKey, ruleDiagnostics) => {
 	}
 	return sections.join("\n") + "\n";
 };
-const buildMarkdownReport = (diagnostics, elapsedMilliseconds, scoreResult, totalSourceFileCount) => {
+const buildMarkdownReport = (diagnostics, elapsedMilliseconds, scoreResult, totalSourceFileCount, isDiffMode = false) => {
 	const errorCount = diagnostics.filter((diagnostic) => diagnostic.severity === "error").length;
 	const warningCount = diagnostics.filter((diagnostic) => diagnostic.severity === "warning").length;
 	const affectedFileCount = collectAffectedFiles(diagnostics).size;
@@ -749,8 +1113,11 @@ const buildMarkdownReport = (diagnostics, elapsedMilliseconds, scoreResult, tota
 		`Generated: ${(/* @__PURE__ */ new Date()).toISOString()}`,
 		""
 	];
-	if (scoreResult) lines.push("## Score", "", `**${scoreResult.score} / ${PERFECT_SCORE}** — ${scoreResult.label}`, "");
-	lines.push("## Summary", "", `- Errors: **${errorCount}**`, `- Warnings: **${warningCount}**`, totalSourceFileCount > 0 ? `- Affected files: **${affectedFileCount}/${totalSourceFileCount}**` : `- Affected files: **${affectedFileCount}**`, `- Elapsed: **${elapsed}**`, "");
+	if (scoreResult) {
+		const labelSuffix = isDiffMode ? " (partial)" : "";
+		lines.push("## Score", "", `**${scoreResult.score} / ${PERFECT_SCORE}** — ${scoreResult.label}${labelSuffix}`, "");
+	}
+	lines.push("## Summary", "", `- Errors: **${errorCount}**`, `- Warnings: **${warningCount}**`, totalSourceFileCount > 0 ? `- Affected files: **${affectedFileCount}/${totalSourceFileCount}**` : `- Affected files: **${affectedFileCount}**`, `- Elapsed: **${elapsed}**`, isDiffMode ? `- Mode: **diff (dead code detection skipped)**` : "", "");
 	if (diagnostics.length === 0) {
 		lines.push("## Diagnostics", "", "No issues found.", "");
 		return lines.join("\n");
@@ -781,7 +1148,7 @@ const resolveReportPath = (report, outputDirectory, baseDirectory) => {
 	}
 	return join(outputDirectory, "report.md");
 };
-const writeDiagnosticsDirectory = (diagnostics, elapsedMilliseconds, scoreResult, totalSourceFileCount, report, baseDirectory) => {
+const writeDiagnosticsDirectory = (diagnostics, elapsedMilliseconds, scoreResult, totalSourceFileCount, report, baseDirectory, isDiffMode = false) => {
 	const outputDirectory = join(tmpdir(), `angular-doctor-${randomUUID()}`);
 	mkdirSync(outputDirectory);
 	const sortedRuleGroups = sortBySeverity([...groupBy(diagnostics, (diagnostic) => `${diagnostic.plugin}/${diagnostic.rule}`).entries()]);
@@ -790,7 +1157,7 @@ const writeDiagnosticsDirectory = (diagnostics, elapsedMilliseconds, scoreResult
 	const markdownPath = resolveReportPath(report, outputDirectory, baseDirectory);
 	if (markdownPath) {
 		mkdirSync(dirname(markdownPath), { recursive: true });
-		writeFileSync(markdownPath, buildMarkdownReport(diagnostics, elapsedMilliseconds, scoreResult, totalSourceFileCount));
+		writeFileSync(markdownPath, buildMarkdownReport(diagnostics, elapsedMilliseconds, scoreResult, totalSourceFileCount, isDiffMode));
 	}
 	return {
 		outputDirectory,
@@ -886,10 +1253,10 @@ const buildCountsSummaryLine = (diagnostics, totalSourceFileCount, elapsedMillis
 	renderedParts.push(highlighter.dim(fileCountText), highlighter.dim(elapsedTimeText));
 	return createFramedLine(plainParts.join("  "), renderedParts.join("  "));
 };
-const printSummary = (diagnostics, elapsedMilliseconds, scoreResult, totalSourceFileCount, report, baseDirectory) => {
+const printSummary = (diagnostics, elapsedMilliseconds, scoreResult, totalSourceFileCount, report, baseDirectory, isDiffMode = false) => {
 	printFramedBox([...buildBrandingLines(scoreResult), buildCountsSummaryLine(diagnostics, totalSourceFileCount, elapsedMilliseconds)]);
 	try {
-		const { outputDirectory, markdownPath } = writeDiagnosticsDirectory(diagnostics, elapsedMilliseconds, scoreResult, totalSourceFileCount, report, baseDirectory);
+		const { outputDirectory, markdownPath } = writeDiagnosticsDirectory(diagnostics, elapsedMilliseconds, scoreResult, totalSourceFileCount, report, baseDirectory, isDiffMode);
 		logger.break();
 		logger.dim(`  Full diagnostics written to ${outputDirectory}`);
 		if (markdownPath) logger.dim(`  Markdown report written to ${markdownPath}`);
@@ -906,9 +1273,24 @@ const mergeScanOptions = (inputOptions, userConfig) => {
 		scoreOnly: inputOptions.scoreOnly ?? false,
 		report: inputOptions.report ?? false,
 		useTypeAwareLint: !fastMode,
-		includePaths: inputOptions.includePaths ?? []
+		includePaths: inputOptions.includePaths ?? [],
+		rules: inputOptions.rules
 	};
 };
+/**
+* Parses the --rules CLI flag and returns a FrameworkInfo override.
+* Categories: signals, ngrx, material
+* Example: "signals,ngrx" force-enables both signals and ngrx rules.
+*/
+const parseRulesOverride = (rules) => {
+	if (!rules || rules === "all") return void 0;
+	const override = {};
+	const categories = rules.split(",").map((c) => c.trim().toLowerCase());
+	for (const category of categories) if (category === "signals") override.hasSignals = true;
+	else if (category === "ngrx") override.hasNgRx = true;
+	else if (category === "material") override.hasAngularMaterial = true;
+	return Object.keys(override).length > 0 ? override : void 0;
+};
 const printProjectDetection = (projectInfo, userConfig, isDiffMode, includePaths) => {
 	const frameworkLabel = formatFrameworkName(projectInfo.framework);
 	const languageLabel = "TypeScript";
@@ -940,9 +1322,34 @@ const scan = async (directory, inputOptions = {}) => {
 		if (!options.lint) return [];
 		const lintSpinner = options.scoreOnly ? null : spinner("Running lint checks...").start();
 		try {
-			const lintDiagnostics = await runEslint(directory, projectInfo.hasTypeScript, computedIncludePaths, { useTypeAware: options.useTypeAwareLint });
+			const detectedFrameworkInfo = {
+				hasNgRx: projectInfo.hasNgRx,
+				hasAngularMaterial: projectInfo.hasAngularMaterial,
+				hasSignals: projectInfo.hasSignals
+			};
+			const rulesOverride = parseRulesOverride(options.rules);
+			const frameworkInfo = rulesOverride ? {
+				...detectedFrameworkInfo,
+				...rulesOverride
+			} : detectedFrameworkInfo;
+			const result = await runEslint(directory, projectInfo.hasTypeScript, computedIncludePaths, {
+				useTypeAware: options.useTypeAwareLint,
+				frameworkInfo
+			});
 			lintSpinner?.succeed("Running lint checks.");
-			return lintDiagnostics;
+			if (result.errors.length > 0) {
+				result.errors;
+				if (options.verbose) {
+					logger.break();
+					logger.error("ESLint encountered the following errors:");
+					for (const error of result.errors) {
+						logger.error(`  ${error.message}`);
+						if (error.stack) logger.dim(error.stack);
+					}
+					logger.break();
+				}
+			}
+			return result.diagnostics;
 		} catch (error) {
 			didLintFail = true;
 			lintSpinner?.fail("Lint checks failed (non-fatal, skipping).");
@@ -964,7 +1371,11 @@ const scan = async (directory, inputOptions = {}) => {
 			return [];
 		}
 	};
-	const [lintDiagnostics, deadCodeDiagnostics] = options.scoreOnly ? await Promise.all([runLint(), runDeadCode()]) : [await runLint(), await runDeadCode()];
+	const parallelStartTime = performance.now();
+	const [lintDiagnostics, deadCodeDiagnostics] = await Promise.all([runLint(), runDeadCode()]);
+	const parallelElapsed = performance.now() - parallelStartTime;
+	const sequentialTime = (options.lint ? parallelElapsed * .6 : 0) + (options.deadCode && !isDiffMode ? parallelElapsed * .4 : 0);
+	if (parallelElapsed > 1e3) logger.dim(`  Parallel scan: ${formatElapsedTime(parallelElapsed)} (sequential would be ~${formatElapsedTime(sequentialTime)})`);
 	const diagnostics = combineDiagnostics(lintDiagnostics, deadCodeDiagnostics, userConfig);
 	const elapsedMilliseconds = performance.now() - startTime;
 	const skippedChecks = [];
@@ -972,12 +1383,16 @@ const scan = async (directory, inputOptions = {}) => {
 	if (didDeadCodeFail) skippedChecks.push("dead code");
 	const hasSkippedChecks = skippedChecks.length > 0;
 	const scoreResult = calculateScore(diagnostics);
+	const errorCount = diagnostics.filter((diagnostic) => diagnostic.severity === "error").length;
+	const warningCount = diagnostics.filter((diagnostic) => diagnostic.severity === "warning").length;
 	if (options.scoreOnly) {
 		logger.log(`${scoreResult.score}`);
 		return {
 			diagnostics,
 			scoreResult,
-			skippedChecks
+			skippedChecks,
+			errorCount,
+			warningCount
 		};
 	}
 	if (diagnostics.length === 0) {
@@ -996,11 +1411,13 @@ const scan = async (directory, inputOptions = {}) => {
 		return {
 			diagnostics,
 			scoreResult,
-			skippedChecks
+			skippedChecks,
+			errorCount,
+			warningCount
 		};
 	}
 	printDiagnostics(diagnostics, options.verbose);
-	printSummary(diagnostics, elapsedMilliseconds, scoreResult, isDiffMode ? includePaths.length : projectInfo.sourceFileCount, options.report, directory);
+	printSummary(diagnostics, elapsedMilliseconds, scoreResult, isDiffMode ? includePaths.length : projectInfo.sourceFileCount, options.report, directory, isDiffMode);
 	if (hasSkippedChecks) {
 		const skippedLabel = skippedChecks.join(" and ");
 		logger.break();
@@ -1009,7 +1426,9 @@ const scan = async (directory, inputOptions = {}) => {
 	return {
 		diagnostics,
 		scoreResult,
-		skippedChecks
+		skippedChecks,
+		errorCount,
+		warningCount
 	};
 };
 
@@ -1160,7 +1579,7 @@ const selectProjects = async (rootDirectory, projectFlag, skipPrompts) => {
 
 //#endregion
 //#region src/cli.ts
-const VERSION = "1.1.3";
+const VERSION = "1.3.0";
 const exitWithHint = () => {
 	logger.break();
 	logger.log("Cancelled.");
@@ -1186,7 +1605,8 @@ const resolveCliScanOptions = (flags, userConfig, programInstance) => {
 		verbose: isCliOverride("verbose") ? Boolean(flags.verbose) : userConfig?.verbose ?? false,
 		scoreOnly: flags.score,
 		report: flags.report,
-		fast: isCliOverride("fast") ? Boolean(flags.fast) : userConfig?.fast ?? false
+		fast: isCliOverride("fast") ? Boolean(flags.fast) : userConfig?.fast ?? false,
+		rules: flags.rules
 	};
 };
 const resolveDiffMode = async (diffInfo, effectiveDiff, shouldSkipPrompts, isScoreOnly) => {
@@ -1204,7 +1624,7 @@ const resolveDiffMode = async (diffInfo, effectiveDiff, shouldSkipPrompts, isSco
 	if (isScoreOnly) return false;
 	return false;
 };
-const program = new Command().name("angular-doctor").description("Diagnose Angular codebase health").version(VERSION, "-v, --version", "display the version number").argument("[directory]", "project directory to scan", ".").option("--no-lint", "skip linting").option("--no-dead-code", "skip dead code detection").option("--verbose", "show file details per rule").option("--score", "output only the score").option("--report [path]", "write a markdown report (optional output path)").option("--fast", "speed up by skipping dead code and type-aware lint").option("-y, --yes", "skip prompts, scan all workspace projects").option("--project <name>", "select workspace project (comma-separated for multiple)").option("--diff [base]", "scan only files changed vs base branch").action(async (directory, flags) => {
+const program = new Command().name("angular-doctor").description("Diagnose Angular codebase health").version(VERSION, "-v, --version", "display the version number").argument("[directory]", "project directory to scan", ".").option("--no-lint", "skip linting").option("--no-dead-code", "skip dead code detection").option("--verbose", "show file details per rule").option("--score", "output only the score").option("--report [path]", "write a markdown report (optional output path)").option("--fast", "speed up by skipping dead code and type-aware lint").option("-y, --yes", "skip prompts, scan all workspace projects").option("--project <name>", "select workspace project (comma-separated for multiple)").option("--diff [base]", "scan only files changed vs base branch (Note: dead code detection is skipped in diff mode)").option("--rules <categories>", "force-enable specific rule categories (signals,ngrx,material) or 'all'").option("--exit-code", "exit with non-zero code when ESLint errors are found (for CI integration)").action(async (directory, flags) => {
 	const isScoreOnly = flags.score;
 	try {
 		const resolvedDirectory = path.resolve(directory);
@@ -1223,6 +1643,7 @@ const program = new Command().name("angular-doctor").description("Diagnose Angul
 		if (isDiffMode && diffInfo && !isScoreOnly) {
 			if (diffInfo.isCurrentChanges) logger.log("Scanning uncommitted changes");
 			else logger.log(`Scanning changes: ${highlighter.info(diffInfo.currentBranch)} → ${highlighter.info(diffInfo.baseBranch)}`);
+			logger.warn("Note: Dead code detection skipped in diff mode. Run without --diff for full scan.");
 			logger.break();
 		}
 		for (const projectDirectory of projectDirectories) {
@@ -1245,10 +1666,11 @@ const program = new Command().name("angular-doctor").description("Diagnose Angul
 				logger.dim(`Scanning ${projectDirectory}...`);
 				logger.break();
 			}
-			await scan(projectDirectory, {
+			const scanResult = await scan(projectDirectory, {
 				...scanOptions,
 				includePaths
 			});
+			if ((flags.exitCode || isAutomatedEnvironment()) && scanResult.errorCount > 0) process.exitCode = 1;
 			if (!isScoreOnly) logger.break();
 		}
 	} catch (error) {