Skip to content

Conversation

@luandro
Copy link
Contributor

@luandro luandro commented Nov 12, 2025

Investigated why "Build documentation" succeeds on PRs but fails on main.

Root cause:

Solution:

  • Created automated fix script (scripts/fix-yaml-frontmatter.ts)
  • Identified 10 files needing fixes in content branch
  • All have unquoted colons (:) and ampersands (&) in titles

Files added:

  • INVESTIGATION-REPORT.md: Detailed analysis and recommendations
  • SOLUTION.md: Step-by-step fix instructions
  • scripts/fix-yaml-frontmatter.ts: Automated YAML frontmatter validator/fixer

Next steps:

  1. Apply fix script to content branch
  2. Commit and push changes to content branch
  3. Verify main branch build succeeds

Related: PR #75, main branch build failures

Investigated why "Build documentation" succeeds on PRs but fails on main.

Root cause:
- PR #75 fixed the Notion fetch scripts to quote YAML special characters
- BUT the content branch still contains old files with unquoted frontmatter
- PRs regenerate content (using fixed scripts) → success ✅
- Main uses existing content branch → failure ❌

Solution:
- Created automated fix script (scripts/fix-yaml-frontmatter.ts)
- Identified 10 files needing fixes in content branch
- All have unquoted colons (:) and ampersands (&) in titles

Files added:
- INVESTIGATION-REPORT.md: Detailed analysis and recommendations
- SOLUTION.md: Step-by-step fix instructions
- scripts/fix-yaml-frontmatter.ts: Automated YAML frontmatter validator/fixer

Next steps:
1. Apply fix script to content branch
2. Commit and push changes to content branch
3. Verify main branch build succeeds

Related: PR #75, main branch build failures
@github-actions
Copy link
Contributor

🚀 Preview Deployment

Your documentation preview is ready!

Preview URL: https://pr-76.comapeo-docs.pages.dev

🔄 Content: Regenerated 5 pages from Notion (content branch was empty, used fallback)

⚠️ Note: Content branch needed regeneration. Consider running the sync-docs workflow.

This preview will update automatically when you push new commits to this PR.


Built with commit c105058

@luandro luandro added the invalid This doesn't seem right label Nov 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

invalid This doesn't seem right

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants