@futdevpro/dynamo-eslint 1.15.25 → 1.15.28

package/__documentations/2026-04-18-ccap-review-integration.md

@@ -1,173 +1,173 @@
-# CCAP Revisioned Review integráció a Dynamo ESLint-be
-
-**Dátum:** 2026-04-18
-
-## Összefoglaló
-
-A `ccap-revisioned/server/src/_modules/developer-tools/` alatti **Code Review
-toolkit** AST-portable részének átültetése a `@futdevpro/dynamo-eslint`
-plugin-be. Eredmény: az IDE a háttér-ben azonnal mutatja a CCAP-szintű
-review-violation-öket — `eslint --fix`-szel a fixálható-k automatikusan
-javíthatók.
-
-Az eredeti `ccap-revisioned` projekt-hez **nem** nyúltunk hozzá.
-
-## Áttekintés: 25 AST-portable review
-
-A 38 CCAP review-ból:
-
-- **8 review** csak filesystem-szinten értelmezhető (cross-file konzisztencia,
-  file-rename autofix) → **csak `dc rev`-ben** él (lásd `dynamo-cli` doc).
-- **6 review** lefedve **ESLint built-in** vagy `@typescript-eslint`
-  szabállyal — `base.ts`-ben aktiválva.
-- **4 review** **megerősítve** meglévő `@futdevpro/dynamo` szabályokon
-  belül (sub-check-ek + szigorúbb logika).
-- **14 review** **új AST-szabály-ként** implementálva.
-
-## 1. Új ESLint built-in / typescript-eslint aktiválás (`base.ts`)
-
-| Szabály                                          | Megfelelő CCAP review        |
-| ------------------------------------------------ | ---------------------------- |
-| `max-params: [warn, 3]` (régről 4)               | `method-input-count`         |
-| `object-shorthand: [warn, never]`                | `no-object-shorthand`        |
-| `dot-notation: warn`                             | `no-bracket-property-access` |
-| `@typescript-eslint/array-type: [warn, array]`   | `no-array-generic`           |
-| `@typescript-eslint/prefer-optional-chain: warn` | `prefer-optional-chaining`   |
-| `@typescript-eslint/no-require-imports: warn`    | `no-dynamic-imports` (CJS)   |
-
-## 2. Megerősített meglévő szabályok
-
-### `naming-patterns` (komplett rewrite)
-
-Konfigurálható sub-check-ek `checks` schema option-nel — mind a 7 CCAP
-naming-review egy szabály-on belül, AST-alapú PropertyDefinition /
-ClassDeclaration / TSEnumDeclaration visitor-okkal.
-
-Sub-check-ek (default: mind aktív):
-
-- `booleanIsPrefix` — boolean property `is`-prefix
-- `signalPostfix` — `signal()` / `computed()` factory `_$` postfix
-- `rxjsPostfix` — `BehaviorSubject` `_BS`, `Subject` `_S`, `Observable` `$`
-- `fileClassPostfix` — class-postfix illeszkedik a fájl-pattern-hez
-- `enumValueFormat` — enum kulcs camelCase, érték kebab-case (`*.type-enum.ts`)
-- `servicePropertyPostfix` — `_DataService` injektált prop `_DS`-re végződik (`_AS`, `_CS`, `_MS` is)
-- `fileClassNameConsistency` — fájl-entity rész = class-entity rész (kebab-átkonvertálva)
-
-### `no-getter-logic` (CCAP-megerősítés)
-
-- Új tiltott AST node-ok: `IfStatement`, `ForStatement`, `WhileStatement`,
-  `SwitchStatement`, `TryStatement`, `ThrowStatement`, `VariableDeclaration`
-- Új ellenőrzés: `BlockStatement.body.length > 1` — több-utasításos getter tilos
-
-### `require-jsdoc-description` (CCAP-megerősítés)
-
-- Új visitor: `TSTypeAliasDeclaration` — exportált `type X = …` is kötelező
-  JSDoc-os
-
-### `import-order` (változatlanul lefedi)
-
-A CCAP `import-group-order` review 4-csoport-os elvét már a meglévő rule
-implementálja.
-
-## 3. Új TypeScript szabályok (`src/plugin/rules/ts/`)
-
-| Szabály                       | CCAP review                  |
-| ----------------------------- | ---------------------------- |
-| `no-as-cast`                  | `no-as-cast` (kivéve `as const`) |
-| `no-any-type`                 | `no-any-type`                |
-| `no-dynamic-imports`          | `no-dynamic-imports` (ES + CJS) |
-| `one-export-per-file`         | `one-export-per-file` (kivételek: `.const.ts`, `.type-enum.ts`, `.interface.ts`, `.data-model.ts`, `.control-model.ts`) |
-| `no-plain-function-export`    | `no-plain-function-export` (kivéve `.spec.ts`) |
-| `data-model-extends`          | `data-model-extends-metadata` + `parallel-date-fields` (egy szabály) |
-| `no-native-browser-dialogs`   | `no-native-browser-dialogs`  |
-
-## 4. Új Angular szabályok (`src/plugin/rules/angular/`)
-
-### TypeScript-AST (alkalmazva `*.ts` fájlokra)
-
-| Szabály                       | CCAP review                  |
-| ----------------------------- | ---------------------------- |
-| `no-async-ngoninit`           | `no-async-ngoninit` — `async ngOnInit` tilos |
-| `no-component-direct-api`     | `no-component-direct-api` — komponens nem importálhat `*ApiService`-t / `HttpClient`-et |
-| `effect-signal-loop`          | `effect-signal-loop` — `effect()` signal read+write detection |
-
-### Template-text-scan (alkalmazva `*.html` fájlokra)
-
-| Szabály                       | CCAP review                  |
-| ----------------------------- | ---------------------------- |
-| `no-method-call-in-template`  | `no-method-call-in-template` — `{{ method() }}` / `[attr]="m()"` / `@if (m())` tilos |
-| `angular-for-track-logic`     | `angular-for-track-logic` — `@for track` csak property access lehet |
-| `no-ngif-directive`           | `no-ngif-directive` — `*ngIf` helyett `@if` |
-| `no-ngfor-directive`          | `no-ngfor-directive` — `*ngFor` helyett `@for` |
-
-> **Megjegyzés**: A 4 template-szabály nem AST-t, hanem source-text regex-et
-> használ a `Program` visitor-ban. Ez direkt döntés — robosztusabb a többféle
-> Angular template-parser verzió ellenében, és a CCAP review is ugyanígy
-> szöveg-szinten skennel.
-
-## Aktiválás
-
-A 7 új TS szabály mind regisztrálódik a `base.ts`-ben (default: `warn`).
-
-A 7 Angular szabály a `ngx.ts`-ben aktiválódik:
-
-- 3 darab a `*.ts` block-ban (no-async-ngoninit, no-component-direct-api, effect-signal-loop)
-- 4 darab a `*.html` block-ban (template-szabályok)
-
-A `ngx.ts` html-block plugins szekcióját kibővítettük `...ngxBaseConfig[1].plugins`-szal,
-hogy a `@futdevpro/dynamo` plugin a html files alatt is regisztrált legyen.
-
-## Tesztek
-
-Minden új szabály-hoz `*.spec.ts` (mock-context pattern, ahogy a meglévő
-specek is csinálták):
-
-- 7 darab `src/plugin/rules/ts/*.spec.ts`
-- 7 darab `src/plugin/rules/angular/*.spec.ts`
-- `naming-patterns.spec.ts` — komplett rewrite a 7 sub-check-hez
-- `no-getter-logic.spec.ts` — multi-statement + if-statement edge case-ek
-- `require-jsdoc-description.spec.ts` — `TSTypeAliasDeclaration` visitor expose
-
-Eredmény: **186 spec, 0 failure** (a meglévő 167-en felül a 19 új).
-
-Futtatás: `pnpm test`.
-
-## Disable comment
-
-ESLint natív disable-komment-tel kapcsolható ki egy adott rule:
-
-```typescript
-// eslint-disable-next-line @futdevpro/dynamo/no-as-cast
-const x: MyType = (someUnknown as MyType);
-```
-
-A CCAP `// ccap-review-disable` és az új `// dy-review-disable` szintaxis
-**csak** a CLI-szintű `dc rev` által értelmezett — ESLint-en nem hat. Ha
-mindkét toolból ki akarsz tiltani egy violation-t, mindkét komment-stílust
-használd.
-
-## When to use which?
-
-| Helyzet                                          | CLI (`dc rev`) | ESLint plugin |
-| ------------------------------------------------ | -------------- | ------------- |
-| IDE inline-hint, save-on-format autofix          |                | ✓             |
-| Pre-commit hook (gyors, fájlonkénti)             |                | ✓             |
-| CI repo-wide audit JSON output-tal               | ✓              |               |
-| Cross-file konzisztencia (module-export-prefix)  | ✓              |               |
-| File-rename autofix (file-class-name-consistency) | ✓             |               |
-| Bulk autofix (`--fix --apply`)                   | ✓              |               |
-| Per-fájl, AST-pontosságú szabály-violation       |                | ✓             |
-| Naming + signal + rxjs postfix                   | ✓              | ✓ (mindkettő) |
-| Template `*ngIf` / `@for track` ellenőrzés       | ✓              | ✓ (mindkettő) |
-
-A két toolkit **komplementer**: az ESLint az IDE-szintű real-time feedback,
-a CLI a repo-szintű audit + bulk-autofix. Ahol mindkét tool implementálja
-ugyanazt a review-t, ott double-coverage van — ez szándékos: az ESLint az
-IDE-ben azonnal jelez, a CLI pedig CI-ben még egyszer leellenőriz.
-
-## Kapcsolódó dokumentumok
-
-- `dynamo-cli/__documentations/2026-04-18-ccap-review-integration.md` — CLI oldal
-- `dynamo-eslint/src/configs/base.ts` — base ruleset (összes új TS szabály)
-- `dynamo-eslint/src/configs/ngx.ts` — Angular ruleset (új Angular + template szabályok)
-- `dynamo-eslint/src/plugin/index.ts` — plugin export + recommended config
+# CCAP Revisioned Review integráció a Dynamo ESLint-be
+
+**Dátum:** 2026-04-18
+
+## Összefoglaló
+
+A `ccap-revisioned/server/src/_modules/developer-tools/` alatti **Code Review
+toolkit** AST-portable részének átültetése a `@futdevpro/dynamo-eslint`
+plugin-be. Eredmény: az IDE a háttér-ben azonnal mutatja a CCAP-szintű
+review-violation-öket — `eslint --fix`-szel a fixálható-k automatikusan
+javíthatók.
+
+Az eredeti `ccap-revisioned` projekt-hez **nem** nyúltunk hozzá.
+
+## Áttekintés: 25 AST-portable review
+
+A 38 CCAP review-ból:
+
+- **8 review** csak filesystem-szinten értelmezhető (cross-file konzisztencia,
+  file-rename autofix) → **csak `dc rev`-ben** él (lásd `dynamo-cli` doc).
+- **6 review** lefedve **ESLint built-in** vagy `@typescript-eslint`
+  szabállyal — `base.ts`-ben aktiválva.
+- **4 review** **megerősítve** meglévő `@futdevpro/dynamo` szabályokon
+  belül (sub-check-ek + szigorúbb logika).
+- **14 review** **új AST-szabály-ként** implementálva.
+
+## 1. Új ESLint built-in / typescript-eslint aktiválás (`base.ts`)
+
+| Szabály                                          | Megfelelő CCAP review        |
+| ------------------------------------------------ | ---------------------------- |
+| `max-params: [warn, 3]` (régről 4)               | `method-input-count`         |
+| `object-shorthand: [warn, never]`                | `no-object-shorthand`        |
+| `dot-notation: warn`                             | `no-bracket-property-access` |
+| `@typescript-eslint/array-type: [warn, array]`   | `no-array-generic`           |
+| `@typescript-eslint/prefer-optional-chain: warn` | `prefer-optional-chaining`   |
+| `@typescript-eslint/no-require-imports: warn`    | `no-dynamic-imports` (CJS)   |
+
+## 2. Megerősített meglévő szabályok
+
+### `naming-patterns` (komplett rewrite)
+
+Konfigurálható sub-check-ek `checks` schema option-nel — mind a 7 CCAP
+naming-review egy szabály-on belül, AST-alapú PropertyDefinition /
+ClassDeclaration / TSEnumDeclaration visitor-okkal.
+
+Sub-check-ek (default: mind aktív):
+
+- `booleanIsPrefix` — boolean property `is`-prefix
+- `signalPostfix` — `signal()` / `computed()` factory `_$` postfix
+- `rxjsPostfix` — `BehaviorSubject` `_BS`, `Subject` `_S`, `Observable` `$`
+- `fileClassPostfix` — class-postfix illeszkedik a fájl-pattern-hez
+- `enumValueFormat` — enum kulcs camelCase, érték kebab-case (`*.type-enum.ts`)
+- `servicePropertyPostfix` — `_DataService` injektált prop `_DS`-re végződik (`_AS`, `_CS`, `_MS` is)
+- `fileClassNameConsistency` — fájl-entity rész = class-entity rész (kebab-átkonvertálva)
+
+### `no-getter-logic` (CCAP-megerősítés)
+
+- Új tiltott AST node-ok: `IfStatement`, `ForStatement`, `WhileStatement`,
+  `SwitchStatement`, `TryStatement`, `ThrowStatement`, `VariableDeclaration`
+- Új ellenőrzés: `BlockStatement.body.length > 1` — több-utasításos getter tilos
+
+### `require-jsdoc-description` (CCAP-megerősítés)
+
+- Új visitor: `TSTypeAliasDeclaration` — exportált `type X = …` is kötelező
+  JSDoc-os
+
+### `import-order` (változatlanul lefedi)
+
+A CCAP `import-group-order` review 4-csoport-os elvét már a meglévő rule
+implementálja.
+
+## 3. Új TypeScript szabályok (`src/plugin/rules/ts/`)
+
+| Szabály                       | CCAP review                  |
+| ----------------------------- | ---------------------------- |
+| `no-as-cast`                  | `no-as-cast` (kivéve `as const`) |
+| `no-any-type`                 | `no-any-type`                |
+| `no-dynamic-imports`          | `no-dynamic-imports` (ES + CJS) |
+| `one-export-per-file`         | `one-export-per-file` (kivételek: `.const.ts`, `.type-enum.ts`, `.interface.ts`, `.data-model.ts`, `.control-model.ts`) |
+| `no-plain-function-export`    | `no-plain-function-export` (kivéve `.spec.ts`) |
+| `data-model-extends`          | `data-model-extends-metadata` + `parallel-date-fields` (egy szabály) |
+| `no-native-browser-dialogs`   | `no-native-browser-dialogs`  |
+
+## 4. Új Angular szabályok (`src/plugin/rules/angular/`)
+
+### TypeScript-AST (alkalmazva `*.ts` fájlokra)
+
+| Szabály                       | CCAP review                  |
+| ----------------------------- | ---------------------------- |
+| `no-async-ngoninit`           | `no-async-ngoninit` — `async ngOnInit` tilos |
+| `no-component-direct-api`     | `no-component-direct-api` — komponens nem importálhat `*ApiService`-t / `HttpClient`-et |
+| `effect-signal-loop`          | `effect-signal-loop` — `effect()` signal read+write detection |
+
+### Template-text-scan (alkalmazva `*.html` fájlokra)
+
+| Szabály                       | CCAP review                  |
+| ----------------------------- | ---------------------------- |
+| `no-method-call-in-template`  | `no-method-call-in-template` — `{{ method() }}` / `[attr]="m()"` / `@if (m())` tilos |
+| `angular-for-track-logic`     | `angular-for-track-logic` — `@for track` csak property access lehet |
+| `no-ngif-directive`           | `no-ngif-directive` — `*ngIf` helyett `@if` |
+| `no-ngfor-directive`          | `no-ngfor-directive` — `*ngFor` helyett `@for` |
+
+> **Megjegyzés**: A 4 template-szabály nem AST-t, hanem source-text regex-et
+> használ a `Program` visitor-ban. Ez direkt döntés — robosztusabb a többféle
+> Angular template-parser verzió ellenében, és a CCAP review is ugyanígy
+> szöveg-szinten skennel.
+
+## Aktiválás
+
+A 7 új TS szabály mind regisztrálódik a `base.ts`-ben (default: `warn`).
+
+A 7 Angular szabály a `ngx.ts`-ben aktiválódik:
+
+- 3 darab a `*.ts` block-ban (no-async-ngoninit, no-component-direct-api, effect-signal-loop)
+- 4 darab a `*.html` block-ban (template-szabályok)
+
+A `ngx.ts` html-block plugins szekcióját kibővítettük `...ngxBaseConfig[1].plugins`-szal,
+hogy a `@futdevpro/dynamo` plugin a html files alatt is regisztrált legyen.
+
+## Tesztek
+
+Minden új szabály-hoz `*.spec.ts` (mock-context pattern, ahogy a meglévő
+specek is csinálták):
+
+- 7 darab `src/plugin/rules/ts/*.spec.ts`
+- 7 darab `src/plugin/rules/angular/*.spec.ts`
+- `naming-patterns.spec.ts` — komplett rewrite a 7 sub-check-hez
+- `no-getter-logic.spec.ts` — multi-statement + if-statement edge case-ek
+- `require-jsdoc-description.spec.ts` — `TSTypeAliasDeclaration` visitor expose
+
+Eredmény: **186 spec, 0 failure** (a meglévő 167-en felül a 19 új).
+
+Futtatás: `pnpm test`.
+
+## Disable comment
+
+ESLint natív disable-komment-tel kapcsolható ki egy adott rule:
+
+```typescript
+// eslint-disable-next-line @futdevpro/dynamo/no-as-cast
+const x: MyType = (someUnknown as MyType);
+```
+
+A CCAP `// ccap-review-disable` és az új `// dy-review-disable` szintaxis
+**csak** a CLI-szintű `dc rev` által értelmezett — ESLint-en nem hat. Ha
+mindkét toolból ki akarsz tiltani egy violation-t, mindkét komment-stílust
+használd.
+
+## When to use which?
+
+| Helyzet                                          | CLI (`dc rev`) | ESLint plugin |
+| ------------------------------------------------ | -------------- | ------------- |
+| IDE inline-hint, save-on-format autofix          |                | ✓             |
+| Pre-commit hook (gyors, fájlonkénti)             |                | ✓             |
+| CI repo-wide audit JSON output-tal               | ✓              |               |
+| Cross-file konzisztencia (module-export-prefix)  | ✓              |               |
+| File-rename autofix (file-class-name-consistency) | ✓             |               |
+| Bulk autofix (`--fix --apply`)                   | ✓              |               |
+| Per-fájl, AST-pontosságú szabály-violation       |                | ✓             |
+| Naming + signal + rxjs postfix                   | ✓              | ✓ (mindkettő) |
+| Template `*ngIf` / `@for track` ellenőrzés       | ✓              | ✓ (mindkettő) |
+
+A két toolkit **komplementer**: az ESLint az IDE-szintű real-time feedback,
+a CLI a repo-szintű audit + bulk-autofix. Ahol mindkét tool implementálja
+ugyanazt a review-t, ott double-coverage van — ez szándékos: az ESLint az
+IDE-ben azonnal jelez, a CLI pedig CI-ben még egyszer leellenőriz.
+
+## Kapcsolódó dokumentumok
+
+- `dynamo-cli/__documentations/2026-04-18-ccap-review-integration.md` — CLI oldal
+- `dynamo-eslint/src/configs/base.ts` — base ruleset (összes új TS szabály)
+- `dynamo-eslint/src/configs/ngx.ts` — Angular ruleset (új Angular + template szabályok)
+- `dynamo-eslint/src/plugin/index.ts` — plugin export + recommended config

