chore: update maintenance dependencies #150
+216
−49
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,84 @@ | ||
| import { FlatCompat } from '@eslint/eslintrc'; | ||
| import js from '@eslint/js'; | ||
| import tsEslintPlugin from '@typescript-eslint/eslint-plugin'; | ||
| import { createRequire } from 'node:module'; | ||
| import path from 'node:path'; | ||
| import { fileURLToPath } from 'node:url'; | ||
|
|
||
| const __filename = fileURLToPath(import.meta.url); | ||
| const __dirname = path.dirname(__filename); | ||
| const require = createRequire(import.meta.url); | ||
|
|
||
| const compat = new FlatCompat({ | ||
| baseDirectory: __dirname, | ||
| recommendedConfig: js.configs.recommended, | ||
| allConfig: js.configs.all, | ||
| }); | ||
|
|
||
| const recommendedTsRules = new Set( | ||
| Object.keys(tsEslintPlugin.configs.recommended.rules || {}), | ||
| ); | ||
| const noopRule = { | ||
| meta: { type: 'problem', docs: {}, schema: [] }, | ||
| create: () => ({}), | ||
| }; | ||
|
|
||
| function normalizeConfig(config) { | ||
| const next = { ...config }; | ||
|
|
||
| if (next.plugins?.['@typescript-eslint']) { | ||
| next.plugins = { | ||
| ...next.plugins, | ||
| '@typescript-eslint': { | ||
| ...next.plugins['@typescript-eslint'], | ||
| rules: { | ||
| ...next.plugins['@typescript-eslint'].rules, | ||
| 'ban-types': noopRule, | ||
| }, | ||
| }, | ||
| }; | ||
| } | ||
|
|
||
| if (next.rules) { | ||
| next.rules = Object.fromEntries( | ||
| Object.entries(next.rules).filter(([ruleName]) => { | ||
| if (!ruleName.startsWith('@typescript-eslint/')) { | ||
| return true; | ||
| } | ||
| return ( | ||
| recommendedTsRules.has(ruleName) || | ||
| ruleName === '@typescript-eslint/ban-types' | ||
| ); | ||
| }), | ||
| ); | ||
| } | ||
|
|
||
| return next; | ||
| } | ||
|
|
||
| export default [ | ||
| { | ||
| ignores: [ | ||
| 'node_modules/', | ||
| 'coverage/', | ||
| 'es/', | ||
| 'lib/', | ||
| 'dist/', | ||
| 'docs-dist/', | ||
| '.dumi/', | ||
| '.doc/', | ||
| '.vercel/', | ||
| '.eslintrc.js', | ||
| 'src/index.d.ts', | ||
| ], | ||
| }, | ||
| ...compat.config(require('./.eslintrc.js')).map(normalizeConfig), | ||
| { | ||
| rules: { | ||
| '@typescript-eslint/ban-types': 'off', | ||
| '@typescript-eslint/no-empty-object-type': 'off', | ||
| '@typescript-eslint/no-unsafe-function-type': 'off', | ||
| '@typescript-eslint/no-unused-vars': 'off', | ||
| }, | ||
| }, | ||
| ]; | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| @@ -0,0 +1,67 @@ | ||||||||||||||||||||||||
| /// <reference types="jest" /> | ||||||||||||||||||||||||
| /// <reference types="node" /> | ||||||||||||||||||||||||
| /// <reference types="react" /> | ||||||||||||||||||||||||
| /// <reference types="react-dom" /> | ||||||||||||||||||||||||
| /// <reference types="@testing-library/jest-dom" /> | ||||||||||||||||||||||||
|
|
||||||||||||||||||||||||
| declare module '*.css'; | ||||||||||||||||||||||||
| declare module '*.less'; | ||||||||||||||||||||||||
| declare module 'jsonp'; | ||||||||||||||||||||||||
|
|
||||||||||||||||||||||||
| declare namespace JSX { | ||||||||||||||||||||||||
| type Element = React.JSX.Element; | ||||||||||||||||||||||||
| interface ElementClass extends React.JSX.ElementClass {} | ||||||||||||||||||||||||
| interface ElementAttributesProperty | ||||||||||||||||||||||||
| extends React.JSX.ElementAttributesProperty {} | ||||||||||||||||||||||||
| interface ElementChildrenAttribute | ||||||||||||||||||||||||
| extends React.JSX.ElementChildrenAttribute {} | ||||||||||||||||||||||||
| type LibraryManagedAttributes<C, P> = React.JSX.LibraryManagedAttributes< | ||||||||||||||||||||||||
| C, | ||||||||||||||||||||||||
| P | ||||||||||||||||||||||||
| >; | ||||||||||||||||||||||||
| interface IntrinsicAttributes extends React.JSX.IntrinsicAttributes {} | ||||||||||||||||||||||||
| interface IntrinsicClassAttributes<T> extends React.JSX | ||||||||||||||||||||||||
| .IntrinsicClassAttributes<T> {} | ||||||||||||||||||||||||
| interface IntrinsicElements extends React.JSX.IntrinsicElements {} | ||||||||||||||||||||||||
| } | ||||||||||||||||||||||||
|
|
||||||||||||||||||||||||
| declare namespace jest { | ||||||||||||||||||||||||
| interface Matchers<R> { | ||||||||||||||||||||||||
| lastCalledWith(...expected: unknown[]): R; | ||||||||||||||||||||||||
| nthCalledWith(nthCall: number, ...expected: unknown[]): R; | ||||||||||||||||||||||||
| toBeCalled(): R; | ||||||||||||||||||||||||
| toBeCalledTimes(expected: number): R; | ||||||||||||||||||||||||
| toBeCalledWith(...expected: unknown[]): R; | ||||||||||||||||||||||||
| } | ||||||||||||||||||||||||
| } | ||||||||||||||||||||||||
|
|
||||||||||||||||||||||||
| declare const vi: { | ||||||||||||||||||||||||
| fn: <T extends (...args: any[]) => any = (...args: any[]) => any>( | ||||||||||||||||||||||||
| implementation?: T, | ||||||||||||||||||||||||
| ) => jest.MockedFunction<T>; | ||||||||||||||||||||||||
| mock: ( | ||||||||||||||||||||||||
| moduleName: string, | ||||||||||||||||||||||||
| factory?: (importOriginal: <T>() => Promise<T>) => unknown, | ||||||||||||||||||||||||
| ) => void; | ||||||||||||||||||||||||
| spyOn: typeof jest.spyOn; | ||||||||||||||||||||||||
| useFakeTimers: () => void; | ||||||||||||||||||||||||
| useRealTimers: () => void; | ||||||||||||||||||||||||
| advanceTimersByTime: (msToRun: number) => void; | ||||||||||||||||||||||||
| clearAllTimers: () => void; | ||||||||||||||||||||||||
| runAllTimers: () => void; | ||||||||||||||||||||||||
| importActual: <T>(moduleName: string) => Promise<T>; | ||||||||||||||||||||||||
| clearAllMocks: () => void; | ||||||||||||||||||||||||
| resetAllMocks: () => void; | ||||||||||||||||||||||||
| restoreAllMocks: () => void; | ||||||||||||||||||||||||
| }; | ||||||||||||||||||||||||
|
|
||||||||||||||||||||||||
| declare const describe: any; | ||||||||||||||||||||||||
| declare const it: any; | ||||||||||||||||||||||||
| declare const test: any; | ||||||||||||||||||||||||
| declare const beforeEach: any; | ||||||||||||||||||||||||
| declare const afterEach: any; | ||||||||||||||||||||||||
| declare const beforeAll: any; | ||||||||||||||||||||||||
| declare const afterAll: any; | ||||||||||||||||||||||||
| declare const expect: any; | ||||||||||||||||||||||||
|
|
||||||||||||||||||||||||
| declare module 'moment/locale/zh-cn'; | ||||||||||||||||||||||||
|
There was a problem hiding this comment. Declaring these Jest globals as any overrides the strongly-typed definitions provided by @types/jest (referenced on line 1). This disables type safety in your test files. Since @types/jest is already referenced, these declarations are redundant and should be removed.
Suggested change
|
||||||||||||||||||||||||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,21 @@ | ||
| import * as React from 'react'; | ||
|
|
||
| declare module 'react' { | ||
| type ReactText = string | number; | ||
| function useRef<T = undefined>(): React.MutableRefObject<T | undefined>; | ||
| function isValidElement<P = any>( | ||
| object: {} | null | undefined, | ||
| ): object is React.ReactElement<P>; | ||
| function cloneElement<P = any>( | ||
| element: React.ReactElement<P>, | ||
| props?: (Partial<P> & React.Attributes) | null, | ||
| ...children: React.ReactNode[] | ||
| ): React.ReactElement<P>; | ||
| } | ||
|
Comment on lines
+3
to
+14
There was a problem hiding this comment. Monkey-patching the global react module to restore removed types (like ReactText) or override standard signatures (like useRef, isValidElement, cloneElement) is fragile and can cause type conflicts or mask actual type errors. It is highly recommended to update the codebase to be fully compatible with React 19 types (e.g., replacing ReactText with string | number directly) rather than using global overrides. |
||
|
|
||
| declare module 'react-dom' { | ||
| function hydrate( | ||
| element: React.ReactNode, | ||
| container: Element | DocumentFragment, | ||
| ): void; | ||
| } | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The normalizeConfig function dynamically filters and overrides rules from the legacy .eslintrc.js to make it compatible with ESLint 9 and @typescript-eslint v8. This runtime patching is complex and fragile. It is highly recommended to fully migrate the configuration in .eslintrc.js to the new flat config format in eslint.config.mjs directly, which would eliminate the need for FlatCompat and this custom normalization logic.