@agentsfleet/orly 0.10.2 → 0.10.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agentsfleet/orly",
-  "version": "0.10.2",
+  "version": "0.10.4",
   "description": "Repository-scoped engineering harness: renders agent rules, materialises the gates that enforce them, and proves the PR boundary \u2014 for any coding agent.",
   "license": "MIT",
   "type": "module",

package/src/config.test.ts

@@ -180,6 +180,24 @@ describe("managedDrift — content", () => {
     expect(drift[0]).toContain("move the change into the pack source");
   });
 
+  test("a digest is algorithm-prefixed, and the unprefixed spelling still matches", async () => {
+    // Bare hex is indistinguishable from a credential to an entropy scanner:
+    // gitleaks refused the first commit that carried these, in every consuming
+    // repository at once. The prefix is what makes it pass without a scanner
+    // suppression — and 0.10.2's unprefixed records must not read as drift.
+    const bytes = new TextEncoder().encode("managed\n");
+    const digest = contentDigest(bytes);
+    expect(digest.startsWith("sha256:")).toBe(true);
+
+    const root = scratch();
+    writeFileSync(join(root, "managed.md"), "managed\n");
+    const legacy = { ...(await seedConfig(root)), managed: ["managed.md"], digests: { "managed.md": digest.slice("sha256:".length) } };
+    expect(managedDrift(root, legacy)).toEqual([]);
+
+    const prefixed = { ...legacy, digests: { "managed.md": digest } };
+    expect(managedDrift(root, prefixed)).toEqual([]);
+  });
+
   test("a file with no recorded digest makes no claim", async () => {
     // Two cases share this path: a config written before digests existed, and a
     // hook orly manages but did not author. Neither may be reported as drift.

package/src/config.ts

@@ -1,8 +1,7 @@
-import { createHash } from "node:crypto";
 import { existsSync, mkdirSync, readdirSync, readFileSync } from "node:fs";
 import { dirname, extname, join, relative } from "node:path";
 
-import { assertWritableInside, isObject, isString, JsonObject, objectValue, OrlyError, readJsonObject, RulesModel, stringArray } from "./model";
+import { assertWritableInside, hashContent, isObject, isString, JsonObject, objectValue, OrlyError, readJsonObject, RulesModel, stringArray } from "./model";
 import { UNSCOPED_ENVIRONMENT } from "./git_env";
 import { validateCommands, validateSurfaces } from "./validation";
 
@@ -19,7 +18,6 @@ const VERSION_FIELD = "orly_version";
 const MANAGED_FIELD = "managed";
 const DIGESTS_FIELD = "digests";
 const DIGEST_ALGORITHM = "sha256";
-const DIGEST_ENCODING = "hex";
 const COMMANDS_FIELD = "commands";
 const CONFORM_COMMAND = "conform";
 const MAKE_COMMAND = "make";
@@ -130,8 +128,25 @@ function readDigests(value: unknown): Record<string, string> {
 }
 
 /// The digest orly records for content it wrote, and recomputes to check it.
+///
+/// Algorithm-prefixed, and not for elegance: a bare 64-character hex string is
+/// indistinguishable from a credential to an entropy-based secret scanner, and
+/// gitleaks refused the very first commit that carried these — in EVERY
+/// consuming repository at once, since `orly update` writes one per managed
+/// file. The prefix is what makes the value self-describing enough to pass,
+/// and the alternative was a scanner suppression, which is how a real
+/// credential eventually rides through.
 export function contentDigest(bytes: Uint8Array): string {
-  return createHash(DIGEST_ALGORITHM).update(bytes).digest(DIGEST_ENCODING);
+  return `${DIGEST_ALGORITHM}:${hashContent(bytes)}`;
+}
+
+// Tolerates the unprefixed spelling 0.10.2 wrote for its one release, so a
+// checkout installed on that version is not reported as wholly drifted before
+// its next update — a format change is not an edit, and saying so would be the
+// false alarm that teaches people to ignore the real ones.
+function sameDigest(recorded: string, actual: string): boolean {
+  const bare = (digest: string) => digest.startsWith(`${DIGEST_ALGORITHM}:`) ? digest.slice(DIGEST_ALGORITHM.length + 1) : digest;
+  return bare(recorded) === bare(actual);
 }
 
 // What orly installed here, and whether the engine has moved since.
@@ -196,7 +211,7 @@ function editedManaged(targetRoot: string, config: RepoConfig, present: string[]
     const recorded = config.digests[relativePath];
     if (!recorded) return [];
     const actual = contentDigest(readFileSync(join(targetRoot, relativePath)));
-    if (actual === recorded) return [];
+    if (sameDigest(recorded, actual)) return [];
     return [`managed file was edited after orly wrote it: ${relativePath} — move the change into the pack source, or \`orly update\` to discard it`];
   });
 }