From 6d9b874250a5289796f0578846a59be17a64509c Mon Sep 17 00:00:00 2001 From: Adam Moussa <166072409+amoussa1229@users.noreply.github.com> Date: Wed, 6 May 2026 18:38:51 -0400 Subject: [PATCH 1/8] Upgrade to TypeScript 6.0.3 and @types/node 25.6.0 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit TS 6 requires explicit node type resolution — added "types": ["node"] to tsconfig.json to resolve path and __dirname globals. --- package-lock.json | 32 +++++++++++++++++++------------- package.json | 4 ++-- tsconfig.json | 1 + 3 files changed, 22 insertions(+), 15 deletions(-) diff --git a/package-lock.json b/package-lock.json index 9ef86e5..4b41829 100644 --- a/package-lock.json +++ b/package-lock.json @@ -18,11 +18,11 @@ "@aws-sdk/client-dynamodb": "^3.1030.0", "@aws-sdk/lib-dynamodb": "^3.1041.0", "@types/aws-lambda": "^8.10.149", - "@types/node": "^22.0.0", + "@types/node": "^25.6.0", "aws-cdk": "^2.1120.0", "esbuild": "^0.25.0", "ts-node": "^10.9.2", - "typescript": "~5.7.2" + "typescript": "^6.0.3" } }, "node_modules/@aws-cdk/asset-awscli-v1": { @@ -46,6 +46,7 @@ "semver" ], "license": "Apache-2.0", + "peer": true, "dependencies": { "jsonschema": "~1.4.1", "semver": "^7.7.4" @@ -214,6 +215,7 @@ "integrity": "sha512-q0dhIfdm3R9EEElaUXvoLD0EBm4KYPcJC4So/skToMPACiznNgVI2Vup4PKM716KTaDsIcF8V1EA7s33H7oWMg==", "dev": true, "license": "Apache-2.0", + "peer": true, "dependencies": { "@aws-crypto/sha256-browser": "5.2.0", "@aws-crypto/sha256-js": "5.2.0", @@ -2065,13 +2067,14 @@ "license": "MIT" }, "node_modules/@types/node": { - "version": "22.19.17", - "resolved": "https://registry.npmjs.org/@types/node/-/node-22.19.17.tgz", - "integrity": "sha512-wGdMcf+vPYM6jikpS/qhg6WiqSV/OhG+jeeHT/KlVqxYfD40iYJf9/AE1uQxVWFvU7MipKRkRv8NSHiCGgPr8Q==", + "version": "25.6.0", + "resolved": "https://registry.npmjs.org/@types/node/-/node-25.6.0.tgz", + "integrity": "sha512-+qIYRKdNYJwY3vRCZMdJbPLJAtGjQBudzZzdzwQYkEPQd+PJGixUL5QfvCLDaULoLv+RhT3LDkwEfKaAkgSmNQ==", "dev": true, "license": "MIT", + "peer": true, "dependencies": { - "undici-types": "~6.21.0" + "undici-types": "~7.19.0" } }, "node_modules/acorn": { @@ -2139,6 +2142,7 @@ "mime-types" ], "license": "Apache-2.0", + "peer": true, "dependencies": { "@aws-cdk/asset-awscli-v1": "2.2.273", "@aws-cdk/asset-node-proxy-agent-v6": "^2.1.1", @@ -2515,7 +2519,8 @@ "version": "10.6.0", "resolved": "https://registry.npmjs.org/constructs/-/constructs-10.6.0.tgz", "integrity": "sha512-TxHOnBO5zMo/G76ykzGF/wMpEHu257TbWiIxP9K0Yv/+t70UzgBQiTqjkAsWOPC6jW91DzJI0+ehQV6xDRNBuQ==", - "license": "Apache-2.0" + "license": "Apache-2.0", + "peer": true }, "node_modules/create-require": { "version": "1.1.1", @@ -2780,11 +2785,12 @@ "license": "0BSD" }, "node_modules/typescript": { - "version": "5.7.3", - "resolved": "https://registry.npmjs.org/typescript/-/typescript-5.7.3.tgz", - "integrity": "sha512-84MVSjMEHP+FQRPy3pX9sTVV/INIex71s9TL2Gm5FG/WG1SqXeKyZ0k7/blY/4FdOzI12CBy1vGc4og/eus0fw==", + "version": "6.0.3", + "resolved": "https://registry.npmjs.org/typescript/-/typescript-6.0.3.tgz", + "integrity": "sha512-y2TvuxSZPDyQakkFRPZHKFm+KKVqIisdg9/CZwm9ftvKXLP8NRWj38/ODjNbr43SsoXqNuAisEf1GdCxqWcdBw==", "dev": true, "license": "Apache-2.0", + "peer": true, "bin": { "tsc": "bin/tsc", "tsserver": "bin/tsserver" @@ -2794,9 +2800,9 @@ } }, "node_modules/undici-types": { - "version": "6.21.0", - "resolved": "https://registry.npmjs.org/undici-types/-/undici-types-6.21.0.tgz", - "integrity": "sha512-iwDZqg0QAGrg9Rav5H4n0M64c3mkR59cJ6wQp+7C4nI0gsmExaedaYLNO44eT4AtBBwjbTiGPMlt2Md0T9H9JQ==", + "version": "7.19.2", + "resolved": "https://registry.npmjs.org/undici-types/-/undici-types-7.19.2.tgz", + "integrity": "sha512-qYVnV5OEm2AW8cJMCpdV20CDyaN3g0AjDlOGf1OW4iaDEx8MwdtChUp4zu4H0VP3nDRF/8RKWH+IPp9uW0YGZg==", "dev": true, "license": "MIT" }, diff --git a/package.json b/package.json index 9dd5fbb..4262c78 100644 --- a/package.json +++ b/package.json @@ -14,11 +14,11 @@ "@aws-sdk/client-dynamodb": "^3.1030.0", "@aws-sdk/lib-dynamodb": "^3.1041.0", "@types/aws-lambda": "^8.10.149", - "@types/node": "^22.0.0", + "@types/node": "^25.6.0", "aws-cdk": "^2.1120.0", "esbuild": "^0.25.0", "ts-node": "^10.9.2", - "typescript": "~5.7.2" + "typescript": "^6.0.3" }, "dependencies": { "@cdklabs/generative-ai-cdk-constructs": "^0.1.0", diff --git a/tsconfig.json b/tsconfig.json index 89c709b..f9cc1e8 100644 --- a/tsconfig.json +++ b/tsconfig.json @@ -2,6 +2,7 @@ "compilerOptions": { "target": "ES2020", "lib": ["ES2020"], + "types": ["node"], "module": "commonjs", "declaration": true, "strict": true, -- 2.50.1 From 3b84455692ef887eb1b48162635b7860fae95ec6 Mon Sep 17 00:00:00 2001 From: Adam Moussa <166072409+amoussa1229@users.noreply.github.com> Date: Wed, 6 May 2026 19:04:55 -0400 Subject: [PATCH 2/8] Pin typescript to ~6.0.3 and @types/node to ^22 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit TS doesn't follow semver — use tilde to avoid surprise minor-release breakage. Keep @types/node on v22 to match Lambda NODEJS_22_X runtime. --- package-lock.json | 18 +++++++++--------- package.json | 4 ++-- 2 files changed, 11 insertions(+), 11 deletions(-) diff --git a/package-lock.json b/package-lock.json index 4b41829..615eb00 100644 --- a/package-lock.json +++ b/package-lock.json @@ -18,11 +18,11 @@ "@aws-sdk/client-dynamodb": "^3.1030.0", "@aws-sdk/lib-dynamodb": "^3.1041.0", "@types/aws-lambda": "^8.10.149", - "@types/node": "^25.6.0", + "@types/node": "^22.19.17", "aws-cdk": "^2.1120.0", "esbuild": "^0.25.0", "ts-node": "^10.9.2", - "typescript": "^6.0.3" + "typescript": "~6.0.3" } }, "node_modules/@aws-cdk/asset-awscli-v1": { @@ -2067,14 +2067,14 @@ "license": "MIT" }, "node_modules/@types/node": { - "version": "25.6.0", - "resolved": "https://registry.npmjs.org/@types/node/-/node-25.6.0.tgz", - "integrity": "sha512-+qIYRKdNYJwY3vRCZMdJbPLJAtGjQBudzZzdzwQYkEPQd+PJGixUL5QfvCLDaULoLv+RhT3LDkwEfKaAkgSmNQ==", + "version": "22.19.17", + "resolved": "https://registry.npmjs.org/@types/node/-/node-22.19.17.tgz", + "integrity": "sha512-wGdMcf+vPYM6jikpS/qhg6WiqSV/OhG+jeeHT/KlVqxYfD40iYJf9/AE1uQxVWFvU7MipKRkRv8NSHiCGgPr8Q==", "dev": true, "license": "MIT", "peer": true, "dependencies": { - "undici-types": "~7.19.0" + "undici-types": "~6.21.0" } }, "node_modules/acorn": { @@ -2800,9 +2800,9 @@ } }, "node_modules/undici-types": { - "version": "7.19.2", - "resolved": "https://registry.npmjs.org/undici-types/-/undici-types-7.19.2.tgz", - "integrity": "sha512-qYVnV5OEm2AW8cJMCpdV20CDyaN3g0AjDlOGf1OW4iaDEx8MwdtChUp4zu4H0VP3nDRF/8RKWH+IPp9uW0YGZg==", + "version": "6.21.0", + "resolved": "https://registry.npmjs.org/undici-types/-/undici-types-6.21.0.tgz", + "integrity": "sha512-iwDZqg0QAGrg9Rav5H4n0M64c3mkR59cJ6wQp+7C4nI0gsmExaedaYLNO44eT4AtBBwjbTiGPMlt2Md0T9H9JQ==", "dev": true, "license": "MIT" }, diff --git a/package.json b/package.json index 4262c78..172c39e 100644 --- a/package.json +++ b/package.json @@ -14,11 +14,11 @@ "@aws-sdk/client-dynamodb": "^3.1030.0", "@aws-sdk/lib-dynamodb": "^3.1041.0", "@types/aws-lambda": "^8.10.149", - "@types/node": "^25.6.0", + "@types/node": "^22.19.17", "aws-cdk": "^2.1120.0", "esbuild": "^0.25.0", "ts-node": "^10.9.2", - "typescript": "^6.0.3" + "typescript": "~6.0.3" }, "dependencies": { "@cdklabs/generative-ai-cdk-constructs": "^0.1.0", -- 2.50.1 From 9d7182b3bca2abae95272124893844e81f9e62c6 Mon Sep 17 00:00:00 2001 From: Adam Moussa <166072409+amoussa1229@users.noreply.github.com> Date: Wed, 6 May 2026 19:16:01 -0400 Subject: [PATCH 3/8] Regenerate lockfile with npm 11.14.0 Fixes corrupted peer:true flags on direct dependencies caused by npm <11.6.3 bug (npm/cli#8690). --- package-lock.json | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/package-lock.json b/package-lock.json index 615eb00..75186ab 100644 --- a/package-lock.json +++ b/package-lock.json @@ -46,7 +46,6 @@ "semver" ], "license": "Apache-2.0", - "peer": true, "dependencies": { "jsonschema": "~1.4.1", "semver": "^7.7.4" @@ -215,7 +214,6 @@ "integrity": "sha512-q0dhIfdm3R9EEElaUXvoLD0EBm4KYPcJC4So/skToMPACiznNgVI2Vup4PKM716KTaDsIcF8V1EA7s33H7oWMg==", "dev": true, "license": "Apache-2.0", - "peer": true, "dependencies": { "@aws-crypto/sha256-browser": "5.2.0", "@aws-crypto/sha256-js": "5.2.0", @@ -2072,7 +2070,6 @@ "integrity": "sha512-wGdMcf+vPYM6jikpS/qhg6WiqSV/OhG+jeeHT/KlVqxYfD40iYJf9/AE1uQxVWFvU7MipKRkRv8NSHiCGgPr8Q==", "dev": true, "license": "MIT", - "peer": true, "dependencies": { "undici-types": "~6.21.0" } @@ -2142,7 +2139,6 @@ "mime-types" ], "license": "Apache-2.0", - "peer": true, "dependencies": { "@aws-cdk/asset-awscli-v1": "2.2.273", "@aws-cdk/asset-node-proxy-agent-v6": "^2.1.1", @@ -2519,8 +2515,7 @@ "version": "10.6.0", "resolved": "https://registry.npmjs.org/constructs/-/constructs-10.6.0.tgz", "integrity": "sha512-TxHOnBO5zMo/G76ykzGF/wMpEHu257TbWiIxP9K0Yv/+t70UzgBQiTqjkAsWOPC6jW91DzJI0+ehQV6xDRNBuQ==", - "license": "Apache-2.0", - "peer": true + "license": "Apache-2.0" }, "node_modules/create-require": { "version": "1.1.1", @@ -2790,7 +2785,6 @@ "integrity": "sha512-y2TvuxSZPDyQakkFRPZHKFm+KKVqIisdg9/CZwm9ftvKXLP8NRWj38/ODjNbr43SsoXqNuAisEf1GdCxqWcdBw==", "dev": true, "license": "Apache-2.0", - "peer": true, "bin": { "tsc": "bin/tsc", "tsserver": "bin/tsserver" -- 2.50.1 From 84cd633677b0e547b2a4aad93cd34027fcf7a49c Mon Sep 17 00:00:00 2001 From: Adam Moussa <166072409+amoussa1229@users.noreply.github.com> Date: Wed, 6 May 2026 19:29:00 -0400 Subject: [PATCH 4/8] Trigger CI -- 2.50.1 From f9a6e387542ba7ce2e9d75f585a42a6f96951289 Mon Sep 17 00:00:00 2001 From: Adam Moussa <166072409+amoussa1229@users.noreply.github.com> Date: Wed, 6 May 2026 19:39:14 -0400 Subject: [PATCH 5/8] Trigger CI -- 2.50.1 From 862a14a9657f1cb21e550b8925c5a9ec0de3f457 Mon Sep 17 00:00:00 2001 From: Adam Moussa <166072409+amoussa1229@users.noreply.github.com> Date: Wed, 6 May 2026 19:42:37 -0400 Subject: [PATCH 6/8] Trigger CI -- 2.50.1 From 0691cd756cb51608bbaa88f6dbbd7b98cd0c1e8e Mon Sep 17 00:00:00 2001 From: Adam Moussa <166072409+amoussa1229@users.noreply.github.com> Date: Wed, 6 May 2026 19:45:38 -0400 Subject: [PATCH 7/8] Add id-token: write permission for claude-code-action --- .github/workflows/claude-review.yaml | 1 + 1 file changed, 1 insertion(+) diff --git a/.github/workflows/claude-review.yaml b/.github/workflows/claude-review.yaml index aefbd96..38fc6a5 100644 --- a/.github/workflows/claude-review.yaml +++ b/.github/workflows/claude-review.yaml @@ -7,6 +7,7 @@ on: permissions: contents: read pull-requests: write + id-token: write jobs: review: -- 2.50.1 From 4d492f6b3ea63b2cfdabafc401a4b820f780bdcb Mon Sep 17 00:00:00 2001 From: Adam Moussa <166072409+amoussa1229@users.noreply.github.com> Date: Wed, 6 May 2026 19:47:45 -0400 Subject: [PATCH 8/8] Trigger CI -- 2.50.1