Centralize frontmatter parsing + parse frontmatter with yaml library #728
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
richardgill
commented
Jan 14, 2026
| import * as fs from "node:fs"; | ||
| import * as os from "node:os"; | ||
| import * as path from "node:path"; | ||
| import { parseFrontmatter } from "@mariozechner/pi-coding-agent"; |
Contributor
Author
There was a problem hiding this comment.
I think this is ~ok, but I think since this is an example a handrolled function is fine if you want to revert this one.
Owner
There was a problem hiding this comment.
No, that's fine. Whatever built-in stuff we can offer to extensions, we should, so they can stay "dependency-less" for as long as possible.
…matter-utils-lib # Conflicts: # packages/coding-agent/src/modes/interactive/interactive-mode.ts
Resolve conflicts keeping yaml-based frontmatter parser with generic type support.
Resolve version conflict by taking ^0.46.0 versions while keeping the new photon-node dependency for image processing.
Owner
|
Cheers! |
badlogic
added a commit
that referenced
this pull request
Jan 16, 2026
- Add OpenAI Codex official support entry - Add pi-internal:// URL scheme - Add ctx.shutdown() fix (#756) - Add OpenAI Codex retry logic (#733) - Add third-party contributions: - #702 by @aliou (custom tool HTML export) - #728 by @richardgill (frontmatter parsing, SelectList fix) - #747 by @kaofelix (tree filter shortcuts) - #752 by @richardgill (keymap expansion)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Problem: When porting my claude code skills to pi I had some multi-line descriptions. Pi only parsed the first line which is
"|".There are several similar frontmatter functions in the codebase (each was slightly different)
I've consolidated into a single function and used the yaml library to parse the yaml (see below for library choice background).
Finally, the Select component didn't handle multi line descriptions well, so I updated the component to replace new lines with spaces.
I've made a few decisions in here that might not be your preference - if so, just let me know and I can fix them for you 👍
Appendix:
Example with multi line value:
SKILL.md:Yaml library picks (a little 'deep researched', but looks good to me)
yamlis 0 deps and reasonably recently published.yamljs-yamlv4confboxultramatterfront-mattergray-matteryaml-front-matter