Skip to content

Remove ML model artifacts from git and add FCM push notifications#1

Draft
Copilot wants to merge 4 commits intomainfrom
copilot/clean-actions-artifacts
Draft

Remove ML model artifacts from git and add FCM push notifications#1
Copilot wants to merge 4 commits intomainfrom
copilot/clean-actions-artifacts

Conversation

Copy link
Copy Markdown

Copilot AI commented Dec 20, 2025

The repository was bloated with 100MB+ of ML model files, and the system lacked real-time alerting for malicious requests.

Changes

Artifact Management

  • Removed ML models (.h5, .tflite) and tokenizers (.tokens) from git tracking
  • Added GitHub Actions workflows:
    • build-models.yml: Builds models on dataset/script changes, stores as artifacts (30-day retention)
    • clean-artifacts.yml: Daily cleanup keeping only latest artifacts
  • Updated .gitignore to prevent future commits of build artifacts

Push Notifications

  • Implemented FCM integration via notification_service.py (async, aiohttp)
  • Integrated into both Dev and Prod server_socket_main.py:
    if final_result == "MALICIOUS":
        asyncio.create_task(
            notification_service.notify_malicious_request(
                str(addr), uri, uri_final_result, body_final_result
            )
        )
  • Configuration via FCM_SERVER_KEY env var and device_tokens.json
  • Graceful degradation when disabled (no FCM key or device tokens)

Documentation

  • SETUP_GUIDE.md: Firebase setup, device token configuration
  • QUICK_REFERENCE.md: Common operations
  • .env.example and device_tokens.json.example: Configuration templates

Testing

Unit tests in test_notification_service.py validate service initialization, notification sending, and malicious request filtering.

Original prompt

I want you to clean the actions artefacts once the file is added to the repository.

Also I didn't received a push notification, I want to receive a push notification on the app


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI and others added 3 commits December 20, 2025 21:26
Co-authored-by: DevEnVrac <75941535+DevEnVrac@users.noreply.github.com>
Co-authored-by: DevEnVrac <75941535+DevEnVrac@users.noreply.github.com>
Co-authored-by: DevEnVrac <75941535+DevEnVrac@users.noreply.github.com>
Copilot AI changed the title [WIP] Clean actions artifacts after file addition Remove ML model artifacts from git and add FCM push notifications Dec 20, 2025
Copilot AI requested a review from DevEnVrac December 20, 2025 21:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants