fix(php): fix boolean clientDefault quoting and inline path param type handling

[Swimburger]

Description

Follow-up to #15465. Fixes two edge-case bugs in the PHP SDK generator's clientDefault support identified by code review.

Changes Made

• RootClientGenerator.ts: Added getClientDefaultPhpLiteral() helper that returns properly formatted PHP literals — single-quoted strings for string values, bare true/false for boolean values. Previously, boolean clientDefault values were incorrectly wrapped in quotes (e.g., $param ??= 'true' instead of $param ??= true), causing a PHP type mismatch when the parameter is typed as ?bool.

• WrappedEndpointRequestGenerator.ts: Removed the php.Type.optional() wrapping for inline path parameters with clientDefault. The DataClass constructor (line 62-63 of DataClass.ts) uses ?? null for optional-typed fields, which discards the clientDefault initializer. Keeping the non-optional type preserves the initializer via ?? <initializer> (line 64-66).

• Changelog: Added generators/php/sdk/changes/unreleased/fix-client-default-boolean-quoting.yml

• Updated README.md generator (if applicable) — N/A

Testing

• Seed test passes: pnpm seed test --generator php-sdk --fixture x-fern-default --skip-scripts --local
• TypeScript compilation passes: pnpm turbo run compile --filter @fern-api/php-sdk
• Biome formatting passes: pnpm check:biome
• No snapshot changes (the x-fern-default fixture only uses string clientDefaults, so these edge-case fixes don't affect its output)

Link to Devin session: https://app.devin.ai/sessions/302ae462e00f48019f81795e34b6937a
Requested by: @Swimburger

---

[devin-ai-integration]

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

• Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
• Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

• Disable automatic comment and CI monitoring

[claude]

Claude Code Review

This repository is configured for manual code reviews. Comment @claude review to trigger a review and subscribe this PR to future pushes, or @claude review once for a one-time review.

_{Tip: disable this comment in your organization's Code Review settings.}

[github-actions]

SDK Generation Benchmark Results

Comparing PR branch against median of 5 nightly run(s) on main (latest: 2026-04-23T04:59:11Z).

Full benchmark table (click to expand)

Generator	Spec	main (generator)	main (E2E)	PR (generator)	Delta
php-sdk	square	43s (n=5)	66s (n=5)	40s	-3s (-7.0%)

main (generator): generator-only time via --skip-scripts (includes Docker image build, container startup, IR parsing, and code generation — this is the same Docker-based flow customers use via fern generate). main (E2E): full customer-observable time including build/test scripts (nightly baseline, informational). Delta is computed against generator-only baseline.
⚠️ = generation exited with a non-zero exit code (timing may not reflect a successful run).
Baseline from nightly runs on main (latest: 2026-04-23T04:59:11Z). Trigger benchmark-baseline to refresh.
Last updated: 2026-04-27 22:24 UTC

[devin-ai-integration]

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 4 additional findings.

[Swimburger]

Is this still necessary?

[devin-ai-integration]

Devin is archived and cannot be woken up. Please unarchive Devin if you want to continue using it.