Skip to content

Commit ba80bf2

Browse files
authored
Merge pull request microsoft#251235 from microsoft/aeschli/experienced-orca
use regular chatAttachmentWidgets for prompt files
2 parents 89ba12c + 79ff867 commit ba80bf2

16 files changed

+339
-906
lines changed

src/vs/workbench/contrib/chat/browser/attachments/implicitContextAttachment.ts

Lines changed: 2 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -6,12 +6,10 @@
66
import * as dom from '../../../../../base/browser/dom.js';
77
import { StandardKeyboardEvent } from '../../../../../base/browser/keyboardEvent.js';
88
import { StandardMouseEvent } from '../../../../../base/browser/mouseEvent.js';
9-
import { Codicon } from '../../../../../base/common/codicons.js';
109
import { KeyCode } from '../../../../../base/common/keyCodes.js';
1110
import { Disposable, DisposableStore } from '../../../../../base/common/lifecycle.js';
1211
import { Schemas } from '../../../../../base/common/network.js';
1312
import { basename, dirname } from '../../../../../base/common/resources.js';
14-
import { ThemeIcon } from '../../../../../base/common/themables.js';
1513
import { URI } from '../../../../../base/common/uri.js';
1614
import { ILanguageService } from '../../../../../editor/common/languages/language.js';
1715
import { IModelService } from '../../../../../editor/common/services/model.js';
@@ -61,9 +59,7 @@ export class ImplicitContextAttachmentWidget extends Disposable {
6159
const file = URI.isUri(this.attachment.value) ? this.attachment.value : this.attachment.value!.uri;
6260
const range = undefined;
6361

64-
const attachmentTypeName = (this.attachment.isPromptFile === false)
65-
? file.scheme === Schemas.vscodeNotebookCell ? localize('cell.lowercase', "cell") : localize('file.lowercase', "file")
66-
: localize('prompt.lowercase', "prompt");
62+
const attachmentTypeName = file.scheme === Schemas.vscodeNotebookCell ? localize('cell.lowercase', "cell") : localize('file.lowercase', "file");
6763

6864
const fileBasename = basename(file);
6965
const fileDirname = dirname(file);
@@ -74,16 +70,11 @@ export class ImplicitContextAttachmentWidget extends Disposable {
7470
const currentFile = localize('openEditor', "Suggested context (current file)");
7571
const title = `${currentFile}\n${uriLabel}`;
7672

77-
const icon = this.attachment.isPromptFile
78-
? ThemeIcon.fromId(Codicon.bookmark.id)
79-
: undefined;
80-
8173
label.setFile(file, {
8274
fileKind: FileKind.FILE,
8375
hidePath: true,
8476
range,
85-
title,
86-
icon,
77+
title
8778
});
8879
this.domNode.ariaLabel = ariaLabel;
8980
this.domNode.tabIndex = 0;

src/vs/workbench/contrib/chat/browser/attachments/promptInstructions/promptInstructionsCollectionWidget.ts

Lines changed: 0 additions & 200 deletions
This file was deleted.

0 commit comments

Comments
 (0)