qfai 1.2.14 → 1.3.0

package/assets/init/.qfai/assistant/instructions/workflow.md

@@ -14,6 +14,22 @@ This file defines the canonical stages and delegation expectations.
 
 ---
 
+## Change Type (Mandatory)
+
+At the start of any work, classify the change and record it in:
+
+- `delta.md` Change Log (latest CL entry)
+- PR description (Change Type section)
+
+Allowed values:
+
+- Primary: `Initial | Behavior | Structural | Ops`
+- Tags (optional): `@ui @api @db @nfr @docs @test`
+
+Do not proceed without a declared Change Type.
+
+---
+
 ## Stages (canonical)
 
 0. Steering refresh (project memory bootstrap)

package/assets/init/.qfai/assistant/prompts/qfai-spec.md

@@ -51,6 +51,18 @@ When unsure, read inputs in this order:
 - P3: `.qfai/specs/<spec-id>/delta.md` (Decision Records; if no spec yet, state "not applicable")
 - P4: other artifacts (spec.md, scenario.feature, contracts, evidence)
 
+## Change Type (Mandatory)
+
+Before editing `delta.md`, declare the Change Type and record it in:
+
+- `delta.md` Change Log (latest CL entry)
+- PR description (Change Type section)
+
+Allowed values:
+
+- Primary: `Initial | Behavior | Structural | Ops`
+- Tags (optional): `@ui @api @db @nfr @docs @test`
+
 ## Delta Rejected Guard (Mandatory)
 
 - Do NOT reintroduce options marked as rejected in delta.md.

package/assets/init/.qfai/assistant/prompts/qfai-tdd-green.md

@@ -38,6 +38,18 @@ When unsure, read inputs in this order:
 - P3: `.qfai/specs/<spec-id>/delta.md` (Decision Records; if no spec yet, state "not applicable")
 - P4: other artifacts (spec.md, scenario.feature, contracts, evidence)
 
+## Change Type (Mandatory)
+
+Before updating `delta.md`, declare the Change Type and record it in:
+
+- `delta.md` Change Log (latest CL entry)
+- PR description (Change Type section)
+
+Allowed values:
+
+- Primary: `Initial | Behavior | Structural | Ops`
+- Tags (optional): `@ui @api @db @nfr @docs @test`
+
 ## Delta Rejected Guard (Mandatory)
 
 - Do NOT reintroduce options marked as rejected in delta.md.

package/assets/init/.qfai/assistant/prompts/qfai-tdd-red.md

@@ -38,6 +38,18 @@ When unsure, read inputs in this order:
 - P3: `.qfai/specs/<spec-id>/delta.md` (Decision Records; if no spec yet, state "not applicable")
 - P4: other artifacts (spec.md, scenario.feature, contracts, evidence)
 
+## Change Type (Mandatory)
+
+Before updating `delta.md`, declare the Change Type and record it in:
+
+- `delta.md` Change Log (latest CL entry)
+- PR description (Change Type section)
+
+Allowed values:
+
+- Primary: `Initial | Behavior | Structural | Ops`
+- Tags (optional): `@ui @api @db @nfr @docs @test`
+
 ## Delta Rejected Guard (Mandatory)
 
 - Do NOT reintroduce options marked as rejected in delta.md.

package/assets/init/.qfai/assistant/prompts/qfai-tdd-refactor.md

@@ -38,6 +38,18 @@ When unsure, read inputs in this order:
 - P3: `.qfai/specs/<spec-id>/delta.md` (Decision Records; if no spec yet, state "not applicable")
 - P4: other artifacts (spec.md, scenario.feature, contracts, evidence)
 
+## Change Type (Mandatory)
+
+Before updating `delta.md`, declare the Change Type and record it in:
+
+- `delta.md` Change Log (latest CL entry)
+- PR description (Change Type section)
+
+Allowed values:
+
+- Primary: `Initial | Behavior | Structural | Ops`
+- Tags (optional): `@ui @api @db @nfr @docs @test`
+
 ## Delta Rejected Guard (Mandatory)
 
 - Do NOT reintroduce options marked as rejected in delta.md.

package/assets/init/.qfai/specs/README.md

@@ -185,6 +185,9 @@ specs/
 
 - date: <YYYY-MM-DD>
 - author: <AI/role or human>
+- change_type_primary: Initial | Behavior | Structural | Ops
+- change_type_tags: <space-separated tags or empty>
+  - example: @api @db
 - scope: <files/areas>
 - change: <what changed>
 - reason: <why it changed>
@@ -204,6 +207,8 @@ specs/
 - selected: <option>
 - rejected:
   - <option> — <reason>
+    - do_not: <what must not be reintroduced>
+    - temptation: <why it may be tempting to reintroduce>
 - impact: <downstream impact>
 - followups: <todos>
 - related_contracts: <QFAI-CONTRACT-REF or IDs>

package/dist/cli/index.cjs

@@ -1184,8 +1184,8 @@ var import_promises7 = require("fs/promises");
 var import_node_path9 = __toESM(require("path"), 1);
 var import_node_url2 = require("url");
 async function resolveToolVersion() {
-  if ("1.2.14".length > 0) {
-    return "1.2.14";
+  if ("1.3.0".length > 0) {
+    return "1.3.0";
   }
   try {
     const packagePath = resolvePackageJsonPath();
@@ -3568,6 +3568,24 @@ async function validateCaseCatalogues(root, config) {
 // src/core/validators/delta.ts
 var import_promises15 = require("fs/promises");
 var import_node_path19 = __toESM(require("path"), 1);
+var CHANGE_TYPE_PRIMARY_RE = /^\s*[-*]?\s*change_type_primary\s*:\s*(.+)\s*$/im;
+var CHANGE_TYPE_TAGS_RE = /^\s*[-*]?\s*change_type_tags\s*:\s*(.*)\s*$/im;
+var DO_NOT_RE = /^\s*[-*]?\s*do_not\s*:/im;
+var TEMPTATION_RE = /^\s*[-*]?\s*temptation\s*:/im;
+var ALLOWED_CHANGE_TYPE_PRIMARY = /* @__PURE__ */ new Set([
+  "initial",
+  "behavior",
+  "structural",
+  "ops"
+]);
+var ALLOWED_CHANGE_TYPE_TAGS = /* @__PURE__ */ new Set([
+  "@ui",
+  "@api",
+  "@db",
+  "@nfr",
+  "@docs",
+  "@test"
+]);
 async function validateDeltas(root, config) {
   const specsRoot = resolvePath(root, config, "specsDir");
   const packs = await collectSpecPackDirs(specsRoot);
@@ -3629,10 +3647,8 @@ async function validateDeltas(root, config) {
         )
       );
     } else {
-      const hasRejected = /^\s*(?:[-*]\s*)?rejected\s*:/im.test(
-        decisionRecords.body
-      );
-      if (!hasRejected) {
+      const rejectedBlocks = extractRejectedBlocks(decisionRecords.body);
+      if (rejectedBlocks.length === 0) {
         issues.push(
           issue(
             "QFAI-DELTA-101",
@@ -3645,6 +3661,28 @@ async function validateDeltas(root, config) {
             "rejected \u3092\u6700\u4F4E1\u4EF6\u8A18\u8F09\u3057\u3066\u304F\u3060\u3055\u3044\u3002"
           )
         );
+      } else {
+        const hasDoNot = rejectedBlocks.some((block) => DO_NOT_RE.test(block));
+        const hasTemptation = rejectedBlocks.some(
+          (block) => TEMPTATION_RE.test(block)
+        );
+        if (!hasDoNot || !hasTemptation) {
+          const missing = [];
+          if (!hasDoNot) missing.push("do_not");
+          if (!hasTemptation) missing.push("temptation");
+          issues.push(
+            issue(
+              "QFAI-DELTA-204",
+              `Decision Records \u306B ${missing.join(" / ")} \u304C\u898B\u3064\u304B\u308A\u307E\u305B\u3093\u3002`,
+              "warning",
+              deltaPath,
+              "delta.rejectedDetails",
+              void 0,
+              "change",
+              "rejected \u306B\u306F do_not / temptation \u3092\u6700\u4F4E1\u4EF6\u542B\u3081\u3066\u304F\u3060\u3055\u3044\u3002"
+            )
+          );
+        }
       }
     }
     if (changeLog && decisionRecords) {
@@ -3663,6 +3701,54 @@ async function validateDeltas(root, config) {
         );
       }
     }
+    if (changeLog) {
+      const blocks = extractChangeLogBlocks(text, changeLog);
+      for (const block of blocks) {
+        const primary = extractChangeTypePrimary(block);
+        if (!primary) {
+          issues.push(
+            issue(
+              "QFAI-DELTA-201",
+              "Change Log \u306E CL \u30D6\u30ED\u30C3\u30AF\u306B change_type_primary \u304C\u898B\u3064\u304B\u308A\u307E\u305B\u3093\u3002",
+              "warning",
+              deltaPath,
+              "delta.changeTypePrimary",
+              void 0,
+              "change",
+              "change_type_primary \u3092\u8FFD\u52A0\u3057\u3066\u304F\u3060\u3055\u3044\u3002"
+            )
+          );
+        } else if (!isAllowedChangeTypePrimary(primary)) {
+          issues.push(
+            issue(
+              "QFAI-DELTA-202",
+              `change_type_primary \u304C\u4E0D\u6B63\u3067\u3059: ${primary}`,
+              "warning",
+              deltaPath,
+              "delta.changeTypePrimary",
+              void 0,
+              "change",
+              "change_type_primary \u306F Initial | Behavior | Structural | Ops \u3092\u6307\u5B9A\u3057\u3066\u304F\u3060\u3055\u3044\u3002"
+            )
+          );
+        }
+        const invalidTags = extractInvalidChangeTypeTags(block);
+        if (invalidTags && invalidTags.length > 0) {
+          issues.push(
+            issue(
+              "QFAI-DELTA-203",
+              `change_type_tags \u306B\u7121\u52B9\u306A\u30BF\u30B0\u304C\u3042\u308A\u307E\u3059: ${invalidTags.join(", ")}`,
+              "warning",
+              deltaPath,
+              "delta.changeTypeTags",
+              void 0,
+              "change",
+              "change_type_tags \u306F @ui @api @db @nfr @docs @test \u306E\u307F\u6307\u5B9A\u3057\u3066\u304F\u3060\u3055\u3044\u3002"
+            )
+          );
+        }
+      }
+    }
   }
   return issues;
 }
@@ -3675,6 +3761,98 @@ function findSection(sections, title) {
   }
   return null;
 }
+function extractChangeLogBlocks(text, changeLog) {
+  const lines = text.split(/\r?\n/);
+  const headings = parseHeadings(text).filter(
+    (heading) => heading.level === 3 && heading.line >= changeLog.startLine && heading.line <= changeLog.endLine
+  );
+  if (headings.length === 0) {
+    return [changeLog.body];
+  }
+  const blocks = [];
+  for (let i = 0; i < headings.length; i += 1) {
+    const current = headings[i];
+    if (!current) continue;
+    const next = headings[i + 1];
+    const startLine = current.line + 1;
+    const endLine = Math.min(
+      changeLog.endLine,
+      (next?.line ?? changeLog.endLine + 1) - 1
+    );
+    if (startLine > endLine) {
+      blocks.push("");
+      continue;
+    }
+    blocks.push(lines.slice(startLine - 1, endLine).join("\n"));
+  }
+  return blocks;
+}
+function extractChangeTypePrimary(block) {
+  const match = block.match(CHANGE_TYPE_PRIMARY_RE);
+  const value = match?.[1]?.trim() ?? "";
+  return value.length > 0 ? value : null;
+}
+function extractRejectedBlocks(sectionBody) {
+  const lines = sectionBody.split(/\r?\n/);
+  const blocks = [];
+  let currentIndent = null;
+  let buffer = [];
+  const flush = () => {
+    if (currentIndent === null) {
+      return;
+    }
+    blocks.push(buffer.join("\n"));
+    buffer = [];
+    currentIndent = null;
+  };
+  for (let i = 0; i < lines.length; i += 1) {
+    const line = lines[i] ?? "";
+    const match = line.match(/^(\s*)(?:[-*]\s*)?rejected\s*:(.*)$/i);
+    if (match) {
+      flush();
+      const inlineValue = (match[2] ?? "").trim();
+      if (inlineValue.length > 0) {
+        blocks.push(inlineValue);
+        currentIndent = null;
+      } else {
+        currentIndent = (match[1] ?? "").length;
+      }
+      continue;
+    }
+    if (currentIndent === null) {
+      continue;
+    }
+    const indentMatch = line.match(/^(\s*)/);
+    const indent = (indentMatch?.[1] ?? "").length;
+    if (line.trim().length > 0 && indent <= currentIndent) {
+      flush();
+      i -= 1;
+      continue;
+    }
+    buffer.push(line);
+  }
+  flush();
+  return blocks;
+}
+function extractInvalidChangeTypeTags(block) {
+  const match = block.match(CHANGE_TYPE_TAGS_RE);
+  if (!match) {
+    return null;
+  }
+  const raw = match[1]?.trim() ?? "";
+  if (raw.length === 0) {
+    return [];
+  }
+  const tags = raw.split(/[\s,]+/).map((tag) => tag.trim()).filter((tag) => tag.length > 0);
+  const invalid = tags.filter((tag) => !isAllowedChangeTypeTag(tag));
+  return invalid;
+}
+function isAllowedChangeTypePrimary(value) {
+  return ALLOWED_CHANGE_TYPE_PRIMARY.has(value.trim().toLowerCase());
+}
+function isAllowedChangeTypeTag(value) {
+  return ALLOWED_CHANGE_TYPE_TAGS.has(value.trim().toLowerCase());
+}
 
 // src/core/validators/ids.ts
 var import_promises16 = require("fs/promises");