docs(queue): enforce milestone-aware picker ranking #348

Merged

timothy merged 2 commits from codex/queue-picker-claim-comments into main 2026-07-14 08:03:47 +02:00

Conversation 1 Commits 2 Files Changed 1

+19 -7

timothy commented 2026-07-13 23:24:59 +02:00

Owner

Copy Link

Copy Source

Summary
Fixes both queue-selector failures exposed during pickup:

• reviewer audits are classified from claim/deliverable comments, with claims persisting until explicit release or abandonment;
• candidate discovery enumerates every open milestone and its open issues plus open review issues, so milestone work outranks unmilestoned backlog before priority: is compared;
• epic containers are excluded when #237 names eligible children, and equal-tier ties preserve that explicit order before issue number.

The canonical #237 protocol carries the same rules. With live state, #245 is the correct next result: it is the first eligible open child in the open ChicoryTV modularization milestone; unmilestoned high-priority #345 ranks below it.

Verification

• git diff --check
• Pre-push: npm run check:api, npm run lint, npm run typecheck, npm run build

Review
Pure docs/workflow change only; no runtime or server-state effect. Independent code review is exempt under the documented pure-docs leaf rule.

Fixes #347

**Summary** Fixes both queue-selector failures exposed during pickup: - reviewer audits are classified from claim/deliverable comments, with claims persisting until explicit release or abandonment; - candidate discovery enumerates every open milestone and its open issues plus open `review` issues, so milestone work outranks unmilestoned backlog before `priority:` is compared; - epic containers are excluded when #237 names eligible children, and equal-tier ties preserve that explicit order before issue number. The canonical #237 protocol carries the same rules. With live state, #245 is the correct next result: it is the first eligible open child in the open ChicoryTV modularization milestone; unmilestoned high-priority #345 ranks below it. **Verification** - `git diff --check` - Pre-push: `npm run check:api`, `npm run lint`, `npm run typecheck`, `npm run build` **Review** Pure docs/workflow change only; no runtime or server-state effect. Independent code review is exempt under the documented pure-docs leaf rule. Fixes #347

timothy added 1 commit 2026-07-13 23:25:00 +02:00

docs(queue): preserve reviewer comment claims ... d235b91d15

Fixes #347

Co-Authored-By: OpenAI Codex <codex@openai.com>

timothy referenced this pull request 2026-07-13 23:25:49 +02:00

Saving a remote connection falsely triggers the dirty guard #344

timothy added 1 commit 2026-07-13 23:38:03 +02:00

docs(queue): enumerate milestone candidates ... e280e9072c

Refs #347

Co-Authored-By: OpenAI Codex <codex@openai.com>

timothy changed title from docs(queue): preserve reviewer comment claims to docs(queue): enforce milestone-aware picker ranking 2026-07-13 23:38:11 +02:00

timothy referenced this pull request 2026-07-14 08:02:29 +02:00

Saving a remote connection falsely triggers the dirty guard #344

timothy commented 2026-07-14 08:03:40 +02:00

Author

Owner

Copy Link

Copy Source

Review-verdict: MERGEABLE @ e280e9072c

Exact docs diff and canonical #237 alignment verified. Pure docs/workflow change with no executable or server-state effect; independent review and live E2E are explicitly exempt.

Review-verdict: MERGEABLE @ e280e9072c676d6ea255dc5e211ba62930c9dc7a Exact docs diff and canonical #237 alignment verified. Pure docs/workflow change with no executable or server-state effect; independent review and live E2E are explicitly exempt.

timothy merged commit e454ed94fd into main 2026-07-14 08:03:47 +02:00

timothy deleted branch codex/queue-picker-claim-comments 2026-07-14 08:03:47 +02:00

timothy referenced this issue from a commit 2026-07-14 08:03:48 +02:00

Merge pull request 'docs(queue): enforce milestone-aware picker ranking' (#348) from codex/queue-picker-claim-comments into main

timothy referenced this pull request 2026-07-14 08:04:36 +02:00

ChicoryTV rewrite — queue tracker (goal, arc, session protocol) #237

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

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: timothy/ersatztv#348