gitflic-cli-mcp 0.2.0 → 0.3.1

package/README.md

@@ -1,6 +1,6 @@
 # gitflic-cli MCP-сервер
 
-[Model Context Protocol](https://modelcontextprotocol.io/) сервер, оборачивающий CLI `gitflic` и отдающий **77 типизированных тулзов** поверх команд GitFlic REST для AI-агентов (Claude Code, Cursor и т.п.).
+[Model Context Protocol](https://modelcontextprotocol.io/) сервер, оборачивающий CLI `gitflic` и отдающий **76 типизированных тулзов** поверх команд GitFlic REST для AI-агентов (Claude Code, Cursor и т.п.).
 
 ## Что внутри
 
@@ -66,22 +66,24 @@ claude mcp add gitflic --env GITFLIC_TOKEN=xxxxxxxx -- npx -y gitflic-cli-mcp
 
 ## Ограничение набора тулзов (экономия контекста)
 
-Все 77 тулзов уходят хосту по `tools/list` ещё до первого вызова — это ~6K токенов «налога» на контекст модели. Если агенту нужны не все, набор можно сузить через **env-переменные** — прописываются в том же `env`-блоке конфига, где `GITFLIC_TOKEN` (своего конфиг-файла у сервера нет, он читает `process.env` при старте).
+Все 76 тулзов уходят хосту по `tools/list` ещё до первого вызова — это ~6K токенов «налога» на контекст модели. Если агенту нужны не все, набор можно сузить через **env-переменные** — прописываются в том же `env`-блоке конфига, где `GITFLIC_TOKEN` (своего конфиг-файла у сервера нет, он читает `process.env` при старте).
 
 | Переменная | Значение | Что делает |
 |---|---|---|
 | `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` | список имён тулзов | убрать конкретные тулзы (применяется последним) |
 
-Все — опциональные. Без них регистрируются все 77 (как и раньше). Неизвестные группы/имена не валят сервер — он пишет `WARN` в stderr и игнорирует их.
+Все — опциональные. Без них регистрируются все 76 (как и раньше). Неизвестные группы/имена не валят сервер — он пишет `WARN` в stderr и игнорирует их.
 
 **Порядок применения (precedence):**
 
 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` — вычитается в самом конце (всегда побеждает).
 
 Значения списков — через запятую или пробел; регистр не важен ни для групп, ни для имён тулзов (имена всегда в нижнем регистре).
 
@@ -90,7 +92,7 @@ claude mcp add gitflic --env GITFLIC_TOKEN=xxxxxxxx -- npx -y gitflic-cli-mcp
 | Группа | Тулзов | Read / Write |
 |---|---|---|
 | `auth` | 8 | 3 read / 5 write |
-| `blob` | 3 | 3 read / 0 write |
+| `blob` | 2 | 2 read / 0 write |
 | `branch` | 10 | 5 read / 5 write |
 | `commit` | 4 | 4 read / 0 write |
 | `issue` | 12 | 4 read / 8 write |
@@ -103,16 +105,29 @@ 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
-# Только MR и issue (~27 тулзов вместо 77)
+# Только MR и issue (~27 тулзов вместо 76)
 claude mcp add gitflic --env GITFLIC_TOKEN=xxxx \
   --env GITFLIC_MCP_GROUPS=mr,issue -- npx -y gitflic-cli-mcp
 
 # Read-only агент для код-ревью (только тулзы без мутаций)
 GITFLIC_MCP_READONLY=1 npx -y gitflic-cli-mcp
 
+# Может всё, кроме удаления (69 тулзов: создаёт/правит/мёрджит — но не удаляет)
+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
@@ -138,15 +153,15 @@ GITFLIC_MCP_TOOLS="gitflic_mr_list,gitflic_mr_view,gitflic_mr_diff,gitflic_mr_cr
 
 > Можно поднять один и тот же сервер дважды под разными именами — напр. `gitflic` (полный) и `gitflic-ro` (`GITFLIC_MCP_READONLY=1`) — и держать оба подключёнными. Либо класть конфиг в проектный `.mcp.json` / `.cursor/mcp.json`, чтобы в каждом репозитории грузились свои группы.
 
-## Все 77 тулзов
+## Все 76 тулзов
 
 Полный список по группам. **R** = read-only (`readOnlyHint: true`, не меняет состояние — попадает в `GITFLIC_MCP_READONLY=1`); **W** = write/мутация.
 
 ### `auth` (8)
 `R` `gitflic_auth` · `W` `gitflic_auth_login` · `W` `gitflic_auth_logout` · `R` `gitflic_auth_list` · `W` `gitflic_auth_switch` · `W` `gitflic_auth_bind` · `W` `gitflic_auth_unbind` · `R` `gitflic_auth_bindings`
 
-### `blob` (3)
-`R` `gitflic_blob_get` · `R` `gitflic_blob_files` · `R` `gitflic_blob_archive`
+### `blob` (2)
+`R` `gitflic_blob_get` · `R` `gitflic_blob_files`
 
 ### `branch` (10)
 `R` `gitflic_branch_list` · `W` `gitflic_branch_create` · `W` `gitflic_branch_delete` · `R` `gitflic_branch_get` · `R` `gitflic_branch_default` · `R` `gitflic_branch_protection_list` · `R` `gitflic_branch_protection_get` · `W` `gitflic_branch_protection_create` · `W` `gitflic_branch_protection_edit` · `W` `gitflic_branch_protection_delete`
@@ -175,7 +190,7 @@ GITFLIC_MCP_TOOLS="gitflic_mr_list,gitflic_mr_view,gitflic_mr_diff,gitflic_mr_cr
 ### `webhook` (4)
 `R` `gitflic_webhook_list` · `R` `gitflic_webhook_get` · `W` `gitflic_webhook_create` · `W` `gitflic_webhook_delete`
 
-Итого: **40 read / 37 write**. Каждый тул маппится 1-в-1 на подкоманду CLI (`gitflic …`).
+Итого: **39 read / 37 write**. Каждый тул маппится 1-в-1 на подкоманду CLI (`gitflic …`).
 
 ## Как это работает
 

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.1",
   "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": {

package/tools.mjs

@@ -242,10 +242,10 @@ export const TOOLS = [
   {
     name: "gitflic_mr_create",
     description:
-      "Create a new merge request. If --from is omitted, uses current git branch. If --to is omitted, uses project default branch.",
+      "Create a new merge request. `description` is required (GitFlic rejects empty MR descriptions). If `from` is omitted, uses current git branch; if `to` is omitted, uses project default branch.",
     inputSchema: {
       type: "object",
-      required: ["project", "title"],
+      required: ["project", "title", "description"],
       properties: {
         project: { type: "string" },
         title: { type: "string" },
@@ -1430,7 +1430,7 @@ export const TOOLS = [
   },
   {
     name: "gitflic_project_edit",
-    description: "Edit project metadata (title, description, default branch, visibility).",
+    description: "Edit project title, description, or language (via change-setting). Visibility and default branch are not editable over the REST API.",
     inputSchema: {
       type: "object",
       required: ["project"],
@@ -1438,18 +1438,14 @@ export const TOOLS = [
         project: { type: "string" },
         title: { type: "string" },
         description: { type: "string" },
-        isPrivate: { type: "boolean" },
         language: { type: "string" },
-        defaultBranch: { type: "string" },
       },
     },
     handler: (a) => {
       const args = ["project", "edit", a.project, "--format", "json"];
       if (a.title) args.push("--title", a.title);
       if (a.description !== undefined) args.push("--description", a.description);
-      if (a.isPrivate !== undefined) args.push(a.isPrivate ? "--private" : "--public");
       if (a.language) args.push("--language", a.language);
-      if (a.defaultBranch) args.push("--default-branch", a.defaultBranch);
       return args;
     },
     annotations: {
@@ -1540,8 +1536,8 @@ export const TOOLS = [
       },
     },
     handler: (a) => {
-      const args = ["blob", "files", "--project", a.project, "--format", "json"];
-      if (a.path) args.push("--path", a.path);
+      const args = ["blob", "recursive", "--project", a.project, "--format", "json"];
+      if (a.path) args.push("--directory", a.path);
       if (a.branch) args.push("--branch", a.branch);
       return args;
     },
@@ -1553,23 +1549,6 @@ export const TOOLS = [
       openWorldHint: true,
     },
   },
-  {
-    name: "gitflic_blob_archive",
-    description: "Download a repo archive (tarball). Returns JSON metadata; use shell to stream the body.",
-    inputSchema: {
-      type: "object",
-      required: ["project", "branch"],
-      properties: { project: { type: "string" }, branch: { type: "string" } },
-    },
-    handler: (a) => ["blob", "archive", a.branch, "--project", a.project, "--format", "json"],
-    annotations: {
-      title: "Blob Archive",
-      readOnlyHint: true,
-      destructiveHint: false,
-      idempotentHint: false,
-      openWorldHint: true,
-    },
-  },
 
   // ===== User: get / projects / followers =================================
   {