gitflic-cli-mcp 0.2.0 → 0.3.0

package/README.md

@@ -72,6 +72,7 @@ claude mcp add gitflic --env GITFLIC_TOKEN=xxxxxxxx -- npx -y gitflic-cli-mcp
 |---|---|---|
 | `GITFLIC_MCP_GROUPS` | список групп через запятую/пробел | оставить только тулзы указанных групп |
 | `GITFLIC_MCP_READONLY` | `1` / `true` / `yes` / `on` | оставить только read-only тулзы (без мутаций) |
+| `GITFLIC_MCP_NO_DESTRUCTIVE` | `1` / `true` / `yes` / `on` | убрать деструктивные тулзы (удаление) — см. ниже |
 | `GITFLIC_MCP_TOOLS` | список имён тулзов | явный allowlist (имеет приоритет над `GROUPS`) |
 | `GITFLIC_MCP_EXCLUDE` | список имён тулзов | убрать конкретные тулзы (применяется последним) |
 
@@ -81,7 +82,8 @@ claude mcp add gitflic --env GITFLIC_TOKEN=xxxxxxxx -- npx -y gitflic-cli-mcp
 
 1. Базовый набор: `GITFLIC_MCP_TOOLS`, если задан; иначе `GITFLIC_MCP_GROUPS`, если задан; иначе — все тулзы.
 2. `GITFLIC_MCP_READONLY` — пересечение с read-only (накладывается поверх **в т.ч. на `GITFLIC_MCP_TOOLS`** — write-тул, явно указанный в allowlist, всё равно будет отсеян; так read-only режим гарантированно не пропускает мутации).
-3. `GITFLIC_MCP_EXCLUDE` — вычитается в самом конце (всегда побеждает).
+3. `GITFLIC_MCP_NO_DESTRUCTIVE` — вычитает деструктивные (удаление) тулзы (поверх базы/READONLY).
+4. `GITFLIC_MCP_EXCLUDE` — вычитается в самом конце (всегда побеждает).
 
 Значения списков — через запятую или пробел; регистр не важен ни для групп, ни для имён тулзов (имена всегда в нижнем регистре).
 
@@ -103,6 +105,16 @@ claude mcp add gitflic --env GITFLIC_TOKEN=xxxxxxxx -- npx -y gitflic-cli-mcp
 
 > ⚠️ Группа `auth` включает **мутирующие** тулзы (`gitflic_auth_login`, `_logout`, `_switch`, `_bind`, `_unbind`) — они меняют локальный keychain/конфиг, а не сервер, но это всё равно запись. Если нужны только проверки личности — добавь `GITFLIC_MCP_READONLY=1` (останутся `gitflic_auth`, `_list`, `_bindings`) или используй `GITFLIC_MCP_TOOLS` с точным набором.
 
