chore: upgrade dependencies

[mengxi-ream]

Type of Changes

• 🔧 Build or dependencies update (build)

Description

Upgrade project dependencies:

• postcss-rem-to-responsive-pixel 6.x → 7.0.0 (performance improvements, strict half-up rounding)
• jsdom 26.x → 27.x (updated test assertions for behavior changes)
• Various other dev dependency updates (eslint, vitest, tailwindcss, etc.)

How Has This Been Tested?

• Added unit tests
• Verified through manual testing

All tests pass and build succeeds.

Checklist

• I have tested these changes locally
• I have updated the documentation accordingly if necessary
• My code follows the code style of this project
• My changes do not break existing functionality
• If my code was generated by AI, I have proofread and improved it as necessary.

---

Summary by cubic

Upgrade dependencies to improve performance and align tests with jsdom v27 behavior. Key updates include postcss-rem-to-responsive-pixel v7 and jsdom v27; test expectations were adjusted for floated elements in jsdom.

• Dependencies
  • postcss-rem-to-responsive-pixel: 6.x → 7.0.0 (faster, strict half-up rounding)
  • jsdom: 26.x → 27.x (floated spans remain inline in jsdom; tests updated)
  • Bumps for React Query/Form, VChart, Dexie, Hotkeys, Jotai, React Router, TailwindCSS, ESLint/Vitest, and more
  • pnpm: 10.16.1 → 10.27.0
  • Node runtime: 24.9.1 → 24.12.0
  • Added changeset for a patch release

^{Written for commit 00c1793. Summary will update on new commits.}

[changeset-bot]

🦋 Changeset detected

Latest commit: 00c1793

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

This PR includes changesets to release 1 package

Name	Type
@read-frog/extension	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

[dosubot]

Related Documentation

Checked 14 published document(s) in 1 knowledge base(s). No updates required.

^{How did I do? Any feedback?}  

[cubic-dev-ai]

No issues found across 4 files

[claude]

PR Review: Dependency Upgrade

【Taste Score】

🟢 Good taste

This is a straightforward dependency upgrade PR with appropriate test adjustments.

---

Summary

This PR upgrades multiple dependencies including:

• postcss-rem-to-responsive-pixel: 6.x → 7.0.0 (major version bump)
• jsdom: 26.x → 27.x (major version bump)
• pnpm: 10.16.1 → 10.27.0
• type-fest: Moved from dependencies → devDependencies (good catch!)
• Various other dev dependency updates (ESLint, Vitest, TailwindCSS, etc.)

---

Code Quality & Best Practices

✅ Changeset included: Proper changeset for tracking this patch release
✅ type-fest moved to devDependencies: Correct categorization - this is a type-only package
✅ Test adjustments for jsdom v27: The test file includes comments explaining jsdom's behavior regarding floated elements not being blockified
✅ Lock file regenerated properly: pnpm-lock.yaml updated consistently

---

Potential Issues

1. Minor: Changeset file missing newline at EOF

The .changeset/ilbyvzjjasdg.md file is missing a trailing newline. While not critical, it's a common convention.

2. Worth noting: postcss-rem-to-responsive-pixel v7 breaking changes

The PR description mentions "strict half-up rounding" in v7. If the project has pixel-perfect visual tests or screenshots, this could cause minor differences. The PR states all tests pass, so this seems fine.

3. Worth noting: jsdom v27 CSS behavior

The test file comments document that jsdom doesn't implement the CSS "float blockifies display" rule. The getComputedStyle mock in the test file (lines 55-71) handles this correctly by checking for inline style float properties. This is good defensive testing.

---

Security Considerations

✅ All dependencies are from trusted sources (npm registry)
✅ No new dependencies with known vulnerabilities detected in the diff
✅ Major version bumps reviewed for breaking changes

---

Performance Considerations

✅ postcss-rem-to-responsive-pixel v7 claims performance improvements
✅ No runtime dependency changes that would affect bundle size negatively

---

Test Coverage

✅ The test file translate.integration.test.tsx has been updated with comments explaining jsdom v27 behavior differences
✅ Tests explicitly document that floated <span> elements remain display:inline in jsdom (unlike real browsers)
✅ The mock for getComputedStyle handles the float property correctly

---

Recommendations

1. Optional: Add trailing newline to changeset file
2. Optional: Consider adding a brief comment in the test file linking to the jsdom changelog/issue explaining the float behavior if this becomes a maintenance concern

---

Verdict

Approved ✅

This is a well-executed dependency upgrade. The test adjustments are appropriately documented, and the categorization fix for type-fest is a nice cleanup. No blocking issues found.

---

🤖 Automated review by Claude Code

[claude]

Code Review: PR #851 - Dependency Upgrade

【Taste Score】

🟢 Good taste - Clean, well-structured dependency update with appropriate test adjustments.

---

Summary

This PR upgrades project dependencies including a major version bump for postcss-rem-to-responsive-pixel (v6 → v7), jsdom (v26 → v27), and various other dev/runtime dependencies. The changeset and CI improvements are properly included.

---

✅ Strengths

1. CI Improvement: Removing hardcoded pnpm version from .github/workflows/pr-test.yml is a good practice - letting pnpm/action-setup@v4 read from packageManager field ensures consistency.

2. Proper Changeset: Including .changeset/ilbyvzjjasdg.md as a patch release is appropriate for dependency updates.

3. Test Adjustments for jsdom v27: The updated test comments (lines 706-754) correctly document jsdom's behavior difference regarding CSS float blockification. The test expectations properly align with jsdom's actual behavior.

4. type-fest moved to devDependencies: Good catch - type-fest is appropriately a dev dependency (v4 → v5 with location change).

5. Major version bumps handled: postcss-rem-to-responsive-pixel v7 and hotkeys-js v4 are major versions; the fact that tests pass suggests no breaking changes affect this codebase.

---

⚠️ Observations (Not Issues)

1. jsdom Float Behavior: The test file documents that jsdom doesn't implement CSS "float blockifies display" rule. The mock at lines 55-71 properly handles this by returning 'none' as default float. The updated test descriptions (lines 709-754) are accurate and helpful documentation.

2. Missing newline in changeset: .changeset/ilbyvzjjasdg.md is missing a trailing newline (minor style issue, non-blocking).

---

🔍 Potential Considerations

1. postcss-rem-to-responsive-pixel v7 API: The lockfile shows this now requires postcss as a peer dependency ([email protected]([email protected])). Verify that the postcss.config.cjs doesn't need updates for the new API. Based on the passing tests, this appears to work correctly.

2. hotkeys-js v4: Major version bump from v3 to v4. If any breaking changes exist in the API, they should be verified. Tests passing is a good sign.

---

📋 Checklist Verification

• Tests pass (implied by PR description)
• Type checking passes
• Build succeeds
• Changeset included
• No security vulnerabilities introduced (standard dependency updates)
• Test expectations updated for jsdom v27 behavior changes

---

【Core Judgment】

✅ Worth merging - This is a well-executed dependency update with proper documentation of test behavior changes. The jsdom behavior difference is correctly documented, and the test mock appropriately handles the float property detection.

---

Recommendation

Approve - The changes are clean, tests are properly updated, and the CI improvement is a nice addition. No blocking issues identified.

[claude]

PR Review: chore: upgrade dependencies

Overall Assessment

🟢 Good PR - This is a well-structured dependency upgrade with appropriate test adjustments.

---

Code Quality & Best Practices

Positives:

• ✅ Clear PR description documenting the major version bumps (postcss-rem-to-responsive-pixel 6.x → 7.0.0, jsdom 26.x → 27.x)
• ✅ Test assertions properly updated to reflect jsdom v27 behavior changes
• ✅ Helpful comments added explaining the jsdom limitation ("float blockifies display" rule not implemented)
• ✅ Proper changeset file included for release tracking
• ✅ CI workflow simplified by removing hardcoded pnpm version (now reads from packageManager field)

Observations:

• type-fest moved from dependencies to devDependencies - this is correct since it is only used for type definitions
• Node runtime version bump 24.9.1 → 24.12.0 is a minor update within the same major version

---

Potential Issues & Concerns

1. Test Behavior Changes (Medium Priority)

The jsdom v27 upgrade changes test behavior for floated elements. The tests now expect INLINE_ATTRIBUTE instead of BLOCK_ATTRIBUTE for floated spans.

Question: Does the production code rely on detecting floated elements as block-level? If so, the tests may now be passing but testing incorrect behavior. The comments acknowledge jsdom limitation, but this could mask real-world bugs.

Recommendation: Consider adding a TODO or issue to track browser-based E2E tests for float detection if this is critical functionality.

2. Major Version Bumps (Low Risk)

Several major/significant version bumps:

• hotkeys-js: 3.x → 4.0.0
• jsdom: 26.x → 27.x
• postcss-rem-to-responsive-pixel: 6.x → 7.0.0
• type-fest: 4.x → 5.x (moved to devDeps)

These have been tested and the build succeeds, so the risk is mitigated.

---

Performance Considerations

✅ postcss-rem-to-responsive-pixel v7 includes performance improvements per the PR description - this is a net positive.

---

Security Concerns

✅ No security concerns identified. All dependencies appear to be standard, well-maintained packages.

---

Test Coverage

✅ Existing tests updated appropriately for jsdom behavior changes.

Suggestion: The test name changes are good and descriptive:

• "should treat float right as block node" → "float right span remains inline in jsdom (no blockification)"

This makes it clear what the test is actually verifying vs. what the expected real-browser behavior is.

---

Minor Observations

1. Changeset file - Missing newline at end of file (.changeset/ilbyvzjjasdg.md). Not blocking but worth fixing for consistency.

2. Lockfile changes - The pnpm-lock.yaml shows clean version resolution. No dependency conflicts detected.

---

Verdict

✅ Approve - This is a clean dependency upgrade PR. The test adjustments are well-documented and the PR follows the project conventions.

The only follow-up consideration is whether the float detection tests need additional browser-based coverage to ensure real-world behavior matches expectations.