From d234f40134bcbc0c841810dcacae30ae92c0aa2b Mon Sep 17 00:00:00 2001 From: Marcos Paulo Date: Sat, 5 Sep 2026 17:11:48 +0000 Subject: [PATCH] fix: stop leaking a task number into site copy, ignore legacy sources Two unrelated cleanups from the 14-17 wave. The review desk footer told visitors to mirror entries into catalog.js 'until task 16 rewires the page to read the collection'. Introduced by b484302 (task 06), it shipped in the built HTML. The mirroring advice is still correct -- verify.mjs:24,52,54 confirm the desk reads catalog.js -- so only the internal task reference is dropped. .prettierignore now covers the legacy sources. They have very long lines, so lint-staged re-wraps them wholesale as soon as an agent stages one: task 15 added four lines to app.js and produced an 829-line diff. Paths are root-anchored so a bare 'rules' does not swallow .agents/rules/. --- .agents/snapshots/skills-review.txt | 2 +- .prettierignore | 20 ++++++++++++++++++++ skills-review/index.html | 2 +- src/pages/skills-review.astro | 7 +++---- 4 files changed, 25 insertions(+), 6 deletions(-) diff --git a/.agents/snapshots/skills-review.txt b/.agents/snapshots/skills-review.txt index 5364a19..dc9723e 100644 --- a/.agents/snapshots/skills-review.txt +++ b/.agents/snapshots/skills-review.txt @@ -45,7 +45,7 @@ submitted-skills/ src/content/reviews/{new-id}.md (and mirror it into skills-review/catalog.js -until task 16 rewires the page to read the collection), then run +which the desk still reads), then run node scripts/build-skill-review.mjs . Votes call a separate service — see vote-service/ diff --git a/.prettierignore b/.prettierignore index 1b91249..04913b6 100644 --- a/.prettierignore +++ b/.prettierignore @@ -7,3 +7,23 @@ skill-reviews vote-service .agents/snapshots pnpm-lock.yaml +public/submitted-skills + +# Legacy site sources, slated for deletion at cutover (task 20). These are +# hand-written files with very long lines; prettier re-wraps them into hundreds +# of changed lines the moment any agent stages one. Task 15 touched app.js to +# add four lines and produced an 829-line diff. verify.mjs asserts substrings +# against several of these, so a reformat is churn at best and a broken +# assertion at worst. +# +# Root-anchored on purpose: a bare `rules` would also swallow .agents/rules/, +# whose markdown we do want formatted. +/app.js +/styles.css +/landing.css +/chapters.css +/responsive.css +/skills-review/ +/rules/ +/skills/ +/full-guide/ diff --git a/skills-review/index.html b/skills-review/index.html index 807b459..021d316 100644 --- a/skills-review/index.html +++ b/skills-review/index.html @@ -38,7 +38,7 @@

The recommendations follow the open Agent Skills format: valid frontmatter for discovery, progressive disclosure for context economy, deterministic scripts for fragile repeated mechanics, and behavioral evaluation rather than a checklist of pretty headings.

Format specification ↗Writing practices ↗Evaluation loop ↗Scripts guide ↗
- + diff --git a/src/pages/skills-review.astro b/src/pages/skills-review.astro index 1bd01b5..45795b3 100644 --- a/src/pages/skills-review.astro +++ b/src/pages/skills-review.astro @@ -142,10 +142,9 @@ const serializedCatalog = JSON.stringify(catalog).replace(/. To add a submission later: drop a package under submitted-skills/, add an entry under src/content/reviews/{'{'}new-id{'}'}.md (and mirror it into skills-review/catalog.js until task 16 rewires the page to read the collection), then run node scripts/build-skill-review.mjs. Votes call a separate service — see vote-service/ — one per visitor, tracked by - network source. + > which the desk still reads), then run node scripts/build-skill-review.mjs. + Votes call a separate service — see vote-service/ — one per visitor, tracked by network + source.