Fix YAML frontmatter, consolidate transformers, add harness reference#69
Merged
Fix YAML frontmatter, consolidate transformers, add harness reference#69
Conversation
Fixes #67: argument-hint values starting with [ were parsed as YAML flow sequences. Replace structured args arrays in source files with pre-formatted argument-hint strings, and quote values starting with [ or { in generateYamlFrontmatter(). Also consolidates 8 nearly-identical transformer files into a single config-driven createTransformer() factory. Adding a new provider now requires only a config object in providers.js instead of a full file. - Replace args source frontmatter with argument-hint strings - Add YAML quoting for values starting with [ or { - Add quote stripping to parseFrontmatter() for round-trip support - Create factory.js + providers.js, delete 8 individual transformers - Replace 16 explicit build.js calls with a loop over PROVIDERS - Consolidate 8 test files into 2 (factory + providers) Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
Research each harness's official documentation to verify and correct
provider frontmatter configs. Remove Codex/Gemini body transforms that
targeted their commands systems, not skills.
- Add compatibility + metadata to Cursor and Agents (Copilot)
- Add allowed-tools to Pi
- Remove Codex $ARGNAME and Gemini {{args}} body transforms
- Add HARNESSES.md as source of truth for harness capabilities
Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
Deploying impeccable with
|
| Latest commit: |
83b01ec
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://250e5109.impeccable-2rv.pages.dev |
| Branch Preview URL: | https://chore-fix-yaml-frontmatter.impeccable-2rv.pages.dev |
Resolve conflict from codex.js (deleted in our branch, modified on main for command_prefix). The command_prefix feature from main is preserved in factory.js via PROVIDER_PLACEHOLDERS lookup. Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
Skill cross-references like "Use the frontend-design skill" and
"run teach-impeccable" now use {{command_prefix}} so each provider
gets the correct invocation syntax (/ for most, $ for Codex).
Prose references to skill content ("guidelines in the frontend-design
skill") are left unchanged since they're not invocation instructions.
Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Summary
argument-hintvalues starting with[were parsed as YAML flow sequences. Replace structuredargsarrays with pre-formattedargument-hintstrings and quote values starting with[or{ingenerateYamlFrontmatter().createTransformer()factory. Adding a new provider now requires only a config object inproviders.js. Net -6,400 lines.Test plan
bun test-- 178 tests passbun run build-- completes successfully, all 8 providers output correctlydiff -r dist-before/ dist/-- verified output is identical after factory refactor.claude/,.codex/,.agents/for valid YAML frontmatter🤖 Generated with Claude Code