docs: global docstrings

Issue: #13

src/config.py

@@ -1,8 +1,10 @@
 """
-Global configurations: import settings, app_configs
+Global configurations
 
-Classes:
+Exports:
  - CustomBaseSettings - Base class to be used by all modules for loading configs
+ - settings: Global configurations object
+ - app_configs: Dict generated from configs, used in app initialisation
 """
 
 from typing import Any

src/schemas.py

@@ -1,3 +1,10 @@
+"""
+Global Pydantic schemas
+
+Exports:
+ - CustomBaseModel: Schema used for all other Pydantic models
+ - ResourceName
+"""
 from pydantic import BaseModel
 from typing import Optional
 

src/service/config.py

@@ -1,7 +1,3 @@
 """
-Configurations for <this module>
-
-Configurations:
- - List: Description
- - Configs: Description
+Configurations for the services module
 """

src/service/constants.py

@@ -1,7 +1,3 @@
 """
-Constants and error codes for <this module>
-
-Constants:
- - List: Description
- - Consts: Description
+Constants for the services module
 """

src/service/dependencies.py

@@ -1,13 +1,9 @@
 """
-Router dependencies for <this module>
+Dependencies related to the services module
 
-Classes:
- - List: Description
- - Classes: Description
-
-Functions:
- - List: Description
- - Functions: Description
+Exports:
+ - service_model_query_dependency: service_model: Gets service model from db, if it exists. Uses service_id from query param.
+ - service_model_body_dependency: service_model: Gets service model from db, if it exists. Uses service_id from request body.
 """
 from typing import Annotated
 from fastapi import Depends, Query

src/service/exceptions.py

@@ -1,9 +1,8 @@
 """
-Module specific exceptions for <this module>
+Exceptions related to the services module
 
 Exceptions:
- - List: Description
- - Exceptions: Description
+ - ServiceNotFoundException: Takes an optional service_id int
 """
 from typing import Optional
 

src/service/models.py

@@ -2,8 +2,8 @@
 Database models for the services module
 
 Models:
- - List: Description
- - Models: Description
+ - Service:
+  - id[PK], name[U], api_key[U]
 """
 from sqlalchemy import Column, Integer, String
 

src/service/router.py

@@ -1,9 +1,11 @@
 """
-Router endpoints for <this module>
+Router endpoints for the services module
 
 Endpoints:
- - List: Description
- - Endpoints: Description
+ - [GET](/): [root user]: Get a list of all services(id, name)
+ - [POST](/): [super admin]: Register a new service(name) on the hub, returns the API key for the service to access the hub.
+ - [PATCH](/key): [super_admin]: Refreshes the API key for a service(id), returning a new one.
+ - [DELETE](/): [super_admin]: Removes a service(id) from the hub.
 """
 from fastapi import APIRouter, status
 from psycopg.errors import UniqueViolation

src/service/schemas.py

@@ -1,9 +1,10 @@
 """
-Pydantic models for the service module
+Pydantic models for service module
 
-Models:
- - List: Description
- - Models: Description
+Models follow the nomenclature of:
+- Sub-models: "<Resource><Opt:>Schema"
+- Mixins: "<Attribute>Mixin"
+- Models: "<Module><Method><Resource><Opt:Resource><Direction>" ie "ServiceGetServiceResponse"
 """
 from pydantic import ConfigDict
 

src/service/service.py

@@ -1,11 +1,3 @@
 """
-Module specific business logic for <this module>
-
-Classes:
- - List: Description
- - Classes: Description
-
-Functions:
- - List: Description
- - Functions: Description
+Business logic for the services module
 """

src/service/utils.py

@@ -1,13 +1,8 @@
 """
-Non-business logic reusable functions and classes for <this module>
+Non-business logic reusable functions and classes for the services module
 
-Classes:
- - List: Description
- - Classes: Description
-
-Functions:
- - List: Description
- - Functions: Description
+Exports:
+ - generate_api_key(): returns a new UUID
 """
 import uuid
 

src/user/config.py

@@ -1,7 +1,3 @@
 """
