Skip to content

DO-1765: Update constructs peer dependency to ^10.5.0#1638

Merged
TheOrangePuff merged 2 commits intomainfrom
fix/DO-1765_update-constructs-peer-dependency
Apr 1, 2026
Merged

DO-1765: Update constructs peer dependency to ^10.5.0#1638
TheOrangePuff merged 2 commits intomainfrom
fix/DO-1765_update-constructs-peer-dependency

Conversation

@TheOrangePuff
Copy link
Copy Markdown
Member

@TheOrangePuff TheOrangePuff commented Apr 1, 2026

Description of the proposed changes

aws-cdk-lib@2.245.0 (updated in #1636) requires constructs@^10.5.0 for the IMixin export. The previous version constraints (^10.3.0/^10.4.2) resolved to 10.4.5 which does not include IMixin, causing all package builds to fail with:

error TS2305: Module '"constructs"' has no exported member 'IMixin'.

This PR:

  • Updates all constructs peer dependencies to ^10.5.0
  • Adds a root-level constructs resolution (^10.5.0) to force deduplication — @aws-solutions-constructs/aws-lambda-sqs-lambda bundles constructs as a direct dependency, creating duplicate types that caused prerender-fargate build failures

Notes to reviewers

ℹ️ When you've finished leaving feedback, please add a final comment to the PR tagging the author, letting them know that you have finished leaving feedback

aws-cdk-lib@2.245.0 requires constructs@^10.5.0 for the IMixin
export. The previous version constraint (^10.3.0/^10.4.2) resolved
to 10.4.5 which does not include IMixin, breaking all package builds.
…tructs dependency

@aws-solutions-constructs/aws-lambda-sqs-lambda bundles constructs as
a direct dependency, creating duplicate types that cause build failures.
A root-level resolution forces all packages to share a single copy.
@TheOrangePuff TheOrangePuff requested a review from a team as a code owner April 1, 2026 23:32
Copy link
Copy Markdown
Contributor

@aaronmedina-dev aaronmedina-dev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@TheOrangePuff TheOrangePuff merged commit 6107318 into main Apr 1, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants