Skip to content

refactor: isolate Check Project Compliance featureRemoved all extraneous UI...

Bikkumalla Sai Krishna requested to merge seperate_project_compilance into dev

Refactor: Isolate and specialize repository for "Check Project Compliance"

Description

This Merge Request streamlines the application into a highly focused, single-purpose tool strictly dedicated to repository compliance checking. We have safely decoupled and purged all extraneous features (analytics, leaderboards, user tracking), resulting in a profoundly cleaner, lighter, and more maintainable codebase (removing over 9,700 lines of unused code).

Features Removed

  • User Profile Overview: Removed UI, backing logic, and tests.
  • Team Leaderboard & Weekly Performance: Removed UI, services, aggregators, and logic tests.
  • Batch Analytics: Purged the batch interface, APIs, file readers, export interfaces, and legacy controllers.
  • Role-Based Access Control (RBAC): Excised the SQLite-based roster authorization database (temp_roster.db, database.py, roster_service.py), allowing any authenticated GitLab user seamless access to the core tool.

Architectural Changes

  • Routing Simplification: app.py and src/gitlab_compliance_checker/ui/main.py have been deeply refactored so they securely bypass the old mode selector and drop directly into render_compliance_mode(client).
  • Test Suite Audit: Purgated the tests/ directory of all 32 obsolete module tests. pytest cleanly passes on all 127 core compliance checks remaining.
  • Documentation: Overhauled README.md to reflect the newly streamlined application boundaries, updated structure, and modern setup commands (uv sync).

Impact

  • Lines Removed: ~9,700+
  • Security Check: DB teardown explicitly removed role constraints. Local DB credentials/logic removed from .streamlit/secrets.toml usage.
  • Test Coverage: 127/127 native compliance tests pass perfectly.

Verification Checklist

  • Application launches directly into Compliance Mode (uv run streamlit run app.py)
  • All residual unused dependencies & files were cleanly scrubbed
  • Local test suite passes successfully
  • Linter/Formatter (ruff) passes checking updated paths
Edited by Bikkumalla Sai Krishna

Merge request reports

Loading