Remove push-up challenge tests from test suite

- Remove Push-Up Challenge test block from api-client.test.ts
- Remove installChallengeStatusLineFeature and removeChallengeStatusLineFeature tests from claude-settings-manager.test.ts
- Remove PUSHUP_ONLY state test from detector.test.ts

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>

Author: dannyshmueli

tests/unit/lib/api-client.test.ts

@@ -388,41 +388,6 @@ describe('API Client Module', () => {
     });
   });
 
-  describe('Push-Up Challenge', () => {
-    it('should sync push-up challenge data', async () => {
-      const pushUpData = {
-        date: '2024-01-15',
-        count: 50,
-        sets: [10, 10, 10, 10, 10],
-      };
-
-      mockAxiosInstance.post.mockResolvedValue({
-        data: { success: true, totalToday: 50 },
-      });
-
-      const result = await apiClient.syncPushUpChallenge(pushUpData);
-
-      expect(result).toEqual({ success: true, totalToday: 50 });
-      expect(mockAxiosInstance.post).toHaveBeenCalledWith('/api/push-up-challenge/sync', pushUpData);
-    });
-
-    it('should fetch push-up challenge stats', async () => {
-      const mockStats = {
-        totalPushUps: 1500,
-        currentStreak: 7,
-        longestStreak: 14,
-        averagePerDay: 50,
-      };
-
-      mockAxiosInstance.get.mockResolvedValue({ data: mockStats });
-
-      const result = await apiClient.fetchPushUpChallengeStats();
-
-      expect(result).toEqual(mockStats);
-      expect(mockAxiosInstance.get).toHaveBeenCalledWith('/api/push-up-challenge/stats');
-    });
-  });
-
   describe('Upload Error Handling', () => {
     it('should propagate upload errors', async () => {
       const sessions = [testData.createSession()];

tests/unit/lib/claude-settings-manager.test.ts

@@ -423,104 +423,4 @@ describe('ClaudeSettingsManager', () => {
     });
   });
 
-  describe('installChallengeStatusLineFeature', () => {
-    it('should install challenge status line with refresh hook', async () => {
-      const mockSettings = { hooks: {} };
-      mockSettingsReader.readGlobalSettings.mockResolvedValue(mockSettings);
-
-      await manager.installChallengeStatusLineFeature();
-
-      const savedSettings = mockSettingsReader.writeGlobalSettings.mock.calls[0][0];
-      // Should have statusLine config for challenge
-      expect(savedSettings.statusLine).toBeDefined();
-      expect(savedSettings.statusLine?.command).toContain('statusline-challenge');
-
-      // Should have UserPromptSubmit hook for auto-refresh
-      expect(savedSettings.hooks?.UserPromptSubmit).toBeDefined();
-      expect(savedSettings.hooks?.UserPromptSubmit?.[0]?.hooks).toBeDefined();
-    });
-
-    it('should backup existing status line before installing challenge', async () => {
-      const mockSettings = {
-        hooks: {},
-        statusLine: {
-          type: 'command' as const,
-          command: 'existing-statusline',
-          padding: 0
-        }
-      };
-      mockSettingsReader.readGlobalSettings.mockResolvedValue(mockSettings);
-
-      await manager.installChallengeStatusLineFeature();
-
-      expect(mockConfig.saveStatusLineBackup).toHaveBeenCalledWith({
-        originalCommand: 'existing-statusline',
-        originalType: 'command',
-        originalPadding: 0,
-        backupReason: 'Replaced by vibe-log challenge status line'
-      });
-    });
-  });
-
-  describe('removeChallengeStatusLineFeature', () => {
-    it('should remove challenge status line and hooks', async () => {
-      const mockSettings = {
-        hooks: {
-          UserPromptSubmit: [{
-            hooks: [
-              { type: 'command' as const, command: 'npx vibe-log-cli refresh-push-up-challenge-statusline' }
-            ]
-          }]
-        },
-        statusLine: {
-          type: 'command' as const,
-          command: 'npx vibe-log-cli statusline-challenge',
-          padding: 0
-        }
-      };
-      mockSettingsReader.readGlobalSettings.mockResolvedValue(mockSettings);
-
-      await manager.removeChallengeStatusLineFeature();
-
-      const savedSettings = mockSettingsReader.writeGlobalSettings.mock.calls[0][0];
-      expect(savedSettings.hooks?.UserPromptSubmit).toBeUndefined();
-      expect(savedSettings.statusLine).toBeUndefined();
-    });
-
-    it('should restore backup when requested', async () => {
-      const mockSettings = {
-        hooks: {
-          UserPromptSubmit: [{
-            hooks: [
-              { type: 'command' as const, command: 'npx vibe-log-cli refresh-push-up-challenge-statusline' }
-            ]
-          }]
-        },
-        statusLine: {
-          type: 'command' as const,
-          command: 'npx vibe-log-cli statusline-challenge',
-          padding: 0
-        }
-      };
-      mockSettingsReader.readGlobalSettings.mockResolvedValue(mockSettings);
-
-      const backup = {
-        originalCommand: 'echo "restored"',
-        originalType: 'command' as const,
-        originalPadding: 2,
-        backupReason: 'test'
-      };
-      mockConfig.getStatusLineBackup.mockReturnValue(backup);
-
-      await manager.removeChallengeStatusLineFeature(true);
-
-      const savedSettings = mockSettingsReader.writeGlobalSettings.mock.calls[0][0];
-      expect(savedSettings.statusLine).toEqual({
-        type: 'command',
-        command: 'echo "restored"',
-        padding: 2
-      });
-      expect(mockConfig.clearStatusLineBackup).toHaveBeenCalled();
-    });
-  });
 });

tests/unit/lib/detector.test.ts

@@ -225,26 +225,6 @@ describe('detector', () => {
       });
     });
 
-    describe('PUSHUP_ONLY state', () => {
-      it('should detect PUSHUP_ONLY when only push-up challenge enabled', async () => {
-        // Push-up challenge requires some config to not be FIRST_TIME
-        mockGetAllConfig.mockReturnValue({ pushUpChallenge: { enabled: true } });
-        mockGetToken.mockResolvedValue(null);
-        mockGetPushUpChallengeConfig.mockReturnValue({ enabled: true });
-
-        // Explicitly mock fs to ensure no agents detected
-        const mockAccess = vi.spyOn(fs, 'access');
-        mockAccess.mockRejectedValue(new Error('Path does not exist'));
-
-        const result = await detectSetupState();
-
-        expect(result.state).toBe('PUSHUP_ONLY');
-        expect(result.hasAuth).toBe(false);
-        expect(result.hasAgents).toBe(false);
-        expect(result.hasPushUpChallenge).toBe(true);
-      });
-    });
-
     describe('PARTIAL_SETUP state', () => {
       it('should detect PARTIAL_SETUP for incomplete configurations', async () => {
         mockGetAllConfig.mockReturnValue({ someConfig: 'value' });