@oss-autopilot/core 1.17.4 → 3.0.0

package/dist/commands/track.d.ts

@@ -1,23 +1,16 @@
 /**
- * Track/Untrack commands (v2 semantics — see #1001)
+ * Track command (v2 semantics — see #1001)
  *
- * **These commands do not mutate state.** In v2, PRs are discovered and
+ * **This command does not mutate state.** In v2, PRs are discovered and
  * enriched automatically on every `daily` run — there is no local tracking
- * list to add to or remove from. The commands are preserved for backwards
- * compatibility with v1 callers, but:
+ * list to add to. `runTrack` is an **informational lookup** that fetches PR
+ * metadata from GitHub and returns it; useful for inspecting a specific
+ * PR's shape without waiting for the next `daily` run. Nothing is persisted.
  *
- * - `runTrack` is an **informational lookup** that fetches PR metadata from
- *   GitHub and returns it. Useful for inspecting a specific PR's shape
- *   without waiting for the next `daily` run. Nothing is persisted.
- * - `runUntrack` is **deprecated** and always a no-op. Use `shelve` to hide
- *   a PR from the daily digest.
+ * The `runUntrack` v1→v2 stub was removed in v4 (#1133). Use `shelve`/
+ * `unshelve` to hide a PR from the daily digest.
  */
 import type { TrackOutput } from '../formatters/json.js';
-export interface UntrackOutput {
-    removed: boolean;
-    url: string;
-    message: string;
-}
 /**
  * Fetch metadata for a PR URL (informational — does not persist).
  *
@@ -33,17 +26,3 @@ export interface UntrackOutput {
 export declare function runTrack(options: {
     prUrl: string;
 }): Promise<TrackOutput>;
-/**
- * @deprecated No-op in v2. Use `runShelve` to hide a PR from the daily digest.
- *
- * Kept for backwards compatibility with v1 callers. PRs are fetched fresh
- * on each `daily` run, so there is no local tracking list to remove from.
- *
- * @param options - Untrack options
- * @param options.prUrl - Full GitHub PR URL (validated but not used)
- * @returns Output object with `removed: false` and a message explaining v2 behavior
- * @throws {ValidationError} If the URL is not a valid GitHub PR URL
- */
-export declare function runUntrack(options: {
-    prUrl: string;
-}): Promise<UntrackOutput>;

package/dist/commands/track.js

@@ -1,21 +1,19 @@
 /**
- * Track/Untrack commands (v2 semantics — see #1001)
+ * Track command (v2 semantics — see #1001)
  *
- * **These commands do not mutate state.** In v2, PRs are discovered and
+ * **This command does not mutate state.** In v2, PRs are discovered and
  * enriched automatically on every `daily` run — there is no local tracking
- * list to add to or remove from. The commands are preserved for backwards
- * compatibility with v1 callers, but:
+ * list to add to. `runTrack` is an **informational lookup** that fetches PR
+ * metadata from GitHub and returns it; useful for inspecting a specific
+ * PR's shape without waiting for the next `daily` run. Nothing is persisted.
  *
- * - `runTrack` is an **informational lookup** that fetches PR metadata from
- *   GitHub and returns it. Useful for inspecting a specific PR's shape
- *   without waiting for the next `daily` run. Nothing is persisted.
- * - `runUntrack` is **deprecated** and always a no-op. Use `shelve` to hide
- *   a PR from the daily digest.
+ * The `runUntrack` v1→v2 stub was removed in v4 (#1133). Use `shelve`/
+ * `unshelve` to hide a PR from the daily digest.
  */
 import { getOctokit, requireGitHubToken } from '../core/index.js';
 import { ValidationError } from '../core/errors.js';
 import { validateUrl, PR_URL_PATTERN, validateGitHubUrl } from './validation.js';
