feat: support for empty file names

beet/library/base.py

@@ -689,8 +689,6 @@ class NamespaceProxy(
 
     def split_key(self, key: str) -> Tuple[str, str]:
         namespace, _, file_path = key.partition(":")
-        if not file_path:
-            raise KeyError(key)
         return namespace, file_path
 
     def join_key(self, key1: str, key2: str) -> str:

beet/library/utils.py

@@ -63,9 +63,22 @@ def list_origin_folders(prefix: str, origin: FileOrigin) -> Dict[str, List[PureP
     return folders
 
 
+def modified_suffixes(path: PurePath) -> List[str]:
+    """
+    Equivalent to path.suffixes but support file with empty name
+    """
+    name = path.name
+    if name.endswith("."):
+        return []
+    if name.startswith("."):
+        name = name[1:]
+        return ["." + suffix for suffix in name.split(".")]
+    return path.suffixes
+
+
 def list_extensions(path: PurePath) -> List[str]:
     extensions: List[str] = list(
-        accumulate(reversed(path.suffixes), lambda a, b: b + a)  # type: ignore
+        accumulate(reversed(modified_suffixes(path)), lambda a, b: b + a)  # type: ignore
     )
     extensions.reverse()
     extensions.append("")

examples/code_void/add_header.py

@@ -0,0 +1,9 @@
+from beet import Context, Function
+
+
+def beet_default(ctx: Context):
+    for f in ctx.data.functions.keys():
+        ctx.data.functions[f] = Function(f"say {f}")
+
+    ctx.data.functions["namespace:"] = Function("give me sugar")
+    ctx.data.functions["namespace:a/"] = Function("give me apple")

examples/code_void/beet.yaml

@@ -0,0 +1,10 @@
+data_pack:
+  load: [.]
+resource_pack:
+  load: [.]
+
+output: build
+
+
+pipeline:
+  - add_header

tests/snapshots/examples__build_code_void__0.data_pack/data/namespace/function/.mcfunction

@@ -0,0 +1 @@
+give me sugar

tests/snapshots/examples__build_code_void__0.data_pack/data/namespace/function/a/.mcfunction

@@ -0,0 +1 @@
+give me apple

tests/snapshots/examples__build_code_void__0.data_pack/data/test/function/...mcfunction

@@ -0,0 +1 @@
+say test:..

tests/snapshots/examples__build_code_void__0.data_pack/data/test/function/..mcfunction

@@ -0,0 +1 @@
+say test:.

tests/snapshots/examples__build_code_void__0.data_pack/data/test/function/.mcfunction

@@ -0,0 +1 @@
+say test:

tests/snapshots/examples__build_code_void__0.data_pack/data/test/function/.mcfunction.mcfunction

@@ -0,0 +1 @@
+say test:.mcfunction

tests/snapshots/examples__build_code_void__0.data_pack/data/test/function/.mcfunction.mcfunction.mcfunction

@@ -0,0 +1 @@
+say test:.mcfunction.mcfunction

tests/snapshots/examples__build_code_void__0.data_pack/data/test/function/a.mcfunction

@@ -0,0 +1 @@
+say test:a

tests/snapshots/examples__build_code_void__0.data_pack/data/test/function/a.py.git.mcfunction

@@ -0,0 +1 @@
+say test:a.py.git

tests/snapshots/examples__build_code_void__0.data_pack/data/test/function/a/...mcfunction

@@ -0,0 +1 @@
+say test:a/..

tests/snapshots/examples__build_code_void__0.data_pack/data/test/function/a/..mcfunction

@@ -0,0 +1 @@
+say test:a/.

tests/snapshots/examples__build_code_void__0.data_pack/data/test/function/a/.mcfunction

@@ -0,0 +1 @@
+say test:a/

tests/snapshots/examples__build_code_void__0.data_pack/data/test/function/a/.mcfunction.mcfunction

@@ -0,0 +1 @@
+say test:a/.mcfunction

tests/snapshots/examples__build_code_void__0.data_pack/data/test/function/a/a.mcfunction

@@ -0,0 +1 @@
+say test:a/a

tests/snapshots/examples__build_code_void__0.data_pack/data/test/function/a/a.py.git.mcfunction

@@ -0,0 +1 @@
+say test:a/a.py.git

tests/snapshots/examples__build_code_void__0.data_pack/pack.mcmeta

@@ -0,0 +1,6 @@
+{
+  "pack": {
+    "pack_format": 61,
+    "description": ""
+  }
+}

tests/snapshots/examples__build_code_void__1.resource_pack/pack.mcmeta

@@ -0,0 +1,6 @@
+{
+  "pack": {
+    "pack_format": 46,
+    "description": ""
+  }
+}

tests/snapshots/modified_suffixes__paths_suffixes_aaa_mcfunction__aaa...mcfunction.json

@@ -0,0 +1,13 @@
+{
+  "path": "aaa...mcfunction",
+  "suffixes": [
+    ".",
+    ".",
+    ".mcfunction"
+  ],
+  "modified_suffixes": [
+    ".",
+    ".",
+    ".mcfunction"
+  ]
+}

tests/snapshots/modified_suffixes__paths_suffixes_load_mcfunction__load.mcfunction.json

@@ -0,0 +1,9 @@
+{
+  "path": "load.mcfunction",
+  "suffixes": [
+    ".mcfunction"
+  ],
+  "modified_suffixes": [
+    ".mcfunction"
+  ]
+}

tests/snapshots/modified_suffixes__paths_suffixes_load_py_mcfunction__load.py.mcfunction.json

@@ -0,0 +1,11 @@
+{
+  "path": "load.py.mcfunction",
+  "suffixes": [
+    ".py",
+    ".mcfunction"
+  ],
+  "modified_suffixes": [
+    ".py",
+    ".mcfunction"
+  ]
+}

tests/snapshots/modified_suffixes__paths_suffixes_mcfunction__...mcfunction.json

@@ -0,0 +1,9 @@
+{
+  "path": "...mcfunction",
+  "suffixes": [],
+  "modified_suffixes": [
+    ".",
+    ".",
+    ".mcfunction"
+  ]
+}

tests/snapshots/modified_suffixes__paths_suffixes_mcfunction__..mcfunction.json

@@ -0,0 +1,8 @@
+{
+  "path": "..mcfunction",
+  "suffixes": [],
+  "modified_suffixes": [
+    ".",
+    ".mcfunction"
+  ]
+}

tests/snapshots/modified_suffixes__paths_suffixes_mcfunction__.mcfunction.json

@@ -0,0 +1,7 @@
+{
+  "path": ".mcfunction",
+  "suffixes": [],
+  "modified_suffixes": [
+    ".mcfunction"
+  ]
+}

tests/snapshots/modified_suffixes__paths_suffixes_py_mcfunction__.py.mcfunction.json

@@ -0,0 +1,10 @@
+{
+  "path": ".py.mcfunction",
+  "suffixes": [
+    ".mcfunction"
+  ],
+  "modified_suffixes": [
+    ".py",
+    ".mcfunction"
+  ]
+}

tests/test_modified_suffixes.py

@@ -0,0 +1,25 @@
+import pytest
+from pytest_insta import SnapshotFixture
+from beet.library.utils import modified_suffixes
+from pathlib import Path
+
+
+PATHS = [
+    "load.mcfunction",
+    ".mcfunction",
+    "..mcfunction",
+    "load.py.mcfunction",
+    ".py.mcfunction",
+    "aaa...mcfunction",
+    "...mcfunction",
+]
+
+
+@pytest.mark.parametrize("path", PATHS)
+def test_paths_suffixes(snapshot: SnapshotFixture, path: str):
+    real_path = Path(path)
+    assert snapshot(f"{path}.json") == {
+        "path": path,
+        "suffixes": list(real_path.suffixes),
+        "modified_suffixes": list(modified_suffixes(real_path)),
+    }