This pull request makes several significant changes to the API structure and database migration scripts. #83
Merged
goldlabelapps merged 4 commits intomasterfrom Apr 12, 2026
Merged
Conversation
Delete the prospects flag API and its associated SQL migration/util scripts. Removed app/api/prospects/flag.py and numerous files under app/api/prospects/sql, and updated app/api/prospects/__init__.py and app/api/routes.py to stop importing and registering the flag router. Cleans up deprecated flag functionality and related migration helpers.
Update __version__ in app/__init__.py from 2.2.1 to 2.2.2 to reflect a patch release.
Move LLM API implementation and SQL helpers from app/api/llm to app/api/prompt, renaming llm.py to prompt.py and updating function names, route paths (/llm -> /prompt) and docstrings accordingly. Replace deleted app/api/llm/__init__.py with app/api/prompt/__init__.py that exposes the router, and update app/api/routes.py to import and include the new prompt router (remove old llm/gemini imports). SQL migration and helper scripts under llm/sql were relocated to prompt/sql with no functional changes.
Refactor project layout: move notify-related modules from app/api/resend to app/utils/notify (resend.py, send_email.py, and package init), and relocate the health endpoint from app/api/health.py to app/utils/health.py. Update app/api/root.py and app/api/routes.py to reflect the new import paths. Add LinkedInConnections.csv under app/api/prospects. Rename tests/test_llm.py to tests/test_prompt.py and apply related test updates (tests/test_routes.py). These changes separate API routes from utility implementations and add prospect data.
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
1 participant
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.
The most notable updates include refactoring the LLM (Large Language Model) API endpoints to use a new "prompt" naming convention, removing the prospects flagging routes, and cleaning up old database migration scripts.
API refactoring and cleanup:
llmtoprompt, updating both the route paths (e.g.,/llm→/prompt) and the associated import structure. (app/api/llm/__init__.py[1]app/api/prompt/__init__.py[2]app/api/prompt/prompt.py[3] [4]app/api/llm/llm.py) have been removed and replaced byapp/api/prompt/prompt.py, reflecting the new naming and structure. (app/api/llm/__init__.py[1]app/api/prompt/prompt.py[2] [3]Prospect flagging routes removal:
flag.pyfile and its import in the prospects router. (app/api/prospects/flag.py[1]app/api/prospects/__init__.py[2]Database migration script cleanup:
namecolumn, and dropping thecleaned,do_not_call, andemail_confidencecolumns from theprospectstable. (app/api/prospects/sql/alter_add_name_and_populate.sql[1]app/api/prospects/sql/alter_drop_cleaned.sql[2]app/api/prospects/sql/alter_drop_do_not_call.sql[3]app/api/prospects/sql/alter_drop_email_confidence.sql[4]Other:
2.2.1to2.2.2to reflect these changes. (app/__init__.pyapp/init.pyL4-R4)