feat/error-handling-browser (#22)

* feat: Add error handling for expected browser errors

* chore: update gitignore with .claude/ directory

* docs: add pull request and commit convention documentation

* chore: bump version to 0.1.12

Author: nuttycc

.gitignore

@@ -1,3 +1,6 @@
+# Agent
+.claude/
+
 # docs, notes
 .hidden/
 repomix.config.json

docs/agent_docs/COMMIT_CONVENTION.md

@@ -0,0 +1,41 @@
+# Commit Convention
+
+## Format
+
+```
+<type>(<scope>): <description>
+```
+
+## Types
+
+| Type | Description |
+|------|-------------|
+| `feat` | New feature |
+| `fix` | Bug fix |
+| `docs` | Documentation only |
+| `refactor` | Code refactoring (no feature/fix) |
+| `perf` | Performance improvement |
+| `test` | Add or update tests |
+| `chore` | Build, dependencies, configs |
+
+## Scope (optional)
+
+Component or area affected: `popup`, `background`, `content`, `options`, `ui`, `db`, etc.
+
+## Examples
+
+```bash
+feat(popup): add time tracking chart
+fix(background): resolve memory leak in timer
+refactor: migrate to composition API
+docs: update README installation steps
+perf(db): optimize query performance
+chore: update dependencies
+```
+
+## Rules
+
+1. Use lowercase
+2. No period at the end
+3. Use imperative mood ("add" not "added")
+4. Keep under 72 characters

docs/agent_docs/PULL_REQUEST_CONVENTION.md

@@ -0,0 +1,58 @@
+# Pull Request Convention
+
+## Branch Naming
+
+```
+<type>/<short-description>
+```
+
+Examples:
+- `feat/dark-mode`
+- `fix/timer-accuracy`
+- `refactor/state-management`
+
+## PR Title
+
+Same format as commit messages:
+
+```
+<type>(<scope>): <description>
+```
+
+## PR Labels
+
+Add one of these labels for auto-generated release notes:
+
+| Label | Use for |
+|-------|---------|
+| `enhancement` or `feature` | New features |
+| `bug` or `fix` | Bug fixes |
+| `performance` | Performance improvements |
+| `documentation` | Documentation updates |
+
+## PR Body Template
+
+```markdown
+## Summary
+- Brief description of changes
+
+## Type
+<!-- Add label: enhancement, bug, documentation, performance -->
+
+## Changes
+- Change 1
+- Change 2
+
+## Test
+- [ ] Tested locally
+```
+
+## Workflow
+
+1. Create feature branch: `git checkout -b feat/xxx`
+2. Make commits following commit convention
+3. Push: `git push -u origin HEAD`
+4. Create PR on GitHub
+5. Add appropriate label
+6. Request review if needed
+7. Merge to main

package.json

@@ -2,7 +2,7 @@
   "name": "lumin-time",
   "description": "LuminTime is a privacy-first browser extension that automatically tracks your online activity and labels it intelligently.",
   "private": true,
-  "version": "0.1.1",
+  "version": "0.1.12",
   "type": "module",
   "scripts": {
     "dev": "wxt",

src/entrypoints/background.ts

@@ -5,6 +5,7 @@ import { SessionManager, type ActiveSessionData } from "@/utils/SessionManager";
 import { debugTools } from "@/utils/debugTools";
 
 const IDLE_DETECTION_INTERVAL = 30;
+
 const SESSION_TICK_ALARM_NAME = "session-update";
 const SESSION_PER_MINUTE = 1;
 
@@ -18,6 +19,17 @@ const sessionStorage = storage.defineItem<ActiveSessionData>("session:activeSess
   },
 });
 
+const isExpectedBrowserError = (error: unknown): boolean => {
+  if (!(error instanceof Error)) return false;
+  const msg = error.message;
+  return (
+    msg.includes("No tab with id") ||
+    msg.includes("No window with id") ||
+    msg.includes("No last-focused window") ||
+    msg.includes("Tabs cannot be edited")
+  );
+};
+
 const getActiveTabUrl = async () => {
   const tabs = await browser.tabs.query({ active: true, currentWindow: true });
   if (tabs.length === 0) {
@@ -70,6 +82,10 @@ export default defineBackground(() => {
           eventSource: "tab_activated",
         });
       } catch (error) {
+        if (isExpectedBrowserError(error)) {
+          console.debug("[TabActivated] Tab unavailable:", error);
+          return;
+        }
         console.error("Failed to handle tab activation:", error);
       }
     })();
@@ -95,6 +111,10 @@ export default defineBackground(() => {
           eventSource: "navigation",
         });
       } catch (error) {
+        if (isExpectedBrowserError(error)) {
+          console.debug("[Navigation] Tab/window unavailable:", error);
+          return;
+        }
         console.error(`Failed to process navigation to ${details.url}:`, error);
       }
     })();
@@ -116,6 +136,10 @@ export default defineBackground(() => {
           });
         }
       } catch (error) {
+        if (isExpectedBrowserError(error)) {
+          console.debug("[WindowFocus] Window unavailable:", error);
+          return;
+        }
         console.error("Failed to process window focus change:", error);
       }
     })();
@@ -148,6 +172,10 @@ export default defineBackground(() => {
           sessionManager.handleEvent("idle", { url: null });
         }
       } catch (error) {
+        if (isExpectedBrowserError(error)) {
+          console.debug("[IdleState] Browser state unavailable:", error);
+          return;
+        }
         console.error("Failed to handle idle state change:", error);
       }
     })();