fix(ci): repair workflow gates and reconcile remaining issue status - #396
Merged
Merged
Conversation
Co-authored-by: arena-agent <297053741+arena-agent@users.noreply.github.com>
arena-ai-coding-agent
Bot
requested a review
from hyperpolymath
as a code owner
September 25, 2026 20:19
|
Important Review skippedBot user detected. To trigger a single review, invoke the ⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Advanced Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Co-authored-by: arena-agent <297053741+arena-agent@users.noreply.github.com>
hyperpolymath
approved these changes
Sep 25, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Review all 19 remaining issues without falsely closing roadmap or upstream-blocked work.
with:mappings that GitHub rejects before creating jobs.actions-lockmanaged files.docs/reports/2026-09-25-issue-maintenance.adoc.Closes #216 (canonical owner header already on main).
Closes #175 (completed informational audit following #166).
Closes #196 (local cancel-token preemption already implemented by #154; benchmark and transport remain #161/#194).
Closes #240 (legacy source removed by #362; obsolete exemptions removed here; real UI compilation remains #117/#266).
Refs #239, #252, #314, #310. These remain open: classification is not expert review, re-arming is not a completed security burn-down, and this connection cannot rotate the owner-selected secret.
Validation
bash -n scripts/ci/*.sh— passbash scripts/ci/test-issue-regressions.sh— pass, 34 workflows checked plus positive/negative fixturesgit diff --check— passuses:reference, Actions lock, proof, licence, secret, baseline severity, or gate strictness changedKnown limits / pre-existing red checks
Main run 36150127750 already fails baseline validation with 62 unfiltered findings at
info. The baseline was NOT absent/skipped; it must not be broadened without evidence. The latest Security Scan has a pre-existing startup failure. Other main workflows also have failures; this PR does not claim to resolve all infrastructure.This sandbox cannot download Debian toolchain packages or GitHub release assets, so no fresh Hypatia scan or proof compilation is claimed. The integration returns 403 for Actions secrets/code-scanning alerts and cannot download Actions log blobs (job/annotation metadata is readable). Owner's #310 decision was rotate, not delete; no credential change made.
Merge blocker discovered on this PR
All GitHub Actions runs are rejected by actor policy, before creating any jobs:
Evidence: https://github.com/hyperpolymath/echidna/actions/runs/36185037867 (actor
arena-ai-coding-agent[bot]). The main ruleset requiresscan / gitleaks,scan / rust-secrets, andscan / shell-secrets. An authorized maintainer must permit this actor or trigger validation as an allowed actor. These checks are not being bypassed. Local test success must not be confused with CI success.Current disposition
Auto-merge (squash) is enabled, but the PR is not merged while required Actions checks cannot run. Attempts to close the three already-resolved issues #216, #175, and #196 were also rejected with
Resource not accessible by integration(confirmed via REST HTTP 403). No issue was closed; all 19 remain open. The closure keywords above are intended to close the four evidenced resolutions upon merge. Remaining roadmap, proof, credential and upstream work stays open intentionally.The GitHub connection needs Issues write permission to manage closures, and Actions policy must allow this actor (or validation must be run by an authorized maintainer). No administrative bypass was attempted.