From 16a09ae0486b943faf0dc63cbdafadb5ea8785aa Mon Sep 17 00:00:00 2001
From: Saatvik Arya <aryasaatvik@gmail.com>
Date: Wed, 28 Jan 2026 05:46:42 +0530
Subject: [PATCH] fix: support JSONC in config files

Use the existing parseJsonBuffer helper (which leverages jsonc-parser)
instead of JSON.parse() when reading config files. This enables:

- Single-line comments (//)
- Multi-line comments (/* */)
- Trailing commas

The jsonc-parser dependency was already present but wasn't being used
for the main config file parsing.
---
 src/config.ts | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/config.ts b/src/config.ts
index ca56b70..428124f 100644
--- a/src/config.ts
+++ b/src/config.ts
@@ -4,6 +4,7 @@ import os from 'node:os';
 import path from 'node:path';
 import { pathsForImport, readExternalEntries } from './config-imports.js';
 import { normalizeServerEntry } from './config-normalize.js';
+import { parseJsonBuffer } from './config/imports/shared.js';
 import {
   DEFAULT_IMPORTS,
   type LoadConfigOptions,
@@ -209,7 +210,7 @@ async function readConfigFile(configPath: string, explicit: boolean): Promise<Ra
   }
   try {
     const buffer = await fs.readFile(configPath, 'utf8');
-    return RawConfigSchema.parse(JSON.parse(buffer));
+    return RawConfigSchema.parse(parseJsonBuffer(buffer));
   } catch (error) {
     if (!explicit && isMissingConfigError(error)) {
       return { mcpServers: {} };