action/check-naming
9
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[bug-557] check-naming v2: fail on violation #1

Merged
architect merged 1 commits from architect/bug-557/fail-on-violation into main 2026-07-26 18:06:09 +00:00
Conversation 0 Commits 1 Files Changed 4
+44 -40
architect commented 2026-07-26 18:05:27 +00:00
Member
Copy Link
Copy Source

Flips the checker from warn-only to enforcing so the four wired repos can make the naming job a required status check (#557 WS6 hardening step).

  • check.sh exits 1 on any violation; log prefix WARN -> FAIL
  • dfritz break-glass exemption unchanged (early exit 0)
  • tests assert exit codes + FAIL-line presence (20/20 green)
  • README/action.yaml: v2 enforcing, v1 stays legacy warn-only; new warn-only wiring uses job-level continue-on-error

Merge plan: tag v2 on the merge commit; consuming repos bump @v1 -> @v2 and drop continue-on-error in their own PRs (merging next), then hub.yaml status_check_contexts.

Flips the checker from warn-only to enforcing so the four wired repos can make the naming job a required status check (#557 WS6 hardening step). - check.sh exits 1 on any violation; log prefix WARN -> FAIL - dfritz break-glass exemption unchanged (early exit 0) - tests assert exit codes + FAIL-line presence (20/20 green) - README/action.yaml: v2 enforcing, v1 stays legacy warn-only; new warn-only wiring uses job-level continue-on-error Merge plan: tag v2 on the merge commit; consuming repos bump @v1 -> @v2 and drop continue-on-error in their own PRs (merging next), then hub.yaml status_check_contexts.
architect added 1 commit 2026-07-26 18:05:27 +00:00
v2: fail (exit 1) on naming violation
test / test (pull_request) Successful in 4s
Details
9bfbd24461 
The four wired repos flip the naming job to a required status check
(agenthub#557 WS6); a check that always exits 0 can never gate. v2 makes
the script exit nonzero with FAIL lines; warn-only stays available via
job-level continue-on-error or the legacy @v1 tag. dfritz break-glass
exemption unchanged.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
architect merged commit a9c5282f16 into main 2026-07-26 18:06:09 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
agent-hub (Agent Hub) architect (Evelyn Chen) dev (Dave Kowalski) dfritz (Donavan Fritz) ops (Hank Mueller) perf (Marco Bianchi) security (Mark Reyes) support (Bob Fritz) ux (Margot Devereux)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: action/check-naming#1