WEBDIENSTE/ECC
1
0
Fork 0
mirror of https://github.com/affaan-m/ECC.git synced 2026-10-19 06:23:46 +02:00
Code Issues Packages Projects Releases Wiki Activity

fix: ignore directories when listing skill snapshots

Browse Source
This commit is contained in:
Eneko Ruiz
2026-10-11 11:33:41 +02:00
parent 2d515e4fc2
commit acb75bc1ed
2 changed files with 29 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
scripts/lib/skill-evolution/versioning.js
+3
View File
@@ -101,6 +101,9 @@ function listVersions(skillPath) {
const filePath = path.join(versionsDir, fileName);
const stats = fs.statSync(filePath);
if (!stats.isFile()) {
return null;
}
return {
version,
tests/lib/skill-evolution.test.js
+26
View File
@@ -202,6 +202,32 @@ function runTests() {
assert.strictEqual(amendments[1].version, 2);
})) passed++; else failed++;
if (test('ignores directories named like version snapshots', () => {
const fixtureRoot = createTempDir('skill-version-directory-');
try {
const skillDir = createSkill(fixtureRoot, 'demo', '# Demo\n');
const snapshotDirectory = path.join(versioning.getVersionsDir(skillDir), 'v999.md');
fs.mkdirSync(snapshotDirectory, { recursive: true });
fs.writeFileSync(path.join(versioning.getVersionsDir(skillDir), 'notes.txt'), 'Not a snapshot');
assert.deepStrictEqual(versioning.listVersions(skillDir), []);
assert.strictEqual(versioning.getCurrentVersion(skillDir), 1);
const snapshot = versioning.createVersion(skillDir);
assert.strictEqual(snapshot.version, 1);
assert.deepStrictEqual(versioning.listVersions(skillDir).map(entry => entry.version), [1]);
assert.deepStrictEqual(
versioning.listVersions(path.join(skillDir, 'SKILL.md')).map(entry => entry.version),
[1]
);
assert.strictEqual(versioning.getCurrentVersion(skillDir), 1);
assert.ok(fs.statSync(snapshotDirectory).isDirectory());
assert.strictEqual(versioning.getCurrentVersion(path.join(fixtureRoot, 'missing-skill')), 0);
assert.throws(() => versioning.createVersion(snapshotDirectory), /Skill file not found/);
} finally {
cleanupTempDir(fixtureRoot);
}
})) passed++; else failed++;
if (test('rolls back to a previous snapshot without losing history', () => {
const skillDir = path.join(skillsRoot, 'alpha');