Skip to content

fix(theme): accept CRLF frontmatter delimiters - #95

Open
HelloWorld3326 wants to merge 1 commit into
Nutlope:mainfrom
HelloWorld3326:fix/theme-crlf-header
Open

HelloWorld3326 wants to merge 1 commit into
Nutlope:mainfrom
HelloWorld3326:fix/theme-crlf-header

Conversation

@HelloWorld3326

Copy link
Copy Markdown

A theme saved with CRLF line endings was rejected as having no frontmatter, because the opening delimiter required an immediate LF. Accept both LF and CRLF delimiters.

Validation: node --test skills/hallmark/scripts/theme.test.mjs fails on CRLF before the fix and passes afterward. The CLI test uses the existing Hum theme, with both variants below the separate 8,000-byte limit, and checks successful validation for both. Source files are not rewritten.

@vercel

vercel Bot commented Oct 27, 2026

Copy link
Copy Markdown

@HelloWorld3326 is attempting to deploy a commit to the Together AI Team on Vercel.

A member of the Team first needs to authorize it.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant