fix: don't reset history cursor while navigating history with attachments (microsoft#223800)

joyceerhl · web-flow · commit 77565bd9f868 · 2024-07-26T13:12:50.000-07:00
diff --git a/src/vs/workbench/contrib/chat/browser/chatInputPart.ts b/src/vs/workbench/contrib/chat/browser/chatInputPart.ts
@@ -362,7 +362,10 @@ export class ChatInputPart extends Disposable implements IHistoryNavigationWidge
 
 		if (removed.length > 0 || contentReferences.length > 0) {
 			this.initAttachedContext(this.attachedContextContainer);
-			this._onDidChangeContext.fire({ removed, added: contentReferences });
+
+			if (!overwrite) {
+				this._onDidChangeContext.fire({ removed, added: contentReferences });
+			}
 		}
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -362,7 +362,10 @@ export class ChatInputPart extends Disposable implements IHistoryNavigationWidge`
`362`	`362`
`363`	`363`	`if (removed.length > 0 \|\| contentReferences.length > 0) {`
`364`	`364`	`this.initAttachedContext(this.attachedContextContainer);`
`365`		`- this._onDidChangeContext.fire({ removed, added: contentReferences });`
	`365`	`+`
	`366`	`+ if (!overwrite) {`
	`367`	`+ this._onDidChangeContext.fire({ removed, added: contentReferences });`
	`368`	`+ }`
`366`	`369`	`}`
`367`	`370`	`}`
`368`	`371`