-import { parseGitHubUrl } from '../core/utils.js';
+import { parseGitHubUrl } from '../core/urls.js';
 /**
  * Fetch metadata for a PR URL (informational — does not persist).
  *
@@ -48,23 +46,3 @@ export async function runTrack(options) {
         },
     };
 }
-/**
- * @deprecated No-op in v2. Use `runShelve` to hide a PR from the daily digest.
- *
- * Kept for backwards compatibility with v1 callers. PRs are fetched fresh
- * on each `daily` run, so there is no local tracking list to remove from.
- *
- * @param options - Untrack options
- * @param options.prUrl - Full GitHub PR URL (validated but not used)
- * @returns Output object with `removed: false` and a message explaining v2 behavior
- * @throws {ValidationError} If the URL is not a valid GitHub PR URL
- */
-export async function runUntrack(options) {
-    validateUrl(options.prUrl);
-    validateGitHubUrl(options.prUrl, PR_URL_PATTERN, 'PR');
-    return {
-        removed: false,
-        url: options.prUrl,
-        message: 'In v2, PRs are fetched fresh on each daily run — there is no local tracking list to remove from.',
-    };
-}

package/dist/core/auth.d.ts

@@ -0,0 +1,50 @@
+/**
+ * GitHub authentication: token resolution + username detection.
+ *
+ * Both sync and async token getters share a process-level cache so a
+ * successful async fetch makes subsequent sync calls instant.
+ *
+ * Extracted from utils.ts under #1116.
+ */
+/**
+ * Retrieves a GitHub authentication token, checking sources in priority order.
+ *
+ * Checks `GITHUB_TOKEN` environment variable first, then falls back to `gh auth token`
+ * from the GitHub CLI. The result is cached after the first successful lookup (or first
+ * failed attempt), so subsequent calls are instant and do not spawn subprocesses.
+ *
+ * @returns The GitHub token string, or `null` if no token is available
+ */
+export declare function getGitHubToken(): string | null;
+/**
+ * Returns a GitHub token or throws an error with setup instructions.
+ *
+ * Delegates to {@link getGitHubToken} and throws if no token is found. Use this
+ * in commands that cannot proceed without authentication.
+ *
+ * @throws {ConfigurationError} If no token is available, with instructions for `gh auth login` or setting `GITHUB_TOKEN`
+ */
+export declare function requireGitHubToken(): string;
+/**
+ * Resets the cached GitHub token and fetch-attempted flag.
+ *
+ * Intended for use in tests to ensure a clean state between test cases.
+ * After calling this, the next call to {@link getGitHubToken} will re-fetch the token.
+ */
+export declare function resetGitHubTokenCache(): void;
+/**
+ * Asynchronous version of {@link getGitHubToken}.
+ *
+ * Uses `execFile` (non-blocking) instead of `execFileSync` to avoid blocking
+ * the event loop during CLI cold start. Shares the same cache as the synchronous
+ * version, so a successful async fetch makes subsequent sync calls instant.
+ */
+export declare function getGitHubTokenAsync(): Promise<string | null>;
+/**
+ * Detect the authenticated GitHub username via the `gh` CLI.
+ *
+ * Runs `gh api user --jq '.login'` asynchronously and validates the result
+ * against GitHub's username rules. Never throws — returns `null` on any failure
+ * (gh not installed, not authenticated, invalid output, etc.).
+ */
+export declare function detectGitHubUsername(): Promise<string | null>;

package/dist/core/auth.js

@@ -0,0 +1,160 @@
+/**
+ * GitHub authentication: token resolution + username detection.
+ *
+ * Both sync and async token getters share a process-level cache so a
+ * successful async fetch makes subsequent sync calls instant.
+ *
+ * Extracted from utils.ts under #1116.
+ */
+import { execFileSync, execFile } from 'child_process';
+import { ConfigurationError } from './errors.js';
+import { debug } from './logger.js';
+const MODULE = 'auth';
+// Cached GitHub token (fetched once per session)
+let cachedGitHubToken = null;
+let tokenFetchAttempted = false;
+/**
+ * Retrieves a GitHub authentication token, checking sources in priority order.
+ *
+ * Checks `GITHUB_TOKEN` environment variable first, then falls back to `gh auth token`
+ * from the GitHub CLI. The result is cached after the first successful lookup (or first
+ * failed attempt), so subsequent calls are instant and do not spawn subprocesses.
+ *
+ * @returns The GitHub token string, or `null` if no token is available
+ */
+export function getGitHubToken() {
+    if (cachedGitHubToken) {
+        return cachedGitHubToken;
+    }
+    if (tokenFetchAttempted) {
+        return null;
+    }
+    tokenFetchAttempted = true;
+    if (process.env.GITHUB_TOKEN) {
+        cachedGitHubToken = process.env.GITHUB_TOKEN;
+        return cachedGitHubToken;
+    }
+    try {
+        const token = execFileSync('gh', ['auth', 'token'], {
+            encoding: 'utf-8',
+            stdio: ['pipe', 'pipe', 'pipe'],
+            timeout: 2000,
+        }).trim();
+        if (token && token.length > 0) {
+            cachedGitHubToken = token;
+            debug(MODULE, 'Using GitHub token from gh CLI');
+            return cachedGitHubToken;
+        }
+    }
+    catch (err) {
+        debug(MODULE, 'gh auth token failed (CLI unavailable or not authenticated)', err);
+    }
+    return null;
+}
+/**
+ * Returns a GitHub token or throws an error with setup instructions.
+ *
+ * Delegates to {@link getGitHubToken} and throws if no token is found. Use this
+ * in commands that cannot proceed without authentication.
+ *
+ * @throws {ConfigurationError} If no token is available, with instructions for `gh auth login` or setting `GITHUB_TOKEN`
+ */
+export function requireGitHubToken() {
+    const token = getGitHubToken();
+    if (!token) {
+        throw new ConfigurationError('GitHub authentication required.\n\n' +
+            'Options:\n' +
+            '  1. Use gh CLI: gh auth login\n' +
+            '  2. Set GITHUB_TOKEN environment variable\n\n' +
+            'The gh CLI is recommended - install from https://cli.github.com');
+    }
+    return token;
+}
+/**
+ * Resets the cached GitHub token and fetch-attempted flag.
+ *
+ * Intended for use in tests to ensure a clean state between test cases.
+ * After calling this, the next call to {@link getGitHubToken} will re-fetch the token.
+ */
+export function resetGitHubTokenCache() {
+    cachedGitHubToken = null;
+    tokenFetchAttempted = false;
+}
+/**
+ * Asynchronous version of {@link getGitHubToken}.
+ *
+ * Uses `execFile` (non-blocking) instead of `execFileSync` to avoid blocking
+ * the event loop during CLI cold start. Shares the same cache as the synchronous
+ * version, so a successful async fetch makes subsequent sync calls instant.
+ */
+export async function getGitHubTokenAsync() {
+    if (cachedGitHubToken) {
+        return cachedGitHubToken;
+    }
+    if (tokenFetchAttempted) {
+        return null;
+    }
+    tokenFetchAttempted = true;
+    if (process.env.GITHUB_TOKEN) {
+        cachedGitHubToken = process.env.GITHUB_TOKEN;
+        return cachedGitHubToken;
+    }
+    try {
+        const token = await new Promise((resolve, reject) => {
+            execFile('gh', ['auth', 'token'], { encoding: 'utf-8', timeout: 2000 }, (error, stdout) => {
+                if (error) {
+                    reject(error);
+                }
+                else {
+                    resolve(stdout.trim());
+                }
+            });
+        });
+        if (token && token.length > 0) {
+            cachedGitHubToken = token;
+            debug(MODULE, 'Using GitHub token from gh CLI (async)');
+            return cachedGitHubToken;
+        }
+    }
+    catch (err) {
+        debug(MODULE, 'gh auth token failed (CLI unavailable or not authenticated)', err);
+    }
+    return null;
+}
+/**
+ * GitHub username validation pattern.
+ * Usernames must start with an alphanumeric character, can contain hyphens
+ * (but not consecutive ones and not at the end), and be 1-39 characters.
+ */
+const GITHUB_USERNAME_RE = /^[a-zA-Z0-9](?:[a-zA-Z0-9]|-(?=[a-zA-Z0-9])){0,38}$/;
+/**
+ * Detect the authenticated GitHub username via the `gh` CLI.
+ *
+ * Runs `gh api user --jq '.login'` asynchronously and validates the result
+ * against GitHub's username rules. Never throws — returns `null` on any failure
+ * (gh not installed, not authenticated, invalid output, etc.).
+ */
+export async function detectGitHubUsername() {
+    try {
+        const login = await new Promise((resolve, reject) => {
+            execFile('gh', ['api', 'user', '--jq', '.login'], { encoding: 'utf-8', timeout: 5000 }, (error, stdout) => {
+                if (error) {
+                    reject(error);
+                }
+                else {
+                    resolve(stdout.trim());
+                }
+            });
+        });
+        if (login && GITHUB_USERNAME_RE.test(login)) {
+            debug(MODULE, `Detected GitHub username: ${login}`);
+            return login;
+        }
+        debug(MODULE, `gh api user returned invalid username: "${login}"`);
+        return null;
+    }
+    catch (err) {
+        debug(MODULE, 'detectGitHubUsername failed', err);
+        return null;
+    }
+}

package/dist/core/concurrency.d.ts

@@ -1,3 +1,10 @@
+/**
+ * Concurrency primitives shared across modules.
+ */
+/** Default concurrency limit for parallel GitHub API requests. */
+export declare const DEFAULT_CONCURRENCY = 5;
+/** Async sleep — exported for mockability in tests. */
+export declare function sleep(ms: number): Promise<void>;
 /**
  * Runs a worker pool that processes items with bounded concurrency.
  * N workers consume from a shared index. On any worker error, remaining

package/dist/core/concurrency.js

@@ -1,3 +1,12 @@
+/**
+ * Concurrency primitives shared across modules.
+ */
+/** Default concurrency limit for parallel GitHub API requests. */
+export const DEFAULT_CONCURRENCY = 5;
+/** Async sleep — exported for mockability in tests. */
+export function sleep(ms) {
+    return new Promise((resolve) => setTimeout(resolve, ms));
+}
 /**
  * Runs a worker pool that processes items with bounded concurrency.
  * N workers consume from a shared index. On any worker error, remaining

package/dist/core/daily-logic.d.ts

@@ -1,17 +1,20 @@
 /**
- * Domain logic extracted from src/commands/daily.ts.
+ * Domain aggregation logic extracted from src/commands/daily.ts.
  *
- * Pure or near-pure functions that operate on FetchedPR data:
+ * Pure or near-pure functions that operate on FetchedPR data and produce
+ * the data types that the renderers / JSON output consume:
  *   - computeRepoSignals / groupPRsByRepo — per-repo aggregations
  *   - assessCapacity — capacity assessment against active PRs
  *   - collectActionableIssues — ordered list of issues needing attention
  *   - computeActionMenu — pre-computed action menu for orchestration
  *   - toShelvedPRRef — lightweight projection for digest display
- *   - formatActionHint — human-readable maintainer action hint label
- *   - formatBriefSummary / formatSummary / printDigest — rendering
+ *
+ * Rendering functions (formatActionHint, formatBriefSummary, formatSummary,
+ * printDigest) live in src/commands/daily-render.ts (#1117). This file
+ * re-exports them at the bottom so existing imports keep working without
+ * a sweep.
  */
