Skip to content

feat: add admin Harness fleet protocol - #24

Open
HaodiFan wants to merge 3 commits into
mainfrom
codex/dev/20260729-skill-harness-upgrade-all
Open

feat: add admin Harness fleet protocol#24
HaodiFan wants to merge 3 commits into
mainfrom
codex/dev/20260729-skill-harness-upgrade-all

Conversation

@HaodiFan

Copy link
Copy Markdown
Contributor

Instruction surface changed

Root SKILL.md now declares EvoZeus Harness fleet maintenance as a first-class trigger and execution protocol. The product CLI adds read-only planning and explicit admin publication.

Why this change is needed

Historical Skills can keep outdated Harness copies, while users need one EvoZeus command to inspect and align all registered repositories without manually editing each Skill.

Agent behavior before / after

Before: root Skill only contained one routing sentence and did not define channel, permission, isolation, failure, or audit behavior.

After: root Skill requires read-only planning first, Stable-only Harness sources, live ADMIN checks, isolated worktrees, per-repo PRs, partial failure isolation, ledger output, and separate authorization for merge or release.

Safety boundary

  • file access: read registry and manifests; write only isolated target worktrees during explicit publication
  • network access: live GitHub permission checks and PR publication only after explicit authorization
  • shell/tool usage: EvoZeus CLI delegates to the installed CoEvolve backend
  • PR creation: exact ADMIN permission per target repository
  • upload/export: no raw session or customer data
  • user confirmation points: publish, merge, and Skill Release are separate authorizations

Prompt injection risk

Target repository content cannot grant publication authority. GitHub viewerPermission and the current user request are authoritative.

Rollback plan

Revert the three focused commits and switch users back to Stable. Target PRs remain independently reviewable and are never auto-merged.

Versioning

  • version bump: pending formal release
  • pinned URL: Stable Product Manifest
  • hash: resolved from the installed Stable channel

Review State

Next action: review EvoZeus-CoEvolve PR #31, then this PR.

Open reviewer questions: none.

Supersedes closed PR #23 after correcting the branch naming contract.

@vercel

vercel Bot commented Jul 29, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
evozeus-metainflow Ready Ready Preview, Comment Jul 29, 2026 2:07pm

@github-actions github-actions Bot added risk:agent-behavior May change future agent behavior risk:governance Changes governance rules risk:skill-entry Touches SKILL.md or scenario skills size:M Medium PR type:code Runtime, CLI, script, or behavior change type:docs Documentation or example change type:governance Governance, template, branch, or maintainer process change type:skill-instruction Agent-facing instruction or skill change proof:needed Real behavior proof is missing or incomplete labels Jul 29, 2026
@github-actions

Copy link
Copy Markdown

EvoZeus Labeler Report

PR type/risk labels were updated from changed files.

Surfaces

  • code
  • docs
  • governance
  • skill

Labels

  • risk:agent-behavior
  • risk:governance
  • risk:skill-entry
  • size:M
  • type:code
  • type:docs
  • type:governance
  • type:skill-instruction

Changed lines

  • 364

@github-actions

Copy link
Copy Markdown

EvoZeus Proof Gate

Mode: dry-run

Proof labels

  • proof:needed

Missing or weak fields

  • missing code/workflow section: Problem
  • missing code/workflow section: Scope
  • missing code/workflow section: Real behavior proof
  • missing code/workflow section: Tests
  • governance change needs Linked RFC or maintainer discussion
  • real behavior proof is missing

Next action

  • Add real behavior proof, not only mocks/lint/CI.

@github-actions

Copy link
Copy Markdown

EvoZeus Auto Response

Missing real behavior proof

This PR needs real behavior proof before review can continue. Add the command, environment, input, output after the patch, observed result, and what was not tested.

@github-actions

Copy link
Copy Markdown

EvoZeus Privacy Scan

Mode: dry-run

Findings

  • none

Next action

  • No simple privacy pattern matched in the PR diff.

@github-actions github-actions Bot added triage:dirty-pr PR mixes too many surfaces or should be split triage:owner-only Owner review required triage:rfc-needed RFC required before review labels Jul 29, 2026
@github-actions

Copy link
Copy Markdown

EvoZeus Dirty PR Check

Mode: dry-run

Surfaces

  • code
  • docs
  • governance
  • skill

Reasons

  • changes 4 surfaces: code, docs, governance, skill
  • protected path changed
  • governance/workflow change has no RFC or maintainer discussion marker

Next action

  • Split the PR, add RFC context, or wait for owner review.

@github-actions

Copy link
Copy Markdown

EvoZeus Queue Guard

Mode: dry-run

Author open PRs: 2

Branch family: codex/dev/20260729-skill-harness-upgrade-all

Branch family open PRs: 1

Violations

  • none

Next action

  • Queue limits are currently OK.

@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: 8a11307f7b

ℹ️ 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 scripts/evozeus-cli.mjs
Comment on lines +1373 to +1375
channel: "development",
version: coevolveHarnessVersionAt(readiness.detected_path, operation),
root: readiness.detected_path

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Block publication without a Stable Harness source

When channel-state.json has no Stable entry, this fallback labels the resolved environment or sibling checkout as development, and harnessUpgradeAll still passes --publish to it. Thus EVOZEUS_WRAPPER_ROOT can point at an arbitrary development tree and harness upgrade-all --publish will create fleet PRs from that tree, despite SKILL.md requiring every published update to come from installed, verified Stable and forbidding distribution of development files. Permit this fallback only for dry-run testing, and reject publication unless the source channel is Stable.

Useful? React with 👍 / 👎.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

proof:needed Real behavior proof is missing or incomplete risk:agent-behavior May change future agent behavior risk:governance Changes governance rules risk:skill-entry Touches SKILL.md or scenario skills size:M Medium PR triage:dirty-pr PR mixes too many surfaces or should be split triage:owner-only Owner review required triage:rfc-needed RFC required before review type:code Runtime, CLI, script, or behavior change type:docs Documentation or example change type:governance Governance, template, branch, or maintainer process change type:skill-instruction Agent-facing instruction or skill change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant