rm completions cache

sourcegraph · Aug 2, 2023 · 33fc664 · 33fc664
1 parent cc4388b
commit 33fc664
Show file tree

Hide file tree

Showing 13 changed files with 12 additions and 435 deletions.
diff --git a/lib/shared/src/configuration.ts b/lib/shared/src/configuration.ts
@@ -24,7 +24,6 @@ export interface Configuration {
         | 'unstable-azure-openai'
     autocompleteAdvancedServerEndpoint: string | null
     autocompleteAdvancedAccessToken: string | null
-    autocompleteAdvancedCache: boolean
     autocompleteAdvancedEmbeddings: boolean
     autocompleteExperimentalCompleteSuggestWidgetSelection?: boolean
     pluginsEnabled?: boolean

diff --git a/vscode/package.json b/vscode/package.json
@@ -885,11 +885,6 @@
           "type": "string",
           "markdownDescription": "Overwrite the access token used for code autocomplete. This is only supported with a provider other than `anthropic`."
         },
-        "cody.autocomplete.advanced.cache": {
-          "type": "boolean",
-          "default": true,
-          "markdownDescription": "Enables caching of code autocomplete."
-        },
         "cody.autocomplete.advanced.embeddings": {
           "order": 99,
           "type": "boolean",

diff --git a/vscode/src/completions/cache.test.ts b/vscode/src/completions/cache.test.ts
diff --git a/vscode/src/completions/cache.ts b/vscode/src/completions/cache.ts
diff --git a/vscode/src/completions/request-manager.test.ts b/vscode/src/completions/request-manager.test.ts
@@ -2,7 +2,6 @@ import { beforeEach, describe, expect, it, vi } from 'vitest'
 
 import { vsCodeMocks } from '../testutils/mocks'
 
-import { CompletionsCache } from './cache'
 import { Provider } from './providers/provider'
 import { RequestManager } from './request-manager'
 import { Completion } from './types'
@@ -51,8 +50,7 @@ function createProvider(prefix: string) {
 describe('RequestManager', () => {
     let createRequest: (prefix: string, provider: Provider) => Promise<Completion[]>
     beforeEach(() => {
-        const cache = new CompletionsCache()
-        const requestManager = new RequestManager(cache)
+        const requestManager = new RequestManager()
 
         createRequest = (prefix: string, provider: Provider) =>
             requestManager.request(DOCUMENT_URI, LOG_ID, prefix, [provider], [], new AbortController().signal)
@@ -98,25 +96,4 @@ describe('RequestManager', () => {
         expect((await promise1)[0].content).toBe('log();')
         expect(provider1.didFinishNetworkRequest).toBe(true)
     })
-
-    it('serves request from cache when a prior request resolves', async () => {
-        const prefix1 = 'console.'
-        const provider1 = createProvider(prefix1)
-        const promise1 = createRequest(prefix1, provider1)
-
-        const prefix2 = 'console.log('
-        const provider2 = createProvider(prefix2)
-        const promise2 = createRequest(prefix2, provider2)
-
-        provider1.resolveRequest(["log('hello')"])
-
-        expect((await promise1)[0].content).toBe("log('hello')")
-        expect((await promise2)[0].content).toBe("'hello')")
-
-        expect(provider1.didFinishNetworkRequest).toBe(true)
-        expect(provider2.didFinishNetworkRequest).toBe(false)
-
-        // Ensure that the completed network request does not cause issues
-        provider2.resolveRequest(["'world')"])
-    })
 })