From 902e84d7ad5bfec39940539c7d79d4c17b82221c Mon Sep 17 00:00:00 2001
From: Jedrzej Kosinski <kosinkadink1@gmail.com>
Date: Wed, 28 Jan 2026 16:04:19 -0800
Subject: [PATCH] Remove tags from body of
 @ROUTES.put(f"/api/assets/{{id:{UUID_RE}}}"), add note about blake3
 requirement to test out

---
 app/assets/api/routes.py     | 1 -
 app/assets/api/schemas_in.py | 8 ++------
 app/assets/manager.py        | 4 ++++
 3 files changed, 6 insertions(+), 7 deletions(-)

diff --git a/app/assets/api/routes.py b/app/assets/api/routes.py
index 612c34e6d..16ed5a3e2 100644
--- a/app/assets/api/routes.py
+++ b/app/assets/api/routes.py
@@ -338,7 +338,6 @@ async def update_asset(request: web.Request) -> web.Response:
         result = manager.update_asset(
             asset_info_id=asset_info_id,
             name=body.name,
-            tags=body.tags,
             user_metadata=body.user_metadata,
             owner_id=USER_MANAGER.get_request_user_id(request),
         )
diff --git a/app/assets/api/schemas_in.py b/app/assets/api/schemas_in.py
index e56c3adec..6707ffb0c 100644
--- a/app/assets/api/schemas_in.py
+++ b/app/assets/api/schemas_in.py
@@ -58,16 +58,12 @@ class ListAssetsQuery(BaseModel):
 
 class UpdateAssetBody(BaseModel):
     name: str | None = None
-    tags: list[str] | None = None
     user_metadata: dict[str, Any] | None = None
 
     @model_validator(mode="after")
     def _at_least_one(self):
-        if self.name is None and self.tags is None and self.user_metadata is None:
-            raise ValueError("Provide at least one of: name, tags, user_metadata.")
-        if self.tags is not None:
-            if not isinstance(self.tags, list) or not all(isinstance(t, str) for t in self.tags):
-                raise ValueError("Field 'tags' must be an array of strings.")
+        if self.name is None and self.user_metadata is None:
+            raise ValueError("Provide at least one of: name, user_metadata.")
         return self
 
 
diff --git a/app/assets/manager.py b/app/assets/manager.py
index c9ccf6e2f..e3fe5f7a5 100644
--- a/app/assets/manager.py
+++ b/app/assets/manager.py
@@ -172,7 +172,11 @@ def upload_asset_from_temp_path(
     owner_id: str = "",
     expected_asset_hash: str | None = None,
 ) -> schemas_out.AssetCreated:
+    """
+    Create new asset or update existing asset from a temporary file path.
+    """
     try:
+        # NOTE: blake3 is not required right now, so this will fail if blake3 is not installed in local environment
         import app.assets.hashing as hashing
         digest = hashing.blake3_hash(temp_path)
     except Exception as e: