ktalk-cli 4.0.0__tar.gz → 4.0.1__tar.gz

{ktalk_cli-4.0.0 → ktalk_cli-4.0.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: ktalk-cli
-Version: 4.0.0
+Version: 4.0.1
 Summary: CLI for accessing Kontur Talk (KTalk) recordings, transcripts and summaries
 Project-URL: Homepage, https://github.com/mdemyanov/ktalk-cli
 Project-URL: Repository, https://github.com/mdemyanov/ktalk-cli

{ktalk_cli-4.0.0 → ktalk_cli-4.0.1}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "ktalk-cli"
-version = "4.0.0"
+version = "4.0.1"
 description = "CLI for accessing Kontur Talk (KTalk) recordings, transcripts and summaries"
 readme = "README.md"
 license = "MIT"

{ktalk_cli-4.0.0 → ktalk_cli-4.0.1}/src/ktalk_cli/cli_token.py

@@ -65,10 +65,11 @@ def _cmd_status(args: argparse.Namespace) -> int:
         return EXIT_OK
     print(f"present: True, права {mode}")
     if token is None:
-        print(
-            "usable: False — файл пуст или его права шире 0600; "
-            f"почините: chmod 600 {path}"
-        )
+        # Совет приходит из `status_payload` — пара (причина × платформа):
+        # chmod существует только на платформах с выразимыми POSIX-битами и
+        # только для причины «права шире» (ADR-028 §3).
+        advice = payload["advice"] or "запишите токен: ktalk token set -"
+        print(f"usable: False — {advice}")
         return EXIT_OK
     print(f"usable: True, {_mask(token)}")
     return EXIT_OK

{ktalk_cli-4.0.0 → ktalk_cli-4.0.1}/src/ktalk_cli/fs_guard.py

@@ -26,14 +26,29 @@ def open_temp_exclusive(tmp: Path, mode: int) -> int:
     `O_NOFOLLOW`/`O_EXCL` — симлинк, подложенный на место `.tmp` между
     проверкой и открытием, оставался рабочим путём записи.
 
-    `O_NOFOLLOW` отказывается идти по уже существующему симлинку. `O_EXCL`
-    отказывается, если `.tmp` уже существует (обычный файл — остаток
+    `O_NOFOLLOW` отказывается идти по уже существующему симлинку; подставляется
+    `getattr(os, "O_NOFOLLOW", 0)`, а не константой: атрибут есть не на всех
+    платформах (нативный Windows его не имеет — ADR-028 §1), и безусловная
+    константа роняла обе команды записи `AttributeError`-ом ещё до записи
+    значения. На POSIX `getattr` возвращает ту же константу — набор флагов
+    бит-в-бит прежний. Отсутствие `O_NOFOLLOW` защиты не снимает: `O_CREAT|O_EXCL`
+    отказывает на любом уже существующем `.tmp`, включая подложенный симлинк —
+    для create-new ссылка не преследуется ни на POSIX (`O_EXCL`, POSIX.1-2008),
+    ни на Windows (`CREATE_NEW`); ветка повтора снимает остаток `unlink`-ом,
+    который никогда не идёт по симлинку, и вторая попытка снова под `O_EXCL` —
+    выигравший гонку атакующий получает отказ, тихой записи по подмене нет.
+    Остаточная роль `O_NOFOLLOW` — различие errno (ELOOP против EEXIST) на
+    существующей записи; создать симлинк на Windows можно только с привилегией
+    (`SeCreateSymbolicLinkPrivilege` либо DevMode) — у непривилегированного
+    локального атакующего из модели угроз хода нет.
+
+    `O_EXCL` отказывается, если `.tmp` уже существует (обычный файл — остаток
     прерванной записи, либо симлинк, подложенный другим локальным
     пользователем) — обрабатывается только безобидная ветка: остаток
     удаляется (`unlink` никогда не идёт по симлинку) и попытка повторяется
     один раз; если атакующий выиграл гонку заново, вторая попытка тоже
     проваливается — тихого успеха через подмену нет."""
-    flags = os.O_WRONLY | os.O_CREAT | os.O_EXCL | os.O_NOFOLLOW
+    flags = os.O_WRONLY | os.O_CREAT | os.O_EXCL | getattr(os, "O_NOFOLLOW", 0)
     try:
         return os.open(tmp, flags, mode)
     except FileExistsError:

{ktalk_cli-4.0.0 → ktalk_cli-4.0.1}/src/ktalk_cli/token_file.py

@@ -12,7 +12,13 @@
 
 Права шире `0600` читаются как «файла нет» — тот же fail-closed барьер, что у
 санкции записи (SEC-006). Это защита не от владельца учётной записи, а от другого
-пользователя машины, которому файл стал доступен.
+пользователя машины, которому файл стал доступен. Барьер применяется только там,
+где POSIX-битам есть смысл (`os.name != "nt"`, ADR-028 §2): нативный `stat` на
+Windows сообщает `0o666` для любого обычного файла, барьер отвергал бы там каждый
+рукописный файл независимо от содержимого; границей защиты на этой платформе
+служит изоляция профиля пользователя, и инструмент не посылает файлу явного
+послабления. На обеих платформах пригодность означает корректную форму значения
+(`_TOKEN_RE`), а не любое непустое содержимое (FR-51).
 """
 
 from __future__ import annotations
@@ -42,40 +48,81 @@ def token_path() -> Path:
     return root / "ktalk-mcp" / "token"
 
 
-def file_mode() -> str | None:
-    """Права файла как `0600` — для диагностики. `None`, если файла нет."""
+def _verdict() -> tuple[str | None, str | None, str | None]:
+    """Вердикт одним проходом: `(токен, права, причина отказа)`. Права — строка
+    вида `0600` для диагностики, `None`, если файла нет. Причина — `None` для
+    пригодного файла, иначе `not_present` / `wide_mode` / `unreadable` / `empty`
+    / `bad_content`; её читает совет диагностики (`status_payload`, ADR-028 §2-3).
+
+    Платформенная развилка читает `os.name` в момент вызова, а не на импорте:
+    ось тестов (NFR-26) подменяет атрибут в окне вызова."""
+    path = token_path()
     try:
-        return format(stat.S_IMODE(token_path().stat().st_mode), "04o")
+        st_mode = stat.S_IMODE(path.stat().st_mode)
     except OSError:
-        return None
+        return None, None, "not_present"
+    mode = format(st_mode, "04o")
+    if os.name != "nt" and st_mode & 0o077:
+        return None, mode, "wide_mode"
+    try:
+        raw = path.read_text(encoding="utf-8")
+    except (OSError, ValueError, UnicodeDecodeError):
+        return None, mode, "unreadable"
+    token = raw.strip()
+    if not token:
+        return None, mode, "empty"
+    if not _TOKEN_RE.match(token):
+        return None, mode, "bad_content"
+    return token, mode, None
+
+
+def file_mode() -> str | None:
+    """Права файла как `0600` — для диагностики. `None`, если файла нет."""
+    return _verdict()[1]
 
 
 def read_token() -> str | None:
-    """Значение токена или `None` — если файла нет, он нечитаем, пуст или его
-    права шире `0600`. Ни одна из этих причин не является ошибкой: вызывающая
+    """Значение токена или `None` — если файла нет, он нечитаем, пуст, его
+    содержимое не по форме токена или (везде, кроме родной Windows) его права
+    шире `0600`. Ни одна из этих причин не является ошибкой: вызывающая
     сторона просто переходит к следующему источнику (или к отказу конфигурации)."""
-    path = token_path()
-    try:
-        if stat.S_IMODE(path.stat().st_mode) & 0o077:
-            return None
-        raw = path.read_text(encoding="utf-8")
-    except (OSError, ValueError, UnicodeDecodeError):
+    return _verdict()[0]
+
+
+def _remediation_advice(cause: str | None) -> str | None:
+    """Совет для причины «не пригоден» — пара (причина × платформа), собирается
+    здесь же, чтобы `cli_token` и `doctor` печатали одно и то же (ADR-026,
+    ADR-028 §3): chmod-совет существует только там, где POSIX-биты выразимы, и
+    только для причины «права шире» — на родной Windows он невыполним и не
+    печатается."""
+    if cause is None:
         return None
-    token = raw.strip()
-    return token or None
+    if cause == "wide_mode" and os.name != "nt":
+        return f"права шире 0600; почините: chmod 600 {token_path()}"
+    if cause == "empty":
+        return "файл пуст; запишите токен заново: ktalk token set -"
+    if cause == "bad_content":
+        return (
+            "содержимое не похоже на токен (одно слово из букв и цифр, "
+            "16-64 символа); запишите заново: ktalk token set -"
+        )
+    return "запишите токен: ktalk token set -"
 
 
 def status_payload() -> dict:
-    """Путь/права/пригодность файла токена одним словарём — единственная точка
-    сборки, которую переиспользуют `cli_token.py::_cmd_status` и `cli_doctor.py`
+    """Путь/права/пригодность/причина одним словарём — единственная точка сборки,
+    которую переиспользуют `cli_token.py::_cmd_status` и `cli_doctor.py`
     (ADR-026 spec §2: значение совпадает буквальным переиспользованием кода, не
-    соглашением между двумя независимо написанными местами)."""
-    mode = file_mode()
+    соглашением между двумя независимо написанными местами). `cause`/`advice` —
+    пара (причина × платформа) из ADR-028 §3."""
+    token, mode, cause = _verdict()
     return {
         "path": str(token_path()),
         "present": mode is not None,
         "mode": mode,
-        "usable": read_token() is not None,
+        "usable": token is not None,
+        "cause": cause,
+        "advice": _remediation_advice(cause),
     }
 
 

{ktalk_cli-4.0.0 → ktalk_cli-4.0.1}/tests/test_secret_masking.py

@@ -17,7 +17,7 @@ import pytest
 from pytest_httpx import HTTPXMock
 
 SECRET = "pk-supersecret-abcdef123456"
-SESSION_SECRET = "sess-supersecret-abcdef123456"
+SESSION_SECRET = "sesssupersecretabcdef123456"
 
 
 @pytest.fixture