@foundation0/git 1.2.1 → 1.2.2

package/mcp/src/cli.ts

@@ -41,14 +41,18 @@ if (hasFlag('--help') || hasFlag('-h')) {
   process.exit(0)
 }
 
-const owner = getArgValue('--default-owner', process.env.GITEA_TEST_OWNER ?? 'example-org')
-const repo = getArgValue('--default-repo', process.env.GITEA_TEST_REPO ?? 'example-repo')
-const host = getArgValue('--gitea-host', process.env.GITEA_HOST ?? process.env.EXAMPLE_GITEA_HOST ?? 'https://gitea.example.com')?.trim()
+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.',
@@ -63,8 +67,8 @@ void runGitMcpServer({
     giteaHost: host,
     giteaToken: token,
   },
-  defaultOwner: owner,
-  defaultRepo: repo,
+  ...(owner ? { defaultOwner: owner } : {}),
+  ...(repo ? { defaultRepo: repo } : {}),
   toolsPrefix,
 }).catch((error) => {
   console.error('Failed to start MCP git server', error)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@foundation0/git",
-  "version": "1.2.1",
+  "version": "1.2.2",
   "description": "Foundation 0 Git API and MCP server",
   "type": "module",
   "bin": {

package/src/git-service-api.ts

@@ -218,6 +218,22 @@ const buildUrl = (
   return url.toString()
 }
 
+const unresolvedPathParamPattern = /^\{[^{}]+\}$/
+
+const assertResolvedMappedPath = (
+  mappedPath: string[],
+  featurePath: string[],
+): void => {
+  const unresolved = mappedPath.filter((segment) => unresolvedPathParamPattern.test(segment))
+  if (unresolved.length === 0) {
+    return
+  }
+
+  throw new Error(
+    `Missing required path arguments for "${featurePath.join('.')}". Unresolved parameters: ${unresolved.join(', ')}`,
+  )
+}
+
 const canUseAbortSignalTimeout = (): boolean =>
   typeof AbortSignal !== 'undefined' && typeof (AbortSignal as unknown as { timeout?: unknown }).timeout === 'function'
 
@@ -369,6 +385,7 @@ const createMethod = (
 
       return segment
     })
+    assertResolvedMappedPath(hydratedPath, feature.path)
 
     const requestBody = buildRequestBody(mapping.method, bodyOptions, unhandled)
     const headers = {
@@ -530,8 +547,8 @@ export const createGitServiceApi = (options: GitServiceApiFactoryOptions = {}):
   const log = options.log
 
   const defaults = {
-    defaultOwner: options.defaultOwner ?? process.env.GITEA_TEST_OWNER ?? 'example-org',
-    defaultRepo: options.defaultRepo ?? process.env.GITEA_TEST_REPO ?? 'example-repo',
+    defaultOwner: options.defaultOwner,
+    defaultRepo: options.defaultRepo,
   }
 
   const root: GitServiceApi = {}
@@ -562,4 +579,24 @@ export const createGitServiceApi = (options: GitServiceApiFactoryOptions = {}):
   return root
 }
 
-export const gitServiceApi = createGitServiceApi()
+const createUnavailableGitServiceApi = (error: Error): GitServiceApi => {
+  return new Proxy(
+    {},
+    {
+      get: (): never => {
+        throw error
+      },
+    },
+  ) as GitServiceApi
+}
+
+export const gitServiceApi: GitServiceApi = (() => {
+  try {
+    return createGitServiceApi()
+  } catch (error) {
+    const message = error instanceof Error ? error.message : String(error)
+    return createUnavailableGitServiceApi(
+      new Error(`Failed to initialize gitServiceApi singleton: ${message}`),
+    )
+  }
+})()

package/src/issue-dependencies.ts

@@ -2,7 +2,6 @@ import type { GitServiceApiExecutionResult } from './git-service-api'
 import { spawn } from 'node:child_process'
 import crypto from 'node:crypto'
 
-const DEFAULT_GITEA_HOST = 'https://gitea.example.com'
 const DEFAULT_REQUEST_TIMEOUT_MS = 60_000
 
 const parseRequestTimeoutMs = (value: unknown): number | null => {
@@ -274,6 +273,14 @@ const resolveGiteaApiBase = (host: string): string => {
   return trimmed.endsWith('/api/v1') ? trimmed : `${trimmed}/api/v1`
 }
 
+const resolveRequiredGiteaHost = (host: string | undefined): string => {
+  const resolved = host?.trim() ?? process.env.GITEA_HOST?.trim()
+  if (!resolved) {
+    throw new Error('GITEA_HOST is required. Pass host explicitly or set process.env.GITEA_HOST.')
+  }
+  return resolved
+}
+
 const buildIssueDependenciesUrl = (host: string, owner: string, repo: string, issueNumber: number): string => {
   return `${resolveGiteaApiBase(host)}/repos/${encodeURIComponent(owner)}/${encodeURIComponent(repo)}/issues/${issueNumber}/dependencies`
 }
@@ -283,11 +290,12 @@ export async function callIssueDependenciesApi(
   owner: string,
   repo: string,
   issueNumber: number,
-  host: string = DEFAULT_GITEA_HOST,
+  host: string | undefined,
   token: string | undefined,
   payload?: GitIssueDependencyPayload
 ): Promise<GitServiceApiExecutionResult<unknown>> {
-  const requestUrl = buildIssueDependenciesUrl(host, owner, repo, issueNumber)
+  const resolvedHost = resolveRequiredGiteaHost(host)
+  const requestUrl = buildIssueDependenciesUrl(resolvedHost, owner, repo, issueNumber)
   const headers = {
     Accept: 'application/json',
     ...(token ? { Authorization: `token ${token}` } : {}),
@@ -343,7 +351,7 @@ export async function callIssueDependenciesApi(
       method,
       query: [],
       headers: [],
-      apiBase: resolveGiteaApiBase(host),
+      apiBase: resolveGiteaApiBase(resolvedHost),
       swaggerPath: '/repos/{owner}/{repo}/issues/{index}/dependencies',
       mapped: true,
     },

package/src/platform/config.ts

@@ -8,7 +8,6 @@ export interface GitPlatformConfig {
   giteaSwaggerPath?: string
 }
 
-const DEFAULT_GITEA_HOST = 'https://gitea.example.com'
 const DEFAULT_PLATFORM: PlatformName = 'GITEA'
 const DEFAULT_GITEA_SWAGGER_PATH = '/swagger.v1.json'
 
@@ -34,8 +33,7 @@ export const getGitPlatformConfig = (overrides: Partial<GitPlatformConfig> = {})
 
   const giteaHost =
     overrides.giteaHost ??
-    process.env.GITEA_HOST ??
-    DEFAULT_GITEA_HOST
+    process.env.GITEA_HOST
 
   const giteaToken =
     overrides.giteaToken ??
@@ -50,9 +48,13 @@ export const getGitPlatformConfig = (overrides: Partial<GitPlatformConfig> = {})
     process.env.GITEA_SWAGGER_PATH ??
     DEFAULT_GITEA_SWAGGER_PATH
 
+  if (!giteaHost || giteaHost.trim().length === 0) {
+    throw new Error('GITEA_HOST is required. Set process.env.GITEA_HOST or pass config.giteaHost explicitly.')
+  }
+
   return {
     platform,
-    giteaHost,
+    giteaHost: giteaHost.trim(),
     giteaToken,
     giteaApiVersion,
     giteaSwaggerPath,