Repository navigation
chore(deps): update fs-extra to 11.3.6 across all packages - #328
Merged
Merged
Conversation
X-Guardian
force-pushed
the
chore/bump-fs-extra
branch
from
July 13, 2026 14:05
2b38d2f to
7b67260
Compare
X-Guardian
marked this pull request as ready for review
July 13, 2026 14:34
X-Guardian
force-pushed
the
chore/bump-fs-extra
branch
from
July 14, 2026 09:23
7b67260 to
d8f238d
Compare
jsteinich
approved these changes
Jul 17, 2026
so0k
pushed a commit
that referenced
this pull request
Aug 7, 2026
🤖 Release PR — merge to cut a new release. Kept open and rebased as commits land on `main`. --- <details><summary>0.24.0</summary> ## [0.24.0](v0.23.4...v0.24.0) (2026-08-06) ### ⚠ BREAKING CHANGES * **lib:** validate Terraform function versions by default ([#362](#362)) * **deps:** Require Node 22 minimum ([#345](#345)) * **lib:** preserve symlinks in TerraformAsset walkers ([#321](#321)) * **cli:** replace node-fetch with undici ([#306](#306)) ### Features * **lib:** canonical asset hashes behind the canonicalAssetHashes feature flag ([#323](#323)) ([76dd4ff](76dd4ff)) * **lib:** validate Terraform function versions by default ([#362](#362)) ([4ac0736](4ac0736)) * support newer provider plugin-protocol features via targetVersions (RFC-04) ([#296](#296)) ([90322f9](90322f9)) ### Bug Fixes * **cli:** don't downgrade prebuilt providers on a transient registry failure ([#298](#298)) ([a960c5c](a960c5c)) * **cli:** include dev dependencies in npm version lookup ([#280](#280)) ([955204a](955204a)) * **docs:** fix stale constructs pin breaking with()/IMixin docs ([#305](#305)) ([605cf63](605cf63)) * **gha:** Allow pnpm to update the lockfile after package updates ([#318](#318)) ([a899b7c](a899b7c)) * **gha:** Fix pnpm upgrade workflow ([#335](#335)) ([e1a69fc](e1a69fc)) * **gha:** flip merged release PR label to autorelease: tagged ([#302](#302)) ([8d64f6c](8d64f6c)) * **gha:** mint the Go-publish token from the CDKTN Maintainers app ([#369](#369)) ([71921ce](71921ce)) * **gha:** mint the Go-publish token from the open-constructs-cdktn App ([#368](#368)) ([179f10c](179f10c)) * **gha:** pass the Go-publish App token as x-access-token userinfo ([#370](#370)) ([4e3ff19](4e3ff19)) * **lib:** Disallow constructs 10.8 until support can be added ([#363](#363)) ([8bdae0d](8bdae0d)) * **lib:** preserve symlinks in TerraformAsset walkers ([#321](#321)) ([6360e20](6360e20)) * typo in `moveFromId` JSDoc ([#355](#355)) ([e1cf8ce](e1cf8ce)) ### Miscellaneous Chores * **cli:** replace Ink + React with smaller-tree CLI libraries ([#264](#264)) ([a6aff7e](a6aff7e)) * **cli:** replace node-fetch with undici ([#306](#306)) ([1317141](1317141)) * **deps:** bump glob to 13.0.6 ([#307](#307)) ([47ee2bb](47ee2bb)) * **deps:** bump the github-actions-backward-compatible group with 2 updates ([#295](#295)) ([eab2a01](eab2a01)) * **deps:** replace lerna with nx ([#315](#315)) ([94999fc](94999fc)) * **deps:** Require Node 22 minimum ([#345](#345)) ([2bf315d](2bf315d)) * **deps:** update ci-info to 4.4.0 across all packages ([#329](#329)) ([557a163](557a163)) * **deps:** update fs-extra to 11.3.6 across all packages ([#328](#328)) ([95753a9](95753a9)) * **deps:** update minimatch to 10.2.5 ([#330](#330)) ([d1c1c53](d1c1c53)) * **deps:** Update sscaff to v2.0.388 ([#331](#331)) ([38ea0ba](38ea0ba)) * **deps:** Update zod to v4.4.3 ([#332](#332)) ([c10ee35](c10ee35)) * **deps:** Upgrade dependencies ([#347](#347)) ([45d3a66](45d3a66)) * remove cdktf from tests ([#277](#277)) ([dc9a8e9](dc9a8e9)) * ship Terraform 1.15.8 in the jsii-terraform image ([#367](#367)) ([deaa9b0](deaa9b0)) * Upgrade dependencies for lib ([#348](#348)) ([d616f17](d616f17)) </details> --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). Co-authored-by: open-constructs-cdktn[bot] <291052431+open-constructs-cdktn[bot]@users.noreply.github.com>
X-Guardian
added a commit
to X-Guardian/cdk-terrain
that referenced
this pull request
Aug 10, 2026
…tructs#328) ### Description Bumps `fs-extra` to `11.3.6` (and `@types/fs-extra` to `11.0.4`) in every package that depends on it, so the whole monorepo resolves a single copy of each. Before this change the versions were inconsistent: five packages were already on the `11.x` line (`11.3.4` / `^11.3.0`), while three — `@cdktn/provider-generator`, `@cdktn/cli-core`, and `cdktn-cli` — plus the standalone `test/` harness were still pinned to `fs-extra@8.1.0` / `@types/fs-extra@8.1.5`. This aligns all of them on the current release. **Packages updated:** | Package | `fs-extra` | `@types/fs-extra` | | --- | --- | --- | | `@cdktn/provider-generator` | `8.1.0` → `11.3.6` | `8.1.5` → `11.0.4` | | `@cdktn/cli-core` | `8.1.0` → `11.3.6` | `8.1.5` → `11.0.4` | | `cdktn-cli` | `8.1.0` → `11.3.6` | `8.1.5` → `11.0.4` | | `test/` (non-workspace) | `^8.1.0` → `^11.3.6` | `^8.1.5` → `^11.0.4` | | `@cdktn/provider-schema` | `11.3.4` → `11.3.6` | `11.0.4` (unchanged) | | `@cdktn/commons` | `11.3.4` → `11.3.6` | `11.0.4` (unchanged) | | `@cdktn/hcl2cdk` | `11.3.4` → `11.3.6` | `11.0.4` (unchanged) | | `@cdktn/hcl2json` | `11.3.4` → `11.3.6` | `11.0.4` (unchanged) | | `@cdktn/hcl-tools` | `^11.3.0` → `^11.3.6` | `11.0.4` (unchanged) | **Two source changes were required.** The three packages crossing the `8.x` → `11.x` boundary were checked against the fs-extra breaking changes in that window. The `@types/fs-extra@11` typings are stricter than `@types/fs-extra@8` and flagged two existing calls that passed a `recursive` option to a copy — `fs.copy`/`fs.copySync` are always recursive, so `recursive` was never a valid `CopyOptions`/`CopyOptionsSync` key; the looser v8 typings silently accepted it. Removing it is a no-op at runtime: - `@cdktn/provider-generator/src/get/constructs-maker.ts` — `error TS2769` on `fs.copy(source, target, { recursive: true, overwrite: false })` → `fs.copy(source, target, { overwrite: false })`. - `cdktn-cli/src/bin/cmds/helper/init.ts` — `error TS2353` on `fs.copySync(src, dest, { recursive: true })` → `fs.copySync(src, dest)`. No behavioural change was needed for the other differences. `move()` defaulting to `overwrite: false` from v9 onward doesn't affect the three `fs.move` call sites in the same file, which already pass `{ overwrite: true }` explicitly. Every other method used across these packages (`existsSync`, `readFile(Sync)`, `writeFile(Sync)`, `remove(Sync)`, `pathExists`, `mkdirp`/`mkdirs`, `copy(Sync)`, `stat`, `readJson(Sync)`, `appendFileSync`, `rmSync`, `unlinkSync`, `accessSync`, `constants`) is unchanged from 8 through 11. The rest of the 8→11 range only dropped support for Node.js < 14, which the repo already requires. ### Checklist - [ ] I have updated the PR title to match [CDKTN's style guide](https://github.com/open-constructs/cdk-terrain/blob/main/CONTRIBUTING.md#pull-requests-1) - [ ] I have run the linter on my code locally - [ ] I have performed a self-review of my code - [ ] I have commented my code, particularly in hard-to-understand areas - [ ] I have made corresponding changes to the [documentation](https://github.com/open-constructs/cdk-terrain-docs/tree/main/content) if applicable - [ ] My changes generate no new warnings - [ ] I have added tests that prove my fix is effective or that my feature works if applicable - [ ] New and existing unit tests pass locally with my changes --------- Co-authored-by: Simon Heather <simon.heather@yulife.com>
X-Guardian
pushed a commit
to X-Guardian/cdk-terrain
that referenced
this pull request
Aug 10, 2026
🤖 Release PR — merge to cut a new release. Kept open and rebased as commits land on `main`. --- <details><summary>0.24.0</summary> ## [0.24.0](open-constructs/cdk-terrain@v0.23.4...v0.24.0) (2026-08-06) ### ⚠ BREAKING CHANGES * **lib:** validate Terraform function versions by default ([open-constructs#362](open-constructs#362)) * **deps:** Require Node 22 minimum ([open-constructs#345](open-constructs#345)) * **lib:** preserve symlinks in TerraformAsset walkers ([open-constructs#321](open-constructs#321)) * **cli:** replace node-fetch with undici ([open-constructs#306](open-constructs#306)) ### Features * **lib:** canonical asset hashes behind the canonicalAssetHashes feature flag ([open-constructs#323](open-constructs#323)) ([76dd4ff](open-constructs@76dd4ff)) * **lib:** validate Terraform function versions by default ([open-constructs#362](open-constructs#362)) ([4ac0736](open-constructs@4ac0736)) * support newer provider plugin-protocol features via targetVersions (RFC-04) ([open-constructs#296](open-constructs#296)) ([90322f9](open-constructs@90322f9)) ### Bug Fixes * **cli:** don't downgrade prebuilt providers on a transient registry failure ([open-constructs#298](open-constructs#298)) ([a960c5c](open-constructs@a960c5c)) * **cli:** include dev dependencies in npm version lookup ([open-constructs#280](open-constructs#280)) ([955204a](open-constructs@955204a)) * **docs:** fix stale constructs pin breaking with()/IMixin docs ([open-constructs#305](open-constructs#305)) ([605cf63](open-constructs@605cf63)) * **gha:** Allow pnpm to update the lockfile after package updates ([open-constructs#318](open-constructs#318)) ([a899b7c](open-constructs@a899b7c)) * **gha:** Fix pnpm upgrade workflow ([open-constructs#335](open-constructs#335)) ([e1a69fc](open-constructs@e1a69fc)) * **gha:** flip merged release PR label to autorelease: tagged ([open-constructs#302](open-constructs#302)) ([8d64f6c](open-constructs@8d64f6c)) * **gha:** mint the Go-publish token from the CDKTN Maintainers app ([open-constructs#369](open-constructs#369)) ([71921ce](open-constructs@71921ce)) * **gha:** mint the Go-publish token from the open-constructs-cdktn App ([open-constructs#368](open-constructs#368)) ([179f10c](open-constructs@179f10c)) * **gha:** pass the Go-publish App token as x-access-token userinfo ([open-constructs#370](open-constructs#370)) ([4e3ff19](open-constructs@4e3ff19)) * **lib:** Disallow constructs 10.8 until support can be added ([open-constructs#363](open-constructs#363)) ([8bdae0d](open-constructs@8bdae0d)) * **lib:** preserve symlinks in TerraformAsset walkers ([open-constructs#321](open-constructs#321)) ([6360e20](open-constructs@6360e20)) * typo in `moveFromId` JSDoc ([open-constructs#355](open-constructs#355)) ([e1cf8ce](open-constructs@e1cf8ce)) ### Miscellaneous Chores * **cli:** replace Ink + React with smaller-tree CLI libraries ([open-constructs#264](open-constructs#264)) ([a6aff7e](open-constructs@a6aff7e)) * **cli:** replace node-fetch with undici ([open-constructs#306](open-constructs#306)) ([1317141](open-constructs@1317141)) * **deps:** bump glob to 13.0.6 ([open-constructs#307](open-constructs#307)) ([47ee2bb](open-constructs@47ee2bb)) * **deps:** bump the github-actions-backward-compatible group with 2 updates ([open-constructs#295](open-constructs#295)) ([eab2a01](open-constructs@eab2a01)) * **deps:** replace lerna with nx ([open-constructs#315](open-constructs#315)) ([94999fc](open-constructs@94999fc)) * **deps:** Require Node 22 minimum ([open-constructs#345](open-constructs#345)) ([2bf315d](open-constructs@2bf315d)) * **deps:** update ci-info to 4.4.0 across all packages ([open-constructs#329](open-constructs#329)) ([557a163](open-constructs@557a163)) * **deps:** update fs-extra to 11.3.6 across all packages ([open-constructs#328](open-constructs#328)) ([95753a9](open-constructs@95753a9)) * **deps:** update minimatch to 10.2.5 ([open-constructs#330](open-constructs#330)) ([d1c1c53](open-constructs@d1c1c53)) * **deps:** Update sscaff to v2.0.388 ([open-constructs#331](open-constructs#331)) ([38ea0ba](open-constructs@38ea0ba)) * **deps:** Update zod to v4.4.3 ([open-constructs#332](open-constructs#332)) ([c10ee35](open-constructs@c10ee35)) * **deps:** Upgrade dependencies ([open-constructs#347](open-constructs#347)) ([45d3a66](open-constructs@45d3a66)) * remove cdktf from tests ([open-constructs#277](open-constructs#277)) ([dc9a8e9](open-constructs@dc9a8e9)) * ship Terraform 1.15.8 in the jsii-terraform image ([open-constructs#367](open-constructs#367)) ([deaa9b0](open-constructs@deaa9b0)) * Upgrade dependencies for lib ([open-constructs#348](open-constructs#348)) ([d616f17](open-constructs@d616f17)) </details> --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). Co-authored-by: open-constructs-cdktn[bot] <291052431+open-constructs-cdktn[bot]@users.noreply.github.com>
Contributor
|
I'm going to lock this pull request because it has been closed for 30 days. This helps our maintainers find and focus on the active issues. If you've found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Bumps
fs-extrato11.3.6(and@types/fs-extrato11.0.4) in every package that depends on it, so the whole monorepo resolves a single copy of each.Before this change the versions were inconsistent: five packages were already on the
11.xline (11.3.4/^11.3.0), while three —@cdktn/provider-generator,@cdktn/cli-core, andcdktn-cli— plus the standalonetest/harness were still pinned tofs-extra@8.1.0/@types/fs-extra@8.1.5. This aligns all of them on the current release.Packages updated:
fs-extra@types/fs-extra@cdktn/provider-generator8.1.0→11.3.68.1.5→11.0.4@cdktn/cli-core8.1.0→11.3.68.1.5→11.0.4cdktn-cli8.1.0→11.3.68.1.5→11.0.4test/(non-workspace)^8.1.0→^11.3.6^8.1.5→^11.0.4@cdktn/provider-schema11.3.4→11.3.611.0.4(unchanged)@cdktn/commons11.3.4→11.3.611.0.4(unchanged)@cdktn/hcl2cdk11.3.4→11.3.611.0.4(unchanged)@cdktn/hcl2json11.3.4→11.3.611.0.4(unchanged)@cdktn/hcl-tools^11.3.0→^11.3.611.0.4(unchanged)Two source changes were required. The three packages crossing the
8.x→11.xboundary were checked against the fs-extra breaking changes in that window. The@types/fs-extra@11typings are stricter than@types/fs-extra@8and flagged two existing calls that passed arecursiveoption to a copy —fs.copy/fs.copySyncare always recursive, sorecursivewas never a validCopyOptions/CopyOptionsSynckey; the looser v8 typings silently accepted it. Removing it is a no-op at runtime:@cdktn/provider-generator/src/get/constructs-maker.ts—error TS2769onfs.copy(source, target, { recursive: true, overwrite: false })→fs.copy(source, target, { overwrite: false }).cdktn-cli/src/bin/cmds/helper/init.ts—error TS2353onfs.copySync(src, dest, { recursive: true })→fs.copySync(src, dest).No behavioural change was needed for the other differences.
move()defaulting tooverwrite: falsefrom v9 onward doesn't affect the threefs.movecall sites in the same file, which already pass{ overwrite: true }explicitly. Every other method used across these packages (existsSync,readFile(Sync),writeFile(Sync),remove(Sync),pathExists,mkdirp/mkdirs,copy(Sync),stat,readJson(Sync),appendFileSync,rmSync,unlinkSync,accessSync,constants) is unchanged from 8 through 11. The rest of the 8→11 range only dropped support for Node.js < 14, which the repo already requires.Checklist