@opengolfapi/mcp-server 2.2.2 → 2.2.3

package/README.md

@@ -63,9 +63,21 @@ Donor tiers raise the daily limit further (10k / 50k / 250k / 1M).
 
 ## Telemetry
 
-Optional: set `SENTRY_DSN` env var to send errors to your own Sentry instance.
-We do not collect telemetry from end users — this is opt-in for the operator
-running the server.
+This server reports unhandled errors to the OpenGolfAPI Sentry project by default so the maintainers can catch bugs that hit real users. **No request bodies, no API keys, no PII** — only stack traces of exceptions thrown inside the server process. Quotas and sampling live in the Sentry project config; the DSN is an ingest endpoint, not a secret.
+
+To opt out completely:
+
+```
+OPENGOLFAPI_DISABLE_TELEMETRY=1 npx @opengolfapi/mcp-server
+```
+
+To send to your own Sentry project instead (overrides the default DSN):
+
+```
+SENTRY_DSN=https://your-dsn@sentry.io/... npx @opengolfapi/mcp-server
+```
+
+Example MCP config:
 
 ```json
 {
@@ -74,17 +86,13 @@ running the server.
       "command": "npx",
       "args": ["@opengolfapi/mcp-server"],
       "env": {
-        "OPENGOLFAPI_KEY": "ogapi_yourkeyhere",
-        "SENTRY_DSN": "https://<key>@o<org>.ingest.sentry.io/<project>"
+        "OPENGOLFAPI_KEY": "ogapi_yourkeyhere"
       }
     }
   }
 }
 ```
 
-When `SENTRY_DSN` is unset the SDK is a complete no-op — nothing is initialized
-and no network calls are made.
-
 ## License
 
 MIT

package/dist/index.js

@@ -12,13 +12,20 @@
  * Install: npx @opengolfapi/mcp-server
  */
 // Sentry must initialize BEFORE any other imports that could throw, so any
-// load-time error in transitive deps still gets captured. The init is a no-op
-// when SENTRY_DSN is unset, so end users who run the server via npx never
-// send telemetry unless they set the env var themselves.
+// load-time error in transitive deps still gets captured.
+//
+// Default DSN ships baked-in so we get bug telemetry from every install — a
+// Sentry DSN is designed to be public (it's an ingest endpoint, not a secret).
+// Users can override with SENTRY_DSN, or opt out completely with
+// OPENGOLFAPI_DISABLE_TELEMETRY=1.
 import * as Sentry from '@sentry/node';
-if (process.env.SENTRY_DSN) {
+const DEFAULT_SENTRY_DSN = 'https://2cb261cd86bbfe9e105309d3c2edbced@o4511071885000704.ingest.us.sentry.io/4511345201315840';
+const SENTRY_DSN_ACTIVE = process.env.OPENGOLFAPI_DISABLE_TELEMETRY
+    ? ''
+    : (process.env.SENTRY_DSN || DEFAULT_SENTRY_DSN);
+if (SENTRY_DSN_ACTIVE) {
     Sentry.init({
-        dsn: process.env.SENTRY_DSN,
+        dsn: SENTRY_DSN_ACTIVE,
         tracesSampleRate: 0.1,
         release: `opengolfapi-mcp-server@${process.env.npm_package_version || 'unknown'}`,
     });
@@ -27,7 +34,7 @@ import { McpServer } from '@modelcontextprotocol/sdk/server/mcp.js';
 import { StdioServerTransport } from '@modelcontextprotocol/sdk/server/stdio.js';
 import { z } from 'zod';
 // Package version — used in User-Agent so the API can identify MCP traffic.
-const PKG_VERSION = '2.2.2';
+const PKG_VERSION = '2.2.3';
 const API_BASE = process.env.OPENGOLFAPI_BASE ?? 'https://api.opengolfapi.org';
 // Optional API key for higher rate limits. Anonymous (no key) still works
 // at 1k req/day per IP. With a free key from courses.opengolfapi.org/api-keys,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@opengolfapi/mcp-server",
-  "version": "2.2.2",
+  "version": "2.2.3",
   "description": "Open MCP server for AI agents to query the OpenGolfAPI dataset (14,708 US golf courses, ODbL)",
   "type": "module",
   "bin": {
@@ -13,7 +13,8 @@
   ],
   "scripts": {
     "build": "tsc",
-    "prepublishOnly": "npm run build"
+    "prepublishOnly": "npm test",
+    "test": "npm run build && vitest run"
   },
   "repository": {
     "type": "git",
@@ -38,6 +39,7 @@
   },
   "devDependencies": {
     "@types/node": "^20.0.0",
-    "typescript": "^5.0.0"
+    "typescript": "^5.0.0",
+    "vitest": "^4.1.5"
   }
 }