mirror of
https://github.com/multica-ai/multica.git
synced 2026-08-12 10:59:06 +02:00
e9dab0022e
Replaces the row + badge layout under "Pull requests" on the issue
detail sidebar with a card that mirrors the GitHub PR summary look:
title, author/avatar, +N −M · K files diff stats, segmented progress
bar (failed → pending → passed, failure leftmost), and a one-line
status caption following an explicit priority pass-through.
Backend
- Migration 092: github_pull_request adds additions / deletions /
changed_files (INT NOT NULL DEFAULT 0). Zero defaults are what the
new frontend treats as "legacy backend — hide the stats row" so old
PR rows that pre-date this migration don't render "+0 −0 · 0 files".
- pull_request webhook handler reads stats off the top-level payload.
- ListPullRequestsByIssue now surfaces per-suite counts
(checks_passed / failed / pending) alongside the existing aggregate
conclusion, so the segmented bar reuses the already-computed counts
with no new aggregation.
Frontend (packages)
- core/github/pull-request-status.{ts,test.ts}: pure-function module
for the status-kind priority table and the segment derivation; 15
cases covered, includes the "all-zero → hide stats" guard.
- views/issues/components/pull-request-list.tsx: PullRequestCard plus
a compact-row fallback used when count > 4 (first 3 as cards, the
remainder collapsed behind a Show more toggle).
- i18n: new `pull_request_card_*` keys in en + zh-Hans.
Tests
- 12 component tests covering each rule of the priority table, the
legacy-zero stats fallback, and the collapse threshold.
- Reuse of the v3 webhook handler tests confirmed.
Verification
- pnpm typecheck + pnpm test green (60 test files, 536 tests).
- go build ./... + go vet ./... clean.
- 6 demo issues (DEV-2..DEV-7) screenshotted via Playwright; see the
PR comments for the visual check matrix.
Co-authored-by: multica-agent <github@multica.ai>
13 lines
701 B
SQL
13 lines
701 B
SQL
-- PR diff stats (additions / deletions / changed_files) for the card layout.
|
||
-- Source: top-level `pull_request` object on every pull_request webhook event
|
||
-- (opened / synchronize / edited / labeled / ...). NOT NULL DEFAULT 0 means
|
||
-- legacy rows that pre-date this migration read as zero, which the frontend
|
||
-- detects via `total === 0` and hides the entire stats row — so the card never
|
||
-- renders a misleading "+0 −0 · 0 files" caption for rows that just haven't
|
||
-- been refreshed by a webhook yet.
|
||
|
||
ALTER TABLE github_pull_request
|
||
ADD COLUMN additions INT NOT NULL DEFAULT 0,
|
||
ADD COLUMN deletions INT NOT NULL DEFAULT 0,
|
||
ADD COLUMN changed_files INT NOT NULL DEFAULT 0;
|