add state transition

sourcegraph · Jan 10, 2025 · 4f6c6aa · 4f6c6aa
1 parent cd86320
commit 4f6c6aa
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/vscode/src/autoedits/analytics-logger/analytics-logger.ts b/vscode/src/autoedits/analytics-logger/analytics-logger.ts
@@ -86,7 +86,7 @@ const validRequestTransitions = {
     started: ['contextLoaded', 'discarded'],
     contextLoaded: ['loaded', 'discarded'],
     loaded: ['postProcessed', 'discarded'],
-    postProcessed: ['suggested'],
+    postProcessed: ['suggested', 'discarded'],
     suggested: ['read', 'accepted', 'rejected'],
     read: ['accepted', 'rejected'],
     accepted: [],

diff --git a/vscode/src/autoedits/renderer/decorators/default-decorator.ts b/vscode/src/autoedits/renderer/decorators/default-decorator.ts
@@ -211,7 +211,9 @@ export class DefaultDecorator implements AutoEditsDecorator {
         if (addedLinesInfo.length === 0) {
             return { removedRangesInfo: removedRanges }
         }
-        const oldLines = addedLinesInfo.map(info => this.editor.document.lineAt(info.afterLine))
+        const oldLines = addedLinesInfo
+            .filter(info => info.afterLine < this.editor.document.lineCount)
+            .map(info => this.editor.document.lineAt(info.afterLine))
         const replacerCol = Math.max(...oldLines.map(line => line.range.end.character))
         const startLine = Math.min(...oldLines.map(line => line.lineNumber))