070959e133
bake-plugin-previews / Bake plugin previews (push) Waiting to run
landing-page-staging / Deploy landing page to staging (push) Has been skipped
landing-page-ci / Validate landing page (push) Failing after 4s
visual-baseline / Capture visual baselines (push) Has been cancelled
71 lines
2.7 KiB
JavaScript
71 lines
2.7 KiB
JavaScript
#!/usr/bin/env node
|
|
// Decide whether two plugin-preview manifests differ in a way that matters —
|
|
// i.e. whether their `previews` subtree changed.
|
|
//
|
|
// The manifest also carries a `generatedAt` timestamp that moves on EVERY bake
|
|
// run. Comparing the whole file (as the workflow's old `git diff --quiet` did)
|
|
// therefore always reports a change and opens a noise review PR even when no
|
|
// clip actually changed — this is what produced timestamp-only PRs like #4261
|
|
// and the stacked backlog. The bake workflows use this helper to open/update a
|
|
// review PR only when a real preview entry changed.
|
|
//
|
|
// CLI:
|
|
// node scripts/plugin-previews-diff.mjs <oldManifest.json> <newManifest.json>
|
|
// → prints "changed" or "unchanged" to stdout; exit 0 on success, 2 on error.
|
|
// A missing/unreadable OLD manifest is treated as "no previews yet", so any
|
|
// entry in NEW counts as a change.
|
|
|
|
import { readFileSync, realpathSync } from 'node:fs';
|
|
import { fileURLToPath } from 'node:url';
|
|
|
|
// Stable, key-order-independent serialization of a JSON value, so reordering the
|
|
// keys inside a preview entry is not mistaken for a content change.
|
|
function canonical(value) {
|
|
if (Array.isArray(value)) return `[${value.map(canonical).join(',')}]`;
|
|
if (value && typeof value === 'object') {
|
|
return `{${Object.keys(value)
|
|
.sort()
|
|
.map((k) => `${JSON.stringify(k)}:${canonical(value[k])}`)
|
|
.join(',')}}`;
|
|
}
|
|
return JSON.stringify(value) ?? 'null';
|
|
}
|
|
|
|
// True when the `previews` subtree differs between the two manifests, ignoring
|
|
// `generatedAt` and any other top-level metadata.
|
|
export function previewsChanged(oldManifest, newManifest) {
|
|
const oldPreviews = (oldManifest && oldManifest.previews) || {};
|
|
const newPreviews = (newManifest && newManifest.previews) || {};
|
|
return canonical(oldPreviews) !== canonical(newPreviews);
|
|
}
|
|
|
|
function readJsonOrEmpty(path) {
|
|
try {
|
|
return JSON.parse(readFileSync(path, 'utf8'));
|
|
} catch {
|
|
return {};
|
|
}
|
|
}
|
|
|
|
// CLI entry — only when this file is the process entrypoint, not when a test
|
|
// imports `previewsChanged`.
|
|
const invokedDirectly =
|
|
process.argv[1] && realpathSync(process.argv[1]) === fileURLToPath(import.meta.url);
|
|
|
|
if (invokedDirectly) {
|
|
const [oldPath, newPath] = process.argv.slice(2);
|
|
if (!oldPath || !newPath) {
|
|
console.error('usage: plugin-previews-diff.mjs <oldManifest.json> <newManifest.json>');
|
|
process.exit(2);
|
|
}
|
|
let newManifest;
|
|
try {
|
|
newManifest = JSON.parse(readFileSync(newPath, 'utf8'));
|
|
} catch (e) {
|
|
console.error(`failed to read new manifest ${newPath}: ${e.message}`);
|
|
process.exit(2);
|
|
}
|
|
const oldManifest = readJsonOrEmpty(oldPath);
|
|
process.stdout.write(previewsChanged(oldManifest, newManifest) ? 'changed\n' : 'unchanged\n');
|
|
}
|