prreviewbuddy 0.87.2 → 0.90.0

package/static/shell.css

@@ -90,6 +90,22 @@
   --neutral-subtle: light-dark(rgba(15, 23, 42, 0.06), rgba(139, 148, 158, 0.16));
   --merged-subtle:  light-dark(#f3e8ff, rgba(163, 113, 247, 0.18));
   --merged-ink:     light-dark(#6b21a8, #d2a8ff);
+  /* A review's own lifecycle, what happens next with it: Active, Waiting, Done. Named apart from
+     --lifecycle-*, which is a pull request's, so the two cannot be swapped by accident. Done is a
+     muted slate rather than purple, because purple already means the brand, Merged and branch news. */
+  --state-active-subtle:  light-dark(#eaf2fd, rgba(56, 139, 253, 0.14));
+  --state-active-ink:     light-dark(#0b4fae, #9cc4ff);
+  --state-active-border:  light-dark(rgba(11, 79, 174, 0.3), rgba(56, 139, 253, 0.45));
+  --state-waiting-subtle: light-dark(#fff5d6, rgba(210, 153, 34, 0.16));
+  --state-waiting-ink:    light-dark(#7a5200, #ffd18a);
+  --state-waiting-border: light-dark(rgba(138, 92, 0, 0.32), rgba(210, 153, 34, 0.5));
+  --state-done-subtle:    light-dark(#e8ecf1, rgba(148, 163, 184, 0.16));
+  --state-done-ink:       light-dark(#334155, #cbd5e1);
+  --state-done-border:    light-dark(rgba(51, 65, 85, 0.32), rgba(148, 163, 184, 0.45));
+  /* The pills' widths, the widest of their words (NOT REVIEWED, WAITING): a press never moves what
+     is beside them, and a card can reserve exactly the room its controls take. */
+  --status-pill-width:    104px;
+  --lifecycle-pill-width: 70px;
   /* A pull request's lifecycle in Activity: merged, closed, reopened. Blue because purple there
      means unread, and a merged tile must never be read as a new one. */
   --lifecycle-subtle: light-dark(#e7f0fd, rgba(56, 139, 253, 0.16));
@@ -408,6 +424,8 @@ body > * { flex: 0 0 auto; }
    a fixed-width box hanging below the header, and its items have to wrap inside it. */
 .prb-header-stats > :not(.prb-menu) { white-space: nowrap; }
 .prb-header-stats .prb-add { color: var(--success); }
+/* The two pills and the change chip read as one pair of facts, closer to each other than to the stats. */
+.prb-state-pills { display: inline-flex; align-items: center; gap: 6px; }
 .prb-header-stats .prb-del { color: var(--danger); }
 
 /* The build, and where to complain about it. Quieter than the change stats — it is provenance,
@@ -2830,6 +2848,18 @@ a.prb-rail-item { text-decoration: none; }
 /* A notice the reviewer can put away: the sentence, then its Dismiss at the end of the same row. */
 .prb-job[data-notice] { flex-direction: row; align-items: center; justify-content: space-between; gap: 16px; }
 .prb-job p { margin: 0; }
+/* The AI integrations notice: an invitation rather than a status, so it carries the brand edge and
+   its link sits before Not now, both at the end of the row. */
+.prb-integrations-notice { border-left-color: var(--brand); }
+.prb-integrations-notice p { flex: 1; }
+.prb-integrations-notice-link {
+  flex: none;
+  color: var(--brand-ink);
+  font-size: var(--t-meta);
+  font-weight: 600;
+  text-decoration: none;
+}
+.prb-integrations-notice-link:hover { opacity: 0.8; }
 .prb-job strong { color: var(--text-primary); font-weight: 600; }
 .prb-job-progress { color: var(--text-muted); }
 .prb-job-elapsed, .prb-job-note { color: var(--text-muted); font-size: var(--t-micro); }
@@ -3519,12 +3549,17 @@ a.prb-guide-pick { text-decoration: none; }
 .prb-index-more[hidden] { display: none; }
 .prb-index-more:hover { color: var(--text-primary); border-color: var(--border-strong); }
 .prb-card {
+  /* What the controls over the top-right corner take: two pills, the menu and the star, and the
+     gaps between them. The head reserves exactly this, so the title is cut only where they are. */
+  --card-controls-width: calc(var(--status-pill-width) + var(--lifecycle-pill-width) + 26px + 26px + 3 * 6px);
   position: relative;
+  container-type: inline-size;
   border: 1px solid var(--border);
   border-radius: 12px;
   background: var(--surface-raised);
 }
-.prb-card:hover { border-color: var(--border-strong); }
+/* A stronger edge and the faintest change of ground, and nothing that moves. */
+.prb-card:hover { border-color: var(--border-strong); background: color-mix(in srgb, var(--surface-raised), var(--text-primary) 2%); }
 .prb-card[hidden] { display: none; }
 
 /* ── Selecting ──
@@ -3768,10 +3803,9 @@ a.prb-guide-pick { text-decoration: none; }
   align-items: baseline;
   justify-content: space-between;
   gap: 12px;
-  /* Room for the pin and the menu, both positioned over the card's top-right corner. Reserved
-     here rather than left to chance: the status badge is the last thing in this row, and without
-     the space it is what the controls sit on top of. */
-  padding-right: 56px;
+  /* Room for the controls positioned over the card's top-right corner. Reserved here rather than
+     left to chance: without it the PR state and the change chip are what the controls sit on. */
+  padding-right: calc(var(--card-controls-width) + 6px);
 }
 .prb-card-head h2 {
   /* The auto margin takes the free space, so the status and the PR state sit together at the end
@@ -3847,12 +3881,26 @@ a.prb-guide-pick { text-decoration: none; }
 
 /* News about the change rather than a verdict on it: tinted like the brand, never like a risk.
    It sits in the meta line with the other facts, because that is what it is. */
-.prb-card-moved {
-  color: var(--brand);
+.prb-card-moved,
+.prb-card-badge {
+  padding: 1px 7px;
+  border-radius: 999px;
   font-weight: 600;
+  white-space: nowrap;
 }
+.prb-card-moved { background: var(--brand-muted); color: var(--brand-ink); }
 .prb-card-diffstat { font-variant-numeric: tabular-nums; }
-.prb-card-left { color: var(--text-secondary); }
+/* What is left, as facts: a badge only where there is an outcome to see at a glance (resolved and
+   answered, or still open), plain words for the rest, "Nothing flagged" among them. */
+.prb-card-left {
+  display: inline-flex;
+  flex-wrap: wrap;
+  align-items: baseline;
+  gap: 8px;
+  color: var(--text-secondary);
+}
+.prb-card-badge[data-tone="success"] { background: var(--success-subtle); color: var(--success-ink); }
+.prb-card-badge[data-tone="attention"] { background: var(--warning-subtle); color: var(--warning-ink); }
 
 /* What is happening to this review right now.
    A dot that breathes, because the one thing this line has to communicate before it is read is
@@ -4045,6 +4093,51 @@ a.prb-guide-pick { text-decoration: none; }
 .prb-card-status { border: 1px solid var(--border); color: var(--text-tertiary, var(--text-secondary)); }
 .prb-card-status[data-status="new"] { border-color: var(--brand); color: var(--brand-ink); }
 .prb-card-status[data-status="complete"] { border-color: var(--success-border); color: var(--success-ink); }
+
+/* ── Status and lifecycle pills ──
+   Where a review stands, on a card and in the review's header, and the control that changes it
+   (state_pills.ts). Outlined on a faint tint of their own colour, where the facts beside them
+   (Merged, risk, branch news) are filled and borderless, so a control and a fact never look alike.
+   Each kind has one width whatever its words, so pressing one never moves what is beside it. */
+.prb-state-pill {
+  box-sizing: border-box;
+  display: inline-flex;
+  align-items: center;
+  justify-content: center;
+  gap: 4px;
+  flex: none;
+  padding: 2px 8px;
+  border: 1px solid var(--border-strong);
+  border-radius: 999px;
+  background: var(--neutral-subtle);
+  color: var(--text-secondary);
+  font: inherit;
+  font-size: var(--t-pill);
+  font-weight: 700;
+  line-height: 1.4;
+  letter-spacing: 0.04em;
+  text-transform: uppercase;
+  white-space: nowrap;
+}
+.prb-state-pill.prb-card-status { min-width: var(--status-pill-width); }
+.prb-lifecycle-pill { min-width: var(--lifecycle-pill-width); }
+.prb-state-pill[data-status="new"] { border-color: var(--brand-border); background: var(--brand-muted); color: var(--brand-ink); }
+.prb-state-pill[data-status="complete"] { border-color: var(--success-border); background: var(--success-subtle); color: var(--success-ink); }
+/* A mark as well as a colour, so Reviewed is not told apart by colour alone. Silent to a screen
+   reader, which hears the word. */
+.prb-state-pill[data-status="complete"]::before { content: '✓' / ''; }
+.prb-lifecycle-pill[data-lifecycle-pill="active"] { border-color: var(--state-active-border); background: var(--state-active-subtle); color: var(--state-active-ink); }
+.prb-lifecycle-pill[data-lifecycle-pill="waiting"] { border-color: var(--state-waiting-border); background: var(--state-waiting-subtle); color: var(--state-waiting-ink); }
+.prb-lifecycle-pill[data-lifecycle-pill="done"] { border-color: var(--state-done-border); background: var(--state-done-subtle); color: var(--state-done-ink); }
+button.prb-state-pill { cursor: pointer; }
+/* Pressable, said quietly: the outline takes the word's colour and the ground darkens a step. */
+button.prb-state-pill:not([aria-busy="true"]):hover {
+  border-color: color-mix(in srgb, currentColor 55%, transparent);
+  box-shadow: inset 0 0 0 999px var(--wash-strong);
+}
+button.prb-state-pill:focus-visible { outline: 2px solid var(--brand-hover); outline-offset: 2px; }
+button.prb-state-pill[aria-busy="true"] { opacity: 0.6; cursor: progress; }
+.prb-pill-menu .prb-menu-panel { top: 28px; width: 200px; }
 /* The upstream's state, beside the reviewer's. Filled where the status is outlined, so the two read
    as different kinds of fact. Merged takes the forge's own purple; closed is muted rather than red,
    because red on this card already means high risk. */
@@ -4061,19 +4154,42 @@ a.prb-guide-pick { text-decoration: none; }
   padding: 2px 7px;
   border-radius: 999px;
   font-size: var(--t-meta);
-  font-weight: 600;
+  font-weight: 700;
   white-space: nowrap;
   background: var(--brand-muted);
   color: var(--brand-ink);
+  /* A step louder than Branch updated, because a parked review that moved is the thing to find when
+     scanning a long list. An inset edge, so it is no bigger than the pills beside it. */
+  box-shadow: inset 0 0 0 1px var(--brand-border);
 }
 .prb-card-lifecycle[hidden] { display: none; }
 
-/* Over the card rather than inside its link, so pinning cannot be a mis-click into the review. */
-.prb-card-pin {
+/* The card's controls, over its top-right corner rather than inside its link, so no press on them
+   can be a mis-click into the review. No z-index of its own: each control keeps the layer it had
+   when the pin and the menu sat here alone, the pin and the pills under any open panel above them
+   (a collection's menu, the card's above), the menu over them. Waiting's items rise while open. */
+.prb-card-controls {
   position: absolute;
   top: 10px;
   right: 10px;
-  z-index: 1;
+  display: flex;
+  align-items: center;
+  gap: 6px;
+}
+.prb-card-controls > .prb-state-pill,
+.prb-card-controls > .prb-card-pin { position: relative; z-index: 1; }
+.prb-card-controls > .prb-menu { position: relative; top: auto; right: auto; }
+.prb-card-controls > .prb-pill-menu { z-index: 1; }
+.prb-card-controls > .prb-pill-menu:has(> [aria-expanded="true"]) { z-index: 3; }
+/* Where the title would be cut short, the controls take a line of their own under it instead, at
+   the title's own left edge. The title is one line, so the line it leaves is a fixed height. */
+@container (max-width: 560px) {
+  .prb-card[data-card] .prb-card-head { padding-right: 0; margin-bottom: 38px; }
+  .prb-card-controls { top: 42px; right: auto; left: 16px; }
+  .prb-index[data-selecting] .prb-card-controls { left: 42px; }
+}
+
+.prb-card-pin {
   width: 26px;
   height: 26px;
   border: 0;
@@ -4222,18 +4338,6 @@ a.prb-guide-pick { text-decoration: none; }
   line-height: var(--lh-body);
 }
 
-/* The review page's lifecycle pill is the status pill's shape. Changed since it was parked, it
-   reads as the card's chip does: a sentence, tinted like news. */
-.prb-page-lifecycle[data-changed] {
-  border-color: transparent;
-  background: var(--brand-muted);
-  color: var(--brand-ink);
-  font-size: var(--t-meta);
-  font-weight: 600;
-  letter-spacing: 0;
-  text-transform: none;
-}
-.prb-page-lifecycle[hidden] { display: none; }
 
 /* ── Diagrams ──
  *
@@ -5791,6 +5895,89 @@ select.prb-addreview-control {
 }
 .prb-settings-help code { font-size: var(--t-code); color: var(--text-primary); }
 
+/* ── AI integrations ──
+   A list inside Settings, one row per coding agent. Each row's state is a word in a pill and the
+   colour only agrees with it, as the save line does. Buttons here act at once, so the primary one
+   is filled and the rest are outlined like Stop server, never the danger red: Disconnect is
+   recoverable with one press of Connect. */
+#ai-integrations { scroll-margin-top: 96px; }
+.prb-integrations > .prb-settings-help + .prb-settings-help { margin-top: 10px; }
+.prb-integrations-list { list-style: none; margin: 14px 0; padding: 0; display: flex; flex-direction: column; gap: 10px; }
+.prb-integration {
+  padding: 14px 16px;
+  border: 1px solid var(--border);
+  border-radius: 10px;
+  background: var(--surface-2);
+  display: flex;
+  flex-direction: column;
+  gap: 8px;
+}
+.prb-integration[aria-busy='true'] { opacity: 0.8; }
+.prb-integration-head { display: flex; align-items: center; gap: 10px; flex-wrap: wrap; }
+.prb-integration-name { font-size: var(--t-body); font-weight: 600; color: var(--text-primary); }
+.prb-integration-state {
+  padding: 2px 9px;
+  border: 1px solid var(--border);
+  border-radius: 999px;
+  font-size: var(--t-meta);
+  color: var(--text-secondary);
+}
+.prb-integration-state[data-state='connected'] { color: var(--success); border-color: var(--success); }
+.prb-integration-state[data-state='connected-elsewhere'],
+.prb-integration-state[data-state='disabled'],
+.prb-integration-state[data-state='unknown'] { color: var(--warning); border-color: var(--warning); }
+.prb-integration-actions { display: flex; flex-wrap: wrap; gap: 8px; }
+.prb-integration-actions:empty { display: none; }
+.prb-integration-button,
+.prb-integration-primary {
+  padding: 7px 14px;
+  border: 1px solid var(--border);
+  border-radius: 8px;
+  background: transparent;
+  color: var(--text-primary);
+  font: inherit;
+  font-size: var(--t-body);
+  cursor: pointer;
+}
+.prb-integration-button:hover { border-color: var(--border-strong); }
+.prb-integration-primary { border-color: var(--brand); background: var(--brand); color: var(--on-brand); }
+.prb-integration-primary:hover { background: var(--brand-hover); border-color: var(--brand-hover); }
+.prb-integration-button:disabled,
+.prb-integration-primary:disabled { opacity: 0.6; cursor: default; }
+.prb-integration-button:focus-visible,
+.prb-integration-primary:focus-visible { outline: 2px solid var(--brand-hover); outline-offset: 2px; }
+.prb-integration-result { margin: 0; font-size: var(--t-meta); line-height: var(--lh-body); color: var(--text-secondary); }
+.prb-integration-result:empty { display: none; }
+.prb-integration-result[data-kind='ok'] { color: var(--success); }
+.prb-integration-result[data-kind='error'] { color: var(--danger); }
+.prb-integration pre,
+.prb-integration-dialog pre {
+  margin: 6px 0;
+  padding: 8px 10px;
+  border: 1px solid var(--border);
+  border-radius: 8px;
+  background: var(--surface);
+  white-space: pre-wrap;
+  word-break: break-all;
+  font-size: var(--t-code);
+  color: var(--text-primary);
+}
+.prb-integration-manual > summary,
+.prb-integrations-scope > summary {
+  cursor: pointer;
+  font-size: var(--t-meta);
+  color: var(--text-secondary);
+}
+.prb-integration-manual > summary:hover,
+.prb-integrations-scope > summary:hover { color: var(--text-primary); }
+.prb-integration-manual[open] > summary,
+.prb-integrations-scope[open] > summary { margin-bottom: 6px; }
+.prb-integration-manual .prb-settings-field { margin: 8px 0; }
+.prb-integrations-scope { margin-top: 6px; }
+.prb-integrations-scope a { color: var(--brand-ink); text-decoration: none; }
+.prb-integrations-scope a:hover { opacity: 0.8; }
+.prb-integration-dialog { max-width: 560px; }
+
 /* ── Help ──
    Reading, not settings: prose at a readable measure, one topic per section, each with an anchor a
    place elsewhere in the product can link to. */