jd-intel-mcp 0.8.0 → 0.8.1

package/errors.js

@@ -1,16 +1,9 @@
 /**
  * Error code taxonomy for all MCP tools.
  *
- * Codes are short and stable. Messages are short and factual.
- * The tool description teaches the AI what each code means; errors
- * themselves do not need to be verbose.
+ * The canonical codes now live in the jd-intel library (src/errors.js), so the
+ * library (which throws AtsError with a .code) and the MCP layer share one
+ * source of truth. Re-exported here to keep the existing import path stable.
  */
 
-export const ERROR_CODES = {
-  COMPANY_NOT_FOUND: 'company_not_found',  // Slug not in registry and not detected
-  ATS_UNREACHABLE: 'ats_unreachable',      // Known ATS failed (500, timeout)
-  PARTIAL_FAILURE: 'partial_failure',      // Discovery mode; some adapters failed
-  INVALID_ARGS: 'invalid_args',            // Missing required, wrong type, bad pattern
-  NO_RESULTS: 'no_results',                // Query succeeded, filters returned nothing
-  RATE_LIMITED: 'rate_limited',            // Upstream returned 429
-};
+export { ERROR_CODES } from 'jd-intel';

package/package.json

@@ -1,6 +1,7 @@
 {
   "name": "jd-intel-mcp",
-  "version": "0.8.0",
+  "version": "0.8.1",
+  "mcpName": "io.github.prPMDev/jd-intel-mcp",
   "description": "MCP server for jd-intel. Your AI assistant fetches and reads full job descriptions across seven ATS platforms, no copy-paste.",
   "type": "module",
   "main": "server.js",

package/tools.js

@@ -10,7 +10,7 @@
  */
 
 import { z } from 'zod';
-import { fetchJobs, detectAts as libDetectAts, registry, ATS_NAMES } from 'jd-intel';
+import { fetchJobs, detectAts as libDetectAts, registry, ATS_NAMES, AtsError } from 'jd-intel';
 
 const { search: searchRegistry, findAtsBySlug } = registry;
 // Tolerate an older jd-intel that predates getSource. The bundle always
@@ -104,23 +104,19 @@ export function registerTools(server, deps = {}) {
         });
       } catch (err) {
         const msg = err.message || 'Unknown error';
-        // Map the library error to the advertised taxonomy. Order matters: a rate
-        // limit surfaces as "...API error...: 429", so the 429 check must run
-        // before the generic /API error/ check. This string-matches the adapters'
-        // message wording (the contained trade-off vs typed errors in the library).
-        if (config && /Workday API error/.test(msg)) {
-          // Keep the Workday triple-repair hint even on a 429.
-          return error(
-            ERROR_CODES.ATS_UNREACHABLE,
-            `Workday rejected ${config.tenant}/${config.env}/${config.site}: ${msg}. Verify the triple against the careers URL https://{tenant}.{env}.myworkdayjobs.com/{site}.`
-          );
-        }
-        if (/:\s*429\b/.test(msg)) {
-          return error(ERROR_CODES.RATE_LIMITED, msg);
-        }
-        if (/API error/.test(msg)) {
-          return error(ERROR_CODES.ATS_UNREACHABLE, msg);
+        // AtsError carries a stable .code from the adapter (ats_unreachable /
+        // rate_limited), so we map by code, not by parsing the message.
+        if (err instanceof AtsError) {
+          if (config && err.code === ERROR_CODES.ATS_UNREACHABLE) {
+            // Keep the Workday triple-repair hint.
+            return error(
+              ERROR_CODES.ATS_UNREACHABLE,
+              `Workday rejected ${config.tenant}/${config.env}/${config.site}: ${msg}. Verify the triple against the careers URL https://{tenant}.{env}.myworkdayjobs.com/{site}.`
+            );
+          }
+          return error(err.code, msg);
         }
+        // Anything else is an arg-validation error from the library.
         return error(ERROR_CODES.INVALID_ARGS, msg);
       }
     }