ralph-hero-mcp-server 2.5.75 → 2.5.79

package/dist/lib/directions.js

@@ -34,6 +34,7 @@ export const DEFAULT_RANK_CONFIG = {
     lockStaleHours: 24,
     treeRecentDoneDays: 7,
     prStaleHours: 24,
+    audience: "human",
 };
 // ---------------------------------------------------------------------------
 // Internal scoring constants
@@ -60,6 +61,18 @@ const STALE_BOOST = -50;
 const LOCK_STALE_BOOST = -100;
 const TREE_CONTINUE_BOOST = -75;
 const PR_REVIEW_REQUIRED_BOOST = -200;
+/**
+ * Per-estimate penalty applied when audience === "agent". Larger items
+ * cost more (positive score), pushing them down the ranking so agent
+ * loops dispatch on XS/S items first.
+ */
+const ESTIMATE_PENALTY = {
+    XS: 0,
+    S: 0,
+    M: 20,
+    L: 40,
+    XL: 60,
+};
 const HOUR_MS = 60 * 60 * 1000;
 const DAY_MS = 24 * HOUR_MS;
 // ---------------------------------------------------------------------------
@@ -100,6 +113,20 @@ function hasOpenBlockers(item) {
 function isLockState(state) {
     return state !== null && LOCK_STATES.includes(state);
 }
+/**
+ * Returns a positive score adjustment when audience === "agent" so larger
+ * estimates get pushed down the ranking. Returns 0 for human audience
+ * (existing behavior). Items with unknown estimate get a mid-tier penalty
+ * (30) so unestimated work doesn't accidentally outrank XS/S items.
+ */
+function audiencePenalty(item, audience) {
+    if (audience !== "agent")
+        return 0;
+    const est = item.estimate;
+    if (est === null || est === undefined)
+        return 30; // unknown: mid penalty
+    return ESTIMATE_PENALTY[est] ?? 30;
+}
 // ---------------------------------------------------------------------------
 // Detection helpers
 // ---------------------------------------------------------------------------
@@ -176,6 +203,9 @@ export function detectTreeContinue(item, allItems, config) {
 export function scoreIssue(item, allItems, config) {
     const tags = [];
     let score = priorityScore(item.priority) + phaseScore(item.workflowState);
+    // Audience-aware estimate penalty (no-op for "human"; pushes XL items
+    // down for "agent" so autonomous loops favor XS/S work).
+    score += audiencePenalty(item, config.audience);
     const lockStale = detectLockStale(item, config);
     const treeContinue = detectTreeContinue(item, allItems, config);
     // Stale boost (non-lock states only)
@@ -295,7 +325,20 @@ export function buildReason(kind, issue, pr, tags, config, linkedIssueNumber = n
         const hours = Math.round(ageHours(issue.updatedAt, config.now));
         const days = Math.max(1, Math.floor(hours / 24));
         const dayLabel = days === 1 ? "day" : "days";
-        return `${phase} for ${days} ${dayLabel} — likely the most unblocking thing`;
+        const priority = issue.priority;
+        if (priority === "P0") {
+            return `P0 stalled in ${phase} for ${days} ${dayLabel} — top of the queue`;
+        }
+        if (priority === "P1") {
+            return `P1 stalled in ${phase} for ${days} ${dayLabel} — likely the most unblocking thing`;
+        }
+        if (priority === "P2") {
+            return `Sitting in ${phase} for ${days} ${dayLabel} — small unblock if you have a moment`;
+        }
+        if (priority === "P3") {
+            return `Low-priority item in ${phase} for ${days} ${dayLabel}`;
+        }
+        return `Unprioritized in ${phase} for ${days} ${dayLabel}`;
     }
     if (issue.priority === "P0") {
         return `P0 in ${phase} — top of the queue`;
@@ -423,6 +466,7 @@ export function rankDirections(items, openPRs, config) {
             const reason = buildReason(c.kind, c.item, null, c.tags, config, null);
             return {
                 rank,
+                recommended: false,
                 kind: c.kind,
                 issue: toDirectionIssue(c.item),
                 pr: null,
@@ -436,6 +480,7 @@ export function rankDirections(items, openPRs, config) {
         const reason = buildReason("pr", null, p.pr, p.tags, config, p.linkedIssueNumber);
         return {
             rank,
+            recommended: false,
             kind: "pr",
             issue: null,
             pr: toDirectionPR(p.pr),
@@ -444,6 +489,12 @@ export function rankDirections(items, openPRs, config) {
             score: p.score,
         };
     });
+    // Mark the top-ranked entry as recommended. Both modes use this
+    // flag for selection: interactive picker pre-selects it; headless
+    // orchestrators dispatch on it.
+    if (directions.length > 0) {
+        directions[0].recommended = true;
+    }
     return directions;
 }
 //# sourceMappingURL=directions.js.map

package/dist/tools/directions-tools.js

@@ -1,11 +1,14 @@
 /**
- * MCP tool wrapping the pure ranker at `lib/directions.ts`.
+ * MCP tools wrapping the pure ranker at `lib/directions.ts`.
  *
- * Exposes `ralph_hero__hello_directions`: a single-tool, fixed-shape,
- * deterministic surface that returns up to N ranked "directions" for the
- * `hello` skill's session briefing. The skill is responsible for fetching
- * open PRs (via `gh pr list`) and passing them in as a parameter — the
- * MCP server does not itself open an Octokit-style PR API surface.
+ * Exposes two tools that share a single implementation:
+ *   - `ralph_hero__next_actions` (current name; accepts `audience` param)
+ *   - `ralph_hero__hello_directions` (DEPRECATED alias; fixed at audience="human")
+ *
+ * Both return a fixed-shape JSON payload with up to N ranked "directions"
+ * for the `hello` skill's session briefing. The skill is responsible for
+ * fetching open PRs (via `gh pr list`) and passing them in as a parameter
+ * — the MCP server does not itself open an Octokit-style PR API surface.
  *
  * Behaviour:
  *   1. Resolve owner + project numbers from args or client config.
@@ -32,68 +35,27 @@ import { toolSuccess, toolError, resolveProjectOwner, resolveProjectNumbers, } f
 // ---------------------------------------------------------------------------
 const HOUR_MS = 60 * 60 * 1000;
 // ---------------------------------------------------------------------------
-// Register
+// Shared schema fragments
 // ---------------------------------------------------------------------------
-export function registerDirectionsTools(server, client, fieldCache) {
-    server.tool("ralph_hero__hello_directions", "Compute up to N deterministic 'directions' for the hello skill's session briefing. Single tool call returns a fixed-shape JSON payload with ranked issue/PR action items. Open PRs must be passed in as a parameter (the MCP server does not fetch them itself).", {
-        owner: z
-            .string()
-            .optional()
-            .describe("GitHub owner. Defaults to RALPH_GH_OWNER env var."),
-        projectNumbers: z
-            .array(z.coerce.number())
-            .optional()
-            .describe("Project numbers to include. Defaults to RALPH_GH_PROJECT_NUMBERS or single configured project."),
-        limit: z
-            .number()
-            .int()
-            .nonnegative()
-            .optional()
-            .default(3)
-            .describe("Max directions to return (default: 3)."),
-        stuckThresholdHours: z
-            .number()
-            .nonnegative()
-            .optional()
-            .default(48)
-            .describe("Hours before a non-lock issue is considered stale (default: 48)."),
-        lockStaleHours: z
-            .number()
-            .nonnegative()
-            .optional()
-            .default(24)
-            .describe("Hours before a lock-state issue is considered stalled (default: 24)."),
-        treeRecentDoneDays: z
-            .number()
-            .nonnegative()
-            .optional()
-            .default(7)
-            .describe("Days within which a sibling Done event still pulls a tree forward (default: 7)."),
-        prStaleHours: z
-            .number()
-            .nonnegative()
-            .optional()
-            .default(24)
-            .describe("Hours before an open PR is considered stale (default: 24)."),
-        openPRs: z
-            .array(z.object({
-            number: z.number(),
-            title: z.string(),
-            url: z.string(),
-            isDraft: z.boolean(),
-            reviewDecision: z
-                .string()
-                .nullable()
-                .describe("REVIEW_REQUIRED | APPROVED | CHANGES_REQUESTED | null"),
-            headRefName: z.string(),
-            createdAt: z
-                .string()
-                .describe("ISO timestamp from gh pr list"),
-        }))
-            .optional()
-            .default([])
-            .describe("Open PRs gathered by the caller (e.g. via `gh pr list`). Drafts and APPROVED PRs are filtered internally."),
-    }, async (args) => {
+const openPRSchema = z.object({
+    number: z.number(),
+    title: z.string(),
+    url: z.string(),
+    isDraft: z.boolean(),
+    reviewDecision: z
+        .string()
+        .nullable()
+        .describe("REVIEW_REQUIRED | APPROVED | CHANGES_REQUESTED | null"),
+    headRefName: z.string(),
+    createdAt: z.string().describe("ISO timestamp from gh pr list"),
+});
+// ---------------------------------------------------------------------------
+// Shared implementation — extracted so both `hello_directions` (deprecated)
+// and `next_actions` (current) can route through the same code path. Keep
+// file-private (no export) — alias lives in this same file.
+// ---------------------------------------------------------------------------
+function makeRunDirections(client, fieldCache) {
+    return async function runDirections(args) {
         try {
             const owner = args.owner || resolveProjectOwner(client.config);
             if (!owner) {
@@ -124,6 +86,7 @@ export function registerDirectionsTools(server, client, fieldCache) {
                 lockStaleHours: args.lockStaleHours ?? DEFAULT_RANK_CONFIG.lockStaleHours,
                 treeRecentDoneDays: args.treeRecentDoneDays ?? DEFAULT_RANK_CONFIG.treeRecentDoneDays,
                 prStaleHours: args.prStaleHours ?? DEFAULT_RANK_CONFIG.prStaleHours,
+                audience: args.audience,
                 now,
             };
             // Compute PR ageHours at the boundary so the lib never reads the wall clock.
@@ -154,6 +117,113 @@ export function registerDirectionsTools(server, client, fieldCache) {
             const message = error instanceof Error ? error.message : String(error);
             return toolError(`Failed to compute hello directions: ${message}`);
         }
+    };
+}
+// ---------------------------------------------------------------------------
+// Register
+// ---------------------------------------------------------------------------
+export function registerDirectionsTools(server, client, fieldCache) {
+    const runDirections = makeRunDirections(client, fieldCache);
+    server.tool("ralph_hero__hello_directions", "[DEPRECATED — use ralph_hero__next_actions instead. Removed in 2.7.0.] Compute up to N deterministic 'directions' for the hello skill's session briefing.", {
+        owner: z
+            .string()
+            .optional()
+            .describe("GitHub owner. Defaults to RALPH_GH_OWNER env var."),
+        projectNumbers: z
+            .array(z.coerce.number())
+            .optional()
+            .describe("Project numbers to include. Defaults to RALPH_GH_PROJECT_NUMBERS or single configured project."),
+        limit: z
+            .number()
+            .int()
+            .nonnegative()
+            .optional()
+            .default(3)
+            .describe("Max directions to return (default: 3)."),
+        stuckThresholdHours: z
+            .number()
+            .nonnegative()
+            .optional()
+            .default(48)
+            .describe("Hours before a non-lock issue is considered stale (default: 48)."),
+        lockStaleHours: z
+            .number()
+            .nonnegative()
+            .optional()
+            .default(24)
+            .describe("Hours before a lock-state issue is considered stalled (default: 24)."),
+        treeRecentDoneDays: z
+            .number()
+            .nonnegative()
+            .optional()
+            .default(7)
+            .describe("Days within which a sibling Done event still pulls a tree forward (default: 7)."),
+        prStaleHours: z
+            .number()
+            .nonnegative()
+            .optional()
+            .default(24)
+            .describe("Hours before an open PR is considered stale (default: 24)."),
+        openPRs: z
+            .array(openPRSchema)
+            .optional()
+            .default([])
+            .describe("Open PRs gathered by the caller (e.g. via `gh pr list`). Drafts and APPROVED PRs are filtered internally."),
+    }, async (args) => {
+        return await runDirections({ ...args, audience: "human" });
+    });
+    server.tool("ralph_hero__next_actions", "Compute up to N deterministic 'directions' (next actions) with one flagged `recommended: true`. Used by the /hello skill picker (interactive) and by headless orchestrators (auto-select recommended). Open PRs must be passed in as a parameter.", {
+        owner: z
+            .string()
+            .optional()
+            .describe("GitHub owner. Defaults to RALPH_GH_OWNER env var."),
+        projectNumbers: z
+            .array(z.coerce.number())
+            .optional()
+            .describe("Project numbers to include. Defaults to RALPH_GH_PROJECT_NUMBERS or single configured project."),
+        limit: z
+            .number()
+            .int()
+            .nonnegative()
+            .optional()
+            .default(3)
+            .describe("Max directions to return (default: 3)."),
+        audience: z
+            .enum(["human", "agent"])
+            .optional()
+            .default("human")
+            .describe("Tilts scoring per consumer; agent penalizes large estimates to honor autonomous-loop XS/S preference (default: human)."),
+        stuckThresholdHours: z
+            .number()
+            .nonnegative()
+            .optional()
+            .default(48)
+            .describe("Hours before a non-lock issue is considered stale (default: 48)."),
+        lockStaleHours: z
+            .number()
+            .nonnegative()
+            .optional()
+            .default(24)
+            .describe("Hours before a lock-state issue is considered stalled (default: 24)."),
+        treeRecentDoneDays: z
+            .number()
+            .nonnegative()
+            .optional()
+            .default(7)
+            .describe("Days within which a sibling Done event still pulls a tree forward (default: 7)."),
+        prStaleHours: z
+            .number()
+            .nonnegative()
+            .optional()
+            .default(24)
+            .describe("Hours before an open PR is considered stale (default: 24)."),
+        openPRs: z
+            .array(openPRSchema)
+            .optional()
+            .default([])
+            .describe("Open PRs gathered by the caller (e.g. via `gh pr list`). Drafts and APPROVED PRs are filtered internally."),
+    }, async (args) => {
+        return await runDirections({ ...args, audience: args.audience ?? "human" });
     });
 }
 //# sourceMappingURL=directions-tools.js.map

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ralph-hero-mcp-server",
-  "version": "2.5.75",
+  "version": "2.5.79",
   "description": "MCP server for GitHub Projects V2 - Ralph workflow automation",
   "type": "module",
   "main": "dist/index.js",