fix(build): drop redundant dynamic import of branding api

Vite warned that `api/branding.ts` was dynamically imported from
`useAnalyticsCounters.ts` while also being statically imported by 18 other
files. The mixed static+dynamic pattern blocked Vite from splitting the
module into a separate chunk — the dynamic import was a no-op for bundle
size and only added an extra Promise hop.

The same file already had `import { brandingApi } from '../api/branding'`
at the top. Inlined the storeYandexCid call to use it directly.

src/hooks/useAnalyticsCounters.ts

@@ -136,9 +136,10 @@ function syncYandexCid(counterId: string) {
         if (!token) return;
         // Route through brandingApi (apiClient) so baseURL, auth refresh, and
         // error handling all flow through the same interceptors as every other
-        // cabinet API call.
+        // cabinet API call. brandingApi уже импортирован статически — динамический
-        import('../api/branding').then(({ brandingApi: api }) => {
+        // import('../api/branding') ломал code-splitting (Vite warning о том, что
-          api
+        // модуль не может быть вынесен в отдельный chunk при mixed static+dynamic).
+        brandingApi
           .storeYandexCid(cid)
           .then(() => {
             try {
@@ -151,7 +152,6 @@ function syncYandexCid(counterId: string) {
             /* swallow -- non-critical, will retry on next login */
           });
       });
-      });
     } catch {
       /* ignore */
     }