@tomflow/proflow-platform-cli 0.1.5 → 0.1.7

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # @tomflow/proflow-platform-cli
 
+## 0.1.7
+
+### Patch Changes
+
+- Rebuild upgrade apply staleness from the current persisted Workspace configuration so unchanged config remains fresh while configuration drift becomes PLAN_STALE instead of being misreported as missing required config.
+
+## 0.1.6
+
+### Patch Changes
+
+- Carry persisted workspace configuration into Registry-backed upgrade planning, with explicit --config values overlaying stored values, so newly published descriptors cannot falsely report existing required configuration as missing before package mutation.
+
 ## 0.1.5
 
 ### Patch Changes

package/dist/deployment/adapter.d.ts

@@ -5,7 +5,7 @@ export declare const behaviorAdapter: {
             ok: boolean;
             status: "SUCCEEDED";
             moduleRef: "platform-cli";
-            moduleVersion: "0.1.5";
+            moduleVersion: "0.1.7";
             data?: {} | null;
         };
         observedEffects: never[];
@@ -16,7 +16,7 @@ export declare const behaviorAdapter: {
             ok: boolean;
             status: "SUCCEEDED";
             moduleRef: "platform-cli";
-            moduleVersion: "0.1.5";
+            moduleVersion: "0.1.7";
             data?: {} | null;
         };
         observedEffects: never[];
@@ -27,7 +27,7 @@ export declare const behaviorAdapter: {
             ok: boolean;
             status: "SUCCEEDED";
             moduleRef: "platform-cli";
-            moduleVersion: "0.1.5";
+            moduleVersion: "0.1.7";
             data?: {} | null;
         };
         observedEffects: never[];
@@ -38,7 +38,7 @@ export declare const behaviorAdapter: {
             ok: boolean;
             status: "SUCCEEDED";
             moduleRef: "platform-cli";
-            moduleVersion: "0.1.5";
+            moduleVersion: "0.1.7";
             data?: {} | null;
         };
         observedEffects: never[];
@@ -49,7 +49,7 @@ export declare const behaviorAdapter: {
             ok: boolean;
             status: "SUCCEEDED";
             moduleRef: "platform-cli";
-            moduleVersion: "0.1.5";
+            moduleVersion: "0.1.7";
             data?: {} | null;
         };
         observedEffects: never[];

package/dist/deployment/descriptor.d.ts

@@ -3,7 +3,7 @@ export declare const descriptor: {
     readonly contractVersion: "1.0.0";
     readonly moduleRef: "platform-cli";
     readonly packageName: "@tomflow/proflow-platform-cli";
-    readonly moduleVersion: "0.1.5";
+    readonly moduleVersion: "0.1.7";
     readonly kind: "cli";
     readonly templateVersion: "1.0.0";
     readonly platformCompatibility: ">=1.0.0 <2.0.0";

package/dist/deployment/descriptor.js

@@ -3,7 +3,7 @@ export const descriptor = {
     contractVersion: "1.0.0",
     moduleRef: "platform-cli",
     packageName: "@tomflow/proflow-platform-cli",
-    moduleVersion: "0.1.5",
+    moduleVersion: "0.1.7",
     kind: "cli",
     templateVersion: "1.0.0",
     platformCompatibility: ">=1.0.0 <2.0.0",

package/dist/src/apply/current.d.ts

@@ -1,4 +1,5 @@
 import type { DeploymentPlan } from "../contracts.ts";
 import type { ModuleCatalog } from "../modules.ts";
+import type { WorkspacePaths } from "../paths.ts";
 import type { PlanInput } from "../planner/plan.ts";
-export declare function rebuildCurrentAssumptions(catalog: ModuleCatalog, plan: DeploymentPlan): Promise<PlanInput>;
+export declare function rebuildCurrentAssumptions(catalog: ModuleCatalog, plan: DeploymentPlan, paths: WorkspacePaths): Promise<PlanInput>;

package/dist/src/apply/current.js

@@ -1,6 +1,7 @@
 import { parseModuleDescriptor, } from "@tomflow/proflow-module-contract";
 import { discoverModules } from "../discovery/index.js";
 import { doctorModules } from "../doctor/index.js";
+import { mergeEffectiveConfig } from "../persistence/effective-config.js";
 import { diagnoseRepair } from "../planner/repair.js";
 function moduleSourceOf(module) {
     if (module.source.type === "registry") {
@@ -27,7 +28,7 @@ async function loadDescriptors(catalog, modules) {
 // are re-discovered and re-parsed so descriptor drift reaches the staleness
 // gate. Upgrade additionally re-discovers current descriptors and reuses the
 // plan's frozen target descriptors; repair re-runs doctor for fresh facts.
-export async function rebuildCurrentAssumptions(catalog, plan) {
+export async function rebuildCurrentAssumptions(catalog, plan, paths) {
     if ((plan.intent === "install" || plan.intent === "upgrade") &&
         plan.resolvedModules.every((module) => module.source.type === "registry")) {
         return {
@@ -46,12 +47,19 @@ export async function rebuildCurrentAssumptions(catalog, plan) {
     }
     if (plan.intent === "upgrade") {
         const currentDescriptors = await loadDescriptors(catalog, modules);
+        const config = await mergeEffectiveConfig(paths, modules, {});
         if (plan.targetDescriptors === undefined)
-            return { intent: plan.intent, modules, targets: plan.moduleTargets };
+            return {
+                intent: plan.intent,
+                modules,
+                targets: plan.moduleTargets,
+                config,
+            };
         return {
             intent: plan.intent,
             modules,
             targets: plan.moduleTargets,
+            config,
             currentDescriptors,
             targetDescriptors: plan.targetDescriptors,
         };

package/dist/src/cli.js

@@ -20,6 +20,7 @@ import { buildManifest } from "./manifest/manifest.js";
 import { writeDeploymentObserverSummary } from "./observer/deployment-summary.js";
 import { workspacePaths } from "./paths.js";
 import { loadConfig } from "./persistence/config.js";
+import { mergeEffectiveConfig } from "./persistence/effective-config.js";
 import { loadPlan, savePlan } from "./persistence/plans.js";
 import { planDeployment } from "./planner/plan.js";
 import { diagnoseRepair } from "./planner/repair.js";
@@ -309,30 +310,13 @@ async function handleInstallerPreflight(root) {
             : "BLOCKED";
     return outcome("preflight", status, result);
 }
-async function loadEffectiveConfig(ctx, modules, configFile) {
-    const effective = {};
-    for (const module of modules) {
-        const stored = await loadConfig(ctx.paths, module.moduleRef);
-        if (stored === undefined)
-            continue;
-        effective[module.moduleRef] = {
-            ...stored.publicValues,
-            ...stored.secretValues,
-        };
-    }
-    const provided = await loadConfigFile(configFile);
-    for (const [moduleRef, values] of Object.entries(provided)) {
-        effective[moduleRef] = {
-            ...(effective[moduleRef] ?? {}),
-            ...values,
-        };
-    }
-    return effective;
+async function loadEffectiveConfig(paths, modules, configFile) {
+    return mergeEffectiveConfig(paths, modules, await loadConfigFile(configFile));
 }
 async function handlePreflight(ctx, args) {
     const modules = await discoverModules({ catalog: ctx.catalog });
     const selected = selectModules(modules, args.positional[0]);
-    const config = await loadEffectiveConfig(ctx, selected, args.configFile);
+    const config = await loadEffectiveConfig(ctx.paths, selected, args.configFile);
     const result = await runPreflight(selected, {
         config,
         catalog: ctx.catalog,
@@ -381,7 +365,9 @@ async function handlePlan(ctx, args) {
     }
     const modules = await discoverModules({ catalog: ctx.catalog });
     const selected = selectModules(modules, args.positional[0]);
-    const config = await loadConfigFile(args.configFile);
+    const config = args.intent === "upgrade"
+        ? await loadEffectiveConfig(ctx.paths, selected, args.configFile)
+        : await loadConfigFile(args.configFile);
     if (intent === "uninstall") {
         if (args.positional[0] === undefined) {
             throw new PlatformError("INVALID_REQUEST", "uninstall plan requires <moduleRef|packageName>");
@@ -497,7 +483,7 @@ async function handleApply(ctx, args) {
     if (plan === undefined) {
         throw new PlatformError("PLAN_NOT_FOUND", `plan ${planRef} not found`);
     }
-    const current = await rebuildCurrentAssumptions(ctx.catalog, plan);
+    const current = await rebuildCurrentAssumptions(ctx.catalog, plan, ctx.paths);
     const result = await applyPlan({
         paths: ctx.paths,
         planRef,
@@ -839,7 +825,7 @@ async function handlePlatformInstanceUninstall(args, runtime) {
                 uninstallScope: "platform-instance",
             });
             await savePlan(ctx.paths, plan);
-            const current = await rebuildCurrentAssumptions(ctx.catalog, plan);
+            const current = await rebuildCurrentAssumptions(ctx.catalog, plan, ctx.paths);
             const applied = await applyPlan({
                 paths: ctx.paths,
                 planRef: plan.planRef,
@@ -959,7 +945,7 @@ async function handleDoctor(ctx, args) {
 async function handleManifest(ctx, args) {
     const modules = await discoverModules({ catalog: ctx.catalog });
     const selected = selectModules(modules, args.positional[0]);
-    const config = await loadEffectiveConfig(ctx, selected, args.configFile);
+    const config = await loadEffectiveConfig(ctx.paths, selected, args.configFile);
     const manifest = await buildManifest({
         catalog: ctx.catalog,
         modules: selected,

package/dist/src/persistence/effective-config.d.ts

@@ -0,0 +1,3 @@
+import type { ResolvedModule } from "../contracts.ts";
+import type { WorkspacePaths } from "../paths.ts";
+export declare function mergeEffectiveConfig(paths: WorkspacePaths, modules: readonly ResolvedModule[], provided: Record<string, Record<string, string>>): Promise<Record<string, Record<string, string>>>;

package/dist/src/persistence/effective-config.js

@@ -0,0 +1,20 @@
+import { loadConfig } from "./config.js";
+export async function mergeEffectiveConfig(paths, modules, provided) {
+    const effective = {};
+    for (const module of modules) {
+        const stored = await loadConfig(paths, module.moduleRef);
+        if (stored === undefined)
+            continue;
+        effective[module.moduleRef] = {
+            ...stored.publicValues,
+            ...stored.secretValues,
+        };
+    }
+    for (const [moduleRef, values] of Object.entries(provided)) {
+        effective[moduleRef] = {
+            ...(effective[moduleRef] ?? {}),
+            ...values,
+        };
+    }
+    return effective;
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tomflow/proflow-platform-cli",
-  "version": "0.1.5",
+  "version": "0.1.7",
   "type": "module",
   "publishConfig": {
     "access": "public"
@@ -24,8 +24,8 @@
     "@tomflow/proflow-module-contract": "^0.1.1"
   },
   "devDependencies": {
-    "@tomflow/proflow-module-template": "^0.1.2",
-    "@tomflow/proflow-deployment-conformance": "^0.1.2"
+    "@tomflow/proflow-deployment-conformance": "^0.1.2",
+    "@tomflow/proflow-module-template": "^0.1.2"
   },
   "description": "Deterministic platform-level deployment discovery, planning, lifecycle and verification CLI.",
   "keywords": [

package/proflow.module.json

@@ -3,7 +3,7 @@
 	"contractVersion": "1.0.0",
 	"moduleRef": "platform-cli",
 	"packageName": "@tomflow/proflow-platform-cli",
-	"moduleVersion": "0.1.5",
+	"moduleVersion": "0.1.7",
 	"kind": "cli",
 	"templateVersion": "1.0.0",
 	"platformCompatibility": ">=1.0.0 <2.0.0",