msg: working-completely;

sacrodge · sacrodge · commit dc3f6a2e7ab8 · 2025-10-13T18:06:17.000-07:00
- removed previous unnecesary edits
- removed example for now
diff --git a/example/src/commands.ts b/example/src/commands.ts
@@ -28,6 +28,5 @@ export enum Commands {
     CLEAR_CONTEXT_ITEMS = '/clear-context-items',
     CLEAR_LOGS = '/clear-logs',
     SHOW_CUSTOM_FORM = '/show-custom-form',
-    VOTE = '/vote',
-    MODIFIED_FILES_DEMO = '/modified-files-demo'
+    VOTE = '/vote'
 }
diff --git a/example/src/config.ts b/example/src/config.ts
@@ -283,11 +283,6 @@ export const QuickActionCommands: QuickActionCommandGroup[] = [
                 icon: MynahIcons.FLASH,
                 description: 'Test streaming animation with different speeds. Use tab bar buttons to switch modes.',
             },
-            {
-                command: Commands.MODIFIED_FILES_DEMO,
-                icon: MynahIcons.FILE,
-                description: 'Demo the modified files tracker component above the chat interface.',
-            },
         ],
     },
 ];
diff --git a/example/src/main.ts b/example/src/main.ts
@@ -1259,17 +1259,6 @@ here to see if it gets cut off properly as expected, with an ellipsis through cs
       `);
         },
         onChatPrompt: (tabId: string, prompt: ChatPrompt) => {
-            // Clear modified files tracker only for commands that modify files
-            const fileModifyingCommands = [
-                Commands.MODIFIED_FILES_DEMO,
-                '/dev',
-                '/transform',
-                '/generate'
-            ];
-            
-            if (prompt.command && fileModifyingCommands.includes(prompt.command)) {
-                mynahUI.clearModifiedFiles(tabId);
-            }
 
             Log(`New prompt on tab: <b>${tabId}</b><br/>
       prompt: <b>${prompt.prompt !== undefined && prompt.prompt !== '' ? prompt.prompt : '{command only}'}</b><br/>
@@ -1587,9 +1576,6 @@ here to see if it gets cut off properly as expected, with an ellipsis through cs
         onMessageDismiss: (tabId, messageId) => {
             Log(`Card dismissed: tabId: <b>${tabId}</b>, messageId: <b>${messageId}</b>`);
         },
-        onModifiedFileClick: (tabId, filePath) => {
-            Log(`Modified file clicked on tab <b>${tabId}</b>: <b>${filePath}</b>`);
-        },
     });
 
     setTimeout(() => {
@@ -1636,23 +1622,6 @@ here to see if it gets cut off properly as expected, with an ellipsis through cs
                 case Commands.VOTE:
                     mynahUI.addChatItem(tabId, exampleVoteChatItem);
                     mynahUI.addChatItem(tabId, defaultFollowUps);
-                    break;
-                case Commands.MODIFIED_FILES_DEMO:
-                    // Demo the modified files tracker
-                    mynahUI.addChatItem(tabId, {
-                        type: ChatItemType.ANSWER,
-                        messageId: generateUID(),
-                        body: 'Demonstrating the modified files tracker. Watch the component above the chat!',
-                    });
-                    
-                    // Demo will be handled through chatItem approach now
-                    mynahUI.addChatItem(tabId, {
-                        type: ChatItemType.ANSWER,
-                        messageId: generateUID(),
-                        body: 'Modified files tracker demo is now handled through chatItem.header.fileList approach.',
-                    });
-                    
-
                     break;
                 case Commands.CARD_WITH_MARKDOWN_LIST:
                     getGenerativeAIAnswer(tabId, sampleMarkdownList);
diff --git a/example/src/samples/sample-data.ts b/example/src/samples/sample-data.ts
@@ -681,10 +681,6 @@ export const defaultFollowUps: ChatItem = {
                 pillText: 'Some auto reply',
                 prompt: 'Some random auto reply here.',
             },
-            {
-                command: Commands.MODIFIED_FILES_DEMO,
-                pillText: 'Modified files demo',
-            },
         ],
     },
 };
