@warnyin/sdlc 0.5.0 → 0.5.2

package/CHANGELOG.md

@@ -1,5 +1,30 @@
 # Changelog
 
+## 0.5.2 (2026-08-25)
+
+- **Fix (delta)**: a `### MODIFIED Requirement:` body replaces the requirement wholesale,
+  so one that carried over only some of the spec's scenarios dropped the rest in silence —
+  no error, no warning, `spec merged` printed either way. Both shapes are now reported:
+  the scenario name gone from the replacement body, and the name surviving while WHEN/THEN
+  clauses it promised have no counterpart (the one a name-level comparison cannot see).
+  `archive` prints the report before it writes a byte and counts it in the summary;
+  `validate` reports the same at warn level, so the loss is visible while the change folder
+  is still readable rather than after ship archived it. A warning, never an error — removing
+  a scenario is sometimes the point of the change, and only the silence was ever the bug.
+  A reworded clause reports the same as a deleted one: nothing mechanical can tell "said
+  better" from "promises less". Cosmetic churn — indentation, bullet marker, clause order,
+  heading case, whitespace — is normalized away and never warns. (#1)
+
+## 0.5.1 (2026-08-25)
+
+- **Fix (cost)**: `costUsd()` never charged cache-write tokens, the highest-rate of
+  the four classes the usage parser collects. Every cost `/sdlc:observe` and the
+  session summary have printed was therefore low. A model priced without a
+  `cacheWrite` rate now charges nothing for that class rather than inferring one from
+  `input` — the module's rule is never to guess a price, and a guess reports as
+  confidently as a known rate. Existing journalled costs are left alone: backfilling
+  would rewrite history from a rate that was not in force at the time.
+
 ## 0.5.0 (2026-08-25)
 
 - **`--auto` on every pipeline stage.** `/sdlc:auto` already ran the whole pipeline,

package/LICENSE

@@ -1,21 +1,21 @@
-MIT License
-
-Copyright (c) 2026 warnyin
-
-Permission is hereby granted, free of charge, to any person obtaining a copy
-of this software and associated documentation files (the "Software"), to deal
-in the Software without restriction, including without limitation the rights
-to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
-copies of the Software, and to permit persons to whom the Software is
-furnished to do so, subject to the following conditions:
-
-The above copyright notice and this permission notice shall be included in all
-copies or substantial portions of the Software.
-
-THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
-IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
-FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
-AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
-LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
-OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
-SOFTWARE.
+MIT License
+
+Copyright (c) 2026 warnyin
+
+Permission is hereby granted, free of charge, to any person obtaining a copy
+of this software and associated documentation files (the "Software"), to deal
+in the Software without restriction, including without limitation the rights
+to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+copies of the Software, and to permit persons to whom the Software is
+furnished to do so, subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included in all
+copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+SOFTWARE.

package/bin/cli.mjs

@@ -479,14 +479,22 @@ export function cmdArchive(projectRoot, changeId, { strict = true } = {}) {
 
   // Phase 1: compute every merge before writing anything (all-or-nothing).
   const merged = [];
+  const driftWarnings = [];
   for (const d of deltas) {
     const specPath = path.join(sdlcRoot, 'specs', d.capability, 'spec.md');
     const specText = fs.existsSync(specPath) ? fs.readFileSync(specPath, 'utf8') : null;
     const result = mergeDelta(specText, d.ops, d.capability);
     if (!result.ok) throw new Error(`spec merge failed for "${d.capability}": ${result.errors.join('; ')}`);
+    driftWarnings.push(...result.warnings);
     merged.push({ specPath, content: result.content, capability: d.capability });
   }
 
+  // A MODIFIED body replaces the requirement wholesale, so it can carry away a
+  // scenario the spec still promised. That is allowed — but it is said out loud
+  // here, while the change folder is still readable, not discovered in a diff
+  // after the folder moved under changes/archive/.
+  for (const w of driftWarnings) console.error(`⚠ ${w}`);
+
   // Phase 2: write specs, promote evals, stamp status, move to archive.
   for (const m of merged) writeFileNormalized(m.specPath, m.content);
 
@@ -508,8 +516,11 @@ export function cmdArchive(projectRoot, changeId, { strict = true } = {}) {
 
   console.log(`shipped: ${changeId}`);
   for (const m of merged) console.log(`  spec merged: specs/${m.capability}/spec.md`);
+  if (driftWarnings.length) {
+    console.log(`  ⚠ ${driftWarnings.length} scenario warning(s) above — re-read the spec diff before pushing`);
+  }
   console.log(`  archived: changes/archive/${date}-${changeId}/`);
-  return { archived: `${date}-${changeId}`, specs: merged.map((m) => m.capability) };
+  return { archived: `${date}-${changeId}`, specs: merged.map((m) => m.capability), warnings: driftWarnings };
 }
 
 // ---------- shared ----------

package/lib/caps.mjs

@@ -1,45 +1,45 @@
-// Canonical line-cap table — the single source of truth for artifact
-// residency budgets. Templates quote these numbers in HTML comments and
-// tests/caps-sync.test.mjs asserts both stay identical.
-//
-// Rationale (Day-1 economics): every resident line is paid for in every
-// turn. Caps make "learning = distilling" structural, not disciplinary.
-
-import { parseFrontmatter } from './frontmatter.mjs';
-
-export const CAPS = Object.freeze({
-  constitution: 30,     // sdlc/context/constitution.md — always loaded
-  steeringFile: 40,     // each sdlc/context/steering/*.md
-  alwaysBudget: 60,     // constitution + all `inclusion: always` steering, combined
-  harness: 60,          // sdlc/harness.md
-  spec: 150,            // sdlc/specs/<capability>/spec.md (soft — split capability beyond)
-  change: Object.freeze({ vibe: 40, standard: 100, deep: 150 }),
-  contractTests: 60,    // changes/<id>/contract/tests.md
-  contractEvals: 40,    // changes/<id>/contract/evals.md
-});
-
-export const TIERS = Object.freeze(['vibe', 'standard', 'deep']);
-
-export const STATUSES = Object.freeze([
-  'new', 'contracted', 'building', 'verified', 'shipped',
-]);
-
-// Effective lines = body lines after frontmatter, excluding blanks and
-// single-line HTML comments. Caps meter prose the model must carry, not
-// machine metadata or annotation comments.
-export function countEffectiveLines(text) {
-  const { body } = parseFrontmatter(text ?? '');
-  return body
-    .split(/\r?\n/)
-    .filter((line) => {
-      const t = line.trim();
-      if (t === '') return false;
-      if (t.startsWith('<!--') && t.endsWith('-->')) return false;
-      return true;
-    })
-    .length;
-}
-
-export function capForChange(tier) {
-  return CAPS.change[tier] ?? CAPS.change.standard;
-}
+// Canonical line-cap table — the single source of truth for artifact
+// residency budgets. Templates quote these numbers in HTML comments and
+// tests/caps-sync.test.mjs asserts both stay identical.
+//
+// Rationale (Day-1 economics): every resident line is paid for in every
+// turn. Caps make "learning = distilling" structural, not disciplinary.
+
+import { parseFrontmatter } from './frontmatter.mjs';
+
+export const CAPS = Object.freeze({
+  constitution: 30,     // sdlc/context/constitution.md — always loaded
+  steeringFile: 40,     // each sdlc/context/steering/*.md
+  alwaysBudget: 60,     // constitution + all `inclusion: always` steering, combined
+  harness: 60,          // sdlc/harness.md
+  spec: 150,            // sdlc/specs/<capability>/spec.md (soft — split capability beyond)
+  change: Object.freeze({ vibe: 40, standard: 100, deep: 150 }),
+  contractTests: 60,    // changes/<id>/contract/tests.md
+  contractEvals: 40,    // changes/<id>/contract/evals.md
+});
+
+export const TIERS = Object.freeze(['vibe', 'standard', 'deep']);
+
+export const STATUSES = Object.freeze([
+  'new', 'contracted', 'building', 'verified', 'shipped',
+]);
+
+// Effective lines = body lines after frontmatter, excluding blanks and
+// single-line HTML comments. Caps meter prose the model must carry, not
+// machine metadata or annotation comments.
+export function countEffectiveLines(text) {
+  const { body } = parseFrontmatter(text ?? '');
+  return body
+    .split(/\r?\n/)
+    .filter((line) => {
+      const t = line.trim();
+      if (t === '') return false;
+      if (t.startsWith('<!--') && t.endsWith('-->')) return false;
+      return true;
+    })
+    .length;
+}
+
+export function capForChange(tier) {
+  return CAPS.change[tier] ?? CAPS.change.standard;
+}

package/lib/config.mjs

@@ -1,41 +1,41 @@
-// Minimal reader for sdlc/config.yaml — supports exactly what the template
-// documents: scalar keys, inline arrays, and a `prices:` block of inline
-// objects. Anything else is ignored (never crash a hook on config).
-
-export function parseConfig(text) {
-  const config = { language: 'en', tools: [], prices: null };
-  const lines = (text ?? '').split(/\r?\n/);
-  let inPrices = false;
-  for (const raw of lines) {
-    if (!raw.trim() || raw.trim().startsWith('#')) continue;
-    const isIndented = /^\s/.test(raw);
-    if (!isIndented) inPrices = false;
-
-    if (inPrices) {
-      const m = raw.match(/^\s+([^:#]+):\s*\{(.*)\}\s*$/);
-      if (!m) continue;
-      const model = m[1].trim();
-      const obj = {};
-      for (const part of m[2].split(',')) {
-        const kv = part.split(':');
-        if (kv.length !== 2) continue;
-        const num = Number(kv[1].trim());
-        if (!Number.isNaN(num)) obj[kv[0].trim()] = num;
-      }
-      config.prices = { ...(config.prices ?? {}), [model]: obj };
-      continue;
-    }
-
-    const kv = raw.match(/^([A-Za-z0-9_-]+):\s*(.*)$/);
-    if (!kv) continue;
-    const [, key, valueRaw] = kv;
-    const value = valueRaw.replace(/\s+#.*$/, '').trim();
-    if (key === 'prices' && value === '') { inPrices = true; continue; }
-    if (value.startsWith('[') && value.endsWith(']')) {
-      config[key] = value.slice(1, -1).split(',').map((s) => s.trim()).filter(Boolean);
-    } else if (value !== '') {
-      config[key] = value;
-    }
-  }
-  return config;
-}
+// Minimal reader for sdlc/config.yaml — supports exactly what the template
+// documents: scalar keys, inline arrays, and a `prices:` block of inline
+// objects. Anything else is ignored (never crash a hook on config).
+
+export function parseConfig(text) {
+  const config = { language: 'en', tools: [], prices: null };
+  const lines = (text ?? '').split(/\r?\n/);
+  let inPrices = false;
+  for (const raw of lines) {
+    if (!raw.trim() || raw.trim().startsWith('#')) continue;
+    const isIndented = /^\s/.test(raw);
+    if (!isIndented) inPrices = false;
+
+    if (inPrices) {
+      const m = raw.match(/^\s+([^:#]+):\s*\{(.*)\}\s*$/);
+      if (!m) continue;
+      const model = m[1].trim();
+      const obj = {};
+      for (const part of m[2].split(',')) {
+        const kv = part.split(':');
+        if (kv.length !== 2) continue;
+        const num = Number(kv[1].trim());
+        if (!Number.isNaN(num)) obj[kv[0].trim()] = num;
+      }
+      config.prices = { ...(config.prices ?? {}), [model]: obj };
+      continue;
+    }
+
+    const kv = raw.match(/^([A-Za-z0-9_-]+):\s*(.*)$/);
+    if (!kv) continue;
+    const [, key, valueRaw] = kv;
+    const value = valueRaw.replace(/\s+#.*$/, '').trim();
+    if (key === 'prices' && value === '') { inPrices = true; continue; }
+    if (value.startsWith('[') && value.endsWith(']')) {
+      config[key] = value.slice(1, -1).split(',').map((s) => s.trim()).filter(Boolean);
+    } else if (value !== '') {
+      config[key] = value;
+    }
+  }
+  return config;
+}