Skip to content

chore: expand exported paths#2636

Merged
vital4u4ok merged 3 commits intomainfrom
chore/expand-exported-paths
Mar 4, 2026
Merged

chore: expand exported paths#2636
vital4u4ok merged 3 commits intomainfrom
chore/expand-exported-paths

Conversation

@vital4u4ok
Copy link
Contributor

@vital4u4ok vital4u4ok commented Mar 4, 2026

What/Why/How?

This pull request adds new entry points to the packages/respect-core/package.json exports, making it easier to import specific modules directly.

New module exports:

  • Added export for ./runtime-expressions, allowing direct import of runtime expressions module and its types.
  • Added export for ./run, enabling direct import of the run module and its types.

This change helps significantly reduce Replay's bundle size in openapi-docs.

Reference

Testing

Screenshots (optional)

Check yourself

  • Code changed? - Tested with Redoc/Realm/Reunite (internal)
  • All new/updated code is covered by tests
  • New package installed? - Tested in different environments (browser/node)
  • Documentation update considered

Security

  • The security impact of the change has been considered
  • Code follows company security practices and guidelines

@changeset-bot
Copy link

changeset-bot bot commented Mar 4, 2026

⚠️ No Changeset found

Latest commit: 7f30c56

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@vital4u4ok vital4u4ok added the snapshot Create experimental release PR label Mar 4, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Mar 4, 2026

📦 A new experimental 🧪 version v0.0.0-snapshot.1772633752 of Redocly CLI has been published for testing.

Install with NPM:

npm install @redocly/cli@0.0.0-snapshot.1772633752
# or
npm install @redocly/openapi-core@0.0.0-snapshot.1772633752
# or
npm install @redocly/respect-core@0.0.0-snapshot.1772633752

⚠️ Note: This is a development build and may contain unstable features.

@github-actions
Copy link
Contributor

github-actions bot commented Mar 4, 2026

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 79.15% (🎯 79%) 6381 / 8061
🔵 Statements 78.6% (🎯 78%) 6593 / 8387
🔵 Functions 82.4% (🎯 82%) 1307 / 1586
🔵 Branches 71.01% (🎯 70%) 4344 / 6117
File CoverageNo changed files found.
Generated in workflow #8913 for commit 7f30c56 by the Vitest Coverage Report Action

@github-actions
Copy link
Contributor

github-actions bot commented Mar 4, 2026

CLI Version Mean Time ± Std Dev (s) Relative Performance (Lower is Faster)
cli-latest 3.659s ± 0.034s ▓ 1.01x
cli-next 3.616s ± 0.036s ▓ 1.00x (Fastest)

@vital4u4ok vital4u4ok marked this pull request as ready for review March 4, 2026 14:30
@vital4u4ok vital4u4ok requested a review from a team as a code owner March 4, 2026 14:30
"import": "./lib/index.js",
"types": "./lib/index.d.ts"
},
"./runtime-expressions": {
Copy link
Collaborator

Choose a reason for hiding this comment

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

I'd suggest propagating the exports to the root in the code instead of creating new entrypoints. This should be a safer approach in terms of maintenance.

Copy link
Collaborator

@tatomyr tatomyr Mar 4, 2026

Choose a reason for hiding this comment

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

Unless it's for smaller bundle size. If this is the case, please update the PR's description.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, this is for bundle size reduction

@DmitryAnansky DmitryAnansky self-requested a review March 4, 2026 14:38
@DmitryAnansky DmitryAnansky self-requested a review March 4, 2026 14:54
Copy link
Contributor

@DmitryAnansky DmitryAnansky left a comment

Choose a reason for hiding this comment

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

@vital4u4ok please review the Check yourself and Security section in PR description.

@vital4u4ok vital4u4ok requested a review from tatomyr March 4, 2026 15:31
@vital4u4ok vital4u4ok merged commit 6edec68 into main Mar 4, 2026
40 of 41 checks passed
@vital4u4ok vital4u4ok deleted the chore/expand-exported-paths branch March 4, 2026 16:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no changeset needed snapshot Create experimental release PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants