fix: disable automatic verifier follow-up issues

[stranske]

Source: Issue #123

Automated Status Summary

Scope

After merging PR #103 (multi-agent routing infrastructure), we need to:

1. Validate the CLI agent pipeline works end-to-end with the new task-focused prompts
2. Add GITHUB_STEP_SUMMARY output so iteration results are visible in the Actions UI
3. Streamline the Automated Status Summary to reduce clutter when using CLI agents
4. Clean up comment patterns to avoid a mix of old UI-agent and new CLI-agent comments

Context for Agent

Design Decisions & Constraints

• 4. Clean up comment patterns to avoid a mix of old UI-agent and new CLI-agent comments
• The keepalive loop now:
• | <!-- keepalive-loop-summary --> | github-actions[bot] | NEW: CLI agent iteration tracking | ✅ Keep for CLI agents |
• | <!-- keepalive-state:v1 --> | agents-workflows-bot[bot] | State tracking | ⚠️ Multiple copies accumulate |
• | <!-- keepalive-round: N --> | stranske | OLD: Instruction comment | ❌ CLI agents dont need this |
• The goal: For CLI agents (agent:* label), we should have exactly one updating comment (<!-- keepalive-loop-summary -->) instead of accumulating 10+ comments per PR.
• Requires PR #103 to be merged first
• This round you MUST:
• Review the Scope/Tasks/Acceptance below, identify the next incomplete task that requires code, implement it, then post a reply comment with the completed items using their exact original text.

Related Issues/PRs

• #103
• #89
• #123

References

• https://github.com/stranske/Workflows/compare/main...codex/issue-123?expand=1

Blockers & Dependencies

• After merging PR #103 (multi-agent routing infrastructure), we need to:
• 3. Mark a task checkbox complete ONLY after verifying the implementation works.

Tasks

Pipeline Validation

• After PR chore(codex): bootstrap PR for issue #101 #103 merges, create a test PR with agent:codex label
• Verify task appendix appears in Codex prompt (check workflow logs)
• Verify Codex works on actual tasks (not random infrastructure work)
• Verify keepalive comment updates with iteration progress

GITHUB_STEP_SUMMARY

• Add step summary output to agents-keepalive-loop.yml after agent run
• Include: iteration number, tasks completed, files changed, outcome
• Ensure summary is visible in workflow run UI

Conditional Status Summary

• Modify buildStatusBlock() in agents_pr_meta_update_body.js to accept agentType parameter
• When agentType is set (CLI agent): hide workflow table, hide head SHA/required checks
• Keep Scope/Tasks/Acceptance checkboxes for all cases
• Pass agent type from workflow to the update_body job

Comment Pattern Cleanup

• For CLI agents (agent:* label):
• Suppress <!-- gate-summary: --> comment posting (use step summary instead)
• Suppress <!-- keepalive-round: N --> instruction comments (task appendix replaces this)
• Update <!-- keepalive-loop-summary --> to be the single source of truth
• Ensure state marker is embedded in the summary comment (not separate)
• For UI Codex (no agent:* label):
• Keep existing comment patterns (instruction comments, connector bot reports)
• Keep <!-- gate-summary: --> comment
• Add agent_type output to detect job so downstream workflows know the mode
• Update agents-pr-meta.yml to conditionally skip gate summary for CLI agent PRs

Acceptance criteria

• CLI agent receives explicit tasks in prompt and works on them
• Iteration results visible in Actions workflow run summary
• PR body shows checkboxes but not workflow clutter when using CLI agents
• UI Codex path (no agent label) continues to show full status summary
• CLI agent PRs have ≤3 bot comments total (summary, one per iteration update) instead of 10+
• State tracking is consolidated in the summary comment, not scattered

Head SHA: 09cc636
Latest Runs: ❔ in progress — Gate
Required: gate: ❔ in progress

Workflow / Job	Result	Logs
Agents PR meta manager	❔ in progress	View run
CI Autofix Loop	✅ success	View run
Gate	❔ in progress	View run
Health 40 Sweep	✅ success	View run
Health 44 Gate Branch Protection	✅ success	View run
Health 45 Agents Guard	✅ success	View run
Health 50 Security Scan	✅ success	View run
Health 73 Template Completeness	✅ success	View run
Maint 52 Validate Workflows	✅ success	View run
PR 11 - Minimal invariant CI	✅ success	View run
Selftest CI	✅ success	View run
Validate Sync Manifest	✅ success	View run

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 202b598578

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[Copilot]

Pull request overview

This PR re-disables automatic follow-up issue creation in the reusable verifier workflow by restoring the if: false guard that was unintentionally removed in PR #1197 (token minting sweep). The PR also includes additional refactoring changes to agents-guard.yml workflows that are not mentioned in the PR description.

Changes:

• Re-adds if: false guard to prevent automatic follow-up issue creation in reusable-agents-verifier.yml
• Replaces setup-api-client action references with inline npm install scripts in agents-guard.yml (both main and template)
• Maintains consistency between main and template workflow files as required by repository architecture

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File	Description
.github/workflows/reusable-agents-verifier.yml	Re-adds if: false guard on line 627 to disable automatic follow-up issue creation step
.github/workflows/agents-guard.yml	Replaces setup-api-client action with inline npm install fallback script (lines 73-114, 160-201)
templates/consumer-repo/.github/workflows/agents-guard.yml	Identical changes to main agents-guard.yml to maintain template consistency

[stranske-keepalive]

Automated Status Summary

Head SHA: 28a5190
Latest Runs: ⏳ pending — Gate
Required contexts: Gate / gate, Health 45 Agents Guard / Enforce agents workflow protections
Required: core tests (3.11): ⏳ pending, core tests (3.12): ⏳ pending, docker smoke: ⏳ pending, gate: ⏳ pending

Workflow / Job	Result	Logs
(no jobs reported)	⏳ pending	—

Coverage Overview

• Coverage history entries: 1

Coverage Trend

Metric	Value
Current	93.12%
Baseline	85.00%
Delta	+8.12%
Minimum	70.00%
Status	✅ Pass

Top Coverage Hotspots (lowest coverage)

File	Coverage	Missing
src/cli_parser.py	81.8%	4
src/percentile_calculator.py	95.0%	1
src/aggregator.py	95.0%	2
src/__init__.py	100.0%	0
src/ndjson_parser.py	100.0%	0

Updated automatically; will refresh on subsequent CI/Docker completions.

---

Keepalive checklist

Scope

After merging PR #103 (multi-agent routing infrastructure), we need to:

1. Validate the CLI agent pipeline works end-to-end with the new task-focused prompts
2. Add GITHUB_STEP_SUMMARY output so iteration results are visible in the Actions UI
3. Streamline the Automated Status Summary to reduce clutter when using CLI agents
4. Clean up comment patterns to avoid a mix of old UI-agent and new CLI-agent comments

Context for Agent

Design Decisions & Constraints

• 4. Clean up comment patterns to avoid a mix of old UI-agent and new CLI-agent comments
• The keepalive loop now:
• | <!-- keepalive-loop-summary --> | github-actions[bot] | NEW: CLI agent iteration tracking | ✅ Keep for CLI agents |
• | <!-- keepalive-state:v1 --> | agents-workflows-bot[bot] | State tracking | ⚠️ Multiple copies accumulate |
• | <!-- keepalive-round: N --> | stranske | OLD: Instruction comment | ❌ CLI agents dont need this |
• The goal: For CLI agents (agent:* label), we should have exactly one updating comment (<!-- keepalive-loop-summary -->) instead of accumulating 10+ comments per PR.
• Requires PR #103 to be merged first
• This round you MUST:
• Review the Scope/Tasks/Acceptance below, identify the next incomplete task that requires code, implement it, then post a reply comment with the completed items using their exact original text.

Related Issues/PRs

• #103
• #89
• #123

References

• https://github.com/stranske/Workflows/compare/main...codex/issue-123?expand=1

Blockers & Dependencies

• After merging PR #103 (multi-agent routing infrastructure), we need to:
• 3. Mark a task checkbox complete ONLY after verifying the implementation works.

Tasks

Pipeline Validation

• After PR chore(codex): bootstrap PR for issue #101 #103 merges, create a test PR with agent:codex label
• Verify task appendix appears in Codex prompt (check workflow logs)
• Verify Codex works on actual tasks (not random infrastructure work)
• Verify keepalive comment updates with iteration progress

GITHUB_STEP_SUMMARY

• Add step summary output to agents-keepalive-loop.yml after agent run
• Include: iteration number, tasks completed, files changed, outcome
• Ensure summary is visible in workflow run UI

Conditional Status Summary

• Modify buildStatusBlock() in agents_pr_meta_update_body.js to accept agentType parameter
• When agentType is set (CLI agent): hide workflow table, hide head SHA/required checks
• Keep Scope/Tasks/Acceptance checkboxes for all cases
• Pass agent type from workflow to the update_body job

Comment Pattern Cleanup

• For CLI agents (agent:* label):
• Suppress <!-- gate-summary: --> comment posting (use step summary instead)
• Suppress <!-- keepalive-round: N --> instruction comments (task appendix replaces this)
• Update <!-- keepalive-loop-summary --> to be the single source of truth
• Ensure state marker is embedded in the summary comment (not separate)
• For UI Codex (no agent:* label):
• Keep existing comment patterns (instruction comments, connector bot reports)
• Keep <!-- gate-summary: --> comment
• Add agent_type output to detect job so downstream workflows know the mode
• Update agents-pr-meta.yml to conditionally skip gate summary for CLI agent PRs

Acceptance criteria

• CLI agent receives explicit tasks in prompt and works on them
• Iteration results visible in Actions workflow run summary
• PR body shows checkboxes but not workflow clutter when using CLI agents
• UI Codex path (no agent label) continues to show full status summary
• CLI agent PRs have ≤3 bot comments total (summary, one per iteration update) instead of 10+
• State tracking is consolidated in the summary comment, not scattered

[stranske-keepalive]

🤖 Keepalive Loop Status

PR #1259 | Agent: Codex | Iteration 0/5

Current State

Metric	Value
Iteration progress	[----------] 0/5
Action	wait (missing-agent-label)
Disposition	skipped (transient)
Gate	success
Tasks	0/10 complete
Timeout	45 min (default)
Timeout usage	3m elapsed (7%, 42m remaining)
Keepalive	❌ disabled
Autofix	❌ disabled

🔍 Failure Classification

| Error type | infrastructure |
| Error category | resource |
| Suggested recovery | Confirm the referenced resource exists (repo, PR, branch, workflow, or file). |

[agents-workflows-bot]

✅ Codex Completion Checkpoint

Commit: ede6424
Recorded: 2026-02-05T18:39:25.376Z

No new completions recorded this round.

About this comment

This comment is automatically generated to track task completions.
The Automated Status Summary reads these checkboxes to update PR progress.
Do not edit this comment manually.

[github-actions]

Status | ✅ no new diagnostics
History points | 1
Timestamp | 2026-02-05 18:53:37 UTC
Report artifact | autofix-report-pr-1259
Remaining | 0
New | 0
No additional artifacts

[github-actions]

Autofix updated these files:

• tests/scripts/test_issue_consistency_check.py