ci: require space at start of // comment

Problem:
Inconsistent comment style.

Solution:
Add a lint rule.

Author: justinmk3

.eslintrc.js

@@ -12,7 +12,7 @@ module.exports = {
         mocha: true,
         es2024: true,
     },
-    plugins: ['@typescript-eslint', 'unicorn', 'header', 'security-node', 'aws-toolkits'],
+    plugins: ['@typescript-eslint', '@stylistic', 'unicorn', 'header', 'security-node', 'aws-toolkits'],
     extends: [
         'eslint:recommended',
         'plugin:@typescript-eslint/eslint-recommended',
@@ -113,6 +113,20 @@ module.exports = {
         'no-constant-condition': ['error', { checkLoops: false }],
         'no-empty': 'off',
 
+        // https://eslint.style/rules/default/spaced-comment
+        // Require space after // comment.
+        '@stylistic/spaced-comment': [
+            'error',
+            'always',
+            {
+                block: {
+                    markers: ['!'], // Allow the /*!…*/ license header.
+                    // exceptions: ['*'],
+                    // balanced: true
+                },
+            },
+        ],
+
         // Rules from https://github.com/sindresorhus/eslint-plugin-unicorn
         // TODO: 'unicorn/no-useless-promise-resolve-reject': 'error',
         // TODO: 'unicorn/prefer-at': 'error',

package-lock.json

@@ -41,6 +41,7 @@
     "devDependencies": {
         "@aws-toolkits/telemetry": "^1.0.284",
         "@playwright/browser-chromium": "^1.43.1",
+        "@stylistic/eslint-plugin": "^2.11.0",
         "@types/he": "^1.2.3",
         "@types/vscode": "^1.68.0",
         "@types/vscode-webview": "^1.57.1",

package.json

@@ -53,7 +53,7 @@ export async function activateAmazonQCommon(context: vscode.ExtensionContext, is
     errors.init(fs.getUsername(), env.isAutomation())
     await initializeComputeRegion()
 
-    globals.contextPrefix = 'amazonq.' //todo: disconnect from above line
+    globals.contextPrefix = 'amazonq.' // todo: disconnect from above line
 
     // Avoid activation if older toolkit is installed
     // Amazon Q is only compatible with AWS Toolkit >= 3.0.0

packages/amazonq/src/extension.ts

@@ -108,7 +108,7 @@ zoomoutBtn.addEventListener('click', () => {
 
 // Message passing from extension to webview.
 // Capture state machine definition
-window.addEventListener('message', event => {
+window.addEventListener('message', (event) => {
     // event.data is object passed in from postMessage from vscode
     const message = event.data
     switch (message.command) {

packages/core/resources/js/graphStateMachine.js

@@ -360,7 +360,7 @@ export class LspController {
                 })
             }
         } catch (error) {
-            //TODO: use telemetry.run()
+            // TODO: use telemetry.run()
             getLogger().error(`LspController: Failed to build index of project`)
             telemetry.amazonq_indexWorkspace.emit({
                 duration: performance.now() - start,

packages/core/src/amazonq/lsp/lspController.ts

@@ -43,7 +43,7 @@ export const createMynahUI = (
     let mynahUI: MynahUI
     // eslint-disable-next-line prefer-const
     let connector: Connector
-    //Store the mapping between messageId and messageUserIntent for amazonq_interactWithMessage telemetry
+    // Store the mapping between messageId and messageUserIntent for amazonq_interactWithMessage telemetry
     const responseMetadata = new Map<string, string[]>()
 
     window.addEventListener('error', (e) => {

packages/core/src/amazonq/webview/ui/main.ts

@@ -588,7 +588,7 @@ export class IamPolicyChecksWebview extends VueWebview {
                 })
             } catch (err: any) {
                 if (err.status === 2) {
-                    //CLI responds with a status code of 2 when findings are discovered
+                    // CLI responds with a status code of 2 when findings are discovered
                     const findingsCount = this.handleValidatePolicyCliResponse(err.stdout.toString())
                     span.record({
                         findingsCount: findingsCount,
@@ -652,7 +652,7 @@ export class IamPolicyChecksWebview extends VueWebview {
                 })
             } catch (err: any) {
                 if (err.status === 2) {
-                    //CLI responds with a status code of 2 when findings are discovered
+                    // CLI responds with a status code of 2 when findings are discovered
                     const findingsCount = this.handleCustomPolicyChecksCliResponse(err.stdout.toString())
                     span.record({
                         findingsCount: findingsCount,
@@ -752,7 +752,7 @@ export async function renderIamPolicyChecks(context: ExtContext): Promise<VueWeb
     const logger: Logger = getLogger()
     try {
         const client = new AccessAnalyzer({ region: context.regionProvider.defaultRegionId })
-        //Read from settings to auto-fill some inputs
+        // Read from settings to auto-fill some inputs
         const checkNoNewAccessFilePath: string = vscode.workspace
             .getConfiguration()
             .get(IamPolicyChecksConstants.CheckNoNewAccessFilePathSetting)!
@@ -827,7 +827,7 @@ export async function _readCustomChecksFile(input: string): Promise<string> {
     }
 }
 
-//Check if Cfn and Tf tools are installed
+// Check if Cfn and Tf tools are installed
 export function arePythonToolsInstalled(): boolean {
     const logger: Logger = getLogger()
     let cfnToolInstalled = true

packages/core/src/awsService/accessanalyzer/vue/iamPolicyChecks.ts

@@ -38,7 +38,7 @@ export class TimeFilterSubmenu extends Prompter<TimeFilterResponse> {
 
     private get recentTimeItems(): DataQuickPickItem<number>[] {
         const options: DataQuickPickItem<number>[] = []
-        //appromixate 31 days as month length (better to overshoot)
+        // appromixate 31 days as month length (better to overshoot)
         options.push({
             label: 'All time',
             data: 0,

packages/core/src/awsService/cloudWatchLogs/timeFilterSubmenu.ts

@@ -46,7 +46,7 @@ export async function attachCertificateCommand(node: IotThingNode, promptFun = p
 
     getLogger().debug('Attached certificate %O', cert.certificateId)
 
-    //Refresh the Thing node
+    // Refresh the Thing node
     await node.refreshNode()
 }