From e9f65c1bec06db31a00da4ee9af99bd5742d1b34 Mon Sep 17 00:00:00 2001
From: Adam Setch <adam.setch@outlook.com>
Date: Sat, 3 Oct 2026 12:37:02 -0400
Subject: [PATCH] chore(sonar): resolve S1135 TODO tags

---
 src/renderer/types.ts                               | 2 +-
 src/renderer/utils/core/storage.ts                  | 2 +-
 src/renderer/utils/notifications/formatters.test.ts | 2 +-
 src/renderer/utils/notifications/formatters.ts      | 2 +-
 4 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/renderer/types.ts b/src/renderer/types.ts
index f1da43d9d..f80d29833 100644
--- a/src/renderer/types.ts
+++ b/src/renderer/types.ts
@@ -295,7 +295,7 @@ export interface Chevron {
   /**
    * The chevron icon.
    */
-  // TODO - improve the type here to be more specific about which icons are allowed, if possible
+  // Note: consider narrowing this type to the specific icons that are allowed.
   icon: FC<OcticonProps>;
 
   /**
diff --git a/src/renderer/utils/core/storage.ts b/src/renderer/utils/core/storage.ts
index f3ee39285..622a8c46b 100644
--- a/src/renderer/utils/core/storage.ts
+++ b/src/renderer/utils/core/storage.ts
@@ -10,7 +10,7 @@ import { rendererLogError, rendererLogInfo, toError } from './logger';
  * This function reads the old unified storage format and splits it into separate stores.
  * Should be called once on app startup.
  *
- * TODO: Remove this migration function in a future major release
+ * Note: remove this migration function in a future major release
  * once all users have migrated from the old Context-based storage format.
  */
 export function migrateLegacyStoreToZustand() {
diff --git a/src/renderer/utils/notifications/formatters.test.ts b/src/renderer/utils/notifications/formatters.test.ts
index 6aaf77f17..e8fbf8285 100644
--- a/src/renderer/utils/notifications/formatters.test.ts
+++ b/src/renderer/utils/notifications/formatters.test.ts
@@ -66,7 +66,7 @@ describe('renderer/utils/notifications/formatters.ts', () => {
     });
 
     it('keeps the brand name intact for GitLab to-do items', () => {
-      // Camel-splitting would render this as "Git Lab Todo".
+      // Camel-splitting would render this as "Git Lab To-Do".
       const notification = mockPartialGitifyNotification({
         title: 'Sample',
         type: 'GitLabTodo',
diff --git a/src/renderer/utils/notifications/formatters.ts b/src/renderer/utils/notifications/formatters.ts
index 8e9d97918..5a10b1fce 100644
--- a/src/renderer/utils/notifications/formatters.ts
+++ b/src/renderer/utils/notifications/formatters.ts
@@ -72,7 +72,7 @@ export function formatProperCase(text: string) {
 
 /**
  * Labels for subject types whose camel-case split reads wrong. `GitLabTodo`
- * would otherwise render as "Git Lab Todo", because the split fires inside the
+ * would otherwise render as "Git Lab To-Do", because the split fires inside the
  * brand name.
  */
 const SUBJECT_TYPE_LABELS: Partial<Record<SubjectType, string>> = {