[clousky2020]

Summary

Fixes all code review issues from PR #1318:

• Fixed duplicate routing strategy guidance and recommendations translations in zh-CN.json
• Added useMemo import in playground/page.tsx for performance optimization
• Added useMemo in RequestLoggerV2.tsx for performance optimization
• Fixed hardcoded string localization in ProxyConfigModal.tsx
• Replaced dangerouslySetInnerHTML with t.rich in OAuthModal.tsx and PricingModal.tsx
• Added missing translation namespaces to en.json

Files changed

• src/app/(dashboard)/dashboard/combos/page.tsx
• src/app/(dashboard)/dashboard/playground/page.tsx
• src/i18n/messages/en.json
• src/i18n/messages/zh-CN.json
• src/shared/components/OAuthModal.tsx
• src/shared/components/PricingModal.tsx
• src/shared/components/ProxyConfigModal.tsx
• src/shared/components/RequestLoggerV2.tsx

Test plan

• Local test: Playground page loads correctly
• Local test: Combos page loads correctly
• Local test: Logs page loads correctly
• ESLint and Prettier checks pass

🤖 Generated with Claude Code

[gemini-code-assist]

Code Review

This pull request updates strategy guidance and recommendations, refactors several components to use useMemo for performance optimization, and improves translation handling by replacing dangerouslySetInnerHTML with t.rich. It also adds a significant number of translation keys for various dashboard features. Feedback is provided regarding the RequestLoggerV2 component, where the cacheSource and tps columns were omitted from the columns array, and their corresponding translation keys are missing from the English locale file.

[gemini-code-assist]

The columns array is missing the cacheSource and tps columns which were present in the original COLUMNS constant. This causes these columns to be hidden by default (as visibleColumns is initialized from this array) and prevents them from appearing in the column visibility toggle list, resulting in a regression of UI functionality.

Suggested change

	const columns = useMemo(
	() => [
	{ key: "status", label: t("columns.status") },
	{ key: "model", label: t("columns.model") },
	{ key: "requestedModel", label: t("columns.requested") },
	{ key: "provider", label: t("columns.provider") },
	{ key: "protocol", label: t("columns.protocol") },
	{ key: "account", label: t("columns.account") },
	{ key: "apiKey", label: t("columns.apiKey") },
	{ key: "combo", label: t("columns.combo") },
	{ key: "tokens", label: t("columns.tokens") },
	{ key: "duration", label: t("columns.duration") },
	{ key: "time", label: t("columns.time") },
	],
	[t]
	);
	const columns = useMemo(
	() => [
	{ key: "status", label: t("columns.status") },
	{ key: "cacheSource", label: t("columns.cacheSource") },
	{ key: "model", label: t("columns.model") },
	{ key: "requestedModel", label: t("columns.requested") },
	{ key: "provider", label: t("columns.provider") },
	{ key: "protocol", label: t("columns.protocol") },
	{ key: "account", label: t("columns.account") },
	{ key: "apiKey", label: t("columns.apiKey") },
	{ key: "combo", label: t("columns.combo") },
	{ key: "tokens", label: t("columns.tokens") },
	{ key: "tps", label: t("columns.tps") },
	{ key: "duration", label: t("columns.duration") },
	{ key: "time", label: t("columns.time") },
	],
	[t]
	);

[gemini-code-assist]

Missing translation keys for cacheSource and tps columns in the requestLogger namespace. These are required to support the missing columns in the Request Logger UI.

Suggested change

	"columns": {
	"status": "Status",
	"model": "Model",
	"requested": "Requested",
	"provider": "Provider",
	"protocol": "Request Protocol",
	"account": "Account",
	"apiKey": "API Key",
	"combo": "Combo",
	"tokens": "Tokens",
	"duration": "Duration",
	"time": "Time"
	},
	"columns": {
	"status": "Status",
	"cacheSource": "Cache Source",
	"model": "Model",
	"requested": "Requested",
	"provider": "Provider",
	"protocol": "Request Protocol",
	"account": "Account",
	"apiKey": "API Key",
	"combo": "Combo",
	"tokens": "Tokens",
	"tps": "TPS",
	"duration": "Duration",
	"time": "Time"
	},