fix(sidebar): add dynamic export to group-order route to prevent static caching#660
Merged
fix(sidebar): add dynamic export to group-order route to prevent static caching#660
Conversation
…ic caching Next.js statically pre-rendered the GET handler at build time, storing the response in .next/server/app/api/sidebar/group-order.body. This caused PUT requests to fall through to the Pages Router 405 handler instead of reaching the App Router handler. Adding `export const dynamic = 'force-dynamic'` ensures the route is always server-rendered on demand, so both GET and PUT are handled by the route module. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Previously the tooltip only showed branch name, repo name, status, and worktree path. Description was only visible inline with line-clamp-2. Changes: - Add description field to BranchTooltip (full text, no truncation) - Remove global whitespace-nowrap; apply it per-line for header rows - Add max-w-sm to tooltip to prevent overflow on very long descriptions - Use whitespace-pre-wrap + break-words so line breaks in description are preserved Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
/api/sidebar/group-orderにexport const dynamic = 'force-dynamic'を追加GETハンドラを静的プリレンダリングし.next/server/app/api/sidebar/group-order.bodyにキャッシュしていた。これによりPUTリクエストが App Router ハンドラに届かず Pages Router の 405 エラーになっていたforce-dynamicを追加することでルートを常にサーバーサイドで実行し、GET/PUT両方が正常に動作するようになったTest plan
GET /api/sidebar/group-orderが保存済み順序を返す)npm run test:unit)🤖 Generated with Claude Code