fix: Cannot read properties of null (reading 'Symbol(astro.headAndContent)')

[icaliman]

Changes

• Fixed the error that we get in our SSR project. This error occurs after 2-3 seconds of page load.

Testing

Can not provide a minimal reproduction example.

Docs

Docs don't need change.

[changeset-bot]

🦋 Changeset detected

Latest commit: f9ac66d

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[bluwy]

I think it would be best if you can figure out how this happens in the first place. A repro or test would help us prevent regressing this in the future.

[icaliman]

@bluwy I found the issue and added a test.

[matthewp]

great test!

[patheticGeek]

Hey! I have this same issue in prod, could we get this merged faster?

[matthewp]

@bluwy can you take a look at this now that you've changed how the test work?

[bluwy]

LGTM. We had an internal discussion whether this behaviour is desirable as we've not documented returning null as a valid pattern in Astro components, but after further testing, it looks like any values return are rendered as-is. As if you're doing <div>{<some-value>}</div> in an Astro file.

Given that I think handling null here make sense for now and we can revisit this behaviour again in the future. I'll rebase this PR before merging.