Hotfix: Log a warning instead of failing the build when item NBT parsing fails

Author: object-Object

CHANGELOG.md

@@ -4,6 +4,12 @@ All notable changes to this project will be documented in this file.
 
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/) and [Pydantic's HISTORY.md](https://github.com/pydantic/pydantic/blob/main/HISTORY.md), and this project *mostly* adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## `1!0.1.0a23`
+
+### Fixed
+
+* Hotfix: Log a warning instead of failing the build when item NBT parsing fails.
+
 ## `1!0.1.0a22`
 
 ### Changed

src/hexdoc/core/resource.py

@@ -254,6 +254,8 @@ def __repr__(self) -> str:
 ResLoc = ResourceLocation
 
 
+# FIXME: commas?????????????????
+# https://vazkiimods.github.io/Patchouli/docs/patchouli-advanced/itemstack-format
 @dataclass(frozen=True, repr=False)
 class ItemStack(BaseResourceLocation, regex=_make_regex(count=True, nbt=True)):
     """Represents an item with optional count and NBT.
@@ -346,12 +348,15 @@ def _parse_nbt(nbt: str | None) -> Compound | None:
     if nbt is None:
         return None
 
+    # TODO: maybe re-add strict parsing when we're sure it will actually work?
     try:
         result = parse_nbt(nbt)
-    except ValueError as e:
-        raise ValueError(f"Failed to parse sNBT literal '{nbt}': {e}") from e
+    except Exception as e:
+        logger.warning(f"Failed to parse sNBT literal '{nbt}': {e}")
+        return None
 
     if not isinstance(result, Compound):
-        raise ValueError(f"Expected Compound, got {type(result)}: {result}")
+        logger.warning(f"Expected Compound, got {type(result)}: {result}")
+        return None
 
     return result