Skip to content

Commit 76029ca

Browse files
AlemTuzlakriccardoperra
authored andcommitted
fix: resolve lint, knip, and sherif issues
- Fix array-type ESLint rule in angular-devtools - Fix import/consistent-type-specifier-style in devtools-utils angular - Sort peerDependencies alphabetically (sherif) - Remove unused devDependencies flagged by knip - Remove angularCompilerOptions from tsconfig (not using ngc)
1 parent 7c18c00 commit 76029ca

6 files changed

Lines changed: 11 additions & 25 deletions

File tree

packages/angular-devtools/package.json

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -52,11 +52,7 @@
5252
},
5353
"devDependencies": {
5454
"@angular/compiler": "^20.0.0",
55-
"@angular/compiler-cli": "^20.0.0",
56-
"@angular/core": "^20.0.0",
57-
"@angular/platform-browser": "^20.0.0",
58-
"@angular/platform-browser-dynamic": "^20.0.0",
59-
"zone.js": "^0.15.0"
55+
"@angular/core": "^20.0.0"
6056
},
6157
"peerDependencies": {
6258
"@angular/core": ">=19.0.0",

packages/angular-devtools/src/devtools.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ export class TanStackDevtoolsComponent {
3333
private injector = inject(EnvironmentInjector)
3434
private destroyRef = inject(DestroyRef)
3535

36-
private componentRefs: ComponentRef<any>[] = []
36+
private componentRefs: Array<ComponentRef<any>> = []
3737
private devtools: TanStackDevtoolsCore | null = null
3838

3939
constructor() {

packages/angular-devtools/tsconfig.json

Lines changed: 1 addition & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -3,12 +3,5 @@
33
"compilerOptions": {
44
"experimentalDecorators": true
55
},
6-
"include": ["src", "eslint.config.js", "vite.config.ts", "tests"],
7-
"angularCompilerOptions": {
8-
"enableI18nLegacyMessageIdFormat": false,
9-
"strictInjectionParameters": true,
10-
"strictInputAccessModifiers": true,
11-
"strictTemplates": true,
12-
"compilationMode": "partial"
13-
}
6+
"include": ["src", "eslint.config.js", "vite.config.ts", "tests"]
147
}

packages/devtools-utils/package.json

Lines changed: 5 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -79,14 +79,17 @@
7979
"@tanstack/devtools-ui": "workspace:^"
8080
},
8181
"peerDependencies": {
82+
"@angular/core": ">=19.0.0",
8283
"@types/react": ">=17.0.0",
8384
"preact": ">=10.0.0",
8485
"react": ">=17.0.0",
8586
"solid-js": ">=1.9.7",
86-
"vue": ">=3.2.0",
87-
"@angular/core": ">=19.0.0"
87+
"vue": ">=3.2.0"
8888
},
8989
"peerDependenciesMeta": {
90+
"@angular/core": {
91+
"optional": true
92+
},
9093
"@types/react": {
9194
"optional": true
9295
},
@@ -101,9 +104,6 @@
101104
},
102105
"vue": {
103106
"optional": true
104-
},
105-
"@angular/core": {
106-
"optional": true
107107
}
108108
},
109109
"files": [
@@ -121,10 +121,6 @@
121121
"build": "vite build && vite build --config vite.config.preact.ts && vite build --config vite.config.vue.ts && vite build --config vite.config.solid-class.ts && vite build --config vite.config.angular.ts && tsup"
122122
},
123123
"devDependencies": {
124-
"@angular/compiler": "^20.0.0",
125-
"@angular/compiler-cli": "^20.0.0",
126-
"@angular/core": "^20.0.0",
127-
"@angular/platform-browser": "^20.0.0",
128124
"tsup": "^8.5.0",
129125
"tsup-preset-solid": "^2.2.0",
130126
"vite-plugin-solid": "^2.11.8"

packages/devtools-utils/src/angular/panel.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,8 +5,8 @@ import {
55
afterNextRender,
66
inject,
77
input,
8-
type Type,
98
} from '@angular/core'
9+
import type { Type } from '@angular/core'
1010

1111
export interface DevtoolsPanelProps {
1212
theme?: 'dark' | 'light' | 'system'

packages/devtools-utils/src/angular/plugin.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
1-
import { Component, type Type } from '@angular/core'
1+
import { Component } from '@angular/core'
2+
import type { Type } from '@angular/core'
23

34
@Component({
45
selector: 'noop-component',

0 commit comments

Comments
 (0)