feat(torrents): add web seeds table

[s0up4200]

• Implemented GetTorrentWebSeeds handler in the API to retrieve HTTP sources for torrents.
• Added WebSeedsTable component to display web seeds in the UI with search functionality.
• Updated TorrentDetailsPanel to include a new tab for web seeds.
• Enhanced API client to support fetching web seeds from the server.

Summary by CodeRabbit

• New Features
  • Added a Web Seeds tab in torrent details showing HTTP sources with per-item URLs.
  • Live search/filter by URL, per-item copy-to-clipboard, and context actions.
  • Incognito mode masks URLs for privacy; shows loading, empty states, and item counts.
  • Web seeds are now retrievable via the API and reflected in UI tab lifecycle.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Walkthrough

Adds a new backend API endpoint and SyncManager method to return a torrent's HTTP web seeds, wires the route, extends the OpenAPI spec and client API, and integrates a new "Web Seeds" tab and WebSeedsTable component in the torrent details UI.

Changes

Cohort / File(s)	Change Summary
Backend — API handler & routing internal/api/handlers/torrents.go, internal/api/server.go	Added TorrentsHandler.GetTorrentWebSeeds and routed GET /instances/{instanceID}/torrents/{hash}/webseeds; validates inputs, handles disabled instances and errors, and returns JSON list of web seeds.
Backend — qBittorrent sync internal/qbittorrent/sync_manager.go	Added SyncManager.GetTorrentWebSeeds(ctx, instanceID, hash) ([]qbt.WebSeed, error) using the client pool and GetTorrentsWebSeedsCtx; refactored GetTorrentPeers to use a per-hash peerSync manager.
API spec internal/web/swagger/openapi.yaml	Added GET /api/instances/{instanceID}/torrents/{hash}/webseeds operation and WebSeed schema with url (string).
Frontend — types & API client web/src/types/index.ts, web/src/lib/api.ts	Added WebSeed type ({ url: string }) and ApiClient.getTorrentWebSeeds(instanceId, hash): Promise<WebSeed[]>.
Frontend — UI components WebSeedsTable web/src/components/torrents/details/WebSeedsTable.tsx, web/src/components/torrents/details/index.ts	New WebSeedsTable component (memoized) with search, incognito masking, copy-to-clipboard, loading/empty states; exported from details index.
Frontend — Torrent details integration web/src/components/torrents/TorrentDetailsPanel.tsx	Added "webseeds" to TAB_VALUES, fetches web seeds via useQuery when tab active, and renders WebSeedsTable (handles horizontal and stacked layouts, loading and no-data redirects).

Sequence Diagram(s)

sequenceDiagram
  participant Browser as Client (UI)
  participant API as API Server (handler)
  participant Sync as SyncManager
  participant QBT as qBittorrent Client (pool)

  Browser->>API: GET /instances/{id}/torrents/{hash}/webseeds
  API->>API: validate instanceID & hash
  API->>Sync: GetTorrentWebSeeds(ctx, instanceID, hash)
  Sync->>QBT: acquire client from pool
  Sync->>QBT: GetTorrentsWebSeedsCtx(ctx, hash)
  QBT-->>Sync: returns []WebSeed or error
  Sync-->>API: []WebSeed or error
  API-->>Browser: 200 JSON []WebSeed (or 4xx/5xx on error)

Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

• Inspect SyncManager.GetTorrentWebSeeds for proper context/cancellation and client-pool usage.
• Verify GetTorrentPeers refactor (peerSync manager) for behavioral parity and concurrency safety.
• Review handler input validation and respondIfInstanceDisabled logic.
• Check frontend useQuery gating, loading/empty states, incognito masking, and copy-to-clipboard behavior.

Possibly related PRs

• fix(search): download torrent files via backend for remote instances #686 — Modifies TorrentsHandler and server wiring; likely overlaps with handler routing and handler constructor changes.
• feat(web): new torrent details panel for desktop #760 — Changes TorrentDetailsPanel.tsx layout and rendering paths; may conflict with integration of the new tab.
• feat(web): Cross-seeded torrent management #552 — Also updates TorrentDetailsPanel.tsx TAB_VALUES; potential merge conflict on tab enum.

Suggested labels

backend, go

Poem

🐰 I hopped through codefields late at night,
Web seeds tucked in rows so tight.
Links masked shy when moon is high,
Copy, search — a bunny's delight.
New tab blooms under starlit byte.

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately summarizes the main change: adding a web seeds table component to display HTTP sources for torrents across the UI and API layers.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch feat/http-sources

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 0

🧹 Nitpick comments (1)

web/src/components/torrents/details/WebSeedsTable.tsx (1)

42-52: Optional: Consider URL-safe truncation for incognito masking.

Line 47 slices the pathname at character 20, which could truncate in the middle of a URL-encoded sequence (e.g., %2 in %20), potentially displaying broken encoding. While this is cosmetic and only affects the masked display, you might consider truncating at a safe boundary.

Note: The Biome linter warning about "wrapping comments" is a false positive—the // on line 47 is part of the URL protocol string, not a JSX comment.

📜 Review details

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 69ed1a3 and 601b940.

📒 Files selected for processing (8)

• internal/api/handlers/torrents.go (1 hunks)
• internal/api/server.go (1 hunks)
• internal/qbittorrent/sync_manager.go (1 hunks)
• web/src/components/torrents/TorrentDetailsPanel.tsx (5 hunks)
• web/src/components/torrents/details/WebSeedsTable.tsx (1 hunks)
• web/src/components/torrents/details/index.ts (1 hunks)
• web/src/lib/api.ts (2 hunks)
• web/src/types/index.ts (1 hunks)

🧰 Additional context used

🧠 Learnings (6)

📓 Common learnings

Learnt from: finevan
Repo: autobrr/qui PR: 677
File: web/src/components/torrents/AddTorrentDialog.tsx:496-504
Timestamp: 2025-12-03T18:11:08.682Z
Learning: In the AddTorrentDialog component (web/src/components/torrents/AddTorrentDialog.tsx), temporary path settings (useDownloadPath/downloadPath) should be applied per-torrent rather than updating global instance preferences. The UI/UX is designed to suggest that these options apply to the individual torrent being added.

📚 Learning: 2025-11-06T11:59:21.390Z

Learnt from: Audionut
Repo: autobrr/qui PR: 553
File: web/src/components/torrents/TorrentTableOptimized.tsx:1510-1515
Timestamp: 2025-11-06T11:59:21.390Z
Learning: In the qui project, the API layer in web/src/lib/api.ts normalizes backend snake_case responses to camelCase for frontend consumption. For CrossSeed search results, the backend's download_url field is transformed to downloadUrl in the searchCrossSeedTorrent method, so frontend code should always use the camelCase variant (result.downloadUrl).

Applied to files:

• web/src/lib/api.ts

📚 Learning: 2025-11-28T20:32:30.126Z

Learnt from: s0up4200
Repo: autobrr/qui PR: 641
File: internal/services/crossseed/service.go:209-212
Timestamp: 2025-11-28T20:32:30.126Z
Learning: Repo: autobrr/qui PR: 641
File: internal/services/crossseed/service.go
Learning: The cross-seed recheck-resume worker intentionally runs for the process lifetime and keys pending entries by hash only. This is acceptable under the current constraint that background seeded-search runs operate on a single instance at a time; graceful shutdown and instanceID|hash keying are deferred by design.

Applied to files:

• internal/qbittorrent/sync_manager.go

📚 Learning: 2025-11-21T21:11:50.633Z

Learnt from: s0up4200
Repo: autobrr/qui PR: 625
File: internal/qbittorrent/sync_manager.go:1112-1123
Timestamp: 2025-11-21T21:11:50.633Z
Learning: In internal/qbittorrent/sync_manager.go, the GetCachedFilesBatch interface documentation (around line 39-40) should specify "uppercase hex" instead of "lowercase hex" to match the actual normalization practice used throughout the codebase (e.g., normalizeHash in internal/services/crossseed/service.go uses strings.ToUpper, and hash filtering uses uppercase).

Applied to files:

• internal/qbittorrent/sync_manager.go

📚 Learning: 2025-11-25T22:46:03.762Z

Learnt from: s0up4200
Repo: autobrr/qui PR: 632
File: internal/backups/service.go:1401-1404
Timestamp: 2025-11-25T22:46:03.762Z
Learning: In qui's backup service (internal/backups/service.go), background torrent downloads initiated during manifest import intentionally use a fire-and-forget pattern with the shared service context (s.ctx). Per-run cancellation is not needed, as orphaned downloads completing after run deletion are considered harmless and acceptable. This design prioritizes simplicity over per-run lifecycle management for background downloads.

Applied to files:

• internal/qbittorrent/sync_manager.go

📚 Learning: 2025-12-03T18:11:08.682Z

Learnt from: finevan
Repo: autobrr/qui PR: 677
File: web/src/components/torrents/AddTorrentDialog.tsx:496-504
Timestamp: 2025-12-03T18:11:08.682Z
Learning: In the AddTorrentDialog component (web/src/components/torrents/AddTorrentDialog.tsx), temporary path settings (useDownloadPath/downloadPath) should be applied per-torrent rather than updating global instance preferences. The UI/UX is designed to suggest that these options apply to the individual torrent being added.

