make skip reason optional in backend

Author: andreaskoepf

backend/oasst_backend/models/task.py

@@ -32,7 +32,7 @@ class Task(SQLModel, table=True):
     ack: Optional[bool] = None
     done: bool = Field(sa_column=sa.Column(sa.Boolean, nullable=False, server_default=false()))
     skipped: bool = Field(sa_column=sa.Column(sa.Boolean, nullable=False, server_default=false()))
-    skip_reason: str = Field(nullable=True, max_length=512)
+    skip_reason: Optional[str] = Field(nullable=True, max_length=512)
     frontend_message_id: Optional[str] = None
     message_tree_id: Optional[UUID] = None
     parent_message_id: Optional[UUID] = None

backend/oasst_backend/prompt_repository.py

@@ -1063,7 +1063,7 @@ def get_stats(self) -> SystemStats:
         )
 
     @managed_tx_method()
-    def skip_task(self, task_id: UUID, reason: str):
+    def skip_task(self, task_id: UUID, reason: Optional[str]):
         self.ensure_user_is_enabled()
 
         task = self.task_repository.fetch_task_by_id(task_id)

oasst-shared/oasst_shared/schemas/protocol.py

@@ -141,7 +141,7 @@ class TaskAck(BaseModel):
 class TaskNAck(BaseModel):
     """The frontend acknowledges that it has received a task but cannot create a message."""
 
-    reason: str
+    reason: str | None = Field(None, nullable=True)
 
 
 class TaskClose(BaseModel):