fix(getResources): move dotenv to default options

Author: cdcabrera

src/__tests__/__snapshots__/options.defaults.test.ts.snap

@@ -82,5 +82,8 @@ exports[`options defaults should return specific properties: defaults 1`] = `
   "toolModules": [],
   "urlRegex": /\\^\\(https\\?:\\)\\\\/\\\\//i,
   "version": "0.0.0",
+  "xhrFetch": {
+    "timeoutMs": 15000,
+  },
 }
 `;

src/options.defaults.ts

@@ -45,6 +45,7 @@ interface DefaultOptions<TLogOptions = LoggingOptions> {
   contextPath: string;
   contextUrl: string;
   docsPath: string;
+  xhrFetch: XhrFetchOptions;
   http: HttpOptions;
   isHttp: boolean;
   logging: TLogOptions;
@@ -153,13 +154,19 @@ interface LoggingSession extends LoggingOptions {
   readonly channelName: string;
 }
 
+/**
+ * Base stats options.
+ */
 type StatsOptions = {
   reportIntervalMs: {
     health: number;
     transport: number;
   }
 };
 
+/**
+ * Stats channel names.
+ */
 type StatsChannels = {
   readonly health: string;
   readonly session: string;
@@ -179,6 +186,17 @@ interface StatsSession extends StatsOptions {
   channels: StatsChannels
 }
 
+/**
+ * XHR and Fetch options.
+ *
+ * @interface XhrFetchOptions
+ *
+ * @property timeoutMs Timeout for XHR and Fetch requests (ms).
+ */
+interface XhrFetchOptions {
+  timeoutMs: number;
+}
+
 /**
  * Base logging options.
  */
@@ -205,7 +223,7 @@ const HTTP_OPTIONS: HttpOptions = {
  */
 const PLUGIN_HOST_OPTIONS: PluginHostOptions = {
   loadTimeoutMs: 5000,
-  invokeTimeoutMs: 10000,
+  invokeTimeoutMs: 10_000,
   gracePeriodMs: 2000
 };
 
@@ -250,7 +268,7 @@ const TOOL_MEMO_OPTIONS = {
 };
 
 /**
- * Stats options.
+ * Default stats options.
  */
 const STATS_OPTIONS: StatsOptions = {
   reportIntervalMs: {
@@ -259,6 +277,13 @@ const STATS_OPTIONS: StatsOptions = {
   }
 };
 
+/**
+ * Default XHR and Fetch options.
+ */
+const XHR_FETCH_OPTIONS: XhrFetchOptions = {
+  timeoutMs: 15_000
+};
+
 /**
  * Base logging channel name. Fixed to avoid user override.
  */
@@ -387,7 +412,8 @@ const DEFAULT_OPTIONS: DefaultOptions = {
   toolModules: [],
   separator: DEFAULT_SEPARATOR,
   urlRegex: URL_REGEX,
-  version: (process.env.NODE_ENV === 'local' && '0.0.0') || packageJson.version
+  version: (process.env.NODE_ENV === 'local' && '0.0.0') || packageJson.version,
+  xhrFetch: XHR_FETCH_OPTIONS
 };
 
 export {

src/server.getResources.ts

@@ -23,12 +23,12 @@ readLocalFileFunction.memo = memo(readLocalFileFunction, DEFAULT_OPTIONS.resourc
  *
  * @note Review expanding fetch to handle more file types like JSON.
  *
- * @param url
+ * @param url - URL to fetch
+ * @param options - Options
  */
-const fetchUrlFunction = async (url: string) => {
+const fetchUrlFunction = async (url: string, options = getOptions()) => {
   const controller = new AbortController();
-  const timeoutMs = Number(process.env.DOC_MCP_FETCH_TIMEOUT_MS || 15_000);
-  const timeout = setTimeout(() => controller.abort(), timeoutMs);
+  const timeout = setTimeout(() => controller.abort(), options.xhrFetch.timeoutMs);
 
   // Allow the process to exit
   timeout.unref();