@oss-scout/mcp 0.9.0 → 0.10.1

package/dist/tools.js

@@ -3,6 +3,25 @@ import { SearchStrategySchema, ISSUE_URL_PATTERN, validateGitHubUrl, validateUrl
 // Generous ceiling: even with inter-phase delays zeroed, vetting many issues
 // under the budget tracker's sliding-window pacing can take minutes (#143).
 const TOOL_TIMEOUT_MS = 300000;
+// Surfaced on the JSON envelope when scout.checkpoint() returns false (#212).
+// The skip tool reports this inline because it returns a short status line;
+// the JSON tools (search, scout-features, config-set) carry it as a top-level
+// field so a persist failure isn't silently swallowed.
+const PERSIST_WARNING = "Checkpoint failed: changes were applied in memory but could not be persisted. They may be lost when the server restarts.";
+/**
+ * Serialize a tool result, attaching a top-level `persistWarning` field when
+ * the preceding checkpoint failed (#212). Spreads onto the existing object so
+ * the happy-path payload is byte-for-byte unchanged.
+ */
+function withPersistWarning(payload, persisted) {
+    return persisted ? payload : { ...payload, persistWarning: PERSIST_WARNING };
+}
+/**
+ * Rejects when `ms` elapses, but does NOT cancel the underlying work — the
+ * scout call keeps running (consuming API budget and mutating scout state)
+ * after the tool has returned its timeout error. Acceptable for a local
+ * stdio server; revisit with AbortSignal threading if it ever matters (#295).
+ */
 function withTimeout(promise, ms = TOOL_TIMEOUT_MS) {
     let timeoutId;
     const timer = new Promise((_, reject) => {
@@ -14,8 +33,11 @@ export function registerTools(server, scout) {
     server.tool("search", "Search for open source issues matching your preferences", {
         maxResults: z
             .number()
+            .int()
+            .min(1)
+            .max(50)
             .optional()
-            .describe("Maximum number of results to return (default 10)"),
+            .describe("Maximum number of results to return (1-50, default 10)"),
         strategies: z
             .string()
             .optional()
@@ -28,24 +50,41 @@ export function registerTools(server, scout) {
             .array(z.string())
             .optional()
             .describe("Soft-boost ranking for candidates in one of these `owner/repo` slugs. Stronger weight than language match (#1244). Not a filter; only reorders."),
+        avoidRepos: z
+            .array(z.string())
+            .optional()
+            .describe("Soft PENALTY (milder than excludeRepos) for candidates in one of these `owner/repo` slugs (#168). Pushes them down the ranking but does not remove them; a strong boost can still outweigh it."),
+        boostIssueTypes: z
+            .array(z.string())
+            .optional()
+            .describe('Soft-boost ranking for candidates whose issue labels match one of these types, case-insensitive (e.g. "bug", "good first issue") (#168). Not a filter; only reorders.'),
         diversityRatio: z
             .number()
             .min(0)
             .max(1)
             .optional()
             .describe("Diversity counterweight (#1244): fraction of result slots reserved for candidates that matched NEITHER preference list. Counters echo-chamber bias when boosts accumulate over time. 0 disables, 1 reserves every slot for diversity. Default 0."),
-    }, async ({ maxResults, strategies, preferLanguages, preferRepos, diversityRatio, }) => {
+    }, async ({ maxResults, strategies, preferLanguages, preferRepos, avoidRepos, boostIssueTypes, diversityRatio, }) => {
         try {
+            // Validate each CSV entry with a readable error instead of letting a
+            // raw ZodError string reach the model (#295).
             const parsedStrategies = strategies
-                ? strategies
-                    .split(",")
-                    .map((s) => SearchStrategySchema.parse(s.trim()))
+                ? strategies.split(",").map((s) => {
+                    const trimmed = s.trim();
+                    const parsed = SearchStrategySchema.safeParse(trimmed);
+                    if (!parsed.success) {
+                        throw new Error(`Invalid strategy "${trimmed}". Valid strategies: ${SearchStrategySchema.options.join(", ")}`);
+                    }
+                    return parsed.data;
+                })
                 : undefined;
             const result = await withTimeout(scout.search({
                 maxResults: maxResults ?? 10,
                 strategies: parsedStrategies,
                 preferLanguages,
                 preferRepos,
+                avoidRepos,
+                boostIssueTypes,
                 diversityRatio,
                 // No fixed inter-phase sleeps in the request/response MCP context;
                 // the budget tracker still paces the GitHub calls (#143)
@@ -53,9 +92,14 @@ export function registerTools(server, scout) {
                 broadPhaseDelayMs: 0,
             }));
             scout.saveResults(result.candidates);
-            await scout.checkpoint();
+            const persisted = await scout.checkpoint();
             return {
-                content: [{ type: "text", text: JSON.stringify(result, null, 2) }],
+                content: [
+                    {
+                        type: "text",
+                        text: JSON.stringify(withPersistWarning(result, persisted), null, 2),
+                    },
+                ],
             };
         }
         catch (err) {
@@ -69,8 +113,11 @@ export function registerTools(server, scout) {
     server.tool("scout-features", "Surface feature-scoped contribution opportunities in repos where you have 3+ merged PRs", {
         maxResults: z
             .number()
+            .int()
+            .min(1)
+            .max(50)
             .optional()
-            .describe("Maximum number of results to return (default 10)"),
+            .describe("Maximum number of results to return (1-50, default 10)"),
         anchorThreshold: z
             .number()
             .int()
@@ -92,9 +139,14 @@ export function registerTools(server, scout) {
                 splitRatio,
             }));
             scout.saveResults([...result.quickWins, ...result.biggerBets]);
-            await scout.checkpoint();
+            const persisted = await scout.checkpoint();
             return {
-                content: [{ type: "text", text: JSON.stringify(result, null, 2) }],
+                content: [
+                    {
+                        type: "text",
+                        text: JSON.stringify(withPersistWarning(result, persisted), null, 2),
+                    },
+                ],
             };
         }
         catch (err) {
@@ -192,6 +244,9 @@ export function registerTools(server, scout) {
                 };
             }
             if (action === "remove") {
+                // Deliberately no URL validation, matching the CLI's runSkipRemove:
+                // entries stored before skip-add validation existed may be junk,
+                // and exact-match removal is the only way to clean them up.
                 const wasPresent = scout
                     .getSkippedIssues()
                     .some((s) => s.url === issueUrl);
@@ -279,10 +334,15 @@ export function registerTools(server, scout) {
         scout.updatePreferences({
             [key]: validated[key],
         });
-        await scout.checkpoint();
+        const persisted = await scout.checkpoint();
         const updated = scout.getPreferences();
         return {
-            content: [{ type: "text", text: JSON.stringify(updated, null, 2) }],
+            content: [
+                {
+                    type: "text",
+                    text: JSON.stringify(withPersistWarning(updated, persisted), null, 2),
+                },
+            ],
         };
     });
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@oss-scout/mcp",
-  "version": "0.9.0",
+  "version": "0.10.1",
   "description": "MCP server for oss-scout — exposes issue discovery and vetting as MCP tools",
   "type": "module",
   "bin": {
@@ -8,8 +8,8 @@
   },
   "exports": {
     ".": {
-      "import": "./dist/index.js",
-      "types": "./dist/index.d.ts"
+      "types": "./dist/index.d.ts",
+      "import": "./dist/index.js"
     }
   },
   "files": [
@@ -18,7 +18,7 @@
   "dependencies": {
     "@modelcontextprotocol/sdk": "^1.27.1",
     "zod": "^4.3.6",
-    "@oss-scout/core": "^1.0.0"
+    "@oss-scout/core": "^1.5.1"
   },
   "devDependencies": {
     "@types/node": "^25.5.0",