feat(email): add Kafka-backed Mailgun pipeline for signup and forgot-…

[aniebietafia]

…password

Summary by CodeRabbit

• Chores
  • Internal code organization and build process improvements.

[coderabbitai]

📝 Walkthrough

Walkthrough

Import paths for user CRUD functions are reorganized from app.crud.user to app.crud.user.user across two modules, and outdated mypy error output is removed from version control.

Changes

Cohort / File(s)	Summary
Import Path Reorganization app/api/v1/endpoints/auth.py, app/crud/__init__.py	Updated import paths for create_user and get_user_by_email functions from app.crud.user to app.crud.user.user. Signatures and call sites remain unchanged.
Mypy Output Cleanup mypy_output.txt	Removed mypy error report content for aiokafka import-untyped warnings (4 lines removed). No code changes.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 Files hop to their new homes with grace,
Import paths find their rightful place,
Old errors swept, the lint is clear,
Organization brings good cheer! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Title check	⚠️ Warning	The PR title claims to add a Kafka-backed Mailgun email pipeline, but the actual changes only involve updating import paths for existing CRUD functions and removing mypy error output.	Update the title to accurately reflect the changes, such as 'refactor(imports): update CRUD import paths in auth and crud modules' or remove this PR if the main feature work is incomplete.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch feat/mailgun-email-service

📝 Coding Plan

• Generate coding plan for human review comments

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

🧹 Nitpick comments (1)

app/api/v1/endpoints/auth.py (1)

8-8: Prefer importing via the public CRUD API boundary.

This endpoint can import from app.crud instead of app.crud.user.user to avoid tight coupling to internal module structure.

Suggested refactor

-from app.crud.user.user import create_user, get_user_by_email
+from app.crud import create_user, get_user_by_email

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@app/api/v1/endpoints/auth.py` at line 8, The endpoint imports create_user and
get_user_by_email directly from the internal module app.crud.user.user; change
the import to use the public CRUD API (import create_user, get_user_by_email
from app.crud) to avoid tight coupling to internal structure, and if necessary
add or verify re-exports for create_user and get_user_by_email in
app.crud.__init__ (or update the package's export list) so the public import
resolves correctly; update the import line in auth.py to use the app.crud module
and run tests to confirm nothing else breaks.

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In `@app/api/v1/endpoints/auth.py`:
- Line 8: The endpoint imports create_user and get_user_by_email directly from
the internal module app.crud.user.user; change the import to use the public CRUD
API (import create_user, get_user_by_email from app.crud) to avoid tight
coupling to internal structure, and if necessary add or verify re-exports for
create_user and get_user_by_email in app.crud.__init__ (or update the package's
export list) so the public import resolves correctly; update the import line in
auth.py to use the app.crud module and run tests to confirm nothing else breaks.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: f88512f9-b762-4f7b-b8d1-a6a348b8b532

📥 Commits

Reviewing files that changed from the base of the PR and between ac1eade and b997661.

📒 Files selected for processing (8)

• app/api/v1/endpoints/auth.py
• app/crud/__init__.py
• app/crud/auth/__init__.py
• app/crud/auth/api_docs.md
• app/crud/user/__init__.py
• app/crud/user/api_docs.md
• app/crud/user/user.py
• mypy_output.txt

💤 Files with no reviewable changes (1)

• mypy_output.txt