launchline 0.2.0__tar.gz → 0.2.1__tar.gz

launchline-0.2.1/.generated-go-semantic-release-changelog.md

@@ -0,0 +1,18 @@
+## 0.2.1 (2026-03-30)
+
+#### Bug Fixes
+
+* add force-patch input to release workflow (1954a256)
+
+#### Reverts
+
+* remove array syntax for command config key (79d5f175)
+
+#### Documentation
+
+* add PyPI badge and update changelog for v0.2.0 (4aa7e495)
+
+#### Tests
+
+* improve test naming and add diagnostic assertion messages (3169b7cd)
+

{launchline-0.2.0 → launchline-0.2.1}/.github/workflows/release.yml

@@ -2,6 +2,11 @@ name: Release
 
 on:
   workflow_dispatch:
+    inputs:
+      force-patch:
+        description: "Force a patch version bump (use when only docs/test/chore commits exist)"
+        type: boolean
+        default: false
 
 permissions:
   contents: write
@@ -20,6 +25,7 @@ jobs:
         with:
           github-token: ${{ secrets.GITHUB_TOKEN }}
           allow-initial-development-versions: true
+          force-bump-patch-version: ${{ inputs.force-patch }}
           dry: true
 
       - name: Require new version

{launchline-0.2.0 → launchline-0.2.1}/CHANGELOG.md

@@ -7,11 +7,21 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+### Removed
+
+- Reverted array syntax for `command` config key; `command` is string-only,
+  use `args` for arguments
+
+### Changed
+
+- Improved test naming to follow `[unit]_[scenario]_[expectedResult]` pattern
+- Added diagnostic failure messages to all test assertions
+- Release workflow now supports `force-patch` input for non-bumping commits
+
+## [0.2.0] - 2026-03-30
+
 ### Added
 
-- `command` config key now accepts a list of strings (e.g.,
-  `command = ["ssh", "-t", "my server"]`) for easier argument handling,
-  especially with values containing spaces
 - `--config-path` CLI flag to print the active configuration file path
 
 ### Fixed
@@ -19,6 +29,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - PyPI package page now renders README images correctly (paths are rewritten
   to absolute URLs during the release build)
 - Release artifacts no longer accidentally include non-distribution files
+- Release workflow now produces correct version numbers (fixed dirty-tree
+  version detection via `SETUPTOOLS_SCM_PRETEND_VERSION`)
 
 ## [0.1.0] - 2026-03-30
 

{launchline-0.2.0 → launchline-0.2.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: launchline
-Version: 0.2.0
+Version: 0.2.1
 Summary: A lightweight terminal launcher for interactive CLI tools.
 Project-URL: Homepage, https://github.com/mikejhill/launchline
 Project-URL: Repository, https://github.com/mikejhill/launchline
@@ -24,12 +24,14 @@ Description-Content-Type: text/markdown
 
 [![CI](https://github.com/mikejhill/launchline/actions/workflows/ci.yml/badge.svg)](https://github.com/mikejhill/launchline/actions/workflows/ci.yml)
 [![Release](https://github.com/mikejhill/launchline/actions/workflows/release.yml/badge.svg)](https://github.com/mikejhill/launchline/actions/workflows/release.yml)
+[![PyPI](https://img.shields.io/pypi/v/launchline)](https://pypi.org/project/launchline/)
+[![Python](https://img.shields.io/pypi/pyversions/launchline)](https://pypi.org/project/launchline/)
 
 A lightweight terminal launcher for interactive CLI tools. Define your frequently
 used commands in a TOML config file, then pick them from a fuzzy-searchable menu
 instead of remembering flags and paths.
 
-![LaunchLine demo](https://raw.githubusercontent.com/mikejhill/launchline/v0.2.0/docs/images/demo.gif)
+![LaunchLine demo](https://raw.githubusercontent.com/mikejhill/launchline/v0.2.1/docs/images/demo.gif)
 
 ## Why
 
@@ -50,7 +52,7 @@ instead.
 
 |           Fuzzy search            |         Arrow-key navigation          |
 | :-------------------------------: | :-----------------------------------: |
-| ![Search](https://raw.githubusercontent.com/mikejhill/launchline/v0.2.0/docs/images/search.png) | ![Navigate](https://raw.githubusercontent.com/mikejhill/launchline/v0.2.0/docs/images/navigate.png) |
+| ![Search](https://raw.githubusercontent.com/mikejhill/launchline/v0.2.1/docs/images/search.png) | ![Navigate](https://raw.githubusercontent.com/mikejhill/launchline/v0.2.1/docs/images/navigate.png) |
 
 ## Installation
 
@@ -123,24 +125,21 @@ Config file format is [TOML](https://toml.io). The file has an optional
 
 Each `[[entries]]` table defines one launchable tool:
 
-| Key                 | Type                      | Required | Default | Description                                                    |
-| ------------------- | ------------------------- | -------- | ------- | -------------------------------------------------------------- |
-| `name`              | string                    | yes      | —       | Display name shown in the menu                                 |
-| `command`           | string or list of strings | yes      | —       | Executable to run (with optional inline arguments)             |
-| `args`              | list of strings           | no       | `[]`    | Additional arguments appended after any inline command args    |
-| `description`       | string                    | no       | `""`    | Short description shown next to the name                       |
-| `working_directory` | string                    | no       | —       | Working directory for the subprocess                           |
-| `env`               | table                     | no       | `{}`    | Extra environment variables (`KEY = "value"`)                  |
+| Key                 | Type            | Required | Default | Description                                                    |
+| ------------------- | --------------- | -------- | ------- | -------------------------------------------------------------- |
+| `name`              | string          | yes      | —       | Display name shown in the menu                                 |
+| `command`           | string          | yes      | —       | Executable to run                                              |
+| `args`              | list of strings | no       | `[]`    | Arguments passed to the command                                |
+| `description`       | string          | no       | `""`    | Short description shown next to the name                       |
+| `working_directory` | string          | no       | —       | Working directory for the subprocess                           |
+| `env`               | table           | no       | `{}`    | Extra environment variables (`KEY = "value"`)                  |
 
 ### Validation Rules
 
 - At least one `[[entries]]` table is required.
 - Every entry must have both `name` and `command`.
-- `command` may be a string (`"pwsh"`) or a list (`["ssh", "-t", "host"]`).
-  When a list, the first element is the executable and the rest are arguments.
 - `on_exit` must be `"restart"` or `"exit"`.
-- `args` must be a list (not a bare string). These are appended after any
-  arguments provided inline in a `command` list.
+- `args` must be a list (not a bare string).
 - `env` must be a TOML table (not a string or list).
 - If `working_directory` does not exist at load time, it is silently reset to
   `None` (a warning is logged).
@@ -171,10 +170,6 @@ name = "PowerShell"
 command = "pwsh"
 args = ["-NoProfile"]
 env = { TERM = "xterm-256color" }
-
-[[entries]]
-name = "Remote Server"
-command = ["ssh", "-t", "my server", "bash"]
 ```
 
 ## Usage

{launchline-0.2.0 → launchline-0.2.1}/README.md

@@ -2,12 +2,14 @@
 
 [![CI](https://github.com/mikejhill/launchline/actions/workflows/ci.yml/badge.svg)](https://github.com/mikejhill/launchline/actions/workflows/ci.yml)
 [![Release](https://github.com/mikejhill/launchline/actions/workflows/release.yml/badge.svg)](https://github.com/mikejhill/launchline/actions/workflows/release.yml)
+[![PyPI](https://img.shields.io/pypi/v/launchline)](https://pypi.org/project/launchline/)
+[![Python](https://img.shields.io/pypi/pyversions/launchline)](https://pypi.org/project/launchline/)
 
 A lightweight terminal launcher for interactive CLI tools. Define your frequently
 used commands in a TOML config file, then pick them from a fuzzy-searchable menu
 instead of remembering flags and paths.
 
-![LaunchLine demo](https://raw.githubusercontent.com/mikejhill/launchline/v0.2.0/docs/images/demo.gif)
+![LaunchLine demo](https://raw.githubusercontent.com/mikejhill/launchline/v0.2.1/docs/images/demo.gif)
 
 ## Why
 
@@ -28,7 +30,7 @@ instead.
 
 |           Fuzzy search            |         Arrow-key navigation          |
 | :-------------------------------: | :-----------------------------------: |
-| ![Search](https://raw.githubusercontent.com/mikejhill/launchline/v0.2.0/docs/images/search.png) | ![Navigate](https://raw.githubusercontent.com/mikejhill/launchline/v0.2.0/docs/images/navigate.png) |
+| ![Search](https://raw.githubusercontent.com/mikejhill/launchline/v0.2.1/docs/images/search.png) | ![Navigate](https://raw.githubusercontent.com/mikejhill/launchline/v0.2.1/docs/images/navigate.png) |
 
 ## Installation
 
@@ -101,24 +103,21 @@ Config file format is [TOML](https://toml.io). The file has an optional
 
 Each `[[entries]]` table defines one launchable tool:
 
-| Key                 | Type                      | Required | Default | Description                                                    |
-| ------------------- | ------------------------- | -------- | ------- | -------------------------------------------------------------- |
-| `name`              | string                    | yes      | —       | Display name shown in the menu                                 |
-| `command`           | string or list of strings | yes      | —       | Executable to run (with optional inline arguments)             |
-| `args`              | list of strings           | no       | `[]`    | Additional arguments appended after any inline command args    |
-| `description`       | string                    | no       | `""`    | Short description shown next to the name                       |
-| `working_directory` | string                    | no       | —       | Working directory for the subprocess                           |
-| `env`               | table                     | no       | `{}`    | Extra environment variables (`KEY = "value"`)                  |
+| Key                 | Type            | Required | Default | Description                                                    |
+| ------------------- | --------------- | -------- | ------- | -------------------------------------------------------------- |
+| `name`              | string          | yes      | —       | Display name shown in the menu                                 |
+| `command`           | string          | yes      | —       | Executable to run                                              |
+| `args`              | list of strings | no       | `[]`    | Arguments passed to the command                                |
+| `description`       | string          | no       | `""`    | Short description shown next to the name                       |
+| `working_directory` | string          | no       | —       | Working directory for the subprocess                           |
+| `env`               | table           | no       | `{}`    | Extra environment variables (`KEY = "value"`)                  |
 
 ### Validation Rules
 
 - At least one `[[entries]]` table is required.
 - Every entry must have both `name` and `command`.
-- `command` may be a string (`"pwsh"`) or a list (`["ssh", "-t", "host"]`).
-  When a list, the first element is the executable and the rest are arguments.
 - `on_exit` must be `"restart"` or `"exit"`.
-- `args` must be a list (not a bare string). These are appended after any
-  arguments provided inline in a `command` list.
+- `args` must be a list (not a bare string).
 - `env` must be a TOML table (not a string or list).
 - If `working_directory` does not exist at load time, it is silently reset to
   `None` (a warning is logged).
@@ -149,10 +148,6 @@ name = "PowerShell"
 command = "pwsh"
 args = ["-NoProfile"]
 env = { TERM = "xterm-256color" }
-
-[[entries]]
-name = "Remote Server"
-command = ["ssh", "-t", "my server", "bash"]
 ```
 
 ## Usage

{launchline-0.2.0 → launchline-0.2.1}/src/launchline/config.py

@@ -188,19 +188,7 @@ description = "PowerShell 7"
         entries: list[EntryConfig] = []
         for i, entry_raw in enumerate(raw_entries, 1):
             name = entry_raw.get("name", "")
-            raw_command = entry_raw.get("command", "")
-
-            if isinstance(raw_command, list):
-                if not raw_command:
-                    raise ConfigurationError(
-                        f"Entry {i}: 'command' list must not be empty."
-                    )
-                command = str(raw_command[0])
-                command_args = tuple(str(a) for a in raw_command[1:])
-            else:
-                command = str(raw_command)
-                command_args = ()
-
+            command = entry_raw.get("command", "")
             if not name or not command:
                 raise ConfigurationError(
                     f"Entry {i}: 'name' and 'command' are required."
@@ -212,8 +200,6 @@ description = "PowerShell 7"
                     f"Entry {i}: 'args' must be a list of strings."
                 )
 
-            combined_args = command_args + tuple(str(a) for a in raw_args)
-
             wd_raw = entry_raw.get("working_directory")
             wd: Path | None = None
             if wd_raw:
@@ -235,8 +221,8 @@ description = "PowerShell 7"
             entries.append(
                 EntryConfig(
                     name=str(name),
-                    command=command,
-                    args=combined_args,
+                    command=str(command),
+                    args=tuple(str(a) for a in raw_args),
                     description=str(entry_raw.get("description", "")),
                     working_directory=wd,
                     env={str(k): str(v) for k, v in raw_env.items()},

{launchline-0.2.0 → launchline-0.2.1}/tests/test_config.py

@@ -21,7 +21,7 @@ _README = Path(__file__).resolve().parent.parent / "README.md"
 class TestLoadConfig:
     """Tests for ConfigLoader.load."""
 
-    def test_loads_valid_config(self, sample_toml: Path) -> None:
+    def test_valid_toml_loads_all_entry_fields(self, sample_toml: Path) -> None:
         config = ConfigLoader.load(sample_toml)
         assert isinstance(config, LaunchLineConfig), (
             f"Expected LaunchLineConfig, got {type(config).__name__}"
@@ -30,7 +30,7 @@ class TestLoadConfig:
             f"Expected 2 entries, got {len(config.entries)}"
         )
 
-    def test_default_settings(self, tmp_path: Path) -> None:
+    def test_missing_settings_use_defaults(self, tmp_path: Path) -> None:
         cfg = tmp_path / "config.toml"
         cfg.write_text('[[entries]]\nname = "X"\ncommand = "x"\n', encoding="utf-8")
         config = ConfigLoader.load(cfg)
@@ -44,7 +44,7 @@ class TestLoadConfig:
         assert config.ghost_text is True, "Default ghost_text should be True"
         assert config.numeric_trigger is True, "Default numeric_trigger should be True"
 
-    def test_custom_settings(self, tmp_path: Path) -> None:
+    def test_custom_settings_override_defaults(self, tmp_path: Path) -> None:
         cfg = tmp_path / "config.toml"
         cfg.write_text(
             '[settings]\non_exit = "exit"\ntitle = "My Launcher"\n'
@@ -72,7 +72,9 @@ class TestLoadConfig:
             encoding="utf-8",
         )
         config = ConfigLoader.load(cfg)
-        assert config.entries[0].args == ("--flag", "val")
+        assert config.entries[0].args == ("--flag", "val"), (
+            f"Expected args ('--flag', 'val'), got {config.entries[0].args!r}"
+        )
 
     def test_entry_env_parsed(self, tmp_path: Path) -> None:
         cfg = tmp_path / "config.toml"
@@ -81,7 +83,9 @@ class TestLoadConfig:
             encoding="utf-8",
         )
         config = ConfigLoader.load(cfg)
-        assert config.entries[0].env == {"FOO": "bar"}
+        assert config.entries[0].env == {"FOO": "bar"}, (
+            f"Expected env {{'FOO': 'bar'}}, got {config.entries[0].env!r}"
+        )
 
     def test_rejects_empty_entries(self, tmp_path: Path) -> None:
         cfg = tmp_path / "config.toml"
@@ -89,19 +93,19 @@ class TestLoadConfig:
         with pytest.raises(ConfigurationError, match="No entries configured"):
             ConfigLoader.load(cfg)
 
-    def test_rejects_missing_name(self, tmp_path: Path) -> None:
+    def test_missing_name_raises_configuration_error(self, tmp_path: Path) -> None:
         cfg = tmp_path / "config.toml"
         cfg.write_text('[[entries]]\ncommand = "x"\n', encoding="utf-8")
         with pytest.raises(ConfigurationError, match="'name' and 'command'"):
             ConfigLoader.load(cfg)
 
-    def test_rejects_missing_command(self, tmp_path: Path) -> None:
+    def test_missing_command_raises_configuration_error(self, tmp_path: Path) -> None:
         cfg = tmp_path / "config.toml"
         cfg.write_text('[[entries]]\nname = "X"\n', encoding="utf-8")
         with pytest.raises(ConfigurationError, match="'name' and 'command'"):
             ConfigLoader.load(cfg)
 
-    def test_rejects_invalid_on_exit(self, tmp_path: Path) -> None:
+    def test_invalid_on_exit_raises_configuration_error(self, tmp_path: Path) -> None:
         cfg = tmp_path / "config.toml"
         cfg.write_text(
             '[settings]\non_exit = "crash"\n\n[[entries]]\nname = "X"\ncommand = "x"\n',
@@ -110,7 +114,7 @@ class TestLoadConfig:
         with pytest.raises(ConfigurationError, match="Invalid on_exit"):
             ConfigLoader.load(cfg)
 
-    def test_rejects_malformed_toml(self, tmp_path: Path) -> None:
+    def test_malformed_toml_raises_configuration_error(self, tmp_path: Path) -> None:
         cfg = tmp_path / "config.toml"
         cfg.write_text("this is not valid toml [[[", encoding="utf-8")
         with pytest.raises(ConfigurationError, match="Invalid TOML"):
@@ -143,7 +147,7 @@ class TestLoadConfig:
             "Non-existent working_directory should resolve to None"
         )
 
-    def test_rejects_invalid_args_type(self, tmp_path: Path) -> None:
+    def test_invalid_args_type_raises_configuration_error(self, tmp_path: Path) -> None:
         cfg = tmp_path / "config.toml"
         cfg.write_text(
             '[[entries]]\nname = "X"\ncommand = "x"\nargs = "not-a-list"\n',
@@ -152,64 +156,7 @@ class TestLoadConfig:
         with pytest.raises(ConfigurationError, match="'args' must be a list"):
             ConfigLoader.load(cfg)
 
-    def test_command_as_array(self, tmp_path: Path) -> None:
-        """Command as list splits into executable + args."""
-        cfg = tmp_path / "config.toml"
-        cfg.write_text(
-            '[[entries]]\nname = "X"\ncommand = ["ssh", "-t", "my server"]\n',
-            encoding="utf-8",
-        )
-        config = ConfigLoader.load(cfg)
-        assert config.entries[0].command == "ssh", (
-            f"Expected command='ssh', got {config.entries[0].command!r}"
-        )
-        assert config.entries[0].args == ("-t", "my server"), (
-            f"Expected args=('-t', 'my server'), got {config.entries[0].args!r}"
-        )
-
-    def test_command_array_with_extra_args(self, tmp_path: Path) -> None:
-        """Command array args are prepended before explicit args."""
-        cfg = tmp_path / "config.toml"
-        cfg.write_text(
-            '[[entries]]\nname = "X"\n'
-            'command = ["ssh", "-t"]\n'
-            'args = ["my server", "bash"]\n',
-            encoding="utf-8",
-        )
-        config = ConfigLoader.load(cfg)
-        assert config.entries[0].command == "ssh", (
-            f"Expected command='ssh', got {config.entries[0].command!r}"
-        )
-        assert config.entries[0].args == ("-t", "my server", "bash"), (
-            f"Expected combined args, got {config.entries[0].args!r}"
-        )
-
-    def test_command_array_single_element(self, tmp_path: Path) -> None:
-        """Single-element command array is valid."""
-        cfg = tmp_path / "config.toml"
-        cfg.write_text(
-            '[[entries]]\nname = "X"\ncommand = ["pwsh"]\n',
-            encoding="utf-8",
-        )
-        config = ConfigLoader.load(cfg)
-        assert config.entries[0].command == "pwsh", (
-            f"Expected command='pwsh', got {config.entries[0].command!r}"
-        )
-        assert config.entries[0].args == (), (
-            f"Expected empty args, got {config.entries[0].args!r}"
-        )
-
-    def test_command_empty_array_rejected(self, tmp_path: Path) -> None:
-        """Empty command array is an error."""
-        cfg = tmp_path / "config.toml"
-        cfg.write_text(
-            '[[entries]]\nname = "X"\ncommand = []\n',
-            encoding="utf-8",
-        )
-        with pytest.raises(ConfigurationError, match="must not be empty"):
-            ConfigLoader.load(cfg)
-
-    def test_rejects_invalid_env_type(self, tmp_path: Path) -> None:
+    def test_invalid_env_type_raises_configuration_error(self, tmp_path: Path) -> None:
         cfg = tmp_path / "config.toml"
         cfg.write_text(
             '[[entries]]\nname = "X"\ncommand = "x"\nenv = "not-a-table"\n',
@@ -222,21 +169,23 @@ class TestLoadConfig:
 class TestResolveConfigPath:
     """Tests for ConfigLoader.resolve_path."""
 
-    def test_cli_path_takes_priority(self, sample_toml: Path) -> None:
+    def test_cli_path_takes_priority_over_env_var(self, sample_toml: Path) -> None:
         result = ConfigLoader.resolve_path(sample_toml)
-        assert result == sample_toml
+        assert result == sample_toml, f"Expected CLI path {sample_toml}, got {result}"
 
     def test_cli_path_missing_raises(self, tmp_path: Path) -> None:
         missing = tmp_path / "nope.toml"
         with pytest.raises(ConfigurationError, match="not found"):
             ConfigLoader.resolve_path(missing)
 
-    def test_env_var_path(
+    def test_env_var_resolves_config_path(
         self, sample_toml: Path, monkeypatch: pytest.MonkeyPatch
     ) -> None:
         monkeypatch.setenv(ConfigLoader.ENV_VAR_NAME, str(sample_toml))
         result = ConfigLoader.resolve_path(None)
-        assert result == sample_toml
+        assert result == sample_toml, (
+            f"Expected env var path {sample_toml}, got {result}"
+        )
 
     def test_env_var_missing_file_raises(
         self, tmp_path: Path, monkeypatch: pytest.MonkeyPatch
@@ -245,7 +194,7 @@ class TestResolveConfigPath:
         with pytest.raises(ConfigurationError, match=ConfigLoader.ENV_VAR_NAME):
             ConfigLoader.resolve_path(None)
 
-    def test_no_config_creates_default(
+    def test_missing_config_creates_default_file(
         self, tmp_path: Path, monkeypatch: pytest.MonkeyPatch
     ) -> None:
         monkeypatch.delenv(ConfigLoader.ENV_VAR_NAME, raising=False)
@@ -255,7 +204,9 @@ class TestResolveConfigPath:
             default_path,
         )
         result = ConfigLoader.resolve_path(None)
-        assert result == default_path
+        assert result == default_path, (
+            f"Expected default path {default_path}, got {result}"
+        )
         assert default_path.exists(), "Default config file should have been created"
         assert default_path.read_text(encoding="utf-8") == (
             ConfigLoader.STARTER_CONFIG
@@ -305,12 +256,16 @@ class TestConfigReferenceSync:
             "entries"
         }
         readme_fields = self._readme_table_keys("`[settings]`")
-        assert readme_fields == code_fields
+        assert readme_fields == code_fields, (
+            f"Expected settings fields {code_fields}, got {readme_fields}"
+        )
 
     def test_entry_fields_match_readme(self) -> None:
         code_fields = {f.name for f in dataclasses.fields(EntryConfig)}
         readme_fields = self._readme_table_keys("`[[entries]]`")
-        assert readme_fields == code_fields
+        assert readme_fields == code_fields, (
+            f"Expected entry fields {code_fields}, got {readme_fields}"
+        )
 
     def test_valid_on_exit_values_in_readme(self) -> None:
         text = _README.read_text(encoding="utf-8")