Skip to content

Fix Pages build failure by making sitemap.xml Liquid filters Jekyll-3 compatible#2

Merged
Bowenislandsong merged 1 commit into
mainfrom
copilot/fix-jekyll-build-issues
May 22, 2026
Merged

Fix Pages build failure by making sitemap.xml Liquid filters Jekyll-3 compatible#2
Bowenislandsong merged 1 commit into
mainfrom
copilot/fix-jekyll-build-issues

Conversation

Copilot AI commented May 22, 2026

Copy link
Copy Markdown
Contributor

GitHub Pages build was failing in actions/jekyll-build-pages@v1 with a Liquid parse error in docs/sitemap.xml (Expected end_of_string but found id). The failure came from a compound where_exp condition that is not accepted by the GitHub Pages Jekyll/Liquid runtime.

  • Root cause

    • docs/sitemap.xml used a single where_exp with an and expression:
      • page.layout == 'default' and page.sitemap != false
    • Under the Pages runtime (Jekyll 3.10 / Liquid 4), this expression is rejected during sitemap rendering.
  • Change made

    • Split filtering into two supported steps:
      1. where for layout == "default"
      2. where_exp for page.sitemap != false
    • This preserves sitemap inclusion logic while using syntax accepted by Pages.
  • Snippet

    {% assign seo_pages = site.pages
      | where: "layout", "default"
      | where_exp: "page", "page.sitemap != false" %}

Agent-Logs-Url: https://github.com/Bowen-AI/AgenticLocal/sessions/df9b1da0-7df1-4f70-9c32-7601bfc9561a

Co-authored-by: Bowenislandsong <29466283+Bowenislandsong@users.noreply.github.com>
@Bowenislandsong Bowenislandsong marked this pull request as ready for review May 22, 2026 22:31
@Bowenislandsong Bowenislandsong merged commit 7c1e2d6 into main May 22, 2026
4 checks passed
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.

2 participants