Show job selection summary and estimated final size on queue cards #26

Merged

cheapnud merged 4 commits from d/queue-job-summary into master 2026-08-22 15:56:24 +00:00

Conversation 1 Commits 4 Files changed 4 +149

cheapnud commented 2026-08-22 15:16:15 +00:00

Owner

Copy link

No description provided.

cheapnud added 1 commit 2026-08-22 15:16:25 +00:00

Show job selection summary and estimated final size on queue cards ... 532859e398

Queue cards now list what the job was configured with: encoder and quality
(labeled per encoder family), export preset name, resolution, frame rate,
audio settings, selected tracks, render range and post action. While a job
renders, the final file size is extrapolated from bytes written vs progress.

cheapnud added 1 commit 2026-08-22 15:29:47 +00:00

Retrigger CI 659d37715a

forgejo-actions commented 2026-08-22 15:30:00 +00:00

Copy link

🤖 Claude AI Code Review

1. Summary
   Adds a selection summary chip row and estimated final size to render job cards, plus a PresetName field for display.

2. Code Quality ⭐⭐⭐⭐☆
   Clean, defensive parsing with sensible caching and thoughtful edge-case handling (low-progress ratio guard, null codecs).

3. Issues

🟢 RenderJobCard.razor — _estimatedFinalSize is only recalculated inside OnFileSizeTimerElapsed when the output file exists. If progress reaches ≥3% but the file check path changes/misses a tick, the chip may show a stale estimate. Minor; likely acceptable given the timer refresh cadence.

🟢 RenderJobCard.razor — division uses progress / 100.0 after Math.Min(..., 100), but if ProgressPercentage can be negative (unlikely), the estimate would go nonsensical. Guard is >= 3 so effectively safe.

4. Verdict: APPROVE

The code is well-structured and defensive. No real bugs, resource leaks, or breaking changes found.

---

📊 Tokens: 4513 input + 385 output | 💰 Cost: ~$0.0322
Generated by Claude Opus 4.8 via Gitea Actions

## 🤖 Claude AI Code Review 1. **Summary** Adds a selection summary chip row and estimated final size to render job cards, plus a `PresetName` field for display. 2. **Code Quality** ⭐⭐⭐⭐☆ Clean, defensive parsing with sensible caching and thoughtful edge-case handling (low-progress ratio guard, null codecs). 3. **Issues** 🟢 RenderJobCard.razor — `_estimatedFinalSize` is only recalculated inside `OnFileSizeTimerElapsed` when the output file exists. If progress reaches ≥3% but the file check path changes/misses a tick, the chip may show a stale estimate. Minor; likely acceptable given the timer refresh cadence. 🟢 RenderJobCard.razor — division uses `progress / 100.0` after `Math.Min(..., 100)`, but if `ProgressPercentage` can be negative (unlikely), the estimate would go nonsensical. Guard is `>= 3` so effectively safe. 4. **Verdict**: **APPROVE** The code is well-structured and defensive. No real bugs, resource leaks, or breaking changes found. --- 📊 **Tokens**: 4513 input + 385 output | 💰 **Cost**: ~$0.0322 *Generated by Claude Opus 4.8 via Gitea Actions*

cheapnud added 1 commit 2026-08-22 15:31:50 +00:00

Guard against null codec fields in queue card summary 9e79a97936

cheapnud added 1 commit 2026-08-22 15:35:33 +00:00

Retrigger CI c0d644f056

cheapnud merged commit ac8cd11a5a into master 2026-08-22 15:56:24 +00:00

cheapnud referenced this pull request from a commit 2026-08-22 15:56:25 +00:00

Merge pull request 'Show job selection summary and estimated final size on queue cards' (#26) from d/queue-job-summary into master

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels

No items

No labels

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

2 participants

Notifications

Subscribe

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

cheapnud/CheapShotcutRandomizer!26

No description provided.