@ryuenn3123/agentic-senior-core 6.10.0 → 6.11.1

package/.agents/plugins/agentic-senior-core/skills/asc-audit/SKILL.md

@@ -10,6 +10,12 @@ Security and architecture audit. Deeper than review, focused on finding vulnerab
 
 Grounded in: OWASP Top 10 (2025), OWASP ASVS v5.0, OWASP Top 10 for Agentic Applications (ASI01-ASI10, v2.01), CVSS vulnerability report structure, CWE classification.
 
+## Audit Procedure (Empirical First)
+
+1. **Dependency Vulnerability Scan**: Run the project's native security auditor (`npm audit`, `cargo audit`, or `pip-audit`). Report known CVEs immediately.
+2. **Lexical Sink & Secret Scan**: Search for hardcoded credentials, unvalidated redirects (`location.href`), dangerous functions (`eval`, `pickle.load`, `shell=True`), and unverified stubs (`// ASC-STUB:`).
+3. **Trust Boundary Mapping**: Trace all points where untrusted external input (HTTP body/query, CLI args, environment, external files) enters the system.
+
 ## Audit Scope
 
 1. **Trust boundaries**: Every point where external input enters the system. Validate that inputs are sanitized, normalized, and rejected when invalid.

package/.agents/plugins/agentic-senior-core/skills/asc-bootstrap/SKILL.md

@@ -19,6 +19,9 @@ Grounded in: Cold-start preference elicitation literature (Deezer/Netflix) & Khr
    - `ui-colored-left-border` (Colored left-border strip alert/card)
    - `ui-glassmorphism` (Generic glassmorphism backdrop blur)
    - `ui-pill-badge-generic` (Generic rounded pill tag badge)
+   - `ui-gradient-text-cliche` (Overused gradient headline text)
+   - `ui-transition-all-layout-cost` (Overused transition-all layout repaint cost)
+   - `ui-stripped-focus-ring` (Stripped focus ring WCAG 2.4.7 violation)
 2. Prompt user to select which slop patterns to ban for their brand/repo.
 3. Call `bootstrapPreferences()` in `lib/core/bootstrap-wizard.mjs` to seed selected rules into project preferences.
 4. Auto-compile Track A rules into `.git/hooks/pre-commit` and `ascx validate`.

package/.agents/plugins/agentic-senior-core/skills/asc-debt/SKILL.md

@@ -12,13 +12,23 @@ Grounded in: Cunningham's technical debt metaphor (1992). Entry format (ladder s
 
 ## Storage
 
-Entries persist in `.agent-context/state/debt-ledger.json` via MCP `state_read`/`state_write`. The ledger is a JSON array of entry objects.
+Entries persist in `.agent-context/state/debt-ledger.json` (or `.asc/debt-ledger.json`) via standard file read/write operations. The ledger is a JSON array of entry objects.
+
+## Stub Convention
+
+When introducing a temporary stub during development, mark it explicitly:
+```typescript
+// ASC-STUB: [reason for temporary code] [ASC-DEBT-001]
+```
+(or `# ASC-STUB: [reason] [ASC-DEBT-001]` in Python/Ruby).
+
+Every stub marker in source code MUST reference an open debt ID. Unregistered stubs will be blocked by the Git pre-commit runner.
 
 ## Entry Format
 
 ```json
 {
-  "id": "sequential integer",
+  "id": "sequential integer (or ASC-DEBT-xxx)",
   "file": "path/to/file.ts",
   "ladderStep": "3",
   "violation": "Added axios — stdlib fetch covers this",
@@ -34,14 +44,14 @@ Entries persist in `.agent-context/state/debt-ledger.json` via MCP `state_read`/
 
 When an enforcement nudge fires and the violation is deferred:
 
-1. Read the current ledger via `state_read` (path: `debt-ledger.json`). If missing, start with `[]`.
+1. Read the current ledger from `.agent-context/state/debt-ledger.json`. If missing, start with `[]`.
 2. Append a new entry with the next sequential `id`, the file path, ladder step, and a one-line violation summary.
-3. Write the updated ledger via `state_write` (path: `debt-ledger.json`, mode: `overwrite`).
+3. Write the updated ledger back to disk.
 4. Confirm: "Logged to debt ledger: [violation summary]"
 
 ### List Open Debt
 
-1. Read the ledger via `state_read`.
+1. Read the ledger.
 2. Filter to entries where `status` is `"open"`.
 3. Display as a table: ID, file, ladder step, violation, age.
 4. If no open entries, say so explicitly.

package/.agents/plugins/agentic-senior-core/skills/asc-refactor/SKILL.md

@@ -24,17 +24,16 @@ Removing speculative code is the highest-value, lowest-risk refactor type.
 After reading the target code, classify the refactor:
 
 - **Low-level** (rename, extract method, type safety, dead code removal): proceed after explaining the change. No approval gate needed.
-- **High-level** (move module, change architecture, redesign abstractions, multi-file structural changes): output a plan with specific files and rationale, then **stop and wait for user approval** before implementing.
-
-**Known limitation:** The high-level refactor gate is a skill-text instruction — the agent self-classifies and self-stops. Unlike `/asc-add-feature` and `/asc-new-project`, this gate is not backed by the PostToolUse hook or workflow-gate.json. If the agent bypasses the gate, there is no automated nudge.
+- **High-level** (move module, change architecture, redesign abstractions, multi-file structural changes): write `workflow-gate.json` with `{"workflow": "asc-refactor", "phase": "plan"}`, output a plan with specific files and rationale, then **stop and wait for user approval** before implementing.
 
 ## Before Editing
 
-1. Read the target code and understand existing patterns.
-2. Identify the smallest relevant scope for the refactor.
-3. Classify the refactor (see above). If high-level, stop and present a plan.
-4. If the change touches UI, check accessibility and responsive behavior.
-5. If the change touches dependencies, verify current official docs.
+1. **Verify Baseline**: Run project test runner (`ascx test`, `npm test`, or equivalent). Do not refactor on failing tests.
+2. Read the target code and understand existing patterns.
+3. Identify the smallest relevant scope for the refactor.
+4. Classify the refactor (see above). For high-level refactors, update `workflow-gate.json` to phase `implement` only after user approval.
+5. If the change touches UI, check accessibility and responsive behavior.
+6. If the change touches dependencies, verify current official docs.
 
 ## Refactor Rules
 

package/.agents/plugins/agentic-senior-core/skills/asc-reference/SKILL.md

@@ -40,9 +40,10 @@ Grounded in: WCAG 2.2 AA (accessibility), Fowler's Money Pattern (monetary types
 ## Frontend
 
 - Match the project's existing styling paradigm (Tailwind, CSS Modules, Vanilla CSS, styled-components, etc.). Do not introduce a new CSS framework or build tool without explicit user confirmation.
-- Semantic HTML before custom components.
-- WCAG 2.2 AA is the accessibility floor.
-- Responsive by default. Handle empty, loading, error, and offline states.
+- UI Component Reusability: Primitive UI elements (buttons, inputs, cards, dialogs, badges) must have a single source of truth in the shared component directory (`components/` or `ui/`). Never recreate ad-hoc variants or duplicate JSX structures across individual pages. Extend existing components with props or variants instead of cloning.
+- Design Consistency: Consume shared design tokens, CSS variables, or theme configuration for colors, spacing, and border-radius. Do not hardcode arbitrary raw color hexes or inconsistent spacing that causes visual design drift between pages.
+- Data-fetching components must explicitly handle 3 states: loading, error (with user-facing toast/notification and retry option), and data. Never leave an unhandled error state.
+- WCAG 2.2 AA accessibility floor. Responsive by default.
 - No placeholder, lorem, or TODO content in production UI.
 
 ## Infrastructure
@@ -59,10 +60,3 @@ Grounded in: WCAG 2.2 AA (accessibility), Fowler's Money Pattern (monetary types
 - Circuit breakers for unhealthy dependencies.
 - Graceful degradation on non-critical dependency failure.
 
-## Naming
-
-- Variables and properties are nouns: `userList`, `pageCount`, not `getUser`.
-- Functions and methods are verbs: `calculateTotal`, `fetchUsers`, not `total`.
-- Booleans prefixed with `is`/`has`/`can`: `isValid`, `hasPermission`.
-- Constants in UPPER_SNAKE_CASE: `MAX_RETRIES`, `API_BASE_URL`.
-- Follow the casing convention of the language and existing codebase. When starting fresh, use the community default for that language.

package/.agents/plugins/agentic-senior-core/skills/asc-review/SKILL.md

@@ -10,12 +10,13 @@ Production-risk code review. Prioritize findings by severity.
 
 Grounded in: OWASP Risk Rating Methodology, Google Engineering Practices (code review guide), Clean Architecture (transport vs business layer separation).
 
-## Before Reviewing
+## Review Procedure (Empirical First)
 
-1. Read the changed files and understand the scope.
-2. For UI changes, check accessibility and design consistency.
-3. For API changes, check contract stability and documentation sync.
-4. Walk the decision ladder for each new file or dependency: does this need to exist, could the stdlib or an existing dependency handle it?
+1. **Establish Scope**: Run `git diff --stat` (or `git status`) to inspect the exact set of changed files and line deltas.
+2. **Test-Change Correlation**: For every modified application source file, check if a corresponding test file was updated. If application logic changed without test updates, flag as Severity 3 ("Missing tests for changed behavior").
+3. **Run Deterministic Checks**: Run the project linter and test suite (`ascx test` / `npm test`). Do not evaluate architectural quality on a failing test suite.
+4. **Security & Invariants Scan**: Check for unverified `ASC-STUB:`, hardcoded credentials, or empty catch blocks in the diff.
+5. **Contract & Hierarchy**: Walk the decision ladder for each new file or dependency: does this need to exist, could the stdlib or an existing dependency handle it?
 
 ## Finding Priority Order
 

package/bin/agentic-senior-core.js

@@ -8,7 +8,6 @@
  *   uninstall   Remove ASC adapter files from the current project
  *   clean       Remove v4 per-project artifacts (.agent-context/, bridge files)
  *   status      Show detected IDEs and install hints
- *   mcp         Start MCP stdio server
  */
 import { exit } from 'node:process';
 import { readFileSync } from 'node:fs';
@@ -37,7 +36,6 @@ function printUsage() {
   console.log('  uninstall         Remove ASC adapter files and git hooks from this project');
   console.log('  clean             Remove v4 per-project artifacts');
   console.log('  status            Show detected IDEs and install hints');
-  console.log('  mcp               Start MCP stdio server');
   console.log('  --version         Show version');
   console.log('  --help            Show this help');
 }
@@ -92,12 +90,6 @@ async function main() {
     return;
   }
 
-  if (commandArgument === 'mcp') {
-    const { runMcpServerCommand } = await import('../lib/cli/commands/mcp.mjs');
-    await runMcpServerCommand();
-    return;
-  }
-
   console.error(`Unknown command: ${commandArgument}`);
   printUsage();
   exit(1);

package/gemini-extension.json

@@ -1,6 +1,6 @@
 {
   "name": "agentic-senior-core",
-  "version": "6.10.0",
+  "version": "6.11.1",
   "description": "Universal AI coding rules. Write code like a staff engineer.",
   "author": "fatidaprilian",
   "license": "MIT",

package/lib/cli/commands/git-hook-generator.mjs

@@ -202,6 +202,33 @@ function checkLargeFiles(stagedFiles, cwd, maxSizeKB) {
   }
 }
 
+function checkStubsAndTodos(stagedFiles, cwd) {
+  var STUB_REGEX = /(?:\\/\\/|#|\\/\\*)\\s*ASC-STUB:(?!.*\\b(ASC-DEBT|#\\d+|[A-Z]+-\\d+)\\b)/i;
+  var MOCK_SUCCESS_REGEX = /(?:return\\s*\\{\\s*success:\\s*true\\s*\\}|return\\s*True|return\\s*true|return\\s*nil)\\s*;?\\s*(?:\\/\\/|#).*?(mock|fake|stub|placeholder)/i;
+
+  for (var i = 0; i < stagedFiles.length; i++) {
+    var ext = path.extname(stagedFiles[i]).slice(1).toLowerCase();
+    if (!SOURCE_EXTENSIONS.has(ext)) continue;
+    try {
+      var content = fs.readFileSync(path.resolve(cwd, stagedFiles[i]), 'utf8');
+      var lines = content.split('\\n');
+      for (var l = 0; l < lines.length; l++) {
+        var line = lines[l];
+        if (STUB_REGEX.test(line)) {
+          console.error('\\x1b[31m[ASC Stub Gate]\\x1b[0m Unverified stub detected in ' + stagedFiles[i] + ' line ' + (l + 1) + ': ' + line.trim());
+          console.error('\\x1b[33mCommit blocked. Stubs must carry a tracked debt ID: // ASC-STUB: [reason] [ASC-DEBT-xxx]. Use git commit --no-verify to bypass.\\x1b[0m');
+          process.exit(1);
+        }
+        if (MOCK_SUCCESS_REGEX.test(line)) {
+          console.error('\\x1b[31m[ASC Fake-Done Gate]\\x1b[0m Simulated success return detected in ' + stagedFiles[i] + ' line ' + (l + 1) + ': ' + line.trim());
+          console.error('\\x1b[33mCommit blocked. Never commit hardcoded mock success returns mimicking live integration. Use git commit --no-verify to bypass.\\x1b[0m');
+          process.exit(1);
+        }
+      }
+    } catch (_) {}
+  }
+}
+
 function runJscpdScan(scanCmd, cwd, tmpDir) {
   const binaries = ['bunx jscpd', 'npx jscpd@5', 'npx jscpd'];
   for (let i = 0; i < binaries.length; i++) {
@@ -318,6 +345,7 @@ function runPreCommitGate() {
   checkSecrets(stagedFiles, cwd);
   checkMergeConflictMarkers(stagedFiles, cwd);
   checkLargeFiles(stagedFiles, cwd, config.maxFileSizeKB);
+  checkStubsAndTodos(stagedFiles, cwd);
 
   // 2. Filter staged files by SOURCE_EXTENSIONS for dedup scan
   const stagedSourceFiles = stagedFiles.filter(f => {
@@ -442,6 +470,13 @@ export function installGlobalGitPreCommitHook({ homeDir = os.homedir() } = {}) {
     fs.mkdirSync(globalHooksDir, { recursive: true });
   }
 
+  // Generate standalone universal runner in global directory for fallback
+  const globalRunnerPath = path.join(globalHooksDir, 'pre-commit-runner.cjs');
+  const runnerScriptContent = generatePreCommitRunnerScript();
+  fs.writeFileSync(globalRunnerPath, runnerScriptContent, { encoding: 'utf8', mode: 0o755 });
+
+  const globalRunnerPosix = globalRunnerPath.replace(/\\/g, '/');
+
   const hookPath = path.join(globalHooksDir, 'pre-commit');
   const dispatcherContent = `#!/bin/sh
 ${ASC_HOOK_HEADER} -- Global Smart Dispatcher
@@ -476,13 +511,16 @@ if [ -n "$LOCAL_HOOK" ]; then
   fi
 fi
 
-# 3. Run ASC workspace pre-commit runner if present
+# 3. Run ASC workspace pre-commit runner if present, or fallback to global runner
 if [ -n "$GIT_ROOT" ] && [ -f "$GIT_ROOT/.asc/hooks/pre-commit-runner.cjs" ]; then
   node "$GIT_ROOT/.asc/hooks/pre-commit-runner.cjs" "$@"
   exit $?
 elif [ -f ".asc/hooks/pre-commit-runner.cjs" ]; then
   node .asc/hooks/pre-commit-runner.cjs "$@"
   exit $?
+elif [ -f "${globalRunnerPosix}" ]; then
+  node "${globalRunnerPosix}" "$@"
+  exit $?
 fi
 
 exit 0
@@ -497,6 +535,6 @@ exit 0
     return { installed: false, hookPath, reason: 'Could not set git config --global core.hooksPath' };
   }
 
-  return { installed: true, hookPath, global: true };
+  return { installed: true, hookPath, runnerPath: globalRunnerPath, global: true };
 }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ryuenn3123/agentic-senior-core",
-  "version": "6.10.0",
+  "version": "6.11.1",
   "type": "module",
   "description": "Agentic Senior Core: Universal AI coding rules and workflows. Write code like a staff engineer, not a junior.",
   "bin": {
@@ -16,8 +16,6 @@
     "gemini-extension.json",
     "plugin.yaml",
     "__init__.py",
-    "scripts/mcp-server.mjs",
-    "scripts/mcp-server/",
     "scripts/uninstall.js",
     "README.md",
     "LICENSE",

package/plugin.yaml

@@ -1,5 +1,5 @@
 name: agentic-senior-core
-version: 6.10.0
+version: 6.11.1
 description: Universal AI coding rules. Write code like a staff engineer.
 author: fatidaprilian
 provides_hooks:

package/lib/cli/commands/mcp.mjs

@@ -1,3 +0,0 @@
-export async function runMcpServerCommand() {
-  await import('../../../scripts/mcp-server.mjs');
-}

package/scripts/mcp-server/constants.mjs

@@ -1,57 +0,0 @@
-// @ts-check
-
-import { existsSync, readFileSync } from 'node:fs';
-import { dirname, resolve } from 'node:path';
-import { fileURLToPath } from 'node:url';
-
-const SCRIPT_FILE_PATH = fileURLToPath(import.meta.url);
-export const REPOSITORY_ROOT = resolve(dirname(SCRIPT_FILE_PATH), '..', '..');
-export const STATE_DIRECTORY = resolve(REPOSITORY_ROOT, '.agent-context', 'state');
-export const DEFAULT_PROTOCOL_VERSION = '2024-11-05';
-export const DEFAULT_FETCH_TIMEOUT_MS = 15000;
-export const DEFAULT_FETCH_MAX_CHARS = 6000;
-export const MAX_FETCH_MAX_CHARS = 20000;
-export const DEFAULT_TREND_WINDOW_DAYS = 90;
-export const MAX_TREND_PACKAGES = 10;
-export const FALLBACK_PACKAGE_VERSION = '0.0.0-local';
-
-// IMPORTANT: This version extraction logic is intentionally duplicated from lib/cli/constants.mjs.
-// The MCP server is designed to be copied directly into target user workspaces where
-// the original package.json may not exist in the parent tree. This try/catch fallback
-// ensures the server can still run standalone without crashing if package.json is missing.
-function resolvePackageVersion() {
-  try {
-    const parsedPackageManifest = JSON.parse(
-      readFileSync(resolve(REPOSITORY_ROOT, 'package.json'), 'utf8')
-    );
-    const rawVersion = typeof parsedPackageManifest?.version === 'string'
-      ? parsedPackageManifest.version.trim()
-      : '';
-
-    return rawVersion || FALLBACK_PACKAGE_VERSION;
-  } catch {
-    return FALLBACK_PACKAGE_VERSION;
-  }
-}
-
-export const PACKAGE_VERSION = resolvePackageVersion();
-
-export const TEST_SUITE_ARGS = {
-  adapter: ['--test', './tests/adapter.test.mjs'],
-};
-
-export const INTERNAL_SCRIPT_PATHS = {};
-
-function getAvailableTestSuites() {
-  return Object.entries(TEST_SUITE_ARGS)
-    .filter(([, commandArguments]) => (
-      Array.isArray(commandArguments)
-      && commandArguments.length > 1
-      && commandArguments
-        .slice(1)
-        .every((relativeTestPath) => existsSync(resolve(REPOSITORY_ROOT, relativeTestPath)))
-    ))
-    .map(([suiteName]) => suiteName);
-}
-
-export const AVAILABLE_TEST_SUITES = getAvailableTestSuites();

package/scripts/mcp-server/tool-registry.mjs

@@ -1,204 +0,0 @@
-// @ts-check
-
-import { existsSync } from 'node:fs';
-import { AVAILABLE_TEST_SUITES, INTERNAL_SCRIPT_PATHS } from './constants.mjs';
-
-export function buildToolDefinitions() {
-  const toolDefinitions = [];
-
-  if (existsSync(INTERNAL_SCRIPT_PATHS.validate)) {
-    toolDefinitions.push({
-      name: 'validate',
-      description: 'Run repository validation checks.',
-      inputSchema: {
-        type: 'object',
-        properties: {},
-        additionalProperties: false,
-      },
-    });
-  }
-
-  if (AVAILABLE_TEST_SUITES.length > 0) {
-    toolDefinitions.push({
-      name: 'test',
-      description: 'Run test suites (full or targeted).',
-      inputSchema: {
-        type: 'object',
-        properties: {
-          suite: {
-            type: 'string',
-            enum: AVAILABLE_TEST_SUITES,
-            description: 'Target test suite. Defaults to the first available suite.',
-          },
-        },
-        additionalProperties: false,
-      },
-    });
-  }
-
-  if (existsSync(INTERNAL_SCRIPT_PATHS.release_gate)) {
-    toolDefinitions.push({
-      name: 'release_gate',
-      description: 'Run release gate checks.',
-      inputSchema: {
-        type: 'object',
-        properties: {},
-        additionalProperties: false,
-      },
-    });
-  }
-
-  if (existsSync(INTERNAL_SCRIPT_PATHS.forbidden_content_check)) {
-    toolDefinitions.push({
-      name: 'forbidden_content_check',
-      description: 'Run forbidden content scan used by publish gate.',
-      inputSchema: {
-        type: 'object',
-        properties: {},
-        additionalProperties: false,
-      },
-    });
-  }
-
-  toolDefinitions.push(
-    {
-      name: 'lookup_rule',
-      description: 'Look up a canonical .agent-context rule section by stable rule ID.',
-      inputSchema: {
-        type: 'object',
-        properties: {
-          ruleId: {
-            type: 'string',
-            description: 'Stable rule section ID, such as ARCH-003 or API-001.',
-          },
-        },
-        required: ['ruleId'],
-        additionalProperties: false,
-      },
-    },
-    {
-      name: 'validate_against_rules',
-      description: 'Validate that cited rule IDs resolve to canonical rule sections.',
-      inputSchema: {
-        type: 'object',
-        properties: {
-          ruleIds: {
-            type: 'array',
-            items: { type: 'string' },
-            description: 'Rule IDs cited by a response, plan, or review.',
-          },
-          summary: {
-            type: 'string',
-            description: 'Optional one-line context for the validation request.',
-          },
-        },
-        required: ['ruleIds'],
-        additionalProperties: false,
-      },
-    },
-    {
-      name: 'audit_compliance',
-      description: 'Run a lightweight compliance audit over cited rule IDs and scope labels.',
-      inputSchema: {
-        type: 'object',
-        properties: {
-          ruleIds: {
-            type: 'array',
-            items: { type: 'string' },
-            description: 'Rule IDs used as the compliance basis.',
-          },
-          scope: {
-            type: 'string',
-            description: 'Optional changed scope label, such as api, security, testing, architecture, ui, or release.',
-          },
-        },
-        required: ['ruleIds'],
-        additionalProperties: false,
-      },
-    },
-    {
-      name: 'research_fetch',
-      description: 'Fetch external documentation/news content and return query-focused excerpts with citation metadata.',
-      inputSchema: {
-        type: 'object',
-        properties: {
-          url: {
-            type: 'string',
-            description: 'Absolute HTTP/HTTPS URL to fetch.',
-          },
-          query: {
-            type: 'string',
-            description: 'Optional search query used to extract focused excerpts.',
-          },
-          maxChars: {
-            type: 'integer',
-            description: 'Maximum characters to return when query is not provided (default 6000, max 20000).',
-          },
-        },
-        required: ['url'],
-        additionalProperties: false,
-      },
-    },
-    {
-      name: 'trend_snapshot',
-      description: 'Generate ecosystem trend snapshot from npm registry metadata with source timestamps.',
-      inputSchema: {
-        type: 'object',
-        properties: {
-          packages: {
-            type: 'array',
-            items: { type: 'string' },
-            description: 'Package names to inspect (max 10).',
-          },
-          windowDays: {
-            type: 'integer',
-            description: 'Release activity window in days (default 90).',
-          },
-        },
-        required: ['packages'],
-        additionalProperties: false,
-      },
-    },
-    {
-      name: 'state_read',
-      description: 'Read a file from .agent-context/state for cross-session continuity.',
-      inputSchema: {
-        type: 'object',
-        properties: {
-          path: {
-            type: 'string',
-            description: 'Path relative to .agent-context/state, such as memory-continuity-benchmark.json.',
-          },
-        },
-        required: ['path'],
-        additionalProperties: false,
-      },
-    },
-    {
-      name: 'state_write',
-      description: 'Write a file under .agent-context/state for cross-session continuity updates.',
-      inputSchema: {
-        type: 'object',
-        properties: {
-          path: {
-            type: 'string',
-            description: 'Path relative to .agent-context/state.',
-          },
-          content: {
-            type: 'string',
-            description: 'UTF-8 content to write.',
-          },
-          mode: {
-            type: 'string',
-            enum: ['overwrite', 'append'],
-            description: 'Write mode. Defaults to overwrite.',
-          },
-        },
-        required: ['path', 'content'],
-        additionalProperties: false,
-      },
-    },
-  );
-
-  return toolDefinitions;
-}