@@ -2185,7 +2181,7 @@ export const mcpToolRunSampleCardInit:ChatItem = // Summary Card
                     filePaths: ['Running'],
                     details: {
                         'Running': {
-                            description: 'Work in progress...',
+                            description: 'Work in progress!',
                             icon: null,
                             labelIcon: 'progress',
                             labelIconForegroundStatus: 'info',
diff --git a/src/__test__/components/feedback-form/feedback-form-integration.spec.ts b/src/__test__/components/feedback-form/feedback-form-integration.spec.ts
@@ -358,7 +358,7 @@ describe('FeedbackForm Integration Tests', () => {
   describe('Error Handling', () => {
     it('should handle missing tab data gracefully', () => {
       // Override the mock for this specific test
-      (mockTabsStore.getTabDataStore as jest.Mock).mockReturnValueOnce(undefined as any);
+      mockTabsStore.getTabDataStore.mockReturnValueOnce(undefined as any);
 
       feedbackForm = new FeedbackForm();
 
diff --git a/src/components/__test__/chat-item/chat-item-card-content.spec.ts b/src/components/__test__/chat-item/chat-item-card-content.spec.ts
@@ -19,26 +19,28 @@ describe('ChatItemCardContent Animation Speed', () => {
   beforeEach(() => {
     jest.clearAllMocks();
 
-    (mockGetInstance as jest.Mock).mockReturnValue({
+    mockGetInstance.mockReturnValue({
+      // @ts-expect-error
       config: {
         typewriterStackTime: 100,
         typewriterMaxWordTime: 20,
         disableTypewriterAnimation: false,
       }
-    } as any);
+    });
 
     document.body.innerHTML = '<div id="test-container"></div>';
   });
 
   describe('Animation Configuration', () => {
     it('should use fast animation settings', () => {
-      (mockGetInstance as jest.Mock).mockReturnValue({
+      mockGetInstance.mockReturnValue({
+        // @ts-expect-error
         config: {
           typewriterStackTime: 100,
           typewriterMaxWordTime: 20,
           disableTypewriterAnimation: false,
         }
-      } as any);
+      });
 
       const props: ChatItemCardContentProps = {
         body: 'Test content',
@@ -51,11 +53,12 @@ describe('ChatItemCardContent Animation Speed', () => {
     });
 
     it('should disable animation when configured', () => {
-      (mockGetInstance as jest.Mock).mockReturnValue({
+      mockGetInstance.mockReturnValue({
+        // @ts-expect-error
         config: {
           disableTypewriterAnimation: true,
         }
-      } as any);
+      });
 
       const props: ChatItemCardContentProps = {
         body: 'Test content',
@@ -86,9 +89,10 @@ describe('ChatItemCardContent Animation Speed', () => {
 
   describe('Default Values', () => {
     it('should use defaults when config is empty', () => {
-      (mockGetInstance as jest.Mock).mockReturnValue({
+      mockGetInstance.mockReturnValue({
+        // @ts-expect-error
         config: {}
-      } as any);
+      });
 
       const props: ChatItemCardContentProps = {
         body: 'Test content',
diff --git a/src/components/chat-item/chat-item-card.ts b/src/components/chat-item/chat-item-card.ts
@@ -320,19 +320,11 @@ export class ChatItemCard {
             if (header.fileList?.details?.[filePath]?.clickable === false) {
               return;
             }
-            console.log('[ChatItemCard] File pill clicked:', {
-              tabId: this.props.tabId,
-              messageId: this.props.chatItem.messageId,
-              filePath,
-              deleted: isDeleted,
-              fileDetails: header.fileList?.details?.[filePath]
-            });
             MynahUIGlobalEvents.getInstance().dispatch(MynahEventNames.FILE_CLICK, {
               tabId: this.props.tabId,
               messageId: this.props.chatItem.messageId,
               filePath,
-              deleted: isDeleted,
-              fileDetails: header.fileList?.details?.[filePath]
+              deleted: isDeleted
             });
           },
           ...(description !== undefined
@@ -813,13 +805,6 @@ export class ChatItemCard {
         formItems: this.chatFormItems,
         buttons: [],
         onActionClick: action => {
-          console.log('[ChatItemCard] Button clicked:', {
-            tabId: this.props.tabId,
-            messageId: this.props.chatItem.messageId,
-            actionId: action.id,
-            actionText: action.text,
-            formItemValues: this.chatFormItems !== null ? this.chatFormItems.getAllValues() : {}
-          });
           MynahUIGlobalEvents.getInstance().dispatch(MynahEventNames.BODY_ACTION_CLICKED, {
             tabId: this.props.tabId,
             messageId: this.props.chatItem.messageId,
diff --git a/src/components/chat-item/chat-item-tree-file.ts b/src/components/chat-item/chat-item-tree-file.ts
@@ -42,14 +42,9 @@ export class ChatItemTreeFile {
         click: () => {
           this.hideTooltip();
           if (this.props.details?.clickable !== false) {
-            const fileMessageId = this.props.details?.data?.messageId ?? this.props.messageId;
-            console.log('[ChatItemTreeFile] File clicked - originalFilePath:', this.props.originalFilePath);
-            console.log('[ChatItemTreeFile] File clicked - details.data.fullPath:', this.props.details?.data?.fullPath);
-            console.log('[ChatItemTreeFile] File clicked - details.description:', this.props.details?.description);
-            console.log('[ChatItemTreeFile] File clicked - using messageId:', fileMessageId);
             MynahUIGlobalEvents.getInstance().dispatch(MynahEventNames.FILE_CLICK, {
               tabId: this.props.tabId,
-              messageId: fileMessageId,
+              messageId: this.props.messageId,
               filePath: this.props.originalFilePath,
               deleted: this.props.deleted,
               fileDetails: this.props.details
@@ -144,17 +139,9 @@ export class ChatItemTreeFile {
                 onClick: (e) => {
                   cancelEvent(e);
                   this.hideTooltip();
-                  const fileMessageId = this.props.details?.data?.messageId ?? this.props.messageId;
-                  console.log('[ChatItemTreeFile] File action clicked:', {
-                    tabId: this.props.tabId,
-                    messageId: fileMessageId,
-                    filePath: this.props.originalFilePath,
-                    actionName: action.name,
-                    actionDetails: action
-                  });
                   MynahUIGlobalEvents.getInstance().dispatch(MynahEventNames.FILE_ACTION_CLICK, {
                     tabId: this.props.tabId,
-                    messageId: fileMessageId,
+                    messageId: this.props.messageId,
                     filePath: this.props.originalFilePath,
                     actionName: action.name,
                   });
diff --git a/src/main.ts b/src/main.ts

Original file line number	Diff line number	Diff line change
`@@ -28,6 +28,5 @@ export enum Commands {`
`28`	`28`	`CLEAR_CONTEXT_ITEMS = '/clear-context-items',`
`29`	`29`	`CLEAR_LOGS = '/clear-logs',`
`30`	`30`	`SHOW_CUSTOM_FORM = '/show-custom-form',`
`31`		`- VOTE = '/vote',`
`32`		`- MODIFIED_FILES_DEMO = '/modified-files-demo'`
	`31`	`+ VOTE = '/vote'`
`33`	`32`	`}`