ralph-hero-mcp-server 2.5.74 → 2.5.79

package/dist/index.js

@@ -20,6 +20,7 @@ import { registerProjectTools } from "./tools/project-tools.js";
 import { registerIssueTools } from "./tools/issue-tools.js";
 import { registerRelationshipTools } from "./tools/relationship-tools.js";
 import { registerDashboardTools } from "./tools/dashboard-tools.js";
+import { registerDirectionsTools } from "./tools/directions-tools.js";
 import { registerBatchTools } from "./tools/batch-tools.js";
 import { registerProjectManagementTools } from "./tools/project-management-tools.js";
 import { registerHygieneTools } from "./tools/hygiene-tools.js";
@@ -386,6 +387,8 @@ async function main() {
     registerRelationshipTools(server, client, fieldCache);
     // Dashboard and pipeline visualization tools
     registerDashboardTools(server, client, fieldCache);
+    // Hello directions tool — deterministic ranked action items
+    registerDirectionsTools(server, client, fieldCache);
     // Phase 5: Batch operations
     registerBatchTools(server, client, fieldCache);
     // Project management tools (archive, remove, add, link repo, clear field)

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

@@ -0,0 +1,229 @@
+/**
+ * MCP tools wrapping the pure ranker at `lib/directions.ts`.
+ *
+ * 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.
+ *   2. For each project, ensure the field cache is populated, then
+ *      paginate `DASHBOARD_ITEMS_QUERY` to gather up to 500 items.
+ *   3. Convert raw items to `DashboardItem[]` via `toDashboardItems`.
+ *   4. Build a `RankConfig` from the args + defaults (with injected `now`).
+ *   5. Compute each PR's `ageHours` at the boundary and call
+ *      `rankDirections(allItems, enrichedPRs, config)`.
+ *   6. Return `{ directions, fetchedAt, totalCandidates }`.
+ *
+ * Determinism: `fetchedAt` is the only time-varying field. Two consecutive
+ * calls on the same board state produce byte-identical `directions[]`
+ * (after stripping `fetchedAt`).
+ */
+import { z } from "zod";
+import { ensureFieldCache } from "../lib/helpers.js";
+import { paginateConnection } from "../lib/pagination.js";
+import { DASHBOARD_ITEMS_QUERY, toDashboardItems, } from "./dashboard-tools.js";
+import { rankDirections, DEFAULT_RANK_CONFIG, } from "../lib/directions.js";
+import { toolSuccess, toolError, resolveProjectOwner, resolveProjectNumbers, } from "../types.js";
+// ---------------------------------------------------------------------------
+// Constants
+// ---------------------------------------------------------------------------
+const HOUR_MS = 60 * 60 * 1000;
+// ---------------------------------------------------------------------------
+// Shared schema fragments
+// ---------------------------------------------------------------------------
+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) {
+                return toolError("owner is required");
+            }
+            const projectNumbers = args.projectNumbers ?? resolveProjectNumbers(client.config);
+            if (projectNumbers.length === 0) {
+                return toolError("No project numbers configured. Set RALPH_GH_PROJECT_NUMBER or RALPH_GH_PROJECT_NUMBERS.");
+            }
+            // Inject `now` at the boundary so the lib remains deterministic.
+            const now = new Date();
+            const allItems = [];
+            for (const pn of projectNumbers) {
+                await ensureFieldCache(client, fieldCache, owner, pn);
+                const projectId = fieldCache.getProjectId(pn);
+                if (!projectId) {
+                    // Defensive: ensureFieldCache succeeded but no projectId.
+                    // Skip this project rather than blowing up the whole call.
+                    continue;
+                }
+                const result = await paginateConnection((q, v) => client.projectQuery(q, v), DASHBOARD_ITEMS_QUERY, { projectId, first: 100 }, "node.items", { maxItems: 500 });
+                allItems.push(...toDashboardItems(result.nodes, pn));
+            }
+            // Build the RankConfig from args + defaults + injected `now`.
+            const config = {
+                limit: args.limit ?? DEFAULT_RANK_CONFIG.limit,
+                stuckThresholdHours: args.stuckThresholdHours ?? DEFAULT_RANK_CONFIG.stuckThresholdHours,
+                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.
+            const enrichedPRs = (args.openPRs ?? []).map((pr) => {
+                const t = new Date(pr.createdAt).getTime();
+                const ageHours = Number.isNaN(t)
+                    ? 0
+                    : Math.max(0, (now.getTime() - t) / HOUR_MS);
+                return {
+                    number: pr.number,
+                    title: pr.title,
+                    url: pr.url,
+                    isDraft: pr.isDraft,
+                    reviewDecision: pr.reviewDecision,
+                    headRefName: pr.headRefName,
+                    createdAt: pr.createdAt,
+                    ageHours,
+                };
+            });
+            const directions = rankDirections(allItems, enrichedPRs, config);
+            return toolSuccess({
+                directions,
+                fetchedAt: now.toISOString(),
+                totalCandidates: allItems.length,
+            });
+        }
+        catch (error) {
+            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.74",
+  "version": "2.5.79",
   "description": "MCP server for GitHub Projects V2 - Ralph workflow automation",
   "type": "module",
   "main": "dist/index.js",