Skip to content

Fix cache handling for optional rscData#1194

Open
conico974 wants to merge 3 commits into
mainfrom
conico/rsc-undefined
Open

Fix cache handling for optional rscData#1194
conico974 wants to merge 3 commits into
mainfrom
conico/rsc-undefined

Conversation

@conico974

Copy link
Copy Markdown
Contributor

Improve fallback behavior for optional rscData in the cache, ensuring it handles undefined values gracefully. Additionally, apply linting changes and include a changeset for better tracking.

@changeset-bot

changeset-bot Bot commented Jul 5, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 433c33b

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

This PR includes changesets to release 2 packages
Name Type
@opennextjs/aws Patch
app-pages-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

@pkg-pr-new

pkg-pr-new Bot commented Jul 5, 2026

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/@opennextjs/aws@1194

commit: 433c33b

@github-actions

github-actions Bot commented Jul 5, 2026

Copy link
Copy Markdown
Contributor

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 36.15% 4029 / 11143
🔵 Statements 36.15% 4029 / 11143
🔵 Functions 58% 192 / 331
🔵 Branches 79.42% 1085 / 1366
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
packages/open-next/src/adapters/cache.ts 88.42% 87.17% 100% 88.42% 28-29, 93-101, 196, 215-216, 432, 452-473, 507-508, 542-551, 561-569
packages/open-next/src/core/routing/cacheInterceptor.ts 89.33% 79.79% 100% 89.33% 64-70, 91-92, 122-123, 142-144, 175-178, 234-235, 256-258, 263-264, 279-281, 289-290, 379-380
packages/open-next/src/types/cache.ts 0% 0% 0% 0%
packages/open-next/src/types/overrides.ts 0% 0% 0% 0%
Generated in workflow #1844 for commit 433c33b by the Vitest Coverage Report Action

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