Skip to content

Fix V1 release header preview - #21

Merged
senyo888 merged 1 commit into
mainfrom
bella/fix-v1-release-header
Jul 16, 2026
Merged

senyo888 merged 1 commit into
mainfrom
bella/fix-v1-release-header

Conversation

@senyo888

@senyo888 senyo888 commented Jul 16, 2026 •

Copy link
Copy Markdown
Owner

Scope

  • Change the v1.0.0 release header GIF and static fallback URLs from the not-yet-created v1.0.0 tag to the already-merged files on main.
  • Document why the release note deliberately uses main-hosted assets during draft review.

Reason

GitHub does not create the release tag until publication. The previous tag-based raw URLs therefore returned HTTP 404 in the draft preview and GitHub displayed only the image alt text.

Files affected

  • docs/releases/v1.0.0.md

Runtime impact

None. Pet packages, installer routes, metadata, sprite hashes, and application behavior are unchanged.

UI impact

The GitHub release draft now renders the animated laboratory header before the release tag exists. The reduced-motion PNG fallback uses the same live source strategy. README and public-site UI are unchanged.

Migration impact

None. No migration, restart, refresh, or dashboard action is required.

Validation performed

  • Both corrected raw URLs returned HTTP 200 with image/gif and image/png content types.
  • Remote GIF SHA-256 matched the tracked file: ace3035b3259efbc19a0f138eff3f23eeb191be572d04c1cf020f8613c0fc737.
  • Remote PNG SHA-256 matched the tracked file: 8866ccf0c891fd7245f3d7efac9dfe5025c9e8b54500308110184bcb2df9738f.
  • GitHub Markdown API preserved the <picture>, GIF URL, and reduced-motion source.
  • Playwright decoded both images at 1200 x 560; the laboratory composition was visually inspected; console errors/warnings: 0.
  • .codex/scripts/project_preflight.sh passed: 7 published pets, 61 site references, 81 Markdown references, privacy scan 84 candidates/0 matches.
  • git diff --check and staged diff checks passed.

Deterministic lane-ordering risk: none. UI truth consistency risk: none; only the hosting ref changed and exact hashes match.

Rollback safety

Revert this commit to restore tag-based URLs. The GitHub release remains a separate draft and can be edited independently without affecting runtime or package files.

Summary by CodeRabbit

  • Documentation
    • Updated release-page image references to use the main branch assets.
    • Ensured both reduced-motion and animated image variants display correctly.

@coderabbitai

coderabbitai Bot commented Jul 16, 2026 •

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 4cf8856a-e4e2-45de-b7a8-c4f092d6efa5

📥 Commits

Reviewing files that changed from the base of the PR and between 1d5ce77 and 3b83826.

📒 Files selected for processing (1)
  • docs/releases/v1.0.0.md

📝 Walkthrough

Walkthrough

The v1.0.0 release header’s reduced-motion and animated image URLs were changed from version-scoped assets to assets hosted on the main branch.

Changes

Release asset references

Layer / File(s) Summary
Update release header assets
docs/releases/v1.0.0.md
The <picture> header references main-branch URLs for both image variants.

Estimated code review effort: 1 (Trivial) | ~2 minutes

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description covers the change, but it does not follow the repository's required template sections. Reformat it to match the template and add Summary, Change type, Pet details, Package checklist, Validation performed, Impact, and Rights.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title is concise and accurately reflects the release header preview fix.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch bella/fix-v1-release-header

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@senyo888
senyo888 marked this pull request as ready for review July 16, 2026 05:28
@senyo888
senyo888 merged commit 65119b7 into main Jul 16, 2026
2 checks passed
@senyo888
senyo888 deleted the bella/fix-v1-release-header branch July 16, 2026 05:41
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