reviewflow 3.22.0 → 3.23.0

package/dist/tests/acceptance/180-quality-threshold-block-approval-iter-B.acceptance.test.js

@@ -0,0 +1,294 @@
+/**
+ * SPEC-180 — Block approval below quality threshold (Iteration B)
+ *
+ * Outer-loop acceptance test (SDD) for the comment-based bypass mechanism.
+ * Exercises POST /webhooks/gitlab with X-Gitlab-Event: Note Hook events and
+ * the chained recordReviewCompletion → bypass-reset path.
+ *
+ * In-scope scenarios from docs/specs/180-quality-threshold-block-approval.md:
+ *   4: bypass with reason → state transition to approved allowed + bypass recorded
+ *   5: bypass without reason → reject with FR message + no bypass stored + FR comment posted
+ *   9: new review after bypass → bypass cleared + state re-evaluated under normal gate
+ *  10: bypass on already-qualified MR → bypass recorded + no state change
+ *
+ * Out of scope: scenario 6 (platform unapprove on platform-side approval — iter C).
+ */
+import { vi } from 'vitest';
+const mockRepoConfig = {
+    name: 'test-project',
+    platform: 'gitlab',
+    localPath: '/home/user/projects/test-project',
+    remoteUrl: 'https://gitlab.com/test-org/test-project.git',
+    skill: 'review-front',
+    enabled: true,
+};
+const mockConfig = {
+    server: { port: 3000 },
+    user: { gitlabUsername: 'claude-bot', githubUsername: 'claude-bot' },
+    queue: { maxConcurrent: 1, deduplicationWindowMs: 60000 },
+    repositories: [mockRepoConfig],
+};
+vi.mock('@/config/loader.js', () => ({
+    loadConfig: vi.fn(() => mockConfig),
+    findRepositoryByProjectPath: vi.fn(() => mockRepoConfig),
+    findRepositoryByRemoteUrl: vi.fn(() => mockRepoConfig),
+}));
+vi.mock('@/security/verifier.js', () => ({
+    verifyGitLabSignature: vi.fn(() => ({ valid: true })),
+    getGitLabEventType: vi.fn(() => 'Note Hook'),
+}));
+vi.mock('@/frameworks/queue/pQueueAdapter.js', () => ({
+    createJobId: vi.fn((prefix, projectPath, mrNumber) => `${prefix}-${projectPath}-${mrNumber}`),
+    enqueueReview: vi.fn(() => Promise.resolve(true)),
+    updateJobProgress: vi.fn(),
+    cancelJob: vi.fn(),
+}));
+vi.mock('@/claude/invoker.js', () => ({
+    invokeClaudeReview: vi.fn(),
+    sendNotification: vi.fn(),
+}));
+vi.mock('@/main/websocket.js', () => ({
+    startWatchingReviewContext: vi.fn(),
+    stopWatchingReviewContext: vi.fn(),
+}));
+vi.mock('@/config/projectConfig.js', () => ({
+    loadProjectConfig: vi.fn(() => ({ qualityThreshold: 7 })),
+    getProjectAgents: vi.fn(() => null),
+    getProjectAgentsOrFocusDefaults: vi.fn(() => null),
+    getFollowupAgents: vi.fn(() => null),
+    getProjectLanguage: vi.fn(() => 'en'),
+}));
+import { describe, it, expect, beforeEach, afterEach } from 'vitest';
+import { handleGitLabWebhook } from '../../modules/platform-integration/interface-adapters/controllers/webhook/gitlab.controller.js';
+import { InMemoryReviewRequestTrackingGateway } from '../../tests/stubs/reviewRequestTracking.stub.js';
+import { StubNoteCommentPostGateway } from '../../tests/stubs/noteCommentPost.stub.js';
+import { TrackedMrFactory } from '../../tests/factories/trackedMr.factory.js';
+import { createStubLogger } from '../../tests/stubs/logger.stub.js';
+import { RecordBypassUseCase } from '../../modules/tracking/usecases/tracking/recordBypass.usecase.js';
+import { RecordReviewCompletionUseCase } from '../../modules/tracking/usecases/tracking/recordReviewCompletion.usecase.js';
+import { TransitionStateUseCase } from '../../modules/tracking/usecases/tracking/transitionState.usecase.js';
+import { TrackAssignmentUseCase } from '../../modules/tracking/usecases/tracking/trackAssignment.usecase.js';
+import { RecordPushUseCase } from '../../modules/tracking/usecases/tracking/recordPush.usecase.js';
+import { CheckFollowupNeededUseCase } from '../../modules/tracking/usecases/tracking/checkFollowupNeeded.usecase.js';
+import { SyncThreadsUseCase } from '../../modules/tracking/usecases/tracking/syncThreads.usecase.js';
+import { evaluateQualityGate } from '../../modules/tracking/entities/qualityGate/qualityGate.js';
+const PROJECT_PATH = '/home/user/projects/test-project';
+const MR_NUMBER = 42;
+const MR_ID = `gitlab-test-org/test-project-${MR_NUMBER}`;
+function buildNoteEvent(noteBody, authorUsername = 'alice') {
+    return {
+        object_kind: 'note',
+        event_type: 'note',
+        user: { username: authorUsername, name: authorUsername },
+        project: {
+            id: 1,
+            name: 'test-project',
+            path_with_namespace: 'test-org/test-project',
+            web_url: 'https://gitlab.com/test-org/test-project',
+            git_http_url: 'https://gitlab.com/test-org/test-project.git',
+        },
+        object_attributes: {
+            id: 999,
+            note: noteBody,
+            noteable_type: 'MergeRequest',
+            noteable_id: MR_NUMBER,
+        },
+        merge_request: {
+            iid: MR_NUMBER,
+            title: 'Test MR',
+            state: 'opened',
+            source_branch: 'feature/test',
+            target_branch: 'main',
+            url: `https://gitlab.com/test-org/test-project/-/merge_requests/${MR_NUMBER}`,
+        },
+    };
+}
+function createDeterministicNow() {
+    return () => '2026-05-26T12:00:00.000Z';
+}
+function buildDeps(tracking, noteCommentPost) {
+    const threadFetchGateway = { fetchThreads: vi.fn(() => []) };
+    return {
+        reviewContextGateway: {
+            create: vi.fn(() => ({ success: true, filePath: '' })),
+            read: vi.fn(() => null),
+            delete: vi.fn(() => ({ success: true, deleted: true })),
+            exists: vi.fn(() => false),
+            getFilePath: vi.fn(() => ''),
+            appendAction: vi.fn(() => ({ success: true })),
+            updateProgress: vi.fn(() => ({ success: true })),
+            setResult: vi.fn(() => ({ success: true })),
+        },
+        threadFetchGateway,
+        diffMetadataFetchGateway: { fetchDiffMetadata: vi.fn(() => undefined) },
+        diffStatsFetchGateway: { fetchDiffStats: vi.fn(() => null) },
+        trackAssignment: new TrackAssignmentUseCase(tracking),
+        recordCompletion: new RecordReviewCompletionUseCase(tracking),
+        recordPush: new RecordPushUseCase(tracking),
+        transitionState: new TransitionStateUseCase(tracking),
+        checkFollowupNeeded: new CheckFollowupNeededUseCase(tracking),
+        syncThreads: new SyncThreadsUseCase(tracking, threadFetchGateway),
+        recordBypass: new RecordBypassUseCase(tracking),
+        noteCommentPostGateway: noteCommentPost,
+        now: createDeterministicNow(),
+        enforceBudget: {
+            execute: vi.fn(async () => ({
+                accepted: true,
+                status: {
+                    limitUsd: 200,
+                    consumedUsd: 0,
+                    remainingUsd: 200,
+                    percentUsed: 0,
+                    exceeded: false,
+                    periodStart: '2026-05-01T00:00:00.000Z',
+                },
+            })),
+        },
+        broadcastBudgetExceeded: vi.fn(),
+        getRepositories: vi.fn(() => [mockRepoConfig]),
+        removeWorktree: vi.fn(async () => ({ status: 'removed' })),
+    };
+}
+describe('Acceptance — SPEC-180 Iteration B: Comment-based bypass', () => {
+    let tracking;
+    let noteCommentPost;
+    let deps;
+    let mockReply;
+    const logger = createStubLogger();
+    beforeEach(() => {
+        vi.clearAllMocks();
+        tracking = new InMemoryReviewRequestTrackingGateway();
+        noteCommentPost = new StubNoteCommentPostGateway();
+        deps = buildDeps(tracking, noteCommentPost);
+        mockReply = {
+            status: vi.fn().mockReturnThis(),
+            send: vi.fn().mockReturnThis(),
+        };
+    });
+    afterEach(() => {
+        vi.resetAllMocks();
+    });
+    describe('Rule: bypass with reason overrides the quality gate', () => {
+        it('scenario 4 — bypass marker with reason allows approval despite failing gate', async () => {
+            tracking.create(PROJECT_PATH, TrackedMrFactory.create({
+                id: MR_ID,
+                mrNumber: MR_NUMBER,
+                platform: 'gitlab',
+                project: 'test-org/test-project',
+                state: 'pending-approval',
+                latestScore: 5,
+                openThreads: 1,
+            }));
+            const noteBody = '/bypass-quality "hotfix critique"';
+            const request = {
+                body: buildNoteEvent(noteBody),
+                headers: {},
+            };
+            await handleGitLabWebhook(request, mockReply, logger, tracking, deps);
+            const afterBypass = tracking.getById(PROJECT_PATH, MR_ID);
+            expect(afterBypass?.bypass).toEqual({
+                author: 'alice',
+                reason: 'hotfix critique',
+                recordedAt: '2026-05-26T12:00:00.000Z',
+            });
+            const transitionResult = deps.transitionState.execute({
+                projectPath: PROJECT_PATH,
+                mrId: MR_ID,
+                targetState: 'approved',
+                qualityCheck: (mr) => evaluateQualityGate({
+                    latestScore: mr.latestScore,
+                    blockingIssues: mr.openThreads,
+                    threshold: 7,
+                }),
+            });
+            expect(transitionResult.ok).toBe(true);
+            const final = tracking.getById(PROJECT_PATH, MR_ID);
+            expect(final?.state).toBe('approved');
+        });
+    });
+    describe('Rule: bypass without reason is rejected with a French message', () => {
+        it('scenario 5 — bypass marker without reason posts FR comment and does not store bypass', async () => {
+            tracking.create(PROJECT_PATH, TrackedMrFactory.create({
+                id: MR_ID,
+                mrNumber: MR_NUMBER,
+                platform: 'gitlab',
+                project: 'test-org/test-project',
+                state: 'pending-approval',
+                latestScore: 5,
+                openThreads: 0,
+            }));
+            const request = {
+                body: buildNoteEvent('/bypass-quality'),
+                headers: {},
+            };
+            await handleGitLabWebhook(request, mockReply, logger, tracking, deps);
+            expect(noteCommentPost.calls).toHaveLength(1);
+            expect(noteCommentPost.calls[0]).toEqual({
+                projectPath: 'test-org/test-project',
+                mrNumber: MR_NUMBER,
+                body: 'Le bypass nécessite une raison explicite. Format attendu : /bypass-quality "raison"',
+            });
+            const updated = tracking.getById(PROJECT_PATH, MR_ID);
+            expect(updated?.bypass).toBeNull();
+        });
+    });
+    describe('Rule: a new review resets any active bypass', () => {
+        it('scenario 9 — new completed review clears bypass and re-evaluates state', () => {
+            tracking.create(PROJECT_PATH, TrackedMrFactory.create({
+                id: MR_ID,
+                mrNumber: MR_NUMBER,
+                platform: 'gitlab',
+                project: 'test-org/test-project',
+                state: 'pending-approval',
+                latestScore: 5,
+                openThreads: 0,
+                bypass: { author: 'alice', reason: 'hotfix critique', recordedAt: '2026-05-25T08:00:00.000Z' },
+            }));
+            const recordCompletion = new RecordReviewCompletionUseCase(tracking);
+            const result = recordCompletion.execute({
+                projectPath: PROJECT_PATH,
+                mrId: MR_ID,
+                reviewData: {
+                    type: 'review',
+                    durationMs: 30000,
+                    score: 8,
+                    blocking: 0,
+                    warnings: 0,
+                    suggestions: 0,
+                    threadsOpened: 0,
+                    threadsClosed: 0,
+                },
+                qualityThreshold: 7,
+            });
+            expect(result?.bypass).toBeNull();
+            expect(result?.state).toBe('pending-approval');
+        });
+    });
+    describe('Rule: bypass on an already-qualified MR is recorded without state change', () => {
+        it('scenario 10 — valid bypass stored, state untouched', async () => {
+            tracking.create(PROJECT_PATH, TrackedMrFactory.create({
+                id: MR_ID,
+                mrNumber: MR_NUMBER,
+                platform: 'gitlab',
+                project: 'test-org/test-project',
+                state: 'pending-approval',
+                latestScore: 9,
+                openThreads: 0,
+            }));
+            const request = {
+                body: buildNoteEvent('/bypass-quality "par précaution"'),
+                headers: {},
+            };
+            await handleGitLabWebhook(request, mockReply, logger, tracking, deps);
+            const updated = tracking.getById(PROJECT_PATH, MR_ID);
+            expect(updated?.bypass).toEqual({
+                author: 'alice',
+                reason: 'par précaution',
+                recordedAt: '2026-05-26T12:00:00.000Z',
+            });
+            expect(updated?.state).toBe('pending-approval');
+            expect(noteCommentPost.calls).toHaveLength(0);
+        });
+    });
+});
+//# sourceMappingURL=180-quality-threshold-block-approval-iter-B.acceptance.test.js.map

package/dist/tests/acceptance/180-quality-threshold-block-approval-iter-B.acceptance.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"180-quality-threshold-block-approval-iter-B.acceptance.test.js","sourceRoot":"","sources":["../../../src/tests/acceptance/180-quality-threshold-block-approval-iter-B.acceptance.test.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;GAcG;AAEH,OAAO,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AAI5B,MAAM,cAAc,GAAqB;IACvC,IAAI,EAAE,cAAc;IACpB,QAAQ,EAAE,QAAQ;IAClB,SAAS,EAAE,kCAAkC;IAC7C,SAAS,EAAE,8CAA8C;IACzD,KAAK,EAAE,cAAc;IACrB,OAAO,EAAE,IAAI;CACd,CAAC;AAEF,MAAM,UAAU,GAAG;IACjB,MAAM,EAAE,EAAE,IAAI,EAAE,IAAI,EAAE;IACtB,IAAI,EAAE,EAAE,cAAc,EAAE,YAAY,EAAE,cAAc,EAAE,YAAY,EAAE;IACpE,KAAK,EAAE,EAAE,aAAa,EAAE,CAAC,EAAE,qBAAqB,EAAE,KAAK,EAAE;IACzD,YAAY,EAAE,CAAC,cAAc,CAAC;CAC/B,CAAC;AAEF,EAAE,CAAC,IAAI,CAAC,oBAAoB,EAAE,GAAG,EAAE,CAAC,CAAC;IACnC,UAAU,EAAE,EAAE,CAAC,EAAE,CAAC,GAAG,EAAE,CAAC,UAAU,CAAC;IACnC,2BAA2B,EAAE,EAAE,CAAC,EAAE,CAAC,GAAG,EAAE,CAAC,cAAc,CAAC;IACxD,yBAAyB,EAAE,EAAE,CAAC,EAAE,CAAC,GAAG,EAAE,CAAC,cAAc,CAAC;CACvD,CAAC,CAAC,CAAC;AAEJ,EAAE,CAAC,IAAI,CAAC,wBAAwB,EAAE,GAAG,EAAE,CAAC,CAAC;IACvC,qBAAqB,EAAE,EAAE,CAAC,EAAE,CAAC,GAAG,EAAE,CAAC,CAAC,EAAE,KAAK,EAAE,IAAI,EAAE,CAAC,CAAC;IACrD,kBAAkB,EAAE,EAAE,CAAC,EAAE,CAAC,GAAG,EAAE,CAAC,WAAW,CAAC;CAC7C,CAAC,CAAC,CAAC;AAEJ,EAAE,CAAC,IAAI,CAAC,qCAAqC,EAAE,GAAG,EAAE,CAAC,CAAC;IACpD,WAAW,EAAE,EAAE,CAAC,EAAE,CAAC,CAAC,MAAc,EAAE,WAAmB,EAAE,QAAgB,EAAE,EAAE,CAAC,GAAG,MAAM,IAAI,WAAW,IAAI,QAAQ,EAAE,CAAC;IACrH,aAAa,EAAE,EAAE,CAAC,EAAE,CAAC,GAAG,EAAE,CAAC,OAAO,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC;IACjD,iBAAiB,EAAE,EAAE,CAAC,EAAE,EAAE;IAC1B,SAAS,EAAE,EAAE,CAAC,EAAE,EAAE;CACnB,CAAC,CAAC,CAAC;AAEJ,EAAE,CAAC,IAAI,CAAC,qBAAqB,EAAE,GAAG,EAAE,CAAC,CAAC;IACpC,kBAAkB,EAAE,EAAE,CAAC,EAAE,EAAE;IAC3B,gBAAgB,EAAE,EAAE,CAAC,EAAE,EAAE;CAC1B,CAAC,CAAC,CAAC;AAEJ,EAAE,CAAC,IAAI,CAAC,qBAAqB,EAAE,GAAG,EAAE,CAAC,CAAC;IACpC,0BAA0B,EAAE,EAAE,CAAC,EAAE,EAAE;IACnC,yBAAyB,EAAE,EAAE,CAAC,EAAE,EAAE;CACnC,CAAC,CAAC,CAAC;AAEJ,EAAE,CAAC,IAAI,CAAC,2BAA2B,EAAE,GAAG,EAAE,CAAC,CAAC;IAC1C,iBAAiB,EAAE,EAAE,CAAC,EAAE,CAAC,GAAG,EAAE,CAAC,CAAC,EAAE,gBAAgB,EAAE,CAAC,EAAE,CAAC,CAAC;IACzD,gBAAgB,EAAE,EAAE,CAAC,EAAE,CAAC,GAAG,EAAE,CAAC,IAAI,CAAC;IACnC,+BAA+B,EAAE,EAAE,CAAC,EAAE,CAAC,GAAG,EAAE,CAAC,IAAI,CAAC;IAClD,iBAAiB,EAAE,EAAE,CAAC,EAAE,CAAC,GAAG,EAAE,CAAC,IAAI,CAAC;IACpC,kBAAkB,EAAE,EAAE,CAAC,EAAE,CAAC,GAAG,EAAE,CAAC,IAAI,CAAC;CACtC,CAAC,CAAC,CAAC;AAEJ,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,UAAU,EAAE,SAAS,EAAE,MAAM,QAAQ,CAAC;AACrE,OAAO,EAAE,mBAAmB,EAAE,MAAM,4FAA4F,CAAC;AAEjI,OAAO,EAAE,oCAAoC,EAAE,MAAM,6CAA6C,CAAC;AACnG,OAAO,EAAE,0BAA0B,EAAE,MAAM,uCAAuC,CAAC;AACnF,OAAO,EAAE,gBAAgB,EAAE,MAAM,wCAAwC,CAAC;AAC1E,OAAO,EAAE,gBAAgB,EAAE,MAAM,8BAA8B,CAAC;AAChE,OAAO,EAAE,mBAAmB,EAAE,MAAM,8DAA8D,CAAC;AACnG,OAAO,EAAE,6BAA6B,EAAE,MAAM,wEAAwE,CAAC;AACvH,OAAO,EAAE,sBAAsB,EAAE,MAAM,iEAAiE,CAAC;AACzG,OAAO,EAAE,sBAAsB,EAAE,MAAM,iEAAiE,CAAC;AACzG,OAAO,EAAE,iBAAiB,EAAE,MAAM,4DAA4D,CAAC;AAC/F,OAAO,EAAE,0BAA0B,EAAE,MAAM,qEAAqE,CAAC;AACjH,OAAO,EAAE,kBAAkB,EAAE,MAAM,6DAA6D,CAAC;AACjG,OAAO,EAAE,mBAAmB,EAAE,MAAM,wDAAwD,CAAC;AAE7F,MAAM,YAAY,GAAG,kCAAkC,CAAC;AACxD,MAAM,SAAS,GAAG,EAAE,CAAC;AACrB,MAAM,KAAK,GAAG,gCAAgC,SAAS,EAAE,CAAC;AAE1D,SAAS,cAAc,CAAC,QAAgB,EAAE,cAAc,GAAG,OAAO;IAChE,OAAO;QACL,WAAW,EAAE,MAAM;QACnB,UAAU,EAAE,MAAM;QAClB,IAAI,EAAE,EAAE,QAAQ,EAAE,cAAc,EAAE,IAAI,EAAE,cAAc,EAAE;QACxD,OAAO,EAAE;YACP,EAAE,EAAE,CAAC;YACL,IAAI,EAAE,cAAc;YACpB,mBAAmB,EAAE,uBAAuB;YAC5C,OAAO,EAAE,0CAA0C;YACnD,YAAY,EAAE,8CAA8C;SAC7D;QACD,iBAAiB,EAAE;YACjB,EAAE,EAAE,GAAG;YACP,IAAI,EAAE,QAAQ;YACd,aAAa,EAAE,cAAc;YAC7B,WAAW,EAAE,SAAS;SACvB;QACD,aAAa,EAAE;YACb,GAAG,EAAE,SAAS;YACd,KAAK,EAAE,SAAS;YAChB,KAAK,EAAE,QAAQ;YACf,aAAa,EAAE,cAAc;YAC7B,aAAa,EAAE,MAAM;YACrB,GAAG,EAAE,6DAA6D,SAAS,EAAE;SAC9E;KACF,CAAC;AACJ,CAAC;AAED,SAAS,sBAAsB;IAC7B,OAAO,GAAG,EAAE,CAAC,0BAA0B,CAAC;AAC1C,CAAC;AAED,SAAS,SAAS,CAChB,QAA8C,EAC9C,eAA2C;IAE3C,MAAM,kBAAkB,GAAG,EAAE,YAAY,EAAE,EAAE,CAAC,EAAE,CAAC,GAAG,EAAE,CAAC,EAAE,CAAC,EAAE,CAAC;IAC7D,OAAO;QACL,oBAAoB,EAAE;YACpB,MAAM,EAAE,EAAE,CAAC,EAAE,CAAC,GAAG,EAAE,CAAC,CAAC,EAAE,OAAO,EAAE,IAAI,EAAE,QAAQ,EAAE,EAAE,EAAE,CAAC,CAAC;YACtD,IAAI,EAAE,EAAE,CAAC,EAAE,CAAC,GAAG,EAAE,CAAC,IAAI,CAAC;YACvB,MAAM,EAAE,EAAE,CAAC,EAAE,CAAC,GAAG,EAAE,CAAC,CAAC,EAAE,OAAO,EAAE,IAAI,EAAE,OAAO,EAAE,IAAI,EAAE,CAAC,CAAC;YACvD,MAAM,EAAE,EAAE,CAAC,EAAE,CAAC,GAAG,EAAE,CAAC,KAAK,CAAC;YAC1B,WAAW,EAAE,EAAE,CAAC,EAAE,CAAC,GAAG,EAAE,CAAC,EAAE,CAAC;YAC5B,YAAY,EAAE,EAAE,CAAC,EAAE,CAAC,GAAG,EAAE,CAAC,CAAC,EAAE,OAAO,EAAE,IAAI,EAAE,CAAC,CAAC;YAC9C,cAAc,EAAE,EAAE,CAAC,EAAE,CAAC,GAAG,EAAE,CAAC,CAAC,EAAE,OAAO,EAAE,IAAI,EAAE,CAAC,CAAC;YAChD,SAAS,EAAE,EAAE,CAAC,EAAE,CAAC,GAAG,EAAE,CAAC,CAAC,EAAE,OAAO,EAAE,IAAI,EAAE,CAAC,CAAC;SAC5C;QACD,kBAAkB;QAClB,wBAAwB,EAAE,EAAE,iBAAiB,EAAE,EAAE,CAAC,EAAE,CAAC,GAAG,EAAE,CAAC,SAAS,CAAC,EAAE;QACvE,qBAAqB,EAAE,EAAE,cAAc,EAAE,EAAE,CAAC,EAAE,CAAC,GAAG,EAAE,CAAC,IAAI,CAAC,EAAE;QAC5D,eAAe,EAAE,IAAI,sBAAsB,CAAC,QAAQ,CAAC;QACrD,gBAAgB,EAAE,IAAI,6BAA6B,CAAC,QAAQ,CAAC;QAC7D,UAAU,EAAE,IAAI,iBAAiB,CAAC,QAAQ,CAAC;QAC3C,eAAe,EAAE,IAAI,sBAAsB,CAAC,QAAQ,CAAC;QACrD,mBAAmB,EAAE,IAAI,0BAA0B,CAAC,QAAQ,CAAC;QAC7D,WAAW,EAAE,IAAI,kBAAkB,CAAC,QAAQ,EAAE,kBAAkB,CAAC;QACjE,YAAY,EAAE,IAAI,mBAAmB,CAAC,QAAQ,CAAC;QAC/C,sBAAsB,EAAE,eAAe;QACvC,GAAG,EAAE,sBAAsB,EAAE;QAC7B,aAAa,EAAE;YACb,OAAO,EAAE,EAAE,CAAC,EAAE,CAAC,KAAK,IAAI,EAAE,CAAC,CAAC;gBAC1B,QAAQ,EAAE,IAAI;gBACd,MAAM,EAAE;oBACN,QAAQ,EAAE,GAAG;oBACb,WAAW,EAAE,CAAC;oBACd,YAAY,EAAE,GAAG;oBACjB,WAAW,EAAE,CAAC;oBACd,QAAQ,EAAE,KAAK;oBACf,WAAW,EAAE,0BAA0B;iBACxC;aACF,CAAC,CAAC;SACJ;QACD,uBAAuB,EAAE,EAAE,CAAC,EAAE,EAAE;QAChC,eAAe,EAAE,EAAE,CAAC,EAAE,CAAC,GAAG,EAAE,CAAC,CAAC,cAAc,CAAC,CAAC;QAC9C,cAAc,EAAE,EAAE,CAAC,EAAE,CAAC,KAAK,IAAI,EAAE,CAAC,CAAC,EAAE,MAAM,EAAE,SAAkB,EAAE,CAAC,CAAC;KAC5B,CAAC;AAC5C,CAAC;AAED,QAAQ,CAAC,yDAAyD,EAAE,GAAG,EAAE;IACvE,IAAI,QAA8C,CAAC;IACnD,IAAI,eAA2C,CAAC;IAChD,IAAI,IAA+B,CAAC;IACpC,IAAI,SAAuB,CAAC;IAC5B,MAAM,MAAM,GAAG,gBAAgB,EAAE,CAAC;IAElC,UAAU,CAAC,GAAG,EAAE;QACd,EAAE,CAAC,aAAa,EAAE,CAAC;QACnB,QAAQ,GAAG,IAAI,oCAAoC,EAAE,CAAC;QACtD,eAAe,GAAG,IAAI,0BAA0B,EAAE,CAAC;QACnD,IAAI,GAAG,SAAS,CAAC,QAAQ,EAAE,eAAe,CAAC,CAAC;QAC5C,SAAS,GAAG;YACV,MAAM,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC,cAAc,EAAE;YAChC,IAAI,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC,cAAc,EAAE;SACJ,CAAC;IAC/B,CAAC,CAAC,CAAC;IAEH,SAAS,CAAC,GAAG,EAAE;QACb,EAAE,CAAC,aAAa,EAAE,CAAC;IACrB,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,qDAAqD,EAAE,GAAG,EAAE;QACnE,EAAE,CAAC,6EAA6E,EAAE,KAAK,IAAI,EAAE;YAC3F,QAAQ,CAAC,MAAM,CACb,YAAY,EACZ,gBAAgB,CAAC,MAAM,CAAC;gBACtB,EAAE,EAAE,KAAK;gBACT,QAAQ,EAAE,SAAS;gBACnB,QAAQ,EAAE,QAAQ;gBAClB,OAAO,EAAE,uBAAuB;gBAChC,KAAK,EAAE,kBAAkB;gBACzB,WAAW,EAAE,CAAC;gBACd,WAAW,EAAE,CAAC;aACf,CAAC,CACH,CAAC;YAEF,MAAM,QAAQ,GAAG,mCAAmC,CAAC;YACrD,MAAM,OAAO,GAAG;gBACd,IAAI,EAAE,cAAc,CAAC,QAAQ,CAAC;gBAC9B,OAAO,EAAE,EAAE;aACiB,CAAC;YAE/B,MAAM,mBAAmB,CAAC,OAAO,EAAE,SAAS,EAAE,MAAM,EAAE,QAAQ,EAAE,IAAI,CAAC,CAAC;YAEtE,MAAM,WAAW,GAAG,QAAQ,CAAC,OAAO,CAAC,YAAY,EAAE,KAAK,CAAC,CAAC;YAC1D,MAAM,CAAC,WAAW,EAAE,MAAM,CAAC,CAAC,OAAO,CAAC;gBAClC,MAAM,EAAE,OAAO;gBACf,MAAM,EAAE,iBAAiB;gBACzB,UAAU,EAAE,0BAA0B;aACvC,CAAC,CAAC;YAEH,MAAM,gBAAgB,GAAG,IAAI,CAAC,eAAe,CAAC,OAAO,CAAC;gBACpD,WAAW,EAAE,YAAY;gBACzB,IAAI,EAAE,KAAK;gBACX,WAAW,EAAE,UAAU;gBACvB,YAAY,EAAE,CAAC,EAAE,EAAE,EAAE,CACnB,mBAAmB,CAAC;oBAClB,WAAW,EAAE,EAAE,CAAC,WAAW;oBAC3B,cAAc,EAAE,EAAE,CAAC,WAAW;oBAC9B,SAAS,EAAE,CAAC;iBACb,CAAC;aACL,CAAC,CAAC;YACH,MAAM,CAAC,gBAAgB,CAAC,EAAE,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;YAEvC,MAAM,KAAK,GAAG,QAAQ,CAAC,OAAO,CAAC,YAAY,EAAE,KAAK,CAAC,CAAC;YACpD,MAAM,CAAC,KAAK,EAAE,KAAK,CAAC,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;QACxC,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,+DAA+D,EAAE,GAAG,EAAE;QAC7E,EAAE,CAAC,sFAAsF,EAAE,KAAK,IAAI,EAAE;YACpG,QAAQ,CAAC,MAAM,CACb,YAAY,EACZ,gBAAgB,CAAC,MAAM,CAAC;gBACtB,EAAE,EAAE,KAAK;gBACT,QAAQ,EAAE,SAAS;gBACnB,QAAQ,EAAE,QAAQ;gBAClB,OAAO,EAAE,uBAAuB;gBAChC,KAAK,EAAE,kBAAkB;gBACzB,WAAW,EAAE,CAAC;gBACd,WAAW,EAAE,CAAC;aACf,CAAC,CACH,CAAC;YAEF,MAAM,OAAO,GAAG;gBACd,IAAI,EAAE,cAAc,CAAC,iBAAiB,CAAC;gBACvC,OAAO,EAAE,EAAE;aACiB,CAAC;YAE/B,MAAM,mBAAmB,CAAC,OAAO,EAAE,SAAS,EAAE,MAAM,EAAE,QAAQ,EAAE,IAAI,CAAC,CAAC;YAEtE,MAAM,CAAC,eAAe,CAAC,KAAK,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;YAC9C,MAAM,CAAC,eAAe,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC;gBACvC,WAAW,EAAE,uBAAuB;gBACpC,QAAQ,EAAE,SAAS;gBACnB,IAAI,EAAE,qFAAqF;aAC5F,CAAC,CAAC;YAEH,MAAM,OAAO,GAAG,QAAQ,CAAC,OAAO,CAAC,YAAY,EAAE,KAAK,CAAC,CAAC;YACtD,MAAM,CAAC,OAAO,EAAE,MAAM,CAAC,CAAC,QAAQ,EAAE,CAAC;QACrC,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,6CAA6C,EAAE,GAAG,EAAE;QAC3D,EAAE,CAAC,wEAAwE,EAAE,GAAG,EAAE;YAChF,QAAQ,CAAC,MAAM,CACb,YAAY,EACZ,gBAAgB,CAAC,MAAM,CAAC;gBACtB,EAAE,EAAE,KAAK;gBACT,QAAQ,EAAE,SAAS;gBACnB,QAAQ,EAAE,QAAQ;gBAClB,OAAO,EAAE,uBAAuB;gBAChC,KAAK,EAAE,kBAAkB;gBACzB,WAAW,EAAE,CAAC;gBACd,WAAW,EAAE,CAAC;gBACd,MAAM,EAAE,EAAE,MAAM,EAAE,OAAO,EAAE,MAAM,EAAE,iBAAiB,EAAE,UAAU,EAAE,0BAA0B,EAAE;aAC/F,CAAC,CACH,CAAC;YAEF,MAAM,gBAAgB,GAAG,IAAI,6BAA6B,CAAC,QAAQ,CAAC,CAAC;YACrE,MAAM,MAAM,GAAG,gBAAgB,CAAC,OAAO,CAAC;gBACtC,WAAW,EAAE,YAAY;gBACzB,IAAI,EAAE,KAAK;gBACX,UAAU,EAAE;oBACV,IAAI,EAAE,QAAQ;oBACd,UAAU,EAAE,KAAK;oBACjB,KAAK,EAAE,CAAC;oBACR,QAAQ,EAAE,CAAC;oBACX,QAAQ,EAAE,CAAC;oBACX,WAAW,EAAE,CAAC;oBACd,aAAa,EAAE,CAAC;oBAChB,aAAa,EAAE,CAAC;iBACjB;gBACD,gBAAgB,EAAE,CAAC;aACpB,CAAC,CAAC;YAEH,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC,QAAQ,EAAE,CAAC;YAClC,MAAM,CAAC,MAAM,EAAE,KAAK,CAAC,CAAC,IAAI,CAAC,kBAAkB,CAAC,CAAC;QACjD,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,0EAA0E,EAAE,GAAG,EAAE;QACxF,EAAE,CAAC,oDAAoD,EAAE,KAAK,IAAI,EAAE;YAClE,QAAQ,CAAC,MAAM,CACb,YAAY,EACZ,gBAAgB,CAAC,MAAM,CAAC;gBACtB,EAAE,EAAE,KAAK;gBACT,QAAQ,EAAE,SAAS;gBACnB,QAAQ,EAAE,QAAQ;gBAClB,OAAO,EAAE,uBAAuB;gBAChC,KAAK,EAAE,kBAAkB;gBACzB,WAAW,EAAE,CAAC;gBACd,WAAW,EAAE,CAAC;aACf,CAAC,CACH,CAAC;YAEF,MAAM,OAAO,GAAG;gBACd,IAAI,EAAE,cAAc,CAAC,kCAAkC,CAAC;gBACxD,OAAO,EAAE,EAAE;aACiB,CAAC;YAE/B,MAAM,mBAAmB,CAAC,OAAO,EAAE,SAAS,EAAE,MAAM,EAAE,QAAQ,EAAE,IAAI,CAAC,CAAC;YAEtE,MAAM,OAAO,GAAG,QAAQ,CAAC,OAAO,CAAC,YAAY,EAAE,KAAK,CAAC,CAAC;YACtD,MAAM,CAAC,OAAO,EAAE,MAAM,CAAC,CAAC,OAAO,CAAC;gBAC9B,MAAM,EAAE,OAAO;gBACf,MAAM,EAAE,gBAAgB;gBACxB,UAAU,EAAE,0BAA0B;aACvC,CAAC,CAAC;YACH,MAAM,CAAC,OAAO,EAAE,KAAK,CAAC,CAAC,IAAI,CAAC,kBAAkB,CAAC,CAAC;YAChD,MAAM,CAAC,eAAe,CAAC,KAAK,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;QAChD,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/dist/tests/acceptance/180-quality-threshold-block-approval-iter-C.acceptance.test.d.ts

@@ -0,0 +1,16 @@
+/**
+ * SPEC-180 — Block approval below quality threshold (Iteration C)
+ *
+ * Outer-loop acceptance test (SDD) for the platform-side revocation + FR
+ * explanatory comment on platform approval of a non-qualified merge request
+ * without an active bypass.
+ *
+ * In-scope scenarios from docs/specs/180-quality-threshold-block-approval.md:
+ *   6: platform approval on non-qualified MR → unapprove on platform + post
+ *      "Approbation annulée : seuil qualité 7/10 non atteint (6/10).
+ *       Utilisez `/bypass-quality "raison"` pour forcer."
+ *
+ * Both platforms (GitLab + GitHub) — the spec rule applies symmetrically.
+ */
+export {};
+//# sourceMappingURL=180-quality-threshold-block-approval-iter-C.acceptance.test.d.ts.map

package/dist/tests/acceptance/180-quality-threshold-block-approval-iter-C.acceptance.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"180-quality-threshold-block-approval-iter-C.acceptance.test.d.ts","sourceRoot":"","sources":["../../../src/tests/acceptance/180-quality-threshold-block-approval-iter-C.acceptance.test.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;GAaG"}

package/dist/tests/acceptance/180-quality-threshold-block-approval-iter-C.acceptance.test.js

@@ -0,0 +1,316 @@
+/**
+ * SPEC-180 — Block approval below quality threshold (Iteration C)
+ *
+ * Outer-loop acceptance test (SDD) for the platform-side revocation + FR
+ * explanatory comment on platform approval of a non-qualified merge request
+ * without an active bypass.
+ *
+ * In-scope scenarios from docs/specs/180-quality-threshold-block-approval.md:
+ *   6: platform approval on non-qualified MR → unapprove on platform + post
+ *      "Approbation annulée : seuil qualité 7/10 non atteint (6/10).
+ *       Utilisez `/bypass-quality "raison"` pour forcer."
+ *
+ * Both platforms (GitLab + GitHub) — the spec rule applies symmetrically.
+ */
+import { vi } from 'vitest';
+const mockRepoConfig = {
+    name: 'test-project',
+    platform: 'gitlab',
+    localPath: '/home/user/projects/test-project',
+    remoteUrl: 'https://gitlab.com/test-org/test-project.git',
+    skill: 'review-front',
+    enabled: true,
+};
+const mockGitHubRepoConfig = {
+    name: 'test-project',
+    platform: 'github',
+    localPath: '/home/user/projects/test-project',
+    remoteUrl: 'https://github.com/test-org/test-project.git',
+    skill: 'review-front',
+    enabled: true,
+};
+const mockConfig = {
+    server: { port: 3000 },
+    user: { gitlabUsername: 'claude-bot', githubUsername: 'claude-bot' },
+    queue: { maxConcurrent: 1, deduplicationWindowMs: 60000 },
+    repositories: [mockRepoConfig],
+};
+vi.mock('@/config/loader.js', () => ({
+    loadConfig: vi.fn(() => mockConfig),
+    findRepositoryByProjectPath: vi.fn(() => mockRepoConfig),
+    findRepositoryByRemoteUrl: vi.fn(() => mockGitHubRepoConfig),
+}));
+const gitLabEventTypeRef = { value: 'Merge Request Hook' };
+const gitHubEventTypeRef = { value: 'pull_request_review' };
+vi.mock('@/security/verifier.js', () => ({
+    verifyGitLabSignature: vi.fn(() => ({ valid: true })),
+    verifyGitHubSignature: vi.fn(() => ({ valid: true })),
+    getGitLabEventType: vi.fn(() => gitLabEventTypeRef.value),
+    getGitHubEventType: vi.fn(() => gitHubEventTypeRef.value),
+}));
+vi.mock('@/frameworks/queue/pQueueAdapter.js', () => ({
+    createJobId: vi.fn((prefix, projectPath, mrNumber) => `${prefix}-${projectPath}-${mrNumber}`),
+    enqueueReview: vi.fn(() => Promise.resolve(true)),
+    updateJobProgress: vi.fn(),
+    cancelJob: vi.fn(),
+}));
+vi.mock('@/claude/invoker.js', () => ({
+    invokeClaudeReview: vi.fn(),
+    sendNotification: vi.fn(),
+}));
+vi.mock('@/main/websocket.js', () => ({
+    startWatchingReviewContext: vi.fn(),
+    stopWatchingReviewContext: vi.fn(),
+}));
+vi.mock('@/config/projectConfig.js', () => ({
+    loadProjectConfig: vi.fn(() => ({ qualityThreshold: 7 })),
+    getProjectAgents: vi.fn(() => null),
+    getProjectAgentsOrFocusDefaults: vi.fn(() => null),
+    getFollowupAgents: vi.fn(() => null),
+    getProjectLanguage: vi.fn(() => 'en'),
+}));
+import { describe, it, expect, beforeEach, afterEach } from 'vitest';
+import { handleGitLabWebhook } from '../../modules/platform-integration/interface-adapters/controllers/webhook/gitlab.controller.js';
+import { handleGitHubWebhook } from '../../modules/platform-integration/interface-adapters/controllers/webhook/github.controller.js';
+import { InMemoryReviewRequestTrackingGateway } from '../../tests/stubs/reviewRequestTracking.stub.js';
+import { StubNoteCommentPostGateway } from '../../tests/stubs/noteCommentPost.stub.js';
+import { StubApprovalRevocationGateway } from '../../tests/stubs/approvalRevocation.stub.js';
+import { TrackedMrFactory } from '../../tests/factories/trackedMr.factory.js';
+import { createStubLogger } from '../../tests/stubs/logger.stub.js';
+import { RecordBypassUseCase } from '../../modules/tracking/usecases/tracking/recordBypass.usecase.js';
+import { RecordReviewCompletionUseCase } from '../../modules/tracking/usecases/tracking/recordReviewCompletion.usecase.js';
+import { TransitionStateUseCase } from '../../modules/tracking/usecases/tracking/transitionState.usecase.js';
+import { TrackAssignmentUseCase } from '../../modules/tracking/usecases/tracking/trackAssignment.usecase.js';
+import { RecordPushUseCase } from '../../modules/tracking/usecases/tracking/recordPush.usecase.js';
+import { CheckFollowupNeededUseCase } from '../../modules/tracking/usecases/tracking/checkFollowupNeeded.usecase.js';
+import { SyncThreadsUseCase } from '../../modules/tracking/usecases/tracking/syncThreads.usecase.js';
+import { HandlePlatformApprovalUseCase } from '../../modules/tracking/usecases/tracking/handlePlatformApproval.usecase.js';
+const PROJECT_PATH = '/home/user/projects/test-project';
+const MR_NUMBER = 42;
+const GITLAB_MR_ID = `gitlab-test-org/test-project-${MR_NUMBER}`;
+const GITHUB_MR_ID = `github-test-org/test-project-${MR_NUMBER}`;
+function buildGitLabApproveEvent() {
+    return {
+        object_kind: 'merge_request',
+        event_type: 'merge_request',
+        user: { username: 'alice', name: 'Alice' },
+        project: {
+            id: 1,
+            name: 'test-project',
+            path_with_namespace: 'test-org/test-project',
+            web_url: 'https://gitlab.com/test-org/test-project',
+            git_http_url: 'https://gitlab.com/test-org/test-project.git',
+        },
+        object_attributes: {
+            iid: MR_NUMBER,
+            title: 'Test MR',
+            state: 'opened',
+            action: 'approved',
+            source_branch: 'feature/test',
+            target_branch: 'main',
+            url: `https://gitlab.com/test-org/test-project/-/merge_requests/${MR_NUMBER}`,
+            draft: false,
+        },
+    };
+}
+function buildGitHubPullRequestReviewEvent() {
+    return {
+        action: 'submitted',
+        review: {
+            id: 12345,
+            state: 'approved',
+            user: { login: 'alice' },
+        },
+        pull_request: {
+            number: MR_NUMBER,
+            state: 'open',
+            html_url: `https://github.com/test-org/test-project/pull/${MR_NUMBER}`,
+        },
+        repository: {
+            full_name: 'test-org/test-project',
+            html_url: 'https://github.com/test-org/test-project',
+            clone_url: 'https://github.com/test-org/test-project.git',
+        },
+        sender: { login: 'alice' },
+    };
+}
+function buildGitLabDeps(tracking, noteCommentPost, approvalRevocation) {
+    const threadFetchGateway = { fetchThreads: vi.fn(() => []) };
+    return {
+        reviewContextGateway: {
+            create: vi.fn(() => ({ success: true, filePath: '' })),
+            read: vi.fn(() => null),
+            delete: vi.fn(() => ({ success: true, deleted: true })),
+            exists: vi.fn(() => false),
+            getFilePath: vi.fn(() => ''),
+            appendAction: vi.fn(() => ({ success: true })),
+            updateProgress: vi.fn(() => ({ success: true })),
+            setResult: vi.fn(() => ({ success: true })),
+        },
+        threadFetchGateway,
+        diffMetadataFetchGateway: { fetchDiffMetadata: vi.fn(() => undefined) },
+        diffStatsFetchGateway: { fetchDiffStats: vi.fn(() => null) },
+        trackAssignment: new TrackAssignmentUseCase(tracking),
+        recordCompletion: new RecordReviewCompletionUseCase(tracking),
+        recordPush: new RecordPushUseCase(tracking),
+        transitionState: new TransitionStateUseCase(tracking),
+        checkFollowupNeeded: new CheckFollowupNeededUseCase(tracking),
+        syncThreads: new SyncThreadsUseCase(tracking, threadFetchGateway),
+        recordBypass: new RecordBypassUseCase(tracking),
+        noteCommentPostGateway: noteCommentPost,
+        handlePlatformApproval: new HandlePlatformApprovalUseCase(tracking),
+        approvalRevocationGateway: approvalRevocation,
+        getQualityThreshold: () => 7,
+        now: () => '2026-05-26T12:00:00.000Z',
+        enforceBudget: {
+            execute: vi.fn(async () => ({
+                accepted: true,
+                status: {
+                    limitUsd: 200,
+                    consumedUsd: 0,
+                    remainingUsd: 200,
+                    percentUsed: 0,
+                    exceeded: false,
+                    periodStart: '2026-05-01T00:00:00.000Z',
+                },
+            })),
+        },
+        broadcastBudgetExceeded: vi.fn(),
+        getRepositories: vi.fn(() => [mockRepoConfig]),
+        removeWorktree: vi.fn(async () => ({ status: 'removed' })),
+    };
+}
+function buildGitHubDeps(tracking, noteCommentPost, approvalRevocation) {
+    const threadFetchGateway = { fetchThreads: vi.fn(() => []) };
+    return {
+        reviewContextGateway: {
+            create: vi.fn(() => ({ success: true, filePath: '' })),
+            read: vi.fn(() => null),
+            delete: vi.fn(() => ({ success: true, deleted: true })),
+            exists: vi.fn(() => false),
+            getFilePath: vi.fn(() => ''),
+            appendAction: vi.fn(() => ({ success: true })),
+            updateProgress: vi.fn(() => ({ success: true })),
+            setResult: vi.fn(() => ({ success: true })),
+        },
+        threadFetchGateway,
+        diffMetadataFetchGateway: { fetchDiffMetadata: vi.fn(() => undefined) },
+        diffStatsFetchGateway: { fetchDiffStats: vi.fn(() => null) },
+        trackAssignment: new TrackAssignmentUseCase(tracking),
+        recordCompletion: new RecordReviewCompletionUseCase(tracking),
+        recordPush: new RecordPushUseCase(tracking),
+        transitionState: new TransitionStateUseCase(tracking),
+        checkFollowupNeeded: new CheckFollowupNeededUseCase(tracking),
+        syncThreads: new SyncThreadsUseCase(tracking, threadFetchGateway),
+        recordBypass: new RecordBypassUseCase(tracking),
+        noteCommentPostGateway: noteCommentPost,
+        handlePlatformApproval: new HandlePlatformApprovalUseCase(tracking),
+        approvalRevocationGateway: approvalRevocation,
+        getQualityThreshold: () => 7,
+        now: () => '2026-05-26T12:00:00.000Z',
+        enforceBudget: {
+            execute: vi.fn(async () => ({
+                accepted: true,
+                status: {
+                    limitUsd: 200,
+                    consumedUsd: 0,
+                    remainingUsd: 200,
+                    percentUsed: 0,
+                    exceeded: false,
+                    periodStart: '2026-05-01T00:00:00.000Z',
+                },
+            })),
+        },
+        broadcastBudgetExceeded: vi.fn(),
+        getRepositories: vi.fn(() => [mockGitHubRepoConfig]),
+        removeWorktree: vi.fn(async () => ({ status: 'removed' })),
+    };
+}
+describe('Acceptance — SPEC-180 Iteration C: Platform approval revoked on non-qualified MR', () => {
+    let tracking;
+    let noteCommentPost;
+    let approvalRevocation;
+    let mockReply;
+    const logger = createStubLogger();
+    beforeEach(() => {
+        vi.clearAllMocks();
+        tracking = new InMemoryReviewRequestTrackingGateway();
+        noteCommentPost = new StubNoteCommentPostGateway();
+        approvalRevocation = new StubApprovalRevocationGateway();
+        mockReply = {
+            status: vi.fn().mockReturnThis(),
+            send: vi.fn().mockReturnThis(),
+        };
+        gitLabEventTypeRef.value = 'Merge Request Hook';
+        gitHubEventTypeRef.value = 'pull_request_review';
+    });
+    afterEach(() => {
+        vi.resetAllMocks();
+    });
+    describe('Rule: platform approval on non-qualified MR is revoked with FR explanation', () => {
+        it('scenario 6 (GitLab) — approved event below threshold revokes approval and posts FR comment', async () => {
+            tracking.create(PROJECT_PATH, TrackedMrFactory.create({
+                id: GITLAB_MR_ID,
+                mrNumber: MR_NUMBER,
+                platform: 'gitlab',
+                project: 'test-org/test-project',
+                state: 'pending-approval',
+                latestScore: 6,
+                openThreads: 0,
+                bypass: null,
+            }));
+            const deps = buildGitLabDeps(tracking, noteCommentPost, approvalRevocation);
+            const request = {
+                body: buildGitLabApproveEvent(),
+                headers: {},
+            };
+            await handleGitLabWebhook(request, mockReply, logger, tracking, deps);
+            expect(approvalRevocation.calls).toHaveLength(1);
+            expect(approvalRevocation.calls[0]).toMatchObject({
+                projectPath: 'test-org/test-project',
+                mrNumber: MR_NUMBER,
+            });
+            expect(noteCommentPost.calls).toHaveLength(1);
+            expect(noteCommentPost.calls[0]).toEqual({
+                projectPath: 'test-org/test-project',
+                mrNumber: MR_NUMBER,
+                body: 'Approbation annulée : seuil qualité 7/10 non atteint (6/10). Utilisez `/bypass-quality "raison"` pour forcer.',
+            });
+            const finalMr = tracking.getById(PROJECT_PATH, GITLAB_MR_ID);
+            expect(finalMr?.state).toBe('pending-approval');
+        });
+        it('scenario 6 (GitHub) — pull_request_review approved below threshold revokes review and posts FR comment', async () => {
+            tracking.create(PROJECT_PATH, TrackedMrFactory.create({
+                id: GITHUB_MR_ID,
+                mrNumber: MR_NUMBER,
+                platform: 'github',
+                project: 'test-org/test-project',
+                state: 'pending-approval',
+                latestScore: 6,
+                openThreads: 0,
+                bypass: null,
+            }));
+            const deps = buildGitHubDeps(tracking, noteCommentPost, approvalRevocation);
+            const request = {
+                body: buildGitHubPullRequestReviewEvent(),
+                headers: {},
+            };
+            await handleGitHubWebhook(request, mockReply, logger, tracking, deps);
+            expect(approvalRevocation.calls).toHaveLength(1);
+            expect(approvalRevocation.calls[0]).toMatchObject({
+                projectPath: 'test-org/test-project',
+                mrNumber: MR_NUMBER,
+                reviewId: 12345,
+            });
+            expect(noteCommentPost.calls).toHaveLength(1);
+            expect(noteCommentPost.calls[0]).toEqual({
+                projectPath: 'test-org/test-project',
+                mrNumber: MR_NUMBER,
+                body: 'Approbation annulée : seuil qualité 7/10 non atteint (6/10). Utilisez `/bypass-quality "raison"` pour forcer.',
+            });
+            const finalMr = tracking.getById(PROJECT_PATH, GITHUB_MR_ID);
+            expect(finalMr?.state).toBe('pending-approval');
+        });
+    });
+});
+//# sourceMappingURL=180-quality-threshold-block-approval-iter-C.acceptance.test.js.map