Skip to content

Conversation

@break-stuff
Copy link

Description

This crashes in SSR environments (the popover polyfill works fine).

Related Issue(s)

Reminder to add related issue(s), if available.

Steps to test/reproduce

Importing directly from @oddbird/css-anchor-positioning crashes in SSR and JSDOM environments.

We were able to get it working by importing the function and calling it, but it would be nice to have it behave consistently like the popover polyfill.

import polyfill from '@oddbird/css-anchor-positioning/fn';

if (typeof window !== 'undefined') {
  polyfill()
}

Show me

Provide screenshots/animated gifs/videos if necessary.

@netlify
Copy link

netlify bot commented Feb 4, 2026

Deploy Preview for anchor-position-wpt canceled.

Name Link
🔨 Latest commit ef349db
🔍 Latest deploy log https://app.netlify.com/projects/anchor-position-wpt/deploys/698b3b7be3cab40007c085b1

@netlify
Copy link

netlify bot commented Feb 4, 2026

Deploy Preview for anchor-polyfill ready!

Name Link
🔨 Latest commit ef349db
🔍 Latest deploy log https://app.netlify.com/projects/anchor-polyfill/deploys/698b3b77b98ca400083c7b55
😎 Deploy Preview https://deploy-preview-377--anchor-polyfill.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Contributor

@jamesnw jamesnw left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution!

@jamesnw
Copy link
Contributor

jamesnw commented Feb 4, 2026

The lint and test checks are stuck for some reason. It seems like the recommendation is to close the PR and re-open, so I'm going to try that. Apologies for the notification noise.

@jamesnw jamesnw closed this Feb 4, 2026
@jamesnw jamesnw reopened this Feb 4, 2026
@break-stuff
Copy link
Author

It looks like the problem is NODE_OPTIONS='--no-experimental-strip-types'. It's causing the CI tests to silently fail.

@break-stuff break-stuff closed this Feb 5, 2026
@break-stuff break-stuff reopened this Feb 5, 2026
@jamesnw
Copy link
Contributor

jamesnw commented Feb 5, 2026

@break-stuff It looks like it's still stalled out. I've opened #378 to track that issue.

@break-stuff
Copy link
Author

break-stuff commented Feb 6, 2026

@jamesnw I may have found the issue. There are failing tests, but they are being masked by the script parallelization. I updated them to now stop when any of the scripts fail.

I was able to find the tests that were failing and skipped them until further investigation.

@break-stuff
Copy link
Author

@jamesnw are you able to kick off CI actions again?

@jamesnw
Copy link
Contributor

jamesnw commented Feb 9, 2026

It's currently hanging, and I'm pretty sure it did run last week .

I'm grateful for your help troubleshooting the workflow, but I wonder about keeping that investigation separate from this PR. I'll ask some questions I have about the proposed solution in #378.

@break-stuff
Copy link
Author

I reverted my previous changes. Tests are passing after updating to node v24.12.0. I left the --race flags in because it still hangs locally without them.

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