click 'manually edit context' button moves items to input, but doesn'…

…t remove them from context list
sourcegraph · Oct 22, 2024 · 1288f43 · 1288f43
1 parent daa2bdf
commit 1288f43
Show file tree

Hide file tree

Showing 4 changed files with 1 addition and 21 deletions.
diff --git a/lib/shared/src/misc/rpc/webviewAPI.ts b/lib/shared/src/misc/rpc/webviewAPI.ts
@@ -87,11 +87,6 @@ export interface WebviewToExtensionAPI {
      * The current user's product subscription information (Cody Free/Pro).
      */
     userProductSubscription(): Observable<UserProductSubscription | null>
-
-    /**
-     * Move the fetched context into the previous input
-     */
-    clearFetchedContext(index: number): Observable<void>
 }
 
 export function createExtensionAPI(
@@ -119,7 +114,6 @@ export function createExtensionAPI(
         transcript: proxyExtensionAPI(messageAPI, 'transcript'),
         userHistory: proxyExtensionAPI(messageAPI, 'userHistory'),
         userProductSubscription: proxyExtensionAPI(messageAPI, 'userProductSubscription'),
-        clearFetchedContext: proxyExtensionAPI(messageAPI, 'clearFetchedContext'),
     }
 }
 

diff --git a/vscode/src/chat/chat-view/ChatController.ts b/vscode/src/chat/chat-view/ChatController.ts
@@ -1,7 +1,6 @@
 import {
     type ChatModel,
     type CodyClientConfig,
-    EMPTY,
     cenv,
     clientCapabilities,
     currentSiteVersion,
@@ -1668,11 +1667,6 @@ export class ChatController implements vscode.Disposable, vscode.WebviewViewProv
                         userProductSubscription.pipe(
                             map(value => (value === pendingOperation ? null : value))
                         ),
-                    clearFetchedContext: (index: number) => {
-                        this.chatBuilder.removeContextForHumanMessage(index)
-                        this.postViewTranscript()
-                        return EMPTY
-                    },
                 }
             )
         )

diff --git a/vscode/webviews/AppWrapperForTest.tsx b/vscode/webviews/AppWrapperForTest.tsx
@@ -123,7 +123,6 @@ export const AppWrapperForTest: FunctionComponent<{ children: ReactNode }> = ({
                             },
                         }),
                     userProductSubscription: () => Observable.of(null),
-                    clearFetchedContext: () => EMPTY,
                 },
             } satisfies Wrapper<ComponentProps<typeof ExtensionAPIProviderForTestsOnly>['value']>,
             {

diff --git a/vscode/webviews/chat/Transcript.tsx b/vscode/webviews/chat/Transcript.tsx
@@ -366,14 +366,7 @@ const TranscriptInteraction: FC<TranscriptInteractionProps> = memo(props => {
         }
         editor.removeMentions(item => item.type !== 'repository')
         editor.addMentions(contextFiles, undefined, 'before', '\n')
-
-        // Clear the listed context files
-        extensionAPI.clearFetchedContext(humanMessage.index).subscribe({
-            error: error => {
-                console.error('Error editing context', error)
-            },
-        })
-    }, [extensionAPI, humanMessage.contextFiles, humanMessage.index])
+    }, [humanMessage.contextFiles])
 
     return (
         <>