fix: default to None for optional pydantic fields (#45)

closes #44

Co-authored-by: msramalho <[email protected]>

.gitignore

@@ -169,3 +169,7 @@ whisperbox-transcribe.sqlite*
 
 # ruff
 .ruff_cache
+
+# other private files
+/data
+.env.prod

Makefile

@@ -18,8 +18,8 @@ test:
 	pytest
 
 run:
-	docker compose -f docker-compose.base.yml -f docker-compose.prod.yml build -d
-	docker compose -f docker-compose.base.yml -f docker-compose.prod.yml up --remove-orphans
+	docker compose -f docker-compose.base.yml -f docker-compose.prod.yml build
+	docker compose -f docker-compose.base.yml -f docker-compose.prod.yml up -d --remove-orphans
 
 stop:
 	docker compose -f docker-compose.base.yml -f docker-compose.prod.yml down

app/shared/db/models.py

@@ -41,22 +41,25 @@ class JobConfig(BaseModel):
     """(JSON) Configuration for a job."""
 
     language: str | None = Field(
+        default=None,
         description=(
             "Spoken language in the media file. "
             "While optional, this can improve output."
-        )
+        ),
     )
 
 
 class JobMeta(BaseModel):
     """(JSON) Metadata relating to a job's execution."""
 
     error: str | None = Field(
-        description="Will contain a descriptive error message if processing failed."
+        default=None,
+        description="Will contain a descriptive error message if processing failed.",
     )
 
     task_id: uuid.UUID | None = Field(
-        description="Internal celery id of this job submission."
+        default=None,
+        description="Internal celery id of this job submission.",
     )
 
 

app/tests/conftest.py

@@ -47,7 +47,8 @@ def mock_job(db_session):
     job = models.Job(
         url="https://example.com",
         type=models.JobType.transcript,
-        status=models.JobStatus.create,
+        status=models.JobStatus.processing,
+        meta={"task_id": "5c790c76-2cc1-4e91-a305-443df55a4a4c"},
     )
     db_session.add(job)
     db_session.flush()

app/web/dtos.py

@@ -33,6 +33,8 @@ class Job(WithDbFields):
 
 
 class Artifact(WithDbFields):
+    """A transcription artifact."""
+
     job_id: UUID
     data: ArtifactData
     type: ArtifactType

app/web/main.py

@@ -47,7 +47,7 @@ def api_root() -> None:
         response_model=list[dtos.Job],
         summary="Get metadata for all jobs",
     )
-    def get_transcripts(
+    def get_jobs(
         session: DatabaseSession,
         type: dtos.JobType | None = None,
     ) -> list[models.Job]:
@@ -65,7 +65,7 @@ def get_transcripts(
         response_model=dtos.Job,
         summary="Get metadata for one job",
     )
-    def get_transcript(
+    def get_job(
         session: DatabaseSession,
         id: UUID = Path(),
     ) -> models.Job | None: