fastapi/full-stack-fastapi-template

Pattern Findings

78 matches across 6 categories. Click a row to expand file-level details.

Hyper-Verbose Identifiers48 hits · 50 pts

Severity	File	Line	Snippet
LOW	backend/app/utils.py	68	def generate_reset_password_email(email_to: str, email: str, token: str) -> EmailData:
LOW	backend/app/utils.py	85	def generate_new_account_email(
LOW	backend/app/utils.py	103	def generate_password_reset_token(email: str) -> str:
LOW	backend/app/utils.py	116	def verify_password_reset_token(token: str) -> str \| None:
LOW	backend/app/main.py	10	def custom_generate_unique_id(route: APIRoute) -> str:
LOW	backend/app/core/config.py	109	def _enforce_non_default_secrets(self) -> Self:
LOW	backend/app/api/deps.py	52	def get_current_active_superuser(current_user: CurrentUser) -> User:
LOW	backend/app/api/routes/login.py	105	def recover_password_html_content(email: str, session: SessionDep) -> Any:
LOW	backend/tests/conftest.py	39	def normal_user_token_headers(client: TestClient, db: Session) -> dict[str, str]:
LOW	backend/tests/utils/user.py	10	def user_authentication_headers(
LOW	backend/tests/utils/user.py	30	def authentication_token_from_email(
LOW	backend/tests/utils/utils.py	17	def get_superuser_token_headers(client: TestClient) -> dict[str, str]:
LOW	backend/tests/scripts/test_backend_pre_start.py	8	def test_init_successful_connection() -> None:
LOW	backend/tests/scripts/test_test_pre_start.py	8	def test_init_successful_connection() -> None:
LOW	backend/tests/crud/test_user.py	30	def test_not_authenticate_user(db: Session) -> None:
LOW	backend/tests/crud/test_user.py	37	def test_check_if_user_is_active(db: Session) -> None:
LOW	backend/tests/crud/test_user.py	45	def test_check_if_user_is_active_inactive(db: Session) -> None:
LOW	backend/tests/crud/test_user.py	53	def test_check_if_user_is_superuser(db: Session) -> None:
LOW	backend/tests/crud/test_user.py	61	def test_check_if_user_is_superuser_normal_user(db: Session) -> None:
LOW	backend/tests/api/routes/test_login.py	28	def test_get_access_token_incorrect_password(client: TestClient) -> None:
LOW	backend/tests/api/routes/test_login.py	67	def test_recovery_password_user_not_exits(
LOW	backend/tests/api/routes/test_login.py	113	def test_reset_password_invalid_token(
LOW	backend/tests/api/routes/test_items.py	55	def test_read_item_not_enough_permissions(
LOW	backend/tests/api/routes/test_items.py	100	def test_update_item_not_found(
LOW	backend/tests/api/routes/test_items.py	114	def test_update_item_not_enough_permissions(
LOW	backend/tests/api/routes/test_items.py	142	def test_delete_item_not_found(
LOW	backend/tests/api/routes/test_items.py	154	def test_delete_item_not_enough_permissions(
LOW	backend/tests/api/routes/test_users.py	15	def test_get_users_superuser_me(
LOW	backend/tests/api/routes/test_users.py	26	def test_get_users_normal_user_me(
LOW	backend/tests/api/routes/test_users.py	37	def test_create_user_new_email(
LOW	backend/tests/api/routes/test_users.py	60	def test_get_existing_user_as_superuser(
LOW	backend/tests/api/routes/test_users.py	79	def test_get_non_existing_user_as_superuser(
LOW	backend/tests/api/routes/test_users.py	90	def test_get_existing_user_current_user(client: TestClient, db: Session) -> None:
LOW	backend/tests/api/routes/test_users.py	117	def test_get_existing_user_permissions_error(
LOW	backend/tests/api/routes/test_users.py	132	def test_get_non_existing_user_permissions_error(
LOW	backend/tests/api/routes/test_users.py	146	def test_create_user_existing_username(
LOW	backend/tests/api/routes/test_users.py	165	def test_create_user_by_normal_user(
LOW	backend/tests/api/routes/test_users.py	267	def test_update_password_me_incorrect_password(
LOW	backend/tests/api/routes/test_users.py	282	def test_update_user_me_email_exists(
LOW	backend/tests/api/routes/test_users.py	300	def test_update_password_me_same_password_error(
LOW	backend/tests/api/routes/test_users.py	342	def test_register_user_already_exists_error(client: TestClient) -> None:
LOW	backend/tests/api/routes/test_users.py	384	def test_update_user_not_exists(
LOW	backend/tests/api/routes/test_users.py	397	def test_update_user_email_exists(
LOW	backend/tests/api/routes/test_users.py	451	def test_delete_user_me_as_superuser(
LOW	backend/tests/api/routes/test_users.py	463	def test_delete_user_super_user(
LOW	backend/tests/api/routes/test_users.py	482	def test_delete_user_not_found(
LOW	backend/tests/api/routes/test_users.py	493	def test_delete_user_current_super_user_error(
LOW	backend/tests/api/routes/test_users.py	508	def test_delete_user_without_privileges(

Self-Referential Comments11 hits · 38 pts

Severity	File	Line	Snippet
MEDIUM	compose.override.yml	23	# Create an entrypoint "http" listening on port 80
MEDIUM	compose.override.yml	25	# Create an entrypoint "https" listening on port 443
MEDIUM	compose.traefik.yml	15	# Define the port inside of the Docker service to use
MEDIUM	compose.traefik.yml	48	# Create an entrypoint "http" listening on port 80
MEDIUM	compose.traefik.yml	50	# Create an entrypoint "https" listening on port 443
MEDIUM	compose.traefik.yml	52	# Create the certificate resolver "le" for Let's Encrypt, uses the environment variable EMAIL
MEDIUM	compose.traefik.yml	70	# Create a volume to store the certificates, even if the container is recreated
MEDIUM	…98dd8ec85a3_edit_replace_id_integers_in_all_models_.py	25	# Create a new UUID column with a default UUID value
MEDIUM	backend/tests/crud/test_user.py	101	# Create a bcrypt hash directly (simulating legacy password)
MEDIUM	backend/tests/api/routes/test_login.py	136	# Create a bcrypt hash directly (simulating legacy password)
MEDIUM	backend/tests/api/routes/test_login.py	170	# Create an argon2 hash (current default)

Fake / Example Data6 hits · 6 pts

Severity	File	Line	Snippet
LOW	README.md	200	- `first_superuser`: (default: `"admin@example.com"`) The email of the first superuser (in .env).
LOW	frontend/src/routes/login.tsx	91	placeholder="user@example.com"
LOW	frontend/src/routes/recover-password.tsx	101	placeholder="user@example.com"
LOW	frontend/src/routes/signup.tsx	122	placeholder="user@example.com"
LOW	backend/tests/api/routes/test_login.py	54	patch("app.core.config.settings.SMTP_USER", "admin@example.com"),
LOW	backend/tests/api/routes/test_users.py	43	patch("app.core.config.settings.SMTP_USER", "admin@example.com"),

Unused Imports5 hits · 5 pts

Severity	File	Line
LOW	backend/app/alembic/env.py	1
LOW	…rsions/fe56fa70289e_add_created_at_to_user_and_item.py	10
LOW	…98dd8ec85a3_edit_replace_id_integers_in_all_models_.py	10
LOW	…sions/1a31ce608336_add_cascade_delete_relationships.py	10
LOW	…ons/9c0a54914c78_add_max_length_for_string_varchar_.py	10

Over-Commented Block4 hits · 4 pts

Severity	File	Line	Snippet
LOW	frontend/playwright.config.ts	41	storageState: 'playwright/.auth/user.json',
LOW	frontend/playwright.config.ts	61	// dependencies: ['setup'],
LOW	backend/alembic.ini	1	# A generic, single database configuration.
LOW	backend/alembic.ini	21	# the 'revision' command, regardless of autogenerate

Excessive Try-Catch Wrapping4 hits · 4 pts

Severity	File	Line	Snippet
LOW	backend/app/backend_pre_start.py	27	except Exception as e:
LOW	backend/app/tests_pre_start.py	27	except Exception as e:
LOW	backend/tests/scripts/test_backend_pre_start.py	26	except Exception:
LOW	backend/tests/scripts/test_test_pre_start.py	26	except Exception:

fastapi/full-stack-fastapi-template

Score History

Severity Breakdown

Pattern Findings