development-framework-cli 1.8.0__tar.gz → 1.8.1__tar.gz

development_framework_cli-1.8.1/.release-please-manifest.json

@@ -0,0 +1,3 @@
+{
+  ".": "1.8.1"
+}

{development_framework_cli-1.8.0 → development_framework_cli-1.8.1}/CHANGELOG.md

@@ -1,5 +1,13 @@
 # Changelog
 
+## [1.8.1](https://github.com/equinor/dm-cli/compare/v1.8.0...v1.8.1) (2026-11-09)
+
+
+### Bug Fixes
+
+* handle DMSS addresses and zip entry names as POSIX paths on Windows ([075b785](https://github.com/equinor/dm-cli/commit/075b785d76d3bae4fb5243793d69ec0fa98c06ac))
+* handle DMSS addresses and zip entry names as POSIX paths on Windows ([b6f652b](https://github.com/equinor/dm-cli/commit/b6f652b06a034509c9416a3b247370bd66b99733))
+
 ## [1.8.0](https://github.com/equinor/dm-cli/compare/v1.7.3...v1.8.0) (2026-08-06)
 
 

{development_framework_cli-1.8.0 → development_framework_cli-1.8.1}/PKG-INFO

@@ -1,6 +1,6 @@
-Metadata-Version: 2.4
+Metadata-Version: 2.5
 Name: development-framework-cli
-Version: 1.8.0
+Version: 1.8.1
 Summary: CLI tool for working with Data Modelling Storage Service
 Project-URL: Homepage, https://github.com/equinor/dm-cli
 Project-URL: Repository, https://github.com/equinor/dm-cli

development_framework_cli-1.8.1/dm_cli/__init__.py

@@ -0,0 +1 @@
+VERSION = "1.8.1"  # x-release-please-version

{development_framework_cli-1.8.0 → development_framework_cli-1.8.1}/dm_cli/cli.py

@@ -87,7 +87,7 @@ def import_plugin_blueprints(
 def create_lookup(
     name: Annotated[str, typer.Argument(help="Name of the lookup (application context) to create.")],
     paths: Annotated[
-        List[Path],
+        List[str],
         typer.Argument(
             help="one or more remote location for recursively looking for RecipeLinks. "
             "A remote location is a path in DMSS on the format <dataSource>/<rootPackage>/<subPacakge>/..."
@@ -97,8 +97,8 @@ def create_lookup(
     """
     Create a named Ui-/StorageRecipe-lookup-table from all RecipeLinks in a package existing in DMSS (requires admin privileges).
     """
-    # TODO change type of paths argument to be list of str to avoid this path as strings conversion below
-    paths_as_strings = [str(path) for path in paths]
+    # DMSS addresses, not local paths: a Path would turn '/' into '\' on Windows.
+    paths_as_strings = [path.rstrip("/") for path in paths]
     print(f"Creating lookup table from paths: {paths_as_strings}")
     dmss_exception_wrapper(dmss_api.create_lookup, recipe_package=paths_as_strings, application=name)
 

{development_framework_cli-1.8.0 → development_framework_cli-1.8.1}/dm_cli/command_group/entities.py

@@ -42,7 +42,7 @@ def import_entity(
     source_path = Path(source)
     destination = destination.rstrip("/\\")
     # Not replacing a package, but appending to. Can therefore not use "fast mode"
-    fast = destination_is_root(Path(destination))
+    fast = destination_is_root(destination)
 
     def inner_import():
         if source_path.is_dir():

{development_framework_cli-1.8.0 → development_framework_cli-1.8.1}/dm_cli/import_entity.py

@@ -59,7 +59,7 @@ def import_document(source_path: Path, destination: str, document: dict):
 
 
 def import_single_entity(source_path: Path, destination: str, validate: bool = False):
-    ensure_package_structure(Path(destination))
+    ensure_package_structure(destination)
     print(f"Importing ENTITY '{source_path.name}' --> '{destination}'")
 
     try:  # Load the JSON document
@@ -96,8 +96,6 @@ def import_folder_entity(
     raw_package_import: bool = False,
     resolve_local_ids: bool = False,
 ) -> dict:
-    destination_path = Path(destination)
-
     # Check if target already exists on remote. Then delete or raise exception
     target = f"{destination}/{source_path.name}"
     exists = dmss_api.document_check(target)
@@ -108,9 +106,9 @@ def import_folder_entity(
         dmss_api.document_remove(target)
 
     dependencies = {}
-    is_root = destination_is_root(destination_path)
+    is_root = destination_is_root(destination)
     if not is_root:
-        ensure_package_structure(destination_path)
+        ensure_package_structure(destination)
         dependencies = dependencies_of(dmss_api.export_meta(destination))
 
     memory_file = io.BytesIO()
@@ -172,7 +170,7 @@ def import_folders_as_one_package(source_path: Path, destination: str, package_n
         console.print(f"'{target}' already exists.  Replacing it...", style="dark_orange")
         dmss_api.document_remove(target)
 
-    ensure_package_structure(Path(destination))
+    ensure_package_structure(destination)
     dependencies = dependencies_of(dmss_api.export_meta(destination))
 
     entries = sorted(source_path.iterdir())

{development_framework_cli-1.8.0 → development_framework_cli-1.8.1}/dm_cli/import_package.py

@@ -3,7 +3,7 @@ import json
 import os
 from concurrent.futures import ThreadPoolExecutor, as_completed
 from json import JSONDecodeError
-from pathlib import Path
+from pathlib import Path, PurePosixPath
 from typing import Callable, Dict, List, Tuple
 from uuid import uuid4
 
@@ -99,7 +99,7 @@ def _upload_documents(description: str, data_source: str, documents: List[dict])
         run_with_progress(description, documents, lambda document: dmss_api.document_add_simple(data_source, document))
 
 
-def add_object_to_package(path: Path, package: Package, object: io.BytesIO) -> None:
+def add_object_to_package(path: PurePosixPath, package: Package, object: io.BytesIO) -> None:
     if len(path.parts) == 1:  # End of path means the actual document
         file = File(
             uid=str(uuid4()),  # This UID will be the data source ID for this file
@@ -115,11 +115,11 @@ def add_object_to_package(path: Path, package: Package, object: io.BytesIO) -> N
         sub_folder = Package(name=path.parts[0], parent=package)
         package.content.append(sub_folder)
 
-    new_path = str(path).split("/", 1)[1]  # Remove first element in path before stepping down
-    return add_object_to_package(Path(new_path), sub_folder, object)
+    new_path = PurePosixPath(*path.parts[1:])  # Remove first element in path before stepping down
+    return add_object_to_package(new_path, sub_folder, object)
 
 
-def add_file_to_package(path: Path, package: Package, document: dict) -> None:
+def add_file_to_package(path: PurePosixPath, package: Package, document: dict) -> None:
     if len(path.parts) == 1:  # End of path means the actual document
         if path.name.endswith("package.json"):
             # if document is a package.json file, add meta info to package instead of adding it to content list.
@@ -134,11 +134,11 @@ def add_file_to_package(path: Path, package: Package, document: dict) -> None:
         sub_folder = Package(name=path.parts[0], parent=package)
         package.content.append(sub_folder)
 
-    new_path = str(path).split("/", 1)[1]  # Remove first element in path before stepping down
-    return add_file_to_package(Path(new_path), sub_folder, document)
+    new_path = PurePosixPath(*path.parts[1:])  # Remove first element in path before stepping down
+    return add_file_to_package(new_path, sub_folder, document)
 
 
-def add_package_to_package(path: Path, package: Package) -> None:
+def add_package_to_package(path: PurePosixPath, package: Package) -> None:
     if len(path.parts) == 1:
         package.content.append(Package(name=path.parts[0], parent=package))
         return
@@ -148,8 +148,8 @@ def add_package_to_package(path: Path, package: Package) -> None:
         sub_folder = Package(name=path.parts[0], parent=package)
         package.content.append(sub_folder)
 
-    new_path = str(path).split("/", 1)[1]  # Remove first element in path before stepping down
-    return add_package_to_package(Path(new_path), sub_folder)
+    new_path = PurePosixPath(*path.parts[1:])  # Remove first element in path before stepping down
+    return add_package_to_package(new_path, sub_folder)
 
 
 def import_package_tree(package: Package, destination: str, raw_package_import: bool, resolve_local_ids: bool) -> None:

{development_framework_cli-1.8.0 → development_framework_cli-1.8.1}/dm_cli/package_tree_from_zip.py

@@ -1,7 +1,7 @@
 import io
 import json
 from json import JSONDecodeError
-from pathlib import Path
+from pathlib import Path, PurePosixPath
 from typing import Dict, Union
 from zipfile import ZipFile
 
@@ -58,26 +58,27 @@ def package_tree_from_zip(
             is_root=is_root,
             meta=package_entity.get("_meta_"),
         )
-        # Construct a nested Package object of the package to import
+        # Construct a nested Package object of the package to import.
+        # Zip entry names and DMSS addresses always use '/', so they are handled as POSIX paths on every OS.
         for file_info in zip_file.filelist:
             filename = file_info.filename.split("/", 1)[1]  # Remove RootPackage prefix
             if file_info.is_dir():
                 if filename == "":  # Skip rootPackage
                     continue
-                add_package_to_package(Path(filename), root_package)
+                add_package_to_package(PurePosixPath(filename), root_package)
                 continue
-            if Path(filename).suffix != ".json":
+            if PurePosixPath(filename).suffix != ".json":
                 file_like = io.BytesIO(zip_file.read(f"{folder_name}/{filename}"))
-                file_like.name = Path(filename).name  # stem
-                file_like.destination = Path(f"/{destination}/{folder_name}/{filename}").parent
-                add_object_to_package(Path(filename), root_package, file_like)
+                file_like.name = PurePosixPath(filename).name  # stem
+                file_like.destination = PurePosixPath(f"/{destination}/{folder_name}/{filename}").parent
+                add_object_to_package(PurePosixPath(filename), root_package, file_like)
                 continue
             try:
                 json_doc = json.loads(zip_file.read(f"{folder_name}/{filename}"))
             except JSONDecodeError:
                 raise Exception(f"Failed to load the file '{filename}' as a JSON document")
 
-            add_file_to_package(Path(filename), root_package, json_doc)
+            add_file_to_package(PurePosixPath(filename), root_package, json_doc)
 
             # Add dependencies from entity to the global dependencies list
             dependencies = concat_dependencies(

{development_framework_cli-1.8.0 → development_framework_cli-1.8.1}/dm_cli/utils/reference.py

@@ -1,5 +1,6 @@
-from os.path import normpath
+import os
 from pathlib import Path
+from posixpath import normpath  # references are DMSS addresses, always '/'-separated
 from typing import Dict, List, Union
 
 from ..dmss import ApplicationException
@@ -81,8 +82,9 @@ def replace_relative_references(
                 if value["referenceType"] == ReferenceTypes.LINK.value:
                     value["address"] = _resolve_reference(value["address"])
                 else:
-                    # Handle storage references
-                    local_file_path = "/".join(str(source_path).split("/")[:-1])
+                    # Handle storage references. source_path is a folder on disk: take its parent with
+                    # the OS path rules, the address itself is appended as-is ('/' works on Windows too).
+                    local_file_path = os.path.dirname(str(source_path)) if source_path else ""
                     if value["address"][0] == ".":
                         raise ApplicationException(
                             f"Relative references by . are not supported", data=value, debug=file_path

{development_framework_cli-1.8.0 → development_framework_cli-1.8.1}/dm_cli/utils/utils.py

@@ -1,7 +1,7 @@
 import json
 import os
 import pprint
-from pathlib import Path
+from pathlib import Path, PurePosixPath
 from typing import Callable, Dict, List
 
 import typer
@@ -68,7 +68,7 @@ def document_already_exists(api_exception: ApiException) -> bool:
     return False
 
 
-def add_package_to_path(name: str, path: Path):
+def add_package_to_path(name: str, path: PurePosixPath):
     package = Package(name, is_root=len(path.parts) == 2)
     dmss_api.document_add(
         str(path.parent),
@@ -102,14 +102,16 @@ def replace_global_addresses(
     return document
 
 
-def destination_is_root(destination: Path) -> bool:
-    if len(destination.parts) > 1:
+def destination_is_root(destination: str | PurePosixPath) -> bool:
+    if len(PurePosixPath(destination).parts) > 1:
         return False
     return True
 
 
-def ensure_package_structure(path: Path):
+def ensure_package_structure(path: str | PurePosixPath):
     """Create any missing packages in the provided path (mkdir -R)"""
+    # A DMSS address, not a path on disk: keep it '/'-separated on every OS.
+    path = PurePosixPath(path)
     try:
         dmss_api.document_get(f"dmss://{path}/")
     except NotFoundException as e:

{development_framework_cli-1.8.0 → development_framework_cli-1.8.1}/tests/unit/test_replace_references.py

@@ -98,7 +98,7 @@ class ReplaceReferences(unittest.TestCase):
         data_source_id, package = dst_path.split("/", 1)
 
         # Load the document
-        document = test_documents[str(src_path)]
+        document = test_documents[src_path.as_posix()]
         dependencies = concat_dependencies(document.get("_meta_", {}).get("dependencies", []), {}, src_path.name)
 
         # Replace references
@@ -132,7 +132,7 @@ class ReplaceReferences(unittest.TestCase):
         data_source_id, package = dst_path.split("/", 1)
 
         # Load the document
-        document = test_documents[str(src_path)]
+        document = test_documents[src_path.as_posix()]
         dependencies = concat_dependencies(document.get("_meta_", {}).get("dependencies", []), {}, src_path.name)
 
         with self.assertRaises(ApplicationException):

development_framework_cli-1.8.0/.release-please-manifest.json

@@ -1,3 +0,0 @@
-{
-  ".": "1.8.0"
-}

development_framework_cli-1.8.0/dm_cli/__init__.py

@@ -1 +0,0 @@
-VERSION = "1.8.0"  # x-release-please-version