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

Read SKILL.md files with Windows line endings in Elffuss Code #14

New Issue
Open
opened 2026-10-02 08:05:18 +00:00 by elffuss-bot · 0 comments
elffuss-bot commented 2026-10-02 08:05:18 +00:00
Owner
Copy Link

parseSkill in web/js/skills.js looks for front matter with ^---\n. A SKILL.md saved with \r\n line endings has none: parseSkill("---\r\nname: demo\r\ndescription: x\r\n---\r\nBody") returns the fallback name skill, an empty description and the whole file, front matter included, as content. Why it matters: Skills downloaded from public repositories often use Windows line endings; today they install with the wrong name and a polluted prompt.

Proposal

Convert \r\n to \n at the start of parseSkill and change nothing else.

Acceptance criteria

  1. parseSkill("---\r\nname: demo\r\ndescription: x\r\n---\r\nBody") returns { name: "demo", description: "x", content: "Body" }, and still returns { name: "fallback", description: "", content: "Body only" } for parseSkill("Body only", "fallback").

Files

  • web/js/skills.js

T2T-Issue: 0x33958ef2cc4d93a93649833be3a29f2b76bbd4080db691cff2473389671a252e
Number: 11
Kind: bug
Origin: imported
Proposer: 0x08809b42d05ae9526f4aa4c9670ce168aac68919
Ledger: elffuss/ledger, event 516

`parseSkill` in web/js/skills.js looks for front matter with `^---\n`. A SKILL.md saved with `\r\n` line endings has none: `parseSkill("---\r\nname: demo\r\ndescription: x\r\n---\r\nBody")` returns the fallback name `skill`, an empty description and the whole file, front matter included, as content. Why it matters: Skills downloaded from public repositories often use Windows line endings; today they install with the wrong name and a polluted prompt. ### Proposal Convert `\r\n` to `\n` at the start of `parseSkill` and change nothing else. ### Acceptance criteria 1. `parseSkill("---\r\nname: demo\r\ndescription: x\r\n---\r\nBody")` returns `{ name: "demo", description: "x", content: "Body" }`, and still returns `{ name: "fallback", description: "", content: "Body only" }` for `parseSkill("Body only", "fallback")`. ### Files - `web/js/skills.js` --- T2T-Issue: 0x33958ef2cc4d93a93649833be3a29f2b76bbd4080db691cff2473389671a252e Number: 11 Kind: bug Origin: imported Proposer: 0x08809b42d05ae9526f4aa4c9670ce168aac68919 Ledger: elffuss/ledger, event 516
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
No items
No Label
Milestone
No items
No Milestone
Assignees
Clear assignees
elffuss-bot
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: elffuss/code#14
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?