Fix typings and add interfaces #20

Copilot · 2026-02-04T08:21:52Z

The namespace property may be undefined when retrieved from the ExtensionData, as it's optional in the ExtensionMetaData interface and only set for the "automatic-comment-blocks" extension (see line 79 in extensionData.ts). Using it directly in string templates without checking for undefined could result in "undefined" being interpolated into the strings.

Consider adding a null check or using a fallback value when namespace is undefined.

Suggested change

const extensionName = extensionData.get("namespace");

const extensionName = String(extensionData.get("namespace"));

Copilot · 2026-02-04T08:21:53Z

Using the non-null assertion operator (!) on process.env.VSCODE_WSL_EXT_LOCATION assumes this environment variable is always defined when running in WSL. If this environment variable is not set, it will result in path.dirname(undefined), which could cause unexpected behavior.

Consider adding a check to ensure the environment variable exists before using it, or provide a fallback value.

Suggested change

this.extensionDiscoveryPaths.set("WindowsUserExtensionsPathFromWsl", path.dirname(process.env.VSCODE_WSL_EXT_LOCATION!));

this.extensionDiscoveryPaths.set("WindowsBuiltInExtensionsPathFromWsl", path.join(process.env.VSCODE_CWD!, "resources/app/extensions"));

const wslExtLocation = process.env.VSCODE_WSL_EXT_LOCATION;

if (wslExtLocation) {

this.extensionDiscoveryPaths.set("WindowsUserExtensionsPathFromWsl", path.dirname(wslExtLocation));

}

const vscodeCwd = process.env.VSCODE_CWD;

if (vscodeCwd) {

this.extensionDiscoveryPaths.set("WindowsBuiltInExtensionsPathFromWsl", path.join(vscodeCwd, "resources/app/extensions"));

}

Copilot · 2026-02-04T08:21:53Z

Using the non-null assertion operator (!) on process.env.VSCODE_CWD assumes this environment variable is always defined when running in WSL. If this environment variable is not set, it will result in path.join(undefined, ...), which could cause unexpected behavior.

Consider adding a check to ensure the environment variable exists before using it, or provide a fallback value.

Suggested change

this.extensionDiscoveryPaths.set("WindowsUserExtensionsPathFromWsl", path.dirname(process.env.VSCODE_WSL_EXT_LOCATION!));

this.extensionDiscoveryPaths.set("WindowsBuiltInExtensionsPathFromWsl", path.join(process.env.VSCODE_CWD!, "resources/app/extensions"));

const wslExtLocation = process.env.VSCODE_WSL_EXT_LOCATION;

const vscodeCwd = process.env.VSCODE_CWD;

if (wslExtLocation) {

this.extensionDiscoveryPaths.set(

"WindowsUserExtensionsPathFromWsl",

path.dirname(wslExtLocation)

);

}

if (vscodeCwd) {

this.extensionDiscoveryPaths.set(

"WindowsBuiltInExtensionsPathFromWsl",

path.join(vscodeCwd, "resources/app/extensions")

);

}

-Original file line number
+Diff line change
@@ Expand Up / @@ -3,7 +3,7 @@ on: @@
     name: Compile TypeScript to check for errors
     jobs:
-      deploy:
+      check-ts:
         runs-on: ubuntu-latest
         steps:
           - uses: actions/checkout@v4
@@ Expand Down @@

-Original file line number
+Diff line change
@@ Expand Up / @@ -30,7 +30,8 @@ @@
     				"*.ts"
     			],
     			"options": {
-    				"quoteProps": "consistent"
+    				"quoteProps": "consistent",
+    				"trailingComma": "es5"
     			}
     		}
     	]
@@ Expand Down @@

-Original file line number
+Diff line change
@@ -0,0 +1,41 @@
+    /**
+     * Define the single-line comment styles.
+     */
+    export type SingleLineCommentStyle = "//" | "#" | ";";
+    /**
+     * Define the extra single-line comment styles, like `///`, etc.
+     */
+    export type ExtraSingleLineCommentStyles = "##" | ";;" | "///" | "//!";
+    /**
+     * Line Comments
+     *
+     * Taken directly from VScode's commit in June 2025 that changed the line comment config.
+     * https://github.com/microsoft/vscode/commit/d9145a291dcef0bad3ace81a3d55727ca294c122#diff-0dfa7db579eface8250affb76bc88717725a121401d4d8598bc36b92b0b6ef62
+     *
+     * The @types/vscode package does not yet have these changes.
+     * So until they're added, we define them manually.
+     */
+    /**
+     * The line comment token, like `// this is a comment`.
+     * Can be a string, an object with comment and optional noIndent properties, or null.
+     */
+    export type LineComment = string | LineCommentConfig | null;
+    /**
+     * Configuration for line comments.
+     */
+    export interface LineCommentConfig {
+    	/**
+    	 * The line comment token, like `//`
+    	 */
+    	comment: string;
+    	/**
+    	 * Whether the comment token should not be indented and placed at the first column.
+    	 * Defaults to false.
+    	 */
+    	noIndent?: boolean;
+    }

-Original file line number
+Diff line change
@@ -0,0 +1,77 @@
+    import {IPackageJson} from "package-json-type";
+    // Utility types for cleaner Map typing
+    export type ExtensionMetaDataValue = ExtensionMetaData[keyof ExtensionMetaData];
+    /**
+     * Extension metadata for a VSCode extension
+     */
+    export interface ExtensionMetaData {
+    	/**
+    	 * The unique ID in the form of `publisher.name`.
+    	 */
+    	id: string;
+    	/**
+    	 * The name.
+    	 * Directly from package.json "name" key.
+    	 */
+    	name: string;
+    	/**
+    	 * The namespace for this extension's configuration settings,
+    	 * which is a slightly shorter version of the name.
+    	 */
+    	namespace?: string;
+    	/**
+    	 * The display name.
+    	 * Directly from package.json "displayName" key.
+    	 */
+    	displayName: string;
+    	/**
+    	 * The version.
+    	 * Directly from package.json "version" key.
+    	 */
+    	version: string;
+    	/**
+    	 * The absolute path to the extension.
+    	 */
+    	extensionPath: string;
+    	/**
+    	 * The full package.json data
+    	 */
+    	packageJSON: IPackageJson;
+    }
+    /**
+     * Extension discovery paths configuration for this extension
+     */
+    export interface ExtensionPaths {
+    	/**
+    	 * The path to the user extensions.
+    	 */
+    	userExtensionsPath: string;
+    	/**
+    	 * The path to the built-in extensions.
+    	 */
+    	builtInExtensionsPath: string;
+    	/**
+    	 * The Windows path to the user extensions when running in WSL.
+    	 *
+    	 * Only set when running in WSL.
+    	 */
+    	WindowsUserExtensionsPathFromWsl?: string;
+    	/**
+    	 * The Windows path to the built-in extensions when running in WSL.
+    	 *
+    	 * Only set when running in WSL.
+    	 */
+    	WindowsBuiltInExtensionsPathFromWsl?: string;
+    }

-Original file line number
+Diff line change
@@ -0,0 +1,10 @@
+    export interface Settings {
+    	singleLineBlockOnEnter: boolean;
+    	disabledLanguages: string[];
+    	slashStyleBlocks: string[];
+    	hashStyleBlocks: string[];
+    	semicolonStyleBlocks: string[];
+    	multiLineStyleBlocks: string[];
+    	overrideDefaultLanguageMultiLineComments: Record<string, string>;
+    	bladeOverrideComments: boolean;
+    }

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Fix typings and add interfaces #20

Uh oh!

Diff view

Diff view

There are no files selected for viewing

Uh oh!

Copilot AI Feb 4, 2026

Uh oh!

Copilot AI Feb 4, 2026

Uh oh!

Copilot AI Feb 4, 2026

Uh oh!

Uh oh!

Fix typings and add interfaces #20

Uh oh!

Fix typings and add interfaces #20

Uh oh!

Uh oh!

Diff view

Diff view

There are no files selected for viewing

Uh oh!

Copilot AI Feb 4, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 4, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 4, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!