From 6cb47f81f64f9bc5b1cc25cf43224d462068c57c Mon Sep 17 00:00:00 2001 From: Defame1297 Date: Sun, 30 Aug 2026 19:31:21 +0000 Subject: [PATCH] fix(apm-workflow): surface the registries precondition in compile/install MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit SKILL.md says the apm experimental enable registries precondition applies "anywhere — configure, install or publish," but only configure.md actually carried it. compile.md's Publish flow and install.md's dependency resolution can both hit a named registry and silently no-op with no error if the precondition was never run, yet neither file mentioned it — contradicting the skill's own promise that each reference file is self-contained for its concern. Add a one-line cross-reference to configure.md's Gotchas in each. Found by an independent post-closure audit of #99 (agent-audit + skill-audit re-run against every changed skill/agent). --- .../kyberforge/.apm/skills/apm-workflow/references/compile.md | 2 ++ .../kyberforge/.apm/skills/apm-workflow/references/install.md | 2 ++ plugins/kyberforge/skills/apm-workflow/references/compile.md | 2 ++ plugins/kyberforge/skills/apm-workflow/references/install.md | 2 ++ 4 files changed, 8 insertions(+) diff --git a/plugins/kyberforge/.apm/skills/apm-workflow/references/compile.md b/plugins/kyberforge/.apm/skills/apm-workflow/references/compile.md index d9b77d9..ae801ea 100644 --- a/plugins/kyberforge/.apm/skills/apm-workflow/references/compile.md +++ b/plugins/kyberforge/.apm/skills/apm-workflow/references/compile.md @@ -51,6 +51,8 @@ apm publish --package acme/my-skill Publishes a producer package (root containing `apm.yml`, `.apm/`, and optionally a `registries:` block) to a registry. Always dry-run with `-v` first — publishing is not trivially reversible once a version tag is claimed on a registry. +Publishing to a named registry requires `apm experimental enable registries` to have already run — see `references/configure.md`'s Gotchas for the full precondition and its silent-no-op failure mode. + ## Run ```bash diff --git a/plugins/kyberforge/.apm/skills/apm-workflow/references/install.md b/plugins/kyberforge/.apm/skills/apm-workflow/references/install.md index 760c44e..8f30470 100644 --- a/plugins/kyberforge/.apm/skills/apm-workflow/references/install.md +++ b/plugins/kyberforge/.apm/skills/apm-workflow/references/install.md @@ -18,3 +18,5 @@ With no arguments, resolves and installs everything declared under `dependencies `--update` is the escape hatch for a lockfile hash mismatch against upstream — normal `apm install` treats that as drift and won't silently accept it; see `references/audit.md` for the CI-side check (`apm install --frozen`) that fails instead of re-resolving. `--target agent-skills` generates the vendor-neutral output directory instead of a Claude/Copilot-specific one — for IDE-agnostic tool support. + +If a `PACKAGE_REF` resolves through a named registry rather than a plain git source, `apm experimental enable registries` must already have been run — see `references/configure.md`'s Gotchas for the full precondition and its silent-no-op failure mode. diff --git a/plugins/kyberforge/skills/apm-workflow/references/compile.md b/plugins/kyberforge/skills/apm-workflow/references/compile.md index d9b77d9..ae801ea 100644 --- a/plugins/kyberforge/skills/apm-workflow/references/compile.md +++ b/plugins/kyberforge/skills/apm-workflow/references/compile.md @@ -51,6 +51,8 @@ apm publish --package acme/my-skill Publishes a producer package (root containing `apm.yml`, `.apm/`, and optionally a `registries:` block) to a registry. Always dry-run with `-v` first — publishing is not trivially reversible once a version tag is claimed on a registry. +Publishing to a named registry requires `apm experimental enable registries` to have already run — see `references/configure.md`'s Gotchas for the full precondition and its silent-no-op failure mode. + ## Run ```bash diff --git a/plugins/kyberforge/skills/apm-workflow/references/install.md b/plugins/kyberforge/skills/apm-workflow/references/install.md index 760c44e..8f30470 100644 --- a/plugins/kyberforge/skills/apm-workflow/references/install.md +++ b/plugins/kyberforge/skills/apm-workflow/references/install.md @@ -18,3 +18,5 @@ With no arguments, resolves and installs everything declared under `dependencies `--update` is the escape hatch for a lockfile hash mismatch against upstream — normal `apm install` treats that as drift and won't silently accept it; see `references/audit.md` for the CI-side check (`apm install --frozen`) that fails instead of re-resolving. `--target agent-skills` generates the vendor-neutral output directory instead of a Claude/Copilot-specific one — for IDE-agnostic tool support. + +If a `PACKAGE_REF` resolves through a named registry rather than a plain git source, `apm experimental enable registries` must already have been run — see `references/configure.md`'s Gotchas for the full precondition and its silent-no-op failure mode.