-Configurations for user module
-
-Configurations:
- - List: Description
- - Configs: Description
+Configurations for the user module
 """

src/user/constants.py

@@ -1,7 +1,3 @@
 """
-Constants and error codes for user module
-
-Constants:
- - List: Description
- - Consts: Description
+Constants for the user module
 """

src/user/dependencies.py

@@ -1,13 +1,10 @@
 """
-Router dependencies for user module
+Dependencies related to the user module
 
-Classes:
- - List: Description
- - Classes: Description
-
-Functions:
- - List: Description
- - Functions: Description
+Exports:
+ - user_model_claims_dependency: user_model: Gets user model from db, if it exists. Uses db_id from user claims.
+ - user_model_query_dependency: user_model: Gets user model from db, if it exists. Uses user_id from query param
+ - user_model_body_dependency: user_model: Gets user model from db, if it exists. Uses user_id from request body.
 """
 from typing import Annotated
 from fastapi import Depends, Query

src/user/exceptions.py

@@ -1,9 +1,8 @@
 """
-Module specific exceptions for user module
+Exceptions related to the user module
 
 Exceptions:
- - List: Description
- - Exceptions: Description
+ - UserNotFoundException: Takes an optional user_id int
 """
 from typing import Optional
 

src/user/models.py

@@ -2,7 +2,12 @@
 Database models for user module
 
 Models:
- - User - id[pk], email, first_name, last_name, oidc_id
+ - User:
+  - id[PK], email, first_name, last_name, oidc_id
+  - organisation_rel: ORM relationship to Organisation via OrgUsers table
+  - group_rel: ORM relationship to Group via UserGroups table
+  - organisations: Calc property list of organisation_rel.name
+  - groups: Calc property dict of {group_rel.org_rel.name: group_rel.name}
 """
 from collections import defaultdict
 

src/user/router.py

@@ -1,15 +1,11 @@
 """
-Router endpoints for user module
+Router endpoints for the user module
 
 Endpoints:
- - [get]/self/claims - Retrieves user's OIDC claims
- - [get]/self/db - Retrieves the user data from the db that corresponds to the current OIDC user
- - [get]/self/orgs - Retrieves all organisations associated with the current user
- - [get]/self/orgs/admin - Retrieves only admin organisations for the current user
- - [get]/{user_id} - Retrieves a specific user by their ID
- - [get]/{user_id}/orgs - Retrieves all organisations associated with a specific user
- - [get]/{user_id}/orgs/admin - Retrieves only admin organisations for a specific user
- - [delete]/{user_id} - Deletes a user from the db by their db ID
+ - [GET](/user/self/claims): [OIDC claims]: Returns all OIDC claims associated with the currently logged-in user.
+ - [GET](/user/self/db): [OIDC claims]: Returns details about the currently logged-in user from the hub db.
+ - [GET](/user/): [super admin]: Returns user(id) details.
+ - [DELETE](/user/): [super admin]: Removes a User(id) from the hub database.
 """
 from fastapi import APIRouter
 from starlette import status

src/user/schemas.py

@@ -1,9 +1,5 @@
 """
-Pydantic models for user module
-
-Models:
- - List: Description
- - Models: Description
+Pydantic models for the user module
 """
 from typing import Optional
 from src.schemas import CustomBaseModel

src/user/service.py

@@ -1,11 +1,8 @@
 """
 Module specific business logic for user module
 
-Functions:
- - add_user_to_db
-
 Exports:
- - add_user_to_db
+ - add_user_to_db: Creates a User record from OIDC claims, or updates user details
 """
 from typing import Any
 

src/user/utils.py

@@ -1,11 +1,3 @@
 """
-Non-business logic reusable functions and classes for user module
-
-Classes:
- - List: Description
- - Classes: Description
-
-Functions:
- - List: Description
- - Functions: Description
+Non-business logic reusable functions and classes for the user module
 """