pierre-review 0.1.22 → 0.1.24

package/dist/api/routes/claude-review.js

@@ -154,8 +154,8 @@ export async function claudeReviewRoutes(app) {
             return {
                 error: 'Conflict',
                 message: result.reason === 'already_running'
-                    ? 'A review is already running for this PR.'
-                    : 'Another review is in progress; try again shortly.',
+                    ? 'A review is already running or queued for this PR.'
+                    : 'The review queue is full; try again once some finish.',
             };
         }
         reply.status(202);

package/dist/api/routes/insights.js

@@ -0,0 +1,35 @@
+import { getInsights, getRepoAnalytics } from '../../db/queries.js';
+import { accountIdOf } from '../plugins/auth.js';
+function parseIntList(raw) {
+    if (!raw)
+        return null;
+    const ids = raw
+        .split(',')
+        .map((s) => Number.parseInt(s.trim(), 10))
+        .filter((n) => Number.isFinite(n));
+    return ids.length > 0 ? ids : null;
+}
+export async function insightsRoutes(app) {
+    // Per-repo sprint/team stats for the Insights panel. Scoped to the account;
+    // `repoIds` narrows to the active watched-repo selection.
+    app.get('/api/insights', async (req) => {
+        const q = req.query;
+        return getInsights({
+            accountId: accountIdOf(req),
+            repoIds: parseIntList(q.repoIds),
+        });
+    });
+    // Heavier per-repo analytics for the drill-down chart panel — loaded on demand.
+    // Ownership-scoped: a repo not owned by the account 404s.
+    app.get('/api/insights/:repoId/analytics', async (req, reply) => {
+        const { repoId } = req.params;
+        const id = Number.parseInt(repoId, 10);
+        const data = Number.isFinite(id)
+            ? await getRepoAnalytics(accountIdOf(req), id)
+            : null;
+        if (!data)
+            return reply.code(404).send({ error: 'repo not found' });
+        return data;
+    });
+}
+//# sourceMappingURL=insights.js.map

package/dist/api/routes/me.js

@@ -8,7 +8,7 @@ const dismissSchema = {
         required: ['kind', 'refId'],
         additionalProperties: false,
         properties: {
-            kind: { type: 'string', enum: ['review_request', 'thread'] },
+            kind: { type: 'string', enum: ['review_request', 'thread', 'claude_review'] },
             refId: { type: 'integer' },
         },
     },
@@ -23,6 +23,7 @@ export async function meRoutes(app) {
                 awaitingReview: myTurn.awaitingReview.length,
                 yourPrsActivity: myTurn.yourPrs.length,
                 threadsAwaiting: myTurn.threadsAwaiting.length,
+                claudeReviewsToAction: myTurn.claudeReviewsToAction.length,
             },
             claudeReviewEnabled: config.claudeReviewEnabled,
             deploymentMode: config.deploymentMode,
@@ -34,7 +35,8 @@ export async function meRoutes(app) {
         await dismissMyTurn(accountIdOf(req), kind, refId);
         return { status: 'ok' };
     });
-    // The "Done" tab: entries dismissed in the past 90 days (review_request + thread).
+    // The "Done" tab: entries dismissed in the past 90 days (review_request + thread
+    // + claude_review).
     app.get('/api/my-turn/done', async (req) => getCompletedDismissals(accountIdOf(req), 90));
     // Un-dismiss: move a completed entry back to the inbox.
     app.post('/api/my-turn/undismiss', { schema: dismissSchema }, async (req) => {

package/dist/api/routes/prs.js

@@ -1,4 +1,4 @@
-import { getPrDetail, markPrViewed } from '../../db/queries.js';
+import { getPrDetail, markAllViewed, markPrViewed } from '../../db/queries.js';
 import { hydratePrDetail } from '../../sync/hydrate-detail.js';
 import { accountIdOf } from '../plugins/auth.js';
 const idParamSchema = {
@@ -16,7 +16,22 @@ const markViewedSchema = {
         properties: { sha: { type: 'string' } },
     },
 };
+const markAllViewedSchema = {
+    body: {
+        type: 'object',
+        additionalProperties: false,
+        properties: { repoIds: { type: 'array', items: { type: 'integer' } } },
+    },
+};
 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
+    // doesn't collide with /api/prs/:id.
+    app.post('/api/prs/mark-all-viewed', { schema: markAllViewedSchema }, async (req) => {
+        const { repoIds } = (req.body ?? {});
+        const count = await markAllViewed(accountIdOf(req), repoIds && repoIds.length > 0 ? repoIds : null);
+        return { status: 'ok', count };
+    });
     app.get('/api/prs/:id', { schema: idParamSchema }, async (req, reply) => {
         const { id } = req.params;
         const accountId = accountIdOf(req);

package/dist/app.js

@@ -16,6 +16,7 @@ import { threadRoutes } from './api/routes/threads.js';
 import { meRoutes } from './api/routes/me.js';
 import { openPrsRoutes } from './api/routes/open-prs.js';
 import { mergersRoutes } from './api/routes/mergers.js';
+import { insightsRoutes } from './api/routes/insights.js';
 import { claudeReviewRoutes } from './api/routes/claude-review.js';
 export async function buildApp() {
     const app = Fastify({
@@ -126,6 +127,7 @@ export async function buildApp() {
     await app.register(meRoutes);
     await app.register(openPrsRoutes);
     await app.register(mergersRoutes);
+    await app.register(insightsRoutes);
     // Claude Review is local-only + opt-in. Only register its routes when enabled,
     // so the clone-manager / gh-CLI dependency is unreachable in cloud mode.
     if (config.claudeReviewEnabled)

package/dist/config.js

@@ -126,8 +126,15 @@ export const config = {
     // Turn cap for a diff-only run. These are TOOL-LESS (only submit_review), so they
     // should finish in ~2 turns; a tight cap is a cheap runaway guard.
     reviewDiffOnlyMaxTurns: intFromEnv('REVIEW_DIFF_ONLY_MAX_TURNS', 6),
-    // At most one review per PR; this caps concurrent reviews across all PRs.
-    reviewConcurrency: intFromEnv('REVIEW_CONCURRENCY', 1),
+    // At most one review per PR; this caps concurrent reviews across all PRs. Default
+    // 4 so the user can bulk-review (extras queue, see review-manager). Raising this
+    // also DISABLES the pasted-key override (which mutates process.env and is only
+    // safe at concurrency 1 — see local-settings.applyUserAnthropicKey); ambient auth
+    // is used instead. Set REVIEW_CONCURRENCY=1 to restore the pasted-key path.
+    reviewConcurrency: intFromEnv('REVIEW_CONCURRENCY', 4),
+    // Hard ceiling on QUEUED (not-yet-started) reviews, a runaway guard for bulk
+    // triggering; further starts return 'busy' until the queue drains.
+    reviewMaxQueued: intFromEnv('REVIEW_MAX_QUEUED', 50),
     // ---- Claude Review routing (diff-only vs worktree) — THE THRESHOLDS ----
     // The deterministic pre-check (review/routing.ts) decides, BEFORE the agent runs,
     // whether a PR can be reviewed from its diff alone (fast, tool-less, no worktree)