Fixed some @ts-ignore comments

nlkitai · May 11, 2024 · f5de8ac · f5de8ac
1 parent 10b8df7
commit f5de8ac
Show file tree

Hide file tree

Showing 10 changed files with 5 additions and 13 deletions.
diff --git a/packages/css/themes/modules.d.ts b/packages/css/themes/modules.d.ts
@@ -0,0 +1 @@
+declare module 'postcss-csso';
diff --git a/packages/css/themes/rollup.config.ts b/packages/css/themes/rollup.config.ts
@@ -1,7 +1,5 @@
-// @ts-ignore
 import csso from 'postcss-csso';
 import postcssImport from 'postcss-import';
-// @ts-ignore
 import postcssNested from 'postcss-nested';
 import {LogLevel, RollupOptions} from 'rollup';
 import postcss from 'rollup-plugin-postcss';

diff --git a/packages/extra/highlighter/modules.d.ts b/packages/extra/highlighter/modules.d.ts
@@ -0,0 +1 @@
+declare module 'postcss-csso';
diff --git a/packages/extra/highlighter/rollup.config.ts b/packages/extra/highlighter/rollup.config.ts
@@ -3,7 +3,6 @@ import {nodeResolve} from '@rollup/plugin-node-resolve';
 import replace from '@rollup/plugin-replace';
 import strip from '@rollup/plugin-strip';
 import terser from '@rollup/plugin-terser';
-// @ts-ignore
 import csso from 'postcss-csso';
 import postcssImport from 'postcss-import';
 import {LogLevel, RollupOptions} from 'rollup';

diff --git a/packages/js/core/src/exports/aiContext/aiContext.ts b/packages/js/core/src/exports/aiContext/aiContext.ts
@@ -156,7 +156,6 @@ class AiContextImpl implements AiContext {
 
             // Status can change to 'destroyed' if the context is destroyed while adapter.set is in progress.
             // In that case, we should not set the contextId and we should immediately clear the context.
-            // @ts-ignore
             if (this.status === 'destroyed') {
                 if (result.success) {
                     await this.theDataSyncService?.resetContextData();

diff --git a/packages/js/core/src/exports/aiContext/dataSyncService.ts b/packages/js/core/src/exports/aiContext/dataSyncService.ts
@@ -153,9 +153,8 @@ export class DataSyncService {
                     op.operation === 'update' &&
                     (op.data !== undefined || op.description !== undefined)
                 ) {
-                    // @ts-ignore
                     // We know that at least one of the two arguments is not undefined.
-                    acc[itemId] = {value: op.data, description: op.description};
+                    acc[itemId] = {value: op.data!, description: op.description!};
                 }
 
                 return acc;

diff --git a/packages/js/core/src/exports/aiContext/tasksService.ts b/packages/js/core/src/exports/aiContext/tasksService.ts
@@ -399,8 +399,6 @@ export class TasksService {
                     item.operation === 'update' &&
                     (item.description !== undefined || item.paramDescriptions !== undefined)
                 ) {
-                    // @ts-ignore
-                    // We know that at least one of the two arguments is not undefined.
                     const updateData: any = {};
                     if (item.description !== undefined) {
                         updateData.description = item.description;

diff --git a/packages/js/core/src/logic/chat/chatRoom/chatRoom.model.ts b/packages/js/core/src/logic/chat/chatRoom/chatRoom.model.ts
@@ -23,8 +23,8 @@ export class CompChatRoom<AiMsg> extends BaseComp<
     AiMsg, CompChatRoomProps<AiMsg>, CompChatRoomElements, CompChatRoomEvents, CompChatRoomActions
 > {
     private autoScrollController: AutoScrollController | undefined;
-    private conversation: CompConversation<AiMsg>;
-    private promptBoxInstance: CompPromptBox<AiMsg>;
+    private conversation!: CompConversation<AiMsg>;
+    private promptBoxInstance!: CompPromptBox<AiMsg>;
     private promptBoxText: string = '';
 
     constructor(context: ControllerContext<AiMsg>, {
@@ -68,7 +68,6 @@ export class CompChatRoom<AiMsg> extends BaseComp<
             promptBox?.submitShortcut,
         );
 
-        // @ts-ignore
         if (!this.conversation || !this.promptBoxInstance) {
             throw new Error('Conversation is not initialized');
         }

diff --git a/packages/js/langchain/rollup.config.ts b/packages/js/langchain/rollup.config.ts
@@ -5,7 +5,6 @@ import strip from '@rollup/plugin-strip';
 import terser from '@rollup/plugin-terser';
 import {RollupOptions} from 'rollup';
 import esbuild from 'rollup-plugin-esbuild';
-// @ts-ignore
 import {generateDts} from '../../../pipeline/utils/rollup/generateDts';
 import {generateOutputConfig} from '../../../pipeline/utils/rollup/generateOutputConfig';
 

diff --git a/packages/js/nlbridge/rollup.config.ts b/packages/js/nlbridge/rollup.config.ts
@@ -5,7 +5,6 @@ import strip from '@rollup/plugin-strip';
 import terser from '@rollup/plugin-terser';
 import {RollupOptions} from 'rollup';
 import esbuild from 'rollup-plugin-esbuild';
-// @ts-ignore
 import {generateDts} from '../../../pipeline/utils/rollup/generateDts';
 import {generateOutputConfig} from '../../../pipeline/utils/rollup/generateOutputConfig';