Skip to content

chore(deps): bump @remix-run/* to 2.17.5#4102

Merged
nicktrn merged 1 commit into
mainfrom
chore/bump-remix-2-17-5
Jul 1, 2026
Merged

chore(deps): bump @remix-run/* to 2.17.5#4102
nicktrn merged 1 commit into
mainfrom
chore/bump-remix-2-17-5

Conversation

@nicktrn

@nicktrn nicktrn commented Jul 1, 2026

Copy link
Copy Markdown
Collaborator

Bumps the @remix-run/* family in the webapp from 2.17.4 to 2.17.5 to keep dependencies current.

2.17.5 pulls @remix-run/router 1.23.2 → 1.23.3, so the local route-matching perf patch was rebased onto 1.23.3 (regenerated via pnpm patch). It is functionally identical to the previous one - the only difference is that 1.23.3 already hoists decodePath out of the match loop upstream, so that hunk is dropped; the per-route-tree branch cache and the compiled-path cache are unchanged. Also updated the @remix-run/dev>tar-fs override key to track the new dev version.

Verified locally against latest main: typecheck --filter webapp passes, --frozen-lockfile is consistent, and the dev server boots and server-renders pages cleanly (route matching exercised via the patched router).

@nicktrn nicktrn self-assigned this Jul 1, 2026
@changeset-bot

changeset-bot Bot commented Jul 1, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 564993f

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

@coderabbitai

coderabbitai Bot commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Caution

Review failed

Failed to post review comments.

GitHub was unavailable or timed out while CodeRabbit was posting the review. Please request a new review later if the pull request still needs one. Use @coderabbitai full review to retry the review.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 62d3b2ca-6ead-413e-af46-e055fa075702

📥 Commits

Reviewing files that changed from the base of the PR and between 7851f5c and 564993f.

⛔ Files ignored due to path filters (1)
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (3)
  • apps/webapp/package.json
  • package.json
  • patches/@remix-run__router@1.23.3.patch
⏰ Context from checks skipped due to timeout. (36)
  • GitHub Check: webapp / 🧪 Unit Tests: Webapp (9, 10)
  • GitHub Check: webapp / 🧪 Unit Tests: Webapp (6, 10)
  • GitHub Check: webapp / 🧪 Unit Tests: Webapp (7, 10)
  • GitHub Check: webapp / 🧪 Unit Tests: Webapp (5, 10)
  • GitHub Check: webapp / 🧪 Unit Tests: Webapp (10, 10)
  • GitHub Check: webapp / 🧪 Unit Tests: Webapp (3, 10)
  • GitHub Check: webapp / 🧪 Unit Tests: Webapp (2, 10)
  • GitHub Check: internal / 🧪 Unit Tests: Internal (2, 12)
  • GitHub Check: webapp / 🧪 Unit Tests: Webapp (1, 10)
  • GitHub Check: webapp / 🧪 Unit Tests: Webapp (4, 10)
  • GitHub Check: e2e / 🧪 CLI v3 tests (blacksmith-4vcpu-windows-2025 - npm)
  • GitHub Check: internal / 🧪 Unit Tests: Internal (6, 12)
  • GitHub Check: internal / 🧪 Unit Tests: Internal (9, 12)
  • GitHub Check: webapp / 🧪 Unit Tests: Webapp (8, 10)
  • GitHub Check: internal / 🧪 Unit Tests: Internal (4, 12)
  • GitHub Check: internal / 🧪 Unit Tests: Internal (11, 12)
  • GitHub Check: internal / 🧪 Unit Tests: Internal (12, 12)
  • GitHub Check: internal / 🧪 Unit Tests: Internal (10, 12)
  • GitHub Check: internal / 🧪 Unit Tests: Internal (1, 12)
  • GitHub Check: internal / 🧪 Unit Tests: Internal (7, 12)
  • GitHub Check: internal / 🧪 Unit Tests: Internal (3, 12)
  • GitHub Check: internal / 🧪 Unit Tests: Internal (8, 12)
  • GitHub Check: internal / 🧪 Unit Tests: Internal (5, 12)
  • GitHub Check: packages / 🧪 Unit Tests: Packages (1, 3)
  • GitHub Check: sdk-compat / Bun Runtime
  • GitHub Check: sdk-compat / Node.js 22.12 (blacksmith-4vcpu-ubuntu-2404)
  • GitHub Check: e2e / 🧪 CLI v3 tests (blacksmith-4vcpu-ubuntu-2404 - pnpm)
  • GitHub Check: packages / 🧪 Unit Tests: Packages (3, 3)
  • GitHub Check: e2e / 🧪 CLI v3 tests (blacksmith-4vcpu-windows-2025 - pnpm)
  • GitHub Check: e2e / 🧪 CLI v3 tests (blacksmith-4vcpu-ubuntu-2404 - npm)
  • GitHub Check: sdk-compat / Cloudflare Workers
  • GitHub Check: packages / 🧪 Unit Tests: Packages (2, 3)
  • GitHub Check: e2e-webapp / 🧪 E2E Tests: Webapp
  • GitHub Check: typecheck / typecheck
  • GitHub Check: Analyze (javascript-typescript)
  • GitHub Check: Build and publish previews
🧰 Additional context used
📓 Path-based instructions (2)
package.json

📄 CodeRabbit inference engine (CLAUDE.md)

When adding dependencies, edit package.json directly instead of using pnpm add; then run pnpm i from the repo root.

Files:

  • package.json
**/package.json

