Skip to content

Conversation

@amrit110
Copy link
Member

This pull request introduces a new "admin" command suite to the aieng-platform-onboard CLI, enabling administrators to manage bootcamp participants and teams via CSV files. The changes include new CLI entry points, supporting utilities, documentation updates, and unit tests for the admin commands. Additionally, there are minor improvements and type annotation updates for Firestore imports throughout the codebase.

Major features and changes:

Admin CLI Functionality:

  • Added a new onboard admin command group with a setup-participants subcommand, allowing admins to bulk import and validate participants and teams from a CSV file. This includes support for a --dry-run mode for validation without making changes. (src/aieng_platform_onboard/admin/cli.py, src/aieng_platform_onboard/admin/__init__.py, src/aieng_platform_onboard/cli.py, [1] [2] [3]
  • Introduced utility functions for Firestore access, input validation, and formatting, used by admin commands. (src/aieng_platform_onboard/admin/utils.py, src/aieng_platform_onboard/admin/utils.pyR1-R299)

Documentation:

  • Updated docs/cli.md to document the new admin commands, their usage, CSV format requirements, validation rules, and examples. (docs/cli.md, [1] [2] [3] [4] [5]

Testing:

  • Added unit tests for the admin CLI, covering command parsing, help output, argument validation, and dry-run functionality. (tests/admin/test_cli.py, tests/admin/__init__.py, [1] [2]

Codebase Quality:

  • Standardized Firestore imports across the codebase with # type: ignore[attr-defined] for better type-checking compatibility. (scripts/admin/setup_teams.py, scripts/collect_coder_analytics.py, services/token-service/main.py, src/aieng_platform_onboard/utils.py, [1] [2] [3] [4]

These changes enable robust, scriptable admin operations for bootcamp onboarding, improve documentation for both users and admins, and enhance code maintainability and test coverage.

@amrit110 amrit110 self-assigned this Dec 12, 2025
@amrit110 amrit110 added documentation Improvements or additions to documentation enhancement New feature or request labels Dec 12, 2025
@codecov
Copy link

codecov bot commented Dec 12, 2025

Codecov Report

❌ Patch coverage is 95.45455% with 13 lines in your changes missing coverage. Please review.
✅ Project coverage is 90.89%. Comparing base (cfffbd2) to head (c67a8a2).
⚠️ Report is 16 commits behind head on main.

Files with missing lines Patch % Lines
...aieng_platform_onboard/admin/setup_participants.py 96.13% 7 Missing ⚠️
src/aieng_platform_onboard/admin/cli.py 81.25% 3 Missing ⚠️
src/aieng_platform_onboard/cli.py 25.00% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #50      +/-   ##
==========================================
+ Coverage   88.09%   90.89%   +2.80%     
==========================================
  Files           3        7       +4     
  Lines         462      747     +285     
==========================================
+ Hits          407      679     +272     
- Misses         55       68      +13     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@amrit110 amrit110 merged commit 017cca6 into main Dec 12, 2025
6 checks passed
@amrit110 amrit110 deleted the add_admin_module branch December 12, 2025 20:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants