Skip to content

fix(sync): keep legacy orchestrator protected - #2162

Closed
stranske wants to merge 1 commit into
mainfrom
sync-review-1836-guard-orchestrator
Closed

fix(sync): keep legacy orchestrator protected#2162
stranske wants to merge 1 commit into
mainfrom
sync-review-1836-guard-orchestrator

Conversation

@stranske

@stranske stranske commented May 25, 2026

Copy link
Copy Markdown
Owner

Source: Issue #1836

Related to campaign issue #1836

Automated Status Summary

Scope

Sync/Dependabot Campaign Queue

Durable tracker — see docs/ops/DURABLE_TRACKING_ISSUES.md. The body below is regenerated each cycle by maint-82-sync-dependabot-campaign.yml; do not close as part of routine triage.

Remote discovery found more review-thread work than fits in a full GitHub issue body. The marker below retains the compact machine-readable queue for the local watcher.

Context for Agent

Related Issues/PRs

Tasks

  • Updated: 2026-05-25T07:55:55.157Z
  • Current sync hash: f344ec227f07
  • Repos checked: 12/12
  • Open sync PRs: 36
  • Open Dependabot PRs: 0
  • Active review threads queued: 27
  • Items needing local Codex: 8
  • Actionable local Codex items: 8
  • Claimable local Codex items: 8
  • Source-fixed candidates: 0
  • Superseded sync candidates: 9
  • Source sync states: current=9, superseded=9
  • Finished local results without published source changes: 1
  • Claimed local Codex items: 0
  • Next claim lease expires: -

Acceptance criteria

  • Acceptance criteria section missing from source issue.

Head SHA: 56669d6
Latest Runs: ✅ success — Gate
Required: gate: ✅ success

Workflow / Job Result Logs
Auto-label Dependabot PRs ⏭️ skipped View run
Copilot code review ✅ success View run
Gate ✅ success View run
Health 40 Sweep ❔ startup failure View run
Health 44 Gate Branch Protection ❔ in progress View run
Health 45 Agents Guard ✅ success View run
Health 50 Security Scan ✅ success View run
Health 72 Template Sync ✅ 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

Copilot AI review requested due to automatic review settings May 25, 2026 08:04
@stranske
stranske temporarily deployed to agent-standard May 25, 2026 08:04 — with GitHub Actions Inactive

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR tightens the agents-guard removal allowlist so the legacy unnumbered orchestrator workflow (.github/workflows/agents-orchestrator.yml) remains protected (removal is blocked), while still allowlisting cleanup of the numbered legacy orchestrator (agents-70-orchestrator.yml) for template-sync bootstrap scenarios. It also adds regression coverage in both the Node and Python test suites to prevent this policy from drifting again.

Changes:

  • Remove .github/workflows/agents-orchestrator.yml from the consumer-only removal allowlist in agents-guard.
  • Keep .github/workflows/agents-70-orchestrator.yml allowlisted for consumer repos (but still blocked in stranske/Workflows).
  • Add JS + Python regression tests asserting unnumbered orchestrator deletions are blocked and the numbered orchestrator remains allowlisted for consumer repos.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
tests/workflows/test_agents_guard.py Updates deletion-blocking fixtures to a non-allowlisted workflow and adds a Python regression test that deletion of agents-orchestrator.yml is blocked in consumer repos.
templates/consumer-repo/.github/scripts/agents-guard.js Removes agents-orchestrator.yml from the consumer-only removal allowlist (template copy).
.github/scripts/agents-guard.js Removes agents-orchestrator.yml from the consumer-only removal allowlist (source-of-truth copy).
.github/scripts/__tests__/agents-guard.test.js Adjusts allowlist coverage and adds a JS regression test that agents-orchestrator.yml removals are blocked in consumer repos.

@stranske-keepalive

Copy link
Copy Markdown
Contributor

Automated Status Summary

Head SHA: acafe86
Latest Runs: ⏳ pending — Gate
Required contexts: Gate / gate, Health 45 Agents Guard / guard
Required: core tests (3.12): ⏳ pending, core tests (3.13): ⏳ 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

No scope information available

Tasks

  • No tasks defined

Acceptance criteria

  • No acceptance criteria defined

@stranske

Copy link
Copy Markdown
Owner Author

Closing as superseded by #2163. The later source-fix pass and the current campaign tracker settled on retiring/removing the legacy consumer agents-orchestrator.yml template rather than protecting it from sync removal. #2163 has passing checks and carries that source-of-truth update.

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.

2 participants