Compare commits

...

12 Commits

Author SHA1 Message Date
joakimp 47b450694c release: v1.13.1 — pi-extensions 7cb808e (aws-sso ignores a Bedrock tier with a non-Bedrock task fallback)
Lint / hadolint (push) Successful in 13s
Lint / doc-drift (push) Successful in 13s
Lint / skill-floor (push) Successful in 14s
Lint / actionlint (push) Successful in 19s
Lint / skill-frontmatter (push) Successful in 21s
Publish Docker Image / lint-gate (push) Successful in 33s
Publish Docker Image / resolve-versions (push) Successful in 11s
Publish Docker Image / base-decide (push) Successful in 12s
Publish Docker Image / build-base (push) Has been skipped
Publish Docker Image / smoke-studio (push) Successful in 5m26s
Publish Docker Image / build-variant-studio (push) Successful in 19m40s
Publish Docker Image / smoke (push) Successful in 10m45s
Publish Docker Image / build-variant (push) Successful in 20m26s
Publish Docker Image / promote-base-latest (push) Successful in 6s
Publish Docker Image / update-description (push) Successful in 9s
2026-10-25 15:33:46 +01:00
joakimp 7737156aad release: v1.13.0 — CHANGELOG heading, components table, BASE_REBUILD_DATE
Lint / hadolint (push) Successful in 10s
Lint / skill-floor (push) Successful in 11s
Lint / doc-drift (push) Successful in 17s
Lint / actionlint (push) Successful in 34s
Lint / skill-frontmatter (push) Successful in 15s
Publish Docker Image / lint-gate (push) Successful in 26s
Publish Docker Image / resolve-versions (push) Successful in 14s
Publish Docker Image / base-decide (push) Successful in 15s
Publish Docker Image / build-base (push) Successful in 44m19s
Publish Docker Image / smoke (push) Successful in 5m26s
Publish Docker Image / smoke-studio (push) Successful in 11m12s
Publish Docker Image / build-variant (push) Successful in 19m25s
Publish Docker Image / update-description (push) Successful in 15s
Publish Docker Image / promote-base-latest (push) Successful in 17s
Publish Docker Image / build-variant-studio (push) Successful in 17m51s
2026-10-25 13:45:18 +01:00
joakimp 644be67ea2 docs(changelog): aws-sso keeper idle without Bedrock (pi-extensions f032195)
Lint / hadolint (push) Successful in 10s
Lint / skill-floor (push) Successful in 13s
Lint / doc-drift (push) Successful in 15s
Lint / actionlint (push) Successful in 23s
Lint / skill-frontmatter (push) Successful in 13s
2026-10-25 13:29:29 +01:00
joakimp f7e35962d6 docs(changelog): mailbox owed-list fix from mempalace-toolkit c04d488
Lint / hadolint (push) Successful in 10s
Lint / skill-floor (push) Successful in 13s
Lint / doc-drift (push) Successful in 14s
Lint / skill-frontmatter (push) Successful in 18s
Lint / actionlint (push) Successful in 32s
2026-10-25 13:10:00 +01:00
joakimp b06941d714 firstboot skill: say what to do when a command gate blocks a probe
Lint / skill-floor (push) Successful in 10s
Lint / hadolint (push) Successful in 16s
Lint / doc-drift (push) Successful in 16s
Lint / actionlint (push) Successful in 20s
Lint / skill-frontmatter (push) Successful in 21s
erdc-guard (fleet extension, not in the image) judges the text of every
bash statement. Measured with its own assess(): ERDC_GUARD=off on a
child pi does not help (the parent blocks the prompt text, 0.95); pi -p
@file is not flagged; cat heredocs are flagged depending on contents;
/guard off is volume-wide and keeps confirm-destructive stood down.
Order: write tool, @file, /tmp scratch, /guard off last. A blocked probe
is a cannot-run.
2026-10-24 16:16:10 +02:00
joakimp 00345d3536 changelog: confirm-destructive steps aside for another bash gate (pi-extensions 7237855)
Lint / hadolint (push) Successful in 15s
Lint / skill-floor (push) Successful in 14s
Lint / doc-drift (push) Successful in 17s
Lint / actionlint (push) Successful in 21s
Lint / skill-frontmatter (push) Successful in 13s
The next image bakes pi-extensions main, which now has 7237855. With a second
command gate loaded (the fleet's erdc-guard), confirm-destructive still asked
about every command that gate let through, with no time limit. It now skips its
bash check while another extension holds the claim
Symbol.for("pi-extensions.bash-gate"). Its /new, /resume and /fork questions
are unchanged, and with no claim it behaves as before.

check-doc-drift.sh check 9 (every moved component named above the last release
heading) passes: pi-extensions 4c1d28d -> 7237855.
2026-10-24 15:39:10 +02:00
pi@mbp-m1-2020 057827a84b skills: trim two descriptions to a routing sentence, and assert their pointers
Lint / hadolint (push) Successful in 10s
Lint / skill-floor (push) Successful in 14s
Lint / doc-drift (push) Successful in 18s
Lint / actionlint (push) Successful in 32s
Lint / skill-frontmatter (push) Successful in 15s
Every skill's name and description sit in the system prompt of EVERY request --
that block is the routing index -- while bodies are fetched on demand through
the read tool. Measured on this device: 24 discoverable skills cost 10,609
description chars (~2,652 tokens) per request, against 474,656 chars of bodies
(~119k tokens) that cost nothing until used. Bodies are 45x the descriptions,
which is what the spec's 1024-char description limit is protecting.

credential-incident-response (1022, two characters under the limit) and
pi-devbox-environment (923) were each carrying a table of contents of what
their body covers. Trimmed to what a description is actually for -- answering
"should I load this?" -- reclaiming 1,087 chars (~271 tokens) of permanent
context. 4,154 -> 3,067 across the five baked skills.

WHY ONLY THESE TWO. Both are named BY PATH from the always-loaded managed
block, so neither depends on description-matching to be found. pi-extensions is
left alone deliberately: check-skill-floor.sh compares the vendored copy to the
package repo by DIRECTORY HASH, so trimming ours alone registers as drift and
turns that gate red; and it is the one skill that also ships in the skillset
repo, where it runs on hosts with no pi-devbox managed block and genuinely needs
its description. pi-devbox-firstboot stays fuller than needed because it is the
skill a device with nothing else has to find.

NOTHING WAS DELETED, ONLY RELOCATED -- and that was checked rather than assumed:
all 18 enumerated items were confirmed present in the bodies BEFORE cutting
(frontmatter stripped, so the description's own line could not score its own
claim), and `git diff --stat` confirms both bodies are byte-identical.

THE TRIM CREATES A DEPENDENCY, SO IT IS NOW ASSERTED. smoke-test.sh checks both
pointers by path, next to the firstboot one. The premise of the cut is "the
block names them", which holds only while those lines exist -- reword the block,
drop a pointer, and the skill silently stops being reachable for a device that
did not already know it was there. Verified discriminating: a reworded block
fails the grep.

An instrument failure caught on the way, and the reason the coverage check was
re-run: my first pass used `grep -E 'a\|b'`, where \| is a LITERAL pipe in ERE,
so 8 of 18 items read as 0 hits. Every zero was a pattern containing \| and
every non-zero lacked one -- that correlation, not the content, was the finding.
With real alternation all 18 are present.

Loader re-verified on both trimmed skills: rc=0, no skill diagnostics.
2026-10-24 11:26:26 +02:00
pi@mbp-m1-2020 206fd1ec35 ci: gate baked skill frontmatter against the loader's own limits
Lint / skill-floor (push) Successful in 9s
Lint / hadolint (push) Successful in 10s
Lint / doc-drift (push) Successful in 18s
Lint / skill-frontmatter (push) Successful in 19s
Lint / actionlint (push) Successful in 32s
Found while adding pi-devbox-firstboot, whose description was drafted at 1056
characters against pi's MAX_DESCRIPTION_LENGTH of 1024. pi does not reject it:
dist/core/skills.js pushes a diagnostic and then loads the skill anyway --
"Still load the skill even with warnings, unless description is missing or
empty". So the skill works, the violation is real, and the only thing that
would ever report it is a diagnostic inside a live pi session. Nothing in CI
starts one, so the warning is invisible here by construction. A spec violation
that still works is the most durable kind of rot: nothing forces the fix.

Unlike check-skill-floor.sh, this was NOT free to add. Measured headroom on the
five baked skills: credential-incident-response 1022 (2 chars spare),
pi-extensions 1003 (21), pi-devbox-firstboot 952, pi-devbox-environment 923,
mempalace 254. Two of five are within one sentence of the limit, so "nobody
would write a description that long" is already false here.

It also catches the sibling failure that is a silent DROP rather than a
warning: a missing or empty description returns skill: null, so the file ships,
the symlink resolves, smoke-test's `test -f` passes, pi-devbox-version lists it
-- and no agent can ever see it. Name rules are checked too, including against
the directory, because the loader falls back to parentDirName and a mismatch
silently renames the skill while every path-based pointer (the managed AGENTS
block included) keeps naming the directory.

Truth table, each mutation verified to differ materially from the shipped tree:
shipped 0; over-long description 1; missing description 1 (reported as fatal,
not a mere violation); name != directory 1; no frontmatter 1; directory with no
SKILL.md 1; absent skills tree 2. Exit codes match lint-shell.sh,
check-skill-floor.sh and check-doc-drift.sh -- a gate that cannot run must not
pass.

python3-yaml rather than grep/awk because `description: >-` is a folded block
scalar whose value spans lines and joins with spaces, so a line-oriented length
check measures the wrong string. Same package the actionlint job already
installs, same parser check-doc-drift.sh already uses.

The vendored copies (pi-extensions, mempalace) are deliberately in scope: an
upstream edit over the limit turns this red on a file this repo does not author,
which is precedented by check-skill-floor.sh's coupling and correct because the
image ships those bytes. The fix in that case is to refresh the snapshot, not to
exempt the file.
2026-10-24 00:32:13 +02:00
pi@mbp-m1-2020 5d6ff537c4 skills: add pi-devbox-firstboot, and point the managed AGENTS block at it
The fleet's first-boot acceptance round is now scoped by published state
(policy v2, artifact art_20261023T204306_b388e93c3d64): a booting container
reads two claim keys -- firstboot-release-<TAG> and
firstboot-image-<TAG>-<ARCH> -- and derives whether it owes the release tier,
the per-architecture image tier, or only the device tier.

Nothing delivered that procedure to a device that does not already have a
coordination chain. The gate deliberately needs no roster, because fleet
membership changes in both directions, so no message addresses a new device and
nothing enrolls it. The consequence is that a BRAND-NEW device -- empty
mailbox, no chain, and the mempalace skill correctly telling it that an empty
mailbox is a fine answer -- had no path to the round at all. Historically a
peer planted the newcomer's first ask cross-device, which is exactly the roster
dependency the state design removes. This skill is how a device learns to plant
its own.

The skill is a FINDER, not the policy: it locates the newest policy.update on
topic=firstboot-acceptance and fetches the artifact, and says plainly that the
artifact wins on disagreement. A policy baked into an image is pinned to that
image; keeping only the lookup here means a revision needs no release and a
device on an older image still reaches the current policy.

Baked canonical, like pi-devbox-environment, rather than skillset-owned. The
round verifies the image, so pinning it to the image is right -- and a
skillset-owned skill needs the vendoring machinery (ARG SKILLSET_SNAPSHOT_REF,
the manifest/label pair, a per-skill refresh script) that today exists only for
mempalace. No wiring was needed: entrypoint-user.sh and pi-devbox-version both
enumerate the baked skills directory.

smoke-test.sh asserts the AGENTS.md pointer BY PATH, not by prose. A new device
reaches the round only through that line, so rewording the paragraph must not
be able to cut it silently. Both skill-resolution loops gain the new name.
2026-10-24 00:31:15 +02:00
joakimp 779efbf4fe ci: pin the smoke image with a running container during the test
Lint / hadolint (push) Successful in 12s
Lint / skill-floor (push) Successful in 13s
Lint / doc-drift (push) Successful in 17s
Lint / actionlint (push) Successful in 26s
The label only protects against reclaims that honour it. The runner
hosts also run jobs we do not control: runner-2 has a second act_runner
registered with gitea.egl.lan on the same Docker daemon, and
opencode-devbox's workflows run raw `docker system prune -af` in 7
places. No prune removes an image a running container uses, so the
smoke step now starts a holder (`ci-reclaim-docker.sh hold`: --rm,
--network none, sleep 3600) right after the preflight, and the
always() cleanup step runs `release` (holder + image). Unprotected now:
only the seconds between the build step's load and the holder starting.

Verified on an isolated docker:29-dind: an unlabelled held image survives
raw system prune and image prune -af (control without holder: deleted);
hold fails on a missing image; holder is AutoRemove + network none;
release removes holder and image; reclaim behaviour unchanged.
2026-10-23 12:46:39 +02:00
joakimp e0611c787f ci docs: the shared daemon is per runner (runner-a1 runs several jobs at once)
Lint / skill-floor (push) Successful in 11s
Lint / hadolint (push) Successful in 12s
Lint / doc-drift (push) Successful in 13s
Lint / actionlint (push) Successful in 32s
Task logs of run 740: runner-a1 received task 2721 (smoke-studio) at
22:53:02Z and task 2722 (build-variant) at 22:58:38Z while 2721 was still
running. Other runners (runner-1/2/3) have capacity 1. 12e66aa said all
jobs share one daemon; corrected in CHANGELOG, AGENTS.md, the script
header and the lint.yml comment. No behaviour change.
2026-10-23 11:43:48 +02:00
joakimp 12e66aa52e ci: stop disk reclaim from deleting a smoke image mid-test
Lint / skill-floor (push) Successful in 9s
Lint / hadolint (push) Successful in 15s
Lint / actionlint (push) Successful in 22s
Lint / doc-drift (push) Successful in 12s
v1.12.2 run 740: build-variant's `docker system prune -af --volumes`
untagged pi-devbox:smoke-studio 30 ms after smoke-studio loaded it, so
every studio assertion failed and the -studio tags were not published.
All jobs share the runner's Docker daemon, and a smoke image is unused
between smoke-test.sh's one-shot `docker run`s.

- scripts/ci-reclaim-docker.sh: the same reclaim, per object type, but
  never prunes an image labelled se.jordbo.pi-devbox.ci-smoke-image;
  labelled orphans are swept after 6h. `assert-protected IMG` is the
  smoke preflight; `lint WORKFLOW...` rejects raw system/image prunes.
- smoke jobs: label their image, assert it, tag it per run
  (pi-devbox:smoke[-studio]-<run_id>), remove it in an always() step.
- all five reclaim sites call the script; lint.yml runs the guard.
- Job order unchanged: studio jobs still gate only the -studio tags.

Verified on an isolated docker:29-dind (old prune deletes a labelled
image; new reclaim keeps it, removes unlabelled + 1970-created labelled
images, spares an in-use image, no errors); actionlint, shellcheck,
lint-shell, check-workflow-shell, check-doc-drift all clean.
2026-10-23 08:34:39 +02:00
14 changed files with 1031 additions and 33 deletions
+49 -14
View File
@@ -90,6 +90,12 @@ env:
IMAGE: ${{ vars.DOCKERHUB_USERNAME }}/pi-devbox
RELEASE_TAG: ${{ github.ref_type == 'tag' && github.ref_name || inputs.release_tag }}
PROMOTE_LATEST: ${{ github.ref_type == 'tag' && 'true' || inputs.promote_latest }}
# Label that protects a smoke job's loaded image from the disk reclaim every
# job runs on the shared runner daemon (scripts/ci-reclaim-docker.sh says
# why: v1.12.2 run 740 lost pi-devbox:smoke-studio to build-variant's prune
# 30 ms after loading it). Set once here; the smoke builds apply it, the
# reclaim script honours it, the smoke preflight asserts it.
SMOKE_IMAGE_LABEL: se.jordbo.pi-devbox.ci-smoke-image
jobs:
# ── Phase 1: decide whether base needs rebuilding ──────────────────
@@ -538,8 +544,7 @@ jobs:
/usr/lib/jvm 2>/dev/null || true
apt-get clean || true
rm -rf /var/lib/apt/lists/* /tmp/* /var/tmp/* || true
docker system prune -af --volumes || true
docker builder prune -af || true
bash scripts/ci-reclaim-docker.sh
df -h / || true
- name: Set up QEMU
uses: docker/setup-qemu-action@v3
@@ -597,6 +602,10 @@ jobs:
needs.resolve-versions.result == 'success' &&
(needs.build-base.result == 'success' || needs.build-base.result == 'skipped')
runs-on: ubuntu-latest
env:
# Per run, so two runs on different refs cannot test or delete each
# other's image on the shared daemon. run_id is stable across re-runs.
SMOKE_IMAGE: pi-devbox:smoke-${{ github.run_id }}
container:
image: catthehacker/ubuntu:act-latest
steps:
@@ -610,8 +619,7 @@ jobs:
/usr/local/lib/android /usr/local/share/powershell \
/usr/local/share/chromium /usr/local/share/boost \
/usr/lib/jvm 2>/dev/null || true
docker system prune -af --volumes || true
docker builder prune -af || true
bash scripts/ci-reclaim-docker.sh
- uses: docker/setup-buildx-action@v4
with: {driver-opts: network=host}
- uses: docker/login-action@v3
@@ -626,7 +634,8 @@ jobs:
platforms: linux/amd64
push: false
load: true
tags: pi-devbox:smoke
tags: ${{ env.SMOKE_IMAGE }}
labels: ${{ env.SMOKE_IMAGE_LABEL }}=true
build-args: |
BASE_IMAGE=${{ env.IMAGE }}:${{ needs.base-decide.outputs.base_tag }}
PI_VERSION=${{ needs.resolve-versions.outputs.pi_version }}
@@ -648,7 +657,19 @@ jobs:
# Asserting the BUILT image matches it also catches a stale cached layer.
EXPECTED_NODE_MAJOR=$(sed -n 's/^ARG NODE_VERSION=\([0-9][0-9]*\).*/\1/p' Dockerfile.base)
export EXPECTED_NODE_MAJOR
bash scripts/smoke-test.sh pi-devbox:smoke
bash scripts/ci-reclaim-docker.sh assert-protected "$SMOKE_IMAGE"
# Pin the image with a running container for the whole test, so a
# raw prune from a job we do not control (other repos, the other
# Gitea instance's runner on the same VM) cannot remove it either.
bash scripts/ci-reclaim-docker.sh hold "$SMOKE_IMAGE"
bash scripts/smoke-test.sh "$SMOKE_IMAGE"
# The label keeps this image from every job's reclaim and the holder pins
# it, so both are removed here. A job killed before this step leaves an
# orphan: the holder exits after 1 h, and the reclaim's age sweep deletes
# the image once it is older than SMOKE_IMAGE_MAX_AGE.
- name: Remove smoke image
if: always()
run: bash scripts/ci-reclaim-docker.sh release "$SMOKE_IMAGE"
# ── Phase 3b: amd64 smoke for the studio variant ────────────────────
# Additive + independent of the core `smoke` job: gates ONLY
@@ -662,6 +683,10 @@ jobs:
needs.resolve-versions.result == 'success' &&
(needs.build-base.result == 'success' || needs.build-base.result == 'skipped')
runs-on: ubuntu-latest
env:
# Per run, so two runs on different refs cannot test or delete each
# other's image on the shared daemon. run_id is stable across re-runs.
SMOKE_IMAGE: pi-devbox:smoke-studio-${{ github.run_id }}
container:
image: catthehacker/ubuntu:act-latest
steps:
@@ -675,8 +700,7 @@ jobs:
/usr/local/lib/android /usr/local/share/powershell \
/usr/local/share/chromium /usr/local/share/boost \
/usr/lib/jvm 2>/dev/null || true
docker system prune -af --volumes || true
docker builder prune -af || true
bash scripts/ci-reclaim-docker.sh
- uses: docker/setup-buildx-action@v4
with: {driver-opts: network=host}
- uses: docker/login-action@v3
@@ -691,7 +715,8 @@ jobs:
platforms: linux/amd64
push: false
load: true
tags: pi-devbox:smoke-studio
tags: ${{ env.SMOKE_IMAGE }}
labels: ${{ env.SMOKE_IMAGE_LABEL }}=true
build-args: |
BASE_IMAGE=${{ env.IMAGE }}:${{ needs.base-decide.outputs.base_tag }}
PI_VERSION=${{ needs.resolve-versions.outputs.pi_version }}
@@ -716,7 +741,19 @@ jobs:
# Asserting the BUILT image matches it also catches a stale cached layer.
EXPECTED_NODE_MAJOR=$(sed -n 's/^ARG NODE_VERSION=\([0-9][0-9]*\).*/\1/p' Dockerfile.base)
export EXPECTED_NODE_MAJOR
bash scripts/smoke-test.sh pi-devbox:smoke-studio
bash scripts/ci-reclaim-docker.sh assert-protected "$SMOKE_IMAGE"
# Pin the image with a running container for the whole test, so a
# raw prune from a job we do not control (other repos, the other
# Gitea instance's runner on the same VM) cannot remove it either.
bash scripts/ci-reclaim-docker.sh hold "$SMOKE_IMAGE"
bash scripts/smoke-test.sh "$SMOKE_IMAGE"
# The label keeps this image from every job's reclaim and the holder pins
# it, so both are removed here. A job killed before this step leaves an
# orphan: the holder exits after 1 h, and the reclaim's age sweep deletes
# the image once it is older than SMOKE_IMAGE_MAX_AGE.
- name: Remove smoke image
if: always()
run: bash scripts/ci-reclaim-docker.sh release "$SMOKE_IMAGE"
# ── Phase 4: multi-arch publish ─────────────────────────────────────
build-variant:
@@ -741,8 +778,7 @@ jobs:
/usr/local/lib/android /usr/local/share/powershell \
/usr/local/share/chromium /usr/local/share/boost \
/usr/lib/jvm 2>/dev/null || true
docker system prune -af --volumes || true
docker builder prune -af || true
bash scripts/ci-reclaim-docker.sh
- uses: docker/setup-qemu-action@v3
with: {platforms: arm64}
- uses: docker/setup-buildx-action@v4
@@ -834,8 +870,7 @@ jobs:
/usr/local/lib/android /usr/local/share/powershell \
/usr/local/share/chromium /usr/local/share/boost \
/usr/lib/jvm 2>/dev/null || true
docker system prune -af --volumes || true
docker builder prune -af || true
bash scripts/ci-reclaim-docker.sh
- uses: docker/setup-qemu-action@v3
with: {platforms: arm64}
- uses: docker/setup-buildx-action@v4
+55
View File
@@ -90,6 +90,15 @@ jobs:
# actionlint so the more precise diagnostic surfaces first.
run: bash scripts/check-workflow-shell.sh .gitea/workflows
- name: No raw docker prune in workflows (shared runner daemon)
# Everything on one runner host shares its Docker daemon (runner-a1
# runs several jobs at once; runner-2 also serves gitea.egl.lan), and
# a bare `docker system prune -a` in one job deletes the smoke image
# another job is testing (v1.12.2 run 740 shipped without its -studio
# tags that way). Reclaim goes through scripts/ci-reclaim-docker.sh,
# which spares labelled smoke images; this keeps it that way.
run: bash scripts/ci-reclaim-docker.sh lint .gitea/workflows/*.yml
- name: link-pi-models.sh contract (two-sided, no docker needed)
# The entrypoint links ~/.pi/agent/models.json to a mounted shared file
# and must never replace a real file or a user's own link. Tested here
@@ -240,3 +249,49 @@ jobs:
- name: Doc claims match the build files
run: bash scripts/check-doc-drift.sh
skill-frontmatter:
# Gate the image-baked skills' frontmatter against the spec pi's own loader
# measures. Its own job, for the same reason as skill-floor and doc-drift:
# "a shipped skill violates the spec" should be a distinct red name, not a
# line buried in a job about workflow syntax.
#
# The gap it closes, measured 2026-10-23 while adding pi-devbox-firstboot:
# that skill's description was drafted at 1056 chars against the loader's
# MAX_DESCRIPTION_LENGTH of 1024. pi does NOT reject it — dist/core/skills.js
# pushes a diagnostic and then loads the skill anyway ("Still load the skill
# even with warnings, unless description is missing or empty"). So the only
# thing that would ever report the violation is a diagnostic inside a live pi
# session, and nothing in CI starts one. A spec violation that still works is
# the most durable kind of rot: nothing forces the fix.
#
# Not free in the way skill-floor was free: two of the five baked skills are
# already within ~20 chars of the limit (credential-incident-response 1022,
# pi-extensions 1003), so this fires on a plausible next edit rather than on
# a hypothetical one.
#
# The sibling failure it also catches is a silent DROP rather than a warning:
# a missing/empty description returns skill: null, so the file ships, the
# symlink resolves, smoke-test's `test -f` passes, pi-devbox-version lists
# it — and no agent can ever see it.
#
# Exit codes 0 valid / 1 violation / 2 cannot-run, matching lint-shell.sh,
# check-skill-floor.sh and check-doc-drift.sh. A missing skills tree or a
# python3 without the yaml module is a red 2, not a green tick.
runs-on: ubuntu-latest
container:
image: catthehacker/ubuntu:act-latest
steps:
- uses: actions/checkout@v4
# python3-yaml because `description: >-` is a folded block scalar: the
# value spans lines and joins with spaces, so a line-oriented length
# check measures the wrong string. Same package the actionlint job
# installs, same parser check-doc-drift.sh already relies on.
- name: Install python3-yaml
run: |
apt-get update
apt-get install -y --no-install-recommends python3-yaml
- name: Baked skill frontmatter is valid
run: bash scripts/check-skill-frontmatter.sh
+10
View File
@@ -45,6 +45,16 @@ re-brand of opencode-devbox's `pi-only` variant.
update-description). The `-studio` variant adds independent
`smoke-studio` + `build-variant-studio` jobs that gate only the
`-studio` tags (never the core `:latest` release).
- `scripts/ci-reclaim-docker.sh` — the ONLY way a CI job frees disk. Everything
on one runner host shares its Docker daemon (runner-a1 runs several jobs at
once; runner-2 also hosts a runner for gitea.egl.lan), so a bare `docker system prune -a` in one
job deletes the smoke image another job is mid-test on (v1.12.2 run 740 lost
its `-studio` tags that way). The script spares images labelled
`se.jordbo.pi-devbox.ci-smoke-image`; the smoke jobs apply that label,
assert it (`assert-protected`), pin the image with a running container for
the test (`hold`, against raw prunes from jobs we do not control) and
`release` it at the end.
`lint.yml` fails on any raw `docker system|image prune` in a workflow.
## Versioning scheme
+271
View File
@@ -11,6 +11,277 @@ Pre-v1.0.0 tags followed the pi npm version (`v{pi_version}[letter]`).
---
## v1.13.1 — 2026-10-25
Patch release with one change since v1.13.0, the pi-extensions AWS SSO keeper
fix below. Nothing else moved: pi-extensions `f032195` → **`7cb808e`** is the
only component the build bakes differently. The base image is not rebuilt
(no base input changed), so the base tools are byte-identical to v1.13.0.
> **Note:** v1.13.1 exists because v1.13.0 was built minutes before
> pi-extensions `7cb808e` landed. CI resolves pi-extensions `main` when a build
> starts, so v1.13.0 shipped `f032195`, whose keeper still counts a Bedrock
> `deep` fork profile. With `deep` on Bedrock, an ERDC session on v1.13.0 can
> still ask for an AWS login. Prefer v1.13.1 over v1.13.0.
### Fixed
- **The AWS SSO keeper ignores a Bedrock fork tier that has a non-Bedrock
fallback** (pi-extensions `7cb808e`). If `pi-task.fallbackProfiles.<tier>`
names a non-Bedrock model, that tier no longer makes the keeper log in. Then a
setup like `deep` = Bedrock Opus, falling back to `erdc/glm-5.3`, never asks
for an AWS login in an ERDC session. While the login is valid, `deep` tasks
run on Opus. Once it has expired, the `task` tool retries once on the
fallback (only after an auth error before any tool call), and its report says
`[FALLBACK]`. Measured 2026-10-25 on tor-ms22 with a deliberately invalid AWS
key: Bedrock answered `UnrecognizedClientException`, and the task finished on
glm-5.3. The `fork` tool has no fallback: a `deep` fork still fails until you
log in (`/aws-sso now`).
## v1.13.0 — 2026-10-25
Minor release. Headline: the **`pi-devbox-firstboot` skill** (first-boot
acceptance scoped by published state) and two fixes our own repos carry: the
**mailbox no longer lists long-answered asks as owed** against the 3.11.0
central palace (mempalace-toolkit `c04d488`), and the **AWS SSO keeper stays
idle when nothing in the session uses Bedrock** (pi-extensions `f032195`). Also
`confirm-destructive` steps aside for another bash gate, a skill-frontmatter
lint job, and the CI smoke-image reclaim fix.
Surveyed 2026-10-25, each upstream queried directly. Unchanged since v1.12.2:
pi `1.4.0` (npm `latest`), mempalace `3.11.0` (PyPI `latest`), pi-atelier
`v0.16.0` (`3b42825`, newest tag), pi-observational-memory `022da5e`
(= `master`), pi-fork `e69725c` (= `master`), pi-studio `v0.11.1` (`8a410b4`,
= `main`, newest tag), pi-toolkit `7c05901`, Node `v24.22.1` (newest 24.x; Node
26 is not LTS yet). The skillset snapshot needs no refresh
(`vendor-mempalace-skill.sh --check`: OK, still `96b840e`). The base image is
rebuilt, because `rootfs/` changed and the mempalace-toolkit SHA is part of the
base hash, so apt and the base tools pinned to `latest` re-resolve. Of those,
gosu, gitea-mcp and tealdeer moved upstream (table below). nvim `0.12.6`, fzf
`0.74.5`, git-lfs `3.8.1`, gitleaks `8.30.1`, bat `0.26.1`, eza `0.23.5`, zoxide
`0.10.0`, uv `0.13.5`, micro `2.0.15`, typst `0.15.1`, yq `4.54.1`,
agent-browser `0.39.2` and playwright `1.64.1` match their newest releases.
Each was compared against the v1.12.2 image on tor-ms22. The new download
assets for both architectures exist (HTTP 200).
### Components that move with this release
| Component | Baked in v1.12.2 | v1.13.0 | Why |
|---|---|---|---|
| **mempalace-toolkit** | `d5a03f4` | **`c04d488`** | mailbox fix, see *Fixed* below |
| **pi-extensions** | `4c1d28d` | **`f032195`** | `7237855` (bash gate, *Changed*) + `f032195` (aws-sso, *Fixed*) |
| gosu (base) | `1.19` | **`1.20`** (or later: `latest` at build time) | Go security update (GO-2026-6686); upstream sees no realistic attack path in gosu |
| gitea-mcp (base) | `v1.10.0` | **`v1.11.0`** (or later) | additive: `project_write` column options, `get_file_contents` `ref`/`path` optional; Go 1.27.3 security fixes |
| tealdeer (base) | `1.9.0` | **`1.9.1`** (or later) | patch release |
### Added
- **`pi-devbox-firstboot` skill + a managed-block pointer to it, so a new
device can find the fleet's first-boot acceptance round at all.** The round
is now *scoped by published state* (policy v2, artifact
`art_20261023T204306_b388e93c3d64`): a booting container reads two claim
keys — `firstboot-release-<TAG>` and `firstboot-image-<TAG>-<ARCH>` — and
derives whether it owes the release tier, the per-architecture image tier, or
only the device tier. Previously every device re-ran the entire sequence on
every release, which was drift rather than a decision: each device plants its
own successor ask carrying the previous round's checklist, so scope only grew
(4 checks at v1.10.2 → B1–B5 at v1.11.2 → B1–B7 at v1.12.2) while a peer's
PASS on *identical image bytes* was recorded as context rather than as grounds
to skip. The v1.8.10-era "image half — already settled, do not repeat" split
was never retired; it simply lapsed.
Two properties the skill exists to protect. First, **the gate needs no
roster**: fleet membership changes in both directions, so nothing addresses a
device and nothing enrolls it — state is read, not pushed. The case that
therefore has no delivery path at all is a *brand-new* device, whose mailbox
is legitimately empty and whose coordination chain does not exist yet; it must
plant its own first ask, and this skill is how it learns that. Second, the
skill is a **finder, not the policy**: it locates the current policy artifact
in the logstream instead of restating it, because a policy baked into an image
is pinned to that image — a device on an older release still reaches the
current policy, and a revision needs no release.
Baked canonical (like `pi-devbox-environment`), not skillset-owned: the round
verifies the image, so pinning it to the image is correct, and a skillset-owned
skill would need the vendoring machinery (`ARG SKILLSET_SNAPSHOT_REF`, the
manifest/label pair, a per-skill refresh script) that today exists only for
`mempalace`. `entrypoint-user.sh` and `pi-devbox-version` both enumerate the
baked skills directory, so no wiring change was needed for either to pick it
up. `scripts/smoke-test.sh` asserts the baked file, both resolution loops, and
the `AGENTS.md` pointer **by path rather than by prose** — a new device reaches
the round only through that line, so rewording the paragraph must not be able
to break the link silently.
The skill also says what to do when **a command gate on the device blocks a
probe**. The gate today is `erdc-guard`, the fleet extension that
docker-compose-repo mounts at `/opt/fleet-pi`, not part of this image. On
2026-10-24 it blocked tor-ms22's own probe-style commands, each after its
120 s wait: a child `pi -p` whose prompt mentioned `rm -rf`, heredoc test
fixtures, and `git rebase`. Unattended, that is a stalled round. Read
carelessly, it is a false defect or a false pass.
The guard's own `assess()`, run against each probe shape, showed that the
obvious workarounds don't work:
- `ERDC_GUARD=off` on the child is no help, because the parent judges the
prompt text first (0.95). `pi -p @file` is not flagged.
- A `cat` heredoc is flagged or not depending on its contents (0.93 vs not
flagged for the same shape).
- `/guard off` is volume-wide, not scoped to the round: the setting is
re-read before every bash call. And the guard keeps its bash-gate claim
while off, so confirm-destructive stays stood down too.
So the skill orders the fixes: the `write` tool for fixtures and prompt
files, `@file` for child prompts, scratch under `/tmp`, and `/guard off` last,
read first and switched back on afterwards. A blocked probe is a cannot-run.
This is device-environment method, not policy, so it belongs in the finder
rather than the policy artifact. Until this ships, the same advice reaches
tor-ms22 only through its own ask (seq 289).
- **`scripts/check-skill-frontmatter.sh` + a `skill-frontmatter` lint job:
baked skill frontmatter is now gated against the limits pi's own loader
measures.** Found while adding the skill above, whose `description` was
drafted at 1056 characters against the loader's `MAX_DESCRIPTION_LENGTH` of
1024. pi does **not** reject that — `dist/core/skills.js` pushes a diagnostic
and then loads the skill anyway ("Still load the skill even with warnings,
unless description is missing or empty"), so the only thing that would ever
report the violation is a diagnostic inside a live pi session — and nothing in
CI starts one. A spec violation that still works is the most durable kind of
rot, because nothing forces the fix.
Unlike `check-skill-floor.sh`, this one was **not** free to add: two of the
five baked skills are already within ~20 characters of the limit
(`credential-incident-response` 1022, `pi-extensions` 1003), so it fires on a
plausible next edit rather than a hypothetical one. It also catches the
sibling failure that is a silent **drop** rather than a warning — missing or
empty `description` returns `skill: null`, so the file ships, the symlink
resolves, smoke-test's `test -f` passes, `pi-devbox-version` lists it, and no
agent can ever see it. Exit codes 0 valid / 1 violation / 2 cannot-run, and
the vendored copies are deliberately in scope: the image ships those bytes,
so the violation is ours to carry whoever typed it.
- **Two baked skills' `description` fields trimmed to a routing sentence,
reclaiming ~1,087 characters of permanently-loaded context.** Every skill's
name and description sit in the system prompt of *every* request — that block
is the routing index — while bodies are fetched on demand through the read
tool. Measured here: 24 discoverable skills cost 10,609 description chars
(~2,652 tokens) on every request, against 474,656 chars of bodies (~119k
tokens) that cost nothing until used. `credential-incident-response` (1022,
i.e. two characters under the spec limit) and `pi-devbox-environment` (923)
were each carrying a table of contents of what their body covers.
Both are named **by path** from the always-loaded managed block, so neither
depends on description-matching to be found — which is what makes the cut
safe, and is why only these two were cut. `pi-extensions` was left alone for
two reasons: `check-skill-floor.sh` compares the vendored copy to the package
repo by directory hash, so trimming ours alone registers as drift; and it is
the one skill that also ships in the `skillset` repo, where it runs on hosts
with no pi-devbox managed block and so genuinely needs its description to be
found. All 18 enumerated items were verified present in the bodies first —
the cut relocates nothing, and both bodies are byte-identical.
`smoke-test.sh` now asserts both pointers by path, alongside the firstboot
one. The trim's premise is "the block names them", and that premise holds only
while those lines exist: reword the block, drop a pointer, and the skill
silently stops being reachable for a device that did not already know it was
there. Verified discriminating with a reworded negative control.
### Changed
- **`confirm-destructive` steps aside when another extension claims the bash
check** (pi-extensions `7237855`; CI resolves `main` at build time). pi runs
`tool_call` hooks one after another until one blocks. With a second command
gate loaded, `confirm-destructive` still asked about every command that
gate let through, and its question has no time limit. So an unattended
session could stop at it anyway. Every new `devbox-pi-config` volume gets
`confirm-destructive` switched on. Such a second gate exists: the fleet's
`erdc-guard` (private, mounted from `/opt/fleet-pi`).
- **How the claim works.** An extension sets
`globalThis[Symbol.for("pi-extensions.bash-gate")] = { owner: "<name>" }`.
While it is set, `confirm-destructive` skips its bash check. Its `/new`,
`/resume` and `/fork` questions are unchanged. With no claim it behaves
exactly as before. `confirm-destructive.ts` now also exports
`BASH_GATE_KEY`, which tells a claimer this version honours the claim.
- **An outside user of `matchesDangerous()`.** The file now notes that
`erdc-guard` imports it, as its fallback, from
`/opt/pi-extensions/extensions/confirm-destructive.ts`. That path and both
names must stay.
- **Tests.** `node --test test/confirm-destructive.test.mjs` in pi-extensions
`7237855` runs the real default export with a stand-in for pi: 4 tests, 4
pass. Without the line that steps aside, 3 pass and 1 fails. Whole suite:
51 pass.
- **Measured in pi 1.4.0 (2026-10-24),** with both extensions loaded in
either order: `sudo -n true` ran with `7237855` and was blocked by
v1.12.2's `4c1d28d`.
- **On v1.12.2 and older images** `confirm-destructive` ignores the claim.
There, switch it off once per volume with `/ext`; the image keeps a
`.ts.off` link off at every start.
### Fixed
- **AWS SSO login prompt in sessions that don't use Bedrock (pi-extensions
`f032195`).** The `aws-sso` keeper logged in whenever it was enabled, so
`pi --model glm-5.3` (ERDC) still asked for AWS approval once the SSO session
had ended. It now acts only when the session model, the observational-memory
model or a fork/task profile is `amazon-bedrock`, and otherwise shows
`AWS · idle (no Bedrock)`. For a session with no AWS at all, point
`observational-memory.model` and `pi-fork.effortProfiles` at ERDC models in
`settings.json` too.
- **Mailbox: asks this device answered long ago came back as owed
(mempalace-toolkit `c04d488`).** mempalace 3.11.0 stopped treating
`from_agent` as a filter on `event_list`; the filter is now `writer`. The
baked bridge still sent `from_agent` to find this device's own replies, so
against the central palace (on 3.11.0 since 2026-10-24T22:20Z) it got the
newest 100 events from every writer, and older replies dropped out. On
tor-ms22 the wake-up listed 22 owed asks; all 20 threads already had a
closing reply. The bridge now sends `writer`, and falls back to `from_agent`
only when a 3.10.0-or-older server rejects `writer`. Measured against the
live palace: 0 owed with the fix, 22 with `d5a03f4`. Until a device runs this
release, its wake-up keeps listing them; don't re-close them.
- **CI: a smoke job's image could be deleted mid-test by another job's disk
reclaim.** This is why v1.12.2's `-studio` tags were not published by its
tag run. Everything on one runner host shares its Docker daemon: concurrent
jobs of a runner with capacity > 1 (in run 740, runner-a1 ran `smoke-studio`
and `build-variant` at the same time), and the jobs of other act_runner
containers on the same VM (runner-2 has one for gitea.jordbo.se and one for
gitea.egl.lan). Each pi-devbox job began with `docker system prune -af --volumes`, which removes every
image that no container is using at that moment. A smoke image is unused
between `smoke-test.sh`'s one-shot `docker run`s. In run 740, `smoke`
passed and `build-variant` started. Its prune logged
`untagged: pi-devbox:smoke-studio` at 22:58:49.11Z, 30 ms after
`smoke-studio` had loaded that image. Every studio assertion then failed with
"Unable to find image … locally", and `build-variant-studio` was skipped.
`v1.12.2` and `latest` were published; `v1.12.2-studio` and
`latest-studio` were not, until the failed jobs were re-run. Now every job
reclaims disk through the new `scripts/ci-reclaim-docker.sh`. It does the
same work as before (stopped containers, unused images, networks, anonymous
volumes, build cache) but never prunes an image labelled
`se.jordbo.pi-devbox.ci-smoke-image`. The smoke builds apply that label, a
preflight fails if it is missing, and each smoke job removes its own image at
the end. Jobs we do not control can still run a raw prune on the same
daemon: opencode-devbox's workflows do, in 7 places, and so may whatever
gitea.egl.lan runs. So the smoke step also pins its image with a running
container (`ci-reclaim-docker.sh hold`) for the whole test, because no prune
removes an image a running container uses. Only the few seconds between the
build step loading the image and the pin starting are left unprotected. The
holder exits by itself after 1 h, and smoke images left by a killed job are
swept once they are older than 6 h. Smoke tags carry the run id (`pi-devbox:smoke-<run_id>`), so two
runs on different refs cannot test or delete each other's image. `lint.yml`
fails on any raw `docker system|image prune` in a workflow. Job order is
unchanged: the studio jobs still gate only the `-studio` tags. Reordering
could not have closed the race anyway, since the two smoke jobs also prune
each other whenever their start times drift. Measured 2026-10-23 on an
isolated `docker:29-dind` (engine 29.9.2; the runner has 29.7.2). The old
prune deletes an unused labelled image (the bug). The new reclaim keeps a
fresh labelled image, deletes an unlabelled one and a labelled one created
in 1970, keeps an image a running container uses, and prints no errors.
`assert-protected` passes the labelled image and fails both an unlabelled
and a missing one. With `hold`, an unlabelled image survives both a raw
`docker system prune -af --volumes` and `docker image prune -af` (control:
without the holder, the same prune deletes it). `hold` fails on a missing
image, and `release` removes the holder and the image.
## v1.12.2 — 2026-10-22
Patch release. Headline: **pi `1.4.0`** (audited, see *Changed* below), plus
+1 -1
View File
@@ -14,7 +14,7 @@
# content-addressed over this file, so any byte change invalidates the
# cache. Recommended cadence: once per release for security updates.
#
# BASE_REBUILD_DATE: 2026-10-22 (v1.12.2 — setup-lan-access.sh always Includes ~/.ssh/config; apt and base tools re-resolve (upstream on 2026-10-22: Node v24.22.1 and git-lfs 3.8.1 security releases, uv 0.13.5, agent-browser 0.39.2; gitea-mcp 1.10.0, nvim 0.12.6, fzf 0.74.5 unchanged); previous marker 2026-10-19 / v1.12.1)
# BASE_REBUILD_DATE: 2026-10-25 (v1.13.0 — rootfs gains the pi-devbox-firstboot skill and mempalace-toolkit moves to c04d488; apt and base tools re-resolve (upstream on 2026-10-25: gosu 1.20, gitea-mcp 1.11.0, tealdeer 1.9.1; Node v24.22.1, git-lfs 3.8.1, uv 0.13.5 unchanged); previous marker 2026-10-22 / v1.12.2)
#
# ── Lineage note ─────────────────────────────────────────────────────
# Adapted from opencode-devbox/Dockerfile.base (commit before v1.16.2).
+9 -1
View File
@@ -804,7 +804,14 @@ directory, and they compose:
mechanisms, the interactive-vs-tool-shell alias gotcha (`dssh`/`dscp`),
tmux 0-indexing, uv-first Python, and pi-studio reachability, all as
*mechanisms* (deployment-specific hostnames/domains/nameservers are
discovered at runtime, never hardcoded).
discovered at runtime, never hardcoded). **`pi-devbox-firstboot`** ships the
same way and is the entry point to the fleet's first-boot acceptance round:
it carries the *state-based scope gate* (two claim lookups, keyed by release
tag and by architecture, deciding which tiers of checks this device still
owes) rather than the policy itself, which it locates in the logstream so a
policy revision needs no release. Baked rather than skillset-owned on purpose
— the device that most needs it is a brand-new one with an empty mailbox and
no coordination chain, so nothing would deliver it.
- **Vendored fallback skills.** The pi-toolkit global `AGENTS.md` tells every
pi session to read `~/.agents/skills/pi-extensions/SKILL.md` at start (to fix
fork/recall under-utilisation). That pointer would dangle in a container
@@ -1124,6 +1131,7 @@ pi-devbox v1.8.14
credential-incident-response baked
mempalace live /workspace/skillset @ 4d7c0ea (identical to baked snapshot)
pi-devbox-environment baked
pi-devbox-firstboot baked
pi-extensions baked
```
@@ -71,6 +71,26 @@ rather than merely confusing you:
the MCP tools while both look correct. Use the MCP tools for the central
palace; the CLI only for a local one.
## First boot on a new release: scope the acceptance round
If you are in a pi-devbox container that was just recreated, or one running a
release this device has not accepted before — or a `firstboot-acceptance` ask is
in your mailbox, or your mailbox is **empty and this device has never run that
round** — read `~/.agents/skills/pi-devbox-firstboot/SKILL.md` before starting.
That round is **scoped by published state, not run in full by every device**:
two claim lookups, keyed by release tag and by architecture, decide whether you
owe the release tier, the per-architecture image tier, or only the device tier.
Three rules are load-bearing enough to repeat here. **Derive the release from
`/etc/pi-devbox/build-manifest.json`; never predict it** — four tags in this
image's history published nothing. **Honour a peer's claim only if its keys
match your own first-hand readings**, or one stale claim silently switches off
verification across the whole fleet while every device still reports PASS. And
**the device tier always runs**, because the same image reads differently per
device and both readings are correct (a warn count is not a device property
unless the invocation is pinned). An empty mailbox on a device with no chain
means *you* plant the first ask — nobody is watching for new devices.
## Before you file a finding: second measurement, different route
This is here rather than in a skill because it has to fire *without* a matching
@@ -10,6 +10,7 @@ one", which was a bug).
|-------|-------|------------------|
| `pi-devbox-environment` | pi-devbox (this repo) | authored here; the canonical copy |
| `credential-incident-response` | pi-devbox (this repo) | authored here; the canonical copy |
| `pi-devbox-firstboot` | pi-devbox (this repo) | authored here; the canonical copy |
| `pi-extensions` | the `pi-extensions` package repo (`skill/`) | **vendored fallback** + refreshed at build |
| `mempalace` | the `skillset` repo | **vendored fallback** (snapshot only) |
@@ -27,6 +28,20 @@ actually picks it up rather than relying on description-matching.
`pi-extensions`'s directive already ships in pi-toolkit's `AGENTS.md`, so only
its skill file needed baking.
`pi-devbox-firstboot` is the third case: authored here, baked canonical, **and**
pointed at from the managed block, for a reason specific to what it does. It is
the entry point to the fleet's first-boot acceptance round, and the device that
most needs it is a **brand-new** one whose mailbox is empty and whose chain does
not exist yet — so nothing will deliver the procedure to it, and
description-matching on an empty mailbox is exactly the case that fails. It is
deliberately a *finder*: it locates the current policy artifact in the logstream
rather than restating it, so a policy revision does not require a release and a
device booting an older image still reaches the current policy. It is **not**
skillset-owned, because the round verifies the image and should therefore be
pinned to the image — and because a skillset-owned skill needs the vendoring
machinery (`ARG SKILLSET_SNAPSHOT_REF`, the manifest/label pair, and a
per-skill refresh script), which today exists only for `mempalace`.
## Freshness model (layered — see Dockerfile.variant)
- **`pi-extensions`** — Option 1 + Option 2. The committed copy here is the
@@ -6,13 +6,7 @@ description: >-
agent-authored note. Load this whenever a task involves a leaked/exposed
secret, a token rotation, a "is this credential still live?" question, deciding
whether to delete or scrub stored content, proving a corpus is clean, or
choosing scopes for a new API token. Covers the mandatory order of operations
(probe the issuer FIRST — severity before cleanliness), leak-free identity via
sha256[:8] fingerprints and when publishing one is safe,
why revocation beats deletion for anything already replicated, scopes derived
from measured consumers, the three places a secret hides in a Chroma palace, how to prove ABSENCE rather than assume it (instrument strength,
census vs class passes, the tokenisation trap where quoting decides detectability, why git filters never run on symlinks, self-tests that abort),
where this fleet's secrets live, and what rotation does NOT fix.
choosing scopes for a new API token.
---
# Credential incident response
@@ -6,14 +6,7 @@ description: >-
shell prompt is prefixed `[devbox]`, or `~/.ssh-local/config` is present) and
the task touches any of: reaching the Docker host or its LAN, SSH, DNS name
resolution, what survives container recreate (persistence vs ephemerality),
running Python or other REPLs, tmux, or the pi-studio browser UI. Covers the
persistence model, the interactive-vs-tool-shell alias gotcha
(dssh/dscp/cat=bat exist only in interactive bash), host + LAN SSH
reachability and ControlMaster, split-horizon DNS mechanisms, the tmux
0-index constraint, uv-first Python, and pi-studio reachability. This skill
teaches MECHANISMS only — concrete hostnames, usernames, internal domains,
nameservers, and even the host OS vary per deployment and MUST be discovered
at runtime, never assumed or hardcoded.
running Python or other REPLs, tmux, or the pi-studio browser UI.
---
# pi-devbox environment
@@ -0,0 +1,249 @@
---
name: pi-devbox-firstboot
description: >-
Run and correctly scope the fleet's first-boot acceptance round for a
pi-devbox container. Load this when a container was just recreated or runs a
release it has not accepted before, when a firstboot-acceptance ask is in
your mailbox, when verifying a new pi-devbox release, or when your mailbox is
EMPTY and this device has never run the round — the self-enrollment case,
where no chain exists and nobody will plant one for you. Covers the
state-based scope gate (two claim lookups keyed by release tag and
architecture), which checks settle once per release, which settle once per
architecture, and which must be re-measured per device whatever a peer
proved; validating a claim against your own readings before honouring it;
planting and perpetuating your own chain; and the invariants that keep it
alive (derive the release from the manifest instead of predicting it, reply
terminal before planting a successor, never score a cannot-run as a pass).
---
# First-boot acceptance for a pi-devbox container
**This skill is a FINDER, not the policy.** The authoritative procedure lives in
the logstream as a versioned artifact, because a policy baked into an image is
pinned to that image and cannot be corrected without a release. Your first job
is therefore to fetch the current policy, not to follow this file from memory.
```
mempalace_event_list(topic="firstboot-acceptance", type="policy.update", limit=5)
```
Take the newest, read `metadata.policy_artifact`, and fetch it:
```
mempalace_artifact_get(artifact_id="<policy_artifact>")
```
**If that artifact and this file disagree, the artifact wins** — it is newer by
construction. Treat this file as the thing that tells you the artifact exists.
(As of writing: policy v2, `art_20261023T204306_b388e93c3d64`, sha256
`40048d5d…`, 11252 B. Verify rather than trust that id; policies get revised.)
## Why there is a gate at all
Every device used to run the entire check sequence on every release. That was
not a decision — it was drift. Each device plants its own successor ask carrying
whatever the previous round contained, so scope only ever grew (4 checks at
v1.10.2 → B1–B5 at v1.11.2 → B1–B7 at v1.12.2), while a peer's PASS on the
*identical image bytes* was recorded as interesting context rather than as
grounds to skip anything. The fleet had actually decided the opposite in
2026-08 (v1.8.10: "IMAGE HALF — ALREADY SETTLED, DO NOT REPEAT"), and nothing
ever retired that decision; it simply lapsed.
The gate restores it, in a form that needs **no roster**. Nothing addresses you,
nothing enrolls you, and a device that leaves the fleet leaves nothing behind to
clean up. You derive your own scope by reading published state.
## The gate
```
release_tag, source_revision <- /etc/pi-devbox/build-manifest.json
arch, my_arch_digest <- your architecture, your PER-ARCH manifest digest
R_claim <- event_list(correlation_id="firstboot-release-<TAG>", type="firstboot.release.settled")
I_claim <- event_list(correlation_id="firstboot-image-<TAG>-<ARCH>", type="firstboot.image.settled")
```
| R valid | I valid | you run | meaning |
|---|---|---|---|
| no | no | **R + I + D** | first device on this release, anywhere |
| yes | no | **I + D** | first device on this *architecture* for this release |
| yes | yes | **D only** | cite the claim ids you inherited |
Emit a claim for each tier you newly settle. **Tier D always runs.**
The `correlation_id` *is* the state key, which makes this two exact server-side
lookups — no scanning, no metadata filtering, and the same cost whether the
fleet has three devices or thirty.
### RULE ZERO: derive the release, never predict it
Read `release_tag` from the manifest at boot. Do not write an expected release
into an ask, a drawer or an expectation file. Seven plants on this fleet have
died of predicting a version: v1.9.0, v1.10.0, v1.10.1 and v1.11.1 were all
tagged and then published nothing, so any ask naming them was wrong before it
was read.
### Validate before you honour — the one way this fails unsafely
A claim is honoured **only if its keys equal your own first-hand readings**:
- release claim: `release_tag` **and** `source_revision`
- image claim: `release_tag`, `arch` **and the arch digest you measured**
A mismatch means **absent**: run the tier. Without this check, a single
mis-keyed or stale claim silently switches off real verification across the
whole fleet, and the failure is invisible because every device still reports
PASS. A `firstboot.release.disputed` / `firstboot.image.disputed` event newer
than the newest claim on that key **voids** it — run the tier yourself.
Duplicate claims are harmless and expected: two devices booting the same release
before replication catches up both run full scope and both claim. The gate wants
**at least one** valid claim, not exactly one. There is deliberately no
reservation protocol, because a reservation can strand a tier if its holder dies
mid-round.
### The digest is the per-arch one
Your image key is the **per-architecture registry manifest digest**. On OrbStack
`docker image inspect` reports `Id` == `RepoDigests` == the multi-arch **index**
digest, while the `com.docker.compose.image` label carries the arch digest.
Comparing an index digest with an arch digest is a category error, not a
mismatch. An arm64 pass licenses **nothing** on amd64 — different bytes.
## The three tiers
**Tier R — release-level, architecture-independent.** The seven component refs
are resolved once per build and handed to every variant job, and the harness
lives in the repo rather than the image: component refs, CI run health,
`Dockerfile.variant` harness wiring, the directives truth table, host buildx,
changelog-vs-repo. Key: `release_tag` + `source_revision`.
**Tier I — image-level, per architecture.** Different bytes per arch: the obsmem
turn cap (two-sided, in both `src` and `dist` forms), dormancy-bridge
byte-identity against the manifest commit, base tool versions as installed, the
component `/opt` HEADs. Key: `release_tag` + `arch` + arch digest.
**Tier D — device-level, always, never inheritable.** Recreate timeline,
is-the-release-really-here, your own digest reading, the sanity script on every
invocation you report, ControlPath, volume survival, palace wiring and
extensions, `.env` and credential probes, KG and fleet triples, chain
bookkeeping — and the **config-coupled runtime checks**.
That last group is *why* D can never be inherited. The same image reads
differently per device and **both readings are correct**:
- `showSidebarOnStartup` is FALSE on some devices and TRUE on others, so an
absent startup sidebar is a defect on one device and by-design on another.
- Sanity warn counts differ by device **and by invocation** — bare vs
`--expected-image-version` gives 2 vs 1 on the same image in the same minute.
**A warn count is not a device property unless the invocation is pinned.**
Record the invocation next to every number.
- ControlPath warnings track the *host's* ssh config, not any image defect.
Carrying a peer's number across has already manufactured false defects here.
## Self-enrollment: if your mailbox is empty
The mempalace skill tells you to check your mailbox and that an empty result is
a fine answer. For this round, empty has two very different causes:
1. **You have a chain and its ask is dormant** — correct, nothing is owed.
2. **You have no chain at all** — you are a new device. Nothing is owed and
nothing ever will be, because asks are planted per device *by that device*.
In case 2, **plant your own first ask** rather than waiting for a peer to notice
you joined. Historically a peer did plant it cross-device (one device's ask was
authored by another), which is exactly the roster dependency the state design
removes. Query your own chain before concluding:
```
mempalace_event_list(correlation_id="firstboot-acceptance-<device>", limit=5)
```
## Chain invariants — the ways this chain dies quietly
- **Reply terminal FIRST, then plant the successor.** `isAnswered` joins on
`correlation_id` with a strictly-after comparison, so planting first silently
retires your own successor and the chain dies with no error at the next
recreate.
- **Dormancy baselines must be ISO-with-Z**, and verified against the *real*
`evaluateDormancy` from the booted bridge before you append. UTC digits
without the `Z` parse 7200 s low on a CEST host; an epoch (string or number)
is `NaN` and makes the condition unevaluable, so the ask nags forever while
looking like a broken feature. Import the function; do not regex a copy of it.
- **A cannot-run is never a pass.** Gates here deliberately separate rc=1
(violation) from rc=2 (cannot run). An empty file is a violation; an absent
path is a cannot-run. Scoring a cannot-run as a pass is how a gate that never
ran reports green.
- **Only the original requester can withdraw an ask.** `isWithdrawn` requires
the terminal event to be addressed to the exact mailbox, so you cannot retire
another device's ask and no broadcast can empty every mailbox at once.
- **Claims are `status=ready`.** Not `open` — the owed-work query selects
`status=open` and a `to_agent="*"` broadcast matches every mailbox, so an open
claim would appear as an owed ask on every device in the fleet. Not terminal
either, because terminal events join `isAnswered` and could clear real work.
## A command gate on the device can fail your probes
A device may run a bash gate that the image does not ship. Today that is
`erdc-guard`, a fleet extension mounted from docker-compose-repo at
`/opt/fleet-pi`. It scores every bash statement with an ERDC classifier and
asks before running anything at or above its threshold (0.8 by default). In
the TUI it waits 120 s, then blocks; under `pi -p` it blocks at once.
A probe it stops failed for a reason that has nothing to do with the image.
**That is a cannot-run: never a pass, and never a defect either.** Read
`/guard status` before you probe, so you know whether the gate is on.
The gate judges the **text** of a command, not what will actually run.
Measured on tor-ms22 on 2026-10-24 with the guard's own `assess()`:
| probe shape | flagged |
|---|---|
| `ERDC_GUARD=off pi -p "… rm -rf data …"` | yes, 0.95 |
| the same prompt, as `pi -p @/tmp/…/prompt.md` | no |
| `cat >> f <<'EOF'` with test lines holding `bash("rm -rf build/")` | yes, 0.93 |
| `cat > f <<'EOF'` with one `"rm -rf build/"` string | no |
| `rm -rf /tmp/<scratch>` | no (skipped as scratch) |
| `git rebase origin/main` | yes, 0.89 |
Try these in order:
1. **Write fixture files and prompt files with the `write` tool**, which the
gate does not watch. Whether a heredoc is flagged depends on its contents,
so you cannot predict it. Do not route file content through bash.
2. **Give a child pi its prompt as `@file`.** `ERDC_GUARD=off` in the child's
environment switches off only the *child's* gate. The parent has already
judged the prompt text and blocked it.
3. **Keep scratch under `/tmp`.** The gate skips removals confined to `/tmp`
without sending a request.
4. **Use `/guard off` only as a last resort.** It is not scoped to your round:
- The setting lives in `~/.pi/agent/erdc-guard.json` and is re-read before
every bash call. Switching it off turns off the gate in every pi session on
the volume at once, and it stays off if your round dies.
- While off, erdc-guard keeps its bash-gate claim, so confirm-destructive
stays stood down too. The result is no gate at all.
- It also changes the Tier D reading of `/guard status`.
If you must use it: take that reading first, switch the gate off, switch it
back on afterwards, and record both switches in the evidence.
## When you copy a probe forward, grep it for your own recorded landmines
Not just stale paths and the previous run's parameters. This fleet has a
documented instance of an agent re-importing a probe carrying the exact
`set -u` bug that its *own* retraction list named as having already killed that
probe. **Reading a retraction is not the same as applying it to the artefact the
retraction is about.**
## Limits of this file
- It tells you the procedure exists and where the real one lives. It cannot be
the procedure, because it ships inside the image it would be verifying — if
this file were wrong you would validate a bad image with its own instructions.
- It arrives only with a release. A device booting an older image gets an older
copy of this finder, which is the main reason it carries the *lookup* rather
than the policy: the lookup is stable across policy revisions.
- Nothing here is automatic. A session must load this skill (the pi-devbox
managed block in the global `AGENTS.md` points at it), and the device must
then act on it.
+189
View File
@@ -0,0 +1,189 @@
#!/usr/bin/env bash
# check-skill-frontmatter.sh — fail when an image-baked skill's frontmatter
# violates the Agent Skills spec that pi's own loader measures against.
#
# THE DEFECT THIS EXISTS TO CATCH, measured 2026-10-23 while adding the
# pi-devbox-firstboot skill. Its `description` was drafted at 1056 characters.
# pi's shipped loader (dist/core/skills.js) defines MAX_DESCRIPTION_LENGTH =
# 1024 and pushes a diagnostic when a description is longer — and then says, in
# a comment directly above the return:
#
# // Still load the skill even with warnings, unless description is
# // missing or empty.
#
# So an over-long description does NOT break the skill. That is exactly why it
# needs a gate rather than trust: the skill works, the violation is real, and
# the only thing that would ever tell you is a diagnostic emitted inside a pi
# session. Nothing in CI starts a pi session and reads skill diagnostics, so the
# warning is invisible here by construction. A spec violation that still works
# is the most durable kind of rot — nothing ever forces the fix.
#
# One failure in this family is NOT a warning but a silent drop: a missing or
# empty description returns `{ skill: null }`, so the skill is never offered to
# the model at all. The skill file ships, `test -f` passes in smoke-test.sh, the
# symlink resolves, `pi-devbox-version` lists it — and the skill does not exist
# as far as any agent is concerned. This gate separates the two cases in its
# output, because they need different urgency even though both are red.
#
# WHY THE BAKED TREE SPECIFICALLY. These skills are the FALLBACK layer (see
# skills/VENDORED.md): they are what a container resolves with no skillset repo
# mounted, which is also the case of a brand-new device that has nothing else.
# A skillset-owned skill has its own upstream review; a baked one has only this.
#
# MEASURED HEADROOM when this gate was added, which is why it is worth having:
# credential-incident-response 1022 (2 chars spare)
# pi-extensions 1003 (21)
# pi-devbox-firstboot 952 (72)
# pi-devbox-environment 923 (101)
# mempalace 254 (770)
# Two of five are within one sentence of the limit, so "nobody would write a
# description that long" is already false here.
#
# A KNOWN COUPLING, stated rather than discovered later: `pi-extensions` and
# `mempalace` are VENDORED copies, so an upstream edit that pushes one over the
# limit turns this gate red on a file this repo does not author. That is
# deliberate and precedented — check-skill-floor.sh already couples to the
# pi-extensions package — because the image SHIPS those bytes, so the violation
# is ours to carry regardless of who typed it. The fix in that case is to
# refresh the snapshot after upstream trims, not to exempt the file.
#
# WHY NAME IS CHECKED TOO, including against the directory. The loader falls
# back to the PARENT DIRECTORY NAME when frontmatter has no usable `name`
# (`frontmatterName || parentDirName`), so a mismatch does not fail — it
# silently renames the skill, and every pointer written as
# `~/.agents/skills/<dir>/SKILL.md` then refers to a skill the model knows under
# a different name. The managed AGENTS.md block points at skills by path, so
# that divergence is load-bearing here.
#
# EXIT CODES (same contract as lint-shell.sh / check-skill-floor.sh /
# check-doc-drift.sh — a gate that cannot run must not pass):
# 0 every baked skill's frontmatter is valid
# 1 VIOLATION — at least one skill breaks the spec
# 2 cannot run — no skills directory, or no python3 with a YAML module
#
# WHY PYTHON AND NOT grep/awk. `description: >-` is a YAML folded block scalar:
# its value spans lines and is joined with spaces, so a line-oriented length
# check measures the wrong string. python3-yaml is already installed in the
# actionlint job and already used by check-doc-drift.sh, so this adds no new
# dependency. NB the heredoc form below is `python3 - "$arg"` (script on stdin,
# argument on the command line); `echo x | python3 <<'EOF'` cannot work, which
# is the SC2259 defect the shellcheck gate was added for.
set -euo pipefail
REPO_ROOT=$(cd "$(dirname "${BASH_SOURCE[0]}")/.." && pwd)
SKILLS_DIR="${REPO_ROOT}/rootfs/usr/local/share/pi-devbox/skills"
usage() {
cat <<'USAGE'
Usage: scripts/check-skill-frontmatter.sh [options]
Options:
--skills-dir DIR Validate DIR instead of the repo's baked skills tree
(used by the truth-table self-test).
-h, --help This text.
Exit: 0 valid | 1 violation | 2 cannot run
USAGE
}
while [ $# -gt 0 ]; do
case "$1" in
--skills-dir) shift; [ $# -gt 0 ] || { echo "::error::--skills-dir needs a value" >&2; exit 2; }; SKILLS_DIR="$1" ;;
-h|--help) usage; exit 0 ;;
*) echo "::error::unknown option: $1" >&2; usage >&2; exit 2 ;;
esac
shift
done
if [ ! -d "$SKILLS_DIR" ]; then
echo "::error::skills directory not found: ${SKILLS_DIR}"
echo "A gate that cannot run must not pass — this is a red 2, not a green tick."
exit 2
fi
if ! command -v python3 >/dev/null 2>&1; then
echo "::error::python3 not found; cannot parse YAML frontmatter"
exit 2
fi
if ! python3 -c 'import yaml' >/dev/null 2>&1; then
echo "::error::python3 has no yaml module (apt: python3-yaml); cannot parse frontmatter"
exit 2
fi
python3 - "$SKILLS_DIR" <<'PY'
import os, re, sys
import yaml
# Mirrors pi's dist/core/skills.js. If pi changes these, change them here.
MAX_NAME_LENGTH = 64
MAX_DESCRIPTION_LENGTH = 1024
skills_dir = sys.argv[1]
fatal, violations, checked = [], [], 0
for name in sorted(os.listdir(skills_dir)):
d = os.path.join(skills_dir, name)
if not os.path.isdir(d):
continue
path = os.path.join(d, "SKILL.md")
rel = os.path.relpath(path, skills_dir)
if not os.path.isfile(path):
fatal.append(f"{rel}: directory has no SKILL.md — the loader cannot see it")
continue
checked += 1
raw = open(path, encoding="utf-8").read()
m = re.match(r"^---\n(.*?)\n---\n", raw, re.S)
if not m:
fatal.append(f"{rel}: no YAML frontmatter block (must open on line 1 with ---)")
continue
try:
fm = yaml.safe_load(m.group(1)) or {}
except Exception as e:
fatal.append(f"{rel}: frontmatter is not valid YAML: {e}")
continue
if not isinstance(fm, dict):
fatal.append(f"{rel}: frontmatter is not a mapping")
continue
desc = fm.get("description")
if not isinstance(desc, str) or desc.strip() == "":
# The one hard drop: the loader returns skill: null for this.
fatal.append(f"{rel}: description missing or empty — THE SKILL IS NEVER LOADED")
elif len(desc) > MAX_DESCRIPTION_LENGTH:
violations.append(
f"{rel}: description is {len(desc)} chars, exceeds {MAX_DESCRIPTION_LENGTH} "
f"(loader warns and still loads, so nothing else will tell you)"
)
fm_name = fm.get("name") if isinstance(fm.get("name"), str) else None
eff = fm_name or name
if len(eff) > MAX_NAME_LENGTH:
violations.append(f"{rel}: name is {len(eff)} chars, exceeds {MAX_NAME_LENGTH}")
if not re.fullmatch(r"[a-z0-9-]+", eff):
violations.append(f"{rel}: name {eff!r} must be lowercase a-z, 0-9 and hyphens only")
if eff.startswith("-") or eff.endswith("-"):
violations.append(f"{rel}: name {eff!r} must not start or end with a hyphen")
if "--" in eff:
violations.append(f"{rel}: name {eff!r} must not contain a double hyphen")
if fm_name is not None and fm_name != name:
violations.append(
f"{rel}: frontmatter name {fm_name!r} != directory {name!r} — the loader "
f"would use {fm_name!r} while every path-based pointer says {name!r}"
)
print(f"checked {checked} baked skill(s) in {skills_dir}")
if not fatal and not violations:
print("OK: every baked skill's frontmatter is valid.")
sys.exit(0)
for f in fatal:
print(f"::error::{f}")
for v in violations:
print(f"::error::{v}")
print(f"FAIL: {len(fatal)} fatal, {len(violations)} spec violation(s).")
sys.exit(1)
PY
+141
View File
@@ -0,0 +1,141 @@
#!/usr/bin/env bash
# ci-reclaim-docker.sh — free disk on the CI runner's Docker daemon WITHOUT
# deleting a smoke image that another job is still testing.
#
# Used by every docker-publish.yml job that reclaims disk (build-base, smoke,
# smoke-studio, build-variant, build-variant-studio). Not shipped in the image.
#
# bash scripts/ci-reclaim-docker.sh # reclaim (default)
# bash scripts/ci-reclaim-docker.sh assert-protected IMG # smoke preflight
# bash scripts/ci-reclaim-docker.sh hold IMG # pin IMG while testing
# bash scripts/ci-reclaim-docker.sh release IMG # drop the pin + IMG
# bash scripts/ci-reclaim-docker.sh lint WORKFLOW... # lint.yml guard
#
# WHY THIS EXISTS. Everything that runs on one runner host shares its Docker
# daemon: concurrent jobs of a runner with capacity > 1 (runner-a1 had
# smoke-studio and build-variant going together in run 740), and the jobs of
# other act_runner containers on the same VM (runner-2 has one registered with
# gitea.jordbo.se and one with gitea.egl.lan). Each pi-devbox job used to start
# with `docker system prune -af --volumes`. That removes every image no
# container is using at that instant, and a smoke image is
# exactly that for most of its life: smoke-test.sh runs one `docker run --rm`
# per assertion, so between assertions nothing holds the image. v1.12.2 run 740
# (2026-10-22): smoke passed, build-variant started and pruned, and its log
# shows `untagged: pi-devbox:smoke-studio` at 22:58:49.11Z, 30 ms after
# smoke-studio had imported that image (22:58:49.08Z). Every smoke-studio
# assertion then failed with "Unable to find image 'pi-devbox:smoke-studio'
# locally", build-variant-studio was skipped, and the release went out without
# its -studio tags. Job ORDER cannot close this: smoke and smoke-studio prune
# each other too whenever their start times drift apart, and other runs share
# the daemon. Two defences, because we only control our own jobs:
# - the smoke images carry a label, and this script never prunes an image
# that has it (covers pi-devbox's own jobs, before and during the test);
# - `hold` starts a container from the image for the whole smoke step. No
# prune removes an image a running container uses, so this also covers
# jobs that never call this script (opencode-devbox's raw prunes,
# anything gitea.egl.lan runs). What is left is the few seconds between
# the build step loading the image and `hold` starting.
#
# WHAT IT DOES. The same reclaim as `docker system prune -af --volumes` +
# `docker builder prune -af`, done per object type so the image step can take
# a filter:
# 1. stopped containers (running ones are never touched)
# 2. unused images WITHOUT the label (the old `system prune -a` image step)
# 3. unused images WITH the label, older than SMOKE_IMAGE_MAX_AGE
# — orphans of a smoke job that died before its own cleanup step
# 4. unused networks, anonymous volumes (what `system prune --volumes` did)
# 5. the build cache
# Each step is `|| true`, as before: failing to reclaim disk must not fail the
# job. A filter the daemon rejected would show up as an error line in the log.
#
# `assert-protected IMG` is the smoke jobs' preflight: it fails unless IMG is in
# the daemon AND carries the label. Without it, a build that silently lost the
# label would look fine until the next time a sibling job's prune landed
# mid-test; and a missing image gets one clear error line instead of every
# assertion failing with "Unable to find image ... locally".
#
# `lint WORKFLOW...` fails if any workflow runs `docker system prune` or
# `docker image prune` itself. One such line in any job reopens the race, so
# lint.yml runs this on every push rather than relying on review.
#
# Environment:
# SMOKE_IMAGE_LABEL label key that protects an image (default below; the
# workflow sets it once in its top-level env)
# SMOKE_IMAGE_MAX_AGE `until=` value for the orphan sweep (default 6h; a
# smoke job takes ~10 min). Compared against the image's
# Created time, which is the build time: nothing here
# sets SOURCE_DATE_EPOCH.
set -u
SMOKE_IMAGE_LABEL="${SMOKE_IMAGE_LABEL:-se.jordbo.pi-devbox.ci-smoke-image}"
SMOKE_IMAGE_MAX_AGE="${SMOKE_IMAGE_MAX_AGE:-6h}"
step() { printf '+ %s\n' "$*"; "$@" || true; }
reclaim() {
step docker container prune -f
step docker image prune -af --filter "label!=${SMOKE_IMAGE_LABEL}"
step docker image prune -af --filter "label=${SMOKE_IMAGE_LABEL}" \
--filter "until=${SMOKE_IMAGE_MAX_AGE}"
step docker network prune -f
step docker volume prune -f
step docker builder prune -af
}
assert_protected() {
local img="${1:?usage: $0 assert-protected <image>}" val
if ! val=$(docker image inspect \
--format "{{ index .Config.Labels \"${SMOKE_IMAGE_LABEL}\" }}" "$img" 2>&1); then
echo "::error::${img} is not in the runner's Docker daemon (${val}). Was it removed by a concurrent prune that does not honour ${SMOKE_IMAGE_LABEL}?"
return 1
fi
if [ "$val" != "true" ]; then
echo "::error::${img} lacks the label ${SMOKE_IMAGE_LABEL}=true (got '${val}'), so a sibling job's reclaim could delete it mid-test. Check the build step's labels: input."
return 1
fi
echo "OK: ${img} is present and carries ${SMOKE_IMAGE_LABEL}=true"
}
holder_name() { printf 'ci-hold-%s' "$(printf '%s' "$1" | tr -c 'A-Za-z0-9_.-' '_')"; }
# The holder exits on its own after an hour (--rm), so a job killed before
# `release` does not pin its image forever; the age sweep then removes it.
hold() {
local img="${1:?usage: $0 hold <image>}" name
name=$(holder_name "$img")
docker rm -f "$name" >/dev/null 2>&1 || true
if ! docker run -d --rm --name "$name" --network none --entrypoint "" \
"$img" sleep 3600 >/dev/null; then
echo "::error::could not start holder ${name} for ${img}; is the image still in the daemon?"
return 1
fi
echo "OK: ${img} pinned by running container ${name} (exits by itself after 1 h)"
}
release() {
local img="${1:?usage: $0 release <image>}"
docker rm -f "$(holder_name "$img")" >/dev/null 2>&1 || true
docker image rm "$img" >/dev/null 2>&1 || true
echo "released ${img}"
}
lint_workflows() {
[ "$#" -gt 0 ] || { echo "usage: $0 lint <workflow.yml>..." >&2; return 2; }
local hits
hits=$(grep -nE '^[^#]*docker[[:space:]]+(system|image)[[:space:]]+prune' "$@" || true)
if [ -n "$hits" ]; then
echo "::error::raw docker prune in a workflow. It deletes smoke images other jobs are testing on the shared runner daemon; call 'bash scripts/ci-reclaim-docker.sh' instead."
printf '%s\n' "$hits"
return 1
fi
echo "OK: no raw 'docker system|image prune' in $# workflow file(s)"
}
case "${1:-reclaim}" in
reclaim) reclaim ;;
assert-protected) shift; assert_protected "$@" ;;
hold) shift; hold "$@" ;;
release) shift; release "$@" ;;
lint) shift; lint_workflows "$@" ;;
*) echo "usage: $0 [reclaim | assert-protected <image> | hold <image> | release <image> | lint <workflow.yml>...]" >&2; exit 2 ;;
esac
+20 -2
View File
@@ -291,12 +291,30 @@ run "image-baked pi-devbox-environment skill" \
"test -f /usr/local/share/pi-devbox/skills/pi-devbox-environment/SKILL.md"
run "image-baked credential-incident-response skill" \
"test -f /usr/local/share/pi-devbox/skills/credential-incident-response/SKILL.md"
run "image-baked pi-devbox-firstboot skill" \
"test -f /usr/local/share/pi-devbox/skills/pi-devbox-firstboot/SKILL.md"
run "global-AGENTS append snippet present" \
"test -f /usr/local/share/pi-devbox/pi-global-AGENTS.append.md"
run "pi-devbox block merged into pi-global-AGENTS.md" \
"grep -q 'pi-devbox:managed-block' /opt/pi-toolkit/pi-global-AGENTS.md"
run "mempalace session-start pointer merged into global AGENTS.md" \
"grep -q 'load the mempalace skill' /opt/pi-toolkit/pi-global-AGENTS.md"
# The firstboot pointer is asserted by PATH, not prose: a brand-new device with
# an empty mailbox reaches the acceptance round only through this line, and
# rewording the paragraph must not be able to silently break the link.
run "firstboot skill pointer merged into global AGENTS.md" \
"grep -q 'pi-devbox-firstboot/SKILL.md' /opt/pi-toolkit/pi-global-AGENTS.md"
# Same path assertion for the two skills whose descriptions were deliberately
# TRIMMED to a routing sentence. Their frontmatter no longer enumerates what
# they cover, on the explicit premise that this block names them by path and so
# they never depend on description-matching to be found. That premise is only
# true while these lines exist, which is exactly why it is asserted rather than
# trusted: reword the block, drop a pointer, and the skill silently stops being
# reachable for a device that did not already know it was there.
run "pi-devbox-environment pointer merged into global AGENTS.md" \
"grep -q 'pi-devbox-environment/SKILL.md' /opt/pi-toolkit/pi-global-AGENTS.md"
run "credential-incident-response pointer merged into global AGENTS.md" \
"grep -q 'credential-incident-response/SKILL.md' /opt/pi-toolkit/pi-global-AGENTS.md"
# Vendored fallback skills (so a no-skillset container still resolves the
# AGENTS.md 'read the pi-extensions skill' pointer).
run "image-baked pi-extensions fallback skill" \
@@ -811,7 +829,7 @@ exec_test "mempalace skill snapshot is current" 'f=$HOME/.agents/skills/mempa
# Link TARGETS, not just link existence: with no skillset mounted (as here) the
# baked tree must be what resolves, for all four vendored skills.
exec_test "vendored skills resolve to the baked tree (no skillset mounted)" \
'for s in mempalace pi-extensions pi-devbox-environment credential-incident-response; do
'for s in mempalace pi-extensions pi-devbox-environment credential-incident-response pi-devbox-firstboot; do
case "$(readlink -f $HOME/.agents/skills/$s)" in
/usr/local/share/pi-devbox/skills/$s) ;;
*) echo "$s resolves to $(readlink -f $HOME/.agents/skills/$s)" >&2; exit 1 ;;
@@ -825,7 +843,7 @@ exec_test "vendored skills resolve to the baked tree (no skillset mounted)" \
exec_test "pi-devbox-version reports skill sources (all baked, no skillset here)" \
'out=$(pi-devbox-version)
echo "$out" | grep -q "skills:" || { echo "no skills section" >&2; exit 1; }
for s in mempalace pi-extensions pi-devbox-environment credential-incident-response; do
for s in mempalace pi-extensions pi-devbox-environment credential-incident-response pi-devbox-firstboot; do
echo "$out" | grep -qE "^ $s +baked( \([^)]*\))?$" \
|| { echo "$s not reported as baked" >&2; exit 1; }
done; echo ok'