Skip to content

chore(agents): install the zi-install skill - #216

Merged
ss-o merged 3 commits into
mainfrom
chore/zi-install-skill
Sep 19, 2026
Merged

ss-o merged 3 commits into
mainfrom
chore/zi-install-skill

Conversation

@ss-o

@ss-o ss-o commented Sep 19, 2026

Copy link
Copy Markdown
Member

Summary

Refs z-shell/.github#636

Verification

  • Installed with gh skill install z-shell/.github .github/skills/zi-install --pin bc2dae03d43216cca93f1f585cb06266c8262ca1 --dir .github/skills (re-pinned from 0105b04a, then from de88a852); the body is byte-identical to the canonical copy at that commit, and the provenance metadata carries the pinned revision.
  • Runtime discovery: the same skill, delivered into the private workspace from the same revision, appeared in the running agent's skill list; that is discovery evidence for the canonical copy, not for this delivery. Invocation against this repository is unverified.

Agent handoff

No handoff needed.

Pilot delivery of the organization zi-install skill at
.github/skills/zi-install/SKILL.md, pinned to z-shell/.github
0105b04ac91b1c4a124b0b2b13d1288fdea72332 per runbooks/org-review.md,
beside the code-review skill. Body byte-identical to the canonical copy;
metadata in the same shape as the existing delivery.

Refs z-shell/.github#636
Copilot AI balanced review requested due to automatic review settings September 19, 2026 04:06
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Sep 19, 2026

Copy link
Copy Markdown

Deploying src with  Cloudflare Pages  Cloudflare Pages

Latest commit: b79c17b
Status: ✅  Deploy successful!
Preview URL: https://6eb4e5a1.zi-src.pages.dev
Branch Preview URL: https://chore-zi-install-skill.zi-src.pages.dev

View logs

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.

Copilot review overview

🟡 Changes recommended

The loader verification can falsely report success without proving that zzinit ran.

Get a fresh assessment by requesting another Copilot review.

Review effort: Balanced
Findings: 1 Medium severity

Open (1)
What changed in this PR

Adds the pinned organization Zi installation skill to the repository that owns the installer.

Changes:

  • Adds provenance metadata for the canonical skill.
  • Documents installer profiles, checksum verification, failure handling, and post-install checks.
File Description
.github/​skills/​zi-install/​SKILL.md Adds the agent-facing Zi installation workflow.

💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread .github/skills/zi-install/SKILL.md Outdated
z-shell/.github#638 replaced the Loader verification with a positive
probe; pin the delivery to its squash commit de88a85250897123602f3586ca7861cac782bc4b.

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.

Copilot review overview

🟡 Changes recommended

Loader verification can succeed against a different checkout when custom installation paths are used.

Get a fresh assessment by requesting another Copilot review.

Review effort: Balanced
Findings: 1 Medium severity · 1 Low severity

Open (2)
Resolved since last review (1)

Comment thread .github/skills/zi-install/SKILL.md
Comment thread .github/skills/zi-install/SKILL.md Outdated
z-shell/.github#639 restricts explicit ZI_HOME overrides to -i skip until
the loader block carries them (src#217); pin the delivery to its squash
commit bc2dae03d43216cca93f1f585cb06266c8262ca1.

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.

Copilot review overview

🟢 Approval recommended

The previously identified loader-probe, custom-home, and provenance issues are resolved in the updated delivery.

Review effort: Balanced
Findings: None

Resolved since last review (2)

@ss-o
ss-o merged commit 3b6aa67 into main Sep 19, 2026
9 checks passed
@ss-o
ss-o deleted the chore/zi-install-skill branch September 19, 2026 05:20
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