FN-8201: pin pi dependency versions
Pin the pi runtime packages to a single exact version so global npm installs resolve a compatible set. - Pin pi-ai and pi-coding-agent declarations across workspace manifests - Add a guard and tests that reject ranged or mismatched pi versions - Document the source-install fallback and add a patch changeset Files changed: .changeset/fn-8201-pin-pi-versions.md | 7 ++ docs/getting-started.md | 3 + package.json | 6 +- packages/cli/package.json | 4 +- packages/cli/src/__tests__/package-config.test.ts | 18 +++- packages/core/package.json | 2 +- packages/dashboard/package.json | 2 +- packages/engine/package.json | 4 +- packages/pi-claude-cli/package.json | 8 +- .../__tests__/check-pi-versions-pinned.test.mjs | 45 ++++++++ scripts/check-pi-versions-pinned.mjs | 120 +++++++++++++++++++++ 11 files changed, 205 insertions(+), 14 deletions(-) Fusion-Task-Id: FN-8201 Fusion-Task-Lineage: bf0ac363-df5f-4445-835b-cfd2d4909659 Co-authored-by: Fusion (runfusion.ai) <noreply@runfusion.ai>
This commit is contained in:
45
scripts/__tests__/check-pi-versions-pinned.test.mjs
Normal file
45
scripts/__tests__/check-pi-versions-pinned.test.mjs
Normal file
@@ -0,0 +1,45 @@
|
||||
import assert from "node:assert/strict";
|
||||
import { describe, it } from "node:test";
|
||||
import {
|
||||
scanTrackedManifests,
|
||||
validateManifestSet,
|
||||
} from "../check-pi-versions-pinned.mjs";
|
||||
|
||||
const pinnedManifest = {
|
||||
dependencies: {
|
||||
"@earendil-works/pi-ai": "0.80.10",
|
||||
"@earendil-works/pi-coding-agent": "0.80.10",
|
||||
},
|
||||
};
|
||||
|
||||
function validate(manifest) {
|
||||
return validateManifestSet([{ filePath: "packages/cli/package.json", manifest }]);
|
||||
}
|
||||
|
||||
describe("check-pi-versions-pinned", () => {
|
||||
it("passes against every guarded repository manifest", () => {
|
||||
assert.deepEqual(scanTrackedManifests(), []);
|
||||
});
|
||||
|
||||
it("rejects caret, tilde, wildcard, x, and comparator ranges", () => {
|
||||
for (const version of ["^0.80.10", "~0.80.10", "*", "0.80.x", ">=0.80.10"]) {
|
||||
const violations = validate({
|
||||
...pinnedManifest,
|
||||
dependencies: { ...pinnedManifest.dependencies, "@earendil-works/pi-ai": version },
|
||||
});
|
||||
assert.equal(violations.length > 0, true, `${version} must be rejected`);
|
||||
}
|
||||
});
|
||||
|
||||
it("rejects a matched-set version mismatch", () => {
|
||||
const violations = validate({
|
||||
...pinnedManifest,
|
||||
dependencies: { ...pinnedManifest.dependencies, "@earendil-works/pi-coding-agent": "0.80.11" },
|
||||
});
|
||||
assert.equal(violations.some((violation) => violation.includes("same exact version")), true);
|
||||
});
|
||||
|
||||
it("accepts a clean exact matched pair", () => {
|
||||
assert.deepEqual(validate(pinnedManifest), []);
|
||||
});
|
||||
});
|
||||
120
scripts/check-pi-versions-pinned.mjs
Normal file
120
scripts/check-pi-versions-pinned.mjs
Normal file
@@ -0,0 +1,120 @@
|
||||
#!/usr/bin/env node
|
||||
/*
|
||||
FNXC:DependencyPinning 2026-07-17-12:00:
|
||||
FN-8201 / Runfusion/Fusion#2270 requires the pi-ai and pi-coding-agent runtime
|
||||
set to remain version-locked. npm global installs ignore pnpm-lock.yaml and can
|
||||
independently resolve ranges to incompatible pi-mono patches, so every guarded
|
||||
manifest declaration must be an exact semver and all declarations must agree.
|
||||
*/
|
||||
import { readFileSync } from "node:fs";
|
||||
import { spawnSync } from "node:child_process";
|
||||
import { fileURLToPath } from "node:url";
|
||||
|
||||
export const PI_DEPENDENCIES = [
|
||||
"@earendil-works/pi-ai",
|
||||
"@earendil-works/pi-coding-agent",
|
||||
];
|
||||
|
||||
export const GUARDED_MANIFESTS = [
|
||||
"packages/cli/package.json",
|
||||
"packages/core/package.json",
|
||||
"packages/engine/package.json",
|
||||
"packages/dashboard/package.json",
|
||||
"packages/pi-claude-cli/package.json",
|
||||
];
|
||||
|
||||
const DEPENDENCY_BLOCKS = ["dependencies", "devDependencies", "peerDependencies", "optionalDependencies"];
|
||||
const EXACT_SEMVER = /^\d+\.\d+\.\d+(?:-[0-9A-Za-z-]+(?:\.[0-9A-Za-z-]+)*)?(?:\+[0-9A-Za-z-]+(?:\.[0-9A-Za-z-]+)*)?$/;
|
||||
|
||||
export function isExactSemver(version) {
|
||||
return typeof version === "string" && EXACT_SEMVER.test(version);
|
||||
}
|
||||
|
||||
function listTrackedManifests() {
|
||||
const result = spawnSync("git", ["ls-files", "--", ...GUARDED_MANIFESTS], {
|
||||
encoding: "utf8",
|
||||
stdio: ["ignore", "pipe", "pipe"],
|
||||
});
|
||||
if (result.status !== 0) throw new Error(result.stderr?.trim() || "git ls-files failed");
|
||||
return result.stdout.split("\n").map((line) => line.trim()).filter(Boolean);
|
||||
}
|
||||
|
||||
export function validateManifestSet(manifests) {
|
||||
const violations = [];
|
||||
const declaredVersions = new Map();
|
||||
|
||||
for (const { filePath, manifest } of manifests) {
|
||||
for (const blockName of DEPENDENCY_BLOCKS) {
|
||||
const dependencies = manifest?.[blockName];
|
||||
if (!dependencies || typeof dependencies !== "object") continue;
|
||||
|
||||
for (const packageName of PI_DEPENDENCIES) {
|
||||
if (!(packageName in dependencies)) continue;
|
||||
const version = dependencies[packageName];
|
||||
if (!isExactSemver(version)) {
|
||||
violations.push(`${filePath}: ${blockName}.${packageName} must be an exact semver, found ${JSON.stringify(version)}`);
|
||||
continue;
|
||||
}
|
||||
const declarations = declaredVersions.get(packageName) ?? [];
|
||||
declarations.push({ filePath, blockName, version });
|
||||
declaredVersions.set(packageName, declarations);
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
for (const packageName of PI_DEPENDENCIES) {
|
||||
const declarations = declaredVersions.get(packageName) ?? [];
|
||||
const versions = [...new Set(declarations.map(({ version }) => version))];
|
||||
if (versions.length > 1) {
|
||||
violations.push(`${packageName} must use one exact version across guarded manifests; found ${versions.join(", ")} (${declarations.map(({ filePath, blockName, version }) => `${filePath}:${blockName}=${version}`).join(", ")})`);
|
||||
}
|
||||
}
|
||||
|
||||
const allVersions = [...new Set(
|
||||
PI_DEPENDENCIES.flatMap((packageName) =>
|
||||
(declaredVersions.get(packageName) ?? []).map(({ version }) => version),
|
||||
),
|
||||
)];
|
||||
if (allVersions.length > 1) {
|
||||
violations.push(`${PI_DEPENDENCIES.join(" and ")} must resolve to the same exact version; found ${allVersions.join(", ")}`);
|
||||
}
|
||||
|
||||
return violations;
|
||||
}
|
||||
|
||||
export function scanTrackedManifests(files = listTrackedManifests(), options = {}) {
|
||||
const readFile = options.readFile ?? readFileSync;
|
||||
const manifests = [];
|
||||
for (const filePath of files) {
|
||||
let source;
|
||||
try {
|
||||
source = readFile(filePath, "utf8");
|
||||
} catch (error) {
|
||||
if (error && typeof error === "object" && "code" in error && error.code === "ENOENT") continue;
|
||||
throw error;
|
||||
}
|
||||
try {
|
||||
manifests.push({ filePath, manifest: JSON.parse(source) });
|
||||
} catch (error) {
|
||||
return [`${filePath}: invalid JSON (${error instanceof Error ? error.message : String(error)})`];
|
||||
}
|
||||
}
|
||||
return validateManifestSet(manifests);
|
||||
}
|
||||
|
||||
export function formatFailureMessage(violations) {
|
||||
return [
|
||||
"[check-pi-versions-pinned] pi runtime dependencies must be exact, matched versions.",
|
||||
"npm global installs do not use pnpm-lock.yaml; ranges can resolve an incompatible pi-mono patch set.",
|
||||
...violations.map((violation) => `- ${violation}`),
|
||||
].join("\n");
|
||||
}
|
||||
|
||||
export function main() {
|
||||
const violations = scanTrackedManifests();
|
||||
if (!violations.length) return 0;
|
||||
console.error(formatFailureMessage(violations));
|
||||
return 1;
|
||||
}
|
||||
|
||||
if (process.argv[1] && fileURLToPath(import.meta.url) === process.argv[1]) process.exitCode = main();
|
||||
Reference in New Issue
Block a user