@bongos/core 1.20.9 → 1.20.10

package/modules/ui-design/skills/high-end-visual-design/SKILL.md

@@ -5,6 +5,12 @@ description: >-
   one layout archetype per project, the container anatomy, the pill button, the spatial rhythm, the motion
   choreography inside the Kill Switch, the mobile collapse, then a pre-output checklist. Triggers on "make it
   feel premium", "agency-grade", "this looks cheap", "the polish pass".
+plain: >-
+  Makes a page look expensive and professionally made, while staying in the project's own style.
+reach-for: >-
+  When a page looks cheap or unfinished.
+cost: >-
+  Uses your session. It changes the page, and you check it on a preview.
 ---
 
 <!-- BEGIN PLATFORM PREAMBLE · ui-design module · one block, byte-identical in every skill the module ships (vendored or rebuilt in-house) so an upstream refresh is a three-way merge; the source is modules/ui-design/skills/PREAMBLE.md and tests/ui_design_skills.mjs fails on drift -->

package/modules/ui-design/skills/image-to-code/SKILL.md

@@ -5,6 +5,12 @@ description: >-
   design source of truth is the owner's reference images and the instance's world, read per image and
   extracted into a full spec before a line is written. Triggers on "build this from the reference", "implement
   the mock faithfully". Never generates a section, comp or screen.
+plain: >-
+  Builds a page faithfully from reference pictures you provide, studying each one first.
+reach-for: >-
+  When you have images of how a page should look and want it built to match.
+cost: >-
+  Uses your session. It builds the page and never generates images itself.
 ---
 
 <!-- BEGIN PLATFORM PREAMBLE · ui-design module · one block, byte-identical in every skill the module ships (vendored or rebuilt in-house) so an upstream refresh is a three-way merge; the source is modules/ui-design/skills/PREAMBLE.md and tests/ui_design_skills.mjs fails on drift -->

package/modules/ui-design/skills/imagegen-frontend-mobile/SKILL.md

@@ -5,6 +5,12 @@ description: >-
   portrait plates and hero objects for the 390 and 320 compositions the kit renders, sized for the fold and
   clear of what a phone reserves, with a sidecar beside every file. Triggers on "a plate for the mobile hero",
   "the masthead at 390". Never a screen, a phone mockup or code.
+plain: >-
+  Makes the large picture at the top of a page, sized for phones, with no text in it.
+reach-for: >-
+  When a page's phone layout needs its main picture.
+cost: >-
+  Paid: each image is made on the project's image key. It adds the picture files to the project.
 ---
 
 <!-- BEGIN PLATFORM PREAMBLE · ui-design module · one block, byte-identical in every skill the module ships (vendored or rebuilt in-house) so an upstream refresh is a three-way merge; the source is modules/ui-design/skills/PREAMBLE.md and tests/ui_design_skills.mjs fails on drift -->

package/modules/ui-design/skills/imagegen-frontend-web/SKILL.md

@@ -5,6 +5,12 @@ description: >-
   objects and atmosphere backdrops for the slots a composition already has — never one per section, never a
   screen, never a UI comp, and no page code. One material, light and camera across the family, landed with a
   provenance sidecar. Triggers on "make a hero plate", "an atmosphere backdrop".
+plain: >-
+  Makes the large background or hero pictures for a web page, with no text in them.
+reach-for: >-
+  When a page needs its main picture or a background image.
+cost: >-
+  Paid: each image is made on the project's image key. It adds the picture files to the project.
 ---
 
 <!-- BEGIN PLATFORM PREAMBLE · ui-design module · one block, byte-identical in every skill the module ships (vendored or rebuilt in-house) so an upstream refresh is a three-way merge; the source is modules/ui-design/skills/PREAMBLE.md and tests/ui_design_skills.mjs fails on drift -->

package/modules/ui-design/skills/impeccable/SKILL.md

@@ -5,6 +5,12 @@ description: >-
   ("impeccable audit /projects"). Sixteen playbooks — shape, audit, critique, polish, harden, animate,
   typeset, colorize, layout, adapt, distill, clarify, onboard, optimize, document, extract — one per request.
   Triggers on "impeccable", "critique this page", "audit the UI". Web surfaces only.
+plain: >-
+  A toolbox of design checks and fixes for a web page, such as a review, a polish, better type or better layout, one job at a time.
+reach-for: >-
+  When you want a page reviewed or improved in one specific way.
+cost: >-
+  Uses your session. A review changes nothing; a fix changes the page.
 ---
 
 <!-- BEGIN PLATFORM PREAMBLE · ui-design module · one block, byte-identical in every skill the module ships (vendored or rebuilt in-house) so an upstream refresh is a three-way merge; the source is modules/ui-design/skills/PREAMBLE.md and tests/ui_design_skills.mjs fails on drift -->

package/modules/ui-design/skills/industrial-brutalist-ui/SKILL.md

@@ -5,6 +5,12 @@ description: >-
   per project (Swiss industrial print or dark tactical terminal, never mixed), massive uppercase display
   against tracked-out mono metadata, strict grid anchoring, zero corner rounding, halftone and scanline
   degradation inside the Kill Switch. Triggers on "brutalist", "industrial", "make it raw".
+plain: >-
+  Gives a page a raw, mechanical look, like a technical document or a control terminal.
+reach-for: >-
+  When you want a bold, stark look.
+cost: >-
+  Uses your session. It changes the page, and you check it on a preview.
 ---
 
 <!-- BEGIN PLATFORM PREAMBLE · ui-design module · one block, byte-identical in every skill the module ships (vendored or rebuilt in-house) so an upstream refresh is a three-way merge; the source is modules/ui-design/skills/PREAMBLE.md and tests/ui_design_skills.mjs fails on drift -->

package/modules/ui-design/skills/minimalist-ui/SKILL.md

@@ -5,6 +5,12 @@ description: >-
   whitespace first, a capped measure, one display job and one body job, exactly one hairline weight for every
   border, flat tile cards with no shadow, entry motion inside the Kill Switch. Triggers on "make it minimal",
   "clean and editorial", "this feels busy". Declines dense surfaces.
+plain: >-
+  Gives a page a calm, clean look built from text and white space.
+reach-for: >-
+  When a page feels too busy.
+cost: >-
+  Uses your session. It changes the page, and you check it on a preview.
 ---
 
 <!-- BEGIN PLATFORM PREAMBLE · ui-design module · one block, byte-identical in every skill the module ships (vendored or rebuilt in-house) so an upstream refresh is a three-way merge; the source is modules/ui-design/skills/PREAMBLE.md and tests/ui_design_skills.mjs fails on drift -->

package/modules/ui-design/skills/redesign-existing-projects/SKILL.md

@@ -5,6 +5,12 @@ description: >-
   breaking it. Three fixed steps — scan the stack, diagnose against a written audit, apply targeted fixes —
   with the instance's world as the answer key and a fixed impact-per-risk fix order. Triggers on "redesign
   this", "upgrade the existing page", "audit this UI".
+plain: >-
+  Improves the look of an existing site in place, without rebuilding it or breaking what works.
+reach-for: >-
+  When an existing page or app needs a visual upgrade.
+cost: >-
+  Uses your session. It changes the existing pages, and you check them on a preview.
 ---
 
 <!-- BEGIN PLATFORM PREAMBLE · ui-design module · one block, byte-identical in every skill the module ships (vendored or rebuilt in-house) so an upstream refresh is a three-way merge; the source is modules/ui-design/skills/PREAMBLE.md and tests/ui_design_skills.mjs fails on drift -->

package/modules/ui-design/skills/stitch-design-taste/SKILL.md

@@ -5,6 +5,12 @@ description: >-
   platform's own frontmatter shape. Four dials with named bands and stated defaults, then eight fixed sections
   pairing descriptive language with exact values. The thirteen colours are transcribed from the instance's
   pack, never invented. Triggers on "a design brief for Stitch".
+plain: >-
+  Writes a design brief in the project's style for an outside screen-design tool such as Google Stitch.
+reach-for: >-
+  When you want to design screens in Stitch using the project's look.
+cost: >-
+  Uses your session. It writes one brief document and changes no pages.
 ---
 
 <!-- BEGIN PLATFORM PREAMBLE · ui-design module · one block, byte-identical in every skill the module ships (vendored or rebuilt in-house) so an upstream refresh is a three-way merge; the source is modules/ui-design/skills/PREAMBLE.md and tests/ui_design_skills.mjs fails on drift -->

package/modules/ui-design/skills/style/SKILL.md

@@ -5,6 +5,12 @@ description: >-
   the fifteen-token contract, and proves it clean before it lands. Four steps and no more — SEARCH, CREATE,
   PROVE, ADOPT. Triggers on "/style", "make a look", "author a style", "I want our own theme", "add a style to
   the library".
+plain: >-
+  A short guided session for creating your own look for the project, checked before it is used.
+reach-for: >-
+  When you want the project to have its own theme.
+cost: >-
+  Uses your session. The new look is only used once you choose to adopt it.
 ---
 
 <!-- BEGIN PLATFORM PREAMBLE · ui-design module · one block, byte-identical in every skill the module ships (vendored or rebuilt in-house) so an upstream refresh is a three-way merge; the source is modules/ui-design/skills/PREAMBLE.md and tests/ui_design_skills.mjs fails on drift -->

package/package-lock.json

@@ -1,12 +1,12 @@
 {
   "name": "@bongos/core",
-  "version": "1.20.9",
+  "version": "1.20.10",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "@bongos/core",
-      "version": "1.20.9",
+      "version": "1.20.10",
       "license": "AGPL-3.0-or-later",
       "dependencies": {
         "express": "^4.21.2",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bongos/core",
-  "version": "1.20.9",
+  "version": "1.20.10",
   "description": "Cloud Bongos — the AI-first build platform core (GDS + platform surfaces + module system), installed as a versioned dependency (ADR 0108).",
   "license": "AGPL-3.0-or-later",
   "main": "src/platform-server.js",

package/release-notes.json

@@ -8106,5 +8106,11 @@
       "id": "1003268",
       "text": "Added three database lookups so the 'who have I worked near' view stays fast as the project grows. Nothing changes on screen today."
     }
+  ],
+  "1.20.10": [
+    {
+      "id": "1004403",
+      "text": "When you adopt a speciality, each skill it offers is now explained in plain words: what it does, when you would use it, and what it costs, including which ones spend money. Before, those explanations were guessed from technica"
+    }
   ]
 }

package/src/module-api.js

@@ -75,7 +75,7 @@ const { responsibilityFor, ROLE_RESPONSIBILITIES } = require('./role-responsibil
 // MAJOR (see allowBoxScope below): passes the request through untouched.
 function deprecatedNoopMiddleware(_req, _res, next) { next(); }
 
-const CORE_VERSION = '1.20.9'; // CI auto-patch carrier (ADR 0161); changelog: docs/module-api-changelog.md
+const CORE_VERSION = '1.20.10'; // CI auto-patch carrier (ADR 0161); changelog: docs/module-api-changelog.md
 
 // A namespaced logger so a module's log lines are attributable + consistent.
 // Usage: const log = api.logger('discord'); log.info('mounted');

package/tests/speciality_skills.mjs

@@ -73,6 +73,48 @@ await test('every skill in the real tree reads a real description — none is a
   assert.deepEqual(bad, [], `these would teach nothing in the walkthrough: ${bad.join(', ')}`);
 });
 
+// task 1004403: every skill a speciality could offer — every core skill and
+// every module's, enabled here or not — carries its own people-facing lines, so
+// the walkthrough never has to derive them and a new skill cannot arrive without.
+const { JARGON_TERMS } = require('../modules/sessions/accessibility.js');
+// The words task 1004403 named as the ones the derived text leaked, plus the
+// repo nouns a newcomer would not know.
+const SKILL_JARGON = ['DB', 'grep', 'grepping', 'rank-scoped', 'status=', 'residue', 'criterion', 'criteria', 'touches[]', 'branch', 'commit', 'diff', 'SHA', 'CI', 'repo', 'PR'];
+const everySkill = () => skills.installedSkills({ coreRoot: ROOT, isModuleEnabled: () => true });
+
+await test('every installable skill carries plain, reach-for and cost — none is derived', () => {
+  const all = [...everySkill().values()];
+  assert.ok(all.length >= 60, `the whole tree was read (${all.length})`);
+  const missing = all.flatMap((s) => ['plain', 'reachFor', 'cost'].filter((k) => !s[k] || s[k].length < 15).map((k) => `${s.name}.${k}`));
+  assert.deepEqual(missing, [], 'add plain: / reach-for: / cost: to the SKILL.md frontmatter, written for someone with no engineering background');
+  const derived = all.filter((s) => { const d = skills.explainSkill(s).derived; return d.does || d.reachFor || d.cost; }).map((s) => s.name);
+  assert.deepEqual(derived, [], 'the walkthrough shows the author\'s own words for every skill');
+});
+
+await test('the people-facing lines stay out of the vocabulary they exist to translate', () => {
+  const banned = [...JARGON_TERMS, ...SKILL_JARGON];
+  const hits = [];
+  for (const s of everySkill().values()) {
+    for (const k of ['plain', 'reachFor', 'cost']) {
+      for (const term of banned) {
+        const re = new RegExp(`(^|[^A-Za-z])${term.replace(/[.*+?^${}()|[\]\\]/g, '\\$&')}([^A-Za-z]|$)`, /^[A-Z+-]+$/.test(term) ? '' : 'i');
+        if (re.test(s[k] || '')) hits.push(`${s.name}.${k}: "${term}"`);
+      }
+    }
+  }
+  assert.deepEqual(hits, []);
+});
+
+await test('a skill with a paid step says so in its cost line', () => {
+  // The ones whose run spends money: every hand-in pays for a quality review,
+  // and image generation is billed per image. Named, so dropping the word from
+  // one of them is caught rather than trusted.
+  const paid = ['builder-ship', 'fix-task', 'builder-sequence', 'grade-recover', 'tweak', 'otb-tile-generate', 'imagegen-frontend-web', 'imagegen-frontend-mobile'];
+  const all = everySkill();
+  const quiet = paid.filter((n) => !/\bpaid\b|costs money/i.test((all.get(n) || {}).cost || ''));
+  assert.deepEqual(quiet, []);
+});
+
 await test('explainSkill splits a model-facing description into what it does, when, and what it costs', () => {
   const e = skills.explainSkill({
     name: 'goal-review', source: 'core',