XB's `test_site` recipe fails to install its default content through Drush in Playwright

I doubt it's causing this, but just in case: also revealed this (edge case-y!) Recipes bug: 🐛 \Drupal\Core\Recipe\ConfigConfigurator unsets the uuid of active configuration, but it should only do that if the recipe data does not contain a UUID Active .

phenaproxima → credited justafish → .

Trying to credit @justafish for help debugging this in a back channel.

This is now in a much better place thanks to @justafish's targeted simplification of the code that applies recipes to the test site.

It's still failing, but for a completely clear reason:

     /builds/issue/experience_builder-3532268/_drupal/web/../web/modules/contrib  
      /experience_builder/tests/fixtures/recipes/test_site/content/node/c66664af-  
      53b9-42f4-a0ca-8ecc9edacb8c.yml: field_xb_demo.1.inputs.348bfa10-af72-49cd-  
      900b-084d617c87df.image.src=Invalid URL format.    

If I go look at that spot, I see this (truncated for brevity):

inputs: '{"image":{"sourceType":"static:field_item:entity_reference","value":{"target_id":"1"} ...

I don't think we can rely on images by serial ID -- only by UUID. That would be my first guess as to why this isn't working.

Is this a known issue? Fixed in some way? Out of scope here? I could use some input from Wim or another elder XB wizard.

Traced the validation error down to \Drupal\Core\StreamWrapper\PublicStream::baseUrl when we're running that via core/scripts/drupal the base URL ends up being borked and the URL is invalid. Made a tweak to the playwright setup that should fix it

If core/scripts/drupal is setting a bad base URL, that sounds like something that should be fixed in core as well. I'll open a follow-up for that.

Wow, epic work here, all three of you! 🤯👏

Just one question: target_id should not be present in the default content export? Can its presence not cause problems? Should that be tackled in a follow-up, too?

But how can we know it won't cause other side effects?

Besides, AFAICT this currently doesn't actually do what we want it to do? 😅 See https://git.drupalcode.org/project/experience_builder/-/merge_requests/1...

(Found 2 bugs AFAICT.)

Having a test makes of ton of sense and catches the bug you found.

Opened follow-up to actually fix this in core: ✨ EntityReferenceItem should support a virtual `target_uuid` property Active

RTBC as far as I'm concerned now — can't approve the MR because I created it originally 😶‍🌫️

Fixed syntax error 😅

Synced the branch with 0.x, restoring RTBC.

LGTM! Thank you 👏🎉