AI: address review on PR #456

frontend/src/components/PostEmbed.tsx

@@ -196,39 +196,6 @@ const PostEmbed: FC<Props> = (
       return <TwitterEmbed userId={userId} statusId={statusId}/>
     }
 
-  case 'bilibili.com':
-    {
-      const videoId = url.pathname.match (
-	  /^\/video\/(BV[a-zA-Z0-9]{10}|av[1-9]\d*)\/?$/)?.[1]
-      if (!(videoId))
-	break
-
-      if (embedAutoLoadMode === 'off')
-	return externalLink
-      if (shouldLoadEmbed === false)
-	return manualLoadControl
-
-      const playerUrl = new URL ('https://player.bilibili.com/player.html')
-      const parameter = videoId.startsWith ('BV') ? 'bvid' : 'aid'
-      const identifier = parameter === 'bvid' ? videoId : videoId.slice (2)
-      playerUrl.searchParams.set (parameter, identifier)
-      playerUrl.searchParams.set ('autoplay', '1')
-
-      const page = url.searchParams.get ('p')
-      if (page != null && /^[1-9]\d*$/.test (page))
-	playerUrl.searchParams.set ('p', page)
-
-      return (
-	  <iframe
-	      src={playerUrl.toString ()}
-	      title={post.title || post.url}
-	      width={640}
-	      height={360}
-	      className="aspect-video h-auto max-w-full"
-	      allow="autoplay; fullscreen"
-	      allowFullScreen/>)
-    }
-
   case 'youtube.com':
     {
       const videoId = url.searchParams.get ('v')

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 ()

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>
 

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' }))
+    const updatedTag = buildTag ({ id: 7, name: 'new', aliases: ['alias'] })
-    api.apiPut.mockResolvedValueOnce (buildTag ({ id: 7, name: 'new', aliases: ['alias'] }))
+    tagsApi.fetchTag
+	.mockResolvedValue (updatedTag)
+	.mockResolvedValueOnce (buildTag ({ id: 7, name: 'old' }))
+    api.apiPut.mockResolvedValueOnce (updatedTag)
 
     renderPage ()