Skip to content

(CODUX-221) Preserve all provider-failure QA reviewers in latest-cycle projections - #3057

Merged
numnx merged 2 commits into
feature/CODUX-221-clarify-qa-follow-ups-and-ci-statusfrom
task/feature-codux-22-t11-codex-98474405-mrjru9jd
Jul 13, 2026
Merged

(CODUX-221) Preserve all provider-failure QA reviewers in latest-cycle projections#3057
numnx merged 2 commits into
feature/CODUX-221-clarify-qa-follow-ups-and-ci-statusfrom
task/feature-codux-22-t11-codex-98474405-mrjru9jd

Conversation

@numnx

@numnx numnx commented Jul 13, 2026

Copy link
Copy Markdown
Collaborator

🧩 Task: Preserve all provider-failure QA reviewers in latest-cycle projections

Automated task PR opened by Code UX for T11.

📋 Summary

Sprint: Sprint 221 · Clarify QA follow-ups and CI status · Task: T11

Clarify QA change requests and CI progress consistently across Sprint, Task, and Live cards by projecting complete QA follow-up metadata, deriving durable CI workflow states from existing execution evidence, and presenting accessible, responsive badges with end-to-end regression coverage and synchronized documentation.

🤖 Model & Provider

Provider Codex CLI
Model gpt-5.6-sol

⏱️ Timing

Started 2026-07-13 22:05:50 UTC
Finished 2026-07-13 22:10:03 UTC
Duration 4m 12s

📊 Token Usage

Metric Value
Input tokens 69,899
Cached input tokens 712,448
Output tokens 7,093
Total tokens 789,440
Invocations 1
Tool calls 14
Active time 3m 40s

Included usage estimate (subscription/local login): $0.92

✅ QA Review

⏳ QA review runs after this PR is opened. Check the task card in the dashboard for the outcome once it lands.

📝 Full Task Prompt
## Objective
Correct shared QA summary selection so provider-failure reviewers cannot be hidden by passing reviewers in the same latest QA cycle.

## Scope
- Edit `src/repositories/project-management/qa-review-summary-query.ts`.
- Update `tests/backend/repositories/qa-review-summary-query.test.ts`.
- Do not add migrations or alter unrelated dashboard behavior.

## Requirements
1. Apply the same representative-row precedence to task and sprint summaries: latest `run_index` first, then `running`, `changes_requested`, provider failures (`failed`, `cancelled`, and `errored`), `pass`, and other states.
2. Verify that a latest cycle containing both pass and cancelled reviewers projects the cancelled reviewer.
3. Verify that a latest cycle containing both pass and errored reviewers projects the errored reviewer.
4. Preserve approved payload-field normalization, redaction, legacy rows, malformed payload handling, and existing task/sprint trigger scoping.
5. Add regression coverage for both task-level and sprint-level selection.

## Verification
- Run `pnpm exec vitest run tests/backend/repositories/qa-review-summary-query.test.ts`.
- Run `pnpm run lint`.
- Run `pnpm run build`.
🌿 Branch Info

Base: feature/CODUX-221-clarify-qa-follow-ups-and-ci-status
Head: task/feature-codux-22-t11-codex-98474405-mrjru9jd


🤖 Generated by Code UX

@numnx
numnx merged commit d98c9bf into feature/CODUX-221-clarify-qa-follow-ups-and-ci-status Jul 13, 2026
20 checks passed
@numnx
numnx deleted the task/feature-codux-22-t11-codex-98474405-mrjru9jd branch July 13, 2026 22:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant