Update react-router-dom 7.0.1 → 7.1.0 (minor) #265

depfu · 2024-12-21T23:13:55Z

Here is everything you need to know about this update. Please take a good look at what changed and the test results before merging this pull request.

What changed?

✳️ react-router-dom (7.0.1 → 7.1.0) · Repo · Changelog

Release Notes

7.1.0 (from changelog)

Date: 2024-12-20

Minor Changes

Add support for Vite v6 (#12469)

Patch Changes

react-router - Throw unwrapped Single Fetch redirect to align with pre-Single Fetch behavior (#12506)

react-router - Ignore redirects when inferring loader data types (#12527)

react-router - Remove <Link prefetch> warning which suffers from false positives in a lazy route discovery world (#12485)

create-react-router - Fix missing fs-extra dependency (#12556)

@react-router/dev/@react-router/serve - Properly initialize NODE_ENV if not already set for compatibility with React 19 (#12578)

@react-router/dev - Remove the leftover/unused abortDelay prop from ServerRouter and update the default entry.server.tsx to use the new streamTimeout value for Single Fetch (#12478)

The abortDelay functionality was removed in v7 as it was coupled to the defer implementation from Remix v2, but this removal of this prop was missed

If you were still using this prop in your entry.server file, it's likely your app is not aborting streams as you would expect and you will need to adopt the new streamTimeout value introduced with Single Fetch

@react-router/fs-routes - Throw error in flatRoutes if routes directory is missing (#12407)

Changes by Package

create-react-router

react-router

@react-router/architect

@react-router/cloudflare

@react-router/dev

@react-router/express

@react-router/fs-routes

@react-router/node

@react-router/remix-config-routes-adapter

@react-router/serve

Full Changelog: v7.0.2...v7.1.0

7.0.2 (from changelog)

Date: 2024-12-02

Patch Changes

Temporarily only use one build in export map so packages can have a peer dependency on react router (#12437)

Support moduleResolution Node16 and NodeNext (#12440)

Generate wide matches and params types for child routes (#12397)

At runtime, matches includes child route matches and params include child route path parameters. But previously, we only generated types for parent routes and the current route in matches and params. To align our generated types more closely to the runtime behavior, we now generate more permissive, wider types when accessing child route information.

Changes by Package

react-router

@react-router/architect

@react-router/cloudflare

@react-router/dev

@react-router/express

@react-router/fs-routes

@react-router/node

@react-router/remix-config-routes-adapter

@react-router/serve

Full Changelog: v7.0.1...v7.0.2

Does any of this look wrong? Please let us know.

↗️ react-router (indirect, 7.0.1 → 7.1.0) · Repo · Changelog

Release Notes

7.1.0 (from changelog)

Date: 2024-12-20

Minor Changes

Add support for Vite v6 (#12469)

Patch Changes

react-router - Throw unwrapped Single Fetch redirect to align with pre-Single Fetch behavior (#12506)

react-router - Ignore redirects when inferring loader data types (#12527)

react-router - Remove <Link prefetch> warning which suffers from false positives in a lazy route discovery world (#12485)

create-react-router - Fix missing fs-extra dependency (#12556)

@react-router/dev/@react-router/serve - Properly initialize NODE_ENV if not already set for compatibility with React 19 (#12578)

@react-router/dev - Remove the leftover/unused abortDelay prop from ServerRouter and update the default entry.server.tsx to use the new streamTimeout value for Single Fetch (#12478)

The abortDelay functionality was removed in v7 as it was coupled to the defer implementation from Remix v2, but this removal of this prop was missed

If you were still using this prop in your entry.server file, it's likely your app is not aborting streams as you would expect and you will need to adopt the new streamTimeout value introduced with Single Fetch

@react-router/fs-routes - Throw error in flatRoutes if routes directory is missing (#12407)

Changes by Package

create-react-router

react-router

@react-router/architect

@react-router/cloudflare

@react-router/dev

@react-router/express

@react-router/fs-routes

@react-router/node

@react-router/remix-config-routes-adapter

@react-router/serve

Full Changelog: v7.0.2...v7.1.0

7.0.2 (from changelog)

Date: 2024-12-02

Patch Changes

Temporarily only use one build in export map so packages can have a peer dependency on react router (#12437)

Support moduleResolution Node16 and NodeNext (#12440)

Generate wide matches and params types for child routes (#12397)

At runtime, matches includes child route matches and params include child route path parameters. But previously, we only generated types for parent routes and the current route in matches and params. To align our generated types more closely to the runtime behavior, we now generate more permissive, wider types when accessing child route information.

Changes by Package

react-router

@react-router/architect

@react-router/cloudflare

@react-router/dev

@react-router/express

@react-router/fs-routes

@react-router/node

@react-router/remix-config-routes-adapter

@react-router/serve

Full Changelog: v7.0.1...v7.0.2

Does any of this look wrong? Please let us know.

🗑️ string-width-cjs (removed)

🗑️ strip-ansi-cjs (removed)

🗑️ wrap-ansi-cjs (removed)

Depfu will automatically keep this PR conflict-free, as long as you don't add any commits to this branch yourself. You can also trigger a rebase manually by commenting with @depfu rebase.

All Depfu comment commands

@depfu rebase
Rebases against your default branch and redoes this update

@depfu recreate
Recreates this PR, overwriting any edits that you've made to it

@depfu merge
Merges this PR once your tests are passing and conflicts are resolved

@depfu cancel merge
Cancels automatic merging of this PR

@depfu close
Closes this PR and deletes the branch

@depfu reopen
Restores the branch and reopens this PR (if it's closed)

@depfu pause
Ignores all future updates for this dependency and closes this PR

@depfu pause [minor|major]
Ignores all future minor/major updates for this dependency and closes this PR

@depfu resume
Future versions of this dependency will create PRs again (leaves this PR as is)

korbit-ai · 2024-12-21T23:13:59Z

By default, I don't review pull requests opened by bots. If you would like me to review this pull request anyway, you can request a review via the /korbit-review command in a comment.

semanticdiff-com · 2024-12-21T23:13:59Z

Review changes with

Changed Files

File	Status
POCUploadStream/ClientApp/package-lock.json	68% smaller
POCUploadStream/ClientApp/package.json	0% smaller

pr-code-reviewer · 2024-12-21T23:14:00Z

👋 Hi there!

Ensure proper indentation and formatting consistency.
Verify if the added dependencies like "color-convert" and "color-name" are necessary for the project.
Check for any missing or incomplete entries in the code snippet.

_{^{Automatically generated with the help of gpt-3.5-turbo.

Feedback? Please don't hesitate to drop me an email at [email protected].}}

coderabbitai · 2024-12-21T23:14:03Z

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

Review comments: Directly reply to a review comment made by CodeRabbit. Example:
- I pushed a fix in commit <commit_id>, please review it.
- Generate unit testing code for this file.
- Open a follow-up GitHub issue for this discussion.
Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query. Examples:
- @coderabbitai generate unit testing code for this file.
- @coderabbitai modularize this function.
PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
- @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
- @coderabbitai read src/utils.ts and generate unit testing code.
- @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.
- @coderabbitai help me debug CodeRabbit configuration file.

Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments.

CodeRabbit Commands (Invoked using PR comments)

@coderabbitai pause to pause the reviews on a PR.
@coderabbitai resume to resume the paused reviews.
@coderabbitai review to trigger an incremental review. This is useful when automatic reviews are disabled for the repository.
@coderabbitai full review to do a full review from scratch and review all the files again.
@coderabbitai summary to regenerate the summary of the PR.
@coderabbitai generate docstrings to generate docstrings for this PR. (Beta)
@coderabbitai resolve resolve all the CodeRabbit review comments.
@coderabbitai configuration to show the current CodeRabbit configuration for the repository.
@coderabbitai help to get help.

Other keywords and placeholders

Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
Add @coderabbitai anywhere in the PR title to generate the title automatically.

CodeRabbit Configuration File (`.coderabbit.yaml`)

You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
Please see the configuration documentation for more information.
If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Documentation and Community

Visit our Documentation for detailed information on how to use CodeRabbit.
Join our Discord Community to get help, request features, and share feedback.
Follow us on X/Twitter for updates and announcements.

codara-ai-code-review · 2024-12-21T23:14:03Z

Potential issues, bugs, and flaws that can introduce unwanted behavior:

/POCUploadStream/ClientApp/package-lock.json: The addition of multiple new dependencies (like color-convert, color-name, string-width-cjs, etc.) without verifying potential conflicts with existing packages could lead to versioning issues in the future. This is particularly important since some versions are significantly lower or higher and can have breaking changes.
/POCUploadStream/ClientApp/package-lock.json: There are multiple duplicate entries for ansi-regex, color-convert, and others between newly added packages and their dependencies. This may cause conflicts when managing package versions or could lead to larger bundle sizes than necessary, potentially impacting performance.
/POCUploadStream/ClientApp/package-lock.json: The modification of react-router and react-router-dom versions without a clear statement of why this change is made might lead to runtime errors, especially if breaking changes between these versions are not accounted for in the rest of the codebase.

Code suggestions and improvements for better exception handling, logic, standardization, and consistency:

/POCUploadStream/ClientApp/package-lock.json:
- Remove duplicates: Consider reviewing the dependencies section for duplicates (like ansi-regex, color-convert, strip-ansi, etc.), as these can create confusion and maintenance issues. Clean up any unnecessary or redundant dependencies.
/POCUploadStream/ClientApp/package-lock.json:
- Version Locking: Ensure to follow a consistent versioning strategy (like semantic versioning) and possibly use "^" or "~" consistently to dictate if you want to allow new minor versions to be installed. Review the necessity of exact versioning in the context of the whole project.
/POCUploadStream/ClientApp/package.json:
- It is advisable to test the application thoroughly after updating react-router-dom and react-router to version 7.1.0 to ensure that there are no breaking changes affecting navigation or route handling in the app.
General Suggestion:
- Document all dependency changes in your version control (e.g., in the commit messages) to provide better context for future developers. Highlight any specific reasons for changes that might impact the functionality of the project.

gooroo-dev · 2024-12-21T23:14:05Z

Please double check the following review of the pull request:

Issues counts

🐞Mistake	🤪Typo	🚨Security	🚀Performance	💪Best Practices	📖Readability	❓Others
0	0	0	0	0	0	0

Changes in the diff

🛠️ Updated react-router-dom from version 7.0.1 to 7.1.0 in package.json.
🛠️ Updated react-router from version 7.0.1 to 7.1.0 in package-lock.json.
➕ Added new dependencies: color-convert, color-name, string-width-cjs, strip-ansi-cjs, wrap-ansi-cjs.
➖ Removed redundant dependencies: string-width-cjs, strip-ansi-cjs, wrap-ansi-cjs.

Identified Issues

ID	Type	Details	Severity	Confidence

No issues were identified in the incoming changes.

Missing Tests

Since the update involves a minor version change for react-router-dom and react-router, it is essential to ensure that the routing functionality in the application is still working as expected. Here are some suggested tests:

Navigation Test: Verify that navigation between different routes is functioning correctly.
Route Parameter Test: Ensure that routes with parameters are still correctly parsing and displaying data.
404 Handling Test: Check that non-existent routes are handled properly and redirect to a 404 page or similar.
Link Component Test: Ensure that Link components are generating the correct URLs and are navigable.
Redirect Test: Verify that redirects are functioning as expected.

These tests will help ensure that the update to react-router-dom and react-router does not introduce any regressions in the routing functionality of the application.

Summon me to re-review when updated! Yours, Gooroo.dev
React or reply to keep the discussion going!

instapr · 2024-12-21T23:14:06Z

Feedback:

Updated react-router-dom to version 7.1.0 successfully.
Changes in POCUploadStream/ClientApp/package-lock.json:
- Added "color-convert": "2.0.1" and "color-name": "1.1.4".
- Removed "string-width-cjs", "strip-ansi-cjs", and "wrap-ansi-cjs".
Ensure all dependencies are compatible with the updated version of react-router-dom.
Double-check the functionality and compatibility with existing codebase before merging.

Let me know if you need any further assistance!

guibranco

Automatically approved by gstraccini[bot]

guibranco · 2024-12-21T23:14:20Z

@depfu merge

socket-security · 2024-12-21T23:14:33Z

Report too large to display inline

View full report↗︎

socket-security · 2024-12-21T23:14:35Z

Report is too large to display inline.
View full report↗︎

Next steps

Mark a package as acceptable risk

To ignore an alert, reply with a comment starting with @SocketSecurity ignore followed by a space separated list of ecosystem/package-name@version specifiers. e.g. @SocketSecurity ignore npm/[email protected] or ignore all packages with @SocketSecurity ignore-all

@SocketSecurity ignore npm/[email protected]
@SocketSecurity ignore npm/[email protected]
@SocketSecurity ignore npm/[email protected]
@SocketSecurity ignore npm/[email protected]
@SocketSecurity ignore npm/[email protected]
@SocketSecurity ignore npm/[email protected]
@SocketSecurity ignore npm/[email protected]
@SocketSecurity ignore npm/[email protected]
@SocketSecurity ignore npm/[email protected]
@SocketSecurity ignore npm/[email protected]
@SocketSecurity ignore npm/[email protected]
@SocketSecurity ignore npm/[email protected]
@SocketSecurity ignore npm/[email protected]
@SocketSecurity ignore npm/[email protected]
@SocketSecurity ignore npm/[email protected]
@SocketSecurity ignore npm/[email protected]
@SocketSecurity ignore npm/[email protected]
@SocketSecurity ignore npm/[email protected]
@SocketSecurity ignore npm/[email protected]
@SocketSecurity ignore npm/[email protected]
@SocketSecurity ignore npm/[email protected]
@SocketSecurity ignore npm/[email protected]
@SocketSecurity ignore npm/[email protected]
@SocketSecurity ignore npm/[email protected]
@SocketSecurity ignore npm/[email protected]
@SocketSecurity ignore npm/[email protected]
@SocketSecurity ignore npm/[email protected]
@SocketSecurity ignore npm/[email protected]
@SocketSecurity ignore npm/[email protected]
@SocketSecurity ignore npm/[email protected]
@SocketSecurity ignore npm/[email protected]
@SocketSecurity ignore npm/[email protected]
@SocketSecurity ignore npm/[email protected]
@SocketSecurity ignore npm/[email protected]
@SocketSecurity ignore npm/[email protected]
@SocketSecurity ignore npm/[email protected]
@SocketSecurity ignore npm/[email protected]
@SocketSecurity ignore npm/[email protected]
@SocketSecurity ignore npm/[email protected]
@SocketSecurity ignore npm/[email protected]
@SocketSecurity ignore npm/[email protected]
@SocketSecurity ignore npm/[email protected]
@SocketSecurity ignore npm/[email protected]
@SocketSecurity ignore npm/[email protected]
@SocketSecurity ignore npm/[email protected]
@SocketSecurity ignore npm/[email protected]
@SocketSecurity ignore npm/[email protected]
@SocketSecurity ignore npm/[email protected]
@SocketSecurity ignore npm/[email protected]
@SocketSecurity ignore npm/[email protected]

github-actions · 2024-12-21T23:14:35Z

Infisical secrets check: ✅ No secrets leaked!

💻 Scan logs

11:14PM INF scanning for exposed secrets...
11:14PM INF 261 commits scanned.
11:14PM INF scan completed in 998ms
11:14PM INF no leaks found

sonarqubecloud · 2024-12-21T23:15:05Z

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

Update react-router-dom to version 7.1.0

208a1ed

depfu bot added the depfu label Dec 21, 2024

github-actions bot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Dec 21, 2024

guibranco enabled auto-merge (squash) December 21, 2024 23:14

gstraccini bot added the ☑️ auto-merge Automatic merging of pull requests (gstraccini-bot) label Dec 21, 2024

gstraccini bot assigned guibranco Dec 21, 2024

gstraccini bot approved these changes Dec 21, 2024

View reviewed changes

guibranco approved these changes Dec 21, 2024

View reviewed changes

gstraccini bot added the 🤖 bot Automated processes or integrations label Dec 21, 2024

guibranco merged commit 0db90f6 into main Dec 21, 2024
18 of 20 checks passed

guibranco deleted the depfu/update/POCUploadStreamClientApp/npm/react-router-dom-7.1.0 branch December 21, 2024 23:18

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Update react-router-dom 7.0.1 → 7.1.0 (minor) #265

Update react-router-dom 7.0.1 → 7.1.0 (minor) #265

depfu bot commented Dec 21, 2024

korbit-ai bot commented Dec 21, 2024

semanticdiff-com bot commented Dec 21, 2024 •

edited

Loading

pr-code-reviewer bot commented Dec 21, 2024

coderabbitai bot commented Dec 21, 2024

Review skipped

Chat

CodeRabbit Commands (Invoked using PR comments)

Other keywords and placeholders

CodeRabbit Configuration File (`.coderabbit.yaml`)

Documentation and Community

codara-ai-code-review bot commented Dec 21, 2024

gooroo-dev bot commented Dec 21, 2024

instapr bot commented Dec 21, 2024

guibranco left a comment

guibranco commented Dec 21, 2024

socket-security bot commented Dec 21, 2024

socket-security bot commented Dec 21, 2024

github-actions bot commented Dec 21, 2024

sonarqubecloud bot commented Dec 21, 2024

Update react-router-dom 7.0.1 → 7.1.0 (minor) #265

Update react-router-dom 7.0.1 → 7.1.0 (minor) #265

Conversation

depfu bot commented Dec 21, 2024

What changed?

✳️ react-router-dom (7.0.1 → 7.1.0) · Repo · Changelog

7.1.0 (from changelog)

Minor Changes

Patch Changes

Changes by Package

7.0.2 (from changelog)

Patch Changes

Changes by Package

↗️ react-router (indirect, 7.0.1 → 7.1.0) · Repo · Changelog

7.1.0 (from changelog)

Minor Changes

Patch Changes

Changes by Package

7.0.2 (from changelog)

Patch Changes

Changes by Package

🗑️ string-width-cjs (removed)

🗑️ strip-ansi-cjs (removed)

🗑️ wrap-ansi-cjs (removed)

korbit-ai bot commented Dec 21, 2024

semanticdiff-com bot commented Dec 21, 2024 • edited Loading

pr-code-reviewer bot commented Dec 21, 2024

👋 Hi there!

coderabbitai bot commented Dec 21, 2024

Review skipped

Chat

CodeRabbit Commands (Invoked using PR comments)

Other keywords and placeholders

CodeRabbit Configuration File (.coderabbit.yaml)

Documentation and Community

codara-ai-code-review bot commented Dec 21, 2024

Potential issues, bugs, and flaws that can introduce unwanted behavior:

Code suggestions and improvements for better exception handling, logic, standardization, and consistency:

gooroo-dev bot commented Dec 21, 2024

Please double check the following review of the pull request:

Issues counts

Changes in the diff

Identified Issues

Missing Tests

instapr bot commented Dec 21, 2024

Feedback:

guibranco left a comment

Choose a reason for hiding this comment

guibranco commented Dec 21, 2024

socket-security bot commented Dec 21, 2024

socket-security bot commented Dec 21, 2024

Next steps

github-actions bot commented Dec 21, 2024

sonarqubecloud bot commented Dec 21, 2024

Quality Gate passed

semanticdiff-com bot commented Dec 21, 2024 •

edited

Loading

CodeRabbit Configuration File (`.coderabbit.yaml`)