docs: added console logs

Author: hanzel98

packages/gator-permissions-snap/src/index.ts

@@ -208,8 +208,28 @@ export const onRpcRequest: OnRpcRequestHandler = async ({
   request,
 }) => {
   logger.debug(`RPC request (origin="${origin}"): method="${request.method}"`);
+  console.log('SNAP================================================1');
+  logger.debug('🔍 Detailed origin info:', {
+    origin,
+    originType: typeof origin,
+    originLength: origin?.length,
+    method: request.method,
+  });
+  console.log('SNAP================================================2');
+  // Special logging for revocation requests
+  if (request.method === 'permissionsProvider_submitRevocation') {
+    logger.debug('🚨 REVOCATION RPC REQUEST DETECTED 🚨');
+    logger.debug('Origin:', origin);
+    logger.debug('Origin type:', typeof origin);
+    logger.debug('Method:', request.method);
+    logger.debug('Params:', request.params);
+  }
 
   if (!isMethodAllowedForOrigin(origin, request.method)) {
+    logger.debug('❌ Permission denied:', {
+      origin,
+      method: request.method,
+    });
     throw new InvalidRequestError(
       `Origin '${origin}' is not allowed to call '${request.method}'`,
     );

packages/gator-permissions-snap/src/rpc/permissions.ts

@@ -1,3 +1,5 @@
+import { logger } from '@metamask/7715-permissions-shared/utils';
+
 import { RpcMethod } from './rpcMethod';
 
 const allowedPermissionsByOrigin: { [origin: string]: string[] } = {
@@ -25,5 +27,24 @@ export const isMethodAllowedForOrigin = (
   origin: string,
   method: string,
 ): boolean => {
-  return allowedPermissionsByOrigin[origin]?.includes(method) ?? false;
+  logger.debug('🔍 Checking origin permissions:', {
+    origin,
+    method,
+    originCharCodes: origin.split('').map((c) => c.charCodeAt(0)),
+    allowedOrigins: Object.keys(allowedPermissionsByOrigin),
+    allowedMethodsForOrigin: allowedPermissionsByOrigin[origin],
+    exactMatch: allowedPermissionsByOrigin[origin],
+    includesCheck: allowedPermissionsByOrigin[origin]?.includes(method),
+  });
+
+  const isAllowed =
+    allowedPermissionsByOrigin[origin]?.includes(method) ?? false;
+
+  logger.debug('🔍 Permission check result:', {
+    origin,
+    method,
+    isAllowed,
+  });
+
+  return isAllowed;
 };

packages/gator-permissions-snap/src/rpc/rpcHandler.ts

@@ -139,62 +139,96 @@ export function createRpcHandler(config: {
    * @returns Success confirmation.
    */
   const submitRevocation = async (params: Json): Promise<Json> => {
-    logger.debug('submitRevocation()', params);
+    console.log('================================================2');
+    logger.debug('=== SUBMIT REVOCATION RPC CALLED ===');
+    logger.debug('submitRevocation() called with params:', params);
+    logger.debug('Params type:', typeof params);
+    logger.debug('Params stringified:', JSON.stringify(params, null, 2));
 
     const { delegationHash } = validateRevocationParams(params);
+    logger.debug('Validated delegationHash:', delegationHash);
 
     // First, get the existing permission to validate it exists
+    logger.debug(
+      'Looking up existing permission for delegationHash:',
+      delegationHash,
+    );
     const existingPermission =
       await profileSyncManager.getGrantedPermissionByDelegationHash(
         delegationHash,
       );
 
     if (!existingPermission) {
+      logger.debug(
+        '❌ Permission not found for delegationHash:',
+        delegationHash,
+      );
       throw new InvalidInputError(
         `Permission not found for delegation hash: ${delegationHash}`,
       );
     }
 
+    logger.debug('✅ Found existing permission:', {
+      delegationHash,
+      isRevoked: existingPermission.isRevoked,
+      siteOrigin: existingPermission.siteOrigin,
+    });
+
     // Extract delegationManager and chainId from the permission response for logging
     const { chainId: permissionChainId, signerMeta } =
       existingPermission.permissionResponse;
     const { delegationManager } = signerMeta;
 
-    logger.debug(
-      `Found permission - chainId: ${permissionChainId}, delegationManager: ${delegationManager ?? 'undefined'}`,
-    );
+    logger.debug('Permission details extracted:', {
+      chainId: permissionChainId,
+      delegationManager: delegationManager ?? 'undefined',
+      signerMeta: signerMeta,
+    });
 
     // Check if the delegation is actually disabled on-chain
     if (!delegationManager) {
+      logger.debug('❌ No delegation manager found');
       throw new InvalidInputError(
         `No delegation manager found for delegation hash: ${delegationHash}`,
       );
     }
 
+    logger.debug('Checking if delegation is disabled on-chain...', {
+      delegationHash,
+      chainId: permissionChainId,
+      delegationManager,
+    });
+
     const isDelegationDisabled =
       await profileSyncManager.checkDelegationDisabledOnChain(
         delegationHash,
         permissionChainId,
         delegationManager,
       );
 
+    logger.debug('On-chain check result:', { isDelegationDisabled });
+
     if (!isDelegationDisabled) {
+      logger.debug('❌ Delegation is not disabled on-chain');
       throw new InvalidInputError(
         `Delegation ${delegationHash} is not disabled on-chain. Cannot process revocation.`,
       );
     }
 
     logger.debug(
-      `Processing revocation for delegation ${delegationHash} - validated on-chain`,
+      '✅ Delegation is disabled on-chain, proceeding with revocation',
     );
 
     // Update the permission's revocation status using the optimized method
     // This avoids re-fetching the permission we already have
+    logger.debug('Updating permission revocation status to true...');
     await profileSyncManager.updatePermissionRevocationStatusWithPermission(
       existingPermission,
       true,
     );
 
+    logger.debug('✅ Revocation completed successfully');
+    logger.debug('=== SUBMIT REVOCATION RPC COMPLETED ===');
     return { success: true };
   };
 

packages/gator-permissions-snap/src/utils/validate.ts

@@ -2,7 +2,10 @@ import {
   type RequestExecutionPermissionsParam,
   zRequestExecutionPermissionsParam,
 } from '@metamask/7715-permissions-shared/types';
-import { extractZodError } from '@metamask/7715-permissions-shared/utils';
+import {
+  extractZodError,
+  logger,
+} from '@metamask/7715-permissions-shared/utils';
 import type { Hex } from '@metamask/delegation-core';
 import { InvalidInputError, type Json } from '@metamask/snaps-sdk';
 import { z } from 'zod';
@@ -53,15 +56,24 @@ export function validateRevocationParams(params: Json): {
   delegationHash: Hex;
 } {
   try {
+    console.log('================================================3');
+    logger.debug('🔍 Validating revocation params:', params);
+    logger.debug('Params type:', typeof params);
+
     if (!params || typeof params !== 'object') {
+      logger.debug('❌ Invalid params: not an object');
       throw new InvalidInputError('Parameters are required');
     }
 
+    logger.debug('✅ Params is valid object, parsing with Zod...');
     const validated = zRevocationParams.parse(params);
+    logger.debug('✅ Zod validation successful:', validated);
+
     return {
       delegationHash: validated.delegationHash as Hex,
     };
   } catch (error) {
+    logger.debug('❌ Validation failed:', error);
     if (error instanceof z.ZodError) {
       throw new InvalidInputError(extractZodError(error.errors));
     }