artefacts-cli 0.17.2__tar.gz → 0.17.3__tar.gz

{artefacts_cli-0.17.2 → artefacts_cli-0.17.3}/CHANGELOG.md

@@ -7,6 +7,17 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.17.3] - 2026-11-12
+
+### Fixed
+
+- Fixed bug where client may in some cases upload `artefacts.yaml` twice.
+- `artefacts run` crashed with `ValueError: placeholder too large for max width` before uploading any artifact when the terminal left the progress bar 1-2 columns for the file name (#576)
+
+### Removed
+
+- Last remnants of ROS1 support: The unused `rospkg` dependency and ROS1-specific documentation and support files (#572)
+
 ## [0.17.2] - 2026-09-15
 
 ### Fixed
@@ -336,7 +347,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - Local metrics errors do not block publication of results.
 - Introduction of Black formatting.
 
-[unreleased]: https://github.com/art-e-fact/artefacts-client/compare/0.16.0...HEAD
+[unreleased]: https://github.com/art-e-fact/artefacts-client/compare/0.17.3...HEAD
+[0.17.3]: https://github.com/art-e-fact/artefacts-client/releases/tag/0.17.3
 [0.16.0]: https://github.com/art-e-fact/artefacts-client/releases/tag/0.16.0
 [0.15.0]: https://github.com/art-e-fact/artefacts-client/releases/tag/0.15.0
 [0.14.0]: https://github.com/art-e-fact/artefacts-client/releases/tag/0.14.0

{artefacts_cli-0.17.2/artefacts_cli.egg-info → artefacts_cli-0.17.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: artefacts_cli
-Version: 0.17.2
+Version: 0.17.3
 Author-email: FD <fabian@artefacts.com>, AGC <alejandro@artefacts.com>, TN <tomo@artefacts.com>, EP <eric@artefacts.com>
 License-Expression: Apache-2.0
 Project-URL: Homepage, https://github.com/art-e-fact/artefacts-client
@@ -21,10 +21,9 @@ Requires-Dist: junitparser>=2.5
 Requires-Dist: pyjwt
 Requires-Dist: mcap
 Requires-Dist: mcap-ros2-support
-Requires-Dist: psutil==7.0.0
+Requires-Dist: psutil<7.3,>=7.2
 Requires-Dist: PyYAML>=6.0
 Requires-Dist: requests>=2.27.1
-Requires-Dist: rospkg
 Requires-Dist: semver==3.0.4
 Requires-Dist: urllib3>=2.4.0
 Requires-Dist: attrs>=21.3.0
@@ -36,10 +35,6 @@ Requires-Dist: awscli; extra == "dev"
 Requires-Dist: babel==2.17.0; extra == "dev"
 Requires-Dist: build; extra == "dev"
 Requires-Dist: lark; extra == "dev"
-Requires-Dist: mkdocs-click==0.8.0; extra == "dev"
-Requires-Dist: mkdocs-material==8.5.6; extra == "dev"
-Requires-Dist: mkdocs-mermaid2-plugin==0.6.0; extra == "dev"
-Requires-Dist: mkdocs==1.4.2; extra == "dev"
 Requires-Dist: packaging; extra == "dev"
 Requires-Dist: pre-commit; extra == "dev"
 Requires-Dist: pyre-check; extra == "dev"
@@ -116,13 +111,7 @@ See [the configuration syntax documentation](docs/configuration-syntax.md)
 
 ## User docs
 
-You can serve docs locally using mkdocs:
-
-```
-mkdocs serve -a 127.0.0.1:7000
-```
-
-The docs are automatically deployed by the documentation workflow.
+Docs are managed in a [separate dedicated repository](https://github.com/art-e-fact/artefacts-docs), and available for consulation at https://docs.artefacts.com.
 
 ## Development notes
 

{artefacts_cli-0.17.2 → artefacts_cli-0.17.3}/README.md

@@ -58,13 +58,7 @@ See [the configuration syntax documentation](docs/configuration-syntax.md)
 
 ## User docs
 
-You can serve docs locally using mkdocs:
-
-```
-mkdocs serve -a 127.0.0.1:7000
-```
-
-The docs are automatically deployed by the documentation workflow.
+Docs are managed in a [separate dedicated repository](https://github.com/art-e-fact/artefacts-docs), and available for consulation at https://docs.artefacts.com.
 
 ## Development notes
 

{artefacts_cli-0.17.2 → artefacts_cli-0.17.3}/README_INTERNAL.md

@@ -98,10 +98,4 @@ Releases are managed with Twine through the `bin/release` script. By default it
 
 ## User docs
 
-You can serve docs locally using mkdocs:
-
-```
-mkdocs serve -a 127.0.0.1:7000
-```
-
-The docs are automatically deployed by the documentation workflow.
+Docs are managed in a [separate dedicated repository](https://github.com/art-e-fact/artefacts-docs), and available for consulation at https://docs.artefacts.com.

{artefacts_cli-0.17.2 → artefacts_cli-0.17.3}/artefacts/cli/__init__.py

@@ -410,11 +410,12 @@ class Run:
                 terminal_width, _ = shutil.get_terminal_size()
                 width = 30
                 file_name_max = max([0, terminal_width - len(label) - width - 10])
+                placeholder = "..."
 
                 def clip_file_name(fname: str) -> str:
-                    if file_name_max > 0:
+                    if file_name_max >= len(placeholder):
                         return textwrap.shorten(
-                            fname, width=file_name_max, placeholder="..."
+                            fname, width=file_name_max, placeholder=placeholder
                         )
                     else:
                         return ""

{artefacts_cli-0.17.2 → artefacts_cli-0.17.3}/artefacts/cli/blob/__init__.py

@@ -173,7 +173,7 @@ def scan_and_hash(
     """
 
     # Parse .artefactsignore
-    ignore_list_str = ".git"
+    ignore_list_str = "\n".join(ALWAYS_IGNORED)
 
     dotartefactsignore = Path(folder) / Path(".artefactsignore")
 
@@ -216,9 +216,10 @@ def scan_and_hash(
                 )
             else:
                 logger.debug(
-                    "file blob %s dest %s",
+                    "file blob %s dest %s: %s",
                     blob.short_hash,
                     blob.destination,
+                    blob.files,
                 )
             if blob.short_hash in existing:
                 final_blobs.append(Blob.from_metadata(blob, existing[blob.short_hash]))

{artefacts_cli-0.17.2 → artefacts_cli-0.17.3}/artefacts/cli/version.py

@@ -18,7 +18,7 @@ version_tuple: tuple[int | str, ...]
 commit_id: str | None
 __commit_id__: str | None
 
-__version__ = version = '0.17.2'
-__version_tuple__ = version_tuple = (0, 17, 2)
+__version__ = version = '0.17.3'
+__version_tuple__ = version_tuple = (0, 17, 3)
 
-__commit_id__ = commit_id = 'gdf013aa6f'
+__commit_id__ = commit_id = 'geafac1697'

{artefacts_cli-0.17.2 → artefacts_cli-0.17.3}/artefacts.yaml

@@ -1,6 +1,6 @@
 version: 0.1.0
 
-project: artefacts/cli-run-remote-testing
+project: artefacts-testing/cli-run-remote-testing
 
 on:
   push:

{artefacts_cli-0.17.2 → artefacts_cli-0.17.3/artefacts_cli.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: artefacts_cli
-Version: 0.17.2
+Version: 0.17.3
 Author-email: FD <fabian@artefacts.com>, AGC <alejandro@artefacts.com>, TN <tomo@artefacts.com>, EP <eric@artefacts.com>
 License-Expression: Apache-2.0
 Project-URL: Homepage, https://github.com/art-e-fact/artefacts-client
@@ -21,10 +21,9 @@ Requires-Dist: junitparser>=2.5
 Requires-Dist: pyjwt
 Requires-Dist: mcap
 Requires-Dist: mcap-ros2-support
-Requires-Dist: psutil==7.0.0
+Requires-Dist: psutil<7.3,>=7.2
 Requires-Dist: PyYAML>=6.0
 Requires-Dist: requests>=2.27.1
-Requires-Dist: rospkg
 Requires-Dist: semver==3.0.4
 Requires-Dist: urllib3>=2.4.0
 Requires-Dist: attrs>=21.3.0
@@ -36,10 +35,6 @@ Requires-Dist: awscli; extra == "dev"
 Requires-Dist: babel==2.17.0; extra == "dev"
 Requires-Dist: build; extra == "dev"
 Requires-Dist: lark; extra == "dev"
-Requires-Dist: mkdocs-click==0.8.0; extra == "dev"
-Requires-Dist: mkdocs-material==8.5.6; extra == "dev"
-Requires-Dist: mkdocs-mermaid2-plugin==0.6.0; extra == "dev"
-Requires-Dist: mkdocs==1.4.2; extra == "dev"
 Requires-Dist: packaging; extra == "dev"
 Requires-Dist: pre-commit; extra == "dev"
 Requires-Dist: pyre-check; extra == "dev"
@@ -116,13 +111,7 @@ See [the configuration syntax documentation](docs/configuration-syntax.md)
 
 ## User docs
 
-You can serve docs locally using mkdocs:
-
-```
-mkdocs serve -a 127.0.0.1:7000
-```
-
-The docs are automatically deployed by the documentation workflow.
+Docs are managed in a [separate dedicated repository](https://github.com/art-e-fact/artefacts-docs), and available for consulation at https://docs.artefacts.com.
 
 ## Development notes
 

{artefacts_cli-0.17.2 → artefacts_cli-0.17.3}/artefacts_cli.egg-info/SOURCES.txt

@@ -160,6 +160,7 @@ tests/cli/test_other.py
 tests/cli/test_parameters.py
 tests/cli/test_pytest.py
 tests/cli/test_reporter.py
+tests/cli/test_ros1_not_supported.py
 tests/cli/test_ros2.py
 tests/cli/test_signals.py
 tests/cli/test_utils.py

{artefacts_cli-0.17.2 → artefacts_cli-0.17.3}/artefacts_cli.egg-info/requires.txt

@@ -9,10 +9,9 @@ junitparser>=2.5
 pyjwt
 mcap
 mcap-ros2-support
-psutil==7.0.0
+psutil<7.3,>=7.2
 PyYAML>=6.0
 requests>=2.27.1
-rospkg
 semver==3.0.4
 urllib3>=2.4.0
 attrs>=21.3.0
@@ -25,10 +24,6 @@ awscli
 babel==2.17.0
 build
 lark
-mkdocs-click==0.8.0
-mkdocs-material==8.5.6
-mkdocs-mermaid2-plugin==0.6.0
-mkdocs==1.4.2
 packaging
 pre-commit
 pyre-check

{artefacts_cli-0.17.2 → artefacts_cli-0.17.3}/artefacts_cli.egg-info/scm_file_list.json

@@ -163,6 +163,7 @@
     "tests/cli/test_parameters.py",
     "tests/cli/test_pytest.py",
     "tests/cli/test_reporter.py",
+    "tests/cli/test_ros1_not_supported.py",
     "tests/cli/test_ros2.py",
     "tests/cli/test_signals.py",
     "tests/cli/test_utils.py",

artefacts_cli-0.17.3/artefacts_cli.egg-info/scm_version.json

@@ -0,0 +1,8 @@
+{
+  "tag": "0.17.3",
+  "distance": 0,
+  "node": "geafac16970f8a2a1da60b1b53c945f67baa8980a",
+  "dirty": false,
+  "branch": "HEAD",
+  "node_date": "2026-11-12"
+}

{artefacts_cli-0.17.2 → artefacts_cli-0.17.3}/pyproject.toml

@@ -30,10 +30,9 @@ dependencies = [
   "pyjwt",
   "mcap",
   "mcap-ros2-support",
-  "psutil==7.0.0",
+  "psutil>=7.2,<7.3",
   "PyYAML>=6.0",
   "requests>=2.27.1",
-  "rospkg",
   "semver==3.0.4",
   "urllib3>=2.4.0",
 
@@ -62,10 +61,6 @@ dev = [
   "babel==2.17.0",
   "build",
   "lark",
-  "mkdocs-click==0.8.0",
-  "mkdocs-material==8.5.6",
-  "mkdocs-mermaid2-plugin==0.6.0",
-  "mkdocs==1.4.2",
   "packaging",
   "pre-commit",
   "pyre-check",

artefacts_cli-0.17.3/tests/cli/test_ros1_not_supported.py

@@ -0,0 +1,48 @@
+from pathlib import Path
+
+from artefacts.cli.app import run
+from artefacts.cli.constants import DEPRECATED_FRAMEWORKS, SUPPORTED_FRAMEWORKS
+
+
+def test_no_ros1_framework_in_supported_frameworks():
+    """ROS1 frameworks are no longer officially supported."""
+    ros1_frameworks = [f for f in SUPPORTED_FRAMEWORKS if "ros1" in str(f).lower()]
+
+    assert ros1_frameworks == []
+
+
+def test_no_ros1_framework_in_deprecated_frameworks():
+    """ROS1 frameworks have no deprecated migration path either."""
+    ros1_frameworks = [f for f in DEPRECATED_FRAMEWORKS if "ros1" in str(f).lower()]
+
+    assert ros1_frameworks == []
+
+
+def test_run_with_ros1_framework_warns_not_supported(
+    cli_runner, authorised_project_with_editable_conf
+):
+    """A job configured with a ROS1 framework is treated as not supported."""
+    authorised_project_with_editable_conf.config["jobs"]["simple_job"]["runtime"] = {
+        "framework": "ros1:noetic",
+        "simulator": "turtlesim",
+    }
+
+    with authorised_project_with_editable_conf as config_path:
+        result = cli_runner.invoke(
+            run,
+            ["simple_job", "--config", config_path, "--dryrun"],
+        )
+
+    assert result.exit_code == 0
+    assert (
+        "WARNING: framework: 'ros1:noetic' is not officially supported. Attempting run."
+        in result.output
+    )
+
+
+def test_rospkg_not_a_project_dependency():
+    """The ROS1 Python package manager must not be a declared dependency."""
+    pyproject = Path(__file__).parents[2] / "pyproject.toml"
+    content = pyproject.read_text()
+
+    assert '"rospkg"' not in content

{artefacts_cli-0.17.2 → artefacts_cli-0.17.3}/tests/cli/test_warp_run.py

@@ -28,6 +28,30 @@ def test_run_with_upload_on_stop(mocker, artefacts_run):
     assert spy.call_args.kwargs["body"].uploads
 
 
+@pytest.mark.parametrize("columns", range(60, 90))
+def test_run_upload_progress_fits_terminal_width(
+    mocker, monkeypatch, artefacts_run, tmp_path, columns
+):
+    """
+    Check that uploading works at any terminal width, including the widths
+    where the progress bar leaves the file name less room than the "..."
+    placeholder needs (1-2 columns: 71-72 for this 30-character label).
+    """
+    artifact = tmp_path / "artifact.txt"
+    artifact.write_text("content")
+    artefacts_run.log_single_artifact(artifact)
+    monkeypatch.setenv("COLUMNS", str(columns))
+    # Click only draws the bar, and the file name in it, on a TTY
+    mocker.patch("click._termui_impl.isatty", return_value=True)
+    mocker.patch.object(artefacts_run.job.api_conf, "sdk")
+    upload = mocker.patch.object(artefacts_run.job.api_conf, "upload")
+    artefacts_run.job.dryrun = False
+
+    artefacts_run.stop()
+
+    upload.assert_called_once()
+
+
 def test_run_without_upload_on_stop(mocker, artefacts_run):
     """
     Check that `uploads` list is not registered to the API when stopping a run

artefacts_cli-0.17.2/artefacts_cli.egg-info/scm_version.json

@@ -1,8 +0,0 @@
-{
-  "tag": "0.17.2",
-  "distance": 0,
-  "node": "gdf013aa6f0b30d038848380fa8ea79e5de5190f7",
-  "dirty": false,
-  "branch": "HEAD",
-  "node_date": "2026-09-15"
-}