Skip to content

Commit 8a2eee5

Browse files
authored
fix: temporarily disable RecentlyVisited, RecentlyEdited, and DocumentHistoryTracker (#8391)
fix: disable expenseive autocommplete and next-edit services
1 parent 4305358 commit 8a2eee5

File tree

3 files changed

+41
-46
lines changed

3 files changed

+41
-46
lines changed

extensions/vscode/src/autocomplete/RecentlyVisitedRangesService.ts

Lines changed: 12 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,6 @@ import {
44
AutocompleteSnippetType,
55
} from "core/autocomplete/snippets/types";
66
import { isSecurityConcern } from "core/indexing/ignore";
7-
import { PosthogFeatureFlag, Telemetry } from "core/util/posthog";
87
import { LRUCache } from "lru-cache";
98
import * as vscode from "vscode";
109

@@ -34,19 +33,18 @@ export class RecentlyVisitedRangesService {
3433
}
3534

3635
private async initWithPostHog() {
37-
const recentlyVisitedRangesNumSurroundingLines =
38-
await Telemetry.getValueForFeatureFlag(
39-
PosthogFeatureFlag.RecentlyVisitedRangesNumSurroundingLines,
40-
);
41-
42-
if (recentlyVisitedRangesNumSurroundingLines) {
43-
this.isEnabled = true;
44-
this.numSurroundingLines = recentlyVisitedRangesNumSurroundingLines;
45-
}
46-
47-
vscode.window.onDidChangeTextEditorSelection(
48-
this.cacheCurrentSelectionContext,
49-
);
36+
// TODO merge this and re-enable https://github.com/continuedev/continue/pull/8364
37+
// const recentlyVisitedRangesNumSurroundingLines =
38+
// await Telemetry.getValueForFeatureFlag(
39+
// PosthogFeatureFlag.RecentlyVisitedRangesNumSurroundingLines,
40+
// );
41+
// if (recentlyVisitedRangesNumSurroundingLines) {
42+
// this.isEnabled = true;
43+
// this.numSurroundingLines = recentlyVisitedRangesNumSurroundingLines;
44+
// }
45+
// vscode.window.onDidChangeTextEditorSelection(
46+
// this.cacheCurrentSelectionContext,
47+
// );
5048
}
5149

5250
private cacheCurrentSelectionContext = async (

extensions/vscode/src/autocomplete/recentlyEdited.ts

Lines changed: 18 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -24,25 +24,24 @@ export class RecentlyEditedTracker {
2424
private static maxRecentlyEditedDocuments = 10;
2525

2626
constructor(private ideUtils: VsCodeIdeUtils) {
27-
vscode.workspace.onDidChangeTextDocument((event) => {
28-
event.contentChanges.forEach((change) => {
29-
const editedRange = {
30-
uri: event.document.uri,
31-
range: new vscode.Range(
32-
new vscode.Position(change.range.start.line, 0),
33-
new vscode.Position(change.range.end.line + 1, 0),
34-
),
35-
timestamp: Date.now(),
36-
};
37-
this.insertRange(editedRange);
38-
});
39-
40-
this.insertDocument(event.document.uri);
41-
});
42-
43-
setInterval(() => {
44-
this.removeOldEntries();
45-
}, 1000 * 15);
27+
// TODO merge this and re-enable https://github.com/continuedev/continue/pull/8364
28+
// vscode.workspace.onDidChangeTextDocument((event) => {
29+
// event.contentChanges.forEach((change) => {
30+
// const editedRange = {
31+
// uri: event.document.uri,
32+
// range: new vscode.Range(
33+
// new vscode.Position(change.range.start.line, 0),
34+
// new vscode.Position(change.range.end.line + 1, 0),
35+
// ),
36+
// timestamp: Date.now(),
37+
// };
38+
// this.insertRange(editedRange);
39+
// });
40+
// this.insertDocument(event.document.uri);
41+
// });
42+
// setInterval(() => {
43+
// this.removeOldEntries();
44+
// }, 1000 * 15);
4645
}
4746

4847
private async insertRange(

extensions/vscode/src/extension/VsCodeExtension.ts

Lines changed: 11 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -44,13 +44,10 @@ import { VsCodeIde } from "../VsCodeIde";
4444
import { ConfigYamlDocumentLinkProvider } from "./ConfigYamlDocumentLinkProvider";
4545
import { VsCodeMessenger } from "./VsCodeMessenger";
4646

47-
import { getAst } from "core/autocomplete/util/ast";
4847
import { modelSupportsNextEdit } from "core/llm/autodetect";
4948
import { NEXT_EDIT_MODELS } from "core/llm/constants";
50-
import { DocumentHistoryTracker } from "core/nextEdit/DocumentHistoryTracker";
5149
import { NextEditProvider } from "core/nextEdit/NextEditProvider";
5250
import { isNextEditTest } from "core/nextEdit/utils";
53-
import { localPathOrUriToPath } from "core/util/pathToUri";
5451
import { JumpManager } from "../activation/JumpManager";
5552
import setupNextEditWindowManager, {
5653
NextEditWindowManager,
@@ -534,16 +531,17 @@ export class VsCodeExtension {
534531
});
535532
});
536533

537-
vscode.workspace.onDidOpenTextDocument(async (event) => {
538-
const ast = await getAst(event.fileName, event.getText());
539-
if (ast) {
540-
DocumentHistoryTracker.getInstance().addDocument(
541-
localPathOrUriToPath(event.fileName),
542-
event.getText(),
543-
ast,
544-
);
545-
}
546-
});
534+
// TODO merge this and re-enable https://github.com/continuedev/continue/pull/8364
535+
// vscode.workspace.onDidOpenTextDocument(async (event) => {
536+
// const ast = await getAst(event.fileName, event.getText());
537+
// if (ast) {
538+
// DocumentHistoryTracker.getInstance().addDocument(
539+
// localPathOrUriToPath(event.fileName),
540+
// event.getText(),
541+
// ast,
542+
// );
543+
// }
544+
// });
547545

548546
// When GitHub sign-in status changes, reload config
549547
vscode.authentication.onDidChangeSessions(async (e) => {

0 commit comments

Comments
 (0)