📄 CodeRabbit inference engine (CLAUDE.md)

When adding zod to any package, use the exact repository-pinned version (3.25.76) and never a different version or a range.

Files:

  • package.json
  • apps/webapp/package.json
🧠 Learnings (1)
📚 Learning: 2026-04-27T16:46:03.861Z
Learnt from: nicktrn
Repo: triggerdotdev/trigger.dev PR: 3456
File: apps/webapp/package.json:152-152
Timestamp: 2026-04-27T16:46:03.861Z
Learning: In `apps/webapp/package.json`, treat the `effect` npm package as an intentional runtime dependency (not unused/misplaced) for the Schedule + Fiber-based metadata update logic. This should apply when reviewing `apps/webapp` code paths used by `apps/webapp/app/utils/updateMetadata.server.ts` (and closely related modules) that use Effect APIs such as `Duration.divide`, `STM.cond`, namespace exports for `Effect`/`Schedule`/`Duration`/`Fiber`, and the `Fiber.RuntimeFiber` type.

Applied to files:

  • apps/webapp/package.json

Walkthrough

This pull request bumps Remix-related dependency versions in apps/webapp/package.json and the root package.json from 2.17.4/1.23.2 to 2.17.5/1.23.3, updates the pnpm patchedDependencies and overrides entries to reference the new versions, and adds a new patch file for @remix-run/router@1.23.3 that introduces memoization caching for route branch computation, path decoding, and compiled path matching in the bundled router code.

Changes

Cohort / File(s) Summary
Dependency version bumps (apps/webapp/package.json, package.json) Bumped @remix-run/* runtime and dev dependency versions to 2.17.5/^1.23.3; updated pnpm patchedDependencies and overrides keys accordingly
Router patch (patches/@remix-run__router@1.23.3.patch) Added WeakMap cache for flattened/ranked route branches, Map cache for compiled paths, and hoisted decodePath computation outside the branch loop

Related PRs: None identified.

Suggested labels: dependencies, patch

Suggested reviewers: None identified.

🐰 A hop, a skip, a version raised,
Remix routes now run unfazed,
Caches tucked in WeakMap's nest,
compilePath takes a well-earned rest,
Carrots patched, the router's blessed.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description is missing required template sections like Closes #issue, checklist, changelog, and screenshots. Restructure the PR body to match the template and add Closes #issue, checklist items, testing steps, changelog, and screenshots.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes the dependency bump and matches the main change in the PR.
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.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/bump-remix-2-17-5

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.

@nicktrn nicktrn enabled auto-merge (squash) July 1, 2026 17:51

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Open in Devin Review

@nicktrn nicktrn merged commit 6cf7677 into main Jul 1, 2026
97 of 99 checks passed
@nicktrn nicktrn deleted the chore/bump-remix-2-17-5 branch July 1, 2026 18:06
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.

2 participants