hostctl 0.1.0__tar.gz → 0.1.2__tar.gz

{hostctl-0.1.0 → hostctl-0.1.2}/CHANGELOG.md

@@ -7,6 +7,48 @@ and this project adheres to [Semantic Versioning](https://semver.org/).
 
 ## [Unreleased]
 
+## [0.1.2] - 2026-07-28
+
+Supersedes 0.1.1, which was tagged but never published: a release-gate test
+timed out on the Windows/Python 3.14 runner, so nothing reached PyPI. There is
+no 0.1.1 release; its fix is included here.
+
+### Added
+
+- `uri_hostname(parsed)` returns the host as it was written in a URI, rather
+  than the case-folded spelling `urlsplit().hostname` produces. Use it in
+  `_from_parsed_uri` wherever a host is stored; presence checks can keep using
+  `.hostname`, since emptiness does not depend on case.
+
+### Changed
+
+- A config built from a URI now stores the hostname as typed, so
+  `HostConfig("ssh://nasA")` keeps `nasA` in `.host`, in `connection_uri`, and
+  in the `HostInfo.hostname` a system host reports without connecting. It
+  previously stored `urlsplit`'s lowercased form, which meant the library
+  echoed a spelling the operator never wrote and left downstream code to
+  recover the original from the URI itself.
+
+  Consequently `.host` is the spelling that was given, not a canonical form:
+  `HostConfig("ssh://nasA")` and `HostConfig("ssh://nasa")` no longer compare
+  equal, so code using a config or its host as a dict key or for
+  deduplication should casefold explicitly. Resolution is unaffected — DNS,
+  SSH, and WinRM all treat the two spellings as one name.
+
+### Fixed
+
+- `redact_uri()` no longer case-folds the hostname. Only the branch that
+  rebuilt the authority — the one taken when a password was present — adopted
+  `urlsplit`'s lowercased `hostname`, so `nasA` rendered as `nasa` with a
+  credential and as `nasA` without one. The same host now renders one way
+  whichever branch runs, and log records stay greppable by the name the
+  operator typed. Redaction removes a credential; normalizing a host is a
+  separate concern and is left to the transport that resolves it.
+- The spawn conformance test no longer fails on a slow runner. It waited 10s
+  for a real `powershell.exe -NoProfile` to start, exit, and be reaped, which
+  a cold Windows CI runner can exceed. The wait is a hang guard rather than a
+  performance assertion, so it is now 60s. Test-only; no library change.
+
 ## [0.1.0] - 2026-07-27
 
 First release. Alpha: the public surface is deliberately small (66 exported
@@ -144,5 +186,6 @@ test suite on Python 3.9 through 3.14.
   assigned to it; a config-less host now builds its own family configuration
   instead.
 
-[Unreleased]: https://github.com/jose-pr/hostctl/compare/v0.1.0...HEAD
+[Unreleased]: https://github.com/jose-pr/hostctl/compare/v0.1.2...HEAD
+[0.1.2]: https://github.com/jose-pr/hostctl/compare/v0.1.0...v0.1.2
 [0.1.0]: https://github.com/jose-pr/hostctl/releases/tag/v0.1.0

{hostctl-0.1.0 → hostctl-0.1.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: hostctl
-Version: 0.1.0
+Version: 0.1.2
 Summary: Run commands and access files on a host, local or remote, protocol-agnostic
 Project-URL: Homepage, https://github.com/jose-pr/hostctl/
 Project-URL: Documentation, https://jose-pr.github.io/hostctl/

{hostctl-0.1.0 → hostctl-0.1.2}/docs/api/reference.md

@@ -84,6 +84,7 @@ without a deprecation cycle.
 ::: hostctl.strict_uri_credentials
 ::: hostctl.strict_uri_query
 ::: hostctl.uri_host
+::: hostctl.uri_hostname
 
 ## Composing transports
 

{hostctl-0.1.0 → hostctl-0.1.2}/docs/guide/transfer.md

@@ -116,11 +116,8 @@ with ProgressReader(source_path.open("rb"), report, total=size) as reader:
   though their read sides stream.
 
 Three gaps are tracked as `pathlib_next` requests rather than hostctl features,
-because each one lives inside machinery hostctl deliberately does not fork.
-Drafts are kept in the repository under `.agents/upstream/`:
+because each one lives inside machinery hostctl deliberately does not fork:
 
-| Request | Draft |
-| --- | --- |
-| Byte-progress hook in `BinaryOpen.copy` / `Path.copy()` | `pathlib_next-copy-progress-hook.md` |
-| Symlink handling in `PathSyncer` (plus its unreachable `ignore_error`) | `pathlib_next-symlink-sync.md` |
-| Optional backend-native checksum protocol | `pathlib_next-backend-checksum-protocol.md` |
+- Byte-progress hook in `BinaryOpen.copy` / `Path.copy()`
+- Symlink handling in `PathSyncer` (plus its unreachable `ignore_error`)
+- Optional backend-native checksum protocol

{hostctl-0.1.0 → hostctl-0.1.2}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "hostctl"
-version = "0.1.0"
+version = "0.1.2"
 authors = [{ name = "Jose A." }]
 description = "Run commands and access files on a host, local or remote, protocol-agnostic"
 readme = "README.md"

{hostctl-0.1.0 → hostctl-0.1.2}/src/hostctl/__init__.py

@@ -46,6 +46,7 @@ from .host import (
     strict_uri_credentials as strict_uri_credentials,
     strict_uri_query as strict_uri_query,
     uri_host as uri_host,
+    uri_hostname as uri_hostname,
     winrm_providers as winrm_providers,
     QemuConfig as QemuConfig,
     QemuHost as QemuHost,
@@ -125,6 +126,7 @@ __all__ = [
     "strict_uri_credentials",
     "strict_uri_query",
     "uri_host",
+    "uri_hostname",
     "winrm_providers",
     "SshConfig",
     "WinRMConfig",

{hostctl-0.1.0 → hostctl-0.1.2}/src/hostctl/host/__init__.py

@@ -10,6 +10,7 @@ from ._common import (
     strict_uri_credentials as strict_uri_credentials,
     strict_uri_query as strict_uri_query,
     uri_host as uri_host,
+    uri_hostname as uri_hostname,
 )
 from ._ssh import ssh_providers as ssh_providers
 from ._winrm import winrm_providers as winrm_providers

{hostctl-0.1.0 → hostctl-0.1.2}/src/hostctl/host/_common.py

@@ -160,9 +160,37 @@ def _reject_authority_control_characters(parsed: _SplitResult) -> None:
         )
 
 
+def uri_hostname(parsed: _SplitResult) -> str:
+    """The host as it was written, not as `urlsplit` case-folded it.
+
+    `urlsplit().hostname` is lowercased by design -- DNS is case-insensitive,
+    so folding is right for *resolution*. It is wrong for text a caller will
+    see again. A config built from a URI stores this value and renders it back
+    through `connection_uri`, so reading `.hostname` there would make the
+    library echo a spelling the operator never typed, and `nasA` would reach
+    logs and `HostInfo` as `nasa`.
+
+    Use this in `_from_parsed_uri` wherever a host is stored. Presence checks
+    (`if not parsed.hostname`) can keep using `.hostname` -- emptiness does not
+    depend on case. Resolution is unaffected either way: DNS, SSH, and WinRM
+    all treat the two spellings as one name.
+
+    Returns `""` for a URI with no host, matching `hostname or ""`.
+
+    `hostname` is a case-folded substring of the authority (after any
+    userinfo), so locating it recovers the original text.
+    """
+    hostname = parsed.hostname or ""
+    authority = parsed.netloc.rpartition("@")[2]
+    index = authority.casefold().find(hostname)
+    if index < 0:
+        return hostname
+    return authority[index : index + len(hostname)]
+
+
 def _rebuild_authority(parsed: _SplitResult, password: _ty.Optional[str]) -> str:
     """Rebuild a URI authority with `password` in place of the original."""
-    host = uri_host(parsed.hostname or "")
+    host = uri_host(uri_hostname(parsed))
     if parsed.port is not None:
         host = f"{host}:{parsed.port}"
     if not parsed.username:

{hostctl-0.1.0 → hostctl-0.1.2}/src/hostctl/host/_ssh.py

@@ -40,6 +40,7 @@ from ._common import (
     strict_uri_credentials,
     strict_uri_query,
     uri_host,
+    uri_hostname,
     reject_stdin_conflict,
 )
 from ..shell import (
@@ -147,7 +148,7 @@ class SshConfig(HostConfig, schemes=("ssh",)):
         if not parsed.hostname or parsed.path not in ("", "/"):
             raise ValueError("SSH URI requires a host and no path")
         return cls(
-            host=parsed.hostname,
+            host=uri_hostname(parsed),
             port=parsed.port or 22,
             username=unquote(parsed.username or "") or "root",
             password=typing.cast(typing.Optional[str], credentials.get("password")),

{hostctl-0.1.0 → hostctl-0.1.2}/src/hostctl/host/_winrm.py

@@ -50,6 +50,7 @@ from ._common import (
     strict_uri_credentials,
     strict_uri_query,
     uri_host,
+    uri_hostname,
 )
 from ..shell import POWERSHELL, ShellFlavour
 
@@ -155,7 +156,7 @@ class WinRMConfig(HostConfig, schemes=("winrm", "winrms")):
         if not parsed.username:
             raise ValueError("WinRM URI requires a username")
         return cls(
-            host=parsed.hostname,
+            host=uri_hostname(parsed),
             username=unquote(parsed.username),
             password=typing.cast(typing.Optional[str], credentials.get("password")),
             port=parsed.port,

{hostctl-0.1.0 → hostctl-0.1.2}/src/hostctl/host/qemu.py

@@ -48,6 +48,7 @@ from ._common import (
     strict_uri_credentials,
     strict_uri_query,
     uri_host,
+    uri_hostname,
 )
 from ._ssh import SshConfig, _SshTransport
 
@@ -222,7 +223,7 @@ class QemuConfig(HostConfig, schemes=("qemu+libvirt", "qga+unix", "qga+ssh")):
         if not parsed.hostname or not parsed.username or not parsed.path.strip("/"):
             raise ValueError("SSH QGA URI requires user, host, and domain")
         ssh = SshConfig(
-            parsed.hostname,
+            uri_hostname(parsed),
             port=parsed.port or 22,
             username=unquote(parsed.username),
             password=typing.cast(typing.Optional[str], credentials.get("password")),

{hostctl-0.1.0 → hostctl-0.1.2}/src/hostctl/shell/_common.py

@@ -386,7 +386,7 @@ class Shell(Executor[_Result], typing.Generic[_Result]):
         provides on its own -- a login profile, rc files, the service
         environment -- because nothing is sent to override it. Requesting a
         genuinely empty environment is a separate feature that does not exist
-        yet (see `.agents/plans/env_clear_semantics.md`).
+        yet.
         """
         if env is None:
             return None

{hostctl-0.1.0 → hostctl-0.1.2}/tests/conformance/test_process_contract.py

@@ -136,7 +136,13 @@ def test_provider_spawn_runs_a_real_process(provider):
                 # closed, which is exactly what the contract promises.
                 assert process.returncode is None
             else:
-                assert process.wait(timeout=10) == 0
+                # Generous because this waits on a real interpreter starting
+                # cold: `powershell.exe -NoProfile` on a Windows CI runner has
+                # taken over 10s. The timeout is a hang guard, not a
+                # performance assertion -- a working adapter exits in well
+                # under a second, so a larger bound costs nothing and a
+                # smaller one fails on runner startup rather than on a defect.
+                assert process.wait(timeout=60) == 0
                 assert process.returncode == 0
         finally:
             process.close()

{hostctl-0.1.0 → hostctl-0.1.2}/tests/test_composite_path_propagation.py

@@ -1,4 +1,4 @@
-"""Provider/pin retention across every path derivation (plan Phase 3).
+"""Provider/pin retention across every path derivation.
 
 ``pathlib.PurePath`` builds several derivations with ``object.__new__``, which
 bypasses the composite constructor entirely.  Any derivation that loses the

{hostctl-0.1.0 → hostctl-0.1.2}/tests/test_host_api.py

@@ -20,6 +20,7 @@ from hostctl import (
     WinRMConfig,
     parse_credentials,
     redact_uri,
+    uri_hostname,
 )
 from hostctl.host._common import _encode_stripped_characters
 
@@ -248,6 +249,44 @@ def test_redact_uri_strips_the_password_leaving_a_reusable_uri():
     assert HostConfig(redacted).username == "admin"
 
 
+def test_redact_uri_preserves_hostname_case():
+    # Redaction removes a credential; it must not also normalize the host.
+    # The branch that rebuilds the authority used to adopt urlsplit's
+    # case-folded hostname, so one host rendered two ways depending on whether
+    # a password was supplied -- which breaks log correlation by the name the
+    # operator typed.
+    assert redact_uri("wss://root:pw@nasA:8443/api") == "wss://root@nasA:8443/api"
+    assert redact_uri("wss://root@nasA") == "wss://root@nasA"
+    assert redact_uri("wss://nasA") == "wss://nasA"
+
+
+def test_config_from_uri_keeps_the_hostname_as_written():
+    # A config renders its host back out through connection_uri, so storing
+    # urlsplit's folded `hostname` would echo a spelling nobody typed.
+    config = HostConfig("ssh://admin@nasA.example.com")
+
+    assert config.host == "nasA.example.com"
+    assert "nasA.example.com" in config.connection_uri
+    # ...on the credentialed path too, which rebuilds the authority.
+    assert HostConfig("ssh://admin:pw@nasA.example.com").host == "nasA.example.com"
+
+
+def test_config_host_case_is_preserved_not_canonicalized():
+    # The deliberate trade-off: `host` is the spelling the operator typed, not
+    # a canonical form, so two spellings of one machine are not equal configs.
+    # Resolution is unaffected -- DNS, SSH, and WinRM treat them as one name.
+    assert HostConfig("ssh://nasA").host != HostConfig("ssh://nasa").host
+
+
+def test_uri_hostname_recovers_the_written_spelling():
+    assert (
+        uri_hostname(urlsplit("ssh://user@NasA.Example.COM:22")) == "NasA.Example.COM"
+    )
+    # No host, and an IPv6 literal (returned unbracketed, as .hostname is).
+    assert uri_hostname(urlsplit("local:")) == ""
+    assert uri_hostname(urlsplit("ssh://u:p@[2001:DB8::1]:22")) == "2001:DB8::1"
+
+
 def test_redact_uri_output_carries_no_credential_back_in():
     config = HostConfig(redact_uri("ssh://admin:hunter2@nas.example.com"))