@foundation0/git 1.3.0 → 1.3.2

package/mcp/src/cli.ts

@@ -1,76 +1,76 @@
-import { runGitMcpServer } from './server'
-
-const getArgValue = (name: string, fallback?: string): string | undefined => {
-  const exactArg = process.argv.find((arg) => arg.startsWith(`${name}=`))
-  if (exactArg) {
-    const [, value] = exactArg.split('=', 2)
-    return value
-  }
-
-  const index = process.argv.indexOf(name)
-  if (index >= 0 && index + 1 < process.argv.length) {
-    const next = process.argv[index + 1]
-    if (!next.startsWith('--')) {
-      return next
-    }
-  }
-
-  return fallback
-}
-
-const hasFlag = (name: string): boolean => process.argv.includes(name)
-
-const isInsecureHttpUrl = (value: string | undefined): boolean => {
-  if (!value) {
-    return false
-  }
-
-  return value.trim().toLowerCase().startsWith('http://')
-}
-
-if (hasFlag('--help') || hasFlag('-h')) {
-  console.log('Usage: f0-git-mcp [options]')
-  console.log('Optional flags:')
-  console.log('  --default-owner <owner>')
-  console.log('  --default-repo <repo>')
-  console.log('  --gitea-host <url>')
-  console.log('  --server-name <name>')
-  console.log('  --server-version <version>')
-  console.log('  --tools-prefix <prefix>')
-  console.log('  --allow-insecure-http')
-  process.exit(0)
-}
-
-const owner = getArgValue('--default-owner')?.trim()
-const repo = getArgValue('--default-repo')?.trim()
-const host = getArgValue('--gitea-host', process.env.GITEA_HOST)?.trim()
-const token = process.env.GITEA_TOKEN
-const serverName = getArgValue('--server-name', 'f0-git-mcp')
-const serverVersion = getArgValue('--server-version', '1.0.0')
-const toolsPrefix = getArgValue('--tools-prefix') ?? process.env.MCP_TOOLS_PREFIX
-
-if (!host) {
-  throw new Error('GITEA_HOST is required. Set process.env.GITEA_HOST or pass --gitea-host.')
-}
-
-if (isInsecureHttpUrl(host) && !hasFlag('--allow-insecure-http')) {
-  throw new Error(
-    'Refusing to send requests to an insecure http:// Gitea host. Use https:// or pass --allow-insecure-http if you really need this for local testing.',
-  )
-}
-
-void runGitMcpServer({
-  serverName: serverName ?? undefined,
-  serverVersion: serverVersion ?? undefined,
-  config: {
-    platform: 'GITEA',
-    giteaHost: host,
-    giteaToken: token,
-  },
-  ...(owner ? { defaultOwner: owner } : {}),
-  ...(repo ? { defaultRepo: repo } : {}),
-  toolsPrefix,
-}).catch((error) => {
-  console.error('Failed to start MCP git server', error)
-  process.exit(1)
-})
+import { runGitMcpServer } from './server'
+
+const getArgValue = (name: string, fallback?: string): string | undefined => {
+  const exactArg = process.argv.find((arg) => arg.startsWith(`${name}=`))
+  if (exactArg) {
+    const [, value] = exactArg.split('=', 2)
+    return value
+  }
+
+  const index = process.argv.indexOf(name)
+  if (index >= 0 && index + 1 < process.argv.length) {
+    const next = process.argv[index + 1]
+    if (!next.startsWith('--')) {
+      return next
+    }
+  }
+
+  return fallback
+}
+
+const hasFlag = (name: string): boolean => process.argv.includes(name)
+
+const isInsecureHttpUrl = (value: string | undefined): boolean => {
+  if (!value) {
+    return false
+  }
+
+  return value.trim().toLowerCase().startsWith('http://')
+}
+
+if (hasFlag('--help') || hasFlag('-h')) {
+  console.log('Usage: f0-git-mcp [options]')
+  console.log('Optional flags:')
+  console.log('  --default-owner <owner>')
+  console.log('  --default-repo <repo>')
+  console.log('  --gitea-host <url>')
+  console.log('  --server-name <name>')
+  console.log('  --server-version <version>')
+  console.log('  --tools-prefix <prefix>')
+  console.log('  --allow-insecure-http')
+  process.exit(0)
+}
+
+const owner = getArgValue('--default-owner')?.trim()
+const repo = getArgValue('--default-repo')?.trim()
+const host = getArgValue('--gitea-host', process.env.GITEA_HOST)?.trim()
+const token = process.env.GITEA_TOKEN
+const serverName = getArgValue('--server-name', 'f0-git-mcp')
+const serverVersion = getArgValue('--server-version', '1.0.0')
+const toolsPrefix = getArgValue('--tools-prefix') ?? process.env.MCP_TOOLS_PREFIX
+
+if (!host) {
+  throw new Error('GITEA_HOST is required. Set process.env.GITEA_HOST or pass --gitea-host.')
+}
+
+if (isInsecureHttpUrl(host) && !hasFlag('--allow-insecure-http')) {
+  throw new Error(
+    'Refusing to send requests to an insecure http:// Gitea host. Use https:// or pass --allow-insecure-http if you really need this for local testing.',
+  )
+}
+
+void runGitMcpServer({
+  serverName: serverName ?? undefined,
+  serverVersion: serverVersion ?? undefined,
+  config: {
+    platform: 'GITEA',
+    giteaHost: host,
+    giteaToken: token,
+  },
+  ...(owner ? { defaultOwner: owner } : {}),
+  ...(repo ? { defaultRepo: repo } : {}),
+  toolsPrefix,
+}).catch((error) => {
+  console.error('Failed to start MCP git server', error)
+  process.exit(1)
+})

package/mcp/src/client.ts

@@ -1,147 +1,156 @@
-import { Client } from '@modelcontextprotocol/sdk/client/index.js'
-import { StdioClientTransport } from '@modelcontextprotocol/sdk/client/stdio.js'
-import { CallToolResultSchema, ListToolsResultSchema } from '@modelcontextprotocol/sdk/types.js'
-
-type ToolResultText = {
-  type: 'text'
-  text: string
-}
-
-type ToolResult = {
-  isError?: boolean
-  content?: ToolResultText[]
-}
-
-const isRecord = (value: unknown): value is Record<string, unknown> =>
-  typeof value === 'object' && value !== null && !Array.isArray(value)
-
-const toText = (result: ToolResult | null): string => {
-  if (!result || !result.content || result.content.length === 0) {
-    return ''
-  }
-
-  return result.content
-    .map((entry) => (entry.type === 'text' ? entry.text : ''))
-    .filter((entry) => entry.length > 0)
-    .join('\n')
-}
-
-const tryParseResult = (text: string): { parsed?: unknown; text: string } => {
-  if (!text) {
-    return { text }
-  }
-
-  try {
-    return {
-      text,
-      parsed: JSON.parse(text),
-    }
-  } catch {
-    return { text }
-  }
-}
-
-export interface GitMcpClientOptions {
-  name?: string
-  version?: string
-}
-
+import { Client } from '@modelcontextprotocol/sdk/client/index.js'
+import { StdioClientTransport } from '@modelcontextprotocol/sdk/client/stdio.js'
+import { CallToolResultSchema, ListToolsResultSchema } from '@modelcontextprotocol/sdk/types.js'
+
+type ToolResultText = {
+  type: 'text'
+  text: string
+}
+
+type ToolResult = {
+  isError?: boolean
+  content?: ToolResultText[]
+}
+
+const isRecord = (value: unknown): value is Record<string, unknown> =>
+  typeof value === 'object' && value !== null && !Array.isArray(value)
+
+const toText = (result: ToolResult | null): string => {
+  if (!result || !result.content || result.content.length === 0) {
+    return ''
+  }
+
+  return result.content
+    .map((entry) => (entry.type === 'text' ? entry.text : ''))
+    .filter((entry) => entry.length > 0)
+    .join('\n')
+}
+
+const tryParseResult = (text: string): { parsed?: unknown; text: string } => {
+  if (!text) {
+    return { text }
+  }
+
+  try {
+    return {
+      text,
+      parsed: JSON.parse(text),
+    }
+  } catch {
+    return { text }
+  }
+}
+
+export interface GitMcpClientOptions {
+  name?: string
+  version?: string
+}
+
 export interface GitMcpCallArgs {
   args?: unknown[]
   options?: Record<string, unknown>
+  format?: 'terse' | 'debug'
+  fields?: string[] | string
+  validateOnly?: boolean
+  full?: boolean
   [key: string]: unknown
 }
-
-export type GitMcpCallResponse = {
-  isError: boolean
-  text: string
-  data: unknown
-}
-
-export class GitMcpClient {
-  private readonly client: Client
-  private transport: StdioClientTransport | null = null
-
-  public constructor(options: GitMcpClientOptions = {}) {
-    this.client = new Client(
-      {
-        name: options.name ?? 'git-mcp-client',
-        version: options.version ?? '1.0.0',
-      },
-      {
-        capabilities: {},
-      },
-    )
-  }
-
-  public async connect(command: string, args: string[] = []): Promise<void> {
-    if (this.transport) {
-      throw new Error('Client transport is already connected')
-    }
-
-    this.transport = new StdioClientTransport({
-      command,
-      args,
-    })
-
-    await this.client.connect(this.transport)
-  }
-
-  public async close(): Promise<void> {
-    if (this.transport) {
-      await this.client.close()
-      this.transport = null
-    }
-  }
-
-  public async listTools(): Promise<string[]> {
-    const response = await this.client.request(
-      {
-        method: 'tools/list',
-      },
-      ListToolsResultSchema,
-    )
-
-    return response.tools.map((tool) => tool.name)
-  }
-
-  public async call(toolName: string, args: GitMcpCallArgs = {}): Promise<GitMcpCallResponse> {
-    const response = await this.client.request(
-      {
-        method: 'tools/call',
-        params: {
-          name: toolName,
-          arguments: args,
-        },
-      },
-      CallToolResultSchema,
-    )
-
-    const text = toText(response as ToolResult)
-    const parsed = tryParseResult(text)
-
-    // Back-compat: if a server returns an { ok:false } envelope but forgets to mark MCP isError,
-    // treat it as an error on the client side as well.
-    const parsedIsError =
-      isRecord(parsed.parsed) && typeof parsed.parsed.ok === 'boolean' ? parsed.parsed.ok === false : false
-
-    return {
-      isError: Boolean(response.isError) || parsedIsError,
-      text: parsed.text,
-      data: parsed.parsed,
-    }
-  }
-
+
+export type GitMcpCallResponse = {
+  isError: boolean
+  text: string
+  data: unknown
+}
+
+export class GitMcpClient {
+  private readonly client: Client
+  private transport: StdioClientTransport | null = null
+
+  public constructor(options: GitMcpClientOptions = {}) {
+    this.client = new Client(
+      {
+        name: options.name ?? 'git-mcp-client',
+        version: options.version ?? '1.0.0',
+      },
+      {
+        capabilities: {},
+      },
+    )
+  }
+
+  public async connect(command: string, args: string[] = []): Promise<void> {
+    if (this.transport) {
+      throw new Error('Client transport is already connected')
+    }
+
+    this.transport = new StdioClientTransport({
+      command,
+      args,
+    })
+
+    await this.client.connect(this.transport)
+  }
+
+  public async close(): Promise<void> {
+    if (this.transport) {
+      await this.client.close()
+      this.transport = null
+    }
+  }
+
+  public async listTools(): Promise<string[]> {
+    const response = await this.client.request(
+      {
+        method: 'tools/list',
+      },
+      ListToolsResultSchema,
+    )
+
+    return response.tools.map((tool) => tool.name)
+  }
+
+  public async call(toolName: string, args: GitMcpCallArgs = {}): Promise<GitMcpCallResponse> {
+    const response = await this.client.request(
+      {
+        method: 'tools/call',
+        params: {
+          name: toolName,
+          arguments: args,
+        },
+      },
+      CallToolResultSchema,
+    )
+
+    const text = toText(response as ToolResult)
+    const parsed = tryParseResult(text)
+
+    // Back-compat: if a server returns an { ok:false } envelope but forgets to mark MCP isError,
+    // treat it as an error on the client side as well.
+    const parsedIsError =
+      isRecord(parsed.parsed) && typeof parsed.parsed.ok === 'boolean' ? parsed.parsed.ok === false : false
+
+    return {
+      isError: Boolean(response.isError) || parsedIsError,
+      text: parsed.text,
+      data: parsed.parsed,
+    }
+  }
+
   public async callByPath(
     path: string,
     methodArgs: unknown[] = [],
     methodOptions: Record<string, unknown> = {},
+    controls: Pick<GitMcpCallArgs, 'format' | 'fields' | 'validateOnly' | 'full'> = {},
   ): Promise<GitMcpCallResponse> {
-    const args = { args: methodArgs.map((value) => String(value)), options: methodOptions }
+    const args = {
+      args: methodArgs.map((value) => String(value)),
+      options: methodOptions,
+      ...controls,
+    }
     return this.call(path, args)
   }
 }
-
-export const createGitMcpClient = (options: GitMcpClientOptions = {}): GitMcpClient => {
-  return new GitMcpClient(options)
-}
+
+export const createGitMcpClient = (options: GitMcpClientOptions = {}): GitMcpClient => {
+  return new GitMcpClient(options)
+}

package/mcp/src/index.ts

@@ -1,7 +1,7 @@
-export {
-  createGitMcpServer,
-  runGitMcpServer,
-  normalizeToolCallNameForServer,
-  type GitMcpServerOptions,
-} from './server'
-export { GitMcpClient, createGitMcpClient, type GitMcpCallArgs, type GitMcpClientOptions, type GitMcpCallResponse } from './client'
+export {
+  createGitMcpServer,
+  runGitMcpServer,
+  normalizeToolCallNameForServer,
+  type GitMcpServerOptions,
+} from './server'
+export { GitMcpClient, createGitMcpClient, type GitMcpCallArgs, type GitMcpClientOptions, type GitMcpCallResponse } from './client'