Merge staging: Twitch VOD support, dependency bumps, bug fixes

[biodrone]

Summary

New Feature: Twitch VOD Download Support

• VOD download implementation with gRPC integration, SQLite tracking DB, config fields (vod, vod_limit), and graceful shutdown
• Multiple bug fixes: atomic ClaimVOD, lazy DB init, RowsAffected checks, server logging on failure paths
• Integration test coverage with dedicated test channel, stale DB cleanup, file match tightening
• README documentation for VOD usage

Dependency Bumps (13 packages)

• Python: urllib3, pytest, requests, pygments, curl-cffi, streamlink, yt-dlp, grpcio, wheel, protobuf
• Node: lodash (security fix for prototype pollution in baseUnset)
• GitHub Actions: checkout, setup-go, setup-node, codeql-action, docker/login-action, docker/setup-buildx-action, docker/setup-qemu-action, WyriHaximus/github-action-get-previous-tag

Misc

• CodeRabbit config for staging PR reviews
• Fix for yt-dlp .get() on format fields to avoid KeyError

Stats

• 35 commits, 46 files changed, +2011 / -622 lines

Test plan

• All staging PRs passed CI before merge
• CI on this PR (software tests, Docker builds)

Summary by CodeRabbit

• New Features

  • Twitch VOD downloads with per-channel limits, claiming/retry logic, and graceful shutdown that waits for in-flight VODs
  • gRPC support to list available VODs

• Documentation

  • README updated with VOD CLI flags, per-channel config, and Docker persistence guidance
  • Added CHANGELOG entry for v3.6.0

• Tests

  • New VOD DB tests, config parser test, and integration test additions for VOD workflows

• Chores

  • CI action version bumps, dependency/version updates, and repo tooling/config additions

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 209097ab-3757-4ca2-913c-83fbf6d7e747

📥 Commits

Reviewing files that changed from the base of the PR and between a90bb2f and 17a7856.

📒 Files selected for processing (2)

• download_stream.go
• streamdl.go

✅ Files skipped from review due to trivial changes (1)

• download_stream.go

---

📝 Walkthrough

Walkthrough

Adds Twitch VOD support end-to-end: new GetVods gRPC/proto, server-side yt-dlp discovery, client-side getVods/VodResult, FFmpeg-based downloadVOD with sanitized filenames, a SQLite VodDB for claiming/tracking, CLI flags and data dir handling, tests, docs, and CI/dependency bumps.

Changes

