Conversation
🦋 Changeset detectedLatest commit: 742b682 The changes in this PR will be included in the next version bump. This PR includes changesets to release 3 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
|
📦 A new experimental 🧪 version v0.0.0-snapshot.1774371282 of Redocly CLI has been published for testing. Install with NPM: npm install @redocly/cli@0.0.0-snapshot.1774371282
# or
npm install @redocly/openapi-core@0.0.0-snapshot.1774371282
# or
npm install @redocly/respect-core@0.0.0-snapshot.1774371282 |
AlbinaBlazhko17
added
no changeset needed
and removed
snapshot
packages/core/package.json
Outdated
| "require": "./lib/bundle/bundle-oas.js", | ||
| "types": "./lib/bundle/bundle-oas.d.ts" | ||
| }, | ||
| "./lib/config/types.js": { |
There was a problem hiding this comment.
I'm not in favour of adding additional export endpoint. It's better to maintain a single one (plus one for bundle-oas). If needed, it's better to reexport the property from the index file.
There was a problem hiding this comment.
Yeah, will do it
tsconfig.json
Outdated
| "sourceMap": true, | ||
| "module": "ESNext", | ||
| "module": "nodenext", | ||
| "target": "ES2021", |
There was a problem hiding this comment.
Let's also try bumping the target (not necessarily in this PR though).
There was a problem hiding this comment.
For sure, let me do it.
.changeset/little-jars-look.md
Outdated
| "@redocly/openapi-core": patch | ||
| --- | ||
|
|
||
| Added export for `./lib/config/types.js` to avoid errors in import. |
There was a problem hiding this comment.
Oh, i thought we have package.json in lib/ folder, but we don't. I will change this changeset to export new type.
AlbinaBlazhko17
added
snapshot
|
📦 A new experimental 🧪 version v0.0.0-snapshot.1774861593 of Redocly CLI has been published for testing. Install with NPM: npm install @redocly/cli@0.0.0-snapshot.1774861593
# or
npm install @redocly/openapi-core@0.0.0-snapshot.1774861593
# or
npm install @redocly/respect-core@0.0.0-snapshot.1774861593 |
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 3 potential issues.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit ed5bfa0. Configure here.
3 participants
What/Why/How?
Upgraded typescript version to v6:
module,moduleResolutiontonodenextbaseUrlandpathsabort-controllerbetter-ajv-errorsfrom v1 to v2.Reference
Second part of #2618
Testing
e2e => https://github.com/Redocly/redocly/actions/runs/23543471148/job/68536960227?pr=21909
Screenshots (optional)
Check yourself
Security
Note
Medium Risk
Moderate risk because it changes TypeScript compiler settings (
nodenext, newpaths, loosenedstrict) and updates runtime-adjacent dependencies (@redocly/ajv,better-ajv-errors) that can affect schema validation behavior and module resolution.Overview
Migrates the monorepo to TypeScript 6 and updates compiler settings to
module/moduleResolution: nodenext, newertarget/lib, and newpathsmapping.Updates validation/tooling dependencies (bumps
@redocly/ajvto8.18.1, upgradesbetter-ajv-errorsto2.0.3) and adjusts Ajv usage/types viaas anycasts to accommodate the upgrade.Cleans up ESM import compatibility across packages (explicit
.jsspecifiers, updated type re-exports, JSON import assertions) and removes theabort-controllerdependency in the CLI.Reviewed by Cursor Bugbot for commit 742b682. Bugbot is set up for automated code reviews on this repo. Configure here.