Contexts are illustrative. Final screens will reference actual GCash patterns.

• Duplicate scope with Overlay component. A separate Overlay record (node 47:329691) already owns the scrim primitive; this Modal should compose it, not re-declare the modal surface. Today the two are maintained independently and there's no annotation describing which is canonical. C7 · Code Connect Linkability
• Two unrelated layouts compressed into one component. type=default and type=with icon are general-purpose dialog shapes, while type=transaction_v1 and type=transaction_v2 are transaction-receipt layouts with their own inner rows and reference-number slot. These are different components masquerading as variants. C4 · Native Mappability
• Mixed enum casing within a single property. type uses default, with icon (space), transaction_v1, transaction_v2 (snake_case). cta uses 1, 1 - vertical, 2 - horizontal, 2 - vertical (spaces around dashes). Neither is consistent with the rest of the DS. C2 · Variant & Property Naming
• Opacity-0 spacer frames instead of gap. Inner layers named _space_16, _space_12 with opacity:0 are used to create vertical rhythm. These are non-semantic and don't translate to native auto-layout. Use gap on the parent instead. C1 · Layer Structure & Naming
• Raster copy-to-clipboard icon. The transaction variants use PNG assets shape_half / shape_full for the copy icon. Should be a vector icon instance bound to main/modal-popup/color/icon-copy. C6 · Asset & Icon Quality
• Icon-placeholder slot is a grey circle, not an instance swap. The with icon variants render a raw #C2C6CF circle (icon-placeholder). Consumers can't swap in a real icon without detaching. Should be a Slot backed by the Icon component. C1 · Layer Structure & Naming
• No interaction states on the modal surface. The component ships only a default state — no pressed / dragging state for the CTA group, no loading state for async actions, and no entrance/exit transition annotation. C5 · Interaction State Coverage
• Missing CTA combos. Transaction variants only ship with cta=1; "with icon" only ships with vertical CTAs. The cta axis is not complete across all type values, so designers resort to detaching when they need another arrangement. C2 · Variant & Property Naming
• No Code Connect mapping. Blocked on restructure — once the transaction layout is extracted and the enum values are cleaned up, mapping is trivial. C7 · Code Connect Linkability

• Consolidate with Overlay — one canonical Modal family. The current split (Overlay = scrim only, Modal = surface + scrim baked in) duplicates intent. Rename and re-partition into: Overlay (scrim primitive, already assessed) + Modal (surface composition that references Overlay) + TransactionReceipt (the transaction_v1/v2 layout pulled out as its own card). Document which component owns which concern. Family
• Extract transaction_v1 / transaction_v2 as a separate component. Move the receipt layout into a new Transaction Receipt component (likely a composition of List + Reference Number + Copy action), and drop the transaction_* values from Modal's type enum. Modal keeps only general-dialog variants. Composition
• Normalise enum casing. Pick one convention for all multi-word values. Recommendation: single lowercase words separated by dashes — default, with-icon, cta-single, cta-single-vertical, cta-double-horizontal, cta-double-vertical. Align with Button and other DS components. Rename
• Replace opacity-0 spacer frames with auto-layout gap. Remove the _space_16 / _space_12 invisible rectangles and set gap on the parent auto-layout frames (using space/space-16 and space/space-12 tokens). Native translators can then emit proper spacing parameters. Property
• Convert icon-placeholder into a Figma Slot. Add a named icon slot to the with icon variants backed by the DS Icon component, so designers can instance-swap without detaching. Default to a neutral status icon. Slot
• Replace raster copy icon with a vector instance. Swap shape_half / shape_full PNGs for the DS vector Copy icon and bind colour to main/modal-popup/color/icon-copy. While there, add a pressed / copied success state. Asset
• Complete the CTA matrix. Ship every type × cta combination (or constrain the schema so unsupported combos aren't implied). Currently default is missing 1-vertical, with icon is missing horizontal pairs, and transactions only ship with cta=1. Either fill the gaps or reshape the enum. Property
• Add loading and destructive states. Modals routinely host async confirmations — add a state=loading variant (CTA replaced with spinner) and surface destructive-action styling via a boolean or via the child Button's existing isError prop. State
• Annotate the dismiss contract. Document on the component: entrance / exit animation, focus trap, restore-focus-on-close, ESC-to-dismiss, tap-outside-dismiss. Developers currently have to infer these from adjacent patterns. Docs
• Title + description copy should come from the DS text styles. Title is bound to Primary/Headlines/Section and description to Secondary/Default/Base (BarkAda). Confirm the secondary-font description is intentional — flag as the standing custom-font action item if not. Docs