Mirrored_Repos/InvokeAI
1
0
Fork 0
mirror of https://github.com/invoke-ai/InvokeAI synced 2024-08-30 20:32:17 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(ui): improve IDE TS performance by not resolving JSON

Browse Source 
The TS Language Server slows down immensely with our translation JSON, which is used to provide kinda-type-safe translation keys. I say "kinda", because you don't get autocomplete - you only get red squigglies when the key is incorrect.

To improve the performance, we can opt out of this process entirely, at the cost of no red squigglies for translation keys. Hopefully we can resolve this in the future.

It's not clear why this became an issue only recently (like past couple weeks). We've tried rolling back the app dependencies, VSCode extensions, VSCode itself, and the TS version to before the time when the issue started, but nothing seems to improve the performance.

1. Disable `resolveJsonModule` in `tsconfig.json`
2. Ignore TS in `i18n.ts` when importing the JSON
3. Comment out the custom types in `i18.d.ts` entirely

It's possible that only `3` is needed to fix the issue.

I've tested building the app and running the build - it works fine, and translation works fine.
This commit is contained in:
psychedelicious 2023-06-29 11:24:20 +10:00 committed by Kent Keirsey
parent 8f6b3660c5
commit 4308d593c3
3 changed files with 22 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

33
invokeai/frontend/web/src/i18.d.ts vendored
View File

@ -1,17 +1,20 @@
import 'i18next'; // TODO: Disabled for IDE performance issues with our translation JSON
import en from '../public/locales/en.json'; // import 'i18next';
declare module 'i18next' { // import en from '../public/locales/en.json';
// Extend CustomTypeOptions
interface CustomTypeOptions { // declare module 'i18next' {
// Setting Default Namespace As English // // Extend CustomTypeOptions
defaultNS: 'en'; // interface CustomTypeOptions {
// Custom Types For Resources // // Setting Default Namespace As English
resources: { // defaultNS: 'en';
en: typeof en; // // Custom Types For Resources
}; // resources: {
// Never Return Null // en: typeof en;
returnNull: false; // };
} // // Never Return Null
} // returnNull: false;
// }
// }
export default {};

2
invokeai/frontend/web/src/i18n.ts
View File

@ -3,6 +3,8 @@ import LanguageDetector from 'i18next-browser-languagedetector';
import Backend from 'i18next-http-backend'; import Backend from 'i18next-http-backend';
import { initReactI18next } from 'react-i18next'; import { initReactI18next } from 'react-i18next';
// TODO: Disabled for IDE performance issues with our translation JSON
// @ts-ignore
import translationEN from '../public/locales/en.json'; import translationEN from '../public/locales/en.json';
import { LOCALSTORAGE_PREFIX } from 'app/store/constants'; import { LOCALSTORAGE_PREFIX } from 'app/store/constants';

3
invokeai/frontend/web/tsconfig.json
View File

@ -11,7 +11,8 @@
"forceConsistentCasingInFileNames": true, "forceConsistentCasingInFileNames": true,
"module": "ESNext", "module": "ESNext",
"moduleResolution": "Node", "moduleResolution": "Node",
"resolveJsonModule": true, // TODO: Disabled for IDE performance issues with our translation JSON
// "resolveJsonModule": true,
"isolatedModules": true, "isolatedModules": true,
"noEmit": true, "noEmit": true,
"jsx": "react-jsx", "jsx": "react-jsx",