Skip to content

fix malformed route id decoding#7474

Open
andrewkernel wants to merge 1 commit into
refinedev:mainfrom
andrewkernel:codex/safe-decode-route-id
Open

fix malformed route id decoding#7474
andrewkernel wants to merge 1 commit into
refinedev:mainfrom
andrewkernel:codex/safe-decode-route-id

Conversation

@andrewkernel

Copy link
Copy Markdown

PR Checklist

Please check if your PR fulfills the following requirements:

Bugs / Features

What is the current behavior?

Malformed percent-encoded route params can throw a URIError while route ids or the to query target are decoded.

What is the new behavior?

Route id and to decoding now preserves valid decoding behavior while falling back to the original string for malformed encoded values. This prevents malformed route params from crashing parsing in @refinedev/core, @refinedev/react-router, and @refinedev/remix-router.

Notes for reviewers

Validation run locally:

  • vitest run src/definitions/helpers/handleUseParams/index.spec.tsx in packages/core
  • vitest run in packages/react-router
  • vitest run in packages/remix-router
  • biome check on touched source/test files
  • tsup and declaration generation for core, react-router, and remix-router

@changeset-bot

changeset-bot Bot commented Jun 29, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 1ae6d52

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 3 packages
Name Type
@refinedev/core Patch
@refinedev/react-router Patch
@refinedev/remix-router Patch

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

@andrewkernel andrewkernel changed the title [codex] fix malformed route id decoding fix malformed route id decoding Jun 29, 2026
@andrewkernel andrewkernel marked this pull request as ready for review June 29, 2026 07:39
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