-import type { FetchedPR, FetchedPRStatus, StalenessTier, ActionReason, DailyDigest, AgentState, ShelvedPRRef, MaintainerActionHint, ComputedRepoSignals, RepoGroup, CommentedIssue, CommentedIssueWithResponse } from './types.js';
-import type { CapacityAssessment, ActionableIssue, ActionMenu } from '../formatters/json.js';
+import type { FetchedPR, FetchedPRStatus, StalenessTier, ActionReason, AgentState, ShelvedPRRef, ComputedRepoSignals, RepoGroup, CommentedIssue, CapacityAssessment, ActionableIssue, ActionMenu } from './types.js';
 /**
  * Statuses indicating action needed from the contributor.
  * Used for auto-unshelving shelved PRs.
@@ -85,12 +88,6 @@ export declare function assessCapacity(activePRs: FetchedPR[], maxActivePRs: num
  * @returns Ordered actionable issues grouped by priority
  */
 export declare function collectActionableIssues(prs: FetchedPR[], lastDigestAt?: string): ActionableIssue[];
-/**
- * Format a maintainer action hint as a human-readable label.
- * @param hint - The maintainer action hint enum value
- * @returns Human-readable label (e.g., "tests requested")
- */
-export declare function formatActionHint(hint: MaintainerActionHint): string;
 /**
  * Compute the action menu from PR data and capacity.
  * The orchestration layer can insert issue-list options (e.g., "Pick from list")
@@ -102,33 +99,4 @@ export declare function formatActionHint(hint: MaintainerActionHint): string;
  * @returns Action menu with context flags for orchestration
  */
 export declare function computeActionMenu(actionableIssues: ActionableIssue[], capacity: CapacityAssessment, commentedIssues?: CommentedIssue[]): ActionMenu;
-/**
- * Format a brief one-liner summary for the action-first flow.
- *
- * @param digest - The daily digest containing PR summary stats
- * @param issueCount - Number of actionable issues needing attention
- * @param issueResponseCount - Number of issue replies from maintainers
- * @returns One-line status string (e.g., "3 Active PRs | 1 needs attention | 2 issue replies")
- */
-export declare function formatBriefSummary(digest: DailyDigest, issueCount: number, issueResponseCount?: number): string;
-/**
- * Format the full dashboard summary as markdown.
- * Used in JSON output for Claude to display verbatim — includes all PR sections,
- * issue replies, and capacity status.
- *
- * @param digest - The daily digest with all PR categories
- * @param capacity - Current capacity assessment for display
- * @param issueResponses - Issue replies from maintainers to display
- * @returns Multi-line markdown string suitable for terminal or chat display
- */
-export declare function formatSummary(digest: DailyDigest, capacity: CapacityAssessment, issueResponses?: CommentedIssueWithResponse[]): string;
-/**
- * Print digest to console as plain text.
- * Unified renderer: uses the same section ordering as {@link formatSummary}
- * but outputs plain text with console.log instead of markdown links.
- *
- * @param digest - The daily digest with all PR categories
- * @param capacity - Current capacity assessment for display
- * @param commentedIssues - All commented issues (filtered to responses internally)
- */
-export declare function printDigest(digest: DailyDigest, capacity: CapacityAssessment, commentedIssues?: CommentedIssue[]): void;
+export { formatActionHint, formatBriefSummary, formatSummary, printDigest } from '../commands/daily-render.js';

package/dist/core/daily-logic.js

@@ -1,16 +1,19 @@
 /**
- * Domain logic extracted from src/commands/daily.ts.
+ * Domain aggregation logic extracted from src/commands/daily.ts.
  *
- * Pure or near-pure functions that operate on FetchedPR data:
+ * Pure or near-pure functions that operate on FetchedPR data and produce
+ * the data types that the renderers / JSON output consume:
  *   - computeRepoSignals / groupPRsByRepo — per-repo aggregations
  *   - assessCapacity — capacity assessment against active PRs
  *   - collectActionableIssues — ordered list of issues needing attention
  *   - computeActionMenu — pre-computed action menu for orchestration
  *   - toShelvedPRRef — lightweight projection for digest display
- *   - formatActionHint — human-readable maintainer action hint label
- *   - formatBriefSummary / formatSummary / printDigest — rendering
+ *
+ * Rendering functions (formatActionHint, formatBriefSummary, formatSummary,
+ * printDigest) live in src/commands/daily-render.ts (#1117). This file
+ * re-exports them at the bottom so existing imports keep working without
+ * a sweep.
  */
