@quark-fw/plugin-react 0.1.25 → 0.1.26

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@quark-fw/plugin-react",
-	"version": "0.1.25",
+	"version": "0.1.26",
 	"main": "src/index.ts",
 	"types": "src/index.ts",
 	"type": "module",
@@ -22,8 +22,8 @@
 		"access": "public"
 	},
 	"dependencies": {
-		"@quark-fw/atom": "0.1.25",
-		"@quark-fw/plugin": "0.1.25",
+		"@quark-fw/atom": "0.1.26",
+		"@quark-fw/plugin": "0.1.26",
 		"@trpc/client": "~11.18.0",
 		"@types/react-dom": "^19.2.3",
 		"express": "^5.2.1",
@@ -36,8 +36,8 @@
 		"@mantine/dates": "^9.0.0",
 		"@mantine/modals": "^9.0.0",
 		"@mantine/notifications": "^9.0.0",
-		"@quark-fw/core": "0.1.25",
-		"@quark-fw/store": "0.1.25",
+		"@quark-fw/core": "0.1.26",
+		"@quark-fw/store": "0.1.26",
 		"@vitejs/plugin-react": "^5.1.0",
 		"react": "^19.2.4",
 		"react-dom": "^19.2.4",

package/src/i18n/config.test.ts

@@ -0,0 +1,16 @@
+import { expect, test } from "vitest";
+import { buildLocalePath } from "./config.js";
+
+// Корень локали — без завершающего слэша: так его рендерит react-router в
+// ссылках (href="/ru") и так его перечисляет sitemap приложения. Форма со
+// слэшем превращалась у Google в дубликат с другим canonical.
+test("buildLocalePath: корень локали без слэша", () => {
+	expect(buildLocalePath("ru", "/")).toBe("/ru");
+	expect(buildLocalePath("ru", "")).toBe("/ru");
+});
+
+test("buildLocalePath: вложенные пути с префиксом и без", () => {
+	expect(buildLocalePath("en", "/tbilisi")).toBe("/en/tbilisi");
+	expect(buildLocalePath("en", "tbilisi")).toBe("/en/tbilisi");
+	expect(buildLocalePath("ka", "/tbilisi?page=2")).toBe("/ka/tbilisi?page=2");
+});

package/src/i18n/config.ts

@@ -79,6 +79,9 @@ export function parseLocalePath(
 }
 
 export function buildLocalePath(locale: string, strippedPath: string): string {
+	// корень локали — без завершающего слэша: так его рендерит react-router
+	// в ссылках и перечисляет sitemap; /ru/ Google считал дубликатом /ru
+	if (strippedPath === "/" || strippedPath === "") return `/${locale}`;
 	return `/${locale}${strippedPath.startsWith("/") ? strippedPath : `/${strippedPath}`}`;
 }
 

package/src/ssr/getRenderMiddleware.ts

@@ -24,6 +24,7 @@ import { createRendererLoader } from "./createRendererLoader.js";
 import { createPreloadTagWriter } from "./createPreloadTagWriter.js";
 import { createRequestTrpcClient } from "./createRequestTrpcClient.js";
 import { createStreamingStore } from "./createStreamingStore.js";
+import { stripTrailingSlash } from "../utils/stripTrailingSlash.js";
 
 export type GetRenderMiddlewareArgs = {
 	/** dev: модули грузятся через vite; в prod — из собранного бандла */
@@ -43,6 +44,21 @@ export default function getRenderMiddleware({ vite }: GetRenderMiddlewareArgs) {
 			const i18nConfig = (rendererModule as { i18nConfig?: I18nConfig })
 				.i18nConfig;
 			const urlWithoutSearch = removeSearchParamsFromUrl(req.originalUrl);
+
+			// одна каноническая форма пути — без завершающего слэша: /ru/ и
+			// /ru/tbilisi/ отдавали 200 со своим canonical и дублировали страницы
+			const normalized = stripTrailingSlash(urlWithoutSearch);
+			if (normalized) {
+				const searchIndex = req.originalUrl.indexOf("?");
+				res.redirect(
+					301,
+					normalized +
+						(searchIndex === -1
+							? ""
+							: req.originalUrl.slice(searchIndex)),
+				);
+				return;
+			}
 			const { locale, strippedPathname, basename, redirect } =
 				parseLocalePath(urlWithoutSearch, i18nConfig);
 

package/src/utils/stripTrailingSlash.test.ts

@@ -0,0 +1,14 @@
+import { expect, test } from "vitest";
+import { stripTrailingSlash } from "./stripTrailingSlash.js";
+
+test("stripTrailingSlash: путь без слэша и корень не меняются", () => {
+	expect(stripTrailingSlash("/")).toBeNull();
+	expect(stripTrailingSlash("/ru")).toBeNull();
+	expect(stripTrailingSlash("/ru/tbilisi")).toBeNull();
+});
+
+test("stripTrailingSlash: завершающие слэши срезаются", () => {
+	expect(stripTrailingSlash("/ru/")).toBe("/ru");
+	expect(stripTrailingSlash("/ru/tbilisi/")).toBe("/ru/tbilisi");
+	expect(stripTrailingSlash("/ru/tbilisi//")).toBe("/ru/tbilisi");
+});

package/src/utils/stripTrailingSlash.ts

@@ -0,0 +1,10 @@
+/**
+ * Путь без завершающих слэшей, если они были; null — если менять нечего.
+ * Корень "/" остаётся как есть. Нужен SSR-мидлваре: /ru/ и /ru/tbilisi/
+ * рендерились как отдельные страницы со своим canonical, и Google считал
+ * их дубликатами.
+ */
+export function stripTrailingSlash(pathname: string): string | null {
+	if (pathname.length <= 1 || !pathname.endsWith("/")) return null;
+	return pathname.replace(/\/+$/, "") || "/";
+}