Remove Fractal

[lancepioch]

Final step of the API contract freeze stack (#2520 → #2521 → this). Both predecessors are merged: the wire format is pinned by the fixture suite and every endpoint renders through the laravel-data layer, so nothing uses Fractal anymore. This PR deletes it. Net −2,424 lines, and almost all of it is pure removal.

What's removed

Removal	Scope
All 31 transformer classes	app/Transformers/**
Custom Fractal wrapper + serializer	Extensions/Spatie/Fractalistic/Fractal, Extensions/League/Fractal/Serializers/PanelSerializer
Fractal config	config/fractal.php
Composer dependency	spatie/laravel-fractal and its league/fractal chain
Fractal-specific Scramble extensions	FractalResponseTypeInfer, TransformerFactoryTypeInfer, TransformerModelBindingExtension

What replaces it

One new class, app/Extensions/Dedoc/Scramble/PanelResponseTypeInfer.php, takes over API doc generation from the three deleted Scramble extensions. It follows the PanelResponse fluent chain and derives attribute schemas from the Data class properties, covering:

• MapOutputName renames
• nullable and Optional keys
• enum backing types
• the same envelope, pagination, and addMeta shapes as before

Suggested review order

1. PanelResponseTypeInfer.php: the only file with new logic (373 lines).
2. Runtime edits: ApplicationApiController, ClientApiController, BackupController, AppServiceProvider, which just drop the Fractal wiring.
3. Test helpers: assertJsonTransformedWith and friends now build expected attributes through the Data classes, the same container path the envelope takes in production.
4. Everything else is deletion, safe to skim.

Verification

• ✅ The Freeze the API wire format with a test fixture suite #2520 fixture suite passes with zero snapshot changes, so the API wire format is byte-for-byte unchanged
• ✅ Full Integration and Unit suites are green
• ✅ ApiDocumentationTest passes against the new inference

Known tradeoff

Array-shaped fields like limits now document as generic arrays rather than keyed objects, since the reflection doesn't parse array-shape docblocks. The old extension occasionally did better there; parsing those docblocks is the upgrade path if it matters. Runtime responses are unaffected, this is docs-only.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 63656c43-7d53-445a-aefc-9a3cba2af47e

📥 Commits

Reviewing files that changed from the base of the PR and between 601d570 and aa2af6a.

📒 Files selected for processing (1)

• tests/TestCase.php

Included review availability: Your plan provides up to 8 included reviews per hour; 5 remain after this review.

---

📝 Walkthrough

Walkthrough

The API moved from Fractal transformers to PanelResponse and Laravel Data resources. Scramble now infers PanelResponse response types. Controllers, dependencies, serializers, transformers, webhook resources, and integration tests were updated or removed.

Changes

PanelResponse migration

Layer / File(s)	Summary
PanelResponse type inference app/Extensions/Dedoc/Scramble/PanelResponseTypeInfer.php	Added fluent-chain state tracking and inference for resource data, collections, metadata, includes, pagination, response status, and attribute types.
Runtime response cutover app/Http/Controllers/Api/Application/ApplicationApiController.php, app/Http/Controllers/Api/Client/ClientApiController.php, app/Providers/AppServiceProvider.php, composer.json	Updated controllers and Scramble registration to use PanelResponse. Removed Fractal transformer wiring and the spatie/laravel-fractal dependency.
Webhook data resources and responses app/Data/Api/Application/Webhook*Data.php, app/Http/Controllers/Api/Application/Webhooks/WebhookController.php	Added webhook configuration and delivery resources. Webhook endpoints now use these resources for collection and item responses.
Legacy transformer removal app/Extensions/League/Fractal/..., app/Extensions/Spatie/Fractalistic/..., app/Transformers/Api/*/*Transformer.php, config/fractal.php	Removed custom Fractal serializers, Fractal integration, transformer factories, transformer bindings, and application and client transformer classes.
Data-class test expectations tests/Feature/ApiDocumentationTest.php, tests/Integration/Api/..., tests/Integration/IntegrationTestCase.php, tests/TestCase.php	Updated documentation and integration tests to resolve data classes, call fromModel, and use ApiResource::RESPONSE_TIMEZONE. Test setup now disables Vite manifest requirements.

Merge Risk: ⚪ Minimal · up to aa2af

The PR removes unused Fractal code and dependency while replacing documentation inference, with API fixtures unchanged and relevant test suites passing; no actionable merge-blocking risk remains.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 70.27% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 37 functions across 13 files.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly and concisely identifies the main change: removing the legacy Fractal API layer.
Description check	✅ Passed	The description directly explains the Fractal removal, its replacement with PanelResponseTypeInfer, affected components, and verification results.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

---

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.}