Skip to content

fix: make blog manifests timezone independent and validate PR builds - #73

Merged
linfangw merged 1 commit into
mainfrom
linfangw/fix-blog-manifest-timezone
Oct 22, 2026
Merged

linfangw merged 1 commit into
mainfrom
linfangw/fix-blog-manifest-timezone

Conversation

@linfangw

@linfangw linfangw commented Oct 22, 2026 •

Copy link
Copy Markdown
Member

Pages publishing failed because timezone-free dates such as Aug 9 2026 were parsed in the host timezone before conversion to UTC. The committed manifests matched Asia/Shanghai, while the UTC deployment runner generated different dates and rejected them as stale.

Parse English calendar dates in UTC and regenerate both locale manifests. This corrects 88 English and 118 Chinese publication dates without changing article membership or other metadata. Add PR/main checks for cross-timezone regression tests, manifest freshness, and a full site build using the same Node 24 / pnpm 10.3.0 versions as Pages.

Validation:

  • Regression tests pass in UTC, Asia/Shanghai, and America/Los_Angeles; fixtures cover YAML/quoted calendar dates, draft exclusion, and rejection after metadata changes.
  • Full local production build passes (220 pages); workflow actionlint and diff checks pass.
  • PR checks passed, and the current commit received a clean automated review and App approval.
  • Blog manifest and site build is now required on main, bound to GitHub Actions, with the branch required to be up to date. Existing rules are preserved.
  • Main checks passed.
  • Pages build and deployment passed; deployment 7248523207 successfully published merged commit 8e7bbab490f29a28104cf272c0b5cf7d3bab3ef6 on 2026-10-22.
  • Live HTTP smoke checks passed for /, /blog/en/, /blog/cn/, /blog/en/agent-financial-tools-retrieval/, /rss.xml, and /sitemap-index.xml at https://qverisai.github.io/. All return HTTP 200; the root redirect, locale HTML, article content, and XML parsing were verified.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 22, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-22T09:20:48.140829Z 99e90b3 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@qverisai-review-bot qverisai-review-bot Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated review completed without findings for 99e90b3. No unresolved automated review findings or change requests remain.

@linfangw
linfangw merged commit 8e7bbab into main Oct 22, 2026
2 checks passed
@linfangw
linfangw deleted the linfangw/fix-blog-manifest-timezone branch October 22, 2026 09:27
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