pierre-review 0.1.67 → 0.1.69

package/dist/api/routes/prs.js

@@ -2,7 +2,8 @@ import { createHash } from 'node:crypto';
 import { config } from '../../config.js';
 import { getAccessToken, getAccountUserId } from '../../auth/account.js';
 import { fetchActionsJobLog } from '../../github/actions-logs.js';
-import { getMentionCandidates, getPrDetail, getPrFilesContext, getPrWriteContext, getReviewerLogins, markAllViewed, markPrMergedLocally, markPrViewed, upsertLocalPrComment, upsertLocalReview, } from '../../db/queries.js';
+import { getMentionCandidates, getPrDetail, getPrFilesContext, getPrWriteContext, getReviewerLogins, getUsersByLogins, markAllViewed, markPrMergedLocally, markPrViewed, upsertLocalPrComment, upsertLocalReview, } from '../../db/queries.js';
+import { getCodeownersMatch } from '../../github/codeowners.js';
 import { buildFileAnchors, fallbackAnchor, isFindingAnchored, } from '../../github/diff-anchor.js';
 import { addIssueComment, fetchHeadShaFor, fetchMergeability, fetchPrFilesWithPatch, fetchPrHeadInfo, fetchRepoMergeConfig, mergePullRequest, postInlineComment, requestReviewers, rerunWorkflowRun, submitPrReview, updatePullRequestBranch, } from '../../github/mutations.js';
 import { hydratePrDetail } from '../../sync/hydrate-detail.js';
@@ -128,18 +129,95 @@ const requestReviewersSchema = {
     ...idParamSchema,
     body: {
         type: 'object',
-        required: ['userIds'],
         additionalProperties: false,
+        // All three are optional; the handler 400s if the combined set is empty. `userIds`
+        // are resolved to logins; `logins` pass through (suggested reviewers we haven't synced);
+        // `teamSlugs` become team review requests (CODEOWNERS @org/team).
         properties: {
-            userIds: {
-                type: 'array',
-                minItems: 1,
-                maxItems: 15,
-                items: { type: 'integer' },
-            },
+            userIds: { type: 'array', maxItems: 15, items: { type: 'integer' } },
+            logins: { type: 'array', maxItems: 15, items: { type: 'string' } },
+            teamSlugs: { type: 'array', maxItems: 15, items: { type: 'string' } },
         },
     },
 };
+// Layer CODEOWNERS-derived suggestions on top of the history-based ones getPrDetail already
+// computed. Runs ONLY when the PR still warrants suggestions (same trigger getPrDetail used).
+// Best-effort: any failure (no CODEOWNERS, org wall, parse error) leaves the history
+// suggestions untouched. CODEOWNERS owners (declared ownership) take precedence, then history
+// fills up to a small cap. Mutates + returns the detail.
+async function enrichSuggestionsWithCodeowners(detail, accountId) {
+    const wants = detail.state === 'open' &&
+        !detail.isDraft &&
+        detail.requestedReviewers.length === 0 &&
+        detail.reviews.length === 0;
+    if (!wants)
+        return detail;
+    const [owner, name] = detail.repoFullName.split('/');
+    if (!owner || !name)
+        return detail;
+    const paths = detail.files.map((f) => f.path);
+    if (paths.length === 0)
+        return detail;
+    try {
+        const token = await getAccessToken(accountId);
+        const match = await getCodeownersMatch(token, owner, name, accountId, paths);
+        if (match.logins.length === 0 && match.teams.length === 0)
+            return detail;
+        const authorLogin = detail.authorId != null
+            ? detail.users.find((u) => u.id === detail.authorId)?.githubLogin ?? null
+            : null;
+        // Resolve @user owners to synced users (avatar/link); unsynced owners show login-only.
+        // Exclude the PR author (GitHub rejects self-review requests).
+        const uniqueLogins = [...new Set(match.logins)].filter((l) => l !== authorLogin);
+        const resolved = await getUsersByLogins(uniqueLogins);
+        const byLogin = new Map(resolved.map((u) => [u.githubLogin, u]));
+        const codeownerUsers = uniqueLogins.map((login) => ({
+            kind: 'user',
+            login,
+            userId: byLogin.get(login)?.id ?? null,
+            teamSlug: null,
+            teamName: null,
+            reason: 'owns this path (CODEOWNERS)',
+            source: 'codeowners',
+        }));
+        const codeownerTeams = match.teams.map((t) => ({
+            kind: 'team',
+            login: null,
+            userId: null,
+            teamSlug: t.slug,
+            teamName: t.name,
+            reason: 'owns this path (CODEOWNERS)',
+            source: 'codeowners',
+        }));
+        // Merge: CODEOWNERS users, then teams, then the history suggestions — dedup users by
+        // login and teams by slug, capped so the row stays digestible.
+        const seenLogins = new Set();
+        const seenTeams = new Set();
+        const merged = [];
+        for (const s of [...codeownerUsers, ...codeownerTeams, ...detail.suggestedReviewers]) {
+            if (s.kind === 'team') {
+                if (s.teamSlug && !seenTeams.has(s.teamSlug)) {
+                    seenTeams.add(s.teamSlug);
+                    merged.push(s);
+                }
+            }
+            else if (s.login && !seenLogins.has(s.login)) {
+                seenLogins.add(s.login);
+                merged.push(s);
+            }
+        }
+        detail.suggestedReviewers = merged.slice(0, 5);
+        // Surface any newly-resolved codeowner users so the client can render avatars/links.
+        const known = new Set(detail.users.map((u) => u.id));
+        for (const u of resolved)
+            if (!known.has(u.id))
+                detail.users.push(u);
+    }
+    catch {
+        /* best-effort: keep the history suggestions */
+    }
+    return detail;
+}
 export async function prRoutes(app) {
     // Bulk "mark all seen": stamp every open PR (optionally scoped to repoIds) viewed
     // at its head, clearing all new-since badges at once. Static path — no :id — so it
@@ -159,7 +237,10 @@ export async function prRoutes(app) {
         }
         // Cloud lean mode: fill in bulky text from GitHub (no-op in local). The client
         // caches the result in IndexedDB keyed by updatedAt so unchanged PRs don't refetch.
-        return hydratePrDetail(pr, accountId);
+        const detail = await hydratePrDetail(pr, accountId);
+        // Layer CODEOWNERS ownership onto the history-based suggestions (best-effort; never
+        // blocks the response). getPrDetail already gated the history part on the same trigger.
+        return enrichSuggestionsWithCodeowners(detail, accountId);
     });
     // Candidates for an @mention autocomplete, ranked by proximity to this PR
     // (participants first, then repo people), self + bots excluded. Account-scoped:
@@ -628,7 +709,7 @@ export async function prRoutes(app) {
     // next sync (reviewRequests are re-derived each pass).
     app.post('/api/prs/:id/request-reviewers', { schema: requestReviewersSchema }, async (req, reply) => {
         const { id } = req.params;
-        const { userIds } = req.body;
+        const { userIds = [], logins: directLogins = [], teamSlugs = [], } = req.body;
         const accountId = accountIdOf(req);
         const ctx = await getPrWriteContext(id, accountId);
         if (!ctx) {
@@ -643,20 +724,23 @@ export async function prRoutes(app) {
                 message: 'You need write access to this repo to request reviewers.',
             };
         }
-        // Drop the PR author (GitHub rejects self-review requests), then resolve to
-        // logins (also drops bots + unknown ids).
+        // Drop the PR author from the id set (GitHub rejects self-review requests), then
+        // resolve to logins (also drops bots + unknown ids). Union with any direct logins
+        // (suggested reviewers we haven't synced as users), deduped.
         const wanted = userIds.filter((uid) => uid !== ctx.authorId);
-        const logins = (await getReviewerLogins(wanted)).map((r) => r.login);
-        if (logins.length === 0) {
+        const resolvedLogins = (await getReviewerLogins(wanted)).map((r) => r.login);
+        const logins = [...new Set([...resolvedLogins, ...directLogins])];
+        const teams = [...new Set(teamSlugs)];
+        if (logins.length === 0 && teams.length === 0) {
             reply.status(400);
             return {
                 error: 'NoReviewers',
-                message: 'None of the selected users can be requested as reviewers.',
+                message: 'None of the selected users or teams can be requested as reviewers.',
             };
         }
         try {
             const token = await getAccessToken(accountId);
-            await requestReviewers(token, ctx.owner, ctx.name, ctx.number, logins);
+            await requestReviewers(token, ctx.owner, ctx.name, ctx.number, logins, teams);
             const result = {
                 status: 'ok',
                 requestedLogins: logins,

package/dist/db/queries.js

@@ -3329,6 +3329,108 @@ async function getThreadsAwaiting(localUserId, accountId, repoNameById) {
     out.sort((a, b) => b.lastReplyAt.localeCompare(a.lastReplyAt));
     return out;
 }
+// Recent-activity window for the CORE reviewer suggester (both the dir-overlap authorship
+// signal and the "reviews here often" pool). 90d = the board's max range; beyond it a
+// contributor is off the radar.
+const REVIEWER_SUGGEST_WINDOW_MS = 90 * 86_400_000;
+// Suggest reviewers for a PR that has NONE assigned, from ALREADY-SYNCED data only (no
+// GitHub calls). Candidate pool = people with merge rights in the repo ∪ people who review
+// it often; ranked by how much they've recently worked in the top-level dirs this PR
+// touches. The PR author + bots are excluded. Returns up to 3 user suggestions
+// (source 'history'). CODEOWNERS suggestions are layered on top in the route (they need a
+// token + network); this is the always-available core signal.
+async function suggestReviewersFromHistory(accountId, repoId, authorId, changedPaths) {
+    const since = new Date(Date.now() - REVIEWER_SUGGEST_WINDOW_MS);
+    const dirs = [...new Set(changedPaths.map(topLevelDir))];
+    // Repo-wide "who recently worked where" (author × top-level dir), from the always-stored
+    // pull_requests.files — real recent activity per area, no commit-file dependency.
+    const dirCount = new Map(); // dir -> uid -> #PRs
+    for (const pr of await db
+        .select({ authorId: pullRequests.authorId, files: pullRequests.files })
+        .from(pullRequests)
+        .where(and(eq(pullRequests.accountId, accountId), eq(pullRequests.repoId, repoId), isNotNull(pullRequests.authorId), isNotNull(pullRequests.files), gte(pullRequests.updatedAt, since)))
+        .execute()) {
+        if (pr.authorId == null || pr.files == null)
+            continue;
+        for (const d of new Set(pr.files.map((f) => topLevelDir(f.path)))) {
+            const m = dirCount.get(d) ?? new Map();
+            m.set(pr.authorId, (m.get(pr.authorId) ?? 0) + 1);
+            dirCount.set(d, m);
+        }
+    }
+    // Merge-rights set for this repo.
+    const repoMergers = new Set((await getMergers(accountId)).find((m) => m.repoId === repoId)?.userIds ?? []);
+    // Frequent reviewers of this repo (review count per author, recent window).
+    const reviewCount = new Map();
+    for (const r of await db
+        .select({ authorId: reviews.authorId })
+        .from(reviews)
+        .innerJoin(pullRequests, eq(pullRequests.id, reviews.prId))
+        .where(and(eq(pullRequests.accountId, accountId), eq(pullRequests.repoId, repoId), isNotNull(reviews.authorId), gte(reviews.submittedAt, since)))
+        .execute()) {
+        if (r.authorId == null)
+            continue;
+        reviewCount.set(r.authorId, (reviewCount.get(r.authorId) ?? 0) + 1);
+    }
+    // Candidate pool = mergers ∪ frequent reviewers, minus the author.
+    const pool = new Set([...repoMergers, ...reviewCount.keys()]);
+    if (authorId != null)
+        pool.delete(authorId);
+    if (pool.size === 0)
+        return [];
+    // Score by dir-overlap; track each candidate's most-touched matching dir for the reason.
+    const overlap = new Map();
+    const topDir = new Map();
+    for (const d of dirs) {
+        const m = dirCount.get(d);
+        if (!m)
+            continue;
+        for (const [uid, cnt] of m) {
+            if (!pool.has(uid))
+                continue;
+            overlap.set(uid, (overlap.get(uid) ?? 0) + cnt);
+            const cur = topDir.get(uid);
+            if (!cur || cnt > cur.cnt)
+                topDir.set(uid, { dir: d, cnt });
+        }
+    }
+    // Resolve to logins (drops bots + null logins), then rank.
+    const resolved = await getReviewerLogins([...pool]);
+    const dirLabel = (d) => (d === '.' ? 'the repo root' : `${d}/`);
+    const ranked = resolved
+        .sort((a, b) => {
+        const ov = (overlap.get(b.userId) ?? 0) - (overlap.get(a.userId) ?? 0);
+        if (ov !== 0)
+            return ov;
+        const rv = (reviewCount.get(b.userId) ?? 0) - (reviewCount.get(a.userId) ?? 0);
+        if (rv !== 0)
+            return rv;
+        return (repoMergers.has(b.userId) ? 1 : 0) - (repoMergers.has(a.userId) ? 1 : 0);
+    })
+        .slice(0, 3);
+    return ranked.map(({ userId, login }) => {
+        const top = topDir.get(userId);
+        const reason = top && (overlap.get(userId) ?? 0) > 0
+            ? `recently changed ${dirLabel(top.dir)}`
+            : (reviewCount.get(userId) ?? 0) > 0
+                ? 'reviews here often'
+                : 'has merge rights here';
+        return { kind: 'user', login, userId, teamSlug: null, teamName: null, reason, source: 'history' };
+    });
+}
+// Resolve GitHub logins to synced User rows (for CODEOWNERS suggestion enrichment in the
+// route — an @user owner may or may not be someone we've synced). Case-insensitive on
+// login is unnecessary: GitHub logins in CODEOWNERS match the stored githubLogin exactly.
+export async function getUsersByLogins(logins) {
+    if (logins.length === 0)
+        return [];
+    const rows = await db
+        .select()
+        .from(users)
+        .where(inArray(users.githubLogin, logins))
+        .execute();
+    return rows.map(mapUser);
+}
 export async function getPrDetail(id, accountId) {
     const rows = await db
         .select()
@@ -3449,6 +3551,17 @@ export async function getPrDetail(id, accountId) {
         userId: r.userId,
         teamName: r.teamName,
     }));
+    // Suggested reviewers (CORE) — only when the PR warrants them: open, non-draft, and
+    // nobody's been requested OR has reviewed yet. The route layers CODEOWNERS suggestions
+    // on top (they need a token + network); this is the history-based signal from synced
+    // data alone. Keep this trigger in lockstep with the route's CODEOWNERS gate.
+    const wantsSuggestions = pr.state === 'open' &&
+        !pr.isDraft &&
+        requestedReviewers.length === 0 &&
+        reviewsOut.length === 0;
+    const suggestedReviewers = wantsSuggestions
+        ? await suggestReviewersFromHistory(accountId, pr.repoId, pr.authorId, (pr.files ?? []).map((f) => f.path))
+        : [];
     // Gather referenced users for client-side lookup.
     const userIds = new Set();
     if (pr.authorId)
@@ -3474,6 +3587,9 @@ export async function getPrDetail(id, accountId) {
     for (const r of reviewerRows)
         if (r.userId)
             userIds.add(r.userId);
+    for (const s of suggestedReviewers)
+        if (s.userId != null)
+            userIds.add(s.userId);
     // A maintainer who only merged the PR (never authored/reviewed/commented) is
     // otherwise absent from userList, leaving "Merged by" unresolved.
     if (pr.mergedById)
@@ -3581,6 +3697,7 @@ export async function getPrDetail(id, accountId) {
         changedFilesCount: pr.changedFiles,
         files: filesOut,
         requestedReviewers,
+        suggestedReviewers,
         viewerCanApprove,
         viewerCanPush,
         viewerHasApprovedStanding,

package/dist/github/client.js

@@ -112,6 +112,22 @@ export async function ghRestGetText(token, path) {
     const text = await res.text().catch(() => '');
     return { status: res.status, ok: res.ok, text };
 }
+// REST GET a repo file's RAW contents (Accept: application/vnd.github.raw), NOT throwing
+// on a non-2xx — returns the status + body so the caller can branch. Used to fetch a
+// repo's CODEOWNERS file (404 when absent → degrade to no CODEOWNERS suggestions). The
+// raw media type returns the file bytes directly (no base64/JSON envelope to decode).
+export async function ghRestGetContentRaw(token, path) {
+    const res = await fetch(`https://api.github.com${path}`, {
+        method: 'GET',
+        headers: {
+            authorization: `token ${token}`,
+            accept: 'application/vnd.github.raw',
+            'x-github-api-version': '2022-11-28',
+        },
+    });
+    const text = await res.text().catch(() => '');
+    return { status: res.status, ok: res.ok, text };
+}
 // REST GET in GitHub's raw `diff` media type (Accept: application/vnd.github.diff),
 // NOT throwing on a non-2xx — returns the status + body so the caller can branch. GitHub
 // caps this media type at 20,000 lines and 406s past it (a huge PR), which the caller

package/dist/github/codeowners.js

@@ -0,0 +1,163 @@
+// CODEOWNERS-based reviewer suggestions (CORE). Fetches a repo's CODEOWNERS file via the
+// contents API, parses it, and — given a PR's changed paths — returns the owning users +
+// teams. Best-effort throughout: any failure (no file, token can't read the repo, a parse
+// hiccup) degrades to "no CODEOWNERS suggestions", NEVER an error on the PR-detail path.
+//
+// Why CODEOWNERS and not the org/teams API: the contents API is reachable in every auth
+// mode (local gh token, cloud OAuth `public_repo`, GitHub App with contents:read), and a
+// `@org/team` owner is directly requestable as a review via `team_reviewers` — no need to
+// expand team membership (which needs `read:org` + org membership and is unreliable under
+// the cloud GitHub App). Individual `@user` owners become user suggestions directly.
+import { ghRestGetContentRaw } from './client.js';
+// The candidate file locations GitHub honours, in the order it resolves them.
+const CODEOWNERS_PATHS = ['.github/CODEOWNERS', 'CODEOWNERS', 'docs/CODEOWNERS'];
+// Per-(account, repo) cache. CODEOWNERS changes rarely, so a short TTL keeps the extra
+// contents-API call off the hot path while staying reasonably fresh. A `null` rules value
+// is cached too (the common "repo has no CODEOWNERS" case) so we don't refetch a 404 on
+// every PR open. Keyed by account so one tenant's token result never leaks to another.
+const CACHE_TTL_MS = 60 * 60 * 1000; // 1 hour
+const cache = new Map();
+// Convert a CODEOWNERS (gitignore-style) path pattern into a regex over POSIX file paths.
+// A best-effort subset matching GitHub's documented CODEOWNERS semantics: `*` matches
+// within a single segment (NOT across `/`), `**` across segments, `?` one non-slash char;
+// a leading `/` or any internal `/` anchors to the repo root, otherwise the pattern may
+// match at any directory depth; a trailing `/` (or a bare directory name) owns everything
+// under that directory. Exported for the unit test that locks these semantics.
+export function globToRegExp(raw) {
+    let p = raw.trim();
+    const dirOnly = p.endsWith('/');
+    if (dirOnly)
+        p = p.slice(0, -1);
+    const startSlash = p.startsWith('/');
+    if (startSlash)
+        p = p.slice(1);
+    // Git: a pattern with a slash anywhere but the end is relative to the root (anchored);
+    // a pattern with no internal slash matches by name at any level.
+    const anchored = startSlash || p.includes('/');
+    let re = '';
+    for (let i = 0; i < p.length; i++) {
+        const c = p[i];
+        if (c === '*') {
+            if (p[i + 1] === '*') {
+                // `**` spans path segments. A segment-aligned `**/` becomes OPTIONAL leading
+                // directories that preserve the `/` boundary (so `**/logs` matches `a/logs` but
+                // NOT `mylogs`); a trailing/standalone `**` matches anything.
+                if (p[i + 2] === '/') {
+                    re += '(?:.*/)?';
+                    i += 2; // consume the second '*' and the '/'
+                }
+                else {
+                    re += '.*';
+                    i += 1; // consume the second '*'
+                }
+            }
+            else {
+                re += '[^/]*';
+            }
+        }
+        else if (c === '?') {
+            re += '[^/]';
+        }
+        else if ('.+^${}()|[]\\'.includes(c)) {
+            re += `\\${c}`;
+        }
+        else {
+            re += c;
+        }
+    }
+    // A trailing-slash directory (or a bare literal path) OWNS everything under it, so it
+    // gets a "…/.*" / "(?:/.*)?" tail. A wildcard-TERMINATED pattern (e.g. `docs/*`) is
+    // single-level per GitHub — it must NOT gain that tail, or it would over-match nested
+    // files (`docs/*` matches `docs/a.md` but not `docs/sub/a.md`).
+    const endsWithWildcard = /[*?]$/.test(p);
+    const body = dirOnly ? `${re}/.*` : endsWithWildcard ? re : `${re}(?:/.*)?`;
+    const full = anchored ? `^${body}$` : `^(?:.*/)?${body}$`;
+    return new RegExp(full);
+}
+// Parse CODEOWNERS text into ordered rules (order matters: the LAST matching rule wins).
+function parseCodeowners(text) {
+    const rules = [];
+    for (const line of text.split('\n')) {
+        const trimmed = line.trim();
+        if (!trimmed || trimmed.startsWith('#'))
+            continue;
+        // Strip an inline comment, then split on whitespace: first token = pattern, rest = owners.
+        const noComment = trimmed.split('#')[0].trim();
+        if (!noComment)
+            continue;
+        const parts = noComment.split(/\s+/);
+        const pattern = parts[0];
+        // Owners are the @-prefixed tokens (drop bare emails — not requestable as reviewers).
+        const owners = parts.slice(1).filter((o) => o.startsWith('@'));
+        if (owners.length === 0)
+            continue;
+        try {
+            rules.push({ re: globToRegExp(pattern), owners });
+        }
+        catch {
+            /* skip an un-compilable pattern rather than fail the whole file */
+        }
+    }
+    return rules;
+}
+// Fetch + parse a repo's CODEOWNERS (cached). Returns null when there's no file / it can't
+// be read. Never throws.
+async function loadRules(token, owner, name, accountId) {
+    const key = `${accountId}:${owner}/${name}`;
+    const hit = cache.get(key);
+    if (hit && Date.now() - hit.at < CACHE_TTL_MS)
+        return hit.rules;
+    let rules = null;
+    try {
+        for (const path of CODEOWNERS_PATHS) {
+            const res = await ghRestGetContentRaw(token, `/repos/${owner}/${name}/contents/${path}`);
+            if (res.ok && res.text) {
+                rules = parseCodeowners(res.text);
+                break;
+            }
+            // Any non-404 (403 org wall / 401) also means "give up on CODEOWNERS" — stop trying.
+            if (res.status !== 404)
+                break;
+        }
+    }
+    catch {
+        rules = null;
+    }
+    cache.set(key, { rules, at: Date.now() });
+    return rules;
+}
+// Given a PR's changed paths, return the CODEOWNERS-derived owners (users + teams). The
+// LAST matching rule for each file wins (git semantics); owners are unioned across files.
+export async function getCodeownersMatch(token, owner, name, accountId, changedPaths) {
+    const empty = { logins: [], teams: [] };
+    if (changedPaths.length === 0)
+        return empty;
+    const rules = await loadRules(token, owner, name, accountId);
+    if (!rules || rules.length === 0)
+        return empty;
+    const ownerTokens = new Set();
+    for (const path of changedPaths) {
+        let last = null;
+        for (const r of rules)
+            if (r.re.test(path))
+                last = r; // last match wins
+        if (last)
+            for (const o of last.owners)
+                ownerTokens.add(o);
+    }
+    const logins = [];
+    const teams = [];
+    for (const tok of ownerTokens) {
+        const handle = tok.slice(1); // drop '@'
+        if (handle.includes('/')) {
+            const slug = handle.slice(handle.indexOf('/') + 1);
+            if (slug)
+                teams.push({ slug, name: handle });
+        }
+        else if (handle) {
+            logins.push(handle);
+        }
+    }
+    return { logins, teams };
+}
+//# sourceMappingURL=codeowners.js.map

package/dist/github/mutations.js

@@ -336,12 +336,17 @@ export async function createPullRequest(token, args) {
 }
 // ---- Request reviewers on a PR (REST) ----
 // Request one or more reviewers on a PR (POST .../pulls/:n/requested_reviewers with
-// { reviewers: [login…] }). Needs a token with write/triage access; GitHub 422s if a
-// login isn't a collaborator or is the PR author (the caller filters the author out).
+// { reviewers: [login…], team_reviewers: [slug…] }). Needs a token with write/triage
+// access; GitHub 422s if a login isn't a collaborator or is the PR author (the caller
+// filters the author out). `teamSlugs` requests a whole team (a CODEOWNERS `@org/team`)
+// WITHOUT expanding its membership — GitHub resolves the slug against the repo's org.
 // Returns 201 with the updated PR body, which we don't need — the refreshed request
 // state arrives on the next sync (reviewRequests are re-derived each sync).
-export async function requestReviewers(token, owner, name, number, logins) {
-    await ghRestPostFor(token, `/repos/${owner}/${name}/pulls/${number}/requested_reviewers`, { reviewers: logins });
+export async function requestReviewers(token, owner, name, number, logins, teamSlugs = []) {
+    const body = { reviewers: logins };
+    if (teamSlugs.length > 0)
+        body.team_reviewers = teamSlugs;
+    await ghRestPostFor(token, `/repos/${owner}/${name}/pulls/${number}/requested_reviewers`, body);
 }
 // ---- Re-trigger a GitHub Actions workflow run ----
 // Re-run a workflow run (per-account). `mode: 'failed'` reruns only the failed jobs

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pierre-review",
-  "version": "0.1.67",
+  "version": "0.1.69",
   "description": "Dashboard for tracking your team's GitHub PR activity across repos — local (SQLite + gh) or self-hosted multi-tenant cloud (Postgres + GitHub App).",
   "type": "module",
   "author": "Alex Wakeman",

package/public/assets/AiFixTab-B1BGyJ-q.js

@@ -0,0 +1,3 @@
+import{r as o,j as e}from"./tanstack-R-s9abdD.js";import{D as te,w as W,A as re,c as ne,d as ae,q as ie,G as ce,I as J,M as K,i as le,l as oe,k as de,C as ue,b as xe,R as Q,a as he,p as X,F as Z,r as I,E as ge,J as me,n as be,m as pe,y as fe,j as Y}from"./index-DNiwLu1C.js";import"./markdown-BUNN1sk6.js";import"./highlight-C8vPehS1.js";import"./vis-C3BCkP5U.js";const S="whitespace-nowrap rounded border border-blue-400 px-2.5 py-1 text-xs text-blue-600 hover:bg-blue-50 disabled:opacity-50 dark:border-blue-600 dark:text-blue-400 dark:hover:bg-blue-900/30",j="whitespace-nowrap rounded border border-gray-300 px-2.5 py-1 text-xs hover:border-gray-400 disabled:opacity-50 dark:border-gray-700 dark:hover:border-gray-500",ye={fetching_diff:"Reading the PR",cloning:"Checking out the code",fixing:"Applying the fix",capturing:"Capturing changes",persisting:"Saving"};function je(s){var t;if(!s||s.status==="idle")return null;if(s.status==="queued")return 6;const r=s.progress;if(!r)return 10;switch(r.phase){case"fetching_diff":return 12;case"cloning":return 28;case"fixing":return Math.min(90,45+(((t=r.recentActivity)==null?void 0:t.length)??0)*3);case"capturing":return 92;case"persisting":return 96;default:return 20}}const ke={cloning:"Checking out the code",applying_fix:"Applying the fix",fetching_trunk:"Fetching the trunk",rebasing:"Rebasing onto the trunk",merging:"Merging the trunk in",resolving_conflicts:"Resolving conflicts with Claude",verifying:"Verifying the result",pushing:"Pushing"};function Ne(s){var t;if(!s||s.status==="idle")return null;if(s.status==="queued")return 6;const r=s.progress;if(!r)return 10;switch(r.phase){case"cloning":return 15;case"applying_fix":return 25;case"fetching_trunk":return 35;case"rebasing":case"merging":return 45;case"resolving_conflicts":return Math.min(90,55+(((t=r.recentActivity)==null?void 0:t.length)??0)*3);case"verifying":return 92;case"pushing":return 96;default:return 20}}function L({children:s}){return e.jsx("div",{className:"px-4 pb-1 pt-3 text-xs font-semibold uppercase tracking-wide text-gray-400",children:s})}const ve={high:"bg-green-100 text-green-700 dark:bg-green-950/40 dark:text-green-300",medium:"bg-amber-100 text-amber-700 dark:bg-amber-950/40 dark:text-amber-300",low:"bg-red-100 text-red-700 dark:bg-red-950/40 dark:text-red-300"};function z({label:s,value:r}){return r?e.jsxs("span",{className:`rounded px-1.5 py-0.5 text-[10px] font-medium uppercase ${ve[r]}`,title:s==="Fixability"?"How confident the analysis is that Pierre's agent could fix this":"How confident the analysis is about the root cause",children:[s,": ",r]}):null}function De({pr:s}){const{aiAnalysis:r,aiFix:t}=te(),n=W(x=>x.aiFixTabFocus),i=W(x=>x.consumeAiFixTabFocus),[c,m]=o.useState(null);return o.useEffect(()=>{n&&n.prId===s.id&&(n.reviewText&&m(n.reviewText),i())},[n,s.id,i]),!r&&!t?e.jsx("div",{className:"p-4 text-sm text-gray-500",children:"AI Analysis and Fix is not enabled."}):e.jsxs("div",{className:"pb-6",children:[r&&e.jsx("div",{className:"border-b border-gray-200 dark:border-gray-800",children:e.jsx(re,{pr:s})}),e.jsx(Ce,{pr:s}),r&&e.jsx(Pe,{pr:s,canFix:t}),t&&e.jsx(Re,{pr:s,seedReviewText:c,onSeedConsumed:()=>m(null)})]})}function Ce({pr:s}){const r=s.checkRuns;return r.length===0?null:e.jsxs("div",{className:"border-b border-gray-200 dark:border-gray-800",children:[e.jsx(L,{children:"CI status"}),e.jsxs("div",{className:"px-4 pb-3",children:[e.jsx(ne,{prId:s.id,checks:r}),e.jsx(ae,{prId:s.id,checks:r,viewerCanPush:s.viewerCanPush})]})]})}function Pe({pr:s,canFix:r}){const{data:t}=ie(s.id,!0),n=ce(s.id),i=J(s.id),c=o.useMemo(()=>s.checkRuns.filter(l=>l.state==="failure"||l.state==="error").map(l=>({name:l.name,jobId:l.jobId,state:l.state})),[s.checkRuns]);if(!(c.length>0||((t==null?void 0:t.hasFailures)??!1))&&!(t!=null&&t.analysis))return null;const x=(t==null?void 0:t.fixability)==="low";return e.jsxs("div",{className:"border-b border-gray-200 dark:border-gray-800",children:[e.jsx(L,{children:"CI failure analysis"}),e.jsxs("div",{className:"px-4 pb-3",children:[(t==null?void 0:t.analysis)&&(t.rootCauseConfidence||t.fixability)&&e.jsxs("div",{className:"mb-1.5 flex flex-wrap items-center gap-1.5",children:[e.jsx("span",{className:"text-[10px] font-semibold uppercase tracking-wide text-gray-400",title:"How confident the analysis is — not a severity rating",children:"Confidence"}),e.jsx(z,{label:"Root cause",value:t.rootCauseConfidence}),e.jsx(z,{label:"Fixability",value:t.fixability})]}),t!=null&&t.analysis?e.jsx("div",{className:"prose prose-sm max-w-none dark:prose-invert",children:e.jsx(K,{children:t.analysis})}):e.jsx("p",{className:"text-sm text-gray-500",children:"Diagnose the failing CI: likely root cause + a suggested fix, from the full logs of every failing check."}),e.jsxs("div",{className:"mt-2 flex flex-wrap items-center gap-2",children:[e.jsx("button",{type:"button",className:j,disabled:n.isPending||c.length===0,onClick:()=>n.mutate(c),children:n.isPending?"Analyzing…":t!=null&&t.analysis?"Re-analyze":"Analyze CI failure"}),r&&(t==null?void 0:t.analysis)&&e.jsx("button",{type:"button",className:S,disabled:i.isPending,onClick:()=>i.mutate({model:"claude-sonnet-5",seed:"ci_analysis"}),title:"Launch an agent to fix the CI failure",children:"Fix it →"}),r&&x&&(t==null?void 0:t.analysis)&&e.jsx("span",{className:"text-[11px] text-gray-500",children:"low confidence this is auto-fixable"}),n.isError&&e.jsx("span",{className:"text-[11px] text-red-500",children:D(n.error)})]})]})]})}function Re({pr:s,seedReviewText:r,onSeedConsumed:t}){var k,F,N,A;const{data:n,isLoading:i}=le(s.id,!0),[c,m]=o.useState("claude-sonnet-5"),x=J(s.id),l=oe(s.id),p=((k=n==null?void 0:n.fix)==null?void 0:k.status)??null,h=p==="running"||p==="queued"||x.isPending,{status:d}=de(s.id,h),R=(d==null?void 0:d.status)??p??"idle",f=R==="running"||R==="queued",w=r?"review":"plain",M=()=>{x.mutate({model:c,seed:w,reviewText:r??void 0},{onSuccess:()=>t()})},u=(n==null?void 0:n.fix)??null;return e.jsxs("div",{children:[e.jsx(L,{children:"AI Fix"}),e.jsxs("div",{className:"px-4",children:[(n==null?void 0:n.enabled)===!1?e.jsx("p",{className:"text-sm text-gray-500",children:"The agentic fixer is disabled."}):(n==null?void 0:n.auth)==="none"?e.jsx("p",{className:"rounded border border-amber-300 bg-amber-50 p-2 text-xs text-amber-700 dark:border-amber-700 dark:bg-amber-950/30 dark:text-amber-300",children:n.authMessage??"No Claude authentication found. Sign in to Claude or set an API key, then restart."}):e.jsxs(e.Fragment,{children:[r&&!f&&e.jsx("div",{className:"mb-2 rounded border border-blue-200 bg-blue-50 p-2 text-xs text-blue-700 dark:border-blue-800 dark:bg-blue-950/30 dark:text-blue-300",children:"Ready to generate a fix from the selected review."}),e.jsxs("div",{className:"flex flex-wrap items-center gap-2",children:[e.jsx("select",{className:"rounded border border-gray-300 bg-transparent px-2 py-1 text-xs dark:border-gray-700",value:c,onChange:y=>m(y.target.value),disabled:f,children:ue.map(y=>e.jsx("option",{value:y,children:xe[y]},y))}),f?e.jsx("button",{type:"button",className:j,onClick:()=>l.mutate(),children:"Cancel"}):e.jsx("button",{type:"button",className:S,disabled:x.isPending,onClick:M,children:u?"Generate new fix":"Generate fix"}),x.isError&&e.jsx("span",{className:"text-[11px] text-red-500",children:D(x.error)})]}),f&&e.jsxs("div",{className:"mt-3",children:[e.jsx(Q,{active:!0,label:"Running AI fix",value:je(d),timeConstantSec:40}),e.jsx("div",{className:"mt-1 text-[11px] text-gray-500",children:ye[((F=d==null?void 0:d.progress)==null?void 0:F.phase)??""]??"Working…"}),((N=d==null?void 0:d.progress)==null?void 0:N.recentActivity)&&d.progress.recentActivity.length>0&&e.jsx("pre",{className:"mt-1 max-h-32 overflow-auto rounded bg-gray-50 p-2 text-[11px] leading-relaxed text-gray-500 dark:bg-gray-900",children:d.progress.recentActivity.slice(-8).join(`
+`)})]}),!f&&u&&e.jsx(we,{pr:s,fix:u,headInfo:(n==null?void 0:n.headInfo)??null,viewerCanPush:(n==null?void 0:n.viewerCanPush)??!1}),i&&!u&&e.jsx("p",{className:"mt-3 text-xs text-gray-400",children:"Loading…"})]}),e.jsx(Fe,{history:(n==null?void 0:n.history)??[],currentFixId:((A=n==null?void 0:n.fix)==null?void 0:A.id)??null})]})]})}function we({pr:s,fix:r,headInfo:t,viewerCanPush:n}){const i=o.useMemo(()=>X(r.patch),[r.patch]);if(r.status==="failed")return e.jsxs("div",{className:"mt-3 rounded border border-red-200 bg-red-50 p-2 text-xs text-red-700 dark:border-red-800 dark:bg-red-950/30 dark:text-red-300",children:["The fix run failed: ",r.error??"unknown error"]});if(r.status==="cancelled")return e.jsx("p",{className:"mt-3 text-xs text-gray-500",children:"The fix run was cancelled."});if(r.status!=="succeeded")return e.jsx(e.Fragment,{});const c=!r.patch||r.filesChanged.length===0;return e.jsxs("div",{className:"mt-3",children:[r.summary&&e.jsx("div",{className:"prose prose-sm max-w-none dark:prose-invert",children:e.jsx(K,{children:r.summary})}),c?e.jsx("p",{className:"mt-2 text-xs text-gray-500",children:"The agent made no changes."}):e.jsxs(e.Fragment,{children:[e.jsxs("div",{className:"mb-1 mt-2 text-[11px] text-gray-500",children:[r.filesChanged.length," file",r.filesChanged.length===1?"":"s"," changed"]}),e.jsx("div",{className:"overflow-hidden rounded border border-gray-200 text-gray-800 dark:border-gray-800 dark:text-gray-200",children:e.jsx(Z,{files:i})}),e.jsx(Se,{pr:s,fix:r,headInfo:t,viewerCanPush:n})]})]})}function _({fix:s}){return e.jsxs("div",{className:"mb-2 rounded border border-green-200 bg-green-50 p-2 text-xs text-green-700 dark:border-green-800 dark:bg-green-950/30 dark:text-green-300",children:[e.jsxs("div",{children:["Pushed to ",e.jsx("span",{className:"font-mono",children:s.pushedBranch}),s.pushedPrUrl&&e.jsxs(e.Fragment,{children:[" · ",e.jsxs("a",{href:s.pushedPrUrl,target:"_blank",rel:"noreferrer",className:"underline",children:["PR #",s.pushedPrNumber]})]}),s.pushedAt&&e.jsxs(e.Fragment,{children:[" · ",I(s.pushedAt)]})]}),s.commitMessage&&e.jsx("div",{className:"mt-1 font-mono text-[11px] text-green-800 dark:text-green-300",children:s.commitMessage})]})}function Fe({history:s,currentFixId:r}){const t=s.filter(n=>n.pushedAt!=null&&n.id!==r);return t.length===0?null:e.jsxs("div",{className:"mt-4 border-t border-gray-200 pt-3 dark:border-gray-800",children:[e.jsx("div",{className:"mb-1.5 text-xs font-semibold uppercase tracking-wide text-gray-400",children:"Fixes pushed via Pierre"}),e.jsx("ul",{className:"space-y-2",children:t.map(n=>e.jsxs("li",{className:"text-xs",children:[e.jsx("div",{className:"font-mono text-gray-700 dark:text-gray-200",children:n.commitMessage??"(no commit message)"}),e.jsxs("div",{className:"mt-0.5 text-[11px] text-gray-400",children:[n.pushedBranch&&e.jsx("span",{className:"font-mono",children:n.pushedBranch}),n.pushedPrUrl&&e.jsxs(e.Fragment,{children:[" · ",e.jsxs("a",{href:n.pushedPrUrl,target:"_blank",rel:"noreferrer",className:"underline",children:["PR #",n.pushedPrNumber]})]}),n.pushedAt&&e.jsxs(e.Fragment,{children:[" · ",I(n.pushedAt)]})]})]},n.id))})]})}function Ae({preview:s,loading:r}){if(r)return e.jsx("p",{className:"text-[11px] text-gray-400",children:"Checking against the trunk…"});if(!s||!s.available||!s.trunkSha)return null;if(!s.clean){const t=s.conflictFiles;return e.jsxs("div",{className:"rounded border border-amber-300 bg-amber-50 p-2 text-[11px] text-amber-700 dark:border-amber-700 dark:bg-amber-950/30 dark:text-amber-300",children:["⚠ Conflicts with ",e.jsx("span",{className:"font-mono",children:s.trunk}),t.length>0?e.jsxs(e.Fragment,{children:[" in ",t.length," file",t.length===1?"":"s",e.jsxs("span",{className:"text-amber-600/90 dark:text-amber-400/90",children:[": ",t.slice(0,6).join(", "),t.length>6?"…":""]})]}):null,". Resolving before you push avoids a conflicted PR."]})}return s.behindBy>0?e.jsxs("p",{className:"text-[11px] text-gray-500",children:[e.jsx("span",{className:"font-mono",children:s.trunk})," is ",s.behindBy," ","commit",s.behindBy===1?"":"s"," ahead — the fix merges cleanly."]}):e.jsxs("p",{className:"text-[11px] text-green-600 dark:text-green-400",children:["Up to date with ",e.jsx("span",{className:"font-mono",children:s.trunk})," — no conflicts."]})}function Ee({resolved:s,target:r,pushing:t,onPush:n,onRedo:i,disabled:c}){const m=o.useMemo(()=>X(s.diff),[s.diff]);return e.jsxs("div",{className:"mt-2 space-y-2",children:[e.jsxs("div",{className:"rounded border border-green-200 bg-green-50 p-2 text-[11px] text-green-700 dark:border-green-800 dark:bg-green-950/30 dark:text-green-300",children:["Rebased onto ",e.jsx("span",{className:"font-mono",children:s.trunk}),".",s.resolvedConflicts?` Claude resolved conflicts in ${s.conflictFiles.length} file${s.conflictFiles.length===1?"":"s"}${s.conflictFiles.length?`: ${s.conflictFiles.join(", ")}`:""}.`:" No conflicts."," ","Review the result below, then push."]}),e.jsxs("div",{className:"text-[11px] text-gray-500",children:[s.filesChanged.length," file",s.filesChanged.length===1?"":"s"," in the rebased result"]}),e.jsx("div",{className:"overflow-hidden rounded border border-gray-200 text-gray-800 dark:border-gray-800 dark:text-gray-200",children:e.jsx(Z,{files:m})}),e.jsxs("div",{className:"flex flex-wrap items-center gap-2",children:[e.jsx("button",{type:"button",className:S,disabled:t||c,onClick:n,children:t?"Pushing…":r==="new"?"Push rebased + open PR":"Push rebased (force-with-lease)"}),e.jsx("button",{type:"button",className:j,disabled:t,onClick:i,children:"Re-rebase"})]})]})}function Se({pr:s,fix:r,headInfo:t,viewerCanPush:n}){var V,G;const i=ge(s.id),c=me(),m=be(),x=pe(),l=fe(),p=(t==null?void 0:t.canPushSameBranch)??!1,[h,d]=o.useState(p?"existing":"new"),[R,f]=o.useState((t==null?void 0:t.suggestedBranch)??""),[w,M]=o.useState(!0),[u,k]=o.useState("idle"),[F,N]=o.useState(null),A=o.useRef(!1),y=o.useRef(!1);o.useEffect(()=>{!A.current&&(t!=null&&t.suggestedBranch)&&(f(t.suggestedBranch),A.current=!0,t.canPushSameBranch||d("new"))},[t]);const g=r.pushedAt!=null,q=g&&r.pushedPrNumber==null,U=!g||q;o.useEffect(()=>{!y.current&&n&&U&&(y.current=!0,l.mutate(r.id))},[n,U,r.id]);const a=l.data??null,{status:v}=Y(s.id,r.id,"rebase",u==="rebasing"),{status:C}=Y(s.id,r.id,"push",u==="pushing");if(o.useEffect(()=>{u==="rebasing"&&v&&v.status!=="running"&&v.status!=="queued"&&(k("idle"),v.error&&N(v.error))},[u,v]),o.useEffect(()=>{u==="pushing"&&C&&C.status!=="running"&&C.status!=="queued"&&(k("idle"),C.error&&N(C.error))},[u,C]),!n)return g?e.jsx(_,{fix:r}):e.jsx("p",{className:"mt-2 text-xs text-gray-500",children:"You need write access to this repository to push this fix."});if(g&&!q)return e.jsx(_,{fix:r});const B=h==="new"&&R.trim().length===0,P=u!=="idle"||i.isPending||c.isPending,H=r.model,ee=g&&!!a&&a.available&&a.clean&&a.behindBy===0,T=E=>{N(null),i.mutate({fixId:r.id,body:{target:h,branch:h==="new"?R.trim():void 0,strategy:E,autoResolve:w,model:H}},{onSuccess:se=>{"pushedBranch"in se||k("pushing")}})},O=()=>{N(null),c.mutate({fixId:r.id,body:{autoResolve:w,model:H}},{onSuccess:()=>k("rebasing")})},b=u==="rebasing"?v:C,$=r.resolved;return e.jsxs("div",{className:"mt-3 rounded border border-gray-200 p-2 dark:border-gray-800",children:[g&&e.jsx(_,{fix:r}),e.jsx("div",{className:"mb-2 text-xs font-semibold text-gray-600 dark:text-gray-300",children:g?"Reconcile with the trunk":"Push this fix"}),!g&&r.commitMessage&&e.jsxs("div",{className:"mb-2 rounded bg-gray-50 p-2 text-[11px] dark:bg-gray-900",children:[e.jsx("span",{className:"uppercase tracking-wide text-gray-400",children:"Commit message"}),e.jsx("div",{className:"mt-0.5 font-mono text-gray-700 dark:text-gray-200",children:r.commitMessage})]}),e.jsxs("label",{className:`flex items-center gap-2 text-xs ${p?"":"opacity-40"}`,title:p?void 0:"The PR head is a fork you cannot push to — use a new branch",children:[e.jsx("input",{type:"radio",checked:h==="existing",disabled:!p||P,onChange:()=>d("existing")}),"Push to the PR branch",(t==null?void 0:t.headRef)&&e.jsxs("span",{className:"font-mono text-gray-400",children:["(",t.headRef,")"]})]}),e.jsxs("label",{className:"mt-1 flex items-center gap-2 text-xs",children:[e.jsx("input",{type:"radio",checked:h==="new",disabled:P,onChange:()=>d("new")}),"New branch + open a PR"]}),h==="new"&&e.jsx("input",{type:"text",className:"mt-1 w-full rounded border border-gray-300 bg-transparent px-2 py-1 font-mono text-xs dark:border-gray-700",value:R,disabled:P,onChange:E=>f(E.target.value),placeholder:"branch-name"}),e.jsxs("div",{className:"mt-2 flex flex-wrap items-center gap-2",children:[e.jsx("div",{className:"min-w-0 flex-1",children:e.jsx(Ae,{preview:a,loading:l.isPending})}),e.jsx("button",{type:"button",className:j,disabled:l.isPending||P,onClick:()=>l.mutate(r.id),title:"Re-check this fix against the current trunk",children:l.isPending?"Checking…":"Re-check trunk"}),l.isError&&!l.isPending&&e.jsx("span",{className:"text-[11px] text-red-500",children:"Couldn't check the trunk — try again."})]}),u!=="idle"?e.jsxs("div",{className:"mt-2",children:[e.jsx(Q,{active:!0,label:u==="rebasing"?"Rebasing & resolving":"Pushing",value:Ne(b),timeConstantSec:40}),e.jsxs("div",{className:"mt-1 flex items-center gap-2 text-[11px] text-gray-500",children:[e.jsx("span",{children:ke[((V=b==null?void 0:b.progress)==null?void 0:V.phase)??""]??"Working…"}),e.jsx("button",{type:"button",className:j,onClick:()=>u==="rebasing"?m.mutate(r.id):x.mutate(r.id),children:"Cancel"})]}),((G=b==null?void 0:b.progress)==null?void 0:G.recentActivity)&&b.progress.recentActivity.length>0&&e.jsx("pre",{className:"mt-1 max-h-32 overflow-auto rounded bg-gray-50 p-2 text-[11px] leading-relaxed text-gray-500 dark:bg-gray-900",children:b.progress.recentActivity.slice(-8).join(`
+`)})]}):$?e.jsx(Ee,{resolved:$,target:h,pushing:i.isPending,disabled:B,onPush:()=>T("rebase"),onRedo:O}):ee?e.jsxs("p",{className:"mt-2 text-[11px] text-gray-500",children:["No trunk changes to reconcile — the pushed fix is up to date with"," ",e.jsx("span",{className:"font-mono",children:a==null?void 0:a.trunk}),"."]}):e.jsxs("div",{className:"mt-2 space-y-2",children:[a&&a.available&&!a.clean&&e.jsxs("div",{className:"text-[11px] text-gray-500",children:["Recommended: rebase onto"," ",e.jsx("span",{className:"font-mono",children:a.trunk})," — Claude resolves the conflicts and you review the result before it pushes."]}),e.jsxs("label",{className:"flex items-center gap-2 text-[11px] text-gray-600 dark:text-gray-300",children:[e.jsx("input",{type:"checkbox",checked:w,onChange:E=>M(E.target.checked)}),"Let Claude resolve conflicts"]}),e.jsxs("div",{className:"flex flex-wrap items-center gap-2",children:[e.jsxs("button",{type:"button",className:a&&!a.clean?S:j,disabled:P||B,onClick:O,title:"Rebase the fix onto the trunk; Claude resolves conflicts and you review before a force-with-lease push",children:["Rebase onto ",(a==null?void 0:a.trunk)??"trunk"]}),!g&&e.jsxs("button",{type:"button",className:j,disabled:P||B,onClick:()=>T("merge"),title:"Merge the trunk into the fix branch (a merge commit; never force-pushes)",children:["Merge ",(a==null?void 0:a.trunk)??"trunk"," in"]}),!g&&e.jsxs("button",{type:"button",className:!a||a.clean?S:j,disabled:P||B,onClick:()=>T("plain"),title:"Push the fix as-is (never force-pushes). If it conflicts with the trunk, the PR will show as conflicted.",children:[h==="new"?"Push + open PR":"Push",a&&!a.clean?" anyway":""]})]}),g&&e.jsxs("p",{className:"text-[11px] text-gray-400",children:["Rebasing replays the fix onto"," ",e.jsx("span",{className:"font-mono",children:(a==null?void 0:a.trunk)??"the trunk"})," ","and force-pushes (with lease) onto the PR branch — the safe way to reconcile an already-pushed fix."]}),h==="existing"&&e.jsxs("p",{className:"text-[11px] text-gray-400",children:["Rebase force-pushes (with lease) onto"," ",e.jsx("span",{className:"font-mono",children:t==null?void 0:t.headRef}),"; merge and push do not."]})]}),(F||i.isError||c.isError)&&e.jsx("div",{className:"mt-2 text-[11px] text-red-500",children:F??D(i.error??c.error)})]})}function D(s){return s instanceof he||s instanceof Error?s.message:"Something went wrong"}export{De as AiFixTab};