Remove video clip support — become snapshot archiver #126

Merged
coding-agent-marvin8 merged 0 commits from refs/pull/126/head into main 2026-07-23 23:20:07 +00:00
coding-agent-marvin8 commented 2026-07-23 02:05:35 +00:00 (Migrated from codeberg.org)

Closes #125

Removes all video clip processing, ffmpeg transcoding, clip storage, and video serving. Cang is now a snapshot-only archiver.

Changes:

  • 🔥 Delete transcode pipeline (pipeline.py), remove watchdog dependency
  • 🔥 Remove video from adapters, config, storage, db, and migration
  • 🔥 Remove video clips from web UI (routes, templates)
  • Fix scrubber performance — remove interval recreation, optimize filmstrip
  • 📝 Update docs and metadata for snapshot archiver identity
  • 🔥 Delete ROADMAP.md and TODO.md (all items complete)
  • 🔧 Remove ffmpeg from container image, update labels
  • 🐛 Add orphaned .dav/.idx cleanup in snapshot scan

Breaking change: video_retention_days config key silently ignored; existing clips table rows orphaned but left intact.

Closes #125 Removes all video clip processing, ffmpeg transcoding, clip storage, and video serving. Cang is now a snapshot-only archiver. **Changes:** - :fire: Delete transcode pipeline (pipeline.py), remove watchdog dependency - :fire: Remove video from adapters, config, storage, db, and migration - :fire: Remove video clips from web UI (routes, templates) - :zap: Fix scrubber performance — remove interval recreation, optimize filmstrip - :memo: Update docs and metadata for snapshot archiver identity - :fire: Delete ROADMAP.md and TODO.md (all items complete) - :wrench: Remove ffmpeg from container image, update labels - :bug: Add orphaned .dav/.idx cleanup in snapshot scan **Breaking change:** `video_retention_days` config key silently ignored; existing clips table rows orphaned but left intact.
forgejo-actions commented 2026-07-23 02:06:11 +00:00 (Migrated from codeberg.org)

This PR is too large to review automatically (5869 lines; limit is 3000). Please consider splitting it into smaller PRs.

This PR is too large to review automatically (5869 lines; limit is 3000). Please consider splitting it into smaller PRs.
forgejo-actions commented 2026-07-23 02:59:09 +00:00 (Migrated from codeberg.org)

This PR is too large to review automatically (5895 lines; limit is 3000). Please consider splitting it into smaller PRs.

This PR is too large to review automatically (5895 lines; limit is 3000). Please consider splitting it into smaller PRs.
forgejo-actions commented 2026-07-23 03:08:39 +00:00 (Migrated from codeberg.org)

This PR is too large to review automatically (5903 lines; limit is 3000). Please consider splitting it into smaller PRs.

This PR is too large to review automatically (5903 lines; limit is 3000). Please consider splitting it into smaller PRs.
coding-agent-marvin8 commented 2026-07-23 03:42:01 +00:00 (Migrated from codeberg.org)

Release-Notes.md line 7

This should be a dot point, i.e. start with "- "

Fixed in commit 792f986.

[`Release-Notes.md` line 7](https://codeberg.org/MinimalNVR/cang/pulls/126#issuecomment-19907866) > This should be a dot point, i.e. start with "- " ✅ Fixed in commit `792f986`.
forgejo-actions commented 2026-07-23 03:42:14 +00:00 (Migrated from codeberg.org)

This PR is too large to review automatically (5903 lines; limit is 3000). Please consider splitting it into smaller PRs.

This PR is too large to review automatically (5903 lines; limit is 3000). Please consider splitting it into smaller PRs.
forgejo-actions commented 2026-07-23 23:16:45 +00:00 (Migrated from codeberg.org)

This PR is too large to review automatically (7397 lines; limit is 3000). Please consider splitting it into smaller PRs.

This PR is too large to review automatically (7397 lines; limit is 3000). Please consider splitting it into smaller PRs.
Sign in to join this conversation.
No reviewers
No labels
No milestone
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
marvin8/cang!126
No description provided.