@awak-app/simy-cli 0.5.15 → 0.5.17

package/src/runner.js

@@ -19,6 +19,7 @@ import {
   redactExecutionText,
 } from "./orchestrator/execution-io.js";
 import { summarizeCodingLoopEvent } from "./orchestrator/presentation.js";
+import { PR_PREPARATION_CHECKS } from "./orchestrator/pr-preparation.js";
 import { increaseProviderTokenBudget, refreshBudgetState } from "./orchestrator/budget.js";
 import { appendEvent } from "./orchestrator/shared.js";
 import {
@@ -73,6 +74,7 @@ const INTERRUPTED_STATES = new Set([
   "re_instructing",
 ]);
 const RESTORABLE_STATES = new Set([...RESUMABLE_STATES, ...INTERRUPTED_STATES]);
+const PR_PREPARATION_IDS = new Set(PR_PREPARATION_CHECKS.map((row) => row.id));
 
 export { LocalRunRegistry };
 
@@ -530,6 +532,18 @@ async function runLocalCodingRun(
 
   markAttachmentsDelivered(run.request.attachments);
 
+  // All collectors and the executor must use the same root. Bind a missing
+  // default only after resolving the authorized checkout, including on resume.
+  if (!run.snapshot.charter.ui_evidence_root) {
+    const evidenceRoot = path.join(repositoryPath, ".artifacts");
+    run.snapshot.charter.ui_evidence_root = evidenceRoot;
+    run.request.ui_evidence_root = evidenceRoot;
+    appendEvent(run.snapshot, run.status, "Local evidence root bound to the verified checkout.", {
+      code: "local_evidence_root_bound",
+    });
+    await updateRun(run, run.status);
+  }
+
   const executor =
     executeAttempt ||
     ((context) =>
@@ -1007,9 +1021,90 @@ function redactedAttemptForLedger(attempt, attachments) {
     },
     raw_output: structuredResultForLedger(attempt.raw_output),
     observed_evidence: observedEvidenceForLedger(attempt.observed_evidence),
+    pr_preparation: prPreparationForLedger(attempt.pr_preparation),
+    skipped_checks: skippedChecksForLedger(attempt.skipped_checks),
   };
 }
 
+function prPreparationForLedger(value) {
+  return (Array.isArray(value) ? value : []).map((row) => ({
+    id: PR_PREPARATION_IDS.has(row?.id) ? row.id : "unrecognized",
+    applicability: row?.applicability || null,
+    passed: row?.passed === true,
+    observation_present: Boolean(row?.observation),
+    reason_present: Boolean(row?.reason),
+    authoritative_source_present: Boolean(row?.authoritative_source),
+    evidence_ref_count: Array.isArray(row?.evidence_refs) ? row.evidence_refs.length : 0,
+    bound_revisions: validatedHashesForLedger(row?.bound_revisions, 40),
+    companion_revisions: (Array.isArray(row?.companion_revisions) ? row.companion_revisions : [])
+      .map((item) => ({
+        repository_present: Boolean(item?.repository),
+        sha: validatedHashForLedger(item?.sha, 40),
+      })),
+    manifest_revisions: (Array.isArray(row?.manifest_revisions) ? row.manifest_revisions : [])
+      .map((item) => ({
+        manifest_present: Boolean(item?.manifest),
+        sha256: validatedHashForLedger(item?.sha256, 64),
+      })),
+    cleanup_residual_count: Number.isInteger(row?.cleanup_residual_count)
+      ? row.cleanup_residual_count
+      : null,
+    security_privacy_check_count: Array.isArray(row?.security_privacy_checks)
+      ? row.security_privacy_checks.length
+      : 0,
+    browser_artifact_count: Array.isArray(row?.browser_artifacts)
+      ? row.browser_artifacts.length
+      : 0,
+    locale_count: Array.isArray(row?.locales_checked) ? row.locales_checked.length : 0,
+    console_error_count: Number.isInteger(row?.console_error_count)
+      ? row.console_error_count
+      : null,
+    network_failure_count: Number.isInteger(row?.network_failure_count)
+      ? row.network_failure_count
+      : null,
+    heading_levels_checked: (Array.isArray(row?.heading_levels_checked)
+      ? row.heading_levels_checked
+      : []).filter((item) => Number.isInteger(item) && item >= 1 && item <= 6),
+    leading_space_checked: row?.leading_space_checked === true,
+    trailing_hash_checked: row?.trailing_hash_checked === true,
+    single_official_checklist: row?.single_official_checklist === true,
+    unverified_model_output_labeled: row?.unverified_model_output_labeled === true,
+    deployment_target_present: Boolean(row?.deployment_target),
+    deployment_target_source_present: Boolean(row?.deployment_target_source),
+    revision_observation_method_present: Boolean(row?.revision_observation_method),
+    live_write_policy: ["no_persistence", "scoped_fixture", "authorized_production_write"]
+      .includes(row?.live_write_policy) ? row.live_write_policy : null,
+    resume_paths_checked: (Array.isArray(row?.resume_paths_checked)
+      ? row.resume_paths_checked
+      : []).filter((item) => ["normal", "paid", "checkpoint"].includes(item)),
+    paid_resume_support: ["supported", "not_supported"].includes(row?.paid_resume_support)
+      ? row.paid_resume_support
+      : null,
+    paid_resume_support_source_present: Boolean(row?.paid_resume_support_source),
+    derived_views_recomputed: row?.derived_views_recomputed === true,
+    artifact_content_preserved: row?.artifact_content_preserved === true,
+  }));
+}
+
+function skippedChecksForLedger(value) {
+  return (Array.isArray(value) ? value : []).map((row) => ({
+    name_present: Boolean(row?.name),
+    reason_present: Boolean(row?.reason),
+    impact_present: Boolean(row?.impact),
+  }));
+}
+
+function validatedHashesForLedger(value, length) {
+  return stringArrayForLedger(value)
+    .map((item) => validatedHashForLedger(item, length))
+    .filter(Boolean);
+}
+
+function validatedHashForLedger(value, length) {
+  const candidate = typeof value === "string" ? value.trim().toLowerCase() : "";
+  return new RegExp(`^[a-f0-9]{${length}}$`).test(candidate) ? candidate : null;
+}
+
 function redactedEventDetail(value, lifecycleState, attachments) {
   const detail = value && typeof value === "object" ? value : {};
   const result = {};
@@ -1070,7 +1165,7 @@ function observedEvidenceForLedger(value) {
       head_branch: github.head_branch || null,
       head_sha: github.head_sha || null,
       base_branch: github.base_branch || null,
-      checks: Array.isArray(github.checks) ? github.checks : [],
+      checks: githubChecksForLedger(github.checks),
       checks_present: github.checks_present === true,
       checks_passing: github.checks_passing === true,
       required_checks_present: github.required_checks_present === true,
@@ -1082,6 +1177,14 @@ function stringArrayForLedger(value) {
   return Array.isArray(value) ? value.filter((item) => typeof item === "string") : [];
 }
 
+function githubChecksForLedger(value) {
+  const allowedStates = new Set(["success", "skipped", "failure", "pending"]);
+  return (Array.isArray(value) ? value : []).map((item) => ({
+    name_present: Boolean(item?.name),
+    state: allowedStates.has(item?.state) ? item.state : "pending",
+  }));
+}
+
 export function toCompatibleCodingLoopState(state) {
   switch (state) {
     case "risk_classifying":

package/src/sqm/agent-skill.js

@@ -104,8 +104,12 @@ export async function installSqmAgentSkill({ home = homedir(), platforms = Objec
 }
 
 export async function inspectCodexPlugin({ home = homedir() } = {}) {
-  const manifest = await bundledPluginManifest();
-  const expectedDigest = await directoryDigest(BUNDLED_CODEX_PLUGIN_ROOT, { manifest });
+  const overrides = await installedPluginOverrides();
+  const manifest = await bundledPluginManifest({ overrides });
+  const expectedDigest = await directoryDigest(BUNDLED_CODEX_PLUGIN_ROOT, {
+    manifest,
+    overrides,
+  });
   const target = codexPluginPath(home);
   const installedDigest = await directoryDigest(target, { missing: null });
   const marketplacePath = codexMarketplacePath(home);
@@ -152,8 +156,12 @@ async function installCodexPlugin({ home }) {
   await rm(staged, { recursive: true, force: true });
   await rm(backup, { recursive: true, force: true });
   await cp(BUNDLED_CODEX_PLUGIN_ROOT, staged, { recursive: true, errorOnExist: true });
+  const overrides = await installedPluginOverrides();
+  for (const [relative, content] of overrides) {
+    await writeFile(path.join(staged, relative), content);
+  }
   await writeFile(path.join(staged, ".codex-plugin", "plugin.json"),
-    `${JSON.stringify(await bundledPluginManifest(), null, 2)}\n`);
+    `${JSON.stringify(await bundledPluginManifest({ overrides }), null, 2)}\n`);
   let backedUp = false;
   try {
     if (await pathExists(target)) {
@@ -303,14 +311,36 @@ async function fileDigest(target) {
   }
 }
 
-async function bundledPluginManifest() {
+async function bundledPluginManifest({ overrides = new Map() } = {}) {
   const manifest = JSON.parse(await readFile(path.join(BUNDLED_CODEX_PLUGIN_ROOT, ".codex-plugin", "plugin.json"), "utf8"));
   // A stable content-derived version refreshes Codex's cache only when the bundle changes.
-  const digest = await directoryDigest(BUNDLED_CODEX_PLUGIN_ROOT);
+  const digest = await directoryDigest(BUNDLED_CODEX_PLUGIN_ROOT, { overrides });
   return { ...manifest, version: `${manifest.version.split("+")[0]}+codex.${digest}` };
 }
 
-async function directoryDigest(root, { missing = "throw", manifest } = {}) {
+async function installedPluginOverrides() {
+  const relative = "hooks/hooks.json";
+  const source = JSON.parse(await readFile(path.join(BUNDLED_CODEX_PLUGIN_ROOT, relative), "utf8"));
+  const commandPrefix = "node ";
+  const runtime = shellQuote(process.execPath);
+  for (const groups of Object.values(source.hooks || {})) {
+    for (const group of groups || []) {
+      for (const hook of group.hooks || []) {
+        if (typeof hook.command === "string" && hook.command.startsWith(commandPrefix)) {
+          hook.command = `${runtime} ${hook.command.slice(commandPrefix.length)}`;
+        }
+      }
+    }
+  }
+  return new Map([[relative, `${JSON.stringify(source, null, 2)}\n`]]);
+}
+
+function shellQuote(value) {
+  if (process.platform === "win32") return `"${value.replaceAll('"', '""')}"`;
+  return `'${value.replaceAll("'", "'\\''")}'`;
+}
+
+async function directoryDigest(root, { missing = "throw", manifest, overrides = new Map() } = {}) {
   let rootStat;
   try {
     rootStat = await lstat(root);
@@ -336,7 +366,9 @@ async function directoryDigest(root, { missing = "throw", manifest } = {}) {
       if (!entry.isFile()) throw new Error(`${target} must be a regular file.`);
       const content = manifest && relative === ".codex-plugin/plugin.json"
         ? `${JSON.stringify(manifest, null, 2)}\n`
-        : await readFile(target);
+        : overrides.has(relative)
+          ? overrides.get(relative)
+          : await readFile(target);
       hash.update(relative).update("\0").update(content).update("\0");
     }
   };

package/src/codex-plugins/simy/skills/agentic-delivery-loop/agents/openai.yaml

@@ -1,5 +0,0 @@
-interface:
-  display_name: "Agentic Delivery Loop"
-  short_description: "Converge code changes through real live proof"
-  default_prompt: "Use $agentic-delivery-loop to take this change from reproduction through verified delivery."
-