Skip to content

fix(wix-manage): quote gift-cards goal description to fix YAML parse error - #827

Open
itayhewix wants to merge 3 commits into
mainfrom
keen-hawk-ded562
Open

fix(wix-manage): quote gift-cards goal description to fix YAML parse error#827
itayhewix wants to merge 3 commits into
mainfrom
keen-hawk-ded562

Conversation

@itayhewix

@itayhewix itayhewix commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Summary

Several skill reference files have a description frontmatter field containing an unquoted Label: text (e.g. Recommend: eCommerce, [Goal: Drive Cross-Sells]). A colon+space inside a plain YAML scalar is parsed as a nested mapping key, which breaks frontmatter parsing with Error in user YAML: (<unknown>): mapping values are not allowed in this context.

Fixed by wrapping each affected description in double quotes, matching the existing convention used elsewhere (e.g. ecom-load-context.md).

Files fixed:

  • ecommerce/gift-cards/ecom-gift-cards-goal-sell-gift-cards.mdRecommend: eCommerce
  • ecommerce/recommend-ecommerce-strategy.md...already has: sales, promotions...
  • ecommerce/pricing-promotions/ecom-pricing-flow-bundle-and-save.md[Goal: Drive Cross-Sells]
  • ecommerce/pricing-promotions/ecom-pricing-flow-stock-mover.md[Goal: Clear Inventory]
  • ecommerce/pricing-promotions/ecom-pricing-flow-upsell-boost.md[Goal: Increase AOV]

Test plan

  • Verified all five files' frontmatter now parses cleanly with yaml.safe_load

🤖 Generated with Claude Code

@github-actions

github-actions Bot commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

❌ EvalForge YAML Gate: Scenario Locked by Another PR

These scenarios are draft-tagged for other PRs. Wait for those PRs to merge/close, or coordinate with their authors:

itayhewix and others added 3 commits August 5, 2026 09:43
…error

The description contained an unquoted "Recommend: eCommerce" — a colon+space
inside a plain YAML scalar is parsed as a nested mapping key, breaking
frontmatter parsing with "mapping values are not allowed in this context".

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
…e skill frontmatter

Same YAML gotcha as the gift-cards goal fix: an unquoted "Label: text" inside
a plain scalar description is parsed as a nested mapping key, breaking
frontmatter parsing with "mapping values are not allowed in this context".

Affects:
- recommend-ecommerce-strategy.md ("...already has: sales, promotions...")
- pricing-promotions/ecom-pricing-flow-bundle-and-save.md ("[Goal: Drive Cross-Sells]")
- pricing-promotions/ecom-pricing-flow-stock-mover.md ("[Goal: Clear Inventory]")
- pricing-promotions/ecom-pricing-flow-upsell-boost.md ("[Goal: Increase AOV]")

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
…ecommerce skill frontmatter"

This reverts commit a64b30c.
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