LearnNewsExamplesServices
Frontmatter
id12916
titleFix stale lint-skill-manifest path in create-skill guide + ADR 0008
stateClosed
labels
bugdocumentationai
assigneesneo-opus-grace
createdAtJun 11, 2026, 10:32 PM
updatedAtJun 11, 2026, 11:35 PM
githubUrlhttps://github.com/neomjs/neo/issues/12916
authorneo-opus-grace
commentsCount0
parentIssuenull
subIssues[]
subIssuesCompleted0
subIssuesTotal0
blockedBy[]
blocking[]
closedAtJun 11, 2026, 11:35 PM

Fix stale lint-skill-manifest path in create-skill guide + ADR 0008

Closed v13.0.0/archive-v13-0-0-chunk-17 bugdocumentationai
neo-opus-grace
neo-opus-grace commented on Jun 11, 2026, 10:32 PM

Context

During PR #12912 (peer-naming skill authoring, 2026-06-11), following the create-skill authoring guide verbatim threw Error: Cannot find module '.../ai/scripts/lint-skill-manifest.mjs' (MODULE_NOT_FOUND). The documented lint command points at a path that no longer exists.

The Problem

The skill-manifest lint script lives at ai/scripts/lint/lint-skill-manifest.mjs (note the lint/ subdir), but several live, agent-consumed docs still reference the pre-lint/ path ai/scripts/lint-skill-manifest.mjs. Any agent running the documented command verbatim hits MODULE_NOT_FOUND and must re-derive the real path. The CI workflow itself already uses the correct path (so CI is unaffected — the drift is docs-only), which is exactly why it went unnoticed.

The Architectural Reality

Live (non-archive) stale references, verified via grep -rn 'ai/scripts/lint-skill-manifest':

  • .agents/skills/create-skill/references/skill-authoring-guide.md:183 — the Verification-checklist command (the one that bit #12912).
  • learn/agentos/decisions/0008-skill-anatomy-and-authoring-contract.md:77, 125, 145 — ADR 0008, the canonical skill-anatomy decision substrate (3 refs).
  • test/playwright/unit/ai/scripts/lint/lintSkillManifest.spec.mjs:22 — a test.describe(...) label (cosmetic; the test file's own path already carries lint/).

Real path confirmed: ls ai/scripts/lint/lint-skill-manifest.mjs → exists.

The Fix

Replace ai/scripts/lint-skill-manifest.mjsai/scripts/lint/lint-skill-manifest.mjs (and the bare describe-label) across the 5 live references. One coherent cleanup, one PR (not micro-PRs).

Decision Record impact

aligned-with ADR 0008 — a factual path correction within the ADR's prose; no change to the decision itself.

Acceptance Criteria

  • skill-authoring-guide.md:183 lint command uses ai/scripts/lint/lint-skill-manifest.mjs.
  • ADR 0008's 3 references corrected to the lint/ path.
  • The test describe label aligned (cosmetic).
  • grep -rn 'ai/scripts/lint-skill-manifest' over non-archive paths returns zero.
  • node ai/scripts/lint/lint-skill-manifest.mjs --base origin/dev runs (proves the corrected command works).
  • resources/content/** archive snapshots left untouched.

Out of Scope

  • The ~55 resources/content/{discussions,pulls}/** archive references (frozen historical record — they document what was written at the time).
  • Any change to the lint script itself or the CI workflow (both already correct).

Related

  • Empirical anchor: PR #12912 (peer-naming skill authoring). Surfaced as a friction→gold spawned-task during that PR.

Release classification: boardless — substrate-doc hygiene, non-blocking.

Origin Session ID: 0bc22bf2-9ded-4cdb-b351-182c3ab1d16f

Retrieval Hint: "stale lint-skill-manifest path create-skill guide ADR 0008 MODULE_NOT_FOUND"

tobiu referenced in commit 03ecb99 - "fix(docs): correct stale lint-skill-manifest path in create-skill guide + ADR 0008 (#12916) (#12917) on Jun 11, 2026, 11:35 PM
tobiu closed this issue on Jun 11, 2026, 11:35 PM