Repository navigation
docs: add the change-maple-ci skill and align CI docs - #1344
Merged
Merged
Conversation
A first read for agents changing, triggering or debugging CI: the hard rules with their reasons, how plan, lanes and ci-ok fit together, what runs on PRs, master, nightly and dispatch, the ci:* labels and re-run semantics, how to add a lane, local validation, common failures, and the release boundary. References hold the design reasoning, the lane and guard-test detail, and a debugging guide. lanes.py and plan.py explain stay the source of truth for volatile lists. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- docs/ci.md: name the advisory PR workflows, the merge model (no queue, no up-to-date rule), master's push lane set, nightly times, dispatch limits, Re-run all jobs for labels, explain's limits, local lane prerequisites, the 10-minute rule, FlakeHub writes, and one runner table that owns lane runners. - docs/repository-workflows.md: MAPLE_HOOK_NO_NIX, FlakeHub writes from same-repository EIF builds, the EIF builder class and fork job, the Pages producer pins, and a runner table for the signed workflows; drop the duplicated updater-lane definition. - AGENTS.md: route shared CI to $change-maple-ci and keep the CI rules short. - Fix stale claims: Agent packaging rehearsals are label-only on PRs, the macOS archive scan runs on signed master builds, master no longer warms the fork EIF cache, Rust SDK and proxy routing for Research, the Windows bundle inventory, the store benchmark is not in CI, the provider suite runs on three OSes only nightly, proxy container builds on PRs, Research Windows PR staging, and os-eif in the OpenSecret skills. - Replace duplicated lane summaries in component docs and skills with pointers, and use just lane repo for the root flake checks. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Keep the docs PR free of build inputs: restore the store-bench and Research Windows resource READMEs, and the Auth and Research command lines that were already correct, so merging starts no signed build or Auth deployment. Skill: typical PR selections, both pr_paths=() and gate=False for slow lanes, re-run and auto-merge mechanics, failover runners, the agent-desktop-build dispatch command, hook wiring for existing components, the merge-base push preview, narrower runner, cache and evidence claims, and no stale flake rows. Docs: os-eif reports approval need, explain applies the approval-edit rule, PR cache exceptions, proxy and Rust SDK routing, the Research Dev re-run advice, and develop-maple routes to change-maple-ci. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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.
Adds the
$change-maple-ciskill so a new agent learns Maple's CI in one read before changing, triggering or debugging it. It also brings the existing docs and skills in line with the CI as it works today, after #1315, #1337, #1339, #1341 and #1342.The skill (
.agents/skills/change-maple-ci/)ci-okstays well under 10 minutes;lanes.py, with a test;[skip ci];plan, then the lanes, thenci-ok, plus a table of what a typical PR runs, and what runs on a PR, a master push, nightly and on dispatch.ci:*label adds, re-run semantics (labels are read whenplanruns), the dispatch traps, and how to get Agent Dev and other master-only builds.$release-maple.references/:design.md(why it's built this way),changing-ci.mdanddebugging.md.Tested on a fresh agent. It read only the skill and answered 18 practical questions, such as "how do I get the Windows installer smoke on my PR?" and "the runner lost communication, what now?". 16 answers were right first time and 2 partly right. The gaps it found are fixed, and every claim was audited against the code.
Docs fixed:
AGENTS.md(routes CI work to the skill),docs/ci.md,docs/repository-workflows.md, the Agent docs, the Research Dev desktop doc,proxy/README.md, the OpenSecret EIF docs, and thedevelop-maple,develop-maple-agent,validate-*andreview-opensecret-securityskills. The fixes cover stale statements from before the recent CI changes, and duplicated CI text that now points to the skill.Safe to merge:
plan.py explainselects no lane.Left for a follow-up code PR: a few stale comments inside
ci.yml,lanes.py,codeql.yml,agent-desktop-build.yml,flake.nixandsetup-hooks.sh.Validation passes:
plan.py lint, the scripts/ci suite (514 tests, including the docs-reference contract C7), the root flake checks, and a link check over every changed Markdown file.🤖 Generated with Claude Code
Need help on this PR? Tag
@codesmith-botwith what you need.