refactor: trim comments on behaviour changes

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

Author: amitsingh-007

apps/extension/src/constants/index.ts

@@ -10,5 +10,4 @@ export const MAX_PANEL_SIZE = {
 
 export const TEST_AUTH_DATA_KEY = '__test_auth_data';
 
-/** Extension popup entry route. Not in @bypass/shared: meaningless on web. */
 export const POPUP_HOMEPAGE = '/popup.html';

apps/extension/src/store/extension.ts

@@ -6,8 +6,7 @@ interface State {
 }
 
 const useExtStore = create<State>()((set) => ({
-  // Defaults to true: Authenticate reads this on mount and would auto-sign-out
-  // if it started false while storage is still resolving
+  // Must default true: Authenticate would auto-sign-out while storage resolves
   isExtensionActive: true,
   setIsExtensionActive: (isExtensionActive: boolean) =>
     set(() => ({ isExtensionActive })),

apps/web/src/app/constants/routes.ts

@@ -1,7 +1,4 @@
-/**
- * Web-app-only routes. Named distinctly from `ROUTES` in @bypass/shared so an
- * auto-import cannot silently swap one for the other.
- */
+/** Named distinctly from `ROUTES` in @bypass/shared to avoid a wrong auto-import. */
 export const WEB_ROUTES = {
   HOMEPAGE: '/',
   BYPASS_LINKS_WEB: '/web-ext',

packages/shared/src/constants/routes.ts

@@ -1,8 +1,3 @@
-/**
- * Panel routes shared by the extension popup and the web app. The extension's
- * own entry route lives in the extension (POPUP_HOMEPAGE) since '/popup.html'
- * is meaningless on web.
- */
 export const ROUTES = {
   SHORTCUTS_PANEL: '/shortcuts-panel/',
   BOOKMARK_PANEL: '/bookmark-panel/',

packages/shared/src/hooks/useIsMobile.ts

@@ -1,17 +1,9 @@
 import { useMediaQuery } from '@mantine/hooks';
 
 /**
- * Viewport width check, not a platform check.
- *
- * The `true` initial value is load-bearing. Mantine defaults to
- * `getInitialValueInEffect: true` and returns `matches || false`, so with no
- * initial value the first render reports "not desktop" everywhere — including
- * the 800px-wide extension popup — making every consumer paint a mobile layout
- * for one frame before correcting. Passing `false` would be a no-op for the
- * same reason; `true` is what makes first paint match steady state on desktop.
- *
- * Deliberately not using `getInitialValueInEffect: false`, which reads
- * window.matchMedia during render and risks a Next.js hydration mismatch.
+ * The `true` initial value is load-bearing: Mantine returns `matches || false`
+ * before its effect runs, so without it every first paint reports mobile.
+ * Passing `false` is a no-op for the same reason.
  */
 const useIsMobile = () => !useMediaQuery('(min-width: 768px)', true);