Skip to content

Backend Cleanup Log and Refactoring Summary - v0.0.3 #62

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.

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

Open
ebinzack15 opened this issue Apr 11, 2025 · 2 comments
Open

Backend Cleanup Log and Refactoring Summary - v0.0.3 #62

ebinzack15 opened this issue Apr 11, 2025 · 2 comments

Comments

@ebinzack15
Copy link
Collaborator

ebinzack15 commented Apr 11, 2025

Branch: cleanup/backend-unused (https://github.com/Zackriya-Solutions/meeting-minutes/tree/cleanup/backend-unused)

This issue documents the changes made focused on removing unused code from v0.0.3.

The primary goal was to remove code paths and dependencies that are no longer triggered by active API endpoints.

Cleanup Summary Table:

File Status Modifications Done TODO Deferred
backend/app/main.py Done - Removed summarization endpoints (/upload-transcript, /start-summarization).
- Removed related helper functions & SummaryProcessor logic.
- Removed agent tools (@processor.agent.tool funcs).
- Removed ChromaDB init & unused imports.
- Monitor for conflicts during merge of new feature branches. -
backend/app/transcript_processor.py Done - Renamed file (Process_transcrip.py -> transcript_processor.py).
- Removed all unused code (classes, funcs, etc.).
- Simplified process_transcript (removed path param, ChromaDB calls).
- Removed unused imports & ChromaDB dependency.
- -
backend/app/db.py On Hold - Commented out get_process and cleanup_old_processes methods. - Re-evaluate necessity of commented functions after feature merge. - Full cleanup deferred pending feature update merge to avoid conflicts.
backend/requirements.txt Done - Added specific versions to dependencies. - -
backend/examples/run_summary_workflow.py Added - New example script created to test the simplified backend API workflow. - -

Potential Conflicts:

  • Significant refactoring occurred in backend/app/main.py. Merging upcoming feature branches that also modify this file will likely require careful conflict resolution.
@ebinzack15
Copy link
Collaborator Author

@sujithatzackriya - Requesting your review on this cleanup(code on cleanup/backend-unused and the summary in the issue description).
Please add any further cleanup suggestions related to this work if any. Thanks!

@ebinzack15 ebinzack15 changed the title Backend Cleanup Log and Refactoring Summary Backend Cleanup Log and Refactoring Summary - v0.0.3 Apr 11, 2025
@sujithatzackriya
Copy link
Collaborator

@ebinzack15 Thank you for cleaning up the code.

After @athulchandroth's changes are done, we shall merge the cleanup branch with the ui changes and see if there are any merge issues. Will review the code during that time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants