From 016974fd0e2777582e381bca125eb65d926768b4 Mon Sep 17 00:00:00 2001 From: "Kamat, Trivikram" <16024985+trivikr@users.noreply.github.com> Date: Tue, 17 Jun 2025 08:31:12 -0700 Subject: [PATCH 1/5] chore(deps-dev): bump biome to v2 --- package.json | 2 +- yarn.lock | 76 ++++++++++++++++++++++++++-------------------------- 2 files changed, 39 insertions(+), 39 deletions(-) diff --git a/package.json b/package.json index dd1e6133..2b894ee5 100644 --- a/package.json +++ b/package.json @@ -45,7 +45,7 @@ "jscodeshift": "17.3.0" }, "devDependencies": { - "@biomejs/biome": "1.9.0", + "@biomejs/biome": "2.0.0", "@changesets/cli": "^2.29.4", "@tsconfig/node18": "^18.2.4", "@types/jscodeshift": "^17.3.0", diff --git a/yarn.lock b/yarn.lock index 24edcf6f..5bb2c2d0 100644 --- a/yarn.lock +++ b/yarn.lock @@ -429,18 +429,18 @@ __metadata: languageName: node linkType: hard -"@biomejs/biome@npm:1.9.0": - version: 1.9.0 - resolution: "@biomejs/biome@npm:1.9.0" - dependencies: - "@biomejs/cli-darwin-arm64": "npm:1.9.0" - "@biomejs/cli-darwin-x64": "npm:1.9.0" - "@biomejs/cli-linux-arm64": "npm:1.9.0" - "@biomejs/cli-linux-arm64-musl": "npm:1.9.0" - "@biomejs/cli-linux-x64": "npm:1.9.0" - "@biomejs/cli-linux-x64-musl": "npm:1.9.0" - "@biomejs/cli-win32-arm64": "npm:1.9.0" - "@biomejs/cli-win32-x64": "npm:1.9.0" +"@biomejs/biome@npm:2.0.0": + version: 2.0.0 + resolution: "@biomejs/biome@npm:2.0.0" + dependencies: + "@biomejs/cli-darwin-arm64": "npm:2.0.0" + "@biomejs/cli-darwin-x64": "npm:2.0.0" + "@biomejs/cli-linux-arm64": "npm:2.0.0" + "@biomejs/cli-linux-arm64-musl": "npm:2.0.0" + "@biomejs/cli-linux-x64": "npm:2.0.0" + "@biomejs/cli-linux-x64-musl": "npm:2.0.0" + "@biomejs/cli-win32-arm64": "npm:2.0.0" + "@biomejs/cli-win32-x64": "npm:2.0.0" dependenciesMeta: "@biomejs/cli-darwin-arm64": optional: true @@ -460,62 +460,62 @@ __metadata: optional: true bin: biome: bin/biome - checksum: 10c0/b07ad2c8dc3d52c0a4eea37f98d36399b33a13759903aa65f9678db916810b773fe94937c304899158479bccd8c3c90f0f30af22b34d6dc5963774f1acc5e245 + checksum: 10c0/a255d2e84e303c6b1bd841877463f358415a35fb39dc4051dec80d9dd44e4f2f546e7e13804f7cd9f0932ca11664600f819e0b0dd75c55c2c0571ed771d86cb5 languageName: node linkType: hard -"@biomejs/cli-darwin-arm64@npm:1.9.0": - version: 1.9.0 - resolution: "@biomejs/cli-darwin-arm64@npm:1.9.0" +"@biomejs/cli-darwin-arm64@npm:2.0.0": + version: 2.0.0 + resolution: "@biomejs/cli-darwin-arm64@npm:2.0.0" conditions: os=darwin & cpu=arm64 languageName: node linkType: hard -"@biomejs/cli-darwin-x64@npm:1.9.0": - version: 1.9.0 - resolution: "@biomejs/cli-darwin-x64@npm:1.9.0" +"@biomejs/cli-darwin-x64@npm:2.0.0": + version: 2.0.0 + resolution: "@biomejs/cli-darwin-x64@npm:2.0.0" conditions: os=darwin & cpu=x64 languageName: node linkType: hard -"@biomejs/cli-linux-arm64-musl@npm:1.9.0": - version: 1.9.0 - resolution: "@biomejs/cli-linux-arm64-musl@npm:1.9.0" +"@biomejs/cli-linux-arm64-musl@npm:2.0.0": + version: 2.0.0 + resolution: "@biomejs/cli-linux-arm64-musl@npm:2.0.0" conditions: os=linux & cpu=arm64 & libc=musl languageName: node linkType: hard -"@biomejs/cli-linux-arm64@npm:1.9.0": - version: 1.9.0 - resolution: "@biomejs/cli-linux-arm64@npm:1.9.0" +"@biomejs/cli-linux-arm64@npm:2.0.0": + version: 2.0.0 + resolution: "@biomejs/cli-linux-arm64@npm:2.0.0" conditions: os=linux & cpu=arm64 & libc=glibc languageName: node linkType: hard -"@biomejs/cli-linux-x64-musl@npm:1.9.0": - version: 1.9.0 - resolution: "@biomejs/cli-linux-x64-musl@npm:1.9.0" +"@biomejs/cli-linux-x64-musl@npm:2.0.0": + version: 2.0.0 + resolution: "@biomejs/cli-linux-x64-musl@npm:2.0.0" conditions: os=linux & cpu=x64 & libc=musl languageName: node linkType: hard -"@biomejs/cli-linux-x64@npm:1.9.0": - version: 1.9.0 - resolution: "@biomejs/cli-linux-x64@npm:1.9.0" +"@biomejs/cli-linux-x64@npm:2.0.0": + version: 2.0.0 + resolution: "@biomejs/cli-linux-x64@npm:2.0.0" conditions: os=linux & cpu=x64 & libc=glibc languageName: node linkType: hard -"@biomejs/cli-win32-arm64@npm:1.9.0": - version: 1.9.0 - resolution: "@biomejs/cli-win32-arm64@npm:1.9.0" +"@biomejs/cli-win32-arm64@npm:2.0.0": + version: 2.0.0 + resolution: "@biomejs/cli-win32-arm64@npm:2.0.0" conditions: os=win32 & cpu=arm64 languageName: node linkType: hard -"@biomejs/cli-win32-x64@npm:1.9.0": - version: 1.9.0 - resolution: "@biomejs/cli-win32-x64@npm:1.9.0" +"@biomejs/cli-win32-x64@npm:2.0.0": + version: 2.0.0 + resolution: "@biomejs/cli-win32-x64@npm:2.0.0" conditions: os=win32 & cpu=x64 languageName: node linkType: hard @@ -1198,7 +1198,7 @@ __metadata: version: 0.0.0-use.local resolution: "aws-sdk-js-codemod@workspace:." dependencies: - "@biomejs/biome": "npm:1.9.0" + "@biomejs/biome": "npm:2.0.0" "@changesets/cli": "npm:^2.29.4" "@tsconfig/node18": "npm:^18.2.4" "@types/jscodeshift": "npm:^17.3.0" From 71c94c20cb570883053e4001b32e2c855901c62e Mon Sep 17 00:00:00 2001 From: "Kamat, Trivikram" <16024985+trivikr@users.noreply.github.com> Date: Tue, 17 Jun 2025 08:32:08 -0700 Subject: [PATCH 2/5] chore: yarn dlx @biomejs/biome migrate --write --- biome.json | 20 +++++++++++++++----- 1 file changed, 15 insertions(+), 5 deletions(-) diff --git a/biome.json b/biome.json index 06a31aa1..1cea1902 100644 --- a/biome.json +++ b/biome.json @@ -1,8 +1,8 @@ { - "$schema": "https://biomejs.dev/schemas/1.8.0/schema.json", + "$schema": "https://biomejs.dev/schemas/2.0.0/schema.json", "files": { "maxSize": 5242880, - "ignore": ["**/__fixtures__/**", "package.json"] + "includes": ["**", "!**/__fixtures__/**", "!**/package.json"] }, "formatter": { "enabled": true, @@ -16,16 +16,26 @@ "noForEach": "off" }, "correctness": { - "noNewSymbol": "error", "noUndeclaredVariables": "error", - "noUnusedVariables": "error" + "noUnusedVariables": "error", + "noInvalidBuiltinInstantiation": "error" }, "style": { "noNamespace": "error", "useConsistentArrayType": { "level": "error", "options": { "syntax": "shorthand" } - } + }, + "noParameterAssign": "error", + "useAsConstAssertion": "error", + "useDefaultParameterLast": "error", + "useEnumInitializers": "error", + "useSelfClosingElements": "error", + "useSingleVarDeclarator": "error", + "noUnusedTemplateLiteral": "error", + "useNumberNamespace": "error", + "noInferrableTypes": "error", + "noUselessElse": "error" }, "suspicious": { "noEmptyBlockStatements": "error" From c2846d20715756635e425ac008b12f9f3841a102 Mon Sep 17 00:00:00 2001 From: "Kamat, Trivikram" <16024985+trivikr@users.noreply.github.com> Date: Tue, 17 Jun 2025 08:34:26 -0700 Subject: [PATCH 3/5] chore: yarn check --- .../getServiceImportDeepWithNameOutput.ts | 2 +- .../getServiceImportEqualsWithNameOutput.ts | 2 +- .../generateNewClientTests/getServiceImportWithNameOutput.ts | 2 +- .../getServiceRequireDeepWithNameOutput.ts | 2 +- .../generateNewClientTests/getServiceRequireWithNameOutput.ts | 2 +- src/transforms/v2-to-v3/apis/renameErrorCodeWithName.ts | 2 +- src/transforms/v2-to-v3/apis/replaceAwsError.ts | 2 +- src/transforms/v2-to-v3/apis/replaceAwsIdentity.ts | 2 +- src/transforms/v2-to-v3/transformer.ts | 2 +- 9 files changed, 9 insertions(+), 9 deletions(-) diff --git a/scripts/generateNewClientTests/getServiceImportDeepWithNameOutput.ts b/scripts/generateNewClientTests/getServiceImportDeepWithNameOutput.ts index 353c98b7..5c0e8dd3 100644 --- a/scripts/generateNewClientTests/getServiceImportDeepWithNameOutput.ts +++ b/scripts/generateNewClientTests/getServiceImportDeepWithNameOutput.ts @@ -1,6 +1,6 @@ import { CLIENTS_TO_TEST } from "./config"; -import { getClientNameWithLocalSuffix } from "./getClientNameWithLocalSuffix"; import { getClientNamesSortedByPackageName } from "./getClientNamesSortedByPackageName"; +import { getClientNameWithLocalSuffix } from "./getClientNameWithLocalSuffix"; import { getV3ClientsNewExpressionCode } from "./getV3ClientsNewExpressionCode"; import { getV3PackageImportsCode } from "./getV3PackageImportsCode"; diff --git a/scripts/generateNewClientTests/getServiceImportEqualsWithNameOutput.ts b/scripts/generateNewClientTests/getServiceImportEqualsWithNameOutput.ts index e3ea5e36..9f496fa6 100644 --- a/scripts/generateNewClientTests/getServiceImportEqualsWithNameOutput.ts +++ b/scripts/generateNewClientTests/getServiceImportEqualsWithNameOutput.ts @@ -1,6 +1,6 @@ import { CLIENTS_TO_TEST } from "./config"; -import { getClientNameWithLocalSuffix } from "./getClientNameWithLocalSuffix"; import { getClientNamesSortedByPackageName } from "./getClientNamesSortedByPackageName"; +import { getClientNameWithLocalSuffix } from "./getClientNameWithLocalSuffix"; import { getV3ClientsNewExpressionCode } from "./getV3ClientsNewExpressionCode"; import { getV3PackageImportEqualsCode } from "./getV3PackageImportEqualsCode"; diff --git a/scripts/generateNewClientTests/getServiceImportWithNameOutput.ts b/scripts/generateNewClientTests/getServiceImportWithNameOutput.ts index 29af9003..a39667ac 100644 --- a/scripts/generateNewClientTests/getServiceImportWithNameOutput.ts +++ b/scripts/generateNewClientTests/getServiceImportWithNameOutput.ts @@ -1,6 +1,6 @@ import { CLIENTS_TO_TEST } from "./config"; -import { getClientNameWithLocalSuffix } from "./getClientNameWithLocalSuffix"; import { getClientNamesSortedByPackageName } from "./getClientNamesSortedByPackageName"; +import { getClientNameWithLocalSuffix } from "./getClientNameWithLocalSuffix"; import { getV3ClientsNewExpressionCode } from "./getV3ClientsNewExpressionCode"; import { getV3PackageImportsCode } from "./getV3PackageImportsCode"; diff --git a/scripts/generateNewClientTests/getServiceRequireDeepWithNameOutput.ts b/scripts/generateNewClientTests/getServiceRequireDeepWithNameOutput.ts index 0fb42a7b..e8cb3aae 100644 --- a/scripts/generateNewClientTests/getServiceRequireDeepWithNameOutput.ts +++ b/scripts/generateNewClientTests/getServiceRequireDeepWithNameOutput.ts @@ -1,6 +1,6 @@ import { CLIENTS_TO_TEST } from "./config"; -import { getClientNameWithLocalSuffix } from "./getClientNameWithLocalSuffix"; import { getClientNamesSortedByPackageName } from "./getClientNamesSortedByPackageName"; +import { getClientNameWithLocalSuffix } from "./getClientNameWithLocalSuffix"; import { getV3ClientsNewExpressionCode } from "./getV3ClientsNewExpressionCode"; import { getV3PackageRequiresCode } from "./getV3PackageRequiresCode"; diff --git a/scripts/generateNewClientTests/getServiceRequireWithNameOutput.ts b/scripts/generateNewClientTests/getServiceRequireWithNameOutput.ts index 2c115531..0fbaa076 100644 --- a/scripts/generateNewClientTests/getServiceRequireWithNameOutput.ts +++ b/scripts/generateNewClientTests/getServiceRequireWithNameOutput.ts @@ -1,6 +1,6 @@ import { CLIENTS_TO_TEST } from "./config"; -import { getClientNameWithLocalSuffix } from "./getClientNameWithLocalSuffix"; import { getClientNamesSortedByPackageName } from "./getClientNamesSortedByPackageName"; +import { getClientNameWithLocalSuffix } from "./getClientNameWithLocalSuffix"; import { getV3ClientsNewExpressionCode } from "./getV3ClientsNewExpressionCode"; import { getV3PackageRequiresCode } from "./getV3PackageRequiresCode"; diff --git a/src/transforms/v2-to-v3/apis/renameErrorCodeWithName.ts b/src/transforms/v2-to-v3/apis/renameErrorCodeWithName.ts index cd21fb4e..2ed28f49 100644 --- a/src/transforms/v2-to-v3/apis/renameErrorCodeWithName.ts +++ b/src/transforms/v2-to-v3/apis/renameErrorCodeWithName.ts @@ -1,6 +1,6 @@ import type { - ASTPath, ArrowFunctionExpression, + ASTPath, CallExpression, Collection, FunctionExpression, diff --git a/src/transforms/v2-to-v3/apis/replaceAwsError.ts b/src/transforms/v2-to-v3/apis/replaceAwsError.ts index 04afc068..185e846f 100644 --- a/src/transforms/v2-to-v3/apis/replaceAwsError.ts +++ b/src/transforms/v2-to-v3/apis/replaceAwsError.ts @@ -1,5 +1,5 @@ import type { Collection, JSCodeshift } from "jscodeshift"; -import { type ImportType, addNamedModule } from "../modules"; +import { addNamedModule, type ImportType } from "../modules"; export interface ReplaceAwsErrorOptions { v2GlobalName?: string; diff --git a/src/transforms/v2-to-v3/apis/replaceAwsIdentity.ts b/src/transforms/v2-to-v3/apis/replaceAwsIdentity.ts index 04f62ea9..3b202b3d 100644 --- a/src/transforms/v2-to-v3/apis/replaceAwsIdentity.ts +++ b/src/transforms/v2-to-v3/apis/replaceAwsIdentity.ts @@ -1,6 +1,6 @@ import type { Collection, JSCodeshift, NewExpression } from "jscodeshift"; import { AWS_CREDENTIALS_MAP, AWS_TOKEN_MAP } from "../config"; -import { type ImportType, addNamedModule } from "../modules"; +import { addNamedModule, type ImportType } from "../modules"; export interface ReplaceAwsCredentialsOptions { v2GlobalName?: string; diff --git a/src/transforms/v2-to-v3/transformer.ts b/src/transforms/v2-to-v3/transformer.ts index db2fbb47..6782c296 100644 --- a/src/transforms/v2-to-v3/transformer.ts +++ b/src/transforms/v2-to-v3/transformer.ts @@ -37,11 +37,11 @@ import { } from "./modules"; import { removeTypesFromTSQualifiedName, replaceTSTypeReference } from "./ts-type"; import { - IndentationType, getFormattedSourceString, getMostUsedIndentationType, getMostUsedStringLiteralQuote, getValueIndentedWithTabs, + IndentationType, isTrailingCommaUsed, isTypeScriptFile, } from "./utils"; From 32c6c62e87810c364978fd6108fcf12d9e83542e Mon Sep 17 00:00:00 2001 From: "Kamat, Trivikram" <16024985+trivikr@users.noreply.github.com> Date: Tue, 17 Jun 2025 08:36:26 -0700 Subject: [PATCH 4/5] chore: remove biome supression comments --- src/cli.ts | 1 - src/transforms/v2-to-v3/apis/removePromiseForCallExpression.ts | 1 - 2 files changed, 2 deletions(-) diff --git a/src/cli.ts b/src/cli.ts index 5e5572cd..faec9fe2 100644 --- a/src/cli.ts +++ b/src/cli.ts @@ -58,7 +58,6 @@ try { ({ options, positionalArguments } = parser.parse()); if (positionalArguments.length === 0 && !options.stdin) { process.stderr.write( - // biome-ignore lint/style/useTemplate: This is a multiline string "Error: You have to provide at least one file/directory to transform." + "\n\n---\n\n" + parser.getHelpText() diff --git a/src/transforms/v2-to-v3/apis/removePromiseForCallExpression.ts b/src/transforms/v2-to-v3/apis/removePromiseForCallExpression.ts index ec9bb56d..c0e83944 100644 --- a/src/transforms/v2-to-v3/apis/removePromiseForCallExpression.ts +++ b/src/transforms/v2-to-v3/apis/removePromiseForCallExpression.ts @@ -32,7 +32,6 @@ export const removePromiseForCallExpression = ( } default: { emitWarning( - // biome-ignore lint/style/useTemplate: This is a multiline string `Removal of .promise() not implemented for parentPath: ${callExpression.parentPath.value.type}\n` + `Code processed: ${j(callExpression.parentPath).toSource()}\n\n` + "Please report your use case on https://github.com/aws/aws-sdk-js-codemod\n" From 17b7724945d12cf73ac550a396ecf1af93f07e22 Mon Sep 17 00:00:00 2001 From: "Kamat, Trivikram" <16024985+trivikr@users.noreply.github.com> Date: Tue, 17 Jun 2025 08:37:22 -0700 Subject: [PATCH 5/5] chore: yarn changeset add --- .changeset/all-symbols-pump.md | 5 +++++ 1 file changed, 5 insertions(+) create mode 100644 .changeset/all-symbols-pump.md diff --git a/.changeset/all-symbols-pump.md b/.changeset/all-symbols-pump.md new file mode 100644 index 00000000..ba623f66 --- /dev/null +++ b/.changeset/all-symbols-pump.md @@ -0,0 +1,5 @@ +--- +"aws-sdk-js-codemod": patch +--- + +Bump biome to v2