Skip to content

refactor(auth): clarify module responsibilities - #26

Merged
rd-code-23 merged 1 commit into
mainfrom
feature/auth
Aug 19, 2026
Merged

rd-code-23 merged 1 commit into
mainfrom
feature/auth

Conversation

@rd-code-23

Copy link
Copy Markdown
Owner

What changed

  • group Google OIDC routes and user provisioning under auth/google
  • group PostgreSQL session middleware under auth/session
  • add descriptive .routes, .service, and .middleware filename suffixes
  • keep each test colocated with the implementation it verifies
  • update imports and concise project architecture guidance

Why

The previous auth filenames identified the feature but not each module’s role. This structure keeps vertical feature organization while making routes, services, middleware, and tests immediately recognizable without adding unnecessary controller or repository layers.

Developer impact

New or modified feature files should use descriptive role suffixes where helpful and focused capability subfolders as features grow. Empty architectural layers are not required.

Validation

  • scoped Prettier check
  • pnpm --filter @goforlift/api lint
  • pnpm --filter @goforlift/api typecheck
  • pnpm --filter @goforlift/api test — 34 tests passed
  • pnpm --filter @goforlift/api build

@rd-code-23
rd-code-23 marked this pull request as ready for review August 19, 2026 03:37
@rd-code-23
rd-code-23 merged commit c56186f into main Aug 19, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant