Skip to content

[Fleet incident] Remove credential-first and broad secret inheritance from hourly loop #37

Description

@seonghobae

Fleet automation incident

Protected main currently exposes two reviewable automation defects in .github/workflows/hourly-commercialization-loop.yml.

1. Product-development eligibility checks NVIDIA before the deterministic PR queue

develop-next-product-gap rejects the run for missing NVIDIA_NIM_API_KEY before it checks whether an open pull request already owns the queue. Deterministic queue/no-op decisions must work without model credentials.

Required ordering:

open-PR / deterministic product gates / dry-run -> NVIDIA credential -> actual NVIDIA-backed OpenCode path

Do not remove the NVIDIA credential from actual inference. Move it to the boundary where inference can genuinely begin.

2. Reusable governance jobs forward all repository/organization secrets

inspect-pr-queue, repair-review-feedback, and revalidate-pr-queue use secrets: inherit when calling the central .github reusable schedulers. This forwards unrelated repository/organization secrets to shared governance jobs and makes the caller dependent on ambient secret inventory. Replace broad inheritance with no secret forwarding where the central workflow's GitHub/OIDC authority is sufficient, or with the minimum explicit named secret mapping if live central contract evidence proves one is genuinely required. Do not invent a new PAT or rename an existing secret to preserve ambient access.

Also re-pin caller SHAs to the exact accepted central protected-main implementation after central prerequisites settle; do not assume historical reusable-workflow SHAs remain the supported contract.

Acceptance criteria

  • RED→GREEN contract proving open-PR and other deterministic no-op gates execute and terminate before model credential validation;
  • dry-run remains credential-independent;
  • no secrets: inherit on central review/fix/merge calls unless a documented, tested, minimum explicit mapping is proven necessary;
  • no COPILOT_GITHUB_TOKEN, guessed PAT, self-modifying repair workflow, or weakened protection/check policy;
  • exact-head CI/security/review evidence;
  • normal protected-main merge;
  • real protected-main scheduled/manual loop run verifies PR maintenance, review-fix and revalidation actually execute downstream, and product development either stops for the correct deterministic reason or reaches the supported NVIDIA boundary.

This is part of the organization-wide GitHub Actions fleet incident. Re-fetch exact current central and RankWeave heads before implementation; no identities in this issue are permanent evidence.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions