itube-specs 0.0.841 → 0.0.842

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

@@ -54,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);
@@ -148,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-noindex.ts

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

package/composables/use-seo-links.ts

@@ -18,19 +18,27 @@ export const useSeoLinks = (baseDomain: string, i18n: I18nMock, localePath: Loca
     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();
@@ -56,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;

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.841",
+  "version": "0.0.842",
   "main": "./nuxt.config.ts",
   "types": "./types/index.d.ts",
   "scripts": {