Skip to content

fix(unhead): preserve numeric zero head content - #887

Open
harlan-zw wants to merge 7 commits into
v2from
fix/v2-meta-numeric-zero
Open

fix(unhead): preserve numeric zero head content#887
harlan-zw wants to merge 7 commits into
v2from
fix/v2-meta-numeric-zero

Conversation

@harlan-zw

@harlan-zw harlan-zw commented Jul 23, 2026

Copy link
Copy Markdown
Collaborator

🔗 Linked issue

Backport of #885.

❓ Type of change

  • 📖 Documentation
  • 🐞 Bug fix
  • 👌 Enhancement
  • ✨ New feature
  • 🧹 Chore
  • ⚠️ Breaking change

📚 Description

Unhead v2 discarded numeric zero in safe input, title rendering, and inferred SEO metadata. A private hasContent predicate now owns presence checks, while empty, false, null, and non-finite values remain omitted.

@coderabbitai

coderabbitai Bot commented Jul 23, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 1f3050a8-fb38-4e9f-98c0-a6f51dac6d4d

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/v2-meta-numeric-zero

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.

@harlan-zw harlan-zw added v2 bug Something isn't working backport Backports an existing change to a maintained release line labels Jul 24, 2026
@harlan-zw harlan-zw changed the title fix(unhead): preserve numeric zero meta content fix(unhead): preserve numeric zero head content Jul 24, 2026
@harlan-zw

Copy link
Copy Markdown
Collaborator Author

🤖 Harlan Agent Kit automated review

This was generated and posted by a robot, not Harlan's personal review.

PASS · 95/100 confidence

  • Reviewed 009a84c6 against v2@9ad444e9
  • Base current; GitHub reports no conflicts
  • PR title and body match the project template; review threads are resolved
  • Adversarial review found no remaining material code issue
  • Complete diff reviewed; required CI passed 2/2; confidence includes limited local execution

Human merge decision still required.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport Backports an existing change to a maintained release line bug Something isn't working v2

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant