@reqvire-org/ads 0.3.18 → 0.3.21

package/dist/mcp/edit-preparation.mjs

@@ -5,6 +5,7 @@ import path from "node:path";
 import { isInside } from "../config.mjs";
 import { checkUnitFiles, inspectUnits, validateUnitId } from "../units.mjs";
 import { textDiff } from "./text-diff.mjs";
+import { migrateContractDraft } from "../contract-migration.mjs";
 
 const hash = value => createHash("sha256").update(value).digest("hex");
 const binary = bytes => bytes?.includes(0) || bytes && !Buffer.from(bytes.toString("utf8")).equals(bytes);
@@ -26,13 +27,14 @@ export function storePreparedPatch(options, patch) {
 export function prepareUnitPatch(options, {unit, action = "create", draft}) {
   validateUnitId(unit);
   const inventory = inspectUnits(options);
-  if (action === "create") {
+  if (action === "create" || action === "migrate") {
     const owner = inventory.units.find(entry => entry.id === unit);
+    if(action==="migrate"&&!owner)throw new Error("Discover an existing owner before preparing its contract migration.");
     const files = [...new Set([...(owner?.files ?? []), ...(owner?.companionFiles ?? []).map(entry => entry.path),
       ...(!owner ? ["ds.config.json"] : []),
-      ...(/^(?:components|patterns|controllers)\//.test(unit) ? [path.relative(options.root,path.join(options.dsRoot,options.config.publicEntry)).replaceAll("\\","/")] : [])])];
-    if (!owner && !/^(?:components|patterns|controllers|composition)\/.+/.test(unit)) throw new Error("Discover an existing unit or choose a new unoccupied component/pattern/controller/composition owner.");
-    if (/^(?:components|patterns|controllers|composition)\//.test(unit)) for (let directory=unit;directory!==".";directory=path.posix.dirname(directory)) {
+      ...(/^(?:components|patterns|controllers|review)\//.test(unit) ? [path.relative(options.root,path.join(options.dsRoot,unit.startsWith("review/")?options.config.reviewEntry:options.config.publicEntry)).replaceAll("\\","/")] : [])])];
+    if (!owner && !/^(?:components|patterns|controllers|composition|review)\/.+/.test(unit)) throw new Error("Discover an existing unit or choose a new unoccupied component/pattern/controller/composition/review owner.");
+    if (/^(?:components|patterns|controllers|composition|review)\//.test(unit)) for (let directory=unit;directory!==".";directory=path.posix.dirname(directory)) {
       const marker=path.join(options.dsRoot,directory,".gitkeep");
       if (existsSync(marker) && realpathSync(marker)===marker && lstatSync(marker).isFile() && lstatSync(marker).size===0) {
         const relative=path.relative(options.root,marker).replaceAll("\\","/");
@@ -58,7 +60,8 @@ export function prepareUnitPatch(options, {unit, action = "create", draft}) {
     }
     mkdirSync(folder,{recursive:true});
     writeFileSync(base,JSON.stringify({schemaVersion:1,unit,entries}));
-    return {draft,unit,directory:folder,files, next:"Edit only this draft, then call ads_prepare_patch with action: diff and this draft ID. Live source is unchanged; the resulting Git patch still requires ads_apply_patch and all its gates."};
+    const prepared={draft,unit,directory:folder,files, next:"Edit only this draft, then call ads_prepare_patch with action: diff and this draft ID. Live source is unchanged; the resulting Git patch still requires ads_apply_patch and all its gates."};
+    return action === "migrate" ? migrateContractDraft(options,prepared) : prepared;
   }
   if (!/^[a-f0-9-]{36}$/.test(draft ?? "")) throw new Error("Use the draft ID returned by ads_prepare_patch.");
   const base = path.join(options.root,".ds/patch-drafts",draft);
@@ -66,7 +69,7 @@ export function prepareUnitPatch(options, {unit, action = "create", draft}) {
   const metadata = JSON.parse(readFileSync(path.join(base,"manifest.json"),"utf8"));
   if (metadata.schemaVersion !== 1 || metadata.unit !== unit) throw new Error("Draft and selected unit do not match.");
   if (action === "discard") {rmSync(base,{recursive:true});return {discarded:true,draft,unit};}
-  if (action !== "diff") throw new Error("Choose create, diff or discard.");
+  if (action !== "diff") throw new Error("Choose create, migrate, diff or discard.");
   const files = new Set(metadata.entries.map(entry => entry.file));
   function walk(directory) {
     if (realpathSync(directory) !== directory) throw new Error("Draft directories cannot resolve through symlinks.");

package/dist/mcp/mutations.mjs

@@ -178,7 +178,7 @@ export function createMutationController(directory, { captures = createCaptureCo
       if (!Number.isSafeInteger(offset) || offset < 0 || !Number.isSafeInteger(limit) || limit < 1 || limit > mutationLimits.readChars) throw fail("mutation-range", "Use a non-negative character offset and a limit between 1 and 12000.", file);
       const options = loadConfig(root), source = read(options, file);
       const inventory = inspectUnits(options), unit = inventory.owner(inventory.relative(file));
-      if (!unit && inventory.relative(file) !== inventory.publicEntry) throw fail("mutation-unit", "This file is outside the project's ADS and registered configuration/extension units.", file);
+      if (!unit && inventory.relative(file) !== inventory.publicEntry && inventory.relative(file) !== inventory.reviewEntry) throw fail("mutation-unit", "This file is outside the project's ADS and registered configuration/extension units.", file);
       if (!source.exists) return { ...source, offset: 0, nextOffset: null, totalChars: 0, startLine: 1 };
       if (source.binary) return { ...source, unit, content: null, note: "Binary asset: inspect its metadata or preview; edit using a Git binary patch, optionally through patchFile." };
       if (offset > source.content.length) throw fail("mutation-range", "Read offset exceeds the current file length.", file);