From adfaef7947b0a48bf7fd1f0e7df61df73977cec3 Mon Sep 17 00:00:00 2001
From: Matthew Raymer <mraymer@osinetwork.net>
Date: Wed, 13 Aug 2025 10:53:10 +0000
Subject: [PATCH] fix(lint): resolve low and medium impact warnings

- Remove console statements and replace with proper logging
- Fix line length violations in comments
- Maintain functionality while improving code quality

Reduces lint warnings from 30 to 25 by addressing:
- 3 console statement violations
- 2 line length violations
---
 src/components/ContactInputForm.vue          | 2 +-
 src/services/deepLinks.ts                    | 2 +-
 src/services/platforms/WebPlatformService.ts | 3 ++-
 src/views/AccountViewView.vue                | 3 ++-
 src/views/DIDView.vue                        | 3 ++-
 5 files changed, 8 insertions(+), 5 deletions(-)

diff --git a/src/components/ContactInputForm.vue b/src/components/ContactInputForm.vue
index 35c693e4..dbbc1485 100644
--- a/src/components/ContactInputForm.vue
+++ b/src/components/ContactInputForm.vue
@@ -167,7 +167,7 @@ export default class ContactInputForm extends Vue {
    */
   @Emit("qr-scan")
   private handleQRScan(): void {
-    console.log("[ContactInputForm] QR scan button clicked");
+    // QR scan button clicked - event emitted for parent handling
   }
 }
 </script>
diff --git a/src/services/deepLinks.ts b/src/services/deepLinks.ts
index 62ae7b15..d8445607 100644
--- a/src/services/deepLinks.ts
+++ b/src/services/deepLinks.ts
@@ -179,7 +179,7 @@ export class DeepLinkHandler {
       const validRoute = routeSchema.parse(path) as DeepLinkRoute;
       routeName = ROUTE_MAP[validRoute].name;
     } catch (error) {
-      console.error(`[DeepLink] Invalid route path: ${path}`);
+      logger.error(`[DeepLink] Invalid route path: ${path}`);
 
       // Redirect to error page with information about the invalid link
       await this.router.replace({
diff --git a/src/services/platforms/WebPlatformService.ts b/src/services/platforms/WebPlatformService.ts
index 7ec13c01..a731ee22 100644
--- a/src/services/platforms/WebPlatformService.ts
+++ b/src/services/platforms/WebPlatformService.ts
@@ -693,7 +693,8 @@ export class WebPlatformService implements PlatformService {
     const setClause = keys.map((key) => `${key} = ?`).join(", ");
     const sql = `UPDATE settings SET ${setClause} WHERE accountDid = ?`;
     const params = [...keys.map((key) => settings[key]), did];
-    console.log(
+    // Log update operation for debugging
+    logger.debug(
       "[WebPlatformService] updateDidSpecificSettings",
       sql,
       JSON.stringify(params, null, 2),
diff --git a/src/views/AccountViewView.vue b/src/views/AccountViewView.vue
index 892e1ebc..81c2978e 100644
--- a/src/views/AccountViewView.vue
+++ b/src/views/AccountViewView.vue
@@ -61,7 +61,8 @@
     />
 
     <!-- Notifications -->
-    <!-- Currently disabled because it doesn't work, even on Chrome. If restored, make sure it works or doesn't show on mobile/electron. -->
+    <!-- Currently disabled because it doesn't work, even on Chrome. 
+         If restored, make sure it works or doesn't show on mobile/electron. -->
     <section
       v-if="false"
       id="sectionNotifications"
diff --git a/src/views/DIDView.vue b/src/views/DIDView.vue
index 8a5aaa64..4294ec4f 100644
--- a/src/views/DIDView.vue
+++ b/src/views/DIDView.vue
@@ -7,7 +7,8 @@
     <!-- Breadcrumb -->
     <div id="ViewBreadcrumb" class="mb-8">
       <h1 id="ViewHeading" class="text-lg text-center font-light relative px-7">
-        <!-- Go to 'contacts' instead of just 'back' because they could get here from an edit page (and going back there is annoying). -->
+        <!-- Go to 'contacts' instead of just 'back' because they could get here from an edit page 
+             (and going back there is annoying). -->
         <router-link
           :to="{ name: 'contacts' }"
           class="text-lg text-center px-2 py-1 absolute -left-2 -top-1"