[FIX] More detailed description about phases#2
Merged
karan-bosamiya-simform merged 1 commit intoJul 2, 2026
Conversation
There was a problem hiding this comment.
Pull request overview
This PR reorganizes and expands the hackathon team plan to provide clearer phase-by-phase execution guidance, with explicit per-person ownership and integration checkpoints.
Changes:
- Restructures the document into a numbered flow (Roster → Squad Structure → How to Read → Phases 1–3 → Pairing points → Trainee review gate → Rehearsal checklist).
- Replaces module-level phase bullets with Phase → Squad → Person assignments, including “done when” criteria and integration dependencies.
- Adds a concrete demo rehearsal checklist and highlights critical cross-squad pairing points.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| ### Phase 2 — Core Feature Build (Hours 3–14) | ||
| - **Ishit Desai (SE Python, lead)** — Scaffolds `app/services/knowledge_hub/` folder structure (§12.2): `connectors/base.py` with the `SourceConnector` protocol (§6.2) defined, empty chunker files stubbed out. Reviews Squad A's stub `retrieve()` signature as soon as it lands to confirm it supports the `chunk_types`/`repository_id` filters Knowledge Hub needs. **Done when**: `SourceConnector` protocol is defined and committed; folder skeleton exists for connectors and chunkers. | ||
| - **Aman Rajput (Trainee Data Engineering)** — Paired with Ishit. Starts on `GitHubConnector` — `list_files()` and `fetch_content()` using `PyGithub` against a real test repo, no chunking yet. **Done when**: can authenticate to a GitHub repo and list its files programmatically, verified with a quick script. | ||
| - **Arjun Mulagapaka (Trainee Python)** — Reads Squad A's published stub contract and Section 5's `knowledge_chunks` schema (§5.3.6). Drafts the SQLAlchemy model for `knowledge_chunks`, `repositories`, `ingestion_jobs` (not yet migrated — just the model classes). **Done when**: model files exist matching §5.3.6's column list, ready for Karan to fold into the migration sequence. |
|
|
||
| --- | ||
| - **Parth Shah (SE Python, lead)** — Scaffolds `app/schemas/meetings.py`, `risks.py`. Stubs `POST /meetings` (accepts transcript text or audio upload, returns `202`, no summarization yet). **Done when**: endpoint accepts input and returns a placeholder `meeting_id` + `status: processing`. | ||
| - **Smit Thakkar (Trainee Python)** — Drafts `meetings`, `meeting_decisions`, `action_items`, `risk_register_snapshots`, `risks` SQLAlchemy models per §5.3.4/§5.3.5. **Done when**: models exist and match the column lists in Section 5, ready for the migration sequence. |
| - **M5 — Frontend Experience**: risk register UI, knowledge hub management UI (only if the golden path is already solid — see §6), final polish on golden-path screens *(Khushal Batham; Bhargav Valani)* | ||
| - **M6 — DevOps & Quality**: end-to-end smoke tests across all six modules together, freeze the deploy, drive **two full demo dress rehearsals**, final regression pass *(Chaitanya Vekaria; Shubham Nayak — leads rehearsal checklist; Chandresh Thakkar)* | ||
| - **Chaitanya Vekaria (Trainee DevOps)** — Writes `docker-compose.yml` (Postgres, Redis, backend skeleton, per §14.3) and a `.github/workflows/ci.yml` running `ruff` + `pytest` on every PR. Paired with Shubham for review since this is foundational infrastructure everyone else depends on. **Done when**: `docker-compose up` brings up Postgres + Redis + a health-check-passing backend container; a test PR shows CI running. | ||
| - **Shubham Nayak (QA Engineer, lead)** — Reviews Chaitanya's CI/compose setup. Drafts the master test scenario list from Section 9's endpoint error tables (happy path / 401 / 403 / 422 / 503 per endpoint) as a shared checklist document every squad can reference when writing their own tests. **Done when**: test scenario checklist is shared with all squad leads. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.