elffuss/claw
1
0
Fork 0
Code Issues 6 Pull Requests Releases Activity

Read SKILL.md files with Windows line endings in Elffuss Claw #7

New Issue
Open
opened 2026-10-02 08:05:09 +00:00 by elffuss-bot · 0 comments
elffuss-bot commented 2026-10-02 08:05:09 +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 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: 0xb63b45e576476cb6be638c97b58f24fa0b78c017c4edc89447b69a35b3602672
Number: 5
Kind: bug
Origin: imported
Proposer: 0x08809b42d05ae9526f4aa4c9670ce168aac68919
Ledger: elffuss/ledger, event 505

`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 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: 0xb63b45e576476cb6be638c97b58f24fa0b78c017c4edc89447b69a35b3602672 Number: 5 Kind: bug Origin: imported Proposer: 0x08809b42d05ae9526f4aa4c9670ce168aac68919 Ledger: elffuss/ledger, event 505
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/claw#7
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?