timothy/ersatztv
1
0
Fork 0
Code Issues 57 Pull Requests 6 Actions Packages Projects Releases 9 Wiki Activity

docs(queue): cascade selector through priority tiers #369

Merged
timothy merged 1 commits from codex/237-cascading-selector into main 2026-07-16 16:14:02 +02:00
Conversation 1 Commits 1 Files Changed 1
+33 -21
timothy commented 2026-07-16 16:03:08 +02:00
Owner
Copy Link
Copy Source

Summary

  • persist a bounded top-five selector cascade: arc → open milestone → open review/audits → priority high → medium → low
  • require priority-label fall-through across all open repository issues instead of stopping at an empty higher tier
  • document the bounded mechanical-evidence fallback when a cheap worker lacks repository-scoped issue listing
  • mirror the same canonical rule in tracker #237

Verification

  • git diff --check
  • stale three-candidate/two-fallback wording absent from the changed selector sections
  • fresh low-cost selector simulation correctly reported its repository-list tool limitation
  • bounded fallback simulation returned exactly #333, #334, #15, #69, #70 and stopped at five
  • pre-push API drift check, ESLint, TypeScript typecheck, and production web build passed
  • .NET and live E2E are not applicable to this documentation-only workflow change

Review

Cold review and fresh fallback re-review both returned MERGEABLE with no findings.

Refs #237

## Summary - persist a bounded top-five selector cascade: arc → open milestone → open review/audits → priority high → medium → low - require priority-label fall-through across all open repository issues instead of stopping at an empty higher tier - document the bounded mechanical-evidence fallback when a cheap worker lacks repository-scoped issue listing - mirror the same canonical rule in tracker #237 ## Verification - `git diff --check` - stale three-candidate/two-fallback wording absent from the changed selector sections - fresh low-cost selector simulation correctly reported its repository-list tool limitation - bounded fallback simulation returned exactly `#333, #334, #15, #69, #70` and stopped at five - pre-push API drift check, ESLint, TypeScript typecheck, and production web build passed - .NET and live E2E are not applicable to this documentation-only workflow change ## Review Cold review and fresh fallback re-review both returned `MERGEABLE` with no findings. Refs #237
timothy added 1 commit 2026-07-16 16:03:08 +02:00
docs(queue): cascade selector through priority tiers
Build ErsatzTV Image / Docs update reminder (pull_request) Successful in 11s
Details
Build ErsatzTV Image / decisions.md append-only (pull_request) Successful in 11s
Details
Build ErsatzTV Image / API docs in sync (OpenAPI + endpoint index) (pull_request) Successful in 13s
Details
Build ErsatzTV Image / Formatting (changed .cs conform to .editorconfig) (pull_request) Successful in 8s
Details
Build ErsatzTV Image / Functional E2E (curl contracts) (pull_request) Successful in 3m12s
Details
Build ErsatzTV Image / Build & test (.NET) (pull_request) Successful in 8m18s
Details
Build ErsatzTV Image / EF migration integrity (SQLite + MySql) (pull_request) Successful in 10m7s
Details
Build ErsatzTV Image / Build & push image (amd64) (pull_request) Has been skipped
Details
e8580f4a84 
Refs #237

Co-Authored-By: OpenAI Codex <codex@openai.com>
timothy commented 2026-07-16 16:03:16 +02:00
Author
Owner
Copy Link
Copy Source

Review-verdict: MERGEABLE @ e8580f4a84

Cold review found no contradictions in the top-five cascade. A fresh re-review also cleared the repository-list tool fallback: mechanical bounded collection remains with the orchestrator, while eligibility filtering and ranking remain with the cheap worker.

Review-verdict: MERGEABLE @ e8580f4a84e97941a7d2943fb9ebd7a9e716f5f8 Cold review found no contradictions in the top-five cascade. A fresh re-review also cleared the repository-list tool fallback: mechanical bounded collection remains with the orchestrator, while eligibility filtering and ranking remain with the cheap worker.
timothy merged commit 5c1d2f63fa into main 2026-07-16 16:14:02 +02:00
timothy deleted branch codex/237-cascading-selector 2026-07-16 16:14:02 +02:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
ad-hoc
One-off / ad-hoc work not tracked by a dedicated issue
 api
REST API / HTTP endpoints
 bug
Something isn't working
 ci-cd
Build, test, deploy pipeline
 content
Channel content / schedules / playlists
 dependencies
Dependency updates (Renovate)
 enhancement
New feature or improvement
 frontend
ChicoryTV React SPA frontend
 in-progress
Claimed by an active session — do not pick up
 jellyfin
Jellyfin tuner / IPTV integration
 parked
Excluded from automatic queue pickup; work only when explicitly selected
 priority: high
priority: low
priority: medium
review
Adversarial review finding
 security
Security / vulnerability fix
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
renovate timothy (Timothy)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: timothy/ersatztv#369