elffuss/code
1
0
Fork 0
Code Issues 11 Pull Requests Releases Activity

Solution to #11: Read SKILL.md files with Windows line endings in Elffuss Code #22

Closed
elffuss-bot wants to merge 1 commits from item/0x6689dd6d2c645e4a9251d1f91e6cd8e6f54caaad3c03cbec98cad659458fe950 into main
pull from: item/0x6689dd6d2c645e4a9251d1f91e6cd8e6f54caaad3c03cbec98cad659458fe950
merge into: elffuss:main
Conversation 1 Commits 1 Files Changed 3 +24
3 changed files with 24 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
tests/acceptance/11/CRITERIA.json Normal file
View File

@@ -0,0 +1,12 @@
{
"schema": "elffuss-t2t/criteria@2",
"issue": 11,
"criteria": [
{
"index": 1,
"tests": [
"tests/acceptance/11/criteria.test.js::criterion 1"
]
}
]
}

11
tests/acceptance/11/criteria.test.js Normal file
View File

@@ -0,0 +1,11 @@
import test from 'node:test';
import assert from 'node:assert/strict';
import * as mod from '../../../web/js/skills.js';
test('criterion 1', () => {
const withCrlf = mod.parseSkill('---\r\nname: demo\r\ndescription: x\r\n---\r\nBody');
assert.deepEqual(withCrlf, { name: 'demo', description: 'x', content: 'Body' });
const fallback = mod.parseSkill('Body only', 'fallback');
assert.deepEqual(fallback, { name: 'fallback', description: '', content: 'Body only' });
});

1
web/js/skills.js
View File

@@ -131,6 +131,7 @@ export async function installFromRepo(entry) {
// SKILL.md → { name, description, content } // SKILL.md → { name, description, content }
export function parseSkill(md, fallbackName = 'skill') { export function parseSkill(md, fallbackName = 'skill') {
md = md.replace(/\r\n/g, '\n');
const fm = md.match(/^---\n([\s\S]*?)\n---/); const fm = md.match(/^---\n([\s\S]*?)\n---/);
const meta = fm?.[1] || ''; const meta = fm?.[1] || '';
const name = meta.match(/^name:\s*(.+)$/m)?.[1]?.trim() || fallbackName; const name = meta.match(/^name:\s*(.+)$/m)?.[1]?.trim() || fallbackName;