Skip to content

ci: host-always gate on all branches; prep metal plane (#101) - #103

Open
tig wants to merge 3 commits into
mainfrom
fix/issue-101-host-always-ci
Open

tig wants to merge 3 commits into
mainfrom
fix/issue-101-host-always-ci

Conversation

@tig

@tig tig commented Jul 26, 2026

Copy link
Copy Markdown
Owner

Summary

First slice of #101: host always, ready for self-hosted metal later.

  • Host-gate on every branch (push + PR + workflow_dispatch) for silico package CI and both GCU plates — lab/session/* work no longer needs firmware on main just so Actions runs.
  • Metal stub job on plates: skipped unless repo var SILICO_METAL_CI=true; then runs on [self-hosted, silico-metal] with concurrency lock and a fail-closed “wire me” step (see knowledge).
  • Codify the contract in silico/knowledge/ci-host-metal.md (INDEX/README linked).
  • AGENTS practice GCUs + Stage E: dual-surface direction, host-always claims honesty, no “push to main for CI.”
  • silicov1 beta note points at host-always / metal-later.

Not in this PR: silico practice CLI, force-restore of dirty practice remotes, or wired metal steps.

Related observer spike: #102 (eyes/ears bench fixture for xuss-class product face).

Test plan

  • Workflow YAML validates structurally (host job unchanged in substance; triggers widened)
  • After merge: confirm Actions runs on a non-main branch push on silico (and on a plate-scaffolded GCU after next scaffold)
  • Confirm metal-bench does not appear when SILICO_METAL_CI is unset
  • Skim AGENTS practice-GCU section for accidental “always main for CI” leftovers

Plate and spine workflows run host-gate on every push/PR/dispatch so lab branches get honest host CI without polluting practice main. Document the two-plane model (host always, self-hosted metal opt-in via SILICO_METAL_CI stub). Align AGENTS practice-GCU + Stage E claims with dual-surface prep.
@cursor

cursor Bot commented Jul 26, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 1e64c7f8a5

ℹ️ 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".

Comment thread AGENTS.md
Comment thread silico/plates/gcu/.github/workflows/ci.yml Outdated
tig added 2 commits July 29, 2026 08:39
Sync scaffolded gcu AGENTS repo-workflow with lab-branch / host-always
guidance so practice GCUs no longer default to public main. Guard
metal-bench with same-repo pull_request check on both plates; document
the YAML contract in ci-host-metal.md.
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.

1 participant