[vavallee]

Summary

Closes #2206.

Torrent indexers get two new optional limits next to the seed ratio: seed time (total minutes) and inactive seed time (minutes without upload). They are carried to the download client at grab time, the same way the ratio is.

• Migration 097 adds seed_time_minutes, inactive_seed_time_minutes (nullable, NULL = client's own rule) and seed_time_source.
• Prowlarr exposes torrentBaseSettings.seedTime (int, minutes) per indexer, so the syncer fills the seed time with the same provenance rules as the ratio. It has its own source column on purpose: every PUT marks seed_ratio_source as user, so sharing it would stop Prowlarr from ever filling the seed time on indexers that have been edited once. Prowlarr has no inactive seed time, so that field is user only.
• API: seedTimeMinutes / inactiveSeedTimeMinutes on indexer create and update. Omitted keeps the stored value, null clears, below 1 or above ten years is a 400.
• UI: both fields under Seed ratio in the add and edit indexer forms, with the per client support in the help text and a "from Prowlarr" note. en.json only.
• Docs: docs/API.md, docs/User-Guide-Wiki.md (support table), docs/QUICKSTART.md (rTorrent note).

Found along the way: the Transmission ratio never applied

AddTorrentDetailed put seedRatioLimit / seedRatioMode on torrent-add. Transmission's torrentAdd reads none of the seed keys; only torrentSet does (libtransmission/rpcimpl.cc, checked on 4.0.6, and the RPC spec lists them under torrent-set only). So the per indexer ratio has been a silent no-op on Transmission. Ratio and idle limit now go out in a torrent-set right after the add, addressed by info hash. A failed torrent-set is logged and the grab still succeeds, matching qBittorrent and Deluge. This has its own ### Fixed line in the fragment.

Client support

Client	Seed ratio	Seed time	Inactive seed time	How
qBittorrent	yes	yes	yes (4.6+)	one setShareLimits call, unset limits sent as -2 (global). Units are minutes: qBit compares seedingTimeLimit * 60 against its seconds counters.
Transmission	yes (now actually applied)	no	yes	torrent-set with seedIdleLimit + seedIdleMode=1. No total seed time exists in the RPC. Idle limit is a uint16 in the daemon, so values above 65535 are clamped.
Deluge	yes	no	no	core options are stop_at_ratio / stop_ratio / remove_at_ratio only
rTorrent	no	no	no	no per torrent limits over XML-RPC
SABnzbd / NZBGet	n/a	n/a	n/a	no seeding

A time limit the client cannot take is logged at debug (unappliedSeedLimits in the adapter) and the client's own rule applies. rTorrent's existing ratio warning is unchanged.

Kept localized for #2713: the Transmission AddTorrent / AddTorrentDetailed signatures are unchanged (new AddTorrentWithLimits underneath), and the adapter change is a few lines per client case.

Fail before

Behavioural, on main's Transmission client:

--- FAIL: TestAddTorrent_SeedRatio_Positive
    seed_limits_test.go:70: torrent-set was not called for a ratio override
--- FAIL: TestAddTorrent_SeedRatio_Unlimited
    seed_limits_test.go:96: torrent-set was not called for the unlimited sentinel

UI tests against main's IndexersTab.tsx: 6 failed (add/edit payloads missing the fields, fields absent). The Go tests for the new fields (db, api, prowlarr, qbittorrent, adapter, scheduler) fail to compile on main since the fields and methods do not exist.

Test plan

• go build ./... && go vet ./internal/... ./cmd/...
• go test ./internal/... ./cmd/... -count=1 (58 packages ok)
• golangci-lint run ./internal/... ./cmd/... (0 issues)
• migration 097 shape and pre-existing rows (TestMigrate097AddsSeedTimeColumns), repo round trip (TestIndexerRepo_SeedTimeLimits)
• API: round trip, omitted keeps, null clears, 0 / negative / over max rejected on create and update (TestIndexerSeedTimeLimits_API)
• Prowlarr: parse seedTime, provenance table, full sync with independent ratio and time ownership
• qBittorrent, Transmission and adapter request bodies against httptest servers
• cd web && npx tsc -b && npx eslint src && npx vitest run (1179 passed; eslint warnings are pre-existing, none in touched files)
• Manual grab against a real qBittorrent 4.6+ / 5.x and a real Transmission 4.x

🤖 Generated with Claude Code

https://claude.ai/code/session_016fJcCVbNnKsmj2MAAMwWj9

[github-actions]

Overall this is a clean, well-reasoned change. The separate provenance column for seed time is the right call — sharing it with seed_ratio_source would freeze every previously-edited indexer out of Prowlarr auto-population. The Transmission ratio fix is a genuine bug fix backed by the libtransmission source.

One behavioral concern: Transmission idle-time clamping is logged at debug

internal/downloader/transmission/client.go — applySeedIdleArgs:

if v > maxSeedIdleMinutes {
    slog.Debug("transmission: inactive seed time clamped to the daemon's maximum", ...)
    v = maxSeedIdleMinutes
}

The API accepts up to 5,256,000 minutes; maxSeedIdleMinutes is 65,535 (~45 days). If a user sets, say, inactiveSeedTimeMinutes: 200000 on a Transmission indexer, they get 65,535 applied with no visible feedback. This is materially different from "limit not supported" (which we log and document): the limit is applied, but at a different value than configured. Debug-level means this is invisible unless the user has debug logging on.

Suggestion: bump to slog.Warn here, or add the Transmission idle-time ceiling to the help text / docs table. The cap is hardware-real (tr_torrentSetIdleLimit is uint16), so there's no workaround — just needs to be surfaced.

---

Everything else looks good:

• Migration 097: additive, nullable defaults, independent down path. Pre-existing rows read back as "no override" correctly.
• Auth / tenancy: no new endpoints; new fields ride the existing indexer CRUD which is already admin-gated. No cross-user visibility risk.
• Prowlarr syncer: independent provenance per field, both reconciled unconditionally (not short-circuited), applyProwlarrSeedTime mirrors applyProwlarrSeedRatio faithfully.
• qBittorrent: -2 for unset fields is correct; the IsZero guard before the call avoids a pointless setShareLimits on torrents with no override.
• SeedLimitsFor(nil) is defended; the idx == nil removal in queue.go is safe because SeedLimitsFor handles it.
• Tests: migration shape, repo round-trip, API (omit-keeps / null-clears / reject-invalid), Prowlarr parse table, qBittorrent form fields, Transmission torrent-set path all covered.

— 🤖 Bindery triage bot (automated). Reply to correct me; a human will see it.

[codecov]

Codecov Report

❌ Patch coverage is 88.54167% with 22 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
internal/downloader/adapter.go	81.63%	8 Missing and 1 partial ⚠️
internal/downloader/transmission/client.go	79.54%	5 Missing and 4 partials ⚠️
internal/api/queue.go	66.66%	1 Missing and 1 partial ⚠️
internal/scheduler/scheduler.go	66.66%	1 Missing and 1 partial ⚠️

📢 Thoughts on this report? Let us know!