Vision

In app/Services/PlanfactService.php at line 64 (the copayMap definition for Vision), the entry is:
'Lenses.Single Vision Lenses Copay' => ['SingleVisionLenses', 'Materials']

The parse loop (around lines 2338–2370 in parseQuoteVision()) iterates over both keys and writes the SAME copay value to both inNetwork1SingleVisionLenses and inNetwork1Materials.

Please:
1. Show me the copayMap entry and the loop.
2. Confirm whether the dual-write is intentional (both fields are meant to hold the same value, perhaps for two different form labels). If yes, add a comment.
3. If not intentional, find the correct Plan Facts source for "Materials" specifically (likely a different key) and split the mapping.
4. Don't change yet — show the analysis.

In app/Services/PlanfactService.php inside the textMap['vision'] definition (around lines 96–108), the In-Network section has entries for:
- "Contacts.Conventional Contacts Discount Beyond Allowance" → inNetwork1ContactsAboveAllowance
- "Frames.Frames Discount Beyond Allowance" → inNetwork1FramesAdditional

But the Out-of-Network section is missing the corresponding entries.

Please:
1. Show me the In-Network and Out-of-Network sections of textMap['vision'].
2. Confirm whether those two fields should have out-of-network mappings (e.g., outNetworkContactsAboveAllowance, outNetworkFramesAdditional). If the Quote model has those fields, add the entries.
3. If the Quote model doesn't have those fields, document why out-of-network is intentionally not captured.
4. Don't change yet — show the analysis.

In app/Services/PlanfactService.php at lines 2398–2411, inside parseQuoteVision(), the Contacts-In-Lieu and Lasik switches match strtolower("yes") / strtolower("no").

If Plan Facts ever returns "Y", "N", "true", "false", or "included" / "not included", they fail silently.

Please:
1. Show me the switch.
2. Broaden the matcher to handle the common variants: "yes"/"y"/"true"/"included" → Yes; "no"/"n"/"false"/"not included" → No.
3. Add a default branch that logs unexpected values rather than silently dropping.
4. Show me the proposed change.

In app/Services/PlanfactService.php at line 2321, inside parseQuoteVision(), the parser hardcodes two network names:
$inNetwork = "In-Network";
$outNetwork = "Out-of-Network";

If a carrier uses different network names (e.g., "Provider Network", "Non-Participating"), every subsequent lookup silently fails and the Quote ends up empty.

Please:
1. Show me the assignment.
2. Replace it with detection: read the actual network names from the response (e.g., $data['Vision']['Networks'] or wherever Plan Facts lists them) and use those.
3. If the carrier really does have non-standard names, the parse should still work — or fall back to the hardcoded names with a log warning.
4. Show me the proposed change.

In app/Services/PlanfactService.php at line 2317, inside parseQuoteVision(), the Quote name field is set from Plan Facts' "Plan Name" with no fallback.

If Plan Facts omits "Plan Name" (or it's empty), the Quote ends up nameless.

Please:
1. Show me the current assignment.
2. Add a fallback: if "Plan Name" is missing or empty, use the carrier name + "Vision Plan" or similar.
3. Show me the proposed change.