-import { formatRelativeTime } from './utils.js';
 import { warn } from './logger.js';
 import { errorMessage } from './errors.js';
 import { getStateManager } from './state.js';
@@ -277,25 +280,6 @@ export function collectActionableIssues(prs, lastDigestAt) {
     }
     return issues;
 }
-/**
- * Format a maintainer action hint as a human-readable label.
- * @param hint - The maintainer action hint enum value
- * @returns Human-readable label (e.g., "tests requested")
- */
-export function formatActionHint(hint) {
-    switch (hint) {
-        case 'demo_requested':
-            return 'demo/screenshot requested';
-        case 'tests_requested':
-            return 'tests requested';
-        case 'changes_requested':
-            return 'code changes requested';
-        case 'docs_requested':
-            return 'documentation requested';
-        case 'rebase_requested':
-            return 'rebase requested';
-    }
-}
 /**
  * Compute the action menu from PR data and capacity.
  * The orchestration layer can insert issue-list options (e.g., "Pick from list")
@@ -369,181 +353,10 @@ export function computeActionMenu(actionableIssues, capacity, commentedIssues =
     };
 }
 // ---------------------------------------------------------------------------
-// Rendering / formatting
+// Rendering re-exports (back-compat)
 // ---------------------------------------------------------------------------
-/**
- * Format a brief one-liner summary for the action-first flow.
- *
- * @param digest - The daily digest containing PR summary stats
- * @param issueCount - Number of actionable issues needing attention
- * @param issueResponseCount - Number of issue replies from maintainers
- * @returns One-line status string (e.g., "3 Active PRs | 1 needs attention | 2 issue replies")
- */
-export function formatBriefSummary(digest, issueCount, issueResponseCount = 0) {
-    const attentionText = issueCount > 0 ? `${issueCount} need${issueCount === 1 ? 's' : ''} attention` : 'all on track';
-    const issueReplyText = issueResponseCount > 0 ? ` | ${issueResponseCount} issue repl${issueResponseCount === 1 ? 'y' : 'ies'}` : '';
-    return `\u{1F4CA} ${digest.summary.totalActivePRs} Active PRs | ${attentionText}${issueReplyText}`;
-}
-/**
- * Format the full dashboard summary as markdown.
- * Used in JSON output for Claude to display verbatim — includes all PR sections,
- * issue replies, and capacity status.
- *
- * @param digest - The daily digest with all PR categories
- * @param capacity - Current capacity assessment for display
- * @param issueResponses - Issue replies from maintainers to display
- * @returns Multi-line markdown string suitable for terminal or chat display
- */
-export function formatSummary(digest, capacity, issueResponses = []) {
-    const lines = [];
-    // Header
-    lines.push('## OSS Dashboard');
-    lines.push('');
-    lines.push(`\u{1F4CA} **${digest.summary.totalActivePRs} Active PRs** | ${digest.summary.totalMergedAllTime} Merged | ${digest.summary.mergeRate}% Merge Rate`);
-    lines.push('\u2713 Dashboard generated \u2014 say "open dashboard" to view in browser');
-    lines.push('');
-    // Needs Addressing
-    if (digest.needsAddressingPRs.length > 0) {
-        lines.push('### \u274C Needs Addressing');
-        for (const pr of digest.needsAddressingPRs) {
-            lines.push(`- [${pr.repo}#${pr.number}](${pr.url}): ${pr.title}`);
-            lines.push(`  \u2514\u2500 ${pr.displayLabel} ${pr.displayDescription}`);
-        }
-        lines.push('');
-    }
-    // Waiting on Maintainer
-    if (digest.waitingOnMaintainerPRs.length > 0) {
-        lines.push('### \u23F3 Waiting on Maintainer');
-        for (const pr of digest.waitingOnMaintainerPRs) {
-            lines.push(`- [${pr.repo}#${pr.number}](${pr.url}): ${pr.title}`);
-            lines.push(`  \u2514\u2500 ${pr.displayDescription}`);
-        }
-        lines.push('');
-    }
-    // Recently Merged (wins!)
-    if (digest.recentlyMergedPRs.length > 0) {
-        lines.push('### \u{1F389} Recently Merged');
-        for (const pr of digest.recentlyMergedPRs) {
-            const mergedDate = pr.mergedAt ? new Date(pr.mergedAt).toLocaleDateString() : '';
-            lines.push(`- [${pr.repo}#${pr.number}](${pr.url}): ${pr.title}${mergedDate ? ` (merged ${mergedDate})` : ''}`);
-        }
-        lines.push('');
-    }
-    // Recently Closed (closed without merge)
-    if (digest.recentlyClosedPRs.length > 0) {
-        lines.push('### \u{1F6AB} Recently Closed');
-        for (const pr of digest.recentlyClosedPRs) {
-            const closedDate = pr.closedAt ? new Date(pr.closedAt).toLocaleDateString() : '';
-            lines.push(`- [${pr.repo}#${pr.number}](${pr.url}): ${pr.title}${closedDate ? ` (closed ${closedDate})` : ''}`);
-        }
-        lines.push('');
-    }
-    // Auto-unshelved (important: maintainer engagement on shelved PRs)
-    if (digest.autoUnshelvedPRs.length > 0) {
-        lines.push('### \u{1F514} Auto-Unshelved');
-        lines.push('> These PRs were shelved but a maintainer engaged \u2014 moved back to active.');
-        for (const pr of digest.autoUnshelvedPRs) {
-            lines.push(`- [${pr.repo}#${pr.number}](${pr.url}): ${pr.title} (${pr.status.replace(/_/g, ' ')})`);
-        }
-        lines.push('');
-    }
-    // Shelved PRs (dimmed, excluded from capacity)
-    if (digest.shelvedPRs.length > 0) {
-        lines.push('### \u{1F4E6} Shelved');
-        for (const pr of digest.shelvedPRs) {
-            lines.push(`- [${pr.repo}#${pr.number}](${pr.url}): ${pr.title}`);
-        }
-        lines.push('');
-    }
-    // Issue Replies
-    if (issueResponses.length > 0) {
-        lines.push('### \u{1F4AC} Issue Replies');
-        for (const issue of issueResponses) {
-            lines.push(`- [${issue.repo}#${issue.number}](${issue.url}): ${issue.title}`);
-            const timeAgo = formatRelativeTime(issue.lastResponseAt);
-            lines.push(`  \u2514\u2500 @${issue.lastResponseAuthor}: "${issue.lastResponseBody.slice(0, 80)}${issue.lastResponseBody.length > 80 ? '...' : ''}"${timeAgo ? ` (${timeAgo})` : ''}`);
-        }
-        lines.push('');
-    }
-    // Capacity
-    const capacityIcon = capacity.hasCapacity ? '\u2705' : '\u26A0\uFE0F';
-    const capacityLabel = capacity.hasCapacity ? 'Ready for new work' : 'Focus on existing PRs';
-    const shelvedNote = capacity.shelvedPRCount > 0 ? ` + ${capacity.shelvedPRCount} shelved` : '';
-    lines.push(`**Capacity:** ${capacityIcon} ${capacityLabel} (${capacity.activePRCount}/${capacity.maxActivePRs} PRs${shelvedNote})`);
-    return lines.join('\n');
-}
-/**
- * Print digest to console as plain text.
- * Unified renderer: uses the same section ordering as {@link formatSummary}
- * but outputs plain text with console.log instead of markdown links.
- *
- * @param digest - The daily digest with all PR categories
- * @param capacity - Current capacity assessment for display
- * @param commentedIssues - All commented issues (filtered to responses internally)
- */
-export function printDigest(digest, capacity, commentedIssues = []) {
-    console.log('\n\u{1F4CA} OSS Daily Check\n');
-    console.log(`Active PRs: ${digest.summary.totalActivePRs}`);
-    console.log(`Needing Attention: ${digest.summary.totalNeedingAttention}`);
-    console.log(`Merged (all time): ${digest.summary.totalMergedAllTime}`);
-    console.log(`Merge Rate: ${digest.summary.mergeRate}%`);
-    console.log(`\nCapacity: ${capacity.hasCapacity ? '\u2705 Ready for new work' : '\u26A0\uFE0F  Focus on existing work'}`);
-    console.log(`  ${capacity.reason}\n`);
-    if (digest.needsAddressingPRs.length > 0) {
-        console.log('\u274C Needs Addressing:');
-        for (const pr of digest.needsAddressingPRs) {
-            console.log(`  - ${pr.repo}#${pr.number}: ${pr.title}`);
-            console.log(`    ${pr.displayLabel} ${pr.displayDescription}`);
-        }
-        console.log('');
-    }
-    if (digest.waitingOnMaintainerPRs.length > 0) {
-        console.log('\u23F3 Waiting on Maintainer:');
-        for (const pr of digest.waitingOnMaintainerPRs) {
-            console.log(`  - ${pr.repo}#${pr.number}: ${pr.title}`);
-            console.log(`    ${pr.displayDescription}`);
-        }
-        console.log('');
-    }
-    if (digest.recentlyMergedPRs.length > 0) {
-        console.log('\u{1F389} Recently Merged:');
-        for (const pr of digest.recentlyMergedPRs) {
-            const mergedDate = pr.mergedAt ? new Date(pr.mergedAt).toLocaleDateString() : '';
-            console.log(`  - ${pr.repo}#${pr.number}: ${pr.title}${mergedDate ? ` (merged ${mergedDate})` : ''}`);
-        }
-        console.log('');
-    }
-    if (digest.recentlyClosedPRs.length > 0) {
-        console.log('\u{1F6AB} Recently Closed:');
-        for (const pr of digest.recentlyClosedPRs) {
-            const closedDate = pr.closedAt ? new Date(pr.closedAt).toLocaleDateString() : '';
-            console.log(`  - ${pr.repo}#${pr.number}: ${pr.title}${closedDate ? ` (closed ${closedDate})` : ''}`);
-        }
-        console.log('');
-    }
-    if (digest.autoUnshelvedPRs.length > 0) {
-        console.log('\u{1F514} Auto-Unshelved:');
-        for (const pr of digest.autoUnshelvedPRs) {
-            console.log(`  - ${pr.repo}#${pr.number}: ${pr.title} (${pr.status.replace(/_/g, ' ')})`);
-        }
-        console.log('');
-    }
-    if (digest.shelvedPRs.length > 0) {
-        console.log('\u{1F4E6} Shelved:');
-        for (const pr of digest.shelvedPRs) {
-            console.log(`  - ${pr.repo}#${pr.number}: ${pr.title}`);
-        }
-        console.log('');
-    }
-    const issueResponses = commentedIssues.filter((i) => i.status === 'new_response');
-    if (issueResponses.length > 0) {
-        console.log('\u{1F4AC} Issue Replies:');
-        for (const issue of issueResponses) {
-            console.log(`  - ${issue.repo}#${issue.number}: ${issue.title}`);
-            console.log(`    @${issue.lastResponseAuthor}: ${issue.lastResponseBody.slice(0, 80)}${issue.lastResponseBody.length > 80 ? '...' : ''}`);
-        }
-        console.log('');
-    }
-    console.log('Run with --json for structured output');
-    console.log('Run "dashboard serve" for browser view');
-}
+//
+// The actual implementations live in src/commands/daily-render.ts. Existing
+// callers that import these from `core/daily-logic` continue to work; new
+// callers should import from `commands/daily-render` directly.
+export { formatActionHint, formatBriefSummary, formatSummary, printDigest } from '../commands/daily-render.js';