@sdd-method/author-cli 0.15.1 → 0.16.1

package/dist/lib/validate/structural-model.js

@@ -20,6 +20,9 @@
 //               element another open change adds names that change in depends_on
 //   no-op       contradicted by a behaviour delta for a feature the model does not hold; a structural delta
 //               must add that feature
+//   removals    each id a delta removes, against the rows outside the model that still name it: this SDD's catalogues
+//               built from its records and its members' catalogues (lib/structure/removals.ts). Reported as [WARN],
+//               not counted as a finding: a removal may be meant, and the warning names what it owes before the push
 // Composition (§4.5; lib/structure/compose.ts), with a change agreed when its agreement record says so:
 //   overlap     two agreed changes writing one element or adding one id; a change overlapping an agreed one
 //   composed    each change's view (with its dependency closure) and the intended view (every agreed change)
@@ -44,6 +47,7 @@ import { agreedOrder, attribute, checkDependencies, checkOverlap, closure, inten
 import { describeState, renderViews, VIEW_PATHS, viewState } from "../structure/render.js";
 import { Additions, CONTRACTS, readAgreement } from "../structure/intended.js";
 import { validateDeltas, validateView, viewOf } from "../structure/view.js";
+import { checkRemovals } from "../structure/removals.js";
 const VIEW_COUNT = Object.keys(VIEW_PATHS).length;
 /** Findings FAIL (from M-ADR 0230's acceptance); false reports them only, as while M-ADR 0230 was Proposed. Also binds a change's structural deltas as agreed content in the behaviour-store check. */
 export const ENFORCED = true;
@@ -222,6 +226,8 @@ export async function checkStructuralModel(repo, options = {}) {
         }
     }
     lines.push(`changes: ${changes.length} open, ${withDeltas} with structural deltas, ${noOps} claiming none, ${without} without the artifact`);
+    const removals = await checkRemovals(repo, schema, structures);
+    lines.push(...removals.lines);
     // Composition (M-ADR 0230 §4.5): the three views and the three checks.
     const archived = await archivedNames(repo);
     checkOverlap(open, problems);
@@ -268,7 +274,7 @@ export async function checkStructuralModel(repo, options = {}) {
     if (raw > 0)
         lines.push(`archived changes whose structure is not applied: ${raw}`);
     lines.push(`views: current; at each of ${[...open.values()].filter((o) => o.structure.deltas.length > 0).length} change(s) with deltas; intended from ${agreed.length} agreed change(s)`);
-    const body = lines.map((l) => `${l}\n`).join("");
+    const body = lines.map((l) => `${l}\n`).join("") + removals.findings.map((w) => `[WARN] ${w}\n`).join("");
     if (problems.length === 0)
         return { exitCode: 0, stdout: `structural-model:\n${body}[PASS] structural-model: ok\n` };
     const tag = enforced ? "FAIL" : "WARN";

package/dist/maintain-version.json

@@ -1,4 +1,4 @@
 {
-  "version": "0.16.1",
-  "built_against_sdd_cli": "5.2.1"
+  "version": "0.17.1",
+  "built_against_sdd_cli": "5.3.1"
 }

package/dist/verify-version.json

@@ -1,4 +1,4 @@
 {
-  "version": "5.2.1",
-  "built_against_sdd_cli": "5.2.1"
+  "version": "5.3.1",
+  "built_against_sdd_cli": "5.3.1"
 }

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@sdd-method/author-cli",
-  "version": "0.15.1",
-  "sddCliVersion": "5.2.1",
+  "version": "0.16.1",
+  "sddCliVersion": "5.3.1",
   "description": "Re-author one SDD's content — scaffold, plan, contracts, conformance, api-surface, snapshots, briefings, catalogue check — plus every read-only verify verb. Ships only the code those verbs reach; it cannot sync the baseline or rebuild the catalogues. Maintenance is in @sdd-method/maintain-cli, administration in @sdd-method/sdd-cli.",
   "license": "UNLICENSED",
   "type": "module",

package/scripts/validate-app-docs.sh

@@ -8,10 +8,16 @@
 #   1. service-architecture.md is present
 #   2. feature-tree.md is present (may be at docs/ or repo root)
 #   3. app-platform-dependencies.yaml is present
-#   4. Feature spec files under features/*/ have required sections:
+#   4. Feature spec files under features/ have required sections:
 #      - ## Overview
-#      - ## Acceptance Criteria
+#      - ## Acceptance Criteria (a prefix: the template's heading is
+#        "## Acceptance Criteria Summary"; once the feature ships it holds the
+#        pointer to its living spec, M-ADR 0221 §4.3)
 #      - ## Platform Contract Dependencies
+#   5. Feature specs sit at features/<feature>/<feature>.md, one directory per
+#      feature with its story files beside it (the app-feature-spec skill's
+#      path). A spec stored flat at features/<name>.md is reported as [WARN],
+#      advisory: it does not fail the run.
 #
 # Does NOT check:
 #   - Glossary alignment (no domain glossary in app repos)
@@ -142,6 +148,7 @@ check_feature_specs() {
     local specs_found=0
     local specs_failed=0
     local specs_passed=0
+    local specs_flat=0
 
     # Discover feature specs under features/*/
     local feature_spec
@@ -157,6 +164,12 @@ check_feature_specs() {
             fi
         done
 
+        if [[ "$(dirname "$feature_spec")" == "$APP_ROOT/features" ]]; then
+            local name; name="$(basename "$feature_spec" .md)"
+            log_warn "$rel: stored flat under features/; a feature spec's path is features/${name}/${name}.md, one directory per feature with its story files beside it (app-feature-spec skill). Advisory: reported, not failed"
+            specs_flat=$((specs_flat + 1))
+        fi
+
         if [[ "$file_failures" -eq 0 ]]; then
             log_pass "$rel: all required sections present"
             specs_passed=$((specs_passed + 1))
@@ -172,6 +185,9 @@ check_feature_specs() {
     fi
 
     log_info "Group 2: $specs_passed/$specs_found feature specs passed, $specs_failed failed"
+    if [[ "$specs_flat" -gt 0 ]]; then
+        log_warn "Group 2: $specs_flat of $specs_found feature spec(s) stored flat under features/ (advisory)"
+    fi
 }
 
 # ──────────────────────────────────────────────────────────────────────────────