Update engineer QA skip wording #1578

Merged
coilysiren merged 1 commit from issue-1577 into main 2026-07-28 03:07:18 +00:00
Member

Update engineer QA skip wording

Ward now describes the in-container review gate as role-separated from QA instead of framing it as pending brokered QA. The engineer seed and headless outcome text now say QA is a separate, opt-in exact-commit verification role bound to the candidate commit, and the PR-and-merge skipped path is covered too.

  • Replaced the stale temporary skip reason in cmd/ward/agent.go.
  • Updated prompt tests to pin the new wording in engineer, pull-request, and pull-request-and-merge flows.
  • Verified ward exec vet, ward exec lint, and pre-commit run --all-files pass.
  • ward exec test is currently blocked by an unrelated TestBuildUpPlanTailnet failure in cmd/ward/container_hostnet_test.go.

closes #1577

Update engineer QA skip wording Ward now describes the in-container review gate as role-separated from QA instead of framing it as pending brokered QA. The engineer seed and headless outcome text now say QA is a separate, opt-in exact-commit verification role bound to the candidate commit, and the PR-and-merge skipped path is covered too. - Replaced the stale temporary skip reason in `cmd/ward/agent.go`. - Updated prompt tests to pin the new wording in engineer, pull-request, and pull-request-and-merge flows. - Verified `ward exec vet`, `ward exec lint`, and `pre-commit run --all-files` pass. - `ward exec test` is currently blocked by an unrelated `TestBuildUpPlanTailnet` failure in `cmd/ward/container_hostnet_test.go`. closes #1577
fix engineer QA outcome wording
All checks were successful
test / test (pull_request) Successful in 42s
06b75094f9
Author
Member

WARD-WORKFLOW: blocked 🛑

details

workflow: pull-request; review summary: blocked: repo-wide ward exec test fails in cmd/ward/container_hostnet_test.go (TestBuildUpPlanTailnet expects host-net on linux but got HostNet=false TSSidecar=true).
The wording fix is in place and the focused checks passed (go test ./cmd/ward -run 'TestReviewGateClauseInSeed|TestEngineerSeedDefaultsSkipReviewGate|TestAgentSeedPromptWorkflow|TestReviewGateWantedHonorsSkipsAndConfig', ward exec vet, ward exec lint, pre-commit run --all-files), but the branch cannot go green until that unrelated test failure is cleared.

— Codex, via ward agent

WARD-WORKFLOW: blocked 🛑 <details><summary>details</summary> workflow: pull-request; review summary: blocked: repo-wide `ward exec test` fails in `cmd/ward/container_hostnet_test.go` (`TestBuildUpPlanTailnet` expects host-net on linux but got `HostNet=false TSSidecar=true`). The wording fix is in place and the focused checks passed (`go test ./cmd/ward -run 'TestReviewGateClauseInSeed|TestEngineerSeedDefaultsSkipReviewGate|TestAgentSeedPromptWorkflow|TestReviewGateWantedHonorsSkipsAndConfig'`, `ward exec vet`, `ward exec lint`, `pre-commit run --all-files`), but the branch cannot go green until that unrelated test failure is cleared. </details> <!-- ward-agent-signature --> — Codex, via `ward agent`
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
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
coilyco-flight-deck/ward!1578
No description provided.