fix(settings): render same-day settings for carriers (trunkrs) with same-day delivery type

[FreekVR]

renders the same-day delivery settings for carriers that expose same-day as a delivery type instead of a shipment option.

Some carriers (like Trunkrs) advertise same-day delivery as a delivery type in their contract, while others (like DHL For You) advertise it as a shipment option. The carrier settings view only checked the shipment option, so for delivery-type carriers the cutoff time field never rendered and the same-day cutoff silently stayed at the 10:00 default. When same-day is all the carrier offers, that hides the carrier from checkout after the cutoff with no way to fix it in the admin. The same-day section now renders for both representations and is the single owner of these fields. This also removes a duplicate same-day toggle that was rendered for shipment-option carriers.

Fixes INT-1561

🤖 Generated with Claude Code

[Copilot]

Pull request overview

This PR updates the PDK carrier settings UI so the same-day delivery settings render for carriers that expose same-day as a delivery type (not only as a shipment option), and consolidates same-day settings into a single section to avoid duplicates.

Changes:

• Render the same-day settings section when SAME_DAY is present in carrier.deliveryTypes, not only when the sameDayDelivery shipment option is supported.
• Prevent duplicate same-day toggles by skipping SAME_DAY in both the dynamic delivery-type list and the shipment-options list.
• Extend unit test coverage and update JSON snapshots to reflect the new settings structure.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File	Description
src/Frontend/View/CarrierSettingsItemView.php	Adds SAME_DAY-delivery-type detection for rendering the same-day section; skips SAME_DAY in other sections to avoid duplicates.
tests/Unit/Frontend/View/CarrierSettingsItemViewTest.php	Adds coverage for same-day-as-delivery-type and asserts same-day settings render exactly once across representations.
tests/snapshots/SettingsViewTest__it_gets_settings_view_with_data_set_carrier_settings__1.json	Snapshot updates reflecting removal of duplicate same-day elements and updated ordering.
tests/snapshots/FrontendRenderServiceTest__it_renders_component_with_data_set_plugin_settings__1.json	Snapshot updates reflecting removal of duplicate same-day elements and updated ordering.

---

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

[codecov]

Codecov Report

❌ Patch coverage is 94.28571% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 94.31%. Comparing base (0a13274) to head (dc7b575).

Files with missing lines	Patch %	Lines
.../Carrier/Service/CapabilitiesValidationService.php	75.00%	1 Missing ⚠️
...tings/Repository/AbstractPdkSettingsRepository.php	85.71%	1 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##               main     #508      +/-   ##
============================================
- Coverage     94.33%   94.31%   -0.02%     
- Complexity     2414     2422       +8     
============================================
  Files           352      352              
  Lines          7726     7745      +19     
============================================
+ Hits           7288     7305      +17     
- Misses          438      440       +2     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[Copilot]

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.