feat(analytics): emit dapp_name + dapp_url on Remote Connection Request Received/Failed

Wire the self-reported dapp identity from the V2 MWP connection request
metadata into both Remote Connection Request analytics events:

- `Remote Connection Request Received` (success path):
  populates `dapp_name` / `dapp_url` from `connReq.metadata.dapp.{name,url}`
- `Remote Connection Request Failed` (catch path):
  same, using optional chaining so both gracefully degrade to `undefined`
  when the failure occurred before `parseConnectionRequest` returned (e.g.
  invalid URL, malformed payload)

Pairs with the Consensys/segment-schema PR that declares both fields as
optional `string` properties on the Received and Failed YAMLs. Mirrors
the `dapp_name` / `dapp_url` already emitted on Connect Request
Started/Completed/Cancelled, closing the parity gap on the earliest
wallet-side connection signal so per-dapp drop-off can be measured
before the permission UI.

Tests updated:
- happy-path Received assertion now also expects `dapp_name`/`dapp_url`
- failed-path Failed assertion now also expects `dapp_name`/`dapp_url`
  + `sdk_version`/`sdk_platform` (which were silently being emitted but
  never asserted)

Author: adonesky1

app/core/SDKConnectV2/services/connection-registry.test.ts

@@ -442,6 +442,8 @@ describe('ConnectionRegistry', () => {
           transport_type: TransportType.MWP,
           sdk_version: '2.0.0',
           sdk_platform: 'JavaScript',
+          dapp_name: 'Test DApp',
+          dapp_url: 'https://test.dapp',
         }),
       );
     });
@@ -583,6 +585,10 @@ describe('ConnectionRegistry', () => {
         expect.objectContaining({
           remote_session_id: mockConnectionRequest.sessionRequest.id,
           transport_type: TransportType.MWP,
+          sdk_version: '2.0.0',
+          sdk_platform: 'JavaScript',
+          dapp_name: 'Test DApp',
+          dapp_url: 'https://test.dapp',
           failure_reason: 'Connection failed',
         }),
       );

app/core/SDKConnectV2/services/connection-registry.ts

@@ -258,6 +258,8 @@ export class ConnectionRegistry {
         transport_type: TransportType.MWP,
         sdk_version: connReq.metadata.sdk.version,
         sdk_platform: connReq.metadata.sdk.platform,
+        dapp_name: connReq.metadata.dapp.name,
+        dapp_url: connReq.metadata.dapp.url,
       });
 
       // Defense-in-depth: block connections whose self-reported dapp metadata
@@ -305,6 +307,8 @@ export class ConnectionRegistry {
         transport_type: TransportType.MWP,
         sdk_version: connReq?.metadata?.sdk?.version,
         sdk_platform: connReq?.metadata?.sdk?.platform,
+        dapp_name: connReq?.metadata?.dapp?.name,
+        dapp_url: connReq?.metadata?.dapp?.url,
         failure_reason: error instanceof Error ? error.message : String(error),
       });