fix buttons duplicating on start/stop

Author: Onatcer

entrypoints/utils/jira.ts

@@ -259,9 +259,15 @@ function createJiraTimeTrackingButton(
 export async function injectJiraTimeTrackingButton(
   actionsWrapper: HTMLElement,
   issueDescription: string,
+  skipExistingCheck = false,
 ): Promise<void> {
-  // Remove existing button if present
+  // Check if button already exists and skip if so (unless explicitly told to replace)
   const existingButton = document.getElementById(BUTTON_WRAPPER_ID);
+  if (existingButton && !skipExistingCheck) {
+    return;
+  }
+
+  // Remove existing button if present
   if (existingButton) {
     existingButton.remove();
   }
@@ -366,7 +372,11 @@ async function handleJiraTrackingClick(
     // Refresh the button
     const actionsWrapper = findJiraActionsWrapper();
     if (actionsWrapper) {
-      await injectJiraTimeTrackingButton(actionsWrapper, issueDescription);
+      await injectJiraTimeTrackingButton(
+        actionsWrapper,
+        issueDescription,
+        true,
+      );
     }
   } catch (error) {
     console.error("Failed to toggle time tracking:", error);

entrypoints/utils/linear.ts

@@ -332,9 +332,15 @@ function createTimeTrackingSection(
 export async function injectTimeTrackingSection(
   container: HTMLElement,
   issueDescription: string,
+  skipExistingCheck = false,
 ): Promise<void> {
-  // Remove existing section if present
+  // Check if section already exists and skip if so (unless explicitly told to replace)
   const existingSection = document.getElementById(SECTION_ID);
+  if (existingSection && !skipExistingCheck) {
+    return;
+  }
+
+  // Remove existing section if present
   if (existingSection) {
     existingSection.remove();
   }
@@ -451,7 +457,11 @@ async function handleTrackingClick(
     // Refresh the section by finding the properties sidebar container
     const propertiesSidebar = findPropertiesSidebar();
     if (propertiesSidebar) {
-      await injectTimeTrackingSection(propertiesSidebar, issueDescription);
+      await injectTimeTrackingSection(
+        propertiesSidebar,
+        issueDescription,
+        true,
+      );
     }
   } catch (error) {
     console.error("Failed to toggle time tracking:", error);

entrypoints/utils/plane.ts

@@ -244,9 +244,15 @@ function createPlaneTimeTrackingButton(
 export async function injectPlaneTimeTrackingButton(
   actionsWrapper: HTMLElement,
   issueDescription: string,
+  skipExistingCheck = false,
 ): Promise<void> {
-  // Remove existing button if present
+  // Check if button already exists and skip if so (unless explicitly told to replace)
   const existingButton = document.getElementById(BUTTON_ID);
+  if (existingButton && !skipExistingCheck) {
+    return;
+  }
+
+  // Remove existing button if present
   if (existingButton) {
     existingButton.remove();
   }
@@ -348,7 +354,11 @@ async function handlePlaneTrackingClick(
     // Refresh the button
     const actionsWrapper = findPlaneActionsWrapper();
     if (actionsWrapper) {
-      await injectPlaneTimeTrackingButton(actionsWrapper, issueDescription);
+      await injectPlaneTimeTrackingButton(
+        actionsWrapper,
+        issueDescription,
+        true,
+      );
     }
   } catch (error) {
     console.error("Failed to toggle time tracking:", error);

package-lock.json

@@ -2,7 +2,7 @@
   "name": "solidtime-browser-extension",
   "description": "Official Browser Extension for solidtime",
   "private": true,
-  "version": "0.0.7",
+  "version": "0.0.8",
   "type": "module",
   "scripts": {
     "dev": "wxt",