Applied to files:

• web/src/components/torrents/TorrentDetailsPanel.tsx

🧬 Code graph analysis (4)

web/src/lib/api.ts (1)

web/src/types/index.ts (1)

• WebSeed (499-501)

internal/api/handlers/torrents.go (1)

internal/api/handlers/helpers.go (2)

• RespondError (34-38)
• RespondJSON (22-31)

internal/qbittorrent/sync_manager.go (1)

web/src/types/index.ts (1)

• WebSeed (499-501)

web/src/components/torrents/TorrentDetailsPanel.tsx (4)

web/src/components/torrents/details/WebSeedsTable.tsx (1)

• WebSeedsTable (30-173)

web/src/components/torrents/details/index.ts (1)

• WebSeedsTable (12-12)

web/src/lib/linkUtils.tsx (1)

• renderTextWithLinks (17-53)

web/src/lib/utils.ts (1)

• copyTextToClipboard (186-205)

🪛 Biome (2.1.2)

web/src/components/torrents/details/WebSeedsTable.tsx

[error] 47-47: Wrap comments inside children within braces.

Unsafe fix: Wrap the comments with braces

(lint/suspicious/noCommentText)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

• GitHub Check: Run tests

🔇 Additional comments (11)

internal/api/handlers/torrents.go (1)

1155-1180: LGTM! Handler implementation follows established patterns.

The GetTorrentWebSeeds handler is well-structured and consistent with other torrent detail handlers in the file (e.g., GetTorrentTrackers, GetTorrentProperties). Proper input validation, error handling, and logging are all in place.

web/src/components/torrents/details/WebSeedsTable.tsx (2)

63-80: LGTM! Efficient filtering and clipboard logic.

The filtered data memoization and copy handler are well-implemented with proper dependencies and incognito mode guards.

---

82-172: LGTM! Clean UI with proper loading and empty states.

The component provides a good user experience with:

• Loading spinner while fetching
• Empty state messaging
• Live search with clear button
• Count indicators (filtered/total)
• Context menu for actions (properly disabled in incognito mode)

web/src/components/torrents/TorrentDetailsPanel.tsx (3)

37-37: LGTM! WebSeedsTable properly integrated.

The import and tab definition follow established patterns and are positioned logically among other torrent detail tabs.

Also applies to: 50-50

---

346-354: LGTM! Web seeds query properly configured.

The data fetching logic mirrors the trackers pattern with appropriate cache times. Unlike peers, web seeds are relatively static, so the absence of a refetch interval is correct.

---

714-719: LGTM! Webseeds tab UI is well-integrated.

Both horizontal and vertical layouts are properly implemented:

• Horizontal layout delegates to the WebSeedsTable component
• Vertical layout follows the same pattern as peers and trackers tabs
• Loading, empty, and data states are handled consistently
• Incognito mode properly masks URLs and disables copy actions
• Context menu integration matches existing patterns

Also applies to: 1378-1434

internal/api/server.go (1)

387-401: Web seeds route wiring matches existing torrent detail routes

Adding GET /webseeds alongside /peers, /files, etc. is consistent with the existing nested routing structure and handler usage; no issues spotted.

web/src/types/index.ts (1)

499-501: WebSeed type is minimal and appropriately scoped

Using a single url: string field aligns with the backend payload and keeps the public type surface focused; looks good.

web/src/components/torrents/details/index.ts (1)

6-12: Re-exporting WebSeedsTable keeps the details module cohesive

Adding WebSeedsTable to this barrel export matches existing conventions and simplifies imports for consumers like TorrentDetailsPanel.

web/src/lib/api.ts (1)

50-50: Web seeds API surface is consistent with existing torrent detail methods

Importing WebSeed and exposing getTorrentWebSeeds(instanceId, hash) via the shared request helper fits the existing pattern used for properties/trackers/peers; the endpoint path and return type line up cleanly with the new backend route and WebSeed shape.

Also applies to: 1154-1156

internal/qbittorrent/sync_manager.go (1)

1328-1341: GetTorrentWebSeeds implementation matches existing SyncManager patterns

The method’s structure (client lookup, wrapped error messages, simple passthrough to GetTorrentsWebSeedsCtx) is in line with other read-only helpers like GetTorrentProperties and GetTorrentTrackers; no functional issues seen.

[coderabbitai]

Actionable comments posted: 0

🧹 Nitpick comments (1)

internal/web/swagger/openapi.yaml (1)

3624-3630: Consider adding format and required field specifications.

The WebSeed schema would benefit from:

1. Adding format: uri to the url field to indicate it's a URI
2. Specifying required: [url] to mark the url field as required

Apply this diff to enhance the schema:

     WebSeed:
       type: object
+      required:
+        - url
       properties:
         url:
           type: string
+          format: uri
           description: HTTP source URL for the torrent

📜 Review details

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 601b940 and b677886.

📒 Files selected for processing (1)

• internal/web/swagger/openapi.yaml (2 hunks)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

• GitHub Check: Run tests

🔇 Additional comments (1)

internal/web/swagger/openapi.yaml (1)

1354-1376: LGTM! Endpoint follows established patterns.

The new web seeds endpoint is well-structured and consistent with other torrent detail endpoints in the API. The response codes and parameter references are appropriate.

[coderabbitai]

Actionable comments posted: 0

🧹 Nitpick comments (2)

web/src/components/torrents/TorrentDetailsPanel.tsx (2)

1408-1415: Consider consistent incognito masking.

The vertical layout completely masks URLs ("***masked***"), while the WebSeedsTable (horizontal layout) shows partial information (protocol//***masked***pathname...). For consistency and better UX, consider applying the same masking logic in both layouts.

Apply this diff to match the horizontal layout's masking approach:

-                                <p className="font-mono text-xs break-all">
-                                  {incognitoMode ? "***masked***" : renderTextWithLinks(webseed.url)}
-                                </p>
+                                <p className="font-mono text-xs break-all">
+                                  {incognitoMode ? (() => {
+                                    try {
+                                      const parsed = new URL(webseed.url)
+                                      return `${parsed.protocol}//***masked***${parsed.pathname.slice(0, 20)}...`
+                                    } catch {
+                                      return "***masked***"
+                                    }
+                                  })() : renderTextWithLinks(webseed.url)}
+                                </p>

---

1418-1429: Consider using the existing error handling helper.

The inline copy implementation doesn't handle errors, while the copyToClipboard helper (lines 112-119) provides consistent error handling used elsewhere in this file. Consider refactoring for consistency.

Apply this diff:

                             <ContextMenuContent>
                               <ContextMenuItem
-                                onClick={() => {
-                                  if (!incognitoMode) {
-                                    copyTextToClipboard(webseed.url)
-                                    toast.success("URL copied to clipboard")
-                                  }
-                                }}
+                                onClick={() => !incognitoMode && copyToClipboard(webseed.url, "URL")}
                                 disabled={incognitoMode}
                               >

📜 Review details

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between b677886 and 3dec604.

📒 Files selected for processing (2)

• internal/web/swagger/openapi.yaml (2 hunks)
• web/src/components/torrents/TorrentDetailsPanel.tsx (5 hunks)

🚧 Files skipped from review as they are similar to previous changes (1)

• internal/web/swagger/openapi.yaml

🧰 Additional context used

🧠 Learnings (1)

📚 Learning: 2025-12-03T18:11:08.682Z

Learnt from: finevan
Repo: autobrr/qui PR: 677
File: web/src/components/torrents/AddTorrentDialog.tsx:496-504
Timestamp: 2025-12-03T18:11:08.682Z
Learning: In the AddTorrentDialog component (web/src/components/torrents/AddTorrentDialog.tsx), temporary path settings (useDownloadPath/downloadPath) should be applied per-torrent rather than updating global instance preferences. The UI/UX is designed to suggest that these options apply to the individual torrent being added.

Applied to files:

• web/src/components/torrents/TorrentDetailsPanel.tsx

🧬 Code graph analysis (1)

web/src/components/torrents/TorrentDetailsPanel.tsx (5)

web/src/lib/api.ts (1)

• api (1716-1716)

web/src/components/ui/tabs.tsx (2)

• TabsTrigger (69-69)
• TabsContent (69-69)

web/src/components/torrents/details/index.ts (1)

• WebSeedsTable (12-12)

web/src/components/torrents/details/WebSeedsTable.tsx (1)

• WebSeedsTable (30-173)

web/src/lib/utils.ts (1)

• copyTextToClipboard (186-205)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

• GitHub Check: Run tests

🔇 Additional comments (5)

web/src/components/torrents/TorrentDetailsPanel.tsx (5)

37-37: LGTM!

The import follows the established pattern for other detail components.

---

50-50: LGTM!

The tab value is correctly added to the type-safe array.

---

346-354: LGTM!

The query setup follows the same pattern as the trackers query, fetching data upfront to determine tab visibility. This is a reasonable trade-off for better UX.

---

356-361: LGTM!

The redirection logic correctly handles the edge case where web seeds become unavailable while viewing the tab.

---

721-728: LGTM!

The conditional rendering ensures the tab only appears when relevant data is available.