Christian Krakau-Louis
5e4633b481
Merge pull request #56 from christianlouis/code-health-unused-imports-models-init-5127216271700083315
...
🧹 [code health improvement] Silence unused imports in models/__init__.py
2026-03-23 17:06:05 +01:00
Christian Krakau-Louis
bfeb98dc04
Merge pull request #57 from christianlouis/testing-auth-store-improvements-16756141904230151053
...
🧪 [testing improvement description] Implement unit tests for `useAuthStore` and refine store logic.
2026-03-23 17:05:50 +01:00
Christian Krakau-Louis
9551aae47c
Merge branch 'main' into testing-auth-store-improvements-16756141904230151053
2026-03-23 17:05:43 +01:00
Christian Krakau-Louis
0e770e92bf
Merge branch 'main' into security-fix-cors-methods-15224374223607102322
2026-03-23 17:05:06 +01:00
google-labs-jules[bot]
2d232bb603
🧪 [testing improvement description] Implement unit tests for useAuthStore and fix linting.
...
🎯 **What:** This PR adds comprehensive unit tests for the `useAuthStore` Zustand store and resolves a CI linting failure by downgrading ESLint to version 9.
📊 **Coverage:** The new tests cover:
- Initial state verification.
- User state updates via `setUser`.
- Token management via `setToken`.
- Loading state toggles with `setLoading`.
- Complete logout flow, including `localStorage` cleanup.
✨ **Result:** Increased reliability of the authentication logic and restored CI health. The store implementation was also refined to properly include and handle the `token` property.
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-03-23 14:42:29 +00:00
google-labs-jules[bot]
a9c2d37336
Restrict overly permissive CORS allow_methods in backend/app/main.py.
...
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-03-23 14:36:57 +00:00
google-labs-jules[bot]
4515042acf
🧪 [testing improvement description] Implement unit tests for useAuthStore and refine store logic.
...
🎯 **What:** This PR adds comprehensive unit tests for the `useAuthStore` Zustand store in the frontend. It addresses a gap where no unit tests were present for authentication state management.
📊 **Coverage:** The new tests cover:
- Initial state verification.
- User state updates via `setUser`.
- Token management via `setToken`.
- Loading state toggles with `setLoading`.
- Complete logout flow, including `localStorage` cleanup.
✨ **Result:** Increased reliability of the authentication logic by ensuring state changes are deterministic and correctly persist/clear tokens as needed. The store implementation was also refined to properly include and handle the `token` property.
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-03-23 14:36:44 +00:00
google-labs-jules[bot]
7f04af53f2
🧹 [code health improvement] Silence unused imports in models/__init__.py
...
Updated `backend/app/models/__init__.py` to use explicit re-exports (e.g., `from app.models.database_models import User as User`) to satisfy static analysis tools and improve code health. This follows PEP 484 and silences "unused import" warnings while maintaining the public API for the models package.
🎯 **What:** The code health issue addressed is unused imports in `models/__init__.py` being flagged by static analysis tools.
💡 **Why:** This improves maintainability and silences noise in linting reports without breaking the package's public API.
✅ **Verification:** Confirmed via code review and manual inspection that models are still correctly exported and correctly used in other parts of the codebase.
✨ **Result:** The codebase is now cleaner and follows Python best practices for package exports.
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
2026-03-23 14:36:24 +00:00
copilot-swe-agent[bot]
09fb87ec7e
fix: downgrade ESLint to ^9 to fix react/display-name TypeError
...
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
Agent-Logs-Url: https://github.com/christianlouis/pop_puller_to_gmail/sessions/402f27be-f4c8-4470-8ef8-9c060d87126e
2026-03-23 14:31:51 +00:00
copilot-swe-agent[bot]
ffc086fd4c
Initial plan
2026-03-23 14:29:23 +00:00
Christian Krakau-Louis
b815df0d26
Merge pull request #54 from christianlouis/copilot/resolve-eslint-peer-dependency
...
Fix npm ci failure: sync eslint-config-next version in package.json with lock file
2026-03-23 15:10:13 +01:00
Christian Krakau-Louis
63bc8529d5
Merge pull request #53 from christianlouis/copilot/update-configuration-from-db
...
Add database-backed configuration and document Gmail API vs SMTP delivery
2026-03-23 14:48:55 +01:00
copilot-swe-agent[bot]
2d3b1065b5
Address code review feedback: improve logging, fix redundant check, clarify docs
...
- Log exceptions at warning level in ConfigService (not debug)
- Include exc_info=True for startup seed failure logging
- Remove redundant `is_secret is not None` guard
- Clarify ★ markers in README environment variables docs
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
Agent-Logs-Url: https://github.com/christianlouis/pop_puller_to_gmail/sessions/ce88d4a8-d8c2-49b3-95a1-30592105a769
2026-03-23 13:43:11 +00:00
copilot-swe-agent[bot]
71a97379a1
Update documentation for hybrid config and Gmail API vs SMTP delivery
...
- README.md: document hybrid config model, Gmail API vs SMTP comparison,
update architecture diagram, replace Postmarkapp with Apprise
- ARCHITECTURE.md: add Gmail API vs SMTP comparison table, document
ConfigService and AppSetting, add settings/gmail API endpoints
- CHANGELOG.md: add entries for database-backed config and documentation
- TODO.md: update progress (59% coverage, production readiness 30%)
- .env.example: document bootstrap vs database-managed settings
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
Agent-Logs-Url: https://github.com/christianlouis/pop_puller_to_gmail/sessions/ce88d4a8-d8c2-49b3-95a1-30592105a769
2026-03-23 13:39:52 +00:00
copilot-swe-agent[bot]
702650376e
Add database-backed configuration (AppSetting model, ConfigService, settings API)
...
- Add AppSetting model for key-value settings storage in PostgreSQL
- Create ConfigService with DB-first, env-var-fallback resolution
- Add admin-only /api/v1/settings CRUD endpoints
- Update tasks.py to use ConfigService for SMTP config
- Seed default settings on application startup
- Add 24 unit tests for ConfigService (all passing)
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
Agent-Logs-Url: https://github.com/christianlouis/pop_puller_to_gmail/sessions/ce88d4a8-d8c2-49b3-95a1-30592105a769
2026-03-23 13:35:52 +00:00
copilot-swe-agent[bot]
7bca1999da
Fix npm ci: sync eslint-config-next to 16.2.1 in package.json to match lock file
...
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
Agent-Logs-Url: https://github.com/christianlouis/pop_puller_to_gmail/sessions/2ec6aaab-a34d-4a91-a2cf-e75a49022638
2026-03-23 13:35:49 +00:00
copilot-swe-agent[bot]
6531419f72
Initial plan
2026-03-23 13:32:49 +00:00
copilot-swe-agent[bot]
91c545ffd4
Initial plan
2026-03-23 13:25:27 +00:00
Christian Krakau-Louis
f3e1afb738
Merge pull request #52 from christianlouis/copilot/merge-dependency-updates
...
Batch dependency updates, remove CodeQL, fix code quality issues
2026-03-23 14:21:34 +01:00
copilot-swe-agent[bot]
f439f887d0
fix: address multiple code quality improvements across backend and frontend
...
Backend fixes:
- Fix JWT sub claim: encode as str(user.id), decode with int() cast (python-jose requirement)
- Replace all deprecated datetime.utcnow() with datetime.now(timezone.utc)
- Replace deprecated FastAPI @app.on_event() with modern lifespan context manager
- Replace deprecated Pydantic class Config with model_config = ConfigDict(...)
- Replace deprecated Pydantic .dict() with .model_dump()
- Fix overly broad except (GmailInjectionError, Exception) → except Exception
- Remove unused GmailInjectionError import
- Fix TokenPayload schema sub field type from int to str
Frontend:
- Create frontend/src/lib/api.ts — API client module with auth, user, mail accounts, processing runs APIs
- Add !frontend/src/lib/ to .gitignore negation
Tests:
- Add 3 new JWT tests (sub string encoding, access token type, refresh token type)
- Update test_token_payload_schema for string sub claim
- All 128 tests pass
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
Agent-Logs-Url: https://github.com/christianlouis/pop_puller_to_gmail/sessions/e0b13eb0-8de7-4f02-81e4-e202cbba4608
2026-03-23 13:09:23 +00:00
Christian Krakau-Louis
a513ef3c20
Update frontend/package.json
...
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com >
2026-03-23 13:58:18 +01:00
Christian Krakau-Louis
19f089f8f6
Update .github/workflows/ci.yml
...
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com >
2026-03-23 13:52:23 +01:00
copilot-swe-agent[bot]
c4d8f392f8
chore: merge dependency updates from PRs #26-#49 and remove CodeQL checks
...
Backend (Python):
- pydantic 2.5.3 → 2.12.5
- pydantic-settings 2.1.0 → 2.13.1
- psycopg2-binary 2.9.9 → 2.9.11
- asyncpg 0.29.0 → 0.31.0
- stripe 7.11.0 → 14.4.1
- aioimaplib 1.0.1 → 2.0.1
- google-auth-httplib2 0.2.0 → 0.3.0
- celery 5.3.6 → 5.6.2
- redis 5.0.1 → 7.3.0
- tenacity 8.2.3 → 9.1.4
Frontend (npm):
- react 19.2.3 → 19.2.4
- @tanstack/react-query ^5.90.20 → ^5.95.0
- axios ^1.13.5 → ^1.13.6
- zustand ^5.0.11 → ^5.0.12
- eslint ^9 → ^10
- eslint-config-next 16.1.6 → 16.2.1
Docker:
- Python base image 3.11-slim → 3.14-slim
CI/CD:
- actions/setup-python v5 → v6
- actions/setup-node v4 → v6
- docker/setup-buildx-action v3 → v4
- codecov/codecov-action v3 → v5
- Python version 3.11 → 3.14, Node.js 18 → 20
- Removed CodeQL checks (was blocking builds)
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
Agent-Logs-Url: https://github.com/christianlouis/pop_puller_to_gmail/sessions/d8a6f888-1c85-4de6-a36d-55e9dcd7489e
2026-03-23 12:33:33 +00:00
copilot-swe-agent[bot]
5f5b4777dd
Initial plan
2026-03-23 12:27:48 +00:00
Christian Krakau-Louis
4338490fa3
Merge pull request #50 from christianlouis/dependabot/npm_and_yarn/frontend/lucide-react-0.577.0
...
chore(deps): bump lucide-react from 0.563.0 to 0.577.0 in /frontend
2026-03-23 13:26:05 +01:00
Christian Krakau-Louis
d1b8850fdc
Merge pull request #51 from christianlouis/dependabot/npm_and_yarn/frontend/react-dom-19.2.4
...
chore(deps): bump react-dom from 19.2.3 to 19.2.4 in /frontend
2026-03-23 13:25:53 +01:00
Christian Krakau-Louis
ec222832ef
Merge branch 'main' into dependabot/npm_and_yarn/frontend/react-dom-19.2.4
2026-03-23 13:25:46 +01:00
dependabot[bot]
8133197d35
chore(deps): bump react-dom from 19.2.3 to 19.2.4 in /frontend
...
Bumps [react-dom](https://github.com/facebook/react/tree/HEAD/packages/react-dom ) from 19.2.3 to 19.2.4.
- [Release notes](https://github.com/facebook/react/releases )
- [Changelog](https://github.com/facebook/react/blob/main/CHANGELOG.md )
- [Commits](https://github.com/facebook/react/commits/v19.2.4/packages/react-dom )
---
updated-dependencies:
- dependency-name: react-dom
dependency-version: 19.2.4
dependency-type: direct:production
update-type: version-update:semver-patch
...
Signed-off-by: dependabot[bot] <support@github.com >
2026-03-23 12:23:49 +00:00
dependabot[bot]
33277717fa
chore(deps): bump lucide-react from 0.563.0 to 0.577.0 in /frontend
...
Bumps [lucide-react](https://github.com/lucide-icons/lucide/tree/HEAD/packages/lucide-react ) from 0.563.0 to 0.577.0.
- [Release notes](https://github.com/lucide-icons/lucide/releases )
- [Commits](https://github.com/lucide-icons/lucide/commits/0.577.0/packages/lucide-react )
---
updated-dependencies:
- dependency-name: lucide-react
dependency-version: 0.577.0
dependency-type: direct:production
update-type: version-update:semver-minor
...
Signed-off-by: dependabot[bot] <support@github.com >
2026-03-23 12:23:40 +00:00
Christian Krakau-Louis
6d80be7abe
Merge pull request #23 from christianlouis/copilot/fix-password-length-issue
...
Fix bcrypt 5.x incompatibility breaking password hashing tests
2026-03-23 13:23:10 +01:00
Christian Krakau-Louis
725d7a42ea
Merge branch 'main' into copilot/fix-password-length-issue
2026-03-23 13:21:20 +01:00
Christian Krakau-Louis
7e74d98cbe
Merge pull request #25 from christianlouis/copilot/update-deployment-documentation
...
Add deployment guide with Docker Compose and Kubernetes examples
2026-03-23 13:21:04 +01:00
Christian Krakau-Louis
9217efb4db
Merge pull request #24 from christianlouis/copilot/secure-ai-agent-infrastructure
...
Milestone 1: Security & infrastructure hardening
2026-03-23 13:20:45 +01:00
copilot-swe-agent[bot]
d66af4d8ef
feat: Milestone 1 Security & Infrastructure improvements
...
- Add Dependabot configuration for pip, npm, GitHub Actions, Docker
- Update copilot instructions to require TODO.md and CHANGELOG.md updates
- Add unit tests for middleware, security, app factory, and schemas (125 total tests, 57% coverage)
- Update TODO.md with current progress
- Update CHANGELOG.md with all changes
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
Agent-Logs-Url: https://github.com/christianlouis/pop_puller_to_gmail/sessions/2f15a52f-6812-4586-9d5c-a144226c842b
2026-03-23 11:23:37 +00:00
copilot-swe-agent[bot]
a84a4fe033
Add comprehensive deployment guide (docs/DEPLOYMENT_GUIDE.md)
...
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
Agent-Logs-Url: https://github.com/christianlouis/pop_puller_to_gmail/sessions/0204699a-6f9d-4481-a23c-b543fee5e3fc
2026-03-23 11:18:33 +00:00
copilot-swe-agent[bot]
1402afb9ce
Initial plan
2026-03-23 11:13:45 +00:00
copilot-swe-agent[bot]
3d48bc6f66
Initial plan
2026-03-23 11:12:14 +00:00
copilot-swe-agent[bot]
5fd0914b07
fix: replace passlib with direct bcrypt calls to fix bcrypt 5.x incompatibility
...
passlib 1.7.4 is incompatible with bcrypt>=5.0.0. When passlib initializes
its bcrypt backend, it calls detect_wrap_bug() with a >72-byte test password.
bcrypt 5.0.0 raises ValueError for such passwords, crashing the initialization
before any user code runs.
Fix: replace passlib[bcrypt] with direct bcrypt==4.3.0 usage:
- security.py: use bcrypt.hashpw()/checkpw() instead of CryptContext
- requirements.txt: replace passlib[bcrypt]==1.7.4 with bcrypt==4.3.0
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
Agent-Logs-Url: https://github.com/christianlouis/pop_puller_to_gmail/sessions/a2327bbe-cc06-4c1f-9010-380fb0fd7d67
2026-03-23 11:07:25 +00:00
Christian Krakau-Louis
319282121f
Merge pull request #21 from christianlouis/copilot/clean-up-repository
...
Clean up repository structure, add standard OSS files, fix documentation accuracy
2026-03-23 12:06:59 +01:00
copilot-swe-agent[bot]
1263ed5266
Update TODO.md: document missing frontend API client, mark completed items, update progress tracking
...
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
Agent-Logs-Url: https://github.com/christianlouis/pop_puller_to_gmail/sessions/71f26285-5584-42b2-8255-8ad2c9e9ecb4
2026-03-23 11:02:09 +00:00
copilot-swe-agent[bot]
df69070951
Initial plan
2026-03-23 10:58:40 +00:00
copilot-swe-agent[bot]
b3a0c4bfd8
Clean up repo: move docs to docs/, add SECURITY.md, .editorconfig, update README with badges, fix cross-references, correct documentation to reflect actual project state
...
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
Agent-Logs-Url: https://github.com/christianlouis/pop_puller_to_gmail/sessions/71f26285-5584-42b2-8255-8ad2c9e9ecb4
2026-03-23 10:57:31 +00:00
Christian Krakau-Louis
dfc074352c
Merge pull request #22 from christianlouis/copilot/fix-unused-imports
...
Fix all CI lint failures: ruff, mypy, and eslint
2026-03-23 11:54:15 +01:00
copilot-swe-agent[bot]
27c50e49cf
Fix all mypy and eslint errors, add copilot lint instructions
...
Backend (mypy - 56 errors fixed):
- database.py: Fix async generator return type to AsyncGenerator
- database_models.py: Add type annotations for SQLEnum columns
- middleware.py: Use explicit Optional for exempt_paths parameter
- mail_processor.py: Fix type narrowing in fetch_emails, add Dict type annotation for KNOWN_PROVIDERS
- users.py, auth.py, providers.py, tasks.py: Add type: ignore comments for SQLAlchemy Column assignment patterns
Frontend (eslint - 3 errors, 2 warnings fixed):
- login/page.tsx: Replace any with unknown + type narrowing, prefix unused vars with underscore
- register/page.tsx: Replace any with unknown + type narrowing
Add .github/copilot-instructions.md with lint-check requirements
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
Agent-Logs-Url: https://github.com/christianlouis/pop_puller_to_gmail/sessions/dac7ab78-fe27-4fe4-890f-32c6c1c6d881
2026-03-23 10:48:16 +00:00
copilot-swe-agent[bot]
a918421945
Fix all 33 ruff linting errors causing CI failures
...
- Remove unused imports (F401) across 17 files
- Fix f-strings without placeholders (F541) in 3 files
- Add noqa: E712 to SQLAlchemy == True comparisons (valid ORM pattern)
- Preserve alembic side-effect import with noqa: F401
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com >
Agent-Logs-Url: https://github.com/christianlouis/pop_puller_to_gmail/sessions/99c3a25f-3479-473d-ac95-9faaa0ddd55b
2026-03-23 10:38:39 +00:00
copilot-swe-agent[bot]
1463ad225c
Initial plan
2026-03-23 10:34:55 +00:00
Christian Krakau-Louis
46104d7d21
Merge pull request #18 from christianlouis/copilot/add-account-management-functionality
...
Fix CI: pytest module resolution, black formatting, authlib security vulnerabilities
2026-03-23 11:33:12 +01:00
Christian Krakau-Louis
29722101cd
Merge pull request #19 from christianlouis/copilot/fix-module-not-found-error
...
Fix test collection failure and patch authlib security vulnerabilities
2026-03-23 11:32:48 +01:00
Christian Krakau-Louis
5cbaacb642
Merge branch 'main' into copilot/fix-module-not-found-error
2026-03-23 11:32:39 +01:00
Christian Krakau-Louis
a0957c55fc
Merge pull request #20 from christianlouis/copilot/rebuild-actions-phased-ci-approach
...
Consolidate CI into single phased pipeline with lint → test → security → build
2026-03-23 11:31:57 +01:00