fix: keep merge-sync checkout intact (#1110)

[stranske]

Source: Issue #1110

Automated Status Summary

Scope

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

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.
• ⚠️ Action Required: Unable to determine source issue for PR #1110. The PR title, branch name, or body must contain the issue number (e.g. #123, branch: issue-123, or the hidden marker ).
• ⚠️ Action Required: Unable to determine source issue for PR #1110. The PR title, branch name, or body must contain the issue number (e.g. #123, branch: issue-123, or the hidden marker ).

Related Issues/PRs

• #123
• #103
• #89
• #1110
• stranske/Workflows#103
• stranske/Workflows#89
• stranske/Workflows#123

References

• chore(codex): bootstrap PR for issue #101 #103
• chore(codex): bootstrap PR for issue #79 #89
• Validate CLI agent pipeline and streamline PR status display #123
• https://github.com/stranske/Workflows/compare/main...codex/issue-123?expand=1
• https://github.com/stranske/Workflows/actions/runs/21365692169
• https://github.com/stranske/Workflows/actions/runs/21365059346
• https://github.com/stranske/Workflows/actions/runs/21365059300
• https://github.com/stranske/Workflows/actions/runs/21365059286
• https://github.com/stranske/Workflows/actions/runs/21365059205
• https://github.com/stranske/Workflows/actions/runs/21365059217
• https://github.com/stranske/Workflows/actions/runs/21365059218
• https://github.com/stranske/Workflows/actions/runs/21365059241
• https://github.com/stranske/Workflows/actions/runs/21365059210
• https://github.com/stranske/Workflows/actions/runs/21365059228
• https://github.com/stranske/Workflows/actions/runs/21365059224
• https://github.com/stranske/Workflows/actions/runs/21365059233
• fix: keep merge-sync checkout intact (#1110) #1110
• https://github.com/stranske/Workflows/actions/runs/21365041567
• https://github.com/stranske/Workflows/actions/runs/21364828142
• https://github.com/stranske/Workflows/actions/runs/21364828239
• https://github.com/stranske/Workflows/actions/runs/21364831738
• https://github.com/stranske/Workflows/actions/runs/21364828210
• https://github.com/stranske/Workflows/actions/runs/21364828238
• https://github.com/stranske/Workflows/actions/runs/21364828162
• https://github.com/stranske/Workflows/actions/runs/21364828159
• https://github.com/stranske/Workflows/actions/runs/21364828160
• https://github.com/stranske/Workflows/actions/runs/21364828157
• https://github.com/stranske/Workflows/actions/runs/21364828152
• https://github.com/stranske/Workflows/actions/runs/21364828140
• https://github.com/stranske/Workflows/actions/runs/21364828146
• https://github.com/stranske/Workflows/actions/runs/21364828148

Blockers & Dependencies

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

Tasks

• 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

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: fa8a9e9
• Latest Runs: ✅ success — Gate
• Required: gate: ✅ success
• | Workflow / Job | Result | Logs |
• |----------------|--------|------|
• | Agents PR meta manager | ❔ in progress | View run |
• | CI Autofix Loop | ✅ success | View run |
• | Gate | ✅ success | 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 |

Head SHA: fa8a9e9
Latest Runs: ✅ success — Gate
Required: gate: ✅ success

Workflow / Job	Result	Logs
Agents PR meta manager	❔ in progress	View run
CI Autofix Loop	✅ success	View run
Gate	✅ success	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

[agents-workflows-bot]

⚠️ Action Required: Unable to determine source issue for PR #1110. The PR title, branch name, or body must contain the issue number (e.g. #123, branch: issue-123, or the hidden marker ).

[Copilot]

Pull request overview

Removes a redundant sparse checkout step in the merge-sync workflow that could overwrite the workspace contents and cause required workflow files (e.g., maint-68-sync-consumer-repos.yml) to be missing during sync PR merge runs.

Changes:

• Deleted the “Checkout retry helpers” step that re-ran actions/checkout with sparse-checkout, which could wipe previously checked-out files.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[github-actions]

Automated Status Summary

Head SHA: 05e9f5b
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

Failure triage

Detected failure types: pytest.

• error_type: pytest
  root_cause: Pytest reported failing tests.
  suggested_fix: Inspect failing tests in the reported files and fix the regression or update expectations.
  playbook_url: docs/INTEGRATION_GUIDE.md#scenario-1-tests-failing

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

---

Keepalive checklist

Scope

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

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.
• ⚠️ Action Required: Unable to determine source issue for PR #1110. The PR title, branch name, or body must contain the issue number (e.g. #123, branch: issue-123, or the hidden marker ).

Related Issues/PRs

• #123
• #103
• #89
• #1110
• stranske/Workflows#103
• stranske/Workflows#89
• stranske/Workflows#123

References

• chore(codex): bootstrap PR for issue #101 #103
• chore(codex): bootstrap PR for issue #79 #89
• Validate CLI agent pipeline and streamline PR status display #123
• https://github.com/stranske/Workflows/compare/main...codex/issue-123?expand=1
• https://github.com/stranske/Workflows/actions/runs/21365041567
• https://github.com/stranske/Workflows/actions/runs/21364828142
• https://github.com/stranske/Workflows/actions/runs/21364828239
• https://github.com/stranske/Workflows/actions/runs/21364831738
• https://github.com/stranske/Workflows/actions/runs/21364828210
• https://github.com/stranske/Workflows/actions/runs/21364828238
• https://github.com/stranske/Workflows/actions/runs/21364828162
• https://github.com/stranske/Workflows/actions/runs/21364828159
• https://github.com/stranske/Workflows/actions/runs/21364828160
• https://github.com/stranske/Workflows/actions/runs/21364828157
• https://github.com/stranske/Workflows/actions/runs/21364828152
• https://github.com/stranske/Workflows/actions/runs/21364828140
• https://github.com/stranske/Workflows/actions/runs/21364828146
• https://github.com/stranske/Workflows/actions/runs/21364828148

Blockers & Dependencies

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

Tasks

• 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

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
• [ ]