package/__documentations/2026-08-18-bfr-ctm-005-eslint9-validator-report.md

@@ -0,0 +1,45 @@
+# BFR-CTM-005 — ESLint 9 validator flat-config correction
+
+**Date:** 2026-08-18
+
+**Target release:** `@futdevpro/dynamo-eslint@1.15.28`
+
+**Consumer:** Company Toolmaker
+
+## Failure reproduced
+
+The published `1.15.25` `dynamo-validate-imports` CLI supplied an eslintrc-shaped `baseConfig` to ESLint 9.
+Every scanned file failed with `ConfigError: Key "parser" ... eslintrc format`, and the existing fail-hard guard
+correctly rejected the untrustworthy result. The naming CLI loaded the package's full NTS config, so formatting,
+JSDoc and TypeScript rules were incorrectly reported as naming violations.
+
+## Correction
+
+- Both CLIs now set `overrideConfigFile: true` and supply an isolated ESLint 9 flat config.
+- Import validation enables only `@futdevpro/dynamo/import-order`.
+- Naming validation enables only `@futdevpro/dynamo/naming-patterns`.
+- Unused disable-directive diagnostics are disabled inside the narrow validators because they are unrelated to
+  either target rule and otherwise produce false failures.
+- Parser resolution remains a consumer peer dependency; the explicit top-level runtime load preserves the
+  package's legacy TypeScript module-resolution compatibility.
+- Runtime/config processing failures remain fail-hard.
+
+## Evidence
+
+- TypeScript compile: PASS.
+- Modified-source ESLint: PASS with zero warnings.
+- Jasmine: 202 specs, 0 failures after the final isolation fixture extension.
+- New subprocess integration coverage proves clean/violating fixtures for both CLIs, absence of `ConfigError`,
+  and isolation from unrelated disable/JSDoc diagnostics.
+- Company Toolmaker verification against the built fix:
+  - server import validation: PASS;
+  - server naming validation: PASS;
+  - client naming validation: PASS;
+  - client import baseline exposed two real ordering findings, corrected in the consumer change-set.
+
+## Release hygiene
+
+The working directory contains a pre-existing untracked `SKILLS.md` owned by another change-set. npm's default
+file discovery would include it in a package built directly from this dirty directory. The `1.15.28` publish must
+therefore run from a clean worktree at the reviewed commit with lifecycle scripts disabled; the packed manifest
+must be rechecked there before publication. The unrelated file is preserved and excluded from this commit.

package/build/configs/base.d.ts

@@ -1,10 +1,10 @@
-/**
- * Base Rulesets For Dynamo Based Systems
- * ESLint v9 Flat Config Format
- */
-declare const tsParser: any;
-declare const tsPlugin: any;
-declare const unusedImportsPlugin: any;
-declare const maxParamsNoConstructorPlugin: any;
-declare const dynamoPlugin: any;
+/**
+ * Base Rulesets For Dynamo Based Systems
+ * ESLint v9 Flat Config Format
+ */
+declare const tsParser: any;
+declare const tsPlugin: any;
+declare const unusedImportsPlugin: any;
+declare const maxParamsNoConstructorPlugin: any;
+declare const dynamoPlugin: any;
 //# sourceMappingURL=base.d.ts.map