Convert YAML boolean defaults to strings

[gOOvER]

Problem:
When importing eggs from YAML format, boolean values (true/false) were being converted to integers (1/0) when saved to the database. This broke validation rules like in:true,false that require exact string matches.

Root Cause:
Symfony YAML parser correctly converts YAML booleans to PHP booleans
Laravel automatically converts PHP booleans to integers when saving to TEXT fields
Result: default_value: true → stored as "1" instead of "true"

Solution:
Added explicit conversion of boolean values to their string representations ("true"/"false") in EggImporterService::parse() before saving to the database.

Impact:
Variables with rules: in:true,false now work correctly after YAML import

Backwards compatible: quoted boolean strings in YAML remain unchanged
Only affects actual YAML boolean values (unquoted true/false)

[coderabbitai]

📝 Walkthrough

Walkthrough

Added a normalization step in EggImporterService::parse to convert boolean default_value entries in YAML-parsed variables to string representations ('true'/'false') to prevent Laravel from storing them as numeric 1/0 in TEXT fields.

Changes

Cohort / File(s)	Summary
Variable Normalization app/Services/Eggs/Sharing/EggImporterService.php	Added boolean-to-string normalization for variable default values during YAML parsing to ensure consistent storage in TEXT database fields.

🚥 Pre-merge checks | ✅ 4

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title 'Convert YAML boolean defaults to strings' directly reflects the main change in the pull request - converting boolean values to strings during YAML import.
Description check	✅ Passed	The description clearly relates to the changeset, explaining the problem, root cause, solution, and impact of the boolean-to-string conversion in YAML egg imports.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.
Merge Conflict Detection	✅ Passed	✅ No merge conflicts detected when merging into main

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

---

No actionable comments were generated in the recent review. 🎉

Tip

Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord.

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[Copilot]

Pull request overview

This PR fixes a bug where YAML boolean values (true/false) in egg variable default values were being incorrectly converted to integers (1/0) when saved to the database, breaking validation rules like in:true,false that require exact string matches.

Changes:

• Added boolean-to-string conversion logic in EggImporterService::parse() to convert PHP boolean values to their string representations ("true"/"false") before saving to the database

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.