Cohort / File(s)	Summary
VOD Core Implementation download_stream.go, vod_db.go, vod_db_test.go	Added sanitizeFilename and downloadVOD (FFmpeg-based VOD downloads); introduced VodDB (SQLite) with InitVodDB, ClaimVOD, MarkVODCompleted, MarkVODFailed; added unit tests covering lifecycle, retries, staleness, and isolation.
gRPC Protocol & Client protos/stream.proto, grpc_client.go	Added GetVods RPC and messages (VodRequest, VodInfo, VodResponse); implemented Go getVods() with timeout, error-code mapping, and VodResult struct.
Python Server Implementation streamdl_proto_srv.py	Implemented StreamServicer.GetVods and helper get_vods() using yt_dlp to extract VOD entries; added error classification (429/404/500) and safe response construction.
Main Application Logic streamdl.go	Added CLI flags -data, -vod-out, -vod-move; lazy VodDB init/close; VOD polling via getVods using VODLimit; claim/dispatch logic spawning downloadVOD goroutines with vodWg; adjusted live-path retry timing and graceful shutdown waiting for VOD goroutines.
Configuration & Tests config.go, config/config.yml.example, config_reader_test.go, README.md, CHANGELOG.md	Added VOD and VODLimit fields to Streamer; updated example config and config parsing test; documented VOD flags/behavior and Docker guidance in README; added CHANGELOG v3.6.0 entry.
Entrypoint & Integration entrypoint_client.sh, tests/integration/docker-compose.integration.yml, tests/integration/run.sh	Ensure /app/data dir exists and is chown'd; mount /app/data in integration compose; extended integration script with a VOD download phase (channel probing, timeout, validation).
Workflows & Dependencies .github/workflows/*.yml, go.mod, pyproject.toml, .coderabbit.yaml	Bumped multiple GitHub Actions versions; updated Go indirect modules; tightened/raised Python dependencies, added grpcio-tools, bumped Python project version to 3.6.0; added .coderabbit.yaml to enable auto-review excluding proto-generated files.
Misc Docs / Changelog CHANGELOG.md, README.md	Added v3.6.0 changelog notes and README sections for VOD usage, flags, and examples.

Sequence Diagram

sequenceDiagram
    participant App as StreamDL App
    participant GRPC as gRPC Server
    participant YtDlp as yt-dlp
    participant DB as SQLite VodDB
    participant FFmpeg as FFmpeg

    loop VOD discovery
        App->>GRPC: GetVods(site,user,limit)
        GRPC->>YtDlp: extract_info(site/user/videos)
        YtDlp-->>GRPC: vod metadata list
        GRPC-->>App: VodResult[]
    end

    loop per VOD
        App->>DB: ClaimVOD(vodID,user,site,title,staleThreshold)
        DB-->>App: claimed true/false

        alt claimed
            App->>GRPC: GetStream(videos/vodID)
            GRPC-->>App: download URL
            App->>FFmpeg: exec(download URL → outPath)
            FFmpeg-->>App: exit success/failure
            App->>DB: MarkVODCompleted / MarkVODFailed
        end
    end

Loading

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~50 minutes

Possibly related PRs

• feat: Twitch VOD support #551: Implements Twitch VOD support with overlapping changes to protos, VodDB, download logic, main loop, tests, and docs.
• chore 🤖(deps): bump yt-dlp from 2025.10.22 to 2026.3.17 #540: Bumps yt-dlp and related Python dependency constraints in pyproject.toml (directly related dependency change).
• Infrastructure, configuration, and core improvements #507: Shares edits to download logic, stream handling, entrypoint scripts, and CI workflows (related code-level changes).

Suggested labels

dependencies

🐰 I hop with glee through code and log,
VODs now queued, each filename snug.
SQLite guards each claimed delight,
FFmpeg hums through day and night,
Fetch, save, and cheer — a rabbit's hop of delight! 🎥🥕

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 40.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.
Description check	❓ Inconclusive	The PR description is comprehensive and well-structured, covering new features, dependency updates, testing, and stats; however, it does not follow the required template structure with 'Fixes #' or a bullet-point 'Changes proposed' section.	Reformat the description to match the repository template: start with 'Fixes #' and organize changes under a 'Changes proposed in this pull request:' section with bullet points.

✅ Passed checks (1 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly summarizes the main changes: Twitch VOD support (primary feature), dependency bumps, and bug fixes.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch staging

---

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

[coderabbitai]

Actionable comments posted: 5

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)

tests/integration/run.sh (1)

233-382: ⚠️ Potential issue | 🟠 Major

Don't let the live-phase skip short-circuit the VOD phase.

Phase 5 is appended after the existing exit 0 at Line 119, so off-peak runs still skip the new VOD coverage entirely even though this phase has its own candidate probing. The === PASS banner at Line 233 is also premature for the same reason. Split the live and VOD phases so a live skip doesn't exit before the VOD path is exercised.

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@tests/integration/run.sh` around lines 233 - 382, The script currently exits
early when the live-phase prints "=== PASS: Live stream integration test
succeeded ===" or when the live-phase skip path does exit 0, preventing the
subsequent VOD phase (the "=== VOD Download Test ===" block and its VOD_CHANNEL
probing logic using CANDIDATE_VOD_CHANNELS, ANY_PROBE_OK, and VOD_CHANNEL) from
running; remove or change that early exit so the VOD phase always runs (or only
skip the live-specific success banner), move or rename the premature "=== PASS:
Live stream integration test succeeded ===" banner so final success is printed
only after both live and VOD checks complete, and ensure the skip branch that
currently does exit 0 continues to the VOD probing logic instead of terminating
the script.

🧹 Nitpick comments (2)

vod_db_test.go (1)

160-162: Consider checking errors from ClaimVOD.

These calls discard the error with _. While the test intent is clear (checking the boolean), silently ignoring errors could mask unexpected failures.

🔧 Suggested improvement

-	d1, _ := db.ClaimVOD("111", "user1", "twitch.tv", "Title A", staleThreshold)
-	d2, _ := db.ClaimVOD("222", "user2", "twitch.tv", "Title B", staleThreshold)
-	d3, _ := db.ClaimVOD("333", "user3", "twitch.tv", "Title C", staleThreshold)
+	d1, err := db.ClaimVOD("111", "user1", "twitch.tv", "Title A", staleThreshold)
+	if err != nil {
+		t.Fatalf("ClaimVOD 111 failed: %v", err)
+	}
+	d2, err = db.ClaimVOD("222", "user2", "twitch.tv", "Title B", staleThreshold)
+	if err != nil {
+		t.Fatalf("ClaimVOD 222 failed: %v", err)
+	}
+	d3, err := db.ClaimVOD("333", "user3", "twitch.tv", "Title C", staleThreshold)
+	if err != nil {
+		t.Fatalf("ClaimVOD 333 failed: %v", err)
+	}

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@vod_db_test.go` around lines 160 - 162, The test is discarding errors from
ClaimVOD calls (d1, d2, d3) which can hide failures; update the test to capture
and assert the returned error for each ClaimVOD invocation (or call
t.Fatalf/t.Errorf if err != nil) so failures surface rather than being
ignored—locate the ClaimVOD(...) calls that assign to d1/d2/d3 and add error
checks immediately after each call.

grpc_client.go (1)

26-84: Extract the shared gRPC setup before this drifts further.

This repeats the same addr/port lookup, grpc.NewClient, close, and timeout plumbing as getStream, and the two paths are already diverging on logging/error translation. A small shared helper would keep live/VOD transport behavior in sync.

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@grpc_client.go` around lines 26 - 84, Refactor the duplicated gRPC setup in
getVods to a shared helper (e.g., newGRPCConn or dialStreamGRPC) that
encapsulates reading STREAMDL_GRPC_ADDR/PORT, calling grpc.NewClient with
grpc.WithTransportCredentials(insecure.NewCredentials()), returning the
grpc.ClientConn (or client interface), a context with timeout (and its cancel
func) and an io.Closer-style cleanup (or ensure conn.Close is called by caller);
then update getVods (and getStream) to call this helper, use the returned ctx
for RPCs, and remove the duplicated addr/port/dial/close/timeout plumbing so
both functions share connection and timeout behavior while keeping their own
logging/error translation.

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@CHANGELOG.md`:
- Around line 1-83: Add a concise bullet under the 3.6.0 changelog (near the New
Features or top-level summary) documenting the Twitch VOD support and the new
config/CLI surface: mention the feature name (Twitch VOD support) and list the
new config/flags (vod, vod_limit, -data, -vod-out, -vod-move) plus a short note
on what each does (e.g., enable VOD downloads, limit VOD count, data dir, VOD
output path, atomically move completed VODs). Ensure this appears alongside
other headline changes in the 3.6.0 notes so users scanning the changelog see
the functional impact.

In `@download_stream.go`:
- Around line 540-541: The code currently calls both cmd.Process.Wait() and
cmd.Wait(), which duplicate reaping the child and can cause errors; remove the
redundant call and use cmd.Wait() to both wait for and reap the process (or, if
you intentionally need low-level Process.Wait(), then remove the subsequent
cmd.Wait() and propagate/handle that error accordingly), updating any error
handling around cmd.Wait()/cmd.Process.Wait() to reflect the single wait path.

In `@streamdl.go`:
- Around line 70-74: The process is exiting with os.Exit(0) before the deferred
vodDB.Close() in main() can run; change the early termination in main() to
return instead of calling os.Exit so the defer that checks vodDB and calls
vodDB.Close() executes, and ensure the defer that wraps vodDB.Close() (the func
that checks if vodDB != nil then vodDB.Close()) remains in main() so resources
are closed on return.
- Around line 191-203: The nested retry shadows err and therefore the "thrice"
branch never sees the second retry result; in the block that calls getStream
after sleeping, replace the short declaration (url, err := getStream(...)) with
an assignment that reuses the outer err (url, err = getStream(...)) or use a new
variable name and then check that variable, then change the subsequent else-if
to inspect the second-retry error (the reused err or the new name) so that the
"rate limited thrice" branch in the getStream / downloadStream retry logic for
streamer.User can be reached and other error paths are not silently dropped;
ensure you still lock/update urls via urlsMu and call downloadStream when the
second retry succeeds.

---

Outside diff comments:
In `@tests/integration/run.sh`:
- Around line 233-382: The script currently exits early when the live-phase
prints "=== PASS: Live stream integration test succeeded ===" or when the
live-phase skip path does exit 0, preventing the subsequent VOD phase (the "===
VOD Download Test ===" block and its VOD_CHANNEL probing logic using
CANDIDATE_VOD_CHANNELS, ANY_PROBE_OK, and VOD_CHANNEL) from running; remove or
change that early exit so the VOD phase always runs (or only skip the
live-specific success banner), move or rename the premature "=== PASS: Live
stream integration test succeeded ===" banner so final success is printed only
after both live and VOD checks complete, and ensure the skip branch that
currently does exit 0 continues to the VOD probing logic instead of terminating
the script.

---

Nitpick comments:
In `@grpc_client.go`:
- Around line 26-84: Refactor the duplicated gRPC setup in getVods to a shared
helper (e.g., newGRPCConn or dialStreamGRPC) that encapsulates reading
STREAMDL_GRPC_ADDR/PORT, calling grpc.NewClient with
grpc.WithTransportCredentials(insecure.NewCredentials()), returning the
grpc.ClientConn (or client interface), a context with timeout (and its cancel
func) and an io.Closer-style cleanup (or ensure conn.Close is called by caller);
then update getVods (and getStream) to call this helper, use the returned ctx
for RPCs, and remove the duplicated addr/port/dial/close/timeout plumbing so
both functions share connection and timeout behavior while keeping their own
logging/error translation.

In `@vod_db_test.go`:
- Around line 160-162: The test is discarding errors from ClaimVOD calls (d1,
d2, d3) which can hide failures; update the test to capture and assert the
returned error for each ClaimVOD invocation (or call t.Fatalf/t.Errorf if err !=
nil) so failures surface rather than being ignored—locate the ClaimVOD(...)
calls that assign to d1/d2/d3 and add error checks immediately after each call.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 482d0821-9fab-49a2-9fd3-2ae2d57f31bd

📥 Commits

Reviewing files that changed from the base of the PR and between 9f841c0 and e07ee5d.

⛔ Files ignored due to path filters (25)

• go.sum is excluded by !**/*.sum
• node_modules/.package-lock.json is excluded by !**/node_modules/**
• node_modules/lodash/README.md is excluded by !**/node_modules/**
• node_modules/lodash/_baseOrderBy.js is excluded by !**/node_modules/**
• node_modules/lodash/_baseUnset.js is excluded by !**/node_modules/**
• node_modules/lodash/_setCacheHas.js is excluded by !**/node_modules/**
• node_modules/lodash/compact.js is excluded by !**/node_modules/**
• node_modules/lodash/core.js is excluded by !**/node_modules/**
• node_modules/lodash/core.min.js is excluded by !**/node_modules/**, !**/*.min.js
• node_modules/lodash/flake.lock is excluded by !**/node_modules/**, !**/*.lock
• node_modules/lodash/flake.nix is excluded by !**/node_modules/**
• node_modules/lodash/fromPairs.js is excluded by !**/node_modules/**
• node_modules/lodash/lodash.js is excluded by !**/node_modules/**
• node_modules/lodash/lodash.min.js is excluded by !**/node_modules/**, !**/*.min.js
• node_modules/lodash/package.json is excluded by !**/node_modules/**
• node_modules/lodash/random.js is excluded by !**/node_modules/**
• node_modules/lodash/release.md is excluded by !**/node_modules/**
• node_modules/lodash/template.js is excluded by !**/node_modules/**
• node_modules/lodash/templateSettings.js is excluded by !**/node_modules/**
• package-lock.json is excluded by !**/package-lock.json
• protos/stream.pb.go is excluded by !**/*.pb.go
• protos/stream_grpc.pb.go is excluded by !**/*.pb.go
• stream_pb2.py is excluded by !stream_pb2.py
• stream_pb2_grpc.py is excluded by !stream_pb2_grpc.py
• uv.lock is excluded by !**/*.lock

📒 Files selected for processing (22)

• .coderabbit.yaml
• .github/workflows/deploy_master.yml
• .github/workflows/deploy_staging.yml
• .github/workflows/test.yml
• .github/workflows/update-actions.yml
• CHANGELOG.md
• README.md
• config.go
• config/config.yml.example
• config_reader_test.go
• download_stream.go
• entrypoint_client.sh
• go.mod
• grpc_client.go
• protos/stream.proto
• pyproject.toml
• streamdl.go
• streamdl_proto_srv.py
• tests/integration/docker-compose.integration.yml
• tests/integration/run.sh
• vod_db.go
• vod_db_test.go

[coderabbitai]

⚠️ Potential issue | 🟡 Minor

Document the VOD feature in the 3.6.0 notes.

The new entry reads mostly like dependency/test churn, but this release's headline change is Twitch VOD support and its new config/CLI surface. Without a bullet for vod, vod_limit, -data, -vod-out, and -vod-move, users scanning the changelog won't discover the biggest functional change in 3.6.0.

🧰 Tools

🪛 markdownlint-cli2 (0.22.0)

[warning] 1-1: First line in a file should be a top-level heading

(MD041, first-line-heading, first-line-h1)

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@CHANGELOG.md` around lines 1 - 83, Add a concise bullet under the 3.6.0
changelog (near the New Features or top-level summary) documenting the Twitch
VOD support and the new config/CLI surface: mention the feature name (Twitch VOD
support) and list the new config/flags (vod, vod_limit, -data, -vod-out,
-vod-move) plus a short note on what each does (e.g., enable VOD downloads,
limit VOD count, data dir, VOD output path, atomically move completed VODs).
Ensure this appears alongside other headline changes in the 3.6.0 notes so users
scanning the changelog see the functional impact.

[coderabbitai]

Actionable comments posted: 2

♻️ Duplicate comments (1)

download_stream.go (1)

449-450: ⚠️ Potential issue | 🔴 Critical

Eliminate double call to cmd.Wait() which causes a race condition.

The code calls cmd.Wait() twice concurrently—once in a goroutine at line 526 and again in the shutdown branch at line 536. Go's os/exec.Cmd.Wait() cannot be called more than once or concurrently from multiple goroutines. The second call will fail with "exec: Wait was already called" or cause a data race, and the control-listener goroutine never exits after a natural completion.

Consolidate to a single cmd.Wait() call by removing the unnecessary sigint channel and control-listener goroutine, then listen for <-control directly in the select. In the shutdown case, read from <-naturalFinish instead of calling cmd.Wait() again, and call MarkVODFailed() immediately to allow a restart to reclaim the VOD right away.

Suggested fix

-	sigint := make(chan bool)
-
 	// Always ensure base directories have correct permissions first
 	if err := createDirWithUmask(outLoc); err != nil {
 		log.Errorf("Failed to create output directory %s: %v", outLoc, err)
@@
-	// Single control listener
-	go func() {
-		for {
-			_, more := <-control
-			if !more {
-				sigint <- true
-				return
-			}
-		}
-	}()
-
 	buf := &bytes.Buffer{}
 	cmd := fluentffmpeg.
@@
 	select {
-	case <-sigint:
+	case <-control:
 		log.Tracef("Sending SIGINT to VOD %s process", vod.ID)
 		if err := cmd.Process.Signal(syscall.SIGINT); err != nil {
 			log.Errorf("Failed to send SIGINT to VOD %s: %v", vod.ID, err)
 		}
-		// Use only cmd.Wait() which handles process reaping internally
-		if err := cmd.Wait(); err != nil {
+		if err := <-naturalFinish; err != nil {
 			log.Tracef("VOD %s process exited after SIGINT: %v", vod.ID, err)
 		}
-		// Interrupted — leave as 'downloading'; stale threshold will handle retry
+		if vodDB != nil {
+			if err := vodDB.MarkVODFailed(vod.ID); err != nil {
+				log.Errorf("Failed to mark VOD %s as failed: %v", vod.ID, err)
+			}
+		}
 		return

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@download_stream.go` around lines 449 - 450, The code currently calls
cmd.Wait() from two places causing races; remove the control-listener goroutine
and the sigint channel, stop calling cmd.Wait() twice, and centralize waiting on
the process through the existing naturalFinish channel: have the control branch
in the select read from the control channel directly (instead of signaling
sigint), and in the shutdown handling read the error from <-naturalFinish (do
not call cmd.Wait() again) and call MarkVODFailed() immediately so the VOD lock
can be reclaimed; update any logic referencing sigint or the removed goroutine
to use naturalFinish and control and ensure only the goroutine that sends to
naturalFinish invokes cmd.Wait() once.

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@download_stream.go`:
- Around line 452-470: The setup block that creates directories
(createDirWithUmask) can return early while the VOD is already claimed
(outLoc/moveLoc/subfolder/user), so mirror the cmd.Start() error-path cleanup:
before each early return on directory/permission errors call the same
VOD-release/cleanup function used in the cmd.Start() failure handling (the
function invoked in that branch to release the VOD claim) so the database row
isn't left in `downloading` state; locate the cmd.Start() failure branch to copy
the exact release call and insert it before each return in this setup block.

In `@streamdl.go`:
- Around line 154-159: ClaimVOD moved the VOD into the in-progress/downloading
state before calling getStream, so if getStream(site.Site, "videos/"+vod.ID,
streamer.Quality) returns an error you must release that claim so the VOD can be
retried; on getStream error call the counterpart that reverts the claim (the
function that unclaims/releases a VOD—e.g., ReleaseVOD or the existing "release"
function used elsewhere for ClaimVOD) with the vod ID, handle/log any error from
that release call, then continue without leaving the row stuck in downloading;
ensure this change is made in the same block where getStream is called (around
resolvedURL, err := getStream(...)) and that no downloader goroutine is started
when releasing the claim.

---

Duplicate comments:
In `@download_stream.go`:
- Around line 449-450: The code currently calls cmd.Wait() from two places
causing races; remove the control-listener goroutine and the sigint channel,
stop calling cmd.Wait() twice, and centralize waiting on the process through the
existing naturalFinish channel: have the control branch in the select read from
the control channel directly (instead of signaling sigint), and in the shutdown
handling read the error from <-naturalFinish (do not call cmd.Wait() again) and
call MarkVODFailed() immediately so the VOD lock can be reclaimed; update any
logic referencing sigint or the removed goroutine to use naturalFinish and
control and ensure only the goroutine that sends to naturalFinish invokes
cmd.Wait() once.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 0e3c8962-3488-411f-83ea-8a30f2ba6683

📥 Commits

Reviewing files that changed from the base of the PR and between e07ee5d and a90bb2f.

📒 Files selected for processing (2)

• download_stream.go
• streamdl.go