@futdevpro/dynamo-eslint 1.20.3 → 1.20.5
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/LICENSE +21 -0
- package/README.md +304 -304
- package/build/plugin/rules/ts/no-as-cast.d.ts.map +1 -1
- package/build/plugin/rules/ts/no-as-cast.js +43 -0
- package/build/plugin/rules/ts/no-as-cast.js.map +1 -1
- package/package.json +14 -7
- package/.c8rc.json +0 -26
- package/.cursor/rules/__assistant_guide.mdc +0 -30
- package/.cursor/rules/__main.mdc +0 -64
- package/.cursor/rules/_ag_backend-structure.mdc +0 -86
- package/.cursor/rules/_ag_backend.mdc +0 -16
- package/.cursor/rules/_ag_debug.mdc +0 -8
- package/.cursor/rules/_ag_documentation_writing_rules.mdc +0 -372
- package/.cursor/rules/_ag_file-refactoring.mdc +0 -113
- package/.cursor/rules/_ag_fixes_rules.mdc +0 -6
- package/.cursor/rules/_ag_frontend-structure.mdc +0 -87
- package/.cursor/rules/_ag_frontend.mdc +0 -40
- package/.cursor/rules/_ag_import-rules.mdc +0 -45
- package/.cursor/rules/_ag_naming.mdc +0 -116
- package/.cursor/rules/_ag_running_commands.mdc +0 -5
- package/.cursor/rules/_ag_server-controller.mdc +0 -6
- package/.cursor/rules/_ag_should-be.mdc +0 -7
- package/.cursor/rules/_ag_swearing.mdc +0 -47
- package/.cursor/rules/ai_development_guide.md +0 -61
- package/.cursor/rules/ai_directives.md +0 -114
- package/.cursor/rules/cursor-rules.md +0 -160
- package/.cursor/rules/default-command.mdc +0 -465
- package/.cursor/rules/error_code_pattern.md +0 -40
- package/.cursor/rules/saved rule mcp server use.md +0 -16
- package/.dynamo/pipeline.cicd.config.json +0 -184
- package/.dynamo/version-bump.config.json +0 -5
- package/.eslintrc.json +0 -4
- package/.github/workflows/main.yml +0 -436
- package/.husky/pre-commit +0 -2
- package/.vscode/settings.json +0 -11
- package/__documentations/2026-04-18-ccap-review-integration.md +0 -173
- package/__documentations/2026-08-18-bfr-ctm-005-eslint9-validator-report.md +0 -45
- package/__documentations/2026-08-22-angular-20-phase2-release.md +0 -83
- package/eslint.config.js +0 -15
- package/pnpm-workspace.yaml +0 -2
- package/samples/.vscode/settings.json +0 -13
- package/samples/base/.eslintrc.json +0 -6
- package/samples/base/eslint.config.js +0 -3
- package/samples/fsm/.eslintrc.json +0 -4
- package/samples/fsm/eslint.config.js +0 -3
- package/samples/ngx/.eslintrc.json +0 -6
- package/samples/ngx/eslint.config.js +0 -3
- package/samples/ngx-package/.eslintrc.json +0 -4
- package/samples/ngx-package/eslint.config.js +0 -3
- package/samples/nts/.eslintrc.json +0 -6
- package/samples/nts/eslint.config.js +0 -3
- package/samples/nts-package/.eslintrc.json +0 -4
- package/samples/nts-package/eslint.config.js +0 -3
- package/samples/package.json.example +0 -22
- package/samples/poc-sample.ts +0 -38
- package/samples/poc-violations.ts +0 -54
- package/scripts/run-coverage-tests.js +0 -24
- package/spec/support/helpers/ts-node-helper.js +0 -9
- package/spec/support/jasmine.coverage.json +0 -23
- package/spec/support/jasmine.json +0 -24
- package/src/configs/angular20-compatibility.spec.ts +0 -73
- package/src/configs/base.ts +0 -156
- package/src/configs/fsm.ts +0 -10
- package/src/configs/ngx-package.ts +0 -10
- package/src/configs/ngx.ts +0 -93
- package/src/configs/nts-package.ts +0 -20
- package/src/configs/nts.ts +0 -34
- package/src/index.ts +0 -4
- package/src/plugin/index.ts +0 -86
- package/src/plugin/rules/angular/angular-for-track-logic.spec.ts +0 -44
- package/src/plugin/rules/angular/angular-for-track-logic.ts +0 -88
- package/src/plugin/rules/angular/effect-signal-loop.spec.ts +0 -89
- package/src/plugin/rules/angular/effect-signal-loop.ts +0 -125
- package/src/plugin/rules/angular/no-async-ngoninit.spec.ts +0 -50
- package/src/plugin/rules/angular/no-async-ngoninit.ts +0 -43
- package/src/plugin/rules/angular/no-component-direct-api.spec.ts +0 -78
- package/src/plugin/rules/angular/no-component-direct-api.ts +0 -82
- package/src/plugin/rules/angular/no-method-call-in-template.spec.ts +0 -63
- package/src/plugin/rules/angular/no-method-call-in-template.ts +0 -89
- package/src/plugin/rules/angular/no-ngfor-directive.spec.ts +0 -44
- package/src/plugin/rules/angular/no-ngfor-directive.ts +0 -62
- package/src/plugin/rules/angular/no-ngif-directive.spec.ts +0 -44
- package/src/plugin/rules/angular/no-ngif-directive.ts +0 -62
- package/src/plugin/rules/explicit-types.spec.ts +0 -190
- package/src/plugin/rules/explicit-types.ts +0 -186
- package/src/plugin/rules/import/import-order.spec.ts +0 -219
- package/src/plugin/rules/import/import-order.ts +0 -197
- package/src/plugin/rules/import/no-import-type.spec.ts +0 -69
- package/src/plugin/rules/import/no-import-type.ts +0 -46
- package/src/plugin/rules/import/no-js-import.spec.ts +0 -82
- package/src/plugin/rules/import/no-js-import.ts +0 -45
- package/src/plugin/rules/naming-patterns.spec.ts +0 -189
- package/src/plugin/rules/naming-patterns.ts +0 -426
- package/src/plugin/rules/no-getter-logic.spec.ts +0 -591
- package/src/plugin/rules/no-getter-logic.ts +0 -220
- package/src/plugin/rules/prefer-enum-over-string-union.spec.ts +0 -583
- package/src/plugin/rules/prefer-enum-over-string-union.ts +0 -333
- package/src/plugin/rules/prefer-interface-over-duplicate-types.spec.ts +0 -404
- package/src/plugin/rules/prefer-interface-over-duplicate-types.ts +0 -314
- package/src/plugin/rules/require-jsdoc-description.spec.ts +0 -779
- package/src/plugin/rules/require-jsdoc-description.ts +0 -572
- package/src/plugin/rules/require-test-description-prefix.spec.ts +0 -459
- package/src/plugin/rules/require-test-description-prefix.ts +0 -154
- package/src/plugin/rules/ts/data-model-extends.spec.ts +0 -84
- package/src/plugin/rules/ts/data-model-extends.ts +0 -100
- package/src/plugin/rules/ts/no-any-type.spec.ts +0 -28
- package/src/plugin/rules/ts/no-any-type.ts +0 -36
- package/src/plugin/rules/ts/no-as-cast.spec.ts +0 -51
- package/src/plugin/rules/ts/no-as-cast.ts +0 -51
- package/src/plugin/rules/ts/no-dynamic-imports.spec.ts +0 -54
- package/src/plugin/rules/ts/no-dynamic-imports.ts +0 -49
- package/src/plugin/rules/ts/no-native-browser-dialogs.spec.ts +0 -54
- package/src/plugin/rules/ts/no-native-browser-dialogs.ts +0 -84
- package/src/plugin/rules/ts/no-plain-function-export.spec.ts +0 -72
- package/src/plugin/rules/ts/no-plain-function-export.ts +0 -73
- package/src/plugin/rules/ts/one-export-per-file.spec.ts +0 -64
- package/src/plugin/rules/ts/one-export-per-file.ts +0 -77
- package/src/scripts/dynamo-fix.ts +0 -172
- package/src/scripts/eslintrc-audit.ts +0 -116
- package/src/scripts/fix-return-types.spec.ts +0 -65
- package/src/scripts/fix-return-types.ts +0 -289
- package/src/scripts/validate-imports.ts +0 -292
- package/src/scripts/validate-naming.ts +0 -213
- package/src/scripts/validator-cli.integration.spec.ts +0 -99
- package/tsconfig.json +0 -32
|
@@ -1,173 +0,0 @@
|
|
|
1
|
-
# CCAP Revisioned Review integráció a Dynamo ESLint-be
|
|
2
|
-
|
|
3
|
-
**Dátum:** 2026-04-18
|
|
4
|
-
|
|
5
|
-
## Összefoglaló
|
|
6
|
-
|
|
7
|
-
A `ccap-revisioned/server/src/_modules/developer-tools/` alatti **Code Review
|
|
8
|
-
toolkit** AST-portable részének átültetése a `@futdevpro/dynamo-eslint`
|
|
9
|
-
plugin-be. Eredmény: az IDE a háttér-ben azonnal mutatja a CCAP-szintű
|
|
10
|
-
review-violation-öket — `eslint --fix`-szel a fixálható-k automatikusan
|
|
11
|
-
javíthatók.
|
|
12
|
-
|
|
13
|
-
Az eredeti `ccap-revisioned` projekt-hez **nem** nyúltunk hozzá.
|
|
14
|
-
|
|
15
|
-
## Áttekintés: 25 AST-portable review
|
|
16
|
-
|
|
17
|
-
A 38 CCAP review-ból:
|
|
18
|
-
|
|
19
|
-
- **8 review** csak filesystem-szinten értelmezhető (cross-file konzisztencia,
|
|
20
|
-
file-rename autofix) → **csak `dc rev`-ben** él (lásd `dynamo-cli` doc).
|
|
21
|
-
- **6 review** lefedve **ESLint built-in** vagy `@typescript-eslint`
|
|
22
|
-
szabállyal — `base.ts`-ben aktiválva.
|
|
23
|
-
- **4 review** **megerősítve** meglévő `@futdevpro/dynamo` szabályokon
|
|
24
|
-
belül (sub-check-ek + szigorúbb logika).
|
|
25
|
-
- **14 review** **új AST-szabály-ként** implementálva.
|
|
26
|
-
|
|
27
|
-
## 1. Új ESLint built-in / typescript-eslint aktiválás (`base.ts`)
|
|
28
|
-
|
|
29
|
-
| Szabály | Megfelelő CCAP review |
|
|
30
|
-
| ------------------------------------------------ | ---------------------------- |
|
|
31
|
-
| `max-params: [warn, 3]` (régről 4) | `method-input-count` |
|
|
32
|
-
| `object-shorthand: [warn, never]` | `no-object-shorthand` |
|
|
33
|
-
| `dot-notation: warn` | `no-bracket-property-access` |
|
|
34
|
-
| `@typescript-eslint/array-type: [warn, array]` | `no-array-generic` |
|
|
35
|
-
| `@typescript-eslint/prefer-optional-chain: warn` | `prefer-optional-chaining` |
|
|
36
|
-
| `@typescript-eslint/no-require-imports: warn` | `no-dynamic-imports` (CJS) |
|
|
37
|
-
|
|
38
|
-
## 2. Megerősített meglévő szabályok
|
|
39
|
-
|
|
40
|
-
### `naming-patterns` (komplett rewrite)
|
|
41
|
-
|
|
42
|
-
Konfigurálható sub-check-ek `checks` schema option-nel — mind a 7 CCAP
|
|
43
|
-
naming-review egy szabály-on belül, AST-alapú PropertyDefinition /
|
|
44
|
-
ClassDeclaration / TSEnumDeclaration visitor-okkal.
|
|
45
|
-
|
|
46
|
-
Sub-check-ek (default: mind aktív):
|
|
47
|
-
|
|
48
|
-
- `booleanIsPrefix` — boolean property `is`-prefix
|
|
49
|
-
- `signalPostfix` — `signal()` / `computed()` factory `_$` postfix
|
|
50
|
-
- `rxjsPostfix` — `BehaviorSubject` `_BS`, `Subject` `_S`, `Observable` `$`
|
|
51
|
-
- `fileClassPostfix` — class-postfix illeszkedik a fájl-pattern-hez
|
|
52
|
-
- `enumValueFormat` — enum kulcs camelCase, érték kebab-case (`*.type-enum.ts`)
|
|
53
|
-
- `servicePropertyPostfix` — `_DataService` injektált prop `_DS`-re végződik (`_AS`, `_CS`, `_MS` is)
|
|
54
|
-
- `fileClassNameConsistency` — fájl-entity rész = class-entity rész (kebab-átkonvertálva)
|
|
55
|
-
|
|
56
|
-
### `no-getter-logic` (CCAP-megerősítés)
|
|
57
|
-
|
|
58
|
-
- Új tiltott AST node-ok: `IfStatement`, `ForStatement`, `WhileStatement`,
|
|
59
|
-
`SwitchStatement`, `TryStatement`, `ThrowStatement`, `VariableDeclaration`
|
|
60
|
-
- Új ellenőrzés: `BlockStatement.body.length > 1` — több-utasításos getter tilos
|
|
61
|
-
|
|
62
|
-
### `require-jsdoc-description` (CCAP-megerősítés)
|
|
63
|
-
|
|
64
|
-
- Új visitor: `TSTypeAliasDeclaration` — exportált `type X = …` is kötelező
|
|
65
|
-
JSDoc-os
|
|
66
|
-
|
|
67
|
-
### `import-order` (változatlanul lefedi)
|
|
68
|
-
|
|
69
|
-
A CCAP `import-group-order` review 4-csoport-os elvét már a meglévő rule
|
|
70
|
-
implementálja.
|
|
71
|
-
|
|
72
|
-
## 3. Új TypeScript szabályok (`src/plugin/rules/ts/`)
|
|
73
|
-
|
|
74
|
-
| Szabály | CCAP review |
|
|
75
|
-
| ----------------------------- | ---------------------------- |
|
|
76
|
-
| `no-as-cast` | `no-as-cast` (kivéve `as const`) |
|
|
77
|
-
| `no-any-type` | `no-any-type` |
|
|
78
|
-
| `no-dynamic-imports` | `no-dynamic-imports` (ES + CJS) |
|
|
79
|
-
| `one-export-per-file` | `one-export-per-file` (kivételek: `.const.ts`, `.type-enum.ts`, `.interface.ts`, `.data-model.ts`, `.control-model.ts`) |
|
|
80
|
-
| `no-plain-function-export` | `no-plain-function-export` (kivéve `.spec.ts`) |
|
|
81
|
-
| `data-model-extends` | `data-model-extends-metadata` + `parallel-date-fields` (egy szabály) |
|
|
82
|
-
| `no-native-browser-dialogs` | `no-native-browser-dialogs` |
|
|
83
|
-
|
|
84
|
-
## 4. Új Angular szabályok (`src/plugin/rules/angular/`)
|
|
85
|
-
|
|
86
|
-
### TypeScript-AST (alkalmazva `*.ts` fájlokra)
|
|
87
|
-
|
|
88
|
-
| Szabály | CCAP review |
|
|
89
|
-
| ----------------------------- | ---------------------------- |
|
|
90
|
-
| `no-async-ngoninit` | `no-async-ngoninit` — `async ngOnInit` tilos |
|
|
91
|
-
| `no-component-direct-api` | `no-component-direct-api` — komponens nem importálhat `*ApiService`-t / `HttpClient`-et |
|
|
92
|
-
| `effect-signal-loop` | `effect-signal-loop` — `effect()` signal read+write detection |
|
|
93
|
-
|
|
94
|
-
### Template-text-scan (alkalmazva `*.html` fájlokra)
|
|
95
|
-
|
|
96
|
-
| Szabály | CCAP review |
|
|
97
|
-
| ----------------------------- | ---------------------------- |
|
|
98
|
-
| `no-method-call-in-template` | `no-method-call-in-template` — `{{ method() }}` / `[attr]="m()"` / `@if (m())` tilos |
|
|
99
|
-
| `angular-for-track-logic` | `angular-for-track-logic` — `@for track` csak property access lehet |
|
|
100
|
-
| `no-ngif-directive` | `no-ngif-directive` — `*ngIf` helyett `@if` |
|
|
101
|
-
| `no-ngfor-directive` | `no-ngfor-directive` — `*ngFor` helyett `@for` |
|
|
102
|
-
|
|
103
|
-
> **Megjegyzés**: A 4 template-szabály nem AST-t, hanem source-text regex-et
|
|
104
|
-
> használ a `Program` visitor-ban. Ez direkt döntés — robosztusabb a többféle
|
|
105
|
-
> Angular template-parser verzió ellenében, és a CCAP review is ugyanígy
|
|
106
|
-
> szöveg-szinten skennel.
|
|
107
|
-
|
|
108
|
-
## Aktiválás
|
|
109
|
-
|
|
110
|
-
A 7 új TS szabály mind regisztrálódik a `base.ts`-ben (default: `warn`).
|
|
111
|
-
|
|
112
|
-
A 7 Angular szabály a `ngx.ts`-ben aktiválódik:
|
|
113
|
-
|
|
114
|
-
- 3 darab a `*.ts` block-ban (no-async-ngoninit, no-component-direct-api, effect-signal-loop)
|
|
115
|
-
- 4 darab a `*.html` block-ban (template-szabályok)
|
|
116
|
-
|
|
117
|
-
A `ngx.ts` html-block plugins szekcióját kibővítettük `...ngxBaseConfig[1].plugins`-szal,
|
|
118
|
-
hogy a `@futdevpro/dynamo` plugin a html files alatt is regisztrált legyen.
|
|
119
|
-
|
|
120
|
-
## Tesztek
|
|
121
|
-
|
|
122
|
-
Minden új szabály-hoz `*.spec.ts` (mock-context pattern, ahogy a meglévő
|
|
123
|
-
specek is csinálták):
|
|
124
|
-
|
|
125
|
-
- 7 darab `src/plugin/rules/ts/*.spec.ts`
|
|
126
|
-
- 7 darab `src/plugin/rules/angular/*.spec.ts`
|
|
127
|
-
- `naming-patterns.spec.ts` — komplett rewrite a 7 sub-check-hez
|
|
128
|
-
- `no-getter-logic.spec.ts` — multi-statement + if-statement edge case-ek
|
|
129
|
-
- `require-jsdoc-description.spec.ts` — `TSTypeAliasDeclaration` visitor expose
|
|
130
|
-
|
|
131
|
-
Eredmény: **186 spec, 0 failure** (a meglévő 167-en felül a 19 új).
|
|
132
|
-
|
|
133
|
-
Futtatás: `pnpm test`.
|
|
134
|
-
|
|
135
|
-
## Disable comment
|
|
136
|
-
|
|
137
|
-
ESLint natív disable-komment-tel kapcsolható ki egy adott rule:
|
|
138
|
-
|
|
139
|
-
```typescript
|
|
140
|
-
// eslint-disable-next-line @futdevpro/dynamo/no-as-cast
|
|
141
|
-
const x: MyType = (someUnknown as MyType);
|
|
142
|
-
```
|
|
143
|
-
|
|
144
|
-
A CCAP `// ccap-review-disable` és az új `// dy-review-disable` szintaxis
|
|
145
|
-
**csak** a CLI-szintű `dc rev` által értelmezett — ESLint-en nem hat. Ha
|
|
146
|
-
mindkét toolból ki akarsz tiltani egy violation-t, mindkét komment-stílust
|
|
147
|
-
használd.
|
|
148
|
-
|
|
149
|
-
## When to use which?
|
|
150
|
-
|
|
151
|
-
| Helyzet | CLI (`dc rev`) | ESLint plugin |
|
|
152
|
-
| ------------------------------------------------ | -------------- | ------------- |
|
|
153
|
-
| IDE inline-hint, save-on-format autofix | | ✓ |
|
|
154
|
-
| Pre-commit hook (gyors, fájlonkénti) | | ✓ |
|
|
155
|
-
| CI repo-wide audit JSON output-tal | ✓ | |
|
|
156
|
-
| Cross-file konzisztencia (module-export-prefix) | ✓ | |
|
|
157
|
-
| File-rename autofix (file-class-name-consistency) | ✓ | |
|
|
158
|
-
| Bulk autofix (`--fix --apply`) | ✓ | |
|
|
159
|
-
| Per-fájl, AST-pontosságú szabály-violation | | ✓ |
|
|
160
|
-
| Naming + signal + rxjs postfix | ✓ | ✓ (mindkettő) |
|
|
161
|
-
| Template `*ngIf` / `@for track` ellenőrzés | ✓ | ✓ (mindkettő) |
|
|
162
|
-
|
|
163
|
-
A két toolkit **komplementer**: az ESLint az IDE-szintű real-time feedback,
|
|
164
|
-
a CLI a repo-szintű audit + bulk-autofix. Ahol mindkét tool implementálja
|
|
165
|
-
ugyanazt a review-t, ott double-coverage van — ez szándékos: az ESLint az
|
|
166
|
-
IDE-ben azonnal jelez, a CLI pedig CI-ben még egyszer leellenőriz.
|
|
167
|
-
|
|
168
|
-
## Kapcsolódó dokumentumok
|
|
169
|
-
|
|
170
|
-
- `dynamo-cli/__documentations/2026-04-18-ccap-review-integration.md` — CLI oldal
|
|
171
|
-
- `dynamo-eslint/src/configs/base.ts` — base ruleset (összes új TS szabály)
|
|
172
|
-
- `dynamo-eslint/src/configs/ngx.ts` — Angular ruleset (új Angular + template szabályok)
|
|
173
|
-
- `dynamo-eslint/src/plugin/index.ts` — plugin export + recommended config
|
|
@@ -1,45 +0,0 @@
|
|
|
1
|
-
# BFR-CTM-005 — ESLint 9 validator flat-config correction
|
|
2
|
-
|
|
3
|
-
**Date:** 2026-08-18
|
|
4
|
-
|
|
5
|
-
**Target release:** `@futdevpro/dynamo-eslint@1.15.28`
|
|
6
|
-
|
|
7
|
-
**Consumer:** Company Toolmaker
|
|
8
|
-
|
|
9
|
-
## Failure reproduced
|
|
10
|
-
|
|
11
|
-
The published `1.15.25` `dynamo-validate-imports` CLI supplied an eslintrc-shaped `baseConfig` to ESLint 9.
|
|
12
|
-
Every scanned file failed with `ConfigError: Key "parser" ... eslintrc format`, and the existing fail-hard guard
|
|
13
|
-
correctly rejected the untrustworthy result. The naming CLI loaded the package's full NTS config, so formatting,
|
|
14
|
-
JSDoc and TypeScript rules were incorrectly reported as naming violations.
|
|
15
|
-
|
|
16
|
-
## Correction
|
|
17
|
-
|
|
18
|
-
- Both CLIs now set `overrideConfigFile: true` and supply an isolated ESLint 9 flat config.
|
|
19
|
-
- Import validation enables only `@futdevpro/dynamo/import-order`.
|
|
20
|
-
- Naming validation enables only `@futdevpro/dynamo/naming-patterns`.
|
|
21
|
-
- Unused disable-directive diagnostics are disabled inside the narrow validators because they are unrelated to
|
|
22
|
-
either target rule and otherwise produce false failures.
|
|
23
|
-
- Parser resolution remains a consumer peer dependency; the explicit top-level runtime load preserves the
|
|
24
|
-
package's legacy TypeScript module-resolution compatibility.
|
|
25
|
-
- Runtime/config processing failures remain fail-hard.
|
|
26
|
-
|
|
27
|
-
## Evidence
|
|
28
|
-
|
|
29
|
-
- TypeScript compile: PASS.
|
|
30
|
-
- Modified-source ESLint: PASS with zero warnings.
|
|
31
|
-
- Jasmine: 202 specs, 0 failures after the final isolation fixture extension.
|
|
32
|
-
- New subprocess integration coverage proves clean/violating fixtures for both CLIs, absence of `ConfigError`,
|
|
33
|
-
and isolation from unrelated disable/JSDoc diagnostics.
|
|
34
|
-
- Company Toolmaker verification against the built fix:
|
|
35
|
-
- server import validation: PASS;
|
|
36
|
-
- server naming validation: PASS;
|
|
37
|
-
- client naming validation: PASS;
|
|
38
|
-
- client import baseline exposed two real ordering findings, corrected in the consumer change-set.
|
|
39
|
-
|
|
40
|
-
## Release hygiene
|
|
41
|
-
|
|
42
|
-
The working directory contains a pre-existing untracked `SKILLS.md` owned by another change-set. npm's default
|
|
43
|
-
file discovery would include it in a package built directly from this dirty directory. The `1.15.28` publish must
|
|
44
|
-
therefore run from a clean worktree at the reviewed commit with lifecycle scripts disabled; the packed manifest
|
|
45
|
-
must be rechecked there before publication. The unrelated file is preserved and excluded from this commit.
|
|
@@ -1,83 +0,0 @@
|
|
|
1
|
-
# Angular 20 fleet upgrade — Phase 2 Dynamo ESLint release
|
|
2
|
-
|
|
3
|
-
**Date:** 2026-08-22
|
|
4
|
-
|
|
5
|
-
**Requirement:** BFR-CTM-006
|
|
6
|
-
|
|
7
|
-
**Release:** `@futdevpro/dynamo-eslint@1.20.3` after the Angular 20 consumer correction
|
|
8
|
-
|
|
9
|
-
**Dependency baseline:** `@futdevpro/fsm-dynamo@1.20.2`
|
|
10
|
-
|
|
11
|
-
## Outcome
|
|
12
|
-
|
|
13
|
-
The shared Dynamo ESLint package now has an Angular 20-compatible, ESLint 9 flat-config contract. The presets use
|
|
14
|
-
the unified `angular-eslint` and `typescript-eslint` public packages instead of requiring their implementation
|
|
15
|
-
subpackages as separate consumer peers. The obsolete Angular rule
|
|
16
|
-
`@angular-eslint/no-host-metadata-property`, which is not exported by Angular ESLint 20, is no longer configured.
|
|
17
|
-
|
|
18
|
-
This is a direct Bedrock correction. Consumers continue to load the existing public preset subpaths
|
|
19
|
-
(`base`, `nts`, `nts-package`, `fsm`, `ngx`, and `ngx-package`); no compatibility wrapper or replacement package
|
|
20
|
-
was introduced.
|
|
21
|
-
|
|
22
|
-
## Published dependency contract
|
|
23
|
-
|
|
24
|
-
- `angular-eslint`: `~20.7.0`
|
|
25
|
-
- `typescript-eslint`: `~8.67.0`
|
|
26
|
-
- `eslint`: `~9.39.5`
|
|
27
|
-
- `typescript`: `>=5.8.2 <6.0.0`
|
|
28
|
-
- runtime Bedrock dependency: exact `@futdevpro/fsm-dynamo@1.20.2`
|
|
29
|
-
|
|
30
|
-
`angular-eslint` is optional at package-install level so Node-only preset consumers do not need the Angular toolchain;
|
|
31
|
-
it remains mandatory whenever an Angular preset is loaded.
|
|
32
|
-
|
|
33
|
-
The previous `eslint-plugin-import`, `eslint-plugin-jsdoc`, `eslint-plugin-prettier`, and `prettier` peers were
|
|
34
|
-
removed because the exported presets do not load or configure them. Keeping unused peers forced unrelated
|
|
35
|
-
consumer installations and created false peer conflicts.
|
|
36
|
-
|
|
37
|
-
## Flat-config and validator corrections
|
|
38
|
-
|
|
39
|
-
- TypeScript parser/plugin resolution now comes from `typescript-eslint`.
|
|
40
|
-
- Angular TypeScript plugin, template plugin, and template parser resolution now comes from `angular-eslint`.
|
|
41
|
-
- An executable compatibility specification verifies that every configured Angular rule exists in the Angular
|
|
42
|
-
20 plugin objects and that the unified parser/plugin instances are the exported preset instances.
|
|
43
|
-
- Angular 20's template-complexity default is no longer inherited implicitly. The fleet policy is explicit:
|
|
44
|
-
warning severity at `maxComplexity: 20`. Legacy `ngFor` trackBy findings are warning-level while the existing
|
|
45
|
-
`no-ngfor-directive` migration warning remains active. Structural template defects such as duplicate attributes
|
|
46
|
-
remain errors.
|
|
47
|
-
- TypeScript 5.9's current ESLint visitor callback declarations are reflected in the direct unit-test invocations.
|
|
48
|
-
Runtime visitor behavior is unchanged; unused synthetic visitor arguments were removed from implementations.
|
|
49
|
-
- Package-local validator scripts now invoke their built entry points directly. The former scripts attempted to
|
|
50
|
-
execute the package's own bin alias before that alias existed in `node_modules/.bin`.
|
|
51
|
-
- Import and naming validators disable consumer inline config and filter only ESLint's corresponding informational
|
|
52
|
-
`noInlineConfig` diagnostic. Parse/runtime failures still fail hard.
|
|
53
|
-
- Validator discovery excludes `samples/**`, because the repository intentionally contains violating samples.
|
|
54
|
-
- Subprocess integration coverage proves that unknown consumer-only inline directives cannot contaminate either
|
|
55
|
-
isolated validator result.
|
|
56
|
-
|
|
57
|
-
## Verification evidence before commit
|
|
58
|
-
|
|
59
|
-
- TypeScript 5.9.3 compilation: PASS.
|
|
60
|
-
- Jasmine: 205 specifications, 0 failures.
|
|
61
|
-
- Angular 20 preset compatibility: 3 executable specifications, PASS.
|
|
62
|
-
- Import validator: 37 production source files, PASS.
|
|
63
|
-
- Naming validator: 37 production source files, PASS.
|
|
64
|
-
- Production dependency audit: 0 known vulnerabilities.
|
|
65
|
-
- Package dependency tree: all declared direct dependencies resolved.
|
|
66
|
-
- Initial package release: `1.20.2`. The first Angular 20 package consumer run then measured the changed implicit
|
|
67
|
-
template-complexity default and drove the explicit fleet policy correction released as `1.20.3`.
|
|
68
|
-
- Full repository ESLint: 0 errors and 2,728 legacy warning-level findings; the existing script exits successfully.
|
|
69
|
-
This release adds no warning-level release gate claim for the historical package source. The consumer release
|
|
70
|
-
gate remains zero-warning lint, and the upgraded presets must be proven there before BFR-CTM-006 resolution.
|
|
71
|
-
|
|
72
|
-
## Release and downstream gates
|
|
73
|
-
|
|
74
|
-
1. Complete two consecutive post-fix review passes.
|
|
75
|
-
2. Commit; inspect any version/dependency changes made by the Bedrock release hook.
|
|
76
|
-
3. Reinstall and rerun compilation, all specifications, validators, audit, and package manifest checks against the
|
|
77
|
-
post-hook tree.
|
|
78
|
-
4. Push and publish the exact reviewed commit.
|
|
79
|
-
5. Read back registry version, integrity, shasum, and peer dependencies.
|
|
80
|
-
6. Adopt the exact release in `@futdevpro/ngx-dynamo`, prove its Angular 20 sandbox and package gates, then publish
|
|
81
|
-
that Bedrock layer.
|
|
82
|
-
7. Adopt the exact Angular 20 Bedrock chain in Company Toolmaker and close BFR-CTM-006 only after the full consumer
|
|
83
|
-
security, browser, test, audit, and user-journey gates are green.
|
package/eslint.config.js
DELETED
|
@@ -1,15 +0,0 @@
|
|
|
1
|
-
const ntsPackageConfig = require('./build/configs/nts-package');
|
|
2
|
-
|
|
3
|
-
module.exports = [
|
|
4
|
-
...ntsPackageConfig,
|
|
5
|
-
{
|
|
6
|
-
files: ['**/*.ts', '**/*.tsx'],
|
|
7
|
-
languageOptions: {
|
|
8
|
-
parser: require('typescript-eslint').parser,
|
|
9
|
-
parserOptions: {
|
|
10
|
-
ecmaVersion: 'latest',
|
|
11
|
-
sourceType: 'module'
|
|
12
|
-
}
|
|
13
|
-
}
|
|
14
|
-
}
|
|
15
|
-
];
|
package/pnpm-workspace.yaml
DELETED
|
@@ -1,13 +0,0 @@
|
|
|
1
|
-
{
|
|
2
|
-
"eslint.run": "onSave",
|
|
3
|
-
"eslint.workingDirectories": [{ "mode": "auto" }],
|
|
4
|
-
"editor.codeActionsOnSave": {
|
|
5
|
-
"source.fixAll.eslint": "explicit"
|
|
6
|
-
},
|
|
7
|
-
"eslint.validate": [
|
|
8
|
-
"javascript",
|
|
9
|
-
"typescript"
|
|
10
|
-
],
|
|
11
|
-
"editor.formatOnSave": false,
|
|
12
|
-
"prettier.requireConfig": true
|
|
13
|
-
}
|
|
@@ -1,22 +0,0 @@
|
|
|
1
|
-
{
|
|
2
|
-
"name": "example-project",
|
|
3
|
-
"version": "1.0.0",
|
|
4
|
-
"scripts": {
|
|
5
|
-
"lint": "eslint src --ext .ts",
|
|
6
|
-
"lint:fix": "eslint src --ext .ts --fix",
|
|
7
|
-
"lint:fix-all": "eslint src --ext .ts --fix && dynamo-fix-return-types",
|
|
8
|
-
"validate:imports": "dynamo-validate-imports",
|
|
9
|
-
"validate:naming": "dynamo-validate-naming",
|
|
10
|
-
"audit:eslintrc": "dynamo-eslintrc-audit",
|
|
11
|
-
"fix": "dynamo-fix",
|
|
12
|
-
"fix:return-types": "dynamo-fix-return-types",
|
|
13
|
-
"test": "npm run lint && npm run validate:imports && npm run validate:naming"
|
|
14
|
-
},
|
|
15
|
-
"devDependencies": {
|
|
16
|
-
"@futdevpro/dynamo-eslint": "1.20.3",
|
|
17
|
-
"angular-eslint": "~20.7.0",
|
|
18
|
-
"eslint": "~9.39.5",
|
|
19
|
-
"typescript": "5.9.3",
|
|
20
|
-
"typescript-eslint": "~8.67.0"
|
|
21
|
-
}
|
|
22
|
-
}
|
package/samples/poc-sample.ts
DELETED
|
@@ -1,38 +0,0 @@
|
|
|
1
|
-
// POC Sample demonstrating Dynamo ESLint import ordering rules
|
|
2
|
-
// This file shows the correct import ordering according to the new rules
|
|
3
|
-
|
|
4
|
-
// Group 1: Non-FutDevPro packages
|
|
5
|
-
import { Component, OnInit } from '@angular/core';
|
|
6
|
-
import { Router } from '@angular/router';
|
|
7
|
-
import { HttpClient } from '@angular/common/http';
|
|
8
|
-
|
|
9
|
-
// Group 2: FutDevPro packages
|
|
10
|
-
import { DyFM_Error, DyFM_Log } from '@futdevpro/fsm-dynamo';
|
|
11
|
-
import { FDP_User, FDP_Account } from '@futdevpro/fdp-templates';
|
|
12
|
-
|
|
13
|
-
// Group 3: Scripts from other modules (3+ levels without underscore, or 3+ levels with underscore)
|
|
14
|
-
import { SomeService } from '../../../services/some.service';
|
|
15
|
-
import { AnotherService } from '../../../../models/another.interface';
|
|
16
|
-
|
|
17
|
-
// Group 4: Scripts from same module (up to 2 levels, or up to 3 levels with underscore)
|
|
18
|
-
import { LocalComponent } from './local.component';
|
|
19
|
-
import { LocalService } from '../_services/local.service';
|
|
20
|
-
import { LocalInterface } from '../../../_models/local.interface';
|
|
21
|
-
|
|
22
|
-
export class PocSampleComponent implements OnInit {
|
|
23
|
-
constructor(
|
|
24
|
-
private router: Router,
|
|
25
|
-
private http: HttpClient,
|
|
26
|
-
private localService: LocalService
|
|
27
|
-
) {}
|
|
28
|
-
|
|
29
|
-
ngOnInit(): void {
|
|
30
|
-
// Example usage
|
|
31
|
-
DyFM_Log.info('Component initialized');
|
|
32
|
-
}
|
|
33
|
-
|
|
34
|
-
// This would be flagged as errors:
|
|
35
|
-
// import { Something } from '../../../NPM-packages/some-package'; // ❌ Forbidden NPM-packages
|
|
36
|
-
// import { Something } from './some-file.js'; // ❌ Forbidden .js extension
|
|
37
|
-
// import type { SomeType } from './some-file'; // ❌ Forbidden import type
|
|
38
|
-
}
|
|
@@ -1,54 +0,0 @@
|
|
|
1
|
-
// POC Sample demonstrating Dynamo ESLint import ordering violations
|
|
2
|
-
// This file shows various violations that should be caught by the linter
|
|
3
|
-
|
|
4
|
-
// ❌ VIOLATIONS: This file contains multiple import ordering violations
|
|
5
|
-
|
|
6
|
-
// Wrong order: FutDevPro packages before non-FutDevPro packages
|
|
7
|
-
|
|
8
|
-
|
|
9
|
-
|
|
10
|
-
|
|
11
|
-
|
|
12
|
-
|
|
13
|
-
|
|
14
|
-
|
|
15
|
-
|
|
16
|
-
|
|
17
|
-
import { DyFM_Error } from '@futdevpro/fsm-dynamo';
|
|
18
|
-
import { Component } from '@angular/core';
|
|
19
|
-
|
|
20
|
-
// Missing empty line between groups
|
|
21
|
-
import { Router } from '@angular/router';
|
|
22
|
-
import { FDP_User } from '@futdevpro/fdp-templates';
|
|
23
|
-
|
|
24
|
-
// Wrong order: same module before other modules
|
|
25
|
-
|
|
26
|
-
|
|
27
|
-
|
|
28
|
-
|
|
29
|
-
|
|
30
|
-
|
|
31
|
-
|
|
32
|
-
|
|
33
|
-
|
|
34
|
-
|
|
35
|
-
import { LocalService } from './local.service';
|
|
36
|
-
|
|
37
|
-
|
|
38
|
-
|
|
39
|
-
|
|
40
|
-
|
|
41
|
-
|
|
42
|
-
|
|
43
|
-
|
|
44
|
-
|
|
45
|
-
import { SomeService } from '../../../services/some.service';
|
|
46
|
-
|
|
47
|
-
// ❌ Forbidden patterns:
|
|
48
|
-
import { Something } from '@futdevpro/some-package'; // Forbidden NPM-packages import
|
|
49
|
-
import { AnotherThing } from './some-file'; // Forbidden .js extension
|
|
50
|
-
import { SomeType } from './some-file'; // Forbidden import type
|
|
51
|
-
|
|
52
|
-
export class PocViolationsComponent {
|
|
53
|
-
// This class demonstrates various import violations
|
|
54
|
-
}
|
|
@@ -1,24 +0,0 @@
|
|
|
1
|
-
const { execSync } = require('child_process');
|
|
2
|
-
const path = require('path');
|
|
3
|
-
|
|
4
|
-
const configPath = path.join(__dirname, '../spec/support/jasmine.coverage.json');
|
|
5
|
-
const tsNodeRegister = require.resolve('ts-node/register/transpile-only');
|
|
6
|
-
|
|
7
|
-
const env = {
|
|
8
|
-
...process.env,
|
|
9
|
-
NODE_OPTIONS: `--require ${tsNodeRegister}`
|
|
10
|
-
};
|
|
11
|
-
|
|
12
|
-
try {
|
|
13
|
-
execSync(`npx jasmine --config="${configPath}"`, {
|
|
14
|
-
stdio: 'inherit',
|
|
15
|
-
cwd: path.join(__dirname, '..'),
|
|
16
|
-
shell: true,
|
|
17
|
-
env: env
|
|
18
|
-
});
|
|
19
|
-
} catch (error) {
|
|
20
|
-
// Exit with the same status code as jasmine (3 = test failures, 1 = other errors)
|
|
21
|
-
// This allows c8 to still generate coverage reports even if tests fail
|
|
22
|
-
process.exit(error.status || 1);
|
|
23
|
-
}
|
|
24
|
-
|
|
@@ -1,23 +0,0 @@
|
|
|
1
|
-
{
|
|
2
|
-
"reporters": [
|
|
3
|
-
{
|
|
4
|
-
"name": "jasmine-json-reporter",
|
|
5
|
-
"options": {
|
|
6
|
-
"pretty": true
|
|
7
|
-
}
|
|
8
|
-
}
|
|
9
|
-
],
|
|
10
|
-
"spec_dir": "src/",
|
|
11
|
-
"spec_files": [
|
|
12
|
-
"**/*.spec.ts",
|
|
13
|
-
"**/*[sS]pec.ts"
|
|
14
|
-
],
|
|
15
|
-
"helpers": [
|
|
16
|
-
"spec/support/helpers/ts-node-helper.js"
|
|
17
|
-
],
|
|
18
|
-
"env": {
|
|
19
|
-
"stopSpecOnExpectationFailure": false,
|
|
20
|
-
"random": true
|
|
21
|
-
}
|
|
22
|
-
}
|
|
23
|
-
|
|
@@ -1,24 +0,0 @@
|
|
|
1
|
-
{
|
|
2
|
-
"reporters": [
|
|
3
|
-
{
|
|
4
|
-
"name": "jasmine-json-reporter",
|
|
5
|
-
"options": {
|
|
6
|
-
"pretty": true
|
|
7
|
-
}
|
|
8
|
-
}
|
|
9
|
-
],
|
|
10
|
-
"spec_dir": "build/",
|
|
11
|
-
"spec_files": [
|
|
12
|
-
"**/*.spec.ts",
|
|
13
|
-
"**/*.spec.js",
|
|
14
|
-
"**/*[sS]pec.ts"
|
|
15
|
-
],
|
|
16
|
-
"helpers": [
|
|
17
|
-
"helpers/**/*.ts",
|
|
18
|
-
"helpers/**/*.js"
|
|
19
|
-
],
|
|
20
|
-
"env": {
|
|
21
|
-
"stopSpecOnExpectationFailure": false,
|
|
22
|
-
"random": true
|
|
23
|
-
}
|
|
24
|
-
}
|