@aliou/pi-guardrails 0.13.0 → 0.13.2

package/extensions/guardrails/commands/settings/index.ts

@@ -441,13 +441,15 @@ export function registerGuardrailsSettings(
 
       if (scope === "global") {
         featureItems.push({
-          id: "onboarding.run",
+          id: "onboarding.completed",
           label: "Onboarding status",
-          description: "Use /guardrails:onboarding to run onboarding",
+          description:
+            "Reset to pending to re-run onboarding (takes effect after reload)",
           currentValue:
             scopedConfig.onboarding?.completed === true
               ? "completed"
               : "pending",
+          values: ["completed", "pending"],
         });
       }
 
@@ -592,5 +594,43 @@ export function registerGuardrailsSettings(
         },
       ];
     },
+
+    onSettingChange: (id, newValue, config) => {
+      const updated = structuredClone(config);
+
+      if (id.startsWith("features.")) {
+        const featureKey = id.slice("features.".length);
+        updated.features = {
+          ...updated.features,
+          [featureKey]: newValue === "enabled",
+        };
+        return updated;
+      }
+
+      if (id === "permissionGate.requireConfirmation") {
+        updated.permissionGate = {
+          ...updated.permissionGate,
+          requireConfirmation: newValue === "on",
+        };
+        return updated;
+      }
+
+      if (id === "onboarding.completed") {
+        updated.onboarding = {
+          ...updated.onboarding,
+          completed: newValue === "completed",
+          completedAt:
+            newValue === "completed"
+              ? (updated.onboarding?.completedAt ?? new Date().toISOString())
+              : undefined,
+          version:
+            newValue === "completed" ? updated.onboarding?.version : undefined,
+        };
+        return updated;
+      }
+
+      // Fall through to default string storage for enums (pathAccess.mode, etc.)
+      return null;
+    },
   });
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@aliou/pi-guardrails",
-  "version": "0.13.0",
+  "version": "0.13.2",
   "license": "MIT",
   "type": "module",
   "private": false,
@@ -39,8 +39,8 @@
     "@aliou/sh": "^0.1.0"
   },
   "peerDependencies": {
-    "@earendil-works/pi-coding-agent": "0.74.0",
-    "@earendil-works/pi-tui": "0.74.0"
+    "@earendil-works/pi-coding-agent": "*",
+    "@earendil-works/pi-tui": "*"
   },
   "devDependencies": {
     "@aliou/biome-plugins": "^0.8.1",

package/src/shared/config/migration/008-normalize-string-booleans.ts

@@ -0,0 +1,57 @@
+import { addPendingWarning } from "../../warnings";
+import type { GuardrailsConfig } from "../types";
+import { CURRENT_VERSION } from "./version";
+
+export function shouldRun(config: GuardrailsConfig): boolean {
+  const features = config.features as Record<string, unknown> | undefined;
+  if (features) {
+    for (const value of Object.values(features)) {
+      if (value === "enabled" || value === "disabled") return true;
+    }
+  }
+
+  const requiresConfirmation = config.permissionGate
+    ?.requireConfirmation as unknown;
+  if (requiresConfirmation === "on" || requiresConfirmation === "off") {
+    return true;
+  }
+
+  return false;
+}
+
+export function run(config: GuardrailsConfig): GuardrailsConfig {
+  const migrated = structuredClone(config) as Record<string, unknown>;
+  let changed = false;
+
+  const features = migrated.features as Record<string, unknown> | undefined;
+  if (features) {
+    for (const [key, value] of Object.entries(features)) {
+      if (value === "enabled" || value === "disabled") {
+        features[key] = value === "enabled";
+        changed = true;
+      }
+    }
+  }
+
+  const permissionGate = migrated.permissionGate as
+    | Record<string, unknown>
+    | undefined;
+  if (
+    permissionGate &&
+    (permissionGate.requireConfirmation === "on" ||
+      permissionGate.requireConfirmation === "off")
+  ) {
+    permissionGate.requireConfirmation =
+      permissionGate.requireConfirmation === "on";
+    changed = true;
+  }
+
+  if (changed) {
+    migrated.version = CURRENT_VERSION;
+    addPendingWarning(
+      "[guardrails] Config migrated: boolean settings stored as strings were converted to true/false.",
+    );
+  }
+
+  return migrated as GuardrailsConfig;
+}

package/src/shared/config/migration/index.ts

@@ -7,6 +7,7 @@ import * as envFilesToPolicies from "./004-env-files-to-policies";
 import * as normalizeAllowedPaths from "./005-normalize-allowed-paths";
 import * as applyBuiltinDefaults from "./006-apply-builtin-defaults";
 import * as markOnboardingDone from "./007-mark-onboarding-done";
+import * as normalizeStringBooleans from "./008-normalize-string-booleans";
 
 export { CURRENT_VERSION } from "./version";
 
@@ -36,6 +37,11 @@ export const migrations: Migration<GuardrailsConfig>[] = [
     shouldRun: normalizeAllowedPaths.shouldRun,
     run: normalizeAllowedPaths.run,
   },
+  {
+    name: "normalize-string-booleans",
+    shouldRun: normalizeStringBooleans.shouldRun,
+    run: normalizeStringBooleans.run,
+  },
 ];
 
 export const globalConfigMigrations = [