pierre-review 0.1.88 → 0.1.90

package/dist/api/routes/bot-triage.js

@@ -1,4 +1,4 @@
-import { addBotMuteRule, deleteBotMuteRule, getBotAnalytics, getBotBehaviourAnalytics, getBotOnlyPrs, getBotVendorPrs, getBotDedupClusters, getResolvableBotThreadPrs, getResolvableBotThreadsForScope, listBotMuteRules, listDetectedReviewers, resolveScopeRepoIds, setReviewerOverride, SCOPE_RESOLVE_THREAD_CAP, } from '../../db/queries.js';
+import { getBotAnalytics, getBotBehaviourAnalytics, getBotOnlyPrs, getBotVendorPrs, getBotDedupClusters, getResolvableBotThreadPrs, getResolvableBotThreadsForScope, listDetectedReviewers, resolveScopeRepoIds, setReviewerOverride, SCOPE_RESOLVE_THREAD_CAP, } from '../../db/queries.js';
 import { resolveThreadsOnGitHub } from '../../bot-triage/resolve.js';
 import { accountIdOf } from '../plugins/auth.js';
 // Parse a comma-separated id list into a positive-int array, or null when empty/absent (so
@@ -82,29 +82,6 @@ const prIdParamSchema = {
         properties: { id: { type: 'integer' } },
     },
 };
-// POST /api/bot-mute-rules — `action` is a closed enum; `autoResolveDays` is floored at 1 so a
-// 0/negative-day auto-resolve rule (which would immediately clear everything) is rejected.
-const muteRuleSchema = {
-    body: {
-        type: 'object',
-        required: ['action'],
-        additionalProperties: false,
-        properties: {
-            vendorKind: { type: ['string', 'null'] },
-            pathGlob: { type: ['string', 'null'] },
-            severity: { type: ['string', 'null'] },
-            action: { type: 'string', enum: ['hide', 'auto_resolve'] },
-            autoResolveDays: { type: ['integer', 'null'], minimum: 1, maximum: 365 },
-        },
-    },
-};
-const ruleIdParamSchema = {
-    params: {
-        type: 'object',
-        required: ['id'],
-        properties: { id: { type: 'integer' } },
-    },
-};
 // GET /api/bot-threads/resolvable?scope=&repoIds= — the scope-wide review list. Same
 // scope/repoIds resolution as the analytics routes (an explicit `repoIds` wins over `scope`).
 const resolvableSchema = {
@@ -139,8 +116,8 @@ const scopeResolveSchema = {
     },
 };
 // Bot-triage platform routes (CORE, always registered). Detection/override, ROI analytics,
-// per-PR cross-bot dedup, and the mute / auto-triage rule store. Every handler is account-
-// scoped via accountIdOf(req); id-addressed reads/writes verify ownership → 404. No AI.
+// per-PR cross-bot dedup, and the confirm-gated scope-wide bot-thread resolve. Every handler is
+// account-scoped via accountIdOf(req); id-addressed reads/writes verify ownership → 404. No AI.
 export async function botTriageRoutes(app) {
     // Every distinct reviewer in the account (human + automated), classified, with 90-day
     // volume + a sample body — powers the Settings "Review bots" detected-reviewers table.
@@ -232,27 +209,6 @@ export async function botTriageRoutes(app) {
         }
         return resp;
     });
-    // The account's mute / auto-triage rules.
-    app.get('/api/bot-mute-rules', async (req) => {
-        const rules = await listBotMuteRules(accountIdOf(req));
-        const resp = { rules };
-        return resp;
-    });
-    app.post('/api/bot-mute-rules', { schema: muteRuleSchema }, async (req) => {
-        const input = req.body;
-        const rule = await addBotMuteRule(accountIdOf(req), input);
-        return rule;
-    });
-    app.delete('/api/bot-mute-rules/:id', { schema: ruleIdParamSchema }, async (req, reply) => {
-        const { id } = req.params;
-        const ok = await deleteBotMuteRule(accountIdOf(req), id);
-        if (!ok) {
-            reply.status(404);
-            return { error: 'NotFound', message: `Mute rule ${id} not found` };
-        }
-        reply.status(204);
-        return null;
-    });
     // The scope-wide review list: every PR with ≥1 `likely_addressed` automated-reviewer thread
     // across the account (or a repo scope), UNCAPPED, newest-thread-first, each row carrying all
     // its resolvable thread ids + a bot thread-state mix + `totalThreads` (the whole backlog). The
@@ -273,7 +229,7 @@ export async function botTriageRoutes(app) {
     // The confirm-gated scope-wide resolve. NEVER blind: the server RE-DERIVES the eligible set
     // (owned + automated-reviewer-originated + `likely_addressed` + unresolved) ∩ the client's
     // explicit reviewed ids, scoped to `repoIds`, then resolves each via the SAME shared helper the
-    // per-PR route + the standing auto-triage job use. An empty list is a no-op (not an error);
+    // per-PR route uses. An empty list is a no-op (not an error);
     // per-thread failures are reported, not fatal. The re-derive path passes `threadIds` so the
     // page cap is bypassed — no requested-and-eligible id is silently dropped.
     app.post('/api/bot-threads/resolve', { schema: scopeResolveSchema }, async (req) => {

package/dist/api/routes/prs.js

@@ -279,7 +279,7 @@ export async function prRoutes(app) {
         const accountId = accountIdOf(req);
         // Server RE-DERIVES the eligible set (owned + automated-reviewer-originated +
         // `likely_addressed` + unresolved) ∩ the client's reviewed list, then resolves each via
-        // the shared helper (the SAME code path the standing auto-triage job uses). An empty
+        // the shared helper (the SAME code path the scope-wide resolve uses). An empty
         // eligible set — PR not owned / no such threads / a fully-stale client list — is a no-op,
         // not an error (the helper short-circuits before any token fetch). Status stays 200 even
         // on partial failure; the body carries per-thread outcomes.

package/dist/app.js

@@ -140,7 +140,7 @@ export async function buildApp() {
     await app.register(activityRoutes);
     await app.register(mentionsRoutes);
     // Bot-triage platform (CORE, always registered): detection/override, ROI analytics,
-    // cross-bot dedup, mute / auto-triage rules. Account-scoped; no AI.
+    // cross-bot dedup, confirm-gated bot-thread resolve. Account-scoped; no AI.
     await app.register(botTriageRoutes);
     // Stripe billing seam (checkout redirect + webhook). Registered in both modes;
     // inert until the STRIPE_* env vars are set (webhook 501s unconfigured).

package/dist/bot-triage/resolve.js

@@ -3,10 +3,11 @@ import { setReviewThreadResolved } from '../github/mutations.js';
 import { stampThreadResolved } from '../db/queries.js';
 // Resolve a batch of already-eligible review threads on GitHub AND stamp them resolved
 // locally. This is the ONE place the GitHub `resolveReviewThread` mutation is driven for
-// bot-thread triage — shared verbatim by BOTH the manual `POST /api/prs/:id/resolve-bot-
-// threads` route and the standing auto-triage scheduled job, so their behaviour can't drift.
+// bot-thread triage — shared verbatim by the manual per-PR `POST /api/prs/:id/resolve-bot-
+// threads` route and the scope-wide `POST /api/bot-threads/resolve`, so their behaviour can't
+// drift. Both are strictly user-initiated + confirm-gated — there is no automatic/cron resolve.
 //
-// Callers own eligibility: both `getResolvableBotThreads` and `getAutoResolveCandidates`
+// Callers own eligibility: `getResolvableBotThreads` / `getResolvableBotThreadsForScope`
 // return ONLY owned + automated-reviewer-originated + `likely_addressed` + unresolved threads
 // (with a GitHub node id), so this helper never has to re-derive it. It NEVER merges — it only
 // resolves. Sequential (not Promise.all) to stay gentle on the GraphQL rate limit (a bot