mirror of
https://github.com/affaan-m/ECC.git
synced 2026-08-17 21:15:40 +02:00
* feat(install): add guided Claude plugin setup * fix: support Claude command shims on Windows * feat: support safe Claude plugin scope migration * fix(install): preserve interactive setup terminal * fix(install): auto-migrate setup scope changes * feat(install): add guided multi-harness installer * fix(install): sync Yarn binary metadata * fix(install): handle wizard EOF on Node 18 * ci: allow installer matrix tests to finish * test(install): allow slower PowerShell delegation * fix(install): harden guided provider reconciliation * test(install): harden packaged and local compatibility * chore: prepare guided installer release 2.2.0 * fix(install): report refreshed Codex marketplace state * fix(install): verify managed content provenance * test(install): allow empty Yarn smoke fixture * test(install): invoke Windows package shims safely * fix(install): close cross-platform release gaps * fix(install): require trusted GitHub origins * fix(install): preserve hook profile precedence * refactor(install): centralize trusted GitHub origins * ci: retrigger workflow run after merge of main Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> --------- Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
186 lines
6.4 KiB
JavaScript
186 lines
6.4 KiB
JavaScript
'use strict';
|
|
/**
|
|
* Tests for scripts/lib/path-safety.js — the install-state containment guard
|
|
* that fixes arbitrary file write/delete via attacker-controlled install-state
|
|
* (GHSA-hfpv-w6mp-5g95).
|
|
*/
|
|
|
|
const assert = require('assert');
|
|
const fs = require('fs');
|
|
const os = require('os');
|
|
const path = require('path');
|
|
|
|
const {
|
|
assertWithinTrustedRoot,
|
|
isWithinRoot,
|
|
realpathNearestExisting
|
|
} = require('../../scripts/lib/path-safety');
|
|
|
|
let passed = 0;
|
|
let failed = 0;
|
|
|
|
function test(name, fn) {
|
|
try {
|
|
fn();
|
|
console.log(` PASS ${name}`);
|
|
passed += 1;
|
|
} catch (error) {
|
|
console.log(` FAIL ${name}`);
|
|
console.log(` ${error.message}`);
|
|
failed += 1;
|
|
}
|
|
}
|
|
|
|
const root = fs.mkdtempSync(path.join(os.tmpdir(), 'path-safety-root-'));
|
|
const outside = fs.mkdtempSync(path.join(os.tmpdir(), 'path-safety-out-'));
|
|
|
|
try {
|
|
test('allows a path inside the trusted root', () => {
|
|
const p = path.join(root, '.cursor', 'rules', 'x.md');
|
|
// Returns the canonicalized path (symlinks like /var -> /private/var resolved).
|
|
assert.doesNotThrow(() => assertWithinTrustedRoot(p, root, 'repair'));
|
|
assert.ok(assertWithinTrustedRoot(p, root, 'repair').endsWith(path.join('.cursor', 'rules', 'x.md')));
|
|
assert.strictEqual(isWithinRoot(p, root), true);
|
|
});
|
|
|
|
test('allows the root itself', () => {
|
|
assert.strictEqual(isWithinRoot(root, root), true);
|
|
});
|
|
|
|
test('allows a non-existent destination beneath a non-existent trusted root', () => {
|
|
const futureRoot = path.join(root, 'future-root');
|
|
const futureDestination = path.join(futureRoot, 'session-data', 'session.json');
|
|
assert.strictEqual(isWithinRoot(futureDestination, futureRoot), true);
|
|
assert.strictEqual(
|
|
assertWithinTrustedRoot(futureDestination, futureRoot, 'write'),
|
|
realpathNearestExisting(futureDestination)
|
|
);
|
|
});
|
|
|
|
test('canonicalizes the nearest existing ancestor for a non-existent trusted root', () => {
|
|
const realParent = fs.mkdtempSync(path.join(os.tmpdir(), 'path-safety-real-'));
|
|
const linkedParent = path.join(
|
|
os.tmpdir(),
|
|
`path-safety-link-${process.pid}-${Date.now()}`
|
|
);
|
|
|
|
try {
|
|
fs.symlinkSync(realParent, linkedParent, 'dir');
|
|
} catch {
|
|
console.log(' (symlink unsupported on this platform; skipping)');
|
|
fs.rmSync(realParent, { recursive: true, force: true });
|
|
return;
|
|
}
|
|
|
|
try {
|
|
const futureRoot = path.join(linkedParent, '.cursor', 'ecc');
|
|
const futureDestination = path.join(futureRoot, 'session-data', 'session.json');
|
|
assert.strictEqual(isWithinRoot(futureDestination, futureRoot), true);
|
|
assert.strictEqual(
|
|
assertWithinTrustedRoot(futureDestination, futureRoot, 'write'),
|
|
path.join(
|
|
fs.realpathSync(realParent),
|
|
'.cursor',
|
|
'ecc',
|
|
'session-data',
|
|
'session.json'
|
|
)
|
|
);
|
|
} finally {
|
|
// Unlink the directory symlink itself. Node 24 rejects rmSync() here
|
|
// with EISDIR even though older supported runtimes accepted it.
|
|
fs.unlinkSync(linkedParent);
|
|
fs.rmSync(realParent, { recursive: true, force: true });
|
|
}
|
|
});
|
|
|
|
test('returns the same canonical destination that was checked for containment', () => {
|
|
const destination = path.join(root, 'single-canonicalization.txt');
|
|
const originalRealpathSync = fs.realpathSync;
|
|
let destinationCanonicalizations = 0;
|
|
fs.writeFileSync(destination, 'safe\n');
|
|
|
|
fs.realpathSync = function countedRealpathSync(candidatePath, options) {
|
|
if (path.resolve(candidatePath) === path.resolve(destination)) {
|
|
destinationCanonicalizations += 1;
|
|
}
|
|
return originalRealpathSync.call(fs, candidatePath, options);
|
|
};
|
|
|
|
try {
|
|
assert.strictEqual(
|
|
assertWithinTrustedRoot(destination, root, 'write'),
|
|
originalRealpathSync(destination)
|
|
);
|
|
} finally {
|
|
fs.realpathSync = originalRealpathSync;
|
|
}
|
|
|
|
assert.strictEqual(destinationCanonicalizations, 1);
|
|
});
|
|
|
|
test('refuses an absolute path outside the root', () => {
|
|
const evil = path.join(outside, 'PWNED.txt');
|
|
assert.throws(() => assertWithinTrustedRoot(evil, root, 'repair'), /outside the install root/);
|
|
assert.strictEqual(isWithinRoot(evil, root), false);
|
|
});
|
|
|
|
test('refuses an escape from a non-existent trusted root', () => {
|
|
const futureRoot = path.join(root, 'future-root');
|
|
const evil = path.join(futureRoot, '..', 'escape.txt');
|
|
assert.throws(() => assertWithinTrustedRoot(evil, futureRoot, 'write'), /outside the install root/);
|
|
assert.strictEqual(isWithinRoot(evil, futureRoot), false);
|
|
});
|
|
|
|
test('refuses a ../ traversal escape', () => {
|
|
const evil = path.join(root, '..', 'escape.txt');
|
|
assert.throws(() => assertWithinTrustedRoot(evil, root, 'uninstall'), /outside the install root/);
|
|
});
|
|
|
|
test('refuses a symlinked intermediate directory that escapes the root', () => {
|
|
const linkDir = path.join(root, 'link');
|
|
try {
|
|
fs.symlinkSync(outside, linkDir, 'dir');
|
|
} catch {
|
|
console.log(' (symlink unsupported on this platform; skipping)');
|
|
return;
|
|
}
|
|
// root/link -> outside, so root/link/PWNED resolves outside the root.
|
|
const evil = path.join(linkDir, 'PWNED.txt');
|
|
assert.throws(() => assertWithinTrustedRoot(evil, root, 'repair'), /outside the install root/);
|
|
});
|
|
|
|
test('refuses a dangling symlinked intermediate directory', () => {
|
|
const danglingTarget = path.join(outside, 'missing-target');
|
|
const linkDir = path.join(root, 'dangling-link');
|
|
try {
|
|
fs.symlinkSync(danglingTarget, linkDir, 'dir');
|
|
} catch {
|
|
console.log(' (symlink unsupported on this platform; skipping)');
|
|
return;
|
|
}
|
|
const evil = path.join(linkDir, 'session-data', 'session.json');
|
|
assert.strictEqual(isWithinRoot(evil, root), false);
|
|
assert.throws(
|
|
() => assertWithinTrustedRoot(evil, root, 'write'),
|
|
/outside the install root/
|
|
);
|
|
});
|
|
|
|
test('refuses when no trusted root is resolved', () => {
|
|
assert.throws(() => assertWithinTrustedRoot(path.join(root, 'x'), null, 'repair'), /no trusted install root/);
|
|
});
|
|
|
|
test('refuses a missing destination path', () => {
|
|
assert.throws(() => assertWithinTrustedRoot('', root, 'repair'), /missing destination path/);
|
|
});
|
|
} finally {
|
|
fs.rmSync(root, { recursive: true, force: true });
|
|
fs.rmSync(outside, { recursive: true, force: true });
|
|
}
|
|
|
|
console.log(`\nResults: Passed: ${passed}, Failed: ${failed}`);
|
|
if (failed > 0) {
|
|
process.exit(1);
|
|
}
|