Skip to content

using fresh fork to splitting false at esbuild#1051

Open
guitavano wants to merge 1 commit intomainfrom
tavano/using-fresh-fork
Open

using fresh fork to splitting false at esbuild#1051
guitavano wants to merge 1 commit intomainfrom
tavano/using-fresh-fork

Conversation

@guitavano
Copy link
Contributor

@guitavano guitavano commented Feb 4, 2026

Summary by cubic

Switched $fresh/ import override to the deco-cx fork via jsDelivr, still pinned to 1.7.3, to prevent accidental Fresh v2 resolution and fix esbuild builds with splitting: false.

Updated scripts/codemod.ts to map "$fresh/" -> "https://cdn.jsdelivr.net/gh/deco-cx/fresh@1.7.3/".

Written for commit 96abe61. Summary will update on new commits.

Summary by CodeRabbit

  • Chores
    • Updated package resolution configuration to use an alternate host for a core dependency, switching from the default source to a managed fork while maintaining version compatibility.

@github-actions
Copy link
Contributor

github-actions bot commented Feb 4, 2026

Tagging Options

Should a new tag be published when this PR is merged?

  • 👍 for Patch 1.137.2 update
  • 🎉 for Minor 1.138.0 update
  • 🚀 for Major 2.0.0 update

@coderabbitai
Copy link

coderabbitai bot commented Feb 4, 2026

📝 Walkthrough

Walkthrough

The change updates the resolution source for the "$fresh/" package namespace from deno.land/x/fresh@1.7.3 to a forked version hosted on jsdelivr (https://cdn.jsdelivr.net/gh/deco-cx/fresh@1.7.3/), maintaining version consistency while switching to a deco-managed variant.

Changes

Cohort / File(s) Summary
Fresh Package Override
scripts/codemod.ts
Updated the "$fresh/" package path resolution from deno.land/x to a jsdelivr-hosted fork, switching the CDN source while preserving the version number.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested reviewers

  • vitoUwu

Poem

🐰 A path in the code, so simple and clean,
From deno to jsdelivr, a redirect unseen,
Fresh stays the same, just a new source of air,
The fork hops along with version to spare! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Title check ❓ Inconclusive The title is vague and does not clearly convey the main change. While it mentions 'fresh fork', the phrase 'splitting false at esbuild' is unclear and doesn't accurately describe the package path override change from deno.land to jsdelivr. Revise the title to be more descriptive and clear, such as 'Switch $fresh/ package path to deco-managed fork on jsdelivr' to better communicate the actual change.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch tavano/using-fresh-fork

Important

Action Needed: IP Allowlist Update

If your organization protects your Git platform with IP whitelisting, please add the new CodeRabbit IP address to your allowlist:

  • 136.113.208.247/32 (new)
  • 34.170.211.100/32
  • 35.222.179.152/32

Reviews will stop working after February 8, 2026 if the new IP is not added to your allowlist.


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.

Copy link

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 1 file

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