Skip to content

Commit

Permalink
Address feedback
Browse files Browse the repository at this point in the history
  • Loading branch information
st0012 committed Aug 19, 2024
1 parent b5df1aa commit ba82b4b
Showing 1 changed file with 11 additions and 8 deletions.
19 changes: 11 additions & 8 deletions vscode/src/issueDiagnosingHelper.ts
Original file line number Diff line number Diff line change
Expand Up @@ -31,16 +31,13 @@ async function collectDiagnosticInfo(
const extensions = await getPublicExtensions();

return {
// eslint-disable-next-line @typescript-eslint/naming-convention
/* eslint-disable @typescript-eslint/naming-convention */
"VS Code Version": vscodeVersion,
// eslint-disable-next-line @typescript-eslint/naming-convention
"Ruby LSP Extension Version": rubyLspExtensionVersion,
// eslint-disable-next-line @typescript-eslint/naming-convention
"Ruby LSP Server Version": rubyLspVersion,
// eslint-disable-next-line @typescript-eslint/naming-convention
"Ruby LSP Addons": rubyLspAddons,
// eslint-disable-next-line @typescript-eslint/naming-convention
"Installed Extensions": extensions,
/* eslint-enable @typescript-eslint/naming-convention */
};
}

Expand All @@ -52,9 +49,15 @@ async function getPublicExtensions(): Promise<string[]> {
return false;
}

// Check for marketplace identifier
const galleryExtension = ext.packageJSON?.extensionPack?.[0] ?? ext.id;
return galleryExtension.includes(".");
// Assume if an extension doesn't have a license, it's private and should not be listed
if (
ext.packageJSON.license === "UNLICENSED" ||
!ext.packageJSON.license
) {
return false;
}

return true;
})
.map((ext) => `${ext.packageJSON.name} (${ext.packageJSON.version})`);
}
Expand Down

0 comments on commit ba82b4b

Please sign in to comment.