devnomads-cli 0.9.1__tar.gz → 0.10.0__tar.gz

{devnomads_cli-0.9.1 → devnomads_cli-0.10.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: devnomads-cli
-Version: 0.9.1
+Version: 0.10.0
 Summary: Manage your DevNomads services from the command line
 Author-email: DevNomads <support@devnomads.nl>
 License: MIT
@@ -81,6 +81,18 @@ dncli dns records set example.com www A 192.0.2.1 --ttl 3600
 dncli dns records delete example.com www A
 ```
 
+Anywhere a zone is expected you may pass the service id of a domain
+instead of its name, so the ids from `dncli domains list` work here
+too:
+
+```sh
+dncli dns records list 42
+```
+
+An id always means that domain's own zone. A delegated subzone such
+as `foo.example.com` has no domain service of its own, so address it
+by name.
+
 ## Moving a zone from another provider
 
 `dncli dns transfer` copies a zone from your current DNS provider

{devnomads_cli-0.9.1 → devnomads_cli-0.10.0}/README.md

@@ -65,6 +65,18 @@ dncli dns records set example.com www A 192.0.2.1 --ttl 3600
 dncli dns records delete example.com www A
 ```
 
+Anywhere a zone is expected you may pass the service id of a domain
+instead of its name, so the ids from `dncli domains list` work here
+too:
+
+```sh
+dncli dns records list 42
+```
+
+An id always means that domain's own zone. A delegated subzone such
+as `foo.example.com` has no domain service of its own, so address it
+by name.
+
 ## Moving a zone from another provider
 
 `dncli dns transfer` copies a zone from your current DNS provider

{devnomads_cli-0.9.1 → devnomads_cli-0.10.0}/devnomads_cli.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: devnomads-cli
-Version: 0.9.1
+Version: 0.10.0
 Summary: Manage your DevNomads services from the command line
 Author-email: DevNomads <support@devnomads.nl>
 License: MIT
@@ -81,6 +81,18 @@ dncli dns records set example.com www A 192.0.2.1 --ttl 3600
 dncli dns records delete example.com www A
 ```
 
+Anywhere a zone is expected you may pass the service id of a domain
+instead of its name, so the ids from `dncli domains list` work here
+too:
+
+```sh
+dncli dns records list 42
+```
+
+An id always means that domain's own zone. A delegated subzone such
+as `foo.example.com` has no domain service of its own, so address it
+by name.
+
 ## Moving a zone from another provider
 
 `dncli dns transfer` copies a zone from your current DNS provider

{devnomads_cli-0.9.1 → devnomads_cli-0.10.0}/dncli.py

@@ -53,7 +53,7 @@ from rich.markup import escape
 from rich.table import Table
 from typer.core import TyperGroup
 
-__version__ = "0.9.1"
+__version__ = "0.10.0"
 
 # Distribution names, for reporting versions via `dncli version`.
 CLI_PACKAGE = "devnomads-cli"
@@ -673,6 +673,45 @@ def zone_id(zone: str) -> str:
     return zone if zone.endswith(".") else f"{zone}."
 
 
+def resolve_zone(state: AppState, zone: str) -> str:
+    """Map an all-digit domain service id to that domain's zone name; any
+    other value is taken to be a zone name already and passed through.
+
+    An id always means the domain product's own zone. A delegated subzone
+    such as foo.example.com has no domain service of its own, so it can
+    only be addressed by name.
+    """
+
+    text = str(zone)
+    if not text.isdigit():
+        return text.rstrip(".")
+    client = get_client(state)
+    rows = client.request("GET", "/services/domains")
+    if isinstance(rows, dict):
+        rows = rows.get("data", [])
+    match = next(
+        (
+            row
+            for row in (rows or [])
+            if isinstance(row, dict) and str(row.get("service_id")) == text
+        ),
+        None,
+    )
+    if match is None:
+        raise CliError(f"no domain service with id {text}")
+    name = str(match.get("entity") or "").rstrip(".")
+    if not name:
+        raise CliError(f"domain service {text} has no domain name")
+    zones = client.request("GET", "/services/dns/zones")
+    known = {str(item.get("name", "")).rstrip(".").lower() for item in zones or []}
+    if name.lower() not in known:
+        raise CliError(
+            f"{name} (service {text}) is not hosted on DevNomads DNS; "
+            "dncli dns zones list shows your zones"
+        )
+    return name
+
+
 def fqdn(name: str, zone: str) -> str:
     """Absolute record name with trailing dot, PowerDNS style."""
 
@@ -1438,13 +1477,17 @@ def zones_list(
 @zones_app.command("show")
 def zones_show(
     ctx: typer.Context,
-    zone: Annotated[str, typer.Argument(help="Zone name, e.g. example.com.")],
+    zone: Annotated[
+        str,
+        typer.Argument(help="Zone name (example.com) or the service id of a domain."),
+    ],
     output: OutputOption = None,
     columns: ColumnsOption = None,
 ) -> None:
     """Show a DNS zone including its records."""
 
     state = state_from(ctx, output, columns)
+    zone = resolve_zone(state, zone)
     data = get_client(state).request("GET", f"/services/dns/zones/{zone_id(zone)}")
     if resolve_format(state) is not OutputFormat.table or not isinstance(data, dict):
         render(state, data)
@@ -1465,7 +1508,10 @@ def zones_show(
 @records_app.command("list")
 def records_list(
     ctx: typer.Context,
-    zone: Annotated[str, typer.Argument(help="Zone name, e.g. example.com.")],
+    zone: Annotated[
+        str,
+        typer.Argument(help="Zone name (example.com) or the service id of a domain."),
+    ],
     sort: SortOption = None,
     output: OutputOption = None,
     columns: ColumnsOption = None,
@@ -1473,6 +1519,7 @@ def records_list(
     """List the records in a zone, one row per value."""
 
     state = state_from(ctx, output, columns)
+    zone = resolve_zone(state, zone)
     data = get_client(state).request("GET", f"/services/dns/zones/{zone_id(zone)}")
     rrsets = data.get("rrsets", []) if isinstance(data, dict) else []
     render(
@@ -1486,7 +1533,10 @@ def records_list(
 @records_app.command("set")
 def records_set(
     ctx: typer.Context,
-    zone: Annotated[str, typer.Argument(help="Zone name, e.g. example.com.")],
+    zone: Annotated[
+        str,
+        typer.Argument(help="Zone name (example.com) or the service id of a domain."),
+    ],
     name: Annotated[
         str, typer.Argument(help="Record name relative to the zone; @ for the apex.")
     ],
@@ -1504,6 +1554,7 @@ def records_set(
     """Create or replace a record set (PowerDNS REPLACE semantics)."""
 
     state = state_from(ctx)
+    zone = resolve_zone(state, zone)
     rrset = build_rrset(
         zone, name, rtype, changetype="REPLACE", ttl=ttl, contents=content
     )
@@ -1520,7 +1571,10 @@ def records_set(
 @records_app.command("delete")
 def records_delete(
     ctx: typer.Context,
-    zone: Annotated[str, typer.Argument(help="Zone name, e.g. example.com.")],
+    zone: Annotated[
+        str,
+        typer.Argument(help="Zone name (example.com) or the service id of a domain."),
+    ],
     name: Annotated[
         str, typer.Argument(help="Record name relative to the zone; @ for the apex.")
     ],
@@ -1534,6 +1588,7 @@ def records_delete(
     """Delete a whole record set (all values for name + type)."""
 
     state = state_from(ctx)
+    zone = resolve_zone(state, zone)
     rrset = build_rrset(zone, name, rtype, changetype="DELETE")
     _confirm(f"Delete all {rrset['type']} records for {rrset['name']}?", yes)
     payload = {"rrsets": [rrset]}
@@ -1555,7 +1610,12 @@ def dns_transfer(
         ),
     ],
     zone: Annotated[
-        str, typer.Option("--zone", help="Zone name on both sides, e.g. example.com.")
+        str,
+        typer.Option(
+            "--zone",
+            help="Zone name on both sides (example.com), or the service id "
+            "of a DevNomads domain.",
+        ),
     ],
     source_profile: Annotated[
         str | None,
@@ -1583,6 +1643,7 @@ def dns_transfer(
 
     state = state_from(ctx, output, columns)
     client = get_client(state)  # resolve DevNomads credentials first: fail fast
+    zone = resolve_zone(state, zone)
     source = build_source(from_, source_profile)
     zones = client.request("GET", "/services/dns/zones")
     known = {str(item.get("name", "")).rstrip(".") for item in zones or []}

{devnomads_cli-0.9.1 → devnomads_cli-0.10.0}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "devnomads-cli"
-version = "0.9.1"
+version = "0.10.0"
 description = "Manage your DevNomads services from the command line"
 readme = "README.md"
 requires-python = ">=3.10"

{devnomads_cli-0.9.1 → devnomads_cli-0.10.0}/tests/test_cli.py

@@ -337,6 +337,92 @@ def test_records_delete_with_yes(configured):
     assert payload["rrsets"][0]["changetype"] == "DELETE"
 
 
+# --- service id -> zone name resolution --------------------------------------
+
+DOMAINS = [
+    {"service_id": 42, "type": "domain", "entity": "example.com"},
+    {"service_id": 43, "type": "domain", "entity": "elsewhere.com"},
+]
+
+
+def mock_domains(zones=("example.com.",)):
+    respx.get(f"{API}/services/domains").mock(
+        return_value=Response(200, json={"data": DOMAINS})
+    )
+    respx.get(f"{API}/services/dns/zones").mock(
+        return_value=Response(200, json=[{"id": z, "name": z} for z in zones])
+    )
+
+
+@respx.mock
+def test_records_list_accepts_a_domain_service_id(configured):
+    mock_domains()
+    respx.get(f"{API}/services/dns/zones/example.com.").mock(
+        return_value=Response(200, json=ZONE)
+    )
+    result = runner.invoke(app, ["--out", "json", "dns", "records", "list", "42"])
+    assert result.exit_code == 0, all_output(result)
+    assert json.loads(result.output)[0]["name"] == "www.example.com."
+
+
+@respx.mock
+def test_records_set_by_service_id_builds_names_from_the_zone(configured):
+    mock_domains()
+    route = respx.patch(f"{API}/services/dns/zones/example.com.").mock(
+        return_value=Response(204)
+    )
+    result = runner.invoke(
+        app, ["dns", "records", "set", "42", "www", "A", "192.0.2.1"]
+    )
+    assert result.exit_code == 0, all_output(result)
+    payload = json.loads(route.calls.last.request.content)
+    assert payload["rrsets"][0]["name"] == "www.example.com."
+
+
+@respx.mock
+def test_zones_show_by_service_id(configured):
+    mock_domains()
+    route = respx.get(f"{API}/services/dns/zones/example.com.").mock(
+        return_value=Response(200, json=ZONE)
+    )
+    result = runner.invoke(app, ["--out", "json", "dns", "zones", "show", "42"])
+    assert result.exit_code == 0, all_output(result)
+    assert route.called
+
+
+@respx.mock
+def test_unknown_service_id_fails_clearly(configured):
+    mock_domains()
+    result = runner.invoke(app, ["dns", "records", "list", "999"])
+    assert result.exit_code == 1
+    assert "no domain service with id 999" in all_output(result)
+
+
+@respx.mock
+def test_domain_without_a_devnomads_zone_fails_clearly(configured):
+    # 43 is elsewhere.com: registered with us, DNS hosted somewhere else.
+    mock_domains()
+    result = runner.invoke(app, ["dns", "records", "list", "43"])
+    assert result.exit_code == 1
+    output = all_output(result)
+    assert "elsewhere.com (service 43) is not hosted on DevNomads DNS" in output
+
+
+@respx.mock
+def test_zone_names_do_not_trigger_a_lookup(configured):
+    domains = respx.get(f"{API}/services/domains").mock(
+        return_value=Response(200, json={"data": DOMAINS})
+    )
+    respx.get(f"{API}/services/dns/zones/example.com.").mock(
+        return_value=Response(200, json=ZONE)
+    )
+    result = runner.invoke(
+        app, ["--out", "json", "dns", "records", "list", "example.com"]
+    )
+    assert result.exit_code == 0, all_output(result)
+    assert not domains.called
+
+
 @respx.mock
 def test_api_error_message_and_exit_code(configured):
     respx.get(f"{API}/services").mock(