@deftai/directive-core 0.98.1 → 0.100.0

package/dist/vbrief-activate/activate.js

@@ -2,6 +2,7 @@ import { existsSync, mkdirSync, readFileSync, renameSync, rmSync, statSync, unli
 import { basename, dirname, resolve } from "node:path";
 import { containedWrite } from "../fs/contained-write.js";
 import { assertProjectionContained, ProjectionContainmentError, } from "../fs/projection-containment.js";
+import { evaluateEffortActivateGate } from "../scope/effort-activate-gate.js";
 import { utcNowIso } from "../scope/vbrief-json.js";
 import { pythonJsonPretty } from "../vbrief-build/json.js";
 import { ACTIVE_FOLDER, ELIGIBLE_STATUSES_FOR_FLIP, formatEligibleStatusList, SOURCE_FOLDERS, TARGET_STATUS, } from "./constants.js";
@@ -121,6 +122,11 @@ export function activate(vbriefPath, options = {}) {
                 `'${TARGET_STATUS}'.`,
         };
     }
+    // #1581: fail closed when any plan item still has effort=XL (needs breakdown).
+    const effortGate = evaluateEffortActivateGate(planObj);
+    if (!effortGate.ok) {
+        return { exitCode: 1, message: effortGate.message };
+    }
     // Resolve destination early so containment can refuse before any mutation.
     // Parity with scope:activate / #2447: projectRoot is parent of the xbrief/ root.
     const vbriefDir = dirname(dirname(vbriefPath));

package/dist/vbrief-validate/constants.d.ts

@@ -9,6 +9,8 @@ export declare const VALID_VBRIEF_VERSIONS: Set<string>;
 export declare const VALID_INFO_ROOT_KEYS: Set<string>;
 /** v0.8 PlanItem.type enum values (optional field). */
 export declare const VALID_PLAN_ITEM_TYPES: Set<string>;
+/** Optional PlanItem.effort enum (#1581). Time anchors: S <2h, M 2-4h, L 1-2d, XL needs breakdown. */
+export declare const VALID_PLAN_ITEM_EFFORTS: Set<string>;
 /** D13: status-to-folder mapping (#533 adds ``failed`` in completed/). */
 export declare const FOLDER_ALLOWED_STATUSES: Readonly<Record<string, ReadonlySet<string>>>;
 export declare const LIFECYCLE_FOLDERS: string[];

package/dist/vbrief-validate/constants.js

@@ -19,6 +19,8 @@ export const VALID_VBRIEF_VERSIONS = new Set(["0.6", "0.8"]);
 export const VALID_INFO_ROOT_KEYS = new Set(["vBRIEFInfo", "xBRIEFInfo"]);
 /** v0.8 PlanItem.type enum values (optional field). */
 export const VALID_PLAN_ITEM_TYPES = new Set(["task", "group", "milestone", "epic"]);
+/** Optional PlanItem.effort enum (#1581). Time anchors: S <2h, M 2-4h, L 1-2d, XL needs breakdown. */
+export const VALID_PLAN_ITEM_EFFORTS = new Set(["S", "M", "L", "XL"]);
 /** D13: status-to-folder mapping (#533 adds ``failed`` in completed/). */
 export const FOLDER_ALLOWED_STATUSES = {
     proposed: new Set(["draft", "proposed"]),

package/dist/vbrief-validate/schema.js

@@ -1,5 +1,5 @@
 import { pyStrRepr, pythonTypeName } from "../triage/scope/python-repr.js";
-import { PROJECT_DEF_EXPECTED_NARRATIVES, VALID_INFO_ROOT_KEYS, VALID_ITEM_STATUSES, VALID_PLAN_ITEM_TYPES, VALID_PLAN_STATUSES, VALID_VBRIEF_VERSIONS, } from "./constants.js";
+import { PROJECT_DEF_EXPECTED_NARRATIVES, VALID_INFO_ROOT_KEYS, VALID_ITEM_STATUSES, VALID_PLAN_ITEM_EFFORTS, VALID_PLAN_ITEM_TYPES, VALID_PLAN_STATUSES, VALID_VBRIEF_VERSIONS, } from "./constants.js";
 function validateNarratives(narratives, path, errors) {
     if (typeof narratives !== "object" || narratives === null || Array.isArray(narratives)) {
         errors.push(`${path} must be an object`);
@@ -50,6 +50,9 @@ function validatePlanItem(item, path, errors) {
     if ("type" in item && !VALID_PLAN_ITEM_TYPES.has(String(item.type))) {
         errors.push(`${itemPath} invalid type: ${pyStrRepr(String(item.type))}`);
     }
+    if ("effort" in item && !VALID_PLAN_ITEM_EFFORTS.has(String(item.effort))) {
+        errors.push(`${itemPath} invalid effort: ${pyStrRepr(String(item.effort))}`);
+    }
     if ("summary" in item && typeof item.summary !== "string") {
         errors.push(`${itemPath}.summary must be a string, got ${pythonTypeName(item.summary)}`);
     }

package/dist/xbrief-migrate/migrate-project.d.ts

@@ -24,6 +24,10 @@ export type XbriefMigrationOutcome = {
     readonly kind: "migrated";
     readonly backupDir: string;
     readonly files: number;
+} | {
+    readonly kind: "rewritten";
+    readonly files: number;
+    readonly message: string;
 } | {
     readonly kind: "converged";
     readonly action: VbriefConvergeAction;

package/dist/xbrief-migrate/migrate-project.js

@@ -5,12 +5,12 @@ import { assertWriteTargetSafe, ProjectionContainmentError } from "../fs/project
 import { checkGitClean } from "../migrate-preflight/index.js";
 import { applyAgentsRefresh } from "../platform/agents-md.js";
 import { patchAgentsMdHeader, renderHeaderPatchSummary } from "./agents-header.js";
-import { LEGACY_ARTIFACT_DIR, LEGACY_ARTIFACT_SUFFIX, MIGRATED_ARTIFACT_DIR, MIGRATED_ARTIFACT_SUFFIX, OBSOLETE_FRAMEWORK_NARRATIVE_FILENAME, VBRIEF_DEPRECATION_MARKER_BODY, VBRIEF_DEPRECATION_MARKER_FILENAME, } from "./constants.js";
+import { LEGACY_ARTIFACT_DIR, LEGACY_ARTIFACT_SUFFIX, LEGACY_VBRIEF_VERSION, MIGRATED_ARTIFACT_DIR, MIGRATED_ARTIFACT_SUFFIX, OBSOLETE_FRAMEWORK_NARRATIVE_FILENAME, VBRIEF_DEPRECATION_MARKER_BODY, VBRIEF_DEPRECATION_MARKER_FILENAME, } from "./constants.js";
 import { detectLegacyVbriefLayout, detectXbriefConvergence } from "./detect.js";
 import { hasVbriefDeprecationMarker, isDirectory, isEffectivelyEmptyDir } from "./fs-helpers.js";
 import { assertMigrationSourceSafe } from "./migration-containment.js";
 import { renderXbriefMigrationLine, xbriefMigrationGuidance } from "./signpost.js";
-import { rewriteEmbeddedTokens, transformArtifactV06ToV08Transactional } from "./transforms.js";
+import { readDeclaredArtifactVersion, rewriteEmbeddedTokens, transformArtifactV06ToV08Transactional, } from "./transforms.js";
 function collectFiles(root, acc = []) {
     if (!isDirectory(root)) {
         return acc;
@@ -74,6 +74,65 @@ function writeMigratedFile(projectRoot, srcPath, destPath) {
         mode: "replace",
     });
 }
+/**
+ * Plan in-place hybrid envelope rewrites for xbrief lifecycle `.xbrief.json`
+ * artifacts that still declare `xBRIEFInfo@0.6` after the layout rename
+ * (#3236 / #2974). Pure probe — no writes. Already-0.8 artifacts, schema
+ * deposits, and non-0.6 JSON are skipped so residual #2368 schema-only markers
+ * still route to `directive update`.
+ */
+function planHybridEnvelopeRewrites(projectRoot) {
+    const migratedDir = join(projectRoot, MIGRATED_ARTIFACT_DIR);
+    const artifactPaths = collectFiles(migratedDir).filter((path) => path.endsWith(MIGRATED_ARTIFACT_SUFFIX));
+    const pending = [];
+    for (const filePath of artifactPaths) {
+        let parsed;
+        try {
+            parsed = JSON.parse(readFileSync(filePath, "utf8"));
+        }
+        catch (err) {
+            const detail = err instanceof Error ? err.message : String(err);
+            return {
+                ok: false,
+                error: `failed to parse hybrid envelope candidate ${relative(projectRoot, filePath)}: ${detail}`,
+            };
+        }
+        // Only residual v0.6 envelopes are rewrite candidates; skip 0.8 and unknown.
+        if (readDeclaredArtifactVersion(parsed) !== LEGACY_VBRIEF_VERSION) {
+            continue;
+        }
+        const result = transformArtifactV06ToV08Transactional(parsed);
+        if (!result.ok) {
+            return {
+                ok: false,
+                error: `failed to transform hybrid envelope ${relative(projectRoot, filePath)}: ${result.error}`,
+            };
+        }
+        if (!result.changed) {
+            continue;
+        }
+        pending.push({
+            path: filePath,
+            body: `${JSON.stringify(result.artifact, null, 2)}\n`,
+        });
+    }
+    return { ok: true, pending };
+}
+/**
+ * Apply a previously planned hybrid-envelope rewrite set via containedWrite (#3236).
+ */
+function applyHybridEnvelopeRewrites(projectRoot, pending) {
+    const root = resolve(projectRoot);
+    for (const entry of pending) {
+        containedWrite({
+            root,
+            target: entry.path,
+            data: entry.body,
+            mode: "replace",
+        });
+    }
+    return pending.length;
+}
 function backupMigrationInputs(projectRoot, legacyDir, migratedDir) {
     const stamp = new Date().toISOString().replace(/[:.]/g, "-");
     const backupRoot = join(projectRoot, ".deft", `xbrief-migrate-backup-${stamp}`);
@@ -256,6 +315,33 @@ export function runXbriefMigration(args, _io) {
         default:
             break;
     }
+    // #3236: on already-xbrief trees with no vbrief/, rewrite residual hybrid
+    // xBRIEFInfo@0.6 envelopes in place (transform accepts hybrid — #2974).
+    // Run before the layout-detection short-circuit so compact `"version":"0.6"`
+    // JSON that the string-scan may miss still gets rewritten.
+    if (convergence.xbriefHasContent && !convergence.vbriefPresent) {
+        const plan = planHybridEnvelopeRewrites(projectRoot);
+        if (!plan.ok) {
+            return { kind: "config", message: plan.error };
+        }
+        if (plan.pending.length > 0) {
+            if (!args.force) {
+                const git = checkGitClean(projectRoot);
+                if (git.status === "WARN") {
+                    return {
+                        kind: "refused",
+                        message: `${git.message} ${xbriefMigrationGuidance()} Pass --force to override.`,
+                    };
+                }
+            }
+            const files = applyHybridEnvelopeRewrites(projectRoot, plan.pending);
+            return {
+                kind: "rewritten",
+                files,
+                message: `Rewrote ${files} hybrid xBRIEFInfo@0.6 envelope(s) in place to xBRIEFInfo@0.8 under '${MIGRATED_ARTIFACT_DIR}/'.`,
+            };
+        }
+    }
     const detection = detectLegacyVbriefLayout(projectRoot);
     if (!detection.legacyLayout) {
         return {
@@ -266,7 +352,7 @@ export function runXbriefMigration(args, _io) {
     if (!isDirectory(legacyDir)) {
         // Already on canonical xbrief/ with no vbrief/ tree — residual markers (e.g. a
         // stale deposited v0.6 schema under xbrief/schemas/) are refreshed by update,
-        // not migrate:xbrief (#2368).
+        // not migrate:xbrief (#2368). Hybrid 0.6 envelopes were handled above (#3236).
         if (convergence.xbriefHasContent && !convergence.vbriefPresent) {
             return {
                 kind: "noop",
@@ -319,6 +405,9 @@ export function emitXbriefMigration(outcome, io, options = {}) {
             io.writeOut(`Migrated ${outcome.files} file(s) from ${LEGACY_ARTIFACT_DIR}/ to ${MIGRATED_ARTIFACT_DIR}/.\n` +
                 `Backup written to ${outcome.backupDir}.\n`);
             return 0;
+        case "rewritten":
+            io.writeOut(`${outcome.message}\n`);
+            return 0;
         case "converged":
             io.writeOut(`${outcome.message}\n`);
             return 0;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@deftai/directive-core",
-  "version": "0.98.1",
+  "version": "0.100.0",
   "description": "TypeScript engine core for the Directive framework.",
   "license": "MIT",
   "type": "module",
@@ -131,6 +131,14 @@
       "types": "./dist/value/readback.d.ts",
       "default": "./dist/value/readback.js"
     },
+    "./decision": {
+      "types": "./dist/decision/index.d.ts",
+      "default": "./dist/decision/index.js"
+    },
+    "./decision/index": {
+      "types": "./dist/decision/index.d.ts",
+      "default": "./dist/decision/index.js"
+    },
     "./events": {
       "types": "./dist/events/attribution-ledger.d.ts",
       "default": "./dist/events/attribution-ledger.js"
@@ -147,6 +155,10 @@
       "types": "./dist/eval/report.d.ts",
       "default": "./dist/eval/report.js"
     },
+    "./eval/version-pin": {
+      "types": "./dist/eval/version-pin.d.ts",
+      "default": "./dist/eval/version-pin.js"
+    },
     "./triage": {
       "types": "./dist/triage/index.d.ts",
       "default": "./dist/triage/index.js"
@@ -271,6 +283,10 @@
       "types": "./dist/lifecycle/index.d.ts",
       "default": "./dist/lifecycle/index.js"
     },
+    "./literal-acceptance": {
+      "types": "./dist/literal-acceptance/index.d.ts",
+      "default": "./dist/literal-acceptance/index.js"
+    },
     "./packs": {
       "types": "./dist/packs/index.d.ts",
       "default": "./dist/packs/index.js"
@@ -354,8 +370,8 @@
     "provenance": true
   },
   "dependencies": {
-    "@deftai/directive-content": "^0.98.1",
-    "@deftai/directive-types": "^0.98.1",
+    "@deftai/directive-content": "^0.100.0",
+    "@deftai/directive-types": "^0.100.0",
     "archiver": "^8.0.0"
   },
   "scripts": {