+**`GITFLIC_MCP_NO_DESTRUCTIVE=1` — выключить удаление одним флагом.**
+
+Жёстко (на уровне регистрации, не через approval-prompt хоста) убирает **только удаляющие** тулзы — модель их вообще не видит. Это надёжнее, чем полагаться на подтверждение в хосте, которое можно прокликать или «разрешить всё агенту».
+
+Деструктивный набор (7 — всё, что заканчивается на `_delete`):
+
+`gitflic_issue_delete` · `gitflic_issue_comments_delete` · `gitflic_mr_discuss_delete` · `gitflic_branch_delete` · `gitflic_branch_protection_delete` · `gitflic_release_delete` · `gitflic_webhook_delete`
+
+Остальные мутации (`*_create`, `*_edit`, `*_close`, `*_cancel`, `gitflic_mr_merge` и т.п.) **остаются** — у них сохраняется `destructiveHint: true`, так что хост по-прежнему спрашивает подтверждение. `gitflic_mr_merge` необратим, но это не удаление — он намеренно вне флага и идёт через approval.
+
 **Примеры:**
 
 ```bash
@@ -113,6 +125,9 @@ claude mcp add gitflic --env GITFLIC_TOKEN=xxxx \
 # Read-only агент для код-ревью (только тулзы без мутаций)
 GITFLIC_MCP_READONLY=1 npx -y gitflic-cli-mcp
 
+# Может всё, кроме удаления (70 тулзов: создаёт/правит/мёрджит — но не удаляет)
+GITFLIC_MCP_NO_DESTRUCTIVE=1 npx -y gitflic-cli-mcp
+
 # Точный набор из четырёх тулзов
 GITFLIC_MCP_TOOLS="gitflic_mr_list,gitflic_mr_view,gitflic_mr_diff,gitflic_mr_create" \
   npx -y gitflic-cli-mcp

package/filter.mjs

@@ -30,6 +30,20 @@ export function isReadOnly(tool) {
   return Boolean(tool.annotations && tool.annotations.readOnlyHint === true);
 }
 
+// "Really destructive" = irreversible deletion. This is a deliberately NARROW
+// set (deletions only) used by GITFLIC_MCP_NO_DESTRUCTIVE, which removes these
+// tools at registration — a hard server-side guarantee, stronger than the
+// host's approval prompt (which a user can blanket-allow).
+//
+// Note: this is independent of the MCP `destructiveHint` annotation. Other
+// mutating ops (create/edit/close/cancel and gitflic_mr_merge) keep
+// destructiveHint:true and still go through the host's approval flow — they are
+// NOT removed by this flag. mr_merge is irreversible but is not a deletion, so
+// it is intentionally excluded here.
+export function isDestructive(tool) {
+  return /_delete$/.test(tool.name);
+}
+
 // Parse a comma/space-separated env value into a clean array.
 function parseList(value, { lower = false } = {}) {
   if (!value) return [];
@@ -43,11 +57,18 @@ function parseList(value, { lower = false } = {}) {
 const TRUTHY = ["1", "true", "yes", "on"];
 const FALSEY = ["0", "false", "no", "off", ""];
 
-// Truthy env flag: "1", "true", "yes", "on" (case-insensitive). Anything
-// else (including "0", "false", "", undefined) is falsey.
-function isTruthy(value) {
-  if (value == null) return false;
-  return TRUTHY.includes(String(value).trim().toLowerCase());
+// Parse a boolean env flag: "1"/"true"/"yes"/"on" → true (case-insensitive);
+// "0"/"false"/"no"/"off"/"" → false. Anything else is treated as false but
+// pushes a warning so a typo (e.g. "enabled") isn't silently ignored.
+function boolFlag(raw, name, warnings) {
+  if (raw == null) return false;
+  const v = String(raw).trim().toLowerCase();
+  if (v !== "" && !TRUTHY.includes(v) && !FALSEY.includes(v)) {
+    warnings.push(
+      `${name}="${raw}" is not a recognized boolean (use 1/true/yes/on); treating as off.`,
+    );
+  }
+  return TRUTHY.includes(v);
 }
 
 // Select which tools to register based on env config.
@@ -58,7 +79,8 @@ function isTruthy(value) {
 //        - GITFLIC_MCP_GROUPS (by group), if set; else
 //        - all tools.
 //   2. GITFLIC_MCP_READONLY truthy → keep only read-only tools (overlay).
-//   3. GITFLIC_MCP_EXCLUDE → drop named tools (applied last, always wins).
+//   3. GITFLIC_MCP_NO_DESTRUCTIVE truthy → drop deletion tools (overlay).
+//   4. GITFLIC_MCP_EXCLUDE → drop named tools (applied last, always wins).
 //
 // Returns { selected, warnings } — warnings are human-readable strings the
 // caller prints to stderr. Unknown names/groups are warned about, not fatal.
@@ -74,16 +96,12 @@ export function selectTools(tools, env = {}) {
   const wantGroups = parseList(env.GITFLIC_MCP_GROUPS, { lower: true });
   const exclude = parseList(env.GITFLIC_MCP_EXCLUDE, { lower: true });
 
-  const readonlyRaw = env.GITFLIC_MCP_READONLY;
-  const readonly = isTruthy(readonlyRaw);
-  if (readonlyRaw != null) {
-    const v = String(readonlyRaw).trim().toLowerCase();
-    if (v !== "" && !TRUTHY.includes(v) && !FALSEY.includes(v)) {
-      warnings.push(
-        `GITFLIC_MCP_READONLY="${readonlyRaw}" is not a recognized boolean (use 1/true/yes/on); treating as off.`,
-      );
-    }
-  }
+  const readonly = boolFlag(env.GITFLIC_MCP_READONLY, "GITFLIC_MCP_READONLY", warnings);
+  const noDestructive = boolFlag(
+    env.GITFLIC_MCP_NO_DESTRUCTIVE,
+    "GITFLIC_MCP_NO_DESTRUCTIVE",
+    warnings,
+  );
 
   let base;
   if (wantTools.length) {
@@ -121,6 +139,10 @@ export function selectTools(tools, env = {}) {
     }
   }
 
+  if (noDestructive) {
+    base = base.filter((t) => !isDestructive(t));
+  }
+
   if (exclude.length) {
     for (const n of exclude) {
       if (!byName.has(n)) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "gitflic-cli-mcp",
-  "version": "0.2.0",
+  "version": "0.3.0",
   "description": "MCP server wrapping the gitflic CLI — exposes the GitFlic REST API as typed tools for AI agents (Claude Code, Cursor, etc.)",
   "type": "module",
   "bin": {