Compare commits
No commits in common. "642b6a0cf43499f3b81f988c100a85977f55f627" and "84bade9a9590af8ea57c51524deaab43bba08391" have entirely different histories.
642b6a0cf4
...
84bade9a95
|
|
@ -108,6 +108,3 @@ jobs:
|
||||||
|
|
||||||
- name: Validate file references
|
- name: Validate file references
|
||||||
run: npm run validate:refs
|
run: npm run validate:refs
|
||||||
|
|
||||||
- name: Validate skills
|
|
||||||
run: npm run validate:skills
|
|
||||||
|
|
|
||||||
|
|
@ -135,8 +135,6 @@ function parseFrontmatterMultiline(content) {
|
||||||
currentKey = line.slice(0, colonIndex).trim();
|
currentKey = line.slice(0, colonIndex).trim();
|
||||||
currentValue = line.slice(colonIndex + 1);
|
currentValue = line.slice(colonIndex + 1);
|
||||||
} else if (currentKey !== null) {
|
} else if (currentKey !== null) {
|
||||||
// Skip YAML comment lines
|
|
||||||
if (line.trimStart().startsWith('#')) continue;
|
|
||||||
// Continuation of multiline value
|
// Continuation of multiline value
|
||||||
currentValue += '\n' + line;
|
currentValue += '\n' + line;
|
||||||
}
|
}
|
||||||
|
|
@ -450,19 +448,19 @@ function validateSkill(skillDir) {
|
||||||
});
|
});
|
||||||
}
|
}
|
||||||
|
|
||||||
// Check content for any mention of installed_path (variable ref, prose, bare text)
|
// Check content for {installed_path}
|
||||||
const stripped = stripCodeBlocks(content);
|
const stripped = stripCodeBlocks(content);
|
||||||
const lines = stripped.split('\n');
|
const lines = stripped.split('\n');
|
||||||
for (const [i, line] of lines.entries()) {
|
for (const [i, line] of lines.entries()) {
|
||||||
if (/installed_path/i.test(line)) {
|
if (line.includes('{installed_path}')) {
|
||||||
findings.push({
|
findings.push({
|
||||||
rule: 'PATH-02',
|
rule: 'PATH-02',
|
||||||
title: 'No installed_path Variable',
|
title: 'No installed_path Variable',
|
||||||
severity: 'HIGH',
|
severity: 'HIGH',
|
||||||
file: relFile,
|
file: relFile,
|
||||||
line: i + 1,
|
line: i + 1,
|
||||||
detail: '`installed_path` reference found in content.',
|
detail: '`{installed_path}` reference found in content.',
|
||||||
fix: 'Remove all installed_path usage. Use relative paths (`./path` or `../path`) instead.',
|
fix: 'Replace `{installed_path}/path` with a relative path (`./path` or `../path`).',
|
||||||
});
|
});
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue