Skip to content

Commit 3e8a8fb

Browse files
joshspicerCopilot
andcommitted
Address Copilot review: fix floating promise, update stale comment
- Add void to unhandled this.refresh() in pluginListWidget itemProvider onChange handler (consistent with all other call sites) - Update comment in remoteAgentHostCustomizationHarness to reflect that synced bundles now preserve per-skill subdirectories Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
1 parent 43e9f21 commit 3e8a8fb

2 files changed

Lines changed: 4 additions & 3 deletions

File tree

src/vs/sessions/contrib/remoteAgentHost/browser/remoteAgentHostCustomizationHarness.ts

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -456,8 +456,9 @@ export class RemoteAgentCustomizationItemProvider extends Disposable implements
456456
/**
457457
* Emits one {@link ICustomizationItem} per child of a per-type
458458
* sub-folder. Skills are conventionally folders containing
459-
* `SKILL.md`, but the local-sync bundler writes them as flat files;
460-
* both layouts are accepted.
459+
* `SKILL.md`, and synced bundles may preserve per-skill
460+
* subdirectories; flat skill files can still appear for legacy
461+
* bundles, so both layouts are accepted.
461462
*/
462463
private _collectFromTypeDir(entries: readonly { name: string; resource: URI; isDirectory: boolean }[], promptType: PromptsType, groupKey: string): ICustomizationItem[] {
463464
const items: ICustomizationItem[] = [];

src/vs/workbench/contrib/chat/browser/aiCustomization/pluginListWidget.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -660,7 +660,7 @@ export class PluginListWidget extends Disposable {
660660
if (itemProvider) {
661661
itemProviderChangeDisposable.value = itemProvider.onDidChange(() => {
662662
if (!this.browseMode) {
663-
this.refresh();
663+
void this.refresh();
664664
}
665665
});
666666
} else {

0 commit comments

Comments
 (0)