mirror of
https://github.com/morluto/rea.git
synced 2026-11-06 14:56:32 +00:00
* fix(artifacts): preserve complete root inventory comparisons * fix(artifacts): declare inventory coverage before comparison * fix(artifacts): preserve traversal omissions and materialization limits * fix(artifacts): preserve coverage declarations and projection scope Remove the duplicate inventory-schema import. Compare original coverage declarations, including field omission, before adapting legacy pages. Require explicit scope for completeness and container-entry coverage for application projections. Source inspection and git diff --check only. Tests, builds and typechecks skipped by explicit user request. GitHub review writes paused globally. * fix(artifacts): require entry coverage for child changes * fix(artifacts): accept source-valid inventory limitations * style(artifacts): format mobile graph projection --------- Co-authored-by: morluto <76467478+morluto@users.noreply.github.com> Co-authored-by: OpenAI Codex <openai-codex@users.noreply.github.com>
74 lines
2.5 KiB
TypeScript
74 lines
2.5 KiB
TypeScript
import { execFile } from "node:child_process";
|
|
import { lstat, rename, rm, writeFile } from "node:fs/promises";
|
|
import { join } from "node:path";
|
|
import { promisify } from "node:util";
|
|
|
|
import { expect, it } from "vitest";
|
|
|
|
import {
|
|
classifyAndHashRoot,
|
|
inventoryArtifact,
|
|
} from "../../fixtures/artifactInventory.js";
|
|
import { createTestTempDirectory } from "../../fixtures/temporaryDirectory.js";
|
|
import { readWithoutFifoWriter } from "../../fixtures/fifoInput.js";
|
|
|
|
it.skipIf(process.platform === "win32")(
|
|
"rejects a FIFO artifact root without waiting for a writer",
|
|
async () => {
|
|
const root = await createTestTempDirectory("rea-artifact-root-fifo-");
|
|
const fifoPath = join(root, "input.asar");
|
|
await promisify(execFile)("mkfifo", [fifoPath]);
|
|
|
|
const outcome = await readWithoutFifoWriter(fifoPath, () =>
|
|
inventoryArtifact(fifoPath).then(
|
|
() => undefined,
|
|
(cause: unknown) => cause,
|
|
),
|
|
);
|
|
expect(outcome.state).toBe("completed");
|
|
if (outcome.state !== "completed")
|
|
throw new Error("Artifact inventory waited for a FIFO writer");
|
|
expect(outcome.result).toMatchObject({
|
|
reason: "format",
|
|
message: `Artifact root is not a regular file: ${fifoPath}`,
|
|
});
|
|
},
|
|
);
|
|
|
|
it("rejects a root path replaced between its initial stat and open", async () => {
|
|
const root = await createTestTempDirectory("rea-artifact-root-replaced-");
|
|
const path = join(root, "input.bin");
|
|
const replacement = join(root, "replacement.bin");
|
|
await writeFile(path, "first bytes\n");
|
|
const expectedMetadata = await lstat(path);
|
|
await writeFile(replacement, "different bytes\n");
|
|
await rm(path);
|
|
await rename(replacement, path);
|
|
|
|
await expect(
|
|
classifyAndHashRoot(path, false, expectedMetadata),
|
|
).rejects.toMatchObject({
|
|
reason: "integrity",
|
|
message: `Root artifact changed before inventory: ${path}`,
|
|
});
|
|
});
|
|
|
|
it("keeps unavailable container traversal distinct from observed root bytes", async () => {
|
|
const root = await createTestTempDirectory("rea-artifact-root-scope-");
|
|
const path = join(root, "fixture.pkg");
|
|
// The recognized XAR prefix selects PKG inventory; no child decoder is installed.
|
|
await writeFile(path, Buffer.from("xar!fixture"));
|
|
const inventory = await inventoryArtifact(path);
|
|
expect(inventory.manifest).toMatchObject({
|
|
root_format: "pkg",
|
|
node_count: 1,
|
|
occurrence_count: 1,
|
|
edge_count: 0,
|
|
});
|
|
expect(inventory.occurrences[0]?.hash_status).toBe("verified");
|
|
expect(inventory.coverage).toEqual({
|
|
scope: "container-entries",
|
|
status: "unknown",
|
|
});
|
|
});
|