Skip to content

(CODUX-221) Correct review-blocked compact CI projection - #3055

Merged
numnx merged 1 commit into
feature/CODUX-221-clarify-qa-follow-ups-and-ci-statusfrom
task/feature-codux-22-t09-codex-50841cea-mrjr8grh
Jul 13, 2026
Merged

(CODUX-221) Correct review-blocked compact CI projection#3055
numnx merged 1 commit into
feature/CODUX-221-clarify-qa-follow-ups-and-ci-statusfrom
task/feature-codux-22-t09-codex-50841cea-mrjr8grh

Conversation

@numnx

@numnx numnx commented Jul 13, 2026

Copy link
Copy Markdown
Collaborator

🧩 Task: Correct review-blocked compact CI projection

Automated task PR opened by Code UX for T09.

📋 Summary

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

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 21:48:54 UTC
Finished 2026-07-13 21:52:27 UTC
Duration 3m 34s

📊 Token Usage

Metric Value
Input tokens 61,221
Cached input tokens 650,752
Output tokens 5,000
Total tokens 716,973
Invocations 1
Tool calls 17
Active time 3m 5s

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

✅ 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
Ensure backend card CI status never represents review-only blockers as failed CI.

## Scope
- Edit `src/domain/sprint/card-ci-status.ts`.
- Update `tests/backend/domain/sprint/card-ci-status.test.ts`.

## Requirements
1. Keep actual `failed_checks` and `hasFailedChecks: true` as failed.
2. Treat `review_blocked` with no failed checks as non-failed and do not produce a red CI status.
3. Preserve active CI-fix attention precedence.
4. Add regression tests covering review-only main-merge blockers and actual failed checks.

## Verification
- Run `pnpm exec vitest run tests/backend/domain/sprint/card-ci-status.test.ts`.
- Run `pnpm run lint` and `pnpm run build`.
🌿 Branch Info

Base: feature/CODUX-221-clarify-qa-follow-ups-and-ci-status
Head: task/feature-codux-22-t09-codex-50841cea-mrjr8grh


🤖 Generated by Code UX

@numnx
numnx merged commit cb3c39e 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-t09-codex-50841cea-mrjr8grh branch July 13, 2026 21:57
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