Skip to content

fix(presets): repair frontmatter that no YAML parser accepts - #6880

Open
RemyLoveLogicAI wants to merge 1 commit into
heygen-com:mainfrom
RemyLoveLogicAI:fix/frame-preset-yaml-frontmatter
Open

RemyLoveLogicAI wants to merge 1 commit into
heygen-com:mainfrom
RemyLoveLogicAI:fix/frame-preset-yaml-frontmatter

Conversation

@RemyLoveLogicAI

Copy link
Copy Markdown

What

13 of the frame presets under skills/hyperframes/assets/frame-presets/ ship YAML
frontmatter that no parser accepts. When the block fails to parse, the document is
rejected whole - a harness loading these presets reads nothing, silently. The preset
is present in the package and dead at runtime.

The defect is one character, 84 times: a flow mapping written key:{ ... } with no space
after the colon. YAML requires that space.

Verification

Every file checked before and after with Ruby YAML.safe_load (psych), not with a
hand-rolled parser:

  • before the patch: rejected, all 13
  • after the patch: parses to a mapping, all 13
  • everything outside the frontmatter block: byte-identical to upstream (asserted per file)

84 lines changed, each adding exactly one space. No wording is touched, so the diff can be
read as a whitespace-only change.

files
  • biennale-yellow
  • blockframe
  • blue-professional
  • bold-poster
  • broadside
  • capsule
  • cartesian
  • cobalt-grid
  • code-editorial
  • coral
  • creative-mode
  • daisy-days
  • editorial-forest

How this was found

A library-wide audit cross-checked every skill's frontmatter against an independent YAML
parser. The audit's own parser had rated these files healthy, because it only looked at
which keys were declared and never at whether the block parsed.

13 frame presets write `key:{ ... }` without the space YAML requires after a flow-map
colon. The block then fails to parse as a whole, so a harness reads nothing from the
preset - it is listed in the package and silently dead at runtime.

Each of the 84 changed lines adds one space; no prose is touched. Verified before and
after with Ruby YAML.safe_load: all 13 blocks were rejected before and parse to a
mapping after, and everything outside the frontmatter is byte-identical to upstream.

🤖 Generated with Codebuff
Co-Authored-By: Codebuff <noreply@codebuff.com>

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