fix(og): resolve OpenGraph query parameter desynchronization

Author: KRUSHAL2956

lib/validations.test.ts

@@ -51,6 +51,15 @@ describe('githubParamsSchema', () => {
     }
   });
 });
+describe('streakParamsSchema user validation', () => {
+  it('should pass when user is valid', () => {
+    const result = streakParamsSchema.safeParse({
+      user: 'octocat',
+    });
+
+    expect(result.success).toBe(true);
+  });
+});
 
 describe('streakParamsSchema', () => {
   it('should fail when user is missing', () => {
@@ -402,4 +411,24 @@ describe('ogParamsSchema', () => {
       expect(result.data.user).toBe('unknown');
     }
   });
+
+  it('should parse "user" parameter successfully', () => {
+    const result = ogParamsSchema.parse({ user: 'octocat' });
+    expect(result.user).toBe('octocat');
+  });
+
+  it('should parse "username" parameter successfully and fallback to user key', () => {
+    const result = ogParamsSchema.parse({ username: 'octocat' });
+    expect(result.user).toBe('octocat');
+  });
+
+  it('should prioritize "user" over "username" when both are provided', () => {
+    const result = ogParamsSchema.parse({ user: 'octocat', username: 'ignored' });
+    expect(result.user).toBe('octocat');
+  });
+
+  it('should fall back to "unknown" when neither are provided', () => {
+    const result = ogParamsSchema.parse({});
+    expect(result.user).toBe('unknown');
+  });
 });

lib/validations.ts

@@ -152,39 +152,48 @@ export const githubParamsSchema = z.object({
     .transform((val) => val === 'true'),
 });
 
-export const ogParamsSchema = z.object({
-  user: z
-    .string()
-    .trim()
-    .optional()
-    .transform((val) => (val === '' ? undefined : val))
-    .default('unknown'),
-  theme: z
-    .string()
-    .trim()
-    .optional()
-    .transform((val) => (val === '' ? undefined : val))
-    .transform((val) => (val && Object.hasOwn(themes, val) ? val : 'dark'))
-    .default('dark'),
-  bg: z
-    .string()
-    .trim()
-    .optional()
-    .transform((val) => (val === '' ? undefined : val))
-    .transform((val) => (val && isValidHex(val) ? sanitizeHexColor(val, '000000') : undefined)),
-  text: z
-    .string()
-    .trim()
-    .optional()
-    .transform((val) => (val === '' ? undefined : val))
-    .transform((val) => (val && isValidHex(val) ? sanitizeHexColor(val, '000000') : undefined)),
-  accent: z
-    .string()
-    .trim()
-    .optional()
-    .transform((val) => (val === '' ? undefined : val))
-    .transform((val) => (val && isValidHex(val) ? sanitizeHexColor(val, '000000') : undefined)),
-});
+export const ogParamsSchema = z
+  .object({
+    user: z
+      .string()
+      .trim()
+      .optional()
+      .transform((val) => (val === '' ? undefined : val)),
+    username: z
+      .string()
+      .trim()
+      .optional()
+      .transform((val) => (val === '' ? undefined : val)),
+    theme: z
+      .string()
+      .trim()
+      .optional()
+      .transform((val) => (val === '' ? undefined : val))
+      .transform((val) => (val && Object.hasOwn(themes, val) ? val : 'dark'))
+      .default('dark'),
+    bg: z
+      .string()
+      .trim()
+      .optional()
+      .transform((val) => (val === '' ? undefined : val))
+      .transform((val) => (val && isValidHex(val) ? sanitizeHexColor(val, '000000') : undefined)),
+    text: z
+      .string()
+      .trim()
+      .optional()
+      .transform((val) => (val === '' ? undefined : val))
+      .transform((val) => (val && isValidHex(val) ? sanitizeHexColor(val, '000000') : undefined)),
+    accent: z
+      .string()
+      .trim()
+      .optional()
+      .transform((val) => (val === '' ? undefined : val))
+      .transform((val) => (val && isValidHex(val) ? sanitizeHexColor(val, '000000') : undefined)),
+  })
+  .transform((data) => ({
+    ...data,
+    user: data.user || data.username || 'unknown',
+  }));
 
 export const statsParamsSchema = z.object({
   user: z.string({ error: 'Missing user parameter' }).min(1, { message: 'Missing user parameter' }),