Merge aws#5307 lint: use gitignore, enforce prettier rules

Author: justinmk3

.eslintignore

@@ -18,7 +18,9 @@ module.exports = {
         'plugin:@typescript-eslint/eslint-recommended',
         'plugin:@typescript-eslint/recommended-requiring-type-checking',
         'plugin:@typescript-eslint/recommended',
-        'prettier',
+        // "Add this as the _last_ item in the extends array, so that eslint-config-prettier has the
+        // opportunity to override other configs." https://github.com/prettier/eslint-plugin-prettier
+        'plugin:prettier/recommended',
     ],
     rules: {
         curly: 2, // Enforce braces on "if"/"for"/etc.

.eslintrc.js

@@ -1,3 +1,5 @@
+# Note: also used for `eslint --ignore-path`.
+
 out
 dist
 node_modules
@@ -27,15 +29,15 @@ packages/core/src/shared/telemetry/vscodeTelemetry.json
 .test-reports
 
 # Auto generated type definitions
-packages/core/src/shared/telemetry/clienttelemetry.d.ts
-packages/core/src/codewhisperer/client/codewhispererclient.d.ts
-packages/core/src/codewhisperer/client/codewhispereruserclient.d.ts
-packages/core/src/amazonqFeatureDev/client/featuredevproxyclient.d.ts
-packages/core/src/auth/sso/oidcclientpkce.d.ts
+**/src/shared/telemetry/clienttelemetry.d.ts
+**/src/codewhisperer/client/codewhispererclient.d.ts
+**/src/codewhisperer/client/codewhispereruserclient.d.ts
+**/src/amazonqFeatureDev/client/featuredevproxyclient.d.ts
+**/src/auth/sso/oidcclientpkce.d.ts
 
 # Generated by tests
-packages/core/src/testFixtures/**/bin
-packages/core/src/testFixtures/**/obj
+**/src/testFixtures/**/bin
+**/src/testFixtures/**/obj
 
 # Generated by copyFiles.ts
 packages/*/LICENSE

.gitignore

@@ -16,7 +16,7 @@
         "singleQuote": true,
         "semi": false,
         "bracketSpacing": true,
-        "arrowParens": "avoid",
+        "arrowParens": "always",
         "endOfLine": "lf"
     },
     "scripts": {
@@ -32,10 +32,8 @@
         "package": "npm run package -w packages/toolkit -w packages/amazonq",
         "newChange": "echo 'Must specify subproject/workspace with -w packages/<subproject>' && false",
         "createRelease": "echo 'Must specify subproject/workspace with -w packages/<subproject>' && false",
-        "format": "prettier --check plugins && npm run format -w packages/ --if-present",
-        "formatfix": "prettier --write plugins && npm run formatfix -w packages/ --if-present",
-        "lint": "npm run lint -w packages/ --if-present && npm run format",
-        "lintfix": "eslint -c .eslintrc.js --fix --ext .ts packages plugins && npm run formatfix",
+        "lint": "npm run lint -w packages/ --if-present",
+        "lintfix": "eslint -c .eslintrc.js --ignore-path .gitignore --ignore-pattern '**/*.json' --ignore-pattern '**/*.gen.ts' --ignore-pattern '**/types/*.d.ts' --ignore-pattern '**/src/testFixtures/**' --ignore-pattern '**/resources/js/graphStateMachine.js' --fix --ext .ts packages plugins",
         "clean": "npm run clean -w packages/ -w plugins/",
         "reset": "npm run clean && ts-node ./scripts/clean.ts node_modules && npm install",
         "generateNonCodeFiles": "npm run generateNonCodeFiles -w packages/ --if-present"
@@ -55,6 +53,7 @@
         "eslint-config-prettier": "^9.1.0",
         "eslint-plugin-aws-toolkits": "file:plugins/eslint-plugin-aws-toolkits",
         "eslint-plugin-header": "^3.1.1",
+        "eslint-plugin-prettier": "^5.1.3",
         "eslint-plugin-security-node": "^1.1.4",
         "eslint-plugin-unicorn": "^54.0.0",
         "husky": "^9.0.7",

package-lock.json

@@ -54,9 +54,7 @@
         "compile": "npm run clean && npm run buildScripts && webpack --mode development",
         "compileOnly": "tsc -p ./",
         "package": "ts-node ../../scripts/package.ts",
-        "format": "prettier --ignore-path ../../.prettierignore --check src scripts",
-        "formatfix": "prettier --ignore-path ../../.prettierignore --write src scripts",
-        "lint": "echo 'Nothing to lint yet!'",
+        "lint": "true",
         "watch": "npm run clean && npm run buildScripts && tsc -watch -p ./",
         "testCompile": "npm run clean && npm run buildScripts && npm run compileOnly",
         "test": "npm run testCompile && c8 ts-node ../core/scripts/test/test.ts dist/test/unit/index.js ../core/dist/src/testFixtures/workspaceFolder",

package.json

@@ -26,7 +26,7 @@ interface CopyTask {
 }
 
 const tasks: CopyTask[] = [
-    ...['LICENSE', 'NOTICE'].map(f => {
+    ...['LICENSE', 'NOTICE'].map((f) => {
         return { target: path.join('../../', f), destination: path.join(projectRoot, f) }
     }),
 

packages/amazonq/package.json

@@ -25,7 +25,7 @@ function main() {
     const coreLibPackageJson = JSON.parse(fs.readFileSync(coreLibPackageJsonFile, { encoding: 'utf-8' }))
 
     const coreSettings = coreLibPackageJson.contributes.configuration.properties
-    Object.keys(coreSettings).forEach(key => {
+    Object.keys(coreSettings).forEach((key) => {
         if (key.startsWith('amazonQ')) {
             packageJson.contributes.configuration.properties[key] = coreSettings[key]
         }

packages/amazonq/scripts/build/copyFiles.ts

@@ -44,7 +44,7 @@ export const amazonQContextPrefix = 'amazonq'
  */
 export async function activateAmazonQCommon(context: vscode.ExtensionContext, isWeb: boolean) {
     initialize(context, isWeb)
-    const homeDirLogs = await fs.init(context, homeDir => {
+    const homeDirLogs = await fs.init(context, (homeDir) => {
         void messages.showViewLogsMessage(`Invalid home directory (check $HOME): "${homeDir}"`)
     })
     errors.init(fs.getUsername(), env.isAutomation())
@@ -73,12 +73,12 @@ export async function activateAmazonQCommon(context: vscode.ExtensionContext, is
                                 } and older. Your AWS Toolkit was updated to version 3.0 or later.`,
                                 'Reload Now'
                             )
-                            .then(async resp => {
+                            .then(async (resp) => {
                                 if (resp === 'Reload Now') {
                                     await vscode.commands.executeCommand('workbench.action.reloadWindow')
                                 }
                             }),
-                    reason => {
+                    (reason) => {
                         getLogger().error('workbench.extensions.installExtension failed: %O', reason)
                     }
                 )

packages/amazonq/scripts/build/syncPackageJson.ts

@@ -78,14 +78,14 @@ describe('Amazon Q Feature Dev', function () {
 
     function waitForButtons(buttons: FollowUpTypes[]) {
         return tab.waitForEvent(() => {
-            return buttons.every(value => tab.hasButton(value))
+            return buttons.every((value) => tab.hasButton(value))
         })
     }
 
     async function waitForText(text: string) {
         await tab.waitForEvent(
             () => {
-                return tab.getChatItems().some(chatItem => chatItem.body === text)
+                return tab.getChatItems().some((chatItem) => chatItem.body === text)
             },
             {
                 waitIntervalInMs: 250,