kd-cli 2.6.3__tar.gz → 2.6.4__tar.gz

{kd_cli-2.6.3/kd_cli.egg-info → kd_cli-2.6.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: kd-cli
-Version: 2.6.3
+Version: 2.6.4
 Summary: Knowledge Dispatcher for issue trackers and knowledge bases
 Author: Sergiy Gladkyy
 License-Expression: MIT

{kd_cli-2.6.3 → kd_cli-2.6.4}/kd/cli/input.py

@@ -25,9 +25,23 @@ def resolve_text_input(value: str | None) -> str | None:
     if not value:
         return value
     if value == "-":
-        return sys.stdin.read()
+        try:
+            return sys.stdin.read()
+        except UnicodeDecodeError as e:
+            raise ValidationError(
+                f"Stdin input could not be decoded as UTF-8: {e.reason}",
+                suggestion="Ensure the upstream shell pipes UTF-8 bytes "
+                "(heredoc, pipe, or redirected file).",
+            ) from e
     if value == "@-":
-        text = sys.stdin.read()
+        try:
+            text = sys.stdin.read()
+        except UnicodeDecodeError as e:
+            raise ValidationError(
+                f"Stdin input could not be decoded as UTF-8: {e.reason}",
+                suggestion="Ensure the upstream shell pipes UTF-8 bytes "
+                "(heredoc, pipe, or redirected file).",
+            ) from e
         if not text:
             raise ValidationError("No input received from stdin")
         return text
@@ -35,7 +49,13 @@ def resolve_text_input(value: str | None) -> str | None:
         file_path = Path(value[1:])
         if not file_path.is_file():
             raise ValidationError(f"File not found: {file_path}")
-        text = file_path.read_text(encoding="utf-8")
+        try:
+            text = file_path.read_text(encoding="utf-8")
+        except UnicodeDecodeError as e:
+            raise ValidationError(
+                f"File {file_path} could not be decoded as UTF-8: {e.reason}",
+                suggestion="Save the file as UTF-8 (without a BOM) and retry.",
+            ) from e
         if not text:
             raise ValidationError(f"File is empty: {file_path}")
         return text

{kd_cli-2.6.3 → kd_cli-2.6.4}/kd/cli/main.py

@@ -142,17 +142,24 @@ def _print_error(error: KdError) -> None:
 
 
 def _configure_windows_stdio() -> None:
-    """Force UTF-8 on stdout/stderr when running on Windows.
-
-    No-op on POSIX. Errors during reconfigure are swallowed; the wrapper's
-    UnicodeEncodeError handler is the defensive backstop for the rare
-    embedded-interpreter case where reconfigure is unavailable.
+    """Force UTF-8 on stdin/stdout/stderr when running on Windows.
+
+    No-op on POSIX. Stdout/stderr use ``errors="replace"`` so a single
+    corrupted byte never aborts a whole command's output. Stdin uses
+    ``errors="strict"`` so invalid bytes raise loudly rather than silently
+    writing replacement chars to disk via the input layer (see
+    ``kd/cli/input.py``). Errors during reconfigure are swallowed; embedded
+    interpreters and pytest captures may lack ``reconfigure`` on any of the
+    three streams.
     """
     if sys.platform != "win32":
         return
-    for stream in (sys.stdout, sys.stderr):
+    for stream in (sys.stdin, sys.stdout, sys.stderr):
         try:
-            stream.reconfigure(encoding="utf-8", errors="replace")  # type: ignore[union-attr,unused-ignore]
+            stream.reconfigure(  # type: ignore[union-attr,unused-ignore]
+                encoding="utf-8",
+                errors="strict" if stream is sys.stdin else "replace",
+            )
         except Exception:
             pass
 

{kd_cli-2.6.3 → kd_cli-2.6.4}/kd/cli/namespaces/system/__init__.py

@@ -709,7 +709,7 @@ def _resolve_target(name: str) -> tuple[Path, Path, str]:
 
 def _diff_summary(snapshot: SnapshotInfo, current_text: str) -> str:
     """One-line diff summary: ``+N -M`` lines vs current file."""
-    snap_lines = snapshot.path.read_text().splitlines()
+    snap_lines = snapshot.path.read_text(encoding="utf-8").splitlines()
     curr_lines = current_text.splitlines()
     added = removed = 0
     for line in difflib.unified_diff(snap_lines, curr_lines, n=0):
@@ -751,7 +751,7 @@ class SystemHistoryCommand(Command):
         rows: list[dict[str, Any]] = []
         for target_name in targets:
             live_path, history_dir, filename = _resolve_target(target_name)
-            current_text = live_path.read_text() if live_path.exists() else ""
+            current_text = live_path.read_text(encoding="utf-8") if live_path.exists() else ""
             for snap in list_snapshots(history_dir, filename):
                 rows.append(
                     {
@@ -791,7 +791,7 @@ class SystemRestoreCommand(Command):
         snap_path = get_snapshot_path(history_dir, filename, timestamp)
         # Snapshot current before restoring (so restore is undoable)
         snapshot_before_write(live_path)
-        restored_text = snap_path.read_text()
+        restored_text = snap_path.read_text(encoding="utf-8")
         atomic_write_text(live_path, restored_text)
         return {
             "ok": True,
@@ -830,15 +830,15 @@ class SystemDiffCommand(Command):
         live_path, history_dir, filename = _resolve_target(target_name)
 
         snap_a = get_snapshot_path(history_dir, filename, args[1])
-        text_a = snap_a.read_text()
+        text_a = snap_a.read_text(encoding="utf-8")
         label_a = f"{filename}.{args[1]}"
 
         if len(args) >= 3:
             snap_b = get_snapshot_path(history_dir, filename, args[2])
-            text_b = snap_b.read_text()
+            text_b = snap_b.read_text(encoding="utf-8")
             label_b = f"{filename}.{args[2]}"
         else:
-            text_b = live_path.read_text() if live_path.exists() else ""
+            text_b = live_path.read_text(encoding="utf-8") if live_path.exists() else ""
             label_b = filename
 
         result = diff_text(text_a, text_b, label_a, label_b)

{kd_cli-2.6.3 → kd_cli-2.6.4}/kd/config.py

@@ -406,7 +406,7 @@ def load_config(path: Path | None = None) -> YtctlConfig:
     if not config_path.exists():
         return YtctlConfig()
     try:
-        raw = yaml.safe_load(config_path.read_text()) or {}
+        raw = yaml.safe_load(config_path.read_text(encoding="utf-8")) or {}
         return YtctlConfig(**raw)
     except (yaml.YAMLError, TypeError, ValueError) as e:
         raise ConfigError(f"Invalid config at {config_path}: {e}") from e
@@ -511,7 +511,7 @@ def update_config_path(path: str, value: str, config_path: Path) -> tuple[str, A
     # Load raw YAML, capture original text for rollback.
     raw = _load_raw_yaml(config_path)
     original_text: str | None
-    original_text = config_path.read_text() if config_path.exists() else None
+    original_text = config_path.read_text(encoding="utf-8") if config_path.exists() else None
 
     retention = _retention_from_raw(raw, DEFAULT_RETENTION)
     snapshot_before_write(config_path, retention=retention)
@@ -619,7 +619,7 @@ def _load_raw_yaml(path: Path) -> dict[str, Any]:
     if not path.exists():
         return {}
     try:
-        data = yaml.safe_load(path.read_text())
+        data = yaml.safe_load(path.read_text(encoding="utf-8"))
     except yaml.YAMLError as e:
         raise ConfigError(f"Invalid YAML in {path}: {e}") from e
     return data or {}
@@ -1395,16 +1395,16 @@ def _manage_gitignore(directory: Path) -> str:
 
     gitignore = git_root / ".gitignore"
     if gitignore.exists():
-        content = gitignore.read_text()
+        content = gitignore.read_text(encoding="utf-8")
         lines = {line.strip() for line in content.splitlines()}
         if lines & _GITIGNORE_PATTERNS:
             return "already_ignored"
         # Append with a trailing newline if file doesn't end with one
         sep = "" if content.endswith("\n") else "\n"
-        gitignore.write_text(f"{content}{sep}\n# kd local config\n.kd/\n")
+        gitignore.write_text(f"{content}{sep}\n# kd local config\n.kd/\n", encoding="utf-8")
         return "added"
 
-    gitignore.write_text("# kd local config\n.kd/\n")
+    gitignore.write_text("# kd local config\n.kd/\n", encoding="utf-8")
     return "created"
 
 
@@ -1466,7 +1466,7 @@ def init_project_config(
         )
 
     config_dir.mkdir(parents=True, exist_ok=True)
-    config_path.write_text(template)
+    config_path.write_text(template, encoding="utf-8")
 
     if global_tier:
         gitignore_action = "not_applicable"

{kd_cli-2.6.3 → kd_cli-2.6.4}/kd/config_read.py

@@ -174,7 +174,7 @@ def _get_history(path: str, key: str) -> dict[str, Any] | None:
         if value is None:
             continue
         # Only surface values actually written to the file, not schema defaults.
-        raw = yaml.safe_load(tier_path.read_text()) or {}
+        raw = yaml.safe_load(tier_path.read_text(encoding="utf-8")) or {}
         history_block = raw.get("history") if isinstance(raw.get("history"), dict) else {}
         if not isinstance(history_block, dict) or key not in history_block:
             continue

{kd_cli-2.6.3 → kd_cli-2.6.4}/kd/primer.py

@@ -121,7 +121,7 @@ def load_primer(path: Path | None = None) -> PrimerData:
     if not primer_file.exists():
         return PrimerData()
     try:
-        raw = yaml.safe_load(primer_file.read_text()) or {}
+        raw = yaml.safe_load(primer_file.read_text(encoding="utf-8")) or {}
         return PrimerData(**raw)
     except (yaml.YAMLError, TypeError, ValueError) as e:
         raise ConfigError(f"Invalid primer at {primer_file}: {e}") from e

{kd_cli-2.6.3 → kd_cli-2.6.4}/kd/resources/CHANGELOG.md

@@ -4,6 +4,34 @@ All notable changes to kd-cli are recorded here. The project follows [Semantic V
 
 ## [Unreleased]
 
+## [2.6.4] — 2026-04-30
+
+This release closes the Windows UTF-8 audit started in 2.6.3. The
+stream-encoding side (stdout/stderr) shipped earlier; 2.6.4 closes the
+file-IO and stdin sides so primer notes, `--content @file`, and heredoc
+or piped input round-trip non-cp1252 characters cleanly.
+
+### Fixed
+- File reads and writes on Windows are now explicit UTF-8 at every
+  kd-owned text-IO call site (primer, config, history/diff snapshots,
+  `--description @file` / `--content @file` content). The headline
+  reproducer was `kd /primer show` raising `'charmap' codec can't
+  decode byte 0x9d` whenever `.kd/primer.yaml` contained any non-cp1252
+  byte (em-dash, arrow, right double quotation mark) — a single primer
+  note authored on macOS/Linux was enough to read-block every
+  downstream Windows session (KDCLI-170).
+- Stdin on Windows is now forced to UTF-8 with strict error handling,
+  so heredocs and pipes carrying non-ASCII content (em-dash, arrow,
+  right double quotation mark) into `kd /primer/note update -`,
+  `kd /article create --content @-`, `--description -`, or any other
+  stdin-consuming command no longer get cp1252-mojibaked at the input
+  layer before being persisted. Invalid UTF-8 bytes on stdin (and
+  non-UTF-8 `@file` content) now surface as a clean
+  `Error: Stdin input could not be decoded as UTF-8 ... / Hint: Ensure
+  the upstream shell pipes UTF-8 bytes ...` (or
+  `Hint: Save the file as UTF-8 ...`) instead of the generic "internal
+  error: ... This is a bug." block (KDCLI-169).
+
 ## [2.6.3] — 2026-04-30
 
 This release focuses on Windows shakedown: encoding crashes, primer

{kd_cli-2.6.3 → kd_cli-2.6.4}/kd/resources/operating-guide.md

@@ -244,7 +244,7 @@ kd -DDD /issue list --project KDCLI
 
 Debug output goes to stderr and tokens are redacted. Use `-DDD` when investigating upstream API behavior because it includes request and response details.
 
-On Windows, kd forces UTF-8 stdout and stderr at the entry point so non-ASCII characters (`→`, `—`, `≤`, em-dash) render correctly regardless of the console's default code page (cp1252 on most installs). No `PYTHONIOENCODING` workaround is needed.
+On Windows, kd forces UTF-8 on stdin, stdout, and stderr at the entry point so non-ASCII characters (`→`, `—`, `≤`, em-dash) render correctly regardless of the console's default code page (cp1252 on most installs), and every kd-owned file read/write (primer, config, history snapshots, `--content @file` and `--description @file`) uses explicit UTF-8 so primer/config files written by macOS or Linux load cleanly on Windows. Stdin uses strict decoding so invalid UTF-8 piped into `kd /primer/note update -` or `kd /article create --content @-` fails loudly with a clear diagnostic (`Error: Stdin input could not be decoded as UTF-8 ...`) rather than silently corrupting persisted content. No `PYTHONIOENCODING` or `PYTHONUTF8` workaround is needed.
 
 Unhandled exceptions render a one-line or one-paragraph friendly diagnostic and exit non-zero. To see the full traceback (for bug reports or local debugging), re-run with `--debug` / `-D`; the wrapper re-raises the original exception so the runtime prints the traceback. `KdError` (expected user-facing failures) keep their existing one-line `Error: ...` / optional `Hint: ...` shape on every platform.
 

{kd_cli-2.6.3 → kd_cli-2.6.4/kd_cli.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: kd-cli
-Version: 2.6.3
+Version: 2.6.4
 Summary: Knowledge Dispatcher for issue trackers and knowledge bases
 Author: Sergiy Gladkyy
 License-Expression: MIT

{kd_cli-2.6.3 → kd_cli-2.6.4}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "kd-cli"
-version = "2.6.3"
+version = "2.6.4"
 description = "Knowledge Dispatcher for issue trackers and knowledge bases"
 readme = "README.md"
 requires-python = ">=3.11"