Skip to content

docs: record deferred improvements with resume checklists#50

Merged
bolin8017 merged 1 commit into
mainfrom
feat/next16-cache-components
Apr 18, 2026
Merged

docs: record deferred improvements with resume checklists#50
bolin8017 merged 1 commit into
mainfrom
feat/next16-cache-components

Conversation

@bolin8017
Copy link
Copy Markdown
Owner

Summary

Captures three review items that were investigated and intentionally deferred:

  • Next.js 16 Cache Components: a full migration was attempted on branch `feat/next16-cache-components` (setting `cacheComponents: true` + 'use cache' helpers across public pages + tag invalidation in `deleteProblem`). The attempt was reverted because `cacheComponents: true` requires every uncached data access to sit inside a `` boundary during static generation — that's a 20+ file UI refactor across admin and mixed public/auth pages rather than a drop-in migration. Doc captures what was tried and a resume checklist.
  • Branded channel-identifier types: narrow compile-time win, wide refactor surface.
  • Node 22 → 24 + pnpm 9 → 10: needs real install + Playwright regression; checklist captured.

Test plan

  • Docs-only change; no code impact
  • Manually inspected `docs/deferred-improvements.md`

🤖 Generated with Claude Code

Full-project review (2026-04-17) left three items deferred. This file
captures what was tried, why it was stopped, and the minimum work to
pick them up later:

- Next.js 16 Cache Components: a full migration attempt is documented
  with its scope, the cacheComponents-enforced Suspense rule that
  blocks the quick path, and a concrete step-by-step for a future
  Suspense-boundary refactor. PRs with rate-limiter + other review
  items merged separately; this one stays for a bigger UI refactor.
- Branded channel-identifier types: narrow compile-time win, wide
  refactor surface — parked with rationale.
- Node 22->24 + pnpm 9->10: needs real install + regression run;
  checklist captured.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 18, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
caffecode Ready Ready Preview, Comment Apr 18, 2026 3:34am

@bolin8017 bolin8017 merged commit 3705b16 into main Apr 18, 2026
3 checks passed
@bolin8017 bolin8017 deleted the feat/next16-cache-components branch April 18, 2026 03:34
@github-actions
Copy link
Copy Markdown

📊 Coverage Report

Package Statements Branches Functions Lines
shared 92.22% ❌ (≥95%) 87.74% ❌ (≥90%) 96.87% ✅ (≥95%) 91.88% ❌ (≥95%)
web 95.47% ✅ (≥90%) 90.75% ✅ (≥85%) 96.42% ✅ (≥90%) 96.18% ✅ (≥90%)

🤖 Generated by CI · thresholds enforced via vitest

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