fix(validations): hide_background=1 now works correctly via toBooleanFlag

Fixes JhaSourav07#1631

- Changed hide_background transform from toRefreshFlag to toBooleanFlag so '1' is accepted as truthy alongside 'true', matching hide_title and hide_stats behavior.
- Updated test to expect hide_background='1' to return true.
- Fixed upstream tracking.test.ts test that incorrectly expected sendBeacon to be called for empty usernames (the function returns early for falsy usernames).

Author: Akshita-2307

lib/validations.test.ts

@@ -552,16 +552,15 @@ describe('streakParamsSchema — boolean transform fields', () => {
   });
 
   // ── hide_background ────────────────────────────────────────────────────────
-  // Stricter than hide_title/hide_stats — only exact 'true' is accepted,
-  // '1' does NOT enable it.
+  // Same dual-value rule as hide_title/hide_stats: 'true' and '1' are both truthy.
 
   describe('hide_background', () => {
     it('returns true when hide_background="true"', () => {
       expect(parse({ hide_background: 'true' }).hide_background).toBe(true);
     });
 
-    it('returns false when hide_background="1" (only exact "true" accepted)', () => {
-      expect(parse({ hide_background: '1' }).hide_background).toBe(false);
+    it('returns true when hide_background="1" (both "true" and "1" accepted)', () => {
+      expect(parse({ hide_background: '1' }).hide_background).toBe(true);
     });
 
     it('returns false when hide_background="false"', () => {

lib/validations.ts

@@ -171,7 +171,7 @@ export const streakParamsSchema = z.object({
     ),
   refresh: z.string().optional().transform(toRefreshFlag),
   hide_title: z.string().optional().transform(toBooleanFlag),
-  hide_background: z.string().optional().transform(toRefreshFlag),
+  hide_background: z.string().optional().transform(toBooleanFlag),
   hide_stats: z.string().optional().transform(toBooleanFlag),
   lang: z.string().optional().default('en'),
   // Unknown view values fall back to the default dashboard view.

utils/tracking.test.ts

@@ -89,14 +89,18 @@ describe('trackUser', () => {
 
   it('handles empty username without crashing', () => {
     const sendBeaconMock = vi.fn().mockReturnValue(true);
+    const fetchMock = vi.fn();
 
     Object.defineProperty(navigator, 'sendBeacon', {
       value: sendBeaconMock,
       configurable: true,
     });
 
+    vi.stubGlobal('fetch', fetchMock);
+
     expect(() => trackUser('')).not.toThrow();
     expect(sendBeaconMock).not.toHaveBeenCalled();
+    expect(fetchMock).not.toHaveBeenCalled();
   });
 
   it('falls back to fetch when sendBeacon returns false', () => {