Skip to content

Feat/GitHub auto sync #3

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
jithin-jz wants to merge 2 commits into from
Closed

Feat/GitHub auto sync #3

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
320a74c
fix: disable executor auth in CI test environment
jithin-jz Aug 9, 2026
d316736
fix: resolve eslint errors — clean FCM disable, remove unused catch vars
jithin-jz Aug 9, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
164 changes: 7 additions & 157 deletions frontend/src/hooks/useFCM.js
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,162 +1,12 @@
import { useCallback, useMemo, useEffect } from "react";
import useNotificationStore from "../stores/useNotificationStore";
import { notify } from "../services/notification";
import { notificationsAPI } from "../services/api";
import { playNotificationSound } from "../utils/playNotificationSound";

/**
* Hook to manage Firebase Cloud Messaging (FCM) notifications.
* Handles permission requests, token registration, and foreground message listening.
*
* TEMPORARILY DISABLED — Firebase FCM is turned off.
* To re-enable: restore the original implementation from git history.
* Commit before disable: 6085f69
*/
export const useFCM = (userId) => {
// ─── DISABLED: Firebase FCM temporarily disabled ───────────────────
// Re-enable by removing this early return block.
return;
// ──────────────────────────────────────────────────────────────────

const showWelcomeNotification = useCallback(async () => {
if (!("Notification" in window) || Notification.permission !== "granted")
return;

try {
const registration = await navigator.serviceWorker.ready;
registration.showNotification("Welcome to CLASHCODE! 🚀", {
body: "You're all set! Prepare for some intense coding battles and stay tuned for challenge alerts.",
icon: "/favicon.svg",
badge: "/favicon.svg",
tag: "welcome-message",
data: { url: "/home" },
});
} catch (err) {
console.error("[FCM] Failed to show welcome notification:", err);
}
}, []);

const registerFCM = useCallback(async () => {
try {
const { requestForToken } = await import("../services/firebase");
const token = await requestForToken();

if (!token) {
console.warn(
"[FCM] No token received from Firebase. Registration aborted.",
);
return;
}

const store = useNotificationStore.getState();

// Sync with backend
await notificationsAPI.registerFCMToken({
token: token,
device_id: navigator.userAgent,
});

const oldToken = store.fcmToken;
useNotificationStore.setState({ fcmToken: token });

if (token !== oldToken) {
notify.success("Push notifications active! 🔔");

// Show welcome notification if it's the first time on this device for this user
const welcomeKey = `fcm_welcome_sent_${userId}`;
if (userId && !localStorage.getItem(welcomeKey)) {
await showWelcomeNotification();
localStorage.setItem(welcomeKey, "true");
}
}
} catch (error) {
console.error("[FCM] Registration error:", error);
notify.error("Registration Failed", {
description: error.message || "Could not sync with backend",
});
}
}, [userId, showWelcomeNotification]);

const requestPermission = useCallback(async () => {
if (typeof Notification === "undefined") return "default";
try {
const permission = await Notification.requestPermission();
useNotificationStore.setState({ permission });

if (permission === "granted") {
notify.success("Permission Granted!", {
description: "Setting up real-time secure channel...",
});
await registerFCM();
} else if (permission === "denied") {
notify.error("Permission Denied", {
description: "You've blocked notifications. Please enable them in site settings.",
});
}
return permission;
} catch (error) {
console.error("Error requesting notification permission:", error);
return "default";
}
}, [registerFCM]);

const initFCM = useCallback(async () => {
if (typeof Notification === "undefined") return;

const store = useNotificationStore.getState();

if (Notification.permission === "default") {
if (!store._promptShown) {
notify.info("Enable Notifications", {
description: "Stay updated with real-time alerts.",
action: {
label: "Allow",
onClick: () => requestPermission(),
},
});
useNotificationStore.setState({ _promptShown: true });
}
} else if (Notification.permission === "granted") {
await registerFCM();
}

// Listener for foreground messages
if (store._listenerSetup) return;
useNotificationStore.setState({ _listenerSetup: true });

try {
const { onMessageListener } = await import("../services/firebase");
onMessageListener((payload) => {
const title = payload.notification?.title || payload.data?.title || "New Notification";
const body = payload.notification?.body || payload.data?.body || "New alert received.";

const currentStore = useNotificationStore.getState();
currentStore._showToast(title, body);
currentStore.scheduleRealtimeSync();
});
} catch (error) {
console.error("Error setting up foreground message listener:", error);
}
}, [registerFCM, requestPermission]);

useEffect(() => {
if (userId) {
initFCM();
}
}, [userId, initFCM]);

// Listen for messages from service worker (background notification clicks)
useEffect(() => {
if (!("serviceWorker" in navigator)) return;

const onSwMessage = (event) => {
if (event.data?.type === "PLAY_NOTIFICATION_SOUND") {
playNotificationSound();
}
};

navigator.serviceWorker.addEventListener("message", onSwMessage);
return () => navigator.serviceWorker.removeEventListener("message", onSwMessage);
}, []);

return useMemo(
() => ({ initFCM, requestPermission, registerFCM }),
[initFCM, requestPermission, registerFCM]
);
// eslint-disable-next-line no-unused-vars
export const useFCM = (userId) => {
// No-op while FCM is disabled
};
10 changes: 5 additions & 5 deletions frontend/src/hooks/useGitHubSync.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ export function useGitHubSync() {
setConnected(data.connected);
setConnection(data.connection);
setError(null);
} catch (err) {
} catch {
setError("Failed to fetch GitHub connection status.");
} finally {
setIsLoading(false);
Expand All @@ -45,7 +45,7 @@ export function useGitHubSync() {
const { data } = await githubSyncService.getConnectUrl();
// Redirect user to GitHub
window.location.href = data.url;
} catch (err) {
} catch {
setError("Failed to initiate GitHub connection.");
}
}, []);
Expand Down Expand Up @@ -75,7 +75,7 @@ export function useGitHubSync() {
setConnected(false);
setConnection(null);
setHistory([]);
} catch (err) {
} catch {
setError("Failed to disconnect GitHub.");
}
}, []);
Expand All @@ -88,7 +88,7 @@ export function useGitHubSync() {
is_enabled: !connection.is_enabled,
});
setConnection(data.connection);
} catch (err) {
} catch {
setError("Failed to update sync setting.");
}
}, [connection]);
Expand All @@ -112,7 +112,7 @@ export function useGitHubSync() {
await githubSyncService.retryPush(logId);
// Refresh history after retry
await fetchHistory();
} catch (err) {
} catch {
setError("Failed to retry push.");
}
}, [fetchHistory]);
Expand Down
11 changes: 11 additions & 0 deletions services/executor/conftest.py
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,11 @@
"""
Test configuration for executor service.
Forces auth to be disabled during test runs regardless of CI environment.
"""

import os

# Force-disable auth for tests (overrides CI env vars)
os.environ["INTERNAL_API_KEY"] = ""
os.environ["INTERNAL_SIGNING_SECRET"] = ""
os.environ["INTERNAL_REQUIRE_SIGNATURE"] = "false"
Loading