refactor: trim comments on DynamicContext refactor

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

Author: amitsingh-007

apps/extension/src/entrypoints/popup/panels/PersonsPanel/components/PersonsPanel.tsx

@@ -150,8 +150,6 @@ function PersonsPanel() {
             persons={filteredAndOrderedPersons}
             bookmarkListProps={{
               fullscreen: true,
-              // The edit route is the extension's concern, so it is supplied
-              // here rather than hardcoded inside the shared component
               onBookmarkEdit: ({ url, parentId }) => {
                 navigate(
                   getBookmarksPanelUrl({

apps/extension/src/entrypoints/popup/provider/DynamicProvider.tsx

@@ -25,9 +25,7 @@ function DynamicProvider({ children }: PropsWithChildren) {
         set: setToChromeStorage,
       },
       tabs: {
-        // Arming the watcher before opening is the invariant this seam exists
-        // to enforce; startHistoryMonitor is idempotent so looping callers can
-        // call this per url
+        // Idempotent, so loop callers can call this per url
         open: (url: string) => {
           startHistoryMonitor();
           browser.tabs.create({ url, active: false });

apps/web/src/app/provider/DynamicProvider.tsx

@@ -20,8 +20,6 @@ function DynamicProvider({ children }: PropsWithChildren) {
         get: async <T,>(key: string) => getFromLocalStorage<T>(key),
         set: async (key: string, value: any) => setToLocalStorage(key, value),
       },
-      // Delegates to openNewTab so the noopener/noreferrer flags and the
-      // focus() call are not silently dropped
       tabs: { open: openNewTab },
       favicon: { getUrl: getYandexFaviconUrl },
     }),

packages/shared/src/components/Bookmarks/utils/index.ts

@@ -15,11 +15,7 @@ export const isFolderEmpty = (
   return !folder || folder.length === 0;
 };
 
-/**
- * Takes the favicon provider as an argument rather than reading DynamicContext:
- * the cache/preload path is plain async functions, not components, so it cannot
- * consume a React context.
- */
+// Provider is a param, not from DynamicContext: callers here are not components
 export const getBookmarkFaviconUrls = (
   urlList: IBookmarksObj['urlList'],
   getFaviconUrl: (url: string) => string

packages/shared/src/components/Persons/components/BookmarksList.tsx

@@ -24,10 +24,7 @@ interface Props {
   personToOpen: IPerson | undefined;
   imageUrl: string;
   fullscreen: boolean;
-  /**
-   * Rendered only when supplied, so the shared component no longer needs to
-   * know the extension's edit route to decide whether editing is possible.
-   */
+  /** The edit button renders only when this is supplied. */
   onBookmarkEdit?: (bookmark: IBookmarkWithFolder) => void;
 }
 

packages/shared/src/provider/DynamicContext.ts

@@ -12,11 +12,6 @@ interface IDynamicContext {
     get: <T>(key: string) => Promise<T | null | undefined>;
     set: (key: string, data: any) => Promise<void>;
   };
-  /**
-   * Opening a link differs per platform (browser.tabs.create vs window.open),
-   * and the extension must arm its history watcher first. Routing it through
-   * the seam keeps that invariant in one place instead of at every call site.
-   */
   tabs: {
     open: (url: string) => void;
   };