itube-specs 0.0.840 → 0.0.842

package/components/layout/footer-language.vue

@@ -66,7 +66,7 @@ function handler() {
   isMenuShow.value = !isMenuShow.value;
 }
 
-const enabledLocales = useState('enabledLocales');
+const enabledLocales = useEnabledLocales();
 
 const localeName = computed(() => locales.value.find(item => item.code === locale.value)?.name || 'English');
 

package/components/layout/language.vue

@@ -120,8 +120,6 @@ defineProps<{
 }>();
 
 const countriesItems = useState<ISelectItem[]>('countriesItems', () => []);
-const enabledLanguageCodesState = useState<string[]>('enabledLanguageCodes', () => []);
-
 const clientCountryCode = useState<string>('clientCountryCode', () => '');
 
 const clientCountryIcon = computed(() => {
@@ -146,13 +144,6 @@ const filteredCountries = computed(() => {
   return countriesItems.value.filter(item => item.title.toLowerCase().includes(query));
 });
 
-if (!enabledLanguageCodesState.value.length) {
-  const { data: enabledLanguageCodes } = await useFetch<string[]>('/bff/site/languages', { default: () => [] });
-  if (enabledLanguageCodes.value?.length) {
-    enabledLanguageCodesState.value = enabledLanguageCodes.value;
-  }
-}
-
 const isCountryLoading = ref(false);
 
 // Список стран не приходит в payload (см. plugins/countries.server.ts) — догружаем
@@ -178,13 +169,7 @@ function onCountryToggle(e: Event) {
   if (isCountryOpen.value) loadCountries();
 }
 
-const enabledLocales = useState('enabledLocales', () => locales.value);
-
-watch(enabledLanguageCodesState, (codes) => {
-  enabledLocales.value = codes?.length
-      ? locales.value.filter(item => codes.includes(item.code))
-      : locales.value;
-}, { immediate: true });
+const enabledLocales = useEnabledLocales();
 
 const selectedLocale = ref(locale.value);
 

package/composables/__tests__/use-seo-links.test.ts

@@ -3,6 +3,7 @@ import { describe, it, expect, beforeEach } from 'vitest';
 import { mockNuxtImport } from '@nuxt/test-utils/runtime';
 import { useSeoLinks } from '../use-seo-links';
 import { useSeoOverridesState } from '../use-seo-overrides';
+import { useEnabledLanguageCodes } from '../use-enabled-locales';
 
 let mockRoute = { path: '/videos', query: {} as Record<string, any> };
 
@@ -23,6 +24,7 @@ describe('useSeoLinks', () => {
     mockRoute = { path: '/videos', query: {} };
     mockI18n.locale.value = 'en';
     useSeoOverridesState().value = null;
+    useEnabledLanguageCodes().value = [];
   });
 
   it('canonical URL', () => {
@@ -52,6 +54,26 @@ describe('useSeoLinks', () => {
     expect(canonicalUrl.value).toContain('sort=popular');
   });
 
+  it('page=1 не попадает в canonical', () => {
+    mockRoute = { path: '/videos', query: { page: '1', sort: 'popular' } };
+    const { canonicalUrl } = useSeoLinks('https://example.com', mockI18n, mockLocalePath);
+    expect(canonicalUrl.value).toBe('https://example.com/videos?sort=popular');
+  });
+
+  it('page > 1 в canonical остаётся', () => {
+    mockRoute = { path: '/videos', query: { page: '2' } };
+    const { canonicalUrl } = useSeoLinks('https://example.com', mockI18n, mockLocalePath);
+    expect(canonicalUrl.value).toContain('page=2');
+  });
+
+  it('noindex убирает hreflang-альтернативы', () => {
+    mockRoute = { path: '/search/anal/videos', query: {} };
+    useSeoOverridesState().value = { path: '/search/anal/videos', noindex: true };
+    const { canonicalUrl, alternateLinks } = useSeoLinks('https://example.com', mockI18n, mockLocalePath);
+    expect(alternateLinks.value).toEqual([]);
+    expect(canonicalUrl.value).toBe('https://example.com/search/anal/videos');
+  });
+
   it('alternate links для всех локалей', () => {
     const { alternateLinks } = useSeoLinks('https://example.com', mockI18n, mockLocalePath);
     const hreflangs = alternateLinks.value.map(l => l.hreflang);
@@ -63,6 +85,26 @@ describe('useSeoLinks', () => {
     expect(hreflangs).toContain('x-default');
   });
 
+  it('alternate links только для включённых на сайте локалей', () => {
+    useEnabledLanguageCodes().value = ['en', 'fr'];
+    const { alternateLinks } = useSeoLinks('https://example.com', mockI18n, mockLocalePath);
+    const hreflangs = alternateLinks.value.map(l => l.hreflang);
+    expect(hreflangs).toContain('en');
+    expect(hreflangs).toContain('fr');
+    expect(hreflangs).toContain('x-default');
+    expect(hreflangs).not.toContain('de');
+    expect(hreflangs).not.toContain('ja-JP');
+    expect(hreflangs).not.toContain('zh-CN');
+  });
+
+  it('текущая локаль остаётся в alternate, даже если её нет в списке включённых', () => {
+    mockRoute = { path: '/ja/videos', query: {} };
+    mockI18n.locale.value = 'ja';
+    useEnabledLanguageCodes().value = ['en', 'fr'];
+    const { alternateLinks } = useSeoLinks('https://example.com', mockI18n, mockLocalePath);
+    expect(alternateLinks.value.map(l => l.hreflang)).toContain('ja-JP');
+  });
+
   it('ja маппится в ja-JP', () => {
     const { alternateLinks } = useSeoLinks('https://example.com', mockI18n, mockLocalePath);
     const jaLink = alternateLinks.value.find(l => l.hreflang === 'ja-JP');
@@ -126,9 +168,9 @@ describe('useSeoLinks', () => {
   });
 
   it('игнорирует неразрешённые query params', () => {
-    mockRoute = { path: '/videos', query: { page: '1', foo: 'bar', categories: 'teen' } };
+    mockRoute = { path: '/videos', query: { page: '2', foo: 'bar', categories: 'teen' } };
     const { canonicalUrl } = useSeoLinks('https://example.com', mockI18n, mockLocalePath);
-    expect(canonicalUrl.value).toContain('page=1');
+    expect(canonicalUrl.value).toContain('page=2');
     expect(canonicalUrl.value).toContain('categories=teen');
     expect(canonicalUrl.value).not.toContain('foo');
   });

package/composables/__tests__/use-seo-overrides.test.ts

@@ -0,0 +1,101 @@
+// @vitest-environment nuxt
+import { describe, it, expect, beforeEach } from 'vitest';
+import { mockNuxtImport } from '@nuxt/test-utils/runtime';
+import { effectScope, nextTick, reactive, ref, watch } from 'vue';
+import { useSeoOverridesState, useSeoOverrides, useSetSeoOverride } from '../use-seo-overrides';
+
+// реактивный, а не переприсваиваемый: useRoute() отдаётся один раз на setup,
+// и смена пути должна долетать через реактивность, как в реальном роутере
+const mockRoute = reactive({ path: '/videos/some-video' });
+
+mockNuxtImport('useRoute', () => () => mockRoute);
+
+const countWrites = () => {
+  const writes = ref(0);
+  watch(useSeoOverridesState(), () => { writes.value += 1; }, { flush: 'sync' });
+  return writes;
+};
+
+describe('useSetSeoOverride', () => {
+  beforeEach(() => {
+    mockRoute.path = '/videos/some-video';
+    useSeoOverridesState().value = null;
+  });
+
+  it('пишет уточнение для текущего пути', async () => {
+    const scope = effectScope();
+    scope.run(() => useSetSeoOverride(() => ({ canonicalLocale: 'en' })));
+    await nextTick();
+
+    expect(useSeoOverridesState().value).toEqual({ path: '/videos/some-video', canonicalLocale: 'en' });
+    scope.stop();
+  });
+
+  it('складывает уточнения от разных источников', async () => {
+    const scope = effectScope();
+    scope.run(() => {
+      useSetSeoOverride(() => ({ canonicalLocale: 'en' }));
+      useSetSeoOverride(() => ({ alonePage: true }));
+    });
+    await nextTick();
+
+    expect(useSeoOverridesState().value).toEqual({
+      path: '/videos/some-video',
+      canonicalLocale: 'en',
+      alonePage: true,
+    });
+    scope.stop();
+  });
+
+  // регрессия: watchEffect, читавший и писавший один и тот же стейт, уходил
+  // в бесконечный цикл на клиенте («Maximum recursive updates exceeded»)
+  it('не зацикливается: конечное число записей в стейт', async () => {
+    const writes = countWrites();
+
+    const scope = effectScope();
+    scope.run(() => {
+      useSetSeoOverride(() => ({ canonicalLocale: 'en' }));
+      useSetSeoOverride(() => ({ alonePage: true }));
+    });
+    await nextTick();
+    await nextTick();
+
+    expect(writes.value).toBeLessThanOrEqual(2);
+    scope.stop();
+  });
+
+  it('повторная запись того же значения стейт не трогает', async () => {
+    const scope = effectScope();
+    const alonePage = ref(true);
+    scope.run(() => useSetSeoOverride(() => ({ alonePage: alonePage.value })));
+    await nextTick();
+
+    const writes = countWrites();
+    alonePage.value = true;
+    await nextTick();
+
+    expect(writes.value).toBe(0);
+    scope.stop();
+  });
+
+  it('уточнение с другого пути не читается', () => {
+    useSeoOverridesState().value = { path: '/other', alonePage: true };
+    expect(useSeoOverrides().value).toEqual({});
+  });
+
+  it('смена пути сбрасывает накопленное', async () => {
+    const scope = effectScope();
+    const canonicalLocale = ref<string | undefined>('en');
+    scope.run(() => useSetSeoOverride(() => ({ canonicalLocale: canonicalLocale.value })));
+    await nextTick();
+
+    useSeoOverridesState().value = { ...useSeoOverridesState().value!, alonePage: true };
+
+    mockRoute.path = '/videos/other-video';
+    canonicalLocale.value = undefined;
+    await nextTick();
+
+    expect(useSeoOverridesState().value).toEqual({ path: '/videos/other-video', canonicalLocale: undefined });
+    scope.stop();
+  });
+});

package/composables/use-enabled-locales.ts

@@ -0,0 +1,17 @@
+/**
+ * Коды локалей, реально включённых на сайте. Заполняет plugins/enabled-locales.server.ts,
+ * значение уезжает в payload. Пустой список = данных нет, потребители используют все локали.
+ */
+export const useEnabledLanguageCodes = () => useState<string[]>('enabledLanguageCodes', () => []);
+
+/**
+ * Локали i18n, отфильтрованные по включённым на сайте — для переключателей языка и hreflang.
+ */
+export const useEnabledLocales = () => {
+    const { locales } = useI18n();
+    const codes = useEnabledLanguageCodes();
+
+    return computed(() => codes.value.length
+        ? locales.value.filter(item => codes.value.includes(item.code))
+        : locales.value);
+};

package/composables/use-noindex.ts

@@ -8,4 +8,6 @@ export function useNoindex() {
       { name: 'robots', content: 'noindex, follow' },
     ],
   });
+
+  useSetSeoOverride(() => ({ noindex: true }));
 }

package/composables/use-seo-links.ts

@@ -15,21 +15,30 @@ type LocalePathMock = (path: string, localeCode: string) => string;
 export const useSeoLinks = (baseDomain: string, i18n: I18nMock, localePath: LocalePathMock) => {
     const route = useRoute();
     const overrides = useSeoOverrides();
+    const enabledCodes = useEnabledLanguageCodes();
 
     const allowedParams: string[] = ['page', 'sort', 'categories'];
+
+    const isParamAllowed = (key: string, value: string) => {
+        if (!allowedParams.includes(key)) return false;
+        // на единственной странице sort не участвует в URL
+        if (overrides.value.alonePage && key === 'sort') return false;
+        // ?page=1 — тот же контент, что и без параметра, дубля быть не должно
+        if (key === 'page' && value === '1') return false;
+
+        return true;
+    };
+
     const createUrlWithParams = (path: string) => {
         const url = new URL(`${baseDomain}${path}`);
 
         Object.entries(route.query).forEach(([key, value]) => {
-            // на единственной странице sort не участвует в URL
-            if (allowedParams.includes(key) && !(overrides.value.alonePage && key === 'sort')) {
-                if (Array.isArray(value)) {
-                    value.forEach(v => url.searchParams.append(key, v));
-                }
-                else if (value != null) {
-                    url.searchParams.append(key, String(value));
-                }
-            }
+            const values = Array.isArray(value) ? value : [value];
+
+            values.forEach((item) => {
+                if (item == null || !isParamAllowed(key, String(item))) return;
+                url.searchParams.append(key, String(item));
+            });
         });
 
         return url.toString();
@@ -55,6 +64,8 @@ export const useSeoLinks = (baseDomain: string, i18n: I18nMock, localePath: Loca
     const alternateLinks = computed(() => {
         // страница неканонична (нет перевода) — hreflang-кластер объявляет канонический URL, а не эта страница
         if (forcedCanonicalLocale.value) return [];
+        // закрытой от индексации странице языковой кластер не нужен
+        if (overrides.value.noindex) return [];
 
         const links = [];
         const currentLocaleCode = i18n.locale.value;
@@ -77,10 +88,14 @@ export const useSeoLinks = (baseDomain: string, i18n: I18nMock, localePath: Loca
             });
         }
 
+        // локаль, выключенная на сайте, отдаёт 404 (server/middleware/locale-guard.ts) —
+        // такие hreflang вели бы на несуществующие страницы
+        const isEnabled = (code: string) => !enabledCodes.value.length || enabledCodes.value.includes(code);
+
         const otherLocales = i18n.locales.value.filter(
             (locale) => {
                 const localeCode = typeof locale === 'string' ? locale : locale.code;
-                return localeCode !== currentLocaleCode && localeCode !== 'en';
+                return localeCode !== currentLocaleCode && localeCode !== 'en' && isEnabled(localeCode);
             }
         );
 

package/composables/use-seo-overrides.ts

@@ -3,6 +3,8 @@ type SeoOverrides = {
     canonicalLocale?: string;
     /** страница единственная в пагинации — из canonical выкидывается sort */
     alonePage?: boolean;
+    /** страница закрыта от индексации — hreflang-кластер ей не нужен */
+    noindex?: boolean;
 };
 
 type SeoOverridesState = SeoOverrides & {
@@ -26,6 +28,13 @@ export const useSeoOverrides = () => {
     return computed<SeoOverrides>(() => state.value?.path === route.path ? state.value : {});
 };
 
+const isSameOverride = (a: SeoOverridesState | null, b: SeoOverridesState) =>
+    !!a
+    && a.path === b.path
+    && a.canonicalLocale === b.canonicalLocale
+    && a.alonePage === b.alonePage
+    && a.noindex === b.noindex;
+
 /**
  * Записывает уточнение для текущего маршрута. Значения от разных источников
  * (страница, пагинация) складываются; при смене маршрута состояние сбрасывается.
@@ -34,11 +43,19 @@ export const useSetSeoOverride = (overrides: MaybeRefOrGetter<SeoOverrides>) =>
     const route = useRoute();
     const state = useSeoOverridesState();
 
-    watchEffect(() => {
-        const patch = toValue(overrides);
+    // именно watch с явными источниками, а не watchEffect: чтение state внутри колбэка
+    // не отслеживается, иначе запись сюда же перезапускала бы эффект по кругу
+    watch(
+        [() => route.path, () => toValue(overrides)],
+        ([path, patch]) => {
+            const next: SeoOverridesState = state.value?.path === path
+                ? { ...state.value, ...patch }
+                : { path, ...patch };
+
+            if (isSameOverride(state.value, next)) return;
 
-        state.value = state.value?.path === route.path
-            ? { ...state.value, ...patch }
-            : { path: route.path, ...patch };
-    });
+            state.value = next;
+        },
+        { immediate: true }
+    );
 };

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "itube-specs",
   "type": "module",
-  "version": "0.0.840",
+  "version": "0.0.842",
   "main": "./nuxt.config.ts",
   "types": "./types/index.d.ts",
   "scripts": {

package/plugins/enabled-locales.server.ts

@@ -0,0 +1,20 @@
+// SSR-плагин: кладёт коды реально включённых на сайте локалей в useState('enabledLanguageCodes').
+// Нужен, чтобы hreflang в app.vue и переключатели языков не предлагали локали, на которые
+// server/middleware/locale-guard.ts отдаёт 404. Раньше список тянул только layout/language.vue,
+// а он рендерится лениво по клику — на SSR не выполнялся никогда.
+// Эндпоинт читает статический SiteDataService и сам падает обратно на runtimeConfig.public.locales.
+export default defineNuxtPlugin({
+  parallel: true,
+  async setup() {
+    const codes = useEnabledLanguageCodes();
+    if (codes.value.length) return;
+
+    try {
+      const languages = await $fetch<string[]>('/api/site/languages');
+      if (languages?.length) codes.value = languages;
+    }
+    catch {
+      // оставляем пустым — потребители падают обратно на полный список локалей
+    }
+  },
+});