netbox-super-cli 1.0.4__tar.gz → 1.0.6__tar.gz

{netbox_super_cli-1.0.4 → netbox_super_cli-1.0.6}/CHANGELOG.md

@@ -2,6 +2,42 @@
 
 All notable changes to netbox-super-cli are tracked here. Format follows [Keep a Changelog](https://keepachangelog.com/en/1.1.0/) loosely. From v1.0.0 onward, releases follow [Semantic Versioning](https://semver.org/) and the version in `pyproject.toml` matches the git tag. Pre-1.0 milestones (Phase 1-5) were pinned by tag while `pyproject.toml` stayed at `0.0.1`.
 
+## v1.0.6 — 2026-06-09
+
+Patch release. Fixes table/CSV column rendering for fields that are nested
+objects or lists of objects (issue #78).
+
+### Fixed
+
+- **Nested-object and list columns now render in `table`/`csv`** (issue #78).
+  A `--columns` (or `columns:` config) entry pointing to a nested object — a
+  NetBox brief FK such as `role`, `site`, `tenant`, `device_type` — rendered as
+  an empty cell; an entry pointing to a list of objects (e.g. `tags`) rendered
+  as a raw JSON blob. Such columns now collapse to the object's `display` label
+  (compact-JSON fallback for objects without one), and list columns render their
+  members' labels joined with `, ` (empty list → blank). Scalars, nulls, and
+  dotted leaf paths (`status.value`, `role.name`) are unchanged, as is the full
+  output of the `json`, `jsonl`, and `yaml` formats.
+
+## v1.0.5 — 2026-05-21
+
+Adds opt-in semantic terminal coloring (issue #24). No config migration is
+needed — `color_mode` defaults to `auto`, which colors a stream only when it
+is a TTY, so piped and redirected output is byte-for-byte unchanged.
+
+### Added
+
+- **Semantic terminal coloring** (issue #24, PR #72). A new `defaults.color_mode`
+  config field — `auto` (default), `on`, or `off` — controls colored output.
+  Table cells are colored by meaning: status values (`active`/`enabled`/`online`
+  green, `planned`/`staged` yellow, `failed`/`disabled`/`offline` red), booleans,
+  and empty cells. `nsc explain` traces and Rich error panels are colored too.
+  stdout and stderr are gated independently by each stream's own TTY, so
+  `nsc … | jq` and `nsc … 2>err.log` no longer mis-color one stream or leak
+  ANSI into the other. Structured formats (`json`, `jsonl`, `yaml`, `csv`) are
+  never colored. All server- and user-sourced text is escaped before Rich
+  markup parsing, so stray `[`/`]` in API responses cannot garble output.
+
 ## v1.0.4 — 2026-05-16
 
 Maintenance release: a codebase-wide internal simplification pass (no behavioural change), a full documentation parity audit, and a dependency bump. There are no CLI, config, or output-contract changes — upgrading from v1.0.3 is transparent.

{netbox_super_cli-1.0.4 → netbox_super_cli-1.0.6}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: netbox-super-cli
-Version: 1.0.4
+Version: 1.0.6
 Summary: Dynamic NetBox CLI generated from the live OpenAPI schema.
 Project-URL: Homepage, https://github.com/thomaschristory/netbox-super-cli
 Project-URL: Issues, https://github.com/thomaschristory/netbox-super-cli/issues

{netbox_super_cli-1.0.4 → netbox_super_cli-1.0.6}/docs/guides/output-formats.md

@@ -43,6 +43,20 @@ columns:
 Unknown column names are silently ignored, so the same config remains valid
 across NetBox versions and plugin changes.
 
+### Nested objects and lists in columns
+
+A column may name a nested field directly or drill in with a dotted path:
+
+- A dotted path selects the leaf scalar: `status.value`, `site.name`.
+- A bare nested object (`site`, `role`, `tenant`, `status`) collapses to its
+  `display` label — `role` renders `Router`, not an empty cell. An object with
+  no `display` field falls back to compact JSON.
+- A list of objects renders as its members' labels joined with `, ` (e.g.
+  `tags` → `prod, edge`); an empty list renders blank.
+
+This applies to both `table` and `csv`. The `json`, `jsonl`, and `yaml` formats
+always keep the full nested structure untouched.
+
 ## Compact JSON
 
 ```sh

{netbox_super_cli-1.0.4 → netbox_super_cli-1.0.6}/docs/reference/config.md

@@ -11,7 +11,7 @@ All fields below describe `~/.nsc/config.yaml`.
 |---|---|---|
 | `default_profile` | `str | None` | `None` |
 | `profiles` | `dict[str, Profile]` | `{}` |
-| `defaults` | `<class 'Defaults'>` | `Defaults(output=<OutputFormat.TABLE: 'table'>, page_size=50, timeout=30.0, schema_refresh=<SchemaRefresh.DAILY: 'daily'>)` |
+| `defaults` | `<class 'Defaults'>` | `Defaults(output=<OutputFormat.TABLE: 'table'>, page_size=50, timeout=30.0, schema_refresh=<SchemaRefresh.DAILY: 'daily'>, color_mode=<ColorMode.AUTO: 'auto'>)` |
 | `columns` | `dict[str, dict[str, list[str]]]` | `{}` |
 
 ## `Profile`
@@ -33,3 +33,4 @@ All fields below describe `~/.nsc/config.yaml`.
 | `page_size` | `<class 'int'>` | `50` |
 | `timeout` | `<class 'float'>` | `30.0` |
 | `schema_refresh` | `<enum 'SchemaRefresh'>` | `<SchemaRefresh.DAILY: 'daily'>` |
+| `color_mode` | `<enum 'ColorMode'>` | `<ColorMode.AUTO: 'auto'>` |

{netbox_super_cli-1.0.4 → netbox_super_cli-1.0.6}/nsc/_version.py

@@ -1,3 +1,3 @@
 """Single source of truth for the package version."""
 
-__version__ = "1.0.4"
+__version__ = "1.0.6"

{netbox_super_cli-1.0.4 → netbox_super_cli-1.0.6}/nsc/cli/globals.py

@@ -9,6 +9,7 @@ from dataclasses import dataclass
 from nsc.cli.runtime import (
     CLIOverrides,
     RuntimeContext,
+    resolve_color,
     resolve_profile,
 )
 from nsc.config import default_paths
@@ -50,6 +51,8 @@ def build_runtime_context(state: GlobalState) -> RuntimeContext:
         output_format=output,
         debug=state.debug,
         page_size=state.config.defaults.page_size,
+        color=resolve_color(state.config.defaults.color_mode, is_tty=sys.stdout.isatty()),
+        color_stderr=resolve_color(state.config.defaults.color_mode, is_tty=sys.stderr.isatty()),
     )
 
 

{netbox_super_cli-1.0.4 → netbox_super_cli-1.0.6}/nsc/cli/handlers.py

@@ -38,6 +38,7 @@ from nsc.config.settings import default_paths
 from nsc.http.audit import AuditEntry, append_audit_jsonl
 from nsc.http.errors import NetBoxAPIError, NetBoxClientError
 from nsc.model.command_model import HttpMethod, Operation, ParameterLocation
+from nsc.output._console import make_console
 from nsc.output.errors import (
     ClientError,
     ErrorEnvelope,
@@ -101,10 +102,11 @@ def handle_list(
             columns=ctx.resolve_columns(op_tag, op_resource, operation),
             stream=_out(stream),
             compact=ctx.compact,
+            color=ctx.color,
         )
     except (NetBoxAPIError, NetBoxClientError) as exc:
         env = map_error(exc, operation_id=operation.operation_id)
-        code = emit_envelope(env, output_format=ctx.output_format)
+        code = emit_envelope(env, output_format=ctx.output_format, color=ctx.color_stderr)
         raise typer.Exit(code) from exc
 
 
@@ -126,10 +128,11 @@ def handle_get(
             columns=ctx.resolve_columns(op_tag, op_resource, operation),
             stream=_out(stream),
             compact=ctx.compact,
+            color=ctx.color,
         )
     except (NetBoxAPIError, NetBoxClientError) as exc:
         env = map_error(exc, operation_id=operation.operation_id)
-        code = emit_envelope(env, output_format=ctx.output_format)
+        code = emit_envelope(env, output_format=ctx.output_format, color=ctx.color_stderr)
         raise typer.Exit(code) from exc
 
 
@@ -316,7 +319,7 @@ def _handle_write(
             is_delete=is_delete,
         )
     except ClientError as exc:
-        code = emit_envelope(exc.envelope, output_format=ctx.output_format)
+        code = emit_envelope(exc.envelope, output_format=ctx.output_format, color=ctx.color_stderr)
         raise typer.Exit(code) from exc
     except NDJSONParseError as exc:
         env = input_error_envelope(
@@ -324,11 +327,11 @@ def _handle_write(
             bad_lines=exc.bad_lines,
             operation_id=operation.operation_id,
         )
-        code = emit_envelope(env, output_format=ctx.output_format)
+        code = emit_envelope(env, output_format=ctx.output_format, color=ctx.color_stderr)
         raise typer.Exit(code) from exc
     except InputError as exc:
         env = client_envelope(str(exc), operation_id=operation.operation_id)
-        code = emit_envelope(env, output_format=ctx.output_format)
+        code = emit_envelope(env, output_format=ctx.output_format, color=ctx.color_stderr)
         raise typer.Exit(code) from exc
     except (NetBoxAPIError, NetBoxClientError) as exc:
         if (
@@ -340,7 +343,7 @@ def _handle_write(
             _render_delete_already_absent(ctx, stream=out)
             return
         env = map_error(exc, operation_id=operation.operation_id)
-        code = emit_envelope(env, output_format=ctx.output_format)
+        code = emit_envelope(env, output_format=ctx.output_format, color=ctx.color_stderr)
         raise typer.Exit(code) from exc
 
 
@@ -364,13 +367,13 @@ def _handle_dry_run_or_preflight(
         _render_explain_or_dry_run(trace, ctx, stream=out)
         if not preflight.ok:
             env = _preflight_envelope(operation, preflight, applied=False)
-            code = emit_envelope(env, output_format=ctx.output_format)
+            code = emit_envelope(env, output_format=ctx.output_format, color=ctx.color_stderr)
             raise typer.Exit(code)
         return True
     if not preflight.ok:
         _emit_dry_run_audit(operation, resolved, preflight, ctx, preflight_blocked=True)
         env = _preflight_envelope(operation, preflight, applied=False)
-        code = emit_envelope(env, output_format=ctx.output_format)
+        code = emit_envelope(env, output_format=ctx.output_format, color=ctx.color_stderr)
         raise typer.Exit(code)
     return False
 
@@ -480,7 +483,7 @@ def _execute_loop(
         operation_id=operation.operation_id,
         total_records=total_records,
     )
-    code = emit_envelope(env, output_format=ctx.output_format)
+    code = emit_envelope(env, output_format=ctx.output_format, color=ctx.color_stderr)
     raise typer.Exit(code)
 
 
@@ -539,7 +542,7 @@ def _render_explain_or_dry_run(trace: ExplainTrace, ctx: RuntimeContext, *, stre
     if ctx.output_format is OutputFormat.JSON:
         print(render_explain_json(trace), file=stream)
     elif ctx.output_format is OutputFormat.TABLE:
-        render_explain_rich(trace, stream=stream)
+        render_explain_rich(trace, stream=stream, color=ctx.color)
     else:
         # CSV/YAML/JSONL on dry-run → JSON to stdout (the formatters expect rows,
         # not a structured trace). Consistent with spec §4.2.3 fallback rules.
@@ -607,6 +610,7 @@ def _render_response(
         columns=ctx.resolve_columns(op_tag, op_resource, operation),
         stream=stream,
         compact=ctx.compact,
+        color=ctx.color,
     )
 
 
@@ -614,6 +618,8 @@ def _render_delete_ok(ctx: RuntimeContext, *, stream: TextIO) -> None:
     payload = {"deleted": True}
     if ctx.output_format is OutputFormat.JSON:
         print(_json.dumps(payload), file=stream)
+    elif ctx.output_format is OutputFormat.TABLE and ctx.color:
+        make_console(stream, color=True).print("[yellow]deleted[/]")
     else:
         print("deleted", file=stream)
 
@@ -622,6 +628,8 @@ def _render_delete_already_absent(ctx: RuntimeContext, *, stream: TextIO) -> Non
     payload = {"deleted": False, "reason": "already_absent"}
     if ctx.output_format is OutputFormat.JSON:
         print(_json.dumps(payload), file=stream)
+    elif ctx.output_format is OutputFormat.TABLE and ctx.color:
+        make_console(stream, color=True).print("[dim]already absent (no change)[/]")
     else:
         print("already absent (no change)", file=stream)
 

{netbox_super_cli-1.0.4 → netbox_super_cli-1.0.6}/nsc/cli/runtime.py

@@ -15,7 +15,7 @@ from typing import Any, Literal
 from pydantic import BaseModel, ConfigDict, HttpUrl, SkipValidation
 
 from nsc.cache.store import ADHOC_PROFILE
-from nsc.config.models import Config, OutputFormat, Profile
+from nsc.config.models import ColorMode, Config, OutputFormat, Profile
 from nsc.config.settings import default_paths
 from nsc.http.client import NetBoxClient
 from nsc.http.errors import NetBoxAPIError, NetBoxClientError
@@ -178,6 +178,14 @@ def _url_only(value: str | None) -> str | None:
     return None
 
 
+def resolve_color(mode: ColorMode, *, is_tty: bool) -> bool:
+    if mode is ColorMode.ON:
+        return True
+    if mode is ColorMode.OFF:
+        return False
+    return is_tty
+
+
 class RuntimeContext(BaseModel):
     """Per-invocation runtime state.
 
@@ -198,6 +206,8 @@ class RuntimeContext(BaseModel):
     limit: int | None = None
     fetch_all: bool = False
     compact: bool = False
+    color: bool = False
+    color_stderr: bool = False
     apply: bool = False
     explain: bool = False
     strict: bool = False
@@ -302,7 +312,7 @@ def map_error(
     )
 
 
-def emit_envelope(env: ErrorEnvelope, *, output_format: OutputFormat) -> int:
+def emit_envelope(env: ErrorEnvelope, *, output_format: OutputFormat, color: bool = False) -> int:
     """Write the envelope to the right target and return the exit code."""
     target = select_render_target(output_format=output_format, stdout_is_tty=sys.stdout.isatty())
     if target is RenderTarget.JSON_STDOUT:
@@ -310,5 +320,5 @@ def emit_envelope(env: ErrorEnvelope, *, output_format: OutputFormat) -> int:
     elif target is RenderTarget.JSON_STDERR:
         print(render_to_json(env), file=sys.stderr)
     else:
-        render_to_rich_stderr(env, stream=sys.stderr)
+        render_to_rich_stderr(env, stream=sys.stderr, color=color)
     return EXIT_CODES.get(env.type, 1)

{netbox_super_cli-1.0.4 → netbox_super_cli-1.0.6}/nsc/config/models.py

@@ -31,11 +31,18 @@ class SchemaRefresh(StrEnum):
     WEEKLY = "weekly"
 
 
+class ColorMode(StrEnum):
+    AUTO = "auto"
+    ON = "on"
+    OFF = "off"
+
+
 class Defaults(_Frozen):
     output: OutputFormat = OutputFormat.TABLE
     page_size: int = 50
     timeout: float = 30.0
     schema_refresh: SchemaRefresh = SchemaRefresh.DAILY
+    color_mode: ColorMode = ColorMode.AUTO
 
 
 class Profile(_Frozen):

netbox_super_cli-1.0.6/nsc/output/_console.py

@@ -0,0 +1,16 @@
+"""Shared Rich Console factory used across all output formatters."""
+
+from __future__ import annotations
+
+from typing import TextIO
+
+from rich.console import Console
+
+
+def make_console(stream: TextIO, *, color: bool, soft_wrap: bool = False) -> Console:
+    # highlight=False on both paths: Rich's auto-highlighter bolds parens and
+    # colorizes numbers/IPs in plain strings, which would corrupt our output and
+    # fight the explicit semantic markup applied by the formatters.
+    if color:
+        return Console(file=stream, force_terminal=True, highlight=False, soft_wrap=soft_wrap)
+    return Console(file=stream, no_color=True, highlight=False, soft_wrap=soft_wrap)

{netbox_super_cli-1.0.4 → netbox_super_cli-1.0.6}/nsc/output/errors.py

@@ -10,11 +10,12 @@ from enum import Enum, StrEnum
 from typing import Any, Literal, TextIO
 
 from pydantic import BaseModel, ConfigDict, Field
-from rich.console import Console
+from rich.markup import escape
 from rich.panel import Panel
 
 from nsc.config.models import OutputFormat
 from nsc.model.command_model import HttpMethod
+from nsc.output._console import make_console
 
 
 class ErrorType(StrEnum):
@@ -97,25 +98,27 @@ def select_render_target(*, output_format: OutputFormat, stdout_is_tty: bool) ->
     return RenderTarget.JSON_STDERR
 
 
-def render_to_rich_stderr(env: ErrorEnvelope, *, stream: TextIO) -> None:
-    console = Console(file=stream, soft_wrap=True, force_terminal=False)
+def render_to_rich_stderr(env: ErrorEnvelope, *, stream: TextIO, color: bool = False) -> None:
+    console = make_console(stream, color=color, soft_wrap=True)
+    # env.error/endpoint/operation_id/audit_log_path/details carry server- and
+    # user-sourced text; escape it so stray [..] is not parsed as Rich markup.
     body_lines = [
-        f"[bold red]{env.type.value}[/]: {env.error}",
+        f"[bold red]{env.type.value}[/]: {escape(env.error)}",
     ]
     if env.endpoint:
-        body_lines.append(f"endpoint: {env.endpoint}")
+        body_lines.append(f"endpoint: {escape(env.endpoint)}")
     if env.method is not None:
         body_lines.append(f"method:   {env.method.value}")
     if env.status_code is not None:
         body_lines.append(f"status:   {env.status_code}")
     if env.operation_id:
-        body_lines.append(f"op:       {env.operation_id}")
+        body_lines.append(f"op:       {escape(env.operation_id)}")
     if env.attempt_n is not None:
         body_lines.append(f"attempt:  {env.attempt_n}")
     if env.audit_log_path:
-        body_lines.append(f"audit:    {env.audit_log_path}")
+        body_lines.append(f"audit:    {escape(env.audit_log_path)}")
     if env.details:
-        body_lines.append(f"details:  {env.details}")
+        body_lines.append(f"details:  {escape(str(env.details))}")
     console.print(Panel("\n".join(body_lines), title="nsc error", border_style="red"))
 
 

{netbox_super_cli-1.0.4 → netbox_super_cli-1.0.6}/nsc/output/explain.py

@@ -11,7 +11,7 @@ from __future__ import annotations
 from typing import Any, Literal, TextIO
 
 from pydantic import BaseModel, ConfigDict, Field
-from rich.console import Console
+from rich.markup import escape
 from rich.panel import Panel
 
 from nsc.cli.writes.apply import ResolvedRequest
@@ -19,6 +19,7 @@ from nsc.cli.writes.bulk import RoutingDecision
 from nsc.cli.writes.input import RawWriteInput
 from nsc.cli.writes.preflight import PreflightResult
 from nsc.model.command_model import Operation
+from nsc.output._console import make_console
 
 DECISION_CAP = 200
 
@@ -151,26 +152,28 @@ def render_to_json(trace: ExplainTrace) -> str:
     return trace.model_dump_json()
 
 
-def render_to_rich_stdout(trace: ExplainTrace, *, stream: TextIO) -> None:
-    console = Console(file=stream, soft_wrap=True, force_terminal=False)
+def render_to_rich_stdout(trace: ExplainTrace, *, stream: TextIO, color: bool = False) -> None:
+    console = make_console(stream, color=color, soft_wrap=True)
+    # Reasoning strings and decision values carry user/schema/server text that
+    # may contain [..]; escape it so it is not parsed as Rich markup.
     body = [
-        f"[bold cyan]operation:[/] {trace.operation_id}",
+        f"[bold cyan]operation:[/] {escape(trace.operation_id)}",
     ]
     if trace.operation_summary:
-        body.append(f"summary:   {trace.operation_summary}")
-    body.append(f"method:    {trace.method_reasoning}")
-    body.append(f"url:       {trace.url_reasoning}")
+        body.append(f"summary:   {escape(trace.operation_summary)}")
+    body.append(f"method:    {escape(trace.method_reasoning)}")
+    body.append(f"url:       {escape(trace.url_reasoning)}")
     if trace.bulk_reasoning:
-        body.append(f"bulk:      {trace.bulk_reasoning}")
+        body.append(f"bulk:      {escape(trace.bulk_reasoning)}")
     for r in trace.requests:
-        body.append(f"  → {r.method.value} {r.url}")
+        body.append(f"  → {r.method.value} {escape(r.url)}")
         if r.body is not None:
-            body.append(f"    body: {r.body}")
+            body.append(f"    body: {escape(str(r.body))}")
     if trace.preflight is not None and not trace.preflight.ok:
         body.append("preflight: [red]FAIL[/]")
         for issue in trace.preflight.issues:
             loc = f"records[{issue.record_index}].{issue.field_path}"
-            body.append(f"  [{issue.kind}] {loc}: {issue.message}")
+            body.append(escape(f"  [{issue.kind}] {loc}: {issue.message}"))
     elif trace.preflight is not None:
         body.append("preflight: [green]OK[/]")
     if trace.decisions:
@@ -178,7 +181,9 @@ def render_to_rich_stdout(trace: ExplainTrace, *, stream: TextIO) -> None:
         for d in trace.decisions:
             extra = f" — {d.note}" if d.note else ""
             body.append(
-                f"  {d.field_path}  [{d.source}]  {d.raw_value!r} → {d.resolved_value!r}{extra}"
+                escape(
+                    f"  {d.field_path}  [{d.source}]  {d.raw_value!r} → {d.resolved_value!r}{extra}"
+                )
             )
         if trace.decisions_truncated:
             body.append(f"  … truncated at {DECISION_CAP} entries")

netbox_super_cli-1.0.6/nsc/output/flatten.py

@@ -0,0 +1,49 @@
+"""Shared dotted-path flattener used by table and csv formatters."""
+
+from __future__ import annotations
+
+import json
+from typing import Any
+
+
+def flatten(record: dict[str, Any], *, columns: list[str] | None = None) -> dict[str, Any]:
+    if columns is None:
+        flat: dict[str, Any] = {}
+        _walk(record, "", flat)
+        return flat
+    return {col: _select(record, col) for col in columns}
+
+
+def _walk(value: Any, prefix: str, out: dict[str, Any]) -> None:
+    if isinstance(value, dict):
+        for k, v in value.items():
+            child = f"{prefix}.{k}" if prefix else k
+            _walk(v, child, out)
+    elif isinstance(value, list):
+        out[prefix] = json.dumps(value)
+    else:
+        out[prefix] = value
+
+
+def _select(record: dict[str, Any], path: str) -> Any:
+    cur: Any = record
+    for part in path.split("."):
+        if isinstance(cur, dict) and part in cur:
+            cur = cur[part]
+        else:
+            return ""
+    return _displayify(cur)
+
+
+def _displayify(value: Any) -> Any:
+    if isinstance(value, dict):
+        display = value.get("display")
+        return display if isinstance(display, str) else json.dumps(value, separators=(",", ":"))
+    if isinstance(value, list):
+        return ", ".join(_as_cell(v) for v in value)
+    return value
+
+
+def _as_cell(value: Any) -> str:
+    rendered = _displayify(value)
+    return "" if rendered is None else str(rendered)

{netbox_super_cli-1.0.4 → netbox_super_cli-1.0.6}/nsc/output/render.py

@@ -16,6 +16,7 @@ def render(
     columns: list[str] | None = None,
     stream: TextIO = sys.stdout,
     compact: bool = False,
+    color: bool = False,
 ) -> None:
     if format is OutputFormat.JSON:
         json_.render(data, stream=stream, compact=compact)
@@ -26,7 +27,7 @@ def render(
     elif format is OutputFormat.CSV:
         csv_.render(data, stream=stream, columns=columns)
     elif format is OutputFormat.TABLE:
-        table.render(data, stream=stream, columns=columns)
+        table.render(data, stream=stream, columns=columns, color=color)
     else:  # pragma: no cover  (StrEnum exhaustively covered above)
         raise ValueError(f"unknown output format: {format!r}")
 

netbox_super_cli-1.0.6/nsc/output/table.py

@@ -0,0 +1,82 @@
+"""Rich table output formatter."""
+
+from __future__ import annotations
+
+import sys
+from typing import Any, TextIO
+
+from rich.markup import escape
+from rich.table import Table
+
+from nsc.output._console import make_console
+from nsc.output.flatten import flatten
+
+_STATUS_COLORS: dict[str, str] = {
+    "active": "green",
+    "enabled": "green",
+    "online": "green",
+    "connected": "green",
+    "planned": "yellow",
+    "staged": "yellow",
+    "decommissioning": "yellow",
+    "failed": "red",
+    "disabled": "red",
+    "offline": "red",
+    "error": "red",
+    "true": "green",
+    "false": "dim",
+}
+
+
+def render(
+    data: list[dict[str, Any]] | dict[str, Any],
+    *,
+    stream: TextIO = sys.stdout,
+    columns: list[str] | None = None,
+    color: bool = False,
+) -> None:
+    records = [data] if isinstance(data, dict) else list(data)
+    if not records:
+        make_console(stream, color=color).print("(no records)")
+        return
+
+    flat_records = [flatten(r, columns=columns) for r in records]
+    fieldnames = columns if columns is not None else _gather_fieldnames(flat_records)
+
+    table = Table(show_header=True, header_style="bold")
+    for col in fieldnames:
+        table.add_column(col)
+    for r in flat_records:
+        table.add_row(*[_format_cell(r.get(col, ""), color=color) for col in fieldnames])
+
+    make_console(stream, color=color).print(table)
+
+
+def _gather_fieldnames(records: list[dict[str, Any]]) -> list[str]:
+    seen: dict[str, None] = {}
+    for r in records:
+        for k in r:
+            seen.setdefault(k, None)
+    return list(seen.keys())
+
+
+def _format_cell(value: Any, *, color: bool = False) -> str:
+    if value is None:
+        text = ""
+    elif isinstance(value, bool):
+        text = "true" if value else "false"
+    else:
+        text = str(value)
+
+    # Rich parses markup in table cells regardless of the no-color setting, so
+    # arbitrary cell values must be escaped on every return path.
+    if not color:
+        return escape(text)
+
+    if not text:
+        return "[dim]-[/]"
+
+    style = _STATUS_COLORS.get(text.lower())
+    if style:
+        return f"[{style}]{escape(text)}[/]"
+    return escape(text)

{netbox_super_cli-1.0.4 → netbox_super_cli-1.0.6}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "netbox-super-cli"
-version = "1.0.4"
+version = "1.0.6"
 description = "Dynamic NetBox CLI generated from the live OpenAPI schema."
 readme = "README.md"
 requires-python = ">=3.12"

netbox_super_cli-1.0.6/tests/cli/test_globals_color.py

@@ -0,0 +1,63 @@
+from __future__ import annotations
+
+import sys
+
+import pytest
+
+from nsc.cli import globals as globals_mod
+from nsc.cli.globals import GlobalState, build_runtime_context
+from nsc.cli.runtime import CLIOverrides
+from nsc.config.models import Config
+from nsc.model.command_model import CommandModel
+
+
+def _state() -> GlobalState:
+    return GlobalState(overrides=CLIOverrides(), config=Config(), debug=False)
+
+
+def _stub_command_model(monkeypatch: pytest.MonkeyPatch) -> None:
+    monkeypatch.setattr(
+        globals_mod,
+        "resolve_command_model",
+        lambda **kwargs: CommandModel(info_title="t", info_version="v", schema_hash="x"),
+    )
+
+
+def test_build_runtime_context_gates_stderr_color_independently(
+    monkeypatch: pytest.MonkeyPatch,
+) -> None:
+    """color (stdout) and color_stderr (stderr) must each follow their own TTY.
+
+    Regression: build_runtime_context derived both flags from sys.stdout.isatty(),
+    so `nsc ... 2>err.log` (stdout a TTY, stderr a file) leaked ANSI into the file.
+    """
+    monkeypatch.setenv("NSC_URL", "https://nb.example/")
+    monkeypatch.setenv("NSC_TOKEN", "tok")
+    for var in ("NSC_PROFILE", "NSC_SCHEMA", "NSC_OUTPUT"):
+        monkeypatch.delenv(var, raising=False)
+    _stub_command_model(monkeypatch)
+    monkeypatch.setattr(sys.stdout, "isatty", lambda: True)
+    monkeypatch.setattr(sys.stderr, "isatty", lambda: False)
+
+    ctx = build_runtime_context(_state())
+
+    assert ctx.color is True
+    assert ctx.color_stderr is False
+
+
+def test_build_runtime_context_gates_stdout_color_independently(
+    monkeypatch: pytest.MonkeyPatch,
+) -> None:
+    """The reverse: stdout piped, stderr a TTY — error panels stay colored."""
+    monkeypatch.setenv("NSC_URL", "https://nb.example/")
+    monkeypatch.setenv("NSC_TOKEN", "tok")
+    for var in ("NSC_PROFILE", "NSC_SCHEMA", "NSC_OUTPUT"):
+        monkeypatch.delenv(var, raising=False)
+    _stub_command_model(monkeypatch)
+    monkeypatch.setattr(sys.stdout, "isatty", lambda: False)
+    monkeypatch.setattr(sys.stderr, "isatty", lambda: True)
+
+    ctx = build_runtime_context(_state())
+
+    assert ctx.color is False
+    assert ctx.color_stderr is True