aienvmp 0.1.41 → 0.1.42

package/BUGFIXES.md

@@ -34,6 +34,12 @@ Short record of bugs, fixes, and follow-up checks.
 - Fix: npm and Python security summaries now include fix versions and advisory references when scanners provide them.
 - Verification: parser tests cover npm remediation objects and pip-audit advisory ids; Windows and macOS tarball tests completed `sync --security`.
 
+### Advisory doctor behavior needed clearer verification
+
+- Issue: `doctor` warnings can look like failures to AI/CI consumers even though local operation should stay non-blocking by default.
+- Fix: `doctor --json` now exposes `exitBehavior`, and enforcement gate metadata explains when strict mode sets a failure exit code.
+- Verification: Windows and macOS candidate smoke checks confirmed default `doctor --json` exits successfully while `doctor --strict policy --json` fails on matching policy warnings.
+
 ## Template
 
 ### Title

package/CHANGELOG.md

@@ -1,5 +1,14 @@
 # Changelog
 
+## 0.1.42
+
+- Added explicit enforcement gate metadata so AI and CI consumers know local checks are warn-only unless `--strict` or `--ci` is requested.
+- Added `doctor --json` exit behavior metadata to distinguish advisory warnings from strict failure conditions.
+- Surfaced enforcement gate details in context, plan, and dashboard outputs.
+- Extended the GitHub Action to write `.aienvmp/schema.json` and `.aienvmp/doctor.json` artifacts by default.
+- Clarified README guidance for advisory-by-default behavior and opt-in strict/CI failure.
+- Documented advisory doctor exit behavior and strict verification steps in troubleshooting and bugfix notes.
+
 ## 0.1.41
 
 - Added preflight contract metadata so AI and CI consumers can rely on stable entry fields while ignoring additive changes.

package/README.md

@@ -28,6 +28,7 @@ npx aienvmp handoff --record --actor agent:codex
 ```
 
 Use `--dir <workspace>` when AI or CI runs outside the target project.
+Warnings are advisory by default. Use `doctor --strict <scope>` only when you want CI-style failure.
 
 ## What It Creates
 
@@ -50,7 +51,8 @@ AIENV.md                 # Markdown env map for AI agents
 - `coordination.conflictTargets` shows where multiple agents are planning changes.
 - `handoff` carries dependency read-set and protocol guidance for the next AI.
 - Light SBOM includes source/confidence hints; verify security claims with dedicated scanners.
-- Everything is advisory by default; strict failure is opt-in with `doctor --strict`.
+- `enforcementProfile.gate` explains when checks warn, fail, and set exit codes.
+- Everything is advisory by default; strict failure is opt-in with `doctor --strict` or `--ci`.
 
 ## Agent Files
 
@@ -78,6 +80,8 @@ aienvmp doctor --strict security|policy|coordination|all
 
 ## CI
 
+The GitHub Action writes status, schema, doctor, plan, and dashboard artifacts. `strict: "off"` reports warnings without failing the job.
+
 ```yaml
 - uses: soovwv/aienvmp@main
   with:

package/TROUBLESHOOTING.md

@@ -101,3 +101,33 @@ Then retry:
 ```bash
 npx aienvmp context
 ```
+
+## Doctor shows warnings but exits successfully
+
+Symptom:
+
+```text
+npx aienvmp doctor --json
+# status: warning
+# exit code: 0
+```
+
+Cause:
+
+- Local checks are advisory by default so shared machines are not blocked unexpectedly.
+
+Fail CI explicitly:
+
+```bash
+npx aienvmp doctor --strict policy
+npx aienvmp doctor --strict security
+npx aienvmp doctor --strict coordination
+```
+
+Check the machine-readable rule:
+
+```bash
+npx aienvmp doctor --json
+```
+
+Read `exitBehavior` and `strict.gate` to see when a failure exit code will be set.

package/action.yml

@@ -19,6 +19,14 @@ inputs:
     description: Write compact AI status artifact
     required: false
     default: "true"
+  write-schema:
+    description: Write stable AI output contract artifact
+    required: false
+    default: "true"
+  write-doctor-json:
+    description: Write doctor JSON with advisory/strict exit behavior
+    required: false
+    default: "true"
 
 runs:
   using: composite
@@ -41,6 +49,22 @@ runs:
           node "$GITHUB_ACTION_PATH/bin/aienvmp.js" status --dir "${{ inputs.directory }}" --write --quiet
         fi
 
+    - name: Write AI schema
+      shell: bash
+      run: |
+        if [ "${{ inputs.write-schema }}" = "true" ]; then
+          mkdir -p "${{ inputs.directory }}/.aienvmp"
+          node "$GITHUB_ACTION_PATH/bin/aienvmp.js" schema --json > "${{ inputs.directory }}/.aienvmp/schema.json"
+        fi
+
+    - name: Write doctor advisory JSON
+      shell: bash
+      run: |
+        if [ "${{ inputs.write-doctor-json }}" = "true" ]; then
+          mkdir -p "${{ inputs.directory }}/.aienvmp"
+          node "$GITHUB_ACTION_PATH/bin/aienvmp.js" doctor --dir "${{ inputs.directory }}" --json > "${{ inputs.directory }}/.aienvmp/doctor.json"
+        fi
+
     - name: Doctor
       shell: bash
       run: |

package/examples/github-action.yml

@@ -14,6 +14,8 @@ jobs:
         with:
           directory: "."
           write-plan: "true"
+          write-schema: "true"
+          write-doctor-json: "true"
           strict: "off" # security, policy, coordination, all, or off
 
       - uses: actions/upload-artifact@v4
@@ -24,6 +26,8 @@ jobs:
             AIENV.md
             .aienvmp/manifest.json
             .aienvmp/status.json
+            .aienvmp/schema.json
+            .aienvmp/doctor.json
             .aienvmp/plan.json
             .aienvmp/plan.md
             .aienvmp/dashboard.html

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "aienvmp",
-  "version": "0.1.41",
+  "version": "0.1.42",
   "description": "AI-first environment maps and lightweight runtime SBOMs for shared development machines.",
   "type": "module",
   "bin": {

package/src/commands/context.js

@@ -50,7 +50,7 @@ export async function contextWorkspace(args) {
     }, null, 2));
     return;
   }
-  console.log(renderContext(manifest, timeline, warnings, intents, policy, actions));
+  console.log(renderContext({ ...manifest, preflight }, timeline, warnings, intents, policy, actions));
 }
 
 function lightSbomSummary(lightSbom = {}) {

package/src/commands/doctor.js

@@ -18,9 +18,18 @@ export async function doctorWorkspace(args) {
   const warnings = [...diagnose(manifest, { timeline, intents }), ...policyWarnings(manifest, policy)];
   const actions = recommendedActions(manifest, { warnings, intents });
   const strict = strictResult(warnings, args);
+  const exitBehavior = {
+    mode: strict.enabled ? "strict" : "advisory",
+    willSetFailureExitCode: strict.fail,
+    reason: strict.enabled
+      ? `strict scope ${strict.scope} is enabled`
+      : "strict mode is off; warnings are reported without failing local operation",
+    gate: strict.gate
+  };
   if (args.json) {
     console.log(JSON.stringify({
       status: strict.fail ? "fail" : warnings.length ? "warning" : "ok",
+      exitBehavior,
       trust: manifest.trust || {},
       policy,
       openIntentCount: intents.length,

package/src/enforcement.js

@@ -3,12 +3,14 @@ const STRICT_SCOPES = ["security", "policy", "coordination", "all"];
 export function strictResult(warnings = [], args = {}) {
   const scope = normalizeStrictScope(args.strict || (args.ci ? "all" : ""));
   const matchedWarnings = scope ? warnings.filter((warning) => warningMatchesScope(warning, scope)) : [];
+  const gate = enforcementGate(scope);
   return {
     enabled: Boolean(scope),
     scope: scope || "off",
     fail: matchedWarnings.length > 0,
     matchedWarningCodes: matchedWarnings.map((warning) => warning.code),
-    availableScopes: STRICT_SCOPES
+    availableScopes: STRICT_SCOPES,
+    gate
   };
 }
 
@@ -28,6 +30,7 @@ export function enforcementAdvice(warnings = []) {
     mode: "advisory-by-default",
     localBehavior: "non-blocking",
     ciBehavior: "strict-only-when-requested",
+    gate: enforcementGate(""),
     suggestedStrictScopes,
     scopes: scopeResults,
     recommendedCommand: suggestedStrictScopes.length
@@ -37,6 +40,18 @@ export function enforcementAdvice(warnings = []) {
   };
 }
 
+export function enforcementGate(scope = "") {
+  const strictScope = normalizeStrictScope(scope);
+  return {
+    defaultMode: "advisory",
+    strictMode: strictScope || "off",
+    localDefault: "warn-only",
+    failCondition: strictScope ? `matching warnings in ${strictScope}` : "never in default mode",
+    exitCode: strictScope ? "1 when matching warnings exist" : "0 unless the command itself errors",
+    rule: "Do not block local or shared machine operation unless --strict or --ci is explicitly requested."
+  };
+}
+
 function normalizeStrictScope(value) {
   if (value === true) return "all";
   const scope = String(value || "").trim().toLowerCase();

package/src/preflight.js

@@ -1,6 +1,6 @@
 import { recommendedActions } from "./actions.js";
 import { aiDecision } from "./decision.js";
-import { enforcementAdvice } from "./enforcement.js";
+import { enforcementAdvice, enforcementGate } from "./enforcement.js";
 import { preflightContract } from "./contract.js";
 
 export function buildPreflight(manifest = {}, warnings = [], intents = []) {
@@ -27,6 +27,7 @@ export function buildPreflight(manifest = {}, warnings = [], intents = []) {
       localOperation: "non-blocking",
       strictMode: "optional",
       strictUse: "CI or explicit human-requested checks only",
+      gate: enforcementGate(""),
       reason: "Avoid disrupting shared servers or developer machines while still making drift visible.",
       recommendedStrictCommand: enforcement.recommendedCommand,
       strictCommands: [

package/src/render.js

@@ -170,6 +170,9 @@ export function renderContext(manifest, timeline = [], warnings = [], intents =
     "Recommended actions:",
     ...(recommendedActions.length ? recommendedActions.map((item) => `- [${item.priority}] ${item.summary}${item.command ? ` (${item.command})` : ""}`) : ["- none"]),
     "",
+    "Enforcement gate:",
+    ...enforcementGateLines(manifest.preflight?.enforcementProfile?.gate),
+    "",
     "Open intents:",
     ...(intents.length ? intents.slice(-5).reverse().map((i) => `- ${i.actor}: ${i.action}`) : ["- none"]),
     "",
@@ -267,6 +270,9 @@ export function renderPlan(plan) {
     "Review gates:",
     ...plan.reviewGates.map((item) => `- ${item}`),
     "",
+    "Enforcement gate:",
+    ...enforcementGateLines(plan.preflight?.enforcementProfile?.gate),
+    "",
     "Dependency protocol:",
     ...dependencyProtocolPlanLines(plan.preflight?.dependencyChangeProtocol),
     "",
@@ -294,6 +300,15 @@ function dependencyProtocolPlanLines(protocol = {}) {
   ];
 }
 
+function enforcementGateLines(gate = {}) {
+  return [
+    `- Default: ${gate.defaultMode || "advisory"} (${gate.localDefault || "warn-only"})`,
+    `- Strict: ${gate.strictMode || "off"}`,
+    `- Fail condition: ${gate.failCondition || "never in default mode"}`,
+    `- Exit code: ${gate.exitCode || "0 unless the command itself errors"}`
+  ];
+}
+
 function environmentLines(item) {
   return [
     `- ${item.category}: ${item.summary}`,
@@ -414,7 +429,8 @@ const ciHasFailure=ciReadiness.some(s=>s.status==='fail');
 const ciReadinessHtml=ciReadiness.length?'<table>'+ciReadiness.map(s=>\`<tr><th>\${esc(s.scope)}</th><td><code>\${esc(s.status)}</code>\${s.matchedWarningCodes?.length?\` \${esc(s.matchedWarningCodes.join(', '))}\`:''}</td></tr>\`).join('')+'</table>':'<div class="okline">Run <code>aienvmp doctor --strict security|policy|coordination|all</code> to choose CI enforcement scope.</div>';
 const enforcementProfile=manifest.preflight?.enforcementProfile||{};
 const strictCommands=enforcementProfile.strictCommands||[];
-const enforcementHtml=\`<table><tr><th>Default</th><td><code>\${esc(enforcementProfile.defaultMode||'advisory')}</code></td></tr><tr><th>Local</th><td>\${esc(enforcementProfile.localOperation||'non-blocking')}</td></tr><tr><th>Strict</th><td>\${esc(enforcementProfile.strictUse||'CI or explicit checks only')}</td></tr><tr><th>Recommended</th><td><code>\${esc(enforcementProfile.recommendedStrictCommand||'aienvmp doctor --strict all')}</code></td></tr></table><div class="timeline">\${strictCommands.slice(0,4).map(cmd=>\`<div class="event"><time>CI</time><div><code>\${esc(cmd)}</code></div></div>\`).join('')}</div><div class="path">\${esc(enforcementProfile.reason||'Warnings stay advisory unless strict mode is requested.')}</div>\`;
+const gate=enforcementProfile.gate||{};
+const enforcementHtml=\`<table><tr><th>Default</th><td><code>\${esc(gate.defaultMode||enforcementProfile.defaultMode||'advisory')}</code> \${esc(gate.localDefault||'warn-only')}</td></tr><tr><th>Strict</th><td><code>\${esc(gate.strictMode||'off')}</code></td></tr><tr><th>Fail</th><td>\${esc(gate.failCondition||'never in default mode')}</td></tr><tr><th>Exit</th><td>\${esc(gate.exitCode||'0 unless the command itself errors')}</td></tr><tr><th>Recommended</th><td><code>\${esc(enforcementProfile.recommendedStrictCommand||'aienvmp doctor --strict all')}</code></td></tr></table><div class="timeline">\${strictCommands.slice(0,4).map(cmd=>\`<div class="event"><time>CI</time><div><code>\${esc(cmd)}</code></div></div>\`).join('')}</div><div class="path">\${esc(gate.rule||enforcementProfile.reason||'Warnings stay advisory unless strict mode is requested.')}</div>\`;
 const contract=manifest.preflight?.contract||{};
 const contractHtml=contract.name?\`<table><tr><th>Name</th><td><code>\${esc(contract.name)}</code></td></tr><tr><th>Version</th><td><code>\${esc(contract.version||1)}</code></td></tr><tr><th>Stability</th><td><code>\${esc(contract.stability||'additive')}</code></td></tr><tr><th>AI fields</th><td>\${esc((contract.aiEntryFields||[]).join(', ')||'none')}</td></tr></table><div class="path">\${esc(contract.rule||'Ignore unknown fields.')}</div>\`:'<div class="okline">Run <code>aienvmp status --write</code> to include AI contract metadata.</div>';
 const intentTargets=manifest.preflight?.intentTargets||[];