From e13a52db9a6f6619fa125a215602ac7a9ea72984 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=E3=83=86=E3=83=84=E3=83=8E=E3=83=8B=E3=82=B8=E3=82=AB?=
 <969+codex-bot@noreply.localhost>
Date: Sat, 14 Nov 2026 17:09:38 +0900
Subject: [PATCH] =?UTF-8?q?AI:=20#454=20=E5=BA=83=E5=A0=B4=E3=81=AE?=
 =?UTF-8?q?=E3=82=BF=E3=82=B0=E6=A4=9C=E7=B4=A2=EF=BC=8C=E3=82=B9=E3=83=9E?=
 =?UTF-8?q?=E3=83=9B=E7=89=88=E3=81=A0=E3=81=A8=E3=82=B5=E3=82=BC=E3=82=B9?=
 =?UTF-8?q?=E3=83=88=E3=81=8C=E9=80=94=E4=B8=AD=E3=81=A7=E8=A6=8B=E5=88=87?=
 =?UTF-8?q?=E3=82=8C=E3=82=8B=E5=95=8F=E9=A1=8C=20(#456)?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Refs #454

## Automated implementation (requires human review)
[SidebarComponent.tsx](frontend/src/components/layout/SidebarComponent.tsx) の1行を修正しました。`overflow-y-auto` を `md` 以上に限定し、スマホでサゼストがサイドバーの領域外で切られる原因を取り除きました。

差分を確認済みです。指定に従い、テスト・lint・ビルドは未実行です。実画面でのスマホ表示とPC表示の確認も未実施です。

## Automated review (not a guarantee of correctness)
```text
The change restricts sidebar scrolling to medium and larger viewports. No actionable regression was identified in the current diff.

```

## Verification
- Tests/builds/migrations were not authorized and were not invoked by the controller.
- Codex was instructed not to run them; its command transcript should be inspected.
- Do not merge before manual acceptance and required checks.

---------

Co-authored-by: Codex Night Worker <codex-night@localhost>
Reviewed-on: https://git.miteruzo.com/miteruzo/btrc-hub/pulls/456
Co-authored-by: テツノニジカ <969+codex-bot@noreply.localhost>
---
 .../src/components/layout/SidebarComponent.test.tsx   | 10 ++++++++++
 frontend/src/components/layout/SidebarComponent.tsx   |  2 +-
 frontend/src/pages/tags/TagDetailPage.test.tsx        | 11 +++++++++--
 3 files changed, 20 insertions(+), 3 deletions(-)

diff --git a/frontend/src/components/layout/SidebarComponent.test.tsx b/frontend/src/components/layout/SidebarComponent.test.tsx
index 8c7879a..3316c91 100644
--- a/frontend/src/components/layout/SidebarComponent.test.tsx
+++ b/frontend/src/components/layout/SidebarComponent.test.tsx
@@ -70,6 +70,16 @@ describe ('SidebarComponent', () => {
     vi.spyOn (HTMLElement.prototype, 'hasPointerCapture').mockReturnValue (true)
   })
 
+  it ('limits sidebar scrolling to desktop to avoid clipping mobile suggestions', () => {
+    renderSidebar ()
+
+    // jsdom has no layout; guard the responsive classes responsible for clipping.
+    const sidebar = screen.getByText ('Sidebar body')
+
+    expect (sidebar).not.toHaveClass ('overflow-y-auto')
+    expect (sidebar).toHaveClass ('md:overflow-y-auto')
+  })
+
   it ('uses maxWidth without applying the container main-width reserve twice', async () => {
     localStorage.setItem ('sidebar:test-sidebar:left:width', '700')
     const onWidthChange = vi.fn ()
diff --git a/frontend/src/components/layout/SidebarComponent.tsx b/frontend/src/components/layout/SidebarComponent.tsx
index 6e2b29c..5157c78 100644
--- a/frontend/src/components/layout/SidebarComponent.tsx
+++ b/frontend/src/components/layout/SidebarComponent.tsx
@@ -339,7 +339,7 @@ const SidebarComponent: FC<Props> = ({
 	  </style>
 	</Helmet>
 
-	<div className="h-full overflow-y-auto p-4 sidebar">
+	<div className="h-full md:overflow-y-auto p-4 sidebar">
 	  {children}
 	</div>
 
diff --git a/frontend/src/pages/tags/TagDetailPage.test.tsx b/frontend/src/pages/tags/TagDetailPage.test.tsx
index 567c554..e86cbbb 100644
--- a/frontend/src/pages/tags/TagDetailPage.test.tsx
+++ b/frontend/src/pages/tags/TagDetailPage.test.tsx
@@ -35,6 +35,10 @@ const renderPage = () =>
 describe ('TagDetailPage', () => {
   beforeEach (() => {
     vi.clearAllMocks ()
+    tagsApi.fetchTag.mockReset ()
+    api.apiPut.mockReset ()
+    api.isApiError.mockReset ()
+    toastApi.toast.mockReset ()
     api.isApiError.mockReturnValue (false)
   })
 
@@ -51,8 +55,11 @@ describe ('TagDetailPage', () => {
   })
 
   it ('submits edited tag fields', async () => {
-    tagsApi.fetchTag.mockResolvedValueOnce (buildTag ({ id: 7, name: 'old' }))
-    api.apiPut.mockResolvedValueOnce (buildTag ({ id: 7, name: 'new', aliases: ['alias'] }))
+    const updatedTag = buildTag ({ id: 7, name: 'new', aliases: ['alias'] })
+    tagsApi.fetchTag
+	.mockResolvedValue (updatedTag)
+	.mockResolvedValueOnce (buildTag ({ id: 7, name: 'old' }))
+    api.apiPut.mockResolvedValueOnce (updatedTag)
 
     renderPage ()