Skip to content

Fix utils export path#37

Open
itosa-kazu wants to merge 1 commit into
rishi-raj-jain:masterfrom
itosa-kazu:codex/fix-utils-export-path
Open

Fix utils export path#37
itosa-kazu wants to merge 1 commit into
rishi-raj-jain:masterfrom
itosa-kazu:codex/fix-utils-export-path

Conversation

@itosa-kazu

@itosa-kazu itosa-kazu commented Jun 9, 2026

Copy link
Copy Markdown

Summary

  • point the ./utils export to dist/utils.js and dist/utils.d.ts
  • keep the package export map aligned with the files produced by the astro-font build

Verification

  • npm run build in packages/astro-font
  • npm pack the fixed package, install the tarball into a clean project, and import astro-font/utils at runtime
  • compile a clean TypeScript project importing generateFonts from astro-font/utils with npx tsc --noEmit

The published package currently points ./utils at dist/util.js and dist/util.d.ts, but the tarball contains dist/utils.js and dist/utils.d.ts.

Summary by CodeRabbit

  • Chores
    • Corrected the package exports configuration for the utilities module to reference the correct build artifacts.

@coderabbitai

coderabbitai Bot commented Jun 9, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 4b4b5dd2-ea2b-4c9c-8cc8-200b3ed50deb

📥 Commits

Reviewing files that changed from the base of the PR and between 32d0cd3 and 489c63f.

📒 Files selected for processing (1)
  • packages/astro-font/package.json

📝 Walkthrough

Walkthrough

The astro-font package export for ./utils is corrected to reference the actual pluralized build artifacts at ./dist/utils.d.ts and ./dist/utils.js, aligning the public API with the build output.

Changes

Package export fix

Layer / File(s) Summary
Export path update
packages/astro-font/package.json
The exports["./utils"] entry types and default fields are updated from ./dist/util.* to ./dist/utils.* to match the actual build artifact names.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A path was wrong, now it's right,
From util to utils—what a sight!
The exports now glow with clarity bright,
One tiny fix makes the package feel light! ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Fix utils export path' directly and clearly summarizes the main change: correcting the package export mapping from singular 'util' to plural 'utils' paths.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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.

1 participant