@mnstry/atelier 0.2.0-alpha.13 → 0.2.0-alpha.14

package/src/upgrade/upgrade.mjs

@@ -15,6 +15,7 @@ import {
   writeJson,
 } from '../project/config.mjs'
 import { packageRootFrom } from '../project/package-root.mjs'
+import { canonicalDiffArgs, gitOutputResult, parseDiffSections } from '../boundary/content-rules.mjs'
 import {
   BOUNDARY_POLICY_SCHEMA,
   checkBoundaryPolicy,
@@ -26,8 +27,10 @@ import { loadExtensionPacks } from '../extension-packs/loader.mjs'
 import { inspectPackageProvenance, legacyPackageSource } from './provenance.mjs'
 import { prepareUpgrade, applySavedUpgrade, upgradeOperationStatus, recoverUpgradeDryRun, explainSavedUpgrade } from './transaction.mjs'
 import { renderUpgradeExplanation } from './explanation.mjs'
+import { planConsumerClosure, runClosurePlanCommand } from './closure.mjs'
 export { prepareUpgrade, applySavedUpgrade, upgradeOperationStatus, recoverUpgradeDryRun, explainSavedUpgrade }
 export { prepareTemplateUpgrade } from './transaction.mjs'
+export { planConsumerClosure }
 
 export const ATELIER_LOCK_SCHEMA = 'mnstry.atelier-lock@v1'
 export const ATELIER_MIGRATION_SCHEMA = 'mnstry.atelier-migration@v1'
@@ -411,13 +414,18 @@ function stagedSemanticFindings(project) {
   const findings = []
   for (const repo of project.repos ?? []) {
     if (!isGitRepo(repo.path)) continue
-    const result = spawnSync('git', ['-C', repo.path, 'diff', '--cached', '--unified=0', '--', '*.md', '*.kg.json'], { encoding: 'utf8' })
-    if (result.status !== 0 || !result.stdout.trim()) continue
-    const diff = result.stdout
-    const changedSemantic = diff
-      .split('\n')
-      .some((line) => /^[+-](?![+-])/.test(line) && SEMANTIC_FIELD_RE.test(line))
-    if (changedSemantic && !REVIEW_MARKER_RE.test(diff)) findings.push(`${repo.name}: staged semantic field change needs Atelier-Boundary-Review marker`)
+    // The boundary guard's pinned diff and parser; only a marker added in this
+    // change approves it, never a removed line or a file name.
+    const result = gitOutputResult(repo.path, [...canonicalDiffArgs(['--cached'], ['--text']), '--', '*.md', '*.kg.json'])
+    if (!result.ok) {
+      findings.push(`${repo.name}: staged semantic fields could not be read (${result.code})`)
+      continue
+    }
+    const sections = parseDiffSections(result.stdout, { detail: true })
+    const changedSemantic = sections.some((section) => section.incomplete
+      || section.hunks.some((hunk) => [...hunk.added, ...hunk.removed].some((line) => SEMANTIC_FIELD_RE.test(line))))
+    const approved = sections.some((section) => section.hunks.some((hunk) => hunk.added.some((line) => REVIEW_MARKER_RE.test(line))))
+    if (changedSemantic && !approved) findings.push(`${repo.name}: staged semantic field change needs Atelier-Boundary-Review marker`)
   }
   return findings
 }
@@ -683,6 +691,10 @@ export function runLockCommand(argv = process.argv.slice(2)) {
 
 export function runUpgradeCommand(argv = process.argv.slice(2)) {
   const args = parseArgs(argv)
+  // `closure` plans against an explicit npm root, which need not be an Atelier
+  // workspace. It runs before the project is resolved, writes nothing, and lets
+  // its typed refusal reach the command wrapper unchanged.
+  if (args._[0] === 'closure') return runClosurePlanCommand(args, argv)
   const project = commandProject({ argv, writeLocalState: args._[0] !== 'explain' })
   if (['plan', 'apply', 'status', 'recover', 'explain'].includes(args._[0])) {
     try {

package/templates/external-project-workspace/pack-lifecycle.json

@@ -11,7 +11,8 @@
         "0.2.0-alpha.10",
         "0.2.0-alpha.11",
         "0.2.0-alpha.12",
-        "0.2.0-alpha.13"
+        "0.2.0-alpha.13",
+        "0.2.0-alpha.14"
       ],
       "status": "active"
     }