@@ -1201,8 +1202,16 @@
Dependency graph (feat_ + infra_)
class infra_e2e_seed_completed_study done;
chore_form_dropdown_primitive["form dropdown primitive"]
class chore_form_dropdown_primitive done;
+ chore_ci_prettier_check["ci prettier check"]
+ class chore_ci_prettier_check done;
chore_data_table_columnvisibility_tanstack["data table columnvisibility tanstack"]
class chore_data_table_columnvisibility_tanstack done;
+ chore_detail_page_shell_primitive["detail page shell primitive"]
+ class chore_detail_page_shell_primitive done;
+ chore_extract_shadcn_select_test_mock["extract shadcn select test mock"]
+ class chore_extract_shadcn_select_test_mock done;
+ chore_form_dropdown_guide_screenshot_refresh["form dropdown guide screenshot refresh"]
+ class chore_form_dropdown_guide_screenshot_refresh done;
infra_e2e_wire_seed_helper_into_studies_spec["e2e wire seed helper into studies spec"]
class infra_e2e_wire_seed_helper_into_studies_spec done;
feat_study_lifecycle --> feat_digest_proposal
@@ -1364,8 +1373,16 @@ Dependency graph (feat_ + infra_)
class infra_e2e_seed_completed_study done;
chore_form_dropdown_primitive["form dropdown primitive"]
class chore_form_dropdown_primitive done;
+ chore_ci_prettier_check["ci prettier check"]
+ class chore_ci_prettier_check done;
chore_data_table_columnvisibility_tanstack["data table columnvisibility tanstack"]
class chore_data_table_columnvisibility_tanstack done;
+ chore_detail_page_shell_primitive["detail page shell primitive"]
+ class chore_detail_page_shell_primitive done;
+ chore_extract_shadcn_select_test_mock["extract shadcn select test mock"]
+ class chore_extract_shadcn_select_test_mock done;
+ chore_form_dropdown_guide_screenshot_refresh["form dropdown guide screenshot refresh"]
+ class chore_form_dropdown_guide_screenshot_refresh done;
infra_e2e_wire_seed_helper_into_studies_spec["e2e wire seed helper into studies spec"]
class infra_e2e_wire_seed_helper_into_studies_spec done;
feat_study_lifecycle --> feat_digest_proposal
diff --git a/state.md b/state.md
index f1b12dbc..27cde82b 100644
--- a/state.md
+++ b/state.md
@@ -2,13 +2,13 @@
> Read this first. Snapshots the active branch, what just shipped, what's in flight, what's queued, and where the project currently sits in the MVP1 → GA roadmap. Updated whenever a feature lands or a priority shifts.
-**Last updated:** 2026-05-19 (after `chore_data_table_columnvisibility_tanstack` shipped via `/impl-execute --ad-hoc` as PR #150 squash commit `c1e4545` — closes the residual DataTable follow-ups: item 5 migrates the primitive from `columns.filter(...)` to TanStack's `state.columnVisibility` API (memoized per Gemini feedback), item 3 locked the flat-prop `DataTableProps` API as canonical with a "Shipped contract addendum" on the historical implementation plan's Story 2.6. Folder renamed `chore_data_table_primitive_followups` → `chore_data_table_columnvisibility_tanstack`. Earlier 2026-05-19 PR #148 (`infra_e2e_wire_seed_helper_into_studies_spec`, squash `65f4150`) — restored the 2 digest-panel E2E tests deferred from PR #130, diagnosed and fixed the real root cause of the original smoke-lane failure (`GET /api/v1/proposals` was silently ignoring the `?study_id=` filter, returning the most-recent global pending proposal), added 5-case integration regression coverage at `backend/tests/integration/test_proposals_study_filter.py`. Plus: (a) earlier 2026-05-18 PR #146 (`bug_install_skip_ui_rebuild`, squash `7299fca`) made `make up` rebuild every Compose service (`docker compose build` no-args), switched `make down` to `docker compose down`, and added a `verify_install_builds_all_services.sh` CI gate to lock the contract; (b) earlier 2026-05-18 PR #147 captured `chore_detail_page_shell_primitive` idea (squash `8854e47`). Two new follow-ups filed: `chore_ci_prettier_check` (CI's frontend job has no `prettier --check` step — surfaced when PR #136 drift in 2 unrelated files blocked an unrelated commit) and the in-flight `chore_detail_page_shell_primitive` (third primitive after DataTable + EntitySelect).)
+**Last updated:** 2026-05-19 (after a 4-PR shipping run drained the actionable post-MVP1 chore backlog: PR #152 `chore_ci_prettier_check` (`476db78`) + PR #153 `chore_extract_shadcn_select_test_mock` (`199e225`) + PR #154 `chore_form_dropdown_guide_screenshot_refresh` (`ed4121f`) + PR #155 `chore_detail_page_shell_primitive` (`9a72514`). PR #155 is the third primitive after `` and `` — 6 detail-page migrations + new lint guard + flattens a latent UX bug where only `proposals/[id]` discriminated 404 from network error. Earlier the same session: PR #150 (`chore_data_table_columnvisibility_tanstack`, `c1e4545`) — closes the residual DataTable follow-ups: item 5 migrates the primitive from `columns.filter(...)` to TanStack's `state.columnVisibility` API (memoized per Gemini feedback), item 3 locked the flat-prop `DataTableProps` API as canonical with a "Shipped contract addendum" on the historical implementation plan's Story 2.6. Folder renamed `chore_data_table_primitive_followups` → `chore_data_table_columnvisibility_tanstack`. Earlier 2026-05-19 PR #148 (`infra_e2e_wire_seed_helper_into_studies_spec`, squash `65f4150`) — restored the 2 digest-panel E2E tests deferred from PR #130, diagnosed and fixed the real root cause of the original smoke-lane failure (`GET /api/v1/proposals` was silently ignoring the `?study_id=` filter, returning the most-recent global pending proposal), added 5-case integration regression coverage at `backend/tests/integration/test_proposals_study_filter.py`. Plus: (a) earlier 2026-05-18 PR #146 (`bug_install_skip_ui_rebuild`, squash `7299fca`) made `make up` rebuild every Compose service (`docker compose build` no-args), switched `make down` to `docker compose down`, and added a `verify_install_builds_all_services.sh` CI gate to lock the contract; (b) earlier 2026-05-18 PR #147 captured `chore_detail_page_shell_primitive` idea (squash `8854e47`). Two new follow-ups filed: `chore_ci_prettier_check` (CI's frontend job has no `prettier --check` step — surfaced when PR #136 drift in 2 unrelated files blocked an unrelated commit) and the in-flight `chore_detail_page_shell_primitive` (third primitive after DataTable + EntitySelect).)
---
## Current branch / execution context
-- **Branch:** `main` (post-merge of PR #150 squash commit `c1e4545` 2026-05-19). Finalization docs PR off `docs/finalize-data-table-columnvisibility-tanstack`. Prior squashes: PR #148 `65f4150` 2026-05-19 (`infra_e2e_wire_seed_helper_into_studies_spec` — `?study_id=` filter bug + E2E test restore), PR #147 `8854e47` 2026-05-18 (capture chore_detail_page_shell_primitive idea), PR #146 `7299fca` 2026-05-18 (bug_install_skip_ui_rebuild — `make up`/`make down` lifecycle fix), PR #136 `cb7d9ee` 2026-05-18 (chore_form_dropdown_primitive), PR #132 `ee4c8d4` 2026-05-17 (chore_data_table_primitive_followups items 1+2+4+6), PR #130 `13b3383` 2026-05-17 (infra_e2e_seed_completed_study), PR #128 `73459d2` 2026-05-17 (bug_cursor_decode_value_validation), PR #126 `d6115b3` 2026-05-16 (feat_data_table_primitive). `v0.1.0` annotated tag still on `main` commit `d099536` 2026-05-13; GitHub Release at https://github.com/SoundMindsAI/relyloop/releases/tag/v0.1.0.
+- **Branch:** `main` (post-merge of PR #155 squash commit `9a72514` 2026-05-19). Finalization docs PR off `docs/finalize-may19-chore-batch`. Prior squashes: PR #154 `ed4121f` 2026-05-19 (`chore_form_dropdown_guide_screenshot_refresh`), PR #153 `199e225` 2026-05-19 (`chore_extract_shadcn_select_test_mock`), PR #152 `476db78` 2026-05-19 (`chore_ci_prettier_check`), PR #151 `110dc5a` 2026-05-19 (finalize chore_data_table_columnvisibility_tanstack), PR #150 `c1e4545` 2026-05-19 (`chore_data_table_columnvisibility_tanstack`), PR #149 `da9506b` 2026-05-19 (finalize infra_e2e_wire_seed_helper_into_studies_spec), PR #148 `65f4150` 2026-05-19 (`infra_e2e_wire_seed_helper_into_studies_spec` — `?study_id=` filter bug + E2E test restore), PR #147 `8854e47` 2026-05-18 (capture chore_detail_page_shell_primitive idea), PR #146 `7299fca` 2026-05-18 (bug_install_skip_ui_rebuild — `make up`/`make down` lifecycle fix), PR #136 `cb7d9ee` 2026-05-18 (chore_form_dropdown_primitive), PR #132 `ee4c8d4` 2026-05-17 (chore_data_table_primitive_followups items 1+2+4+6), PR #130 `13b3383` 2026-05-17 (infra_e2e_seed_completed_study), PR #128 `73459d2` 2026-05-17 (bug_cursor_decode_value_validation), PR #126 `d6115b3` 2026-05-16 (feat_data_table_primitive). `v0.1.0` annotated tag still on `main` commit `d099536` 2026-05-13; GitHub Release at https://github.com/SoundMindsAI/relyloop/releases/tag/v0.1.0.
- **Active feature:** none in flight. **Three PRs shipped 2026-05-15:** PR #122 (Phase 1, 16th MVP1 feature — Tooltip primitive + 26 placements on create-study modal + study detail), PR #123 (Phase 1 finalization docs), PR #124 (Phases 2 + 3 — 17th MVP1 feature; 21 additional tooltips on judgments + proposals + cluster registration + 2 new first-run components: chat ExamplePrompts strip + Stripe-style StartHereChecklist on home page). The original "MVP1 Phase 1 only" scope-lock was reversed mid-day: operator decided to ship Phases 2 + 3 together with a Stripe-style design call rather than wait for MVP2. PR #124 took 2 hours from idea-folder reuse to merge. 47 total tooltip placements + 2 new first-run components live in `main`. **PR #122 shipped 2026-05-15 morning** — `feat_contextual_help` Phase 1 (16th MVP1 feature). Adds the first Tooltip primitive (`@radix-ui/react-tooltip@~1.2.8` + shadcn-style wrapper at `ui/src/components/ui/tooltip.tsx`), two glossary-backed wrappers (`InfoTooltip` standalone + asChild modes; `HelpPopover` click-to-open with `react-markdown` safety filter), and a 49-key glossary source-of-truth at `ui/src/lib/glossary.ts` (8 enum groups parity-tested against `enums.ts`). 26 tooltip placements across the create-study modal (Step 1 target + Step 3 template + 9 Step 5 inputs), study-header (status badge dynamic key + Best metric + Trials), trials-table (5 column headers + Sort label), and digest panel (5 section labels + Open PR enabled + Open PR disabled). The disabled Open PR button refactored from native `disabled` to `aria-disabled="true"` so it stays focusable and the tooltip reveals on focus (AC-11). Gemini Code Assist: 2 findings (1 accepted + fixed, 1 rejected with cited counter-evidence). Final GPT-5.5 review: 1 Medium accepted-framing-but-deferred. Spec converged at GPT-5.5 cycle 3 (24 findings, 23 accepted + 1 rejected); plan converged at cycle 2 (12 findings, 10 accepted + 1 rejected + 1 spec patch). UI vitest now **279 passing across 48 files** (was 249 across 45 — +3 new test files, +30 cases). Playwright E2E **8 passing** (was 5 — +3 new contextual-help tests). One follow-up filed: `infra_e2e_seed_completed_study/idea.md` tracks the E2E gap for digest-panel triggers + AC-11 (cross-subsystem helper for seeding a completed study with digest + proposal; component-level coverage is in place). Phases 2 + 3 deferred to MVP2 via `feat_contextual_help_mvp2/` (judgments + proposals tooltips; chat + cluster + home onboarding; the home-page "Start here" panel is the only product-design-shaped item).
**Earlier — seven PRs shipped 2026-05-14:** `feat_judgments_periodic_resume_sweep` (PR #104, 14th MVP1 feature), `bug_query_inline_crud_since_filter_uuidv7_ms_collision` (PR #106 — UUIDv7 ms-collision test flake), `infra_dashboard_regen_pre_commit_conflict §2+§4` (PR #108 — dashboard regen idempotency + relative-link rewriting), `infra_make_targets_split_backend_only` (PR #110 — `make backend-fmt/lint/typecheck` + symmetric `ui-fmt` so Node-18 contributors aren't blocked), `chore_digest_worker_narrow_except` (PR #112 — narrowed `except Exception` allowlist to `(ValueError,)` + ERROR-level `digest_importance_failed_unexpected` event), `infra_structlog_test_helpers` (PR #114 — factored the two structlog test-assertion patterns into `backend/tests/_log_helpers.py`), and `chore_chat_last_message_preview` (PR #117 — `last_message_preview` + `last_message_at` on `ConversationSummary` via LATERAL JOIN; frontend shows preview under title + swaps displayed timestamp from `created_at` to `last_message_at`). Plus PR #116 dropped `chore_studies_ui_shadcn_polish` as won't-do (forward-compat audit on NavigationMenu primitive + ClusterFilterSelect precedent on native `` for page-level controls). The 7-PR day drained the operational-friction surfaces and the last `/bug-fix`-shaped item. **MVP1 alpha + 17 features shipped (all 3 phases of feat_contextual_help on 2026-05-15) + 26 backlog items drained.** Manual maintainer steps still pending from [`release-checklist.md`](docs/03_runbooks/release-checklist.md): §4 fresh-VM hosted-OpenAI walkthrough, §5 local-LLM walkthrough, §8 feedback Discussion + design-partner channel shares. **Remaining MVP1 backlog: zero actionable items** — the MVP1 dashboard now reads `Path to MVP1: 0`. Two ideas held for MVP2 (visible on the new `MVP2_DASHBOARD.md`): `infra_arq_subprocess_test_mvp2` (folder renamed 2026-05-14 to mirror the `_mvp2` precedent — trigger-locked: arq pin bump, 3rd cron, or MVP3 hardening opt-in) and `bug_chat_long_conversation_truncation_mvp2` (paired with the just-shipped chat preview but kept on MVP2 hold per its preflight — needs `/pipeline` for migration + new LLM round-trip). Plus 4 keep-deferred items by operator decision + `infra_dashboard_regen_pre_commit_conflict §3` follow-up (runbook addendum, MVP1-eligible but low-leverage).
@@ -19,6 +19,14 @@
## Most recent meaningful changes (newest first)
+- **2026-05-19 — `chore_detail_page_shell_primitive` merged into `main`** as PR #155 (squash commit `9a72514`). Shipped via `/impl-execute --ad-hoc` against the preflighted idea (user switched from full `/pipeline` after surfacing the over-ceremonialization tradeoff). Third primitive in the extraction pattern after `` (PR #126) and `` (PR #136). 6 of 7 detail routes (`clusters/[id]`, `studies/[id]`, `proposals/[id]`, `query-sets/[id]`, `templates/[id]`, `judgments/[id]`; `chat/[id]` excluded as structurally different — stream-rendered conversation, not a card scaffold) hand-rolled the same `isPending → isError → data` ternary with identical className strings and slightly inconsistent copy ("deleted" vs "removed") — and only `proposals/[id]` discriminated 404 from network error. Extracted into `>` at `ui/src/components/common/detail-page-shell.tsx`. **Latent UX bug fixed:** all 6 pages now discriminate `errorCode === _NOT_FOUND` from other errors via the primitive's built-in routing — pre-PR, 5 of 6 pages collapsed both into "{Entity} not found" even when the API was just down. **Primitive shape:** `query: UseQueryResult`, `entityLabel: string` (singular), `notFoundErrorCode: string`, optional `entityTitle` / `notFoundMessage` / `unreachableMessage` overrides, children-as-function (Q2 locked default). **Lint guard** at `ui/src/__tests__/components/common/detail-page-shell-discipline.test.tsx` walks `src/app//[id]/page.tsx`, fails on hand-rolled `isPending? + isError?` ternaries without a `` import; escape hatch `// detail-page-shell-allow: `. **Copy normalization:** "may have been deleted." adopted as canonical (4 of 6 pre-migration sites used it); templates/[id] + judgments/[id] migrated from "removed". **Gemini Code Assist:** 1 Medium accepted (`query.data === undefined` → `query.data == null` for defense against 200-OK-null body, commit `9f93a12`). **Tests:** UI vitest 399 → 413 (+14 — 8 primitive + 6 lint-guard self-tests). All migrated pages' existing test files pass unchanged (copy was preserved character-for-character). **Docs:** ui-architecture.md gains a "Detail page shell primitive" section after "Form dropdown primitive" + updates the existing "Modal-level testing" paragraph to cite the now-shared shadcn-select-mock helper from PR #153. **Process win:** locked Q1 in the in-PR-#150 preflight rather than carrying as an open question; Q2 + Q3 recommended-defaults applied without re-asking.
+
+- **2026-05-19 — `chore_form_dropdown_guide_screenshot_refresh` merged into `main`** as PR #154 (squash commit `ed4121f`). Shipped via `/guide-gen --regen` on guide 01, then bulk Playwright run for guides 04, 06, 09. Regenerated 20 PNGs across 4 walkthroughs against the post-PR-#136 EntitySelect UI: `01_register_first_cluster` (Config repo field now always-visible with empty-state CTA), `04_create_query_set` (Cluster picker is `` with health-status dot, label renamed "Cluster ID" → "Cluster"), `06_create_and_monitor_study` (Step-1 cluster picker uses `` with "Choose a cluster" placeholder), `09_generate_judgments_llm` (Template picker dropdown shows versioned options via ``). **Test spec fix in 04_create_query_set.spec.ts:** swapped `option.click()` → `option.dispatchEvent('click')` with `exact: true` after Gemini review, because polluted dev DB (200+ accumulated clusters) puts the Radix popover's option outside the viewport even with `force: true`; synthetic click bypasses both viewport + actionability checks. **Gemini:** 1 Medium accepted (exact:true addition). Pass-1 visual audit done; Pass-2 GPT-5.5 cross-model skipped per the "implement-now" rubric (screenshot refresh validating a UX change already verified by Pass-1 + the existing E2E suite).
+
+- **2026-05-19 — `chore_extract_shadcn_select_test_mock` merged into `main`** as PR #153 (squash commit `199e225`). Shipped via `/impl-execute --ad-hoc`. The Radix-based shadcn `` family crashes inside jsdom + Dialog focus traps (testing-library's `patchedFocus` shim recurses through Radix's internal focus-trap). Three modal test files (`clusters/register-cluster-modal.test.tsx`, `query-sets/create-query-set-modal.test.tsx`, `studies/create-study-modal.test.tsx`) duplicated the same ~50-LOC native-`` shim inline. New shared helper at `ui/src/__tests__/helpers/shadcn-select-mock.tsx` exports `mockShadcnSelect: () => Promise<...>`; each test file invokes via a 3-line dynamic-`import()` inside `vi.mock` that sidesteps vitest's hoisting rule (earlier non-dynamic refactors failed because vi.mock hoists above top-level imports). Used the "richer" mock variant (data-testid forwarding + disabled prop handling) as the helper — superset of the studies inline mock, doesn't affect its assertions. Net -65 LOC of duplicated scaffolding. **Gemini:** 2 Medium accepted (single-pass `findTriggerProps` returning both props, reference-equality check on `child.type === SelectTrigger` for minifier-robustness). **CI fight:** smoke lane flaked once on `OpenAI capability check structured_output='fail'` — external/infra issue (same `OPENAI_API_KEY_TEST` passed smoke on PR #150 + #152 within 30 minutes; a test-mock refactor has no path to affecting backend `/healthz` capability detection). Re-ran failed job → passed.
+
+- **2026-05-19 — `chore_ci_prettier_check` merged into `main`** as PR #152 (squash commit `476db78`). Shipped via `/impl-execute --ad-hoc`. 12-LOC YAML addition to `.github/workflows/pr.yml` adds a "Format check (prettier)" step before ESLint in the frontend job. Argument list mirrors the pre-commit `prettier-ui` hook exactly: `pnpm --dir ui exec prettier --check src package.json tsconfig.json eslint.config.mjs .prettierrc.json`. Closes the gap surfaced during PR #148 where 2 unrelated PR #136 files (`entity-select.tsx`, `form-select-discipline.test.tsx`) had drifted past `prettier@3.8.3` formatting and the drift survived to a follow-up commit because CI's frontend job had no `prettier --check` step. The pre-commit hook is local-only; any path that bypasses it (`--no-verify`, hosted edits, fresh checkout without `pre-commit install`, agent harnesses that invoke `git` without the husky shim) was letting drift through. **Gemini:** zero line comments.
+
- **2026-05-19 — `chore_data_table_columnvisibility_tanstack` merged into `main`** as PR #150 (squash commit `c1e4545`). Shipped via `/impl-execute --ad-hoc`. Closes the residual DataTable follow-ups from PR #126's review cycles (items 1, 2, 4, 6 had landed in PR #132 2026-05-17). **Item 5 — TanStack `state.columnVisibility` migration:** `` previously pre-filtered `columns` via `columns.filter(c => c.sticky || c.hideable === false || !hiddenColumns.has(c.id))` before passing to `useReactTable`. Refactored to pass the full `columns` array + `state: { columnVisibility }` derived from the same `useLocalStorageSet`-backed `hiddenColumns`. Memoized via `useMemo([columns, hiddenColumns.value])` per Gemini feedback for stable identity across re-renders. No `onColumnVisibilityChange` handler — `useLocalStorageSet` remains the single source of truth for persisted visibility (one-way flow). Behavioral contract unchanged: sticky/non-hideable force-show, menu rendering, localStorage persistence, sort/filter/selection/keyboard nav/density all identical. **Item 3 — flat-prop API lock:** the implemented plan's Story 2.6 sketched a `urlState: DataTableUrlState` + `setSort`/`setFilter`/... aggregate-prop shape, but the build-out shipped 10 flat optional props on `DataTableProps` (sort? / onSortChange? / filters? / onFilterChange? / q? / onQChange? / cursor? / onCursorChange? / onPageSizeChange? / sortCodec?). Locked the flat-prop API as canonical (idiomatic React + shadcn-aligned + 8 thin-wrapper consumers depend on it); historical `implemented_features/2026_05_16_feat_data_table_primitive/implementation_plan.md` Story 2.6 gained a "Shipped contract addendum" block at the top with back-reference. **Folder rename:** `chore_data_table_primitive_followups` → `chore_data_table_columnvisibility_tanstack` (4 tokens, intent-clear after the residual scope narrowed to item 5 only); 6 live-code-comment citations in `ui/src/__tests__/helpers/data-table-url-mock.ts`, `ui/src/__tests__/hooks/use-data-table-url-state.test.tsx`, `ui/src/hooks/use-data-table-url-state.ts`, `ui/src/hooks/use-local-storage-set.ts` updated to the new slug. Historical state.md entries + `implemented_features/*` docs intentionally left unchanged (correct historical record of PR #132's name-at-the-time). **Gemini Code Assist:** 1 Medium finding, accepted — wrapped `columnVisibility` derivation in `useMemo` (initially rejected citing state.md PR #136's "React Compiler ESLint rule" precedent, then verified by reading `ui/eslint.config.mjs` that the rule isn't actually configured — flat config loads only `next/core-web-vitals` + `eslint-plugin-security`). state.md PR #136 entry annotated inline with the correction so future agents don't lean on the stale precedent. **CI:** 3 push cycles before final merge (initial + Gemini-fix + state.md inline-correction force-push). **Tests:** UI vitest 399 passing (unchanged — Item 5 is a structural refactor, all existing behavioral tests pass against the new code). Backend unchanged. **Process win:** inline-fixed the React Compiler precedent in state.md rather than capturing as `chore_state_md_react_compiler_precedent` idea, per CLAUDE.md's "default lean: implement-now" rubric. Force-pushed to drop the capture commit, kept the PR at 5 commits with the correction landed.
- **2026-05-19 — `infra_e2e_wire_seed_helper_into_studies_spec` merged into `main`** as PR #148 (squash commit `65f4150`). Shipped via `/impl-execute --ad-hoc`. Resumes the work deferred from PR #130 (which dropped 2 digest-panel E2E tests after the smoke lane failed for an undiagnosed reason). **The "undiagnosed Playwright failure" was actually a real backend bug:** `GET /api/v1/proposals` accepted `?study_id=` from the frontend (`useProposalForStudy` has been issuing it since PR #126) but the endpoint never declared the parameter — FastAPI silently ignored it and returned the most-recent global pending proposal regardless of study. The study-detail page reads that result to decide between the Open PR (enabled) and aria-disabled branches; with the filter silently broken, both restored tests flaked depending on what stray proposals lived in the smoke runner's DB. **Fix:** `study_id: str | None` added to `list_proposals_paginated` + `count_proposals` at `backend/app/db/repo/proposal.py`; endpoint declares `study_id: Annotated[UUID | None, Query()] = None` and plumbs it through. Invalid UUIDs return 422 via FastAPI parsing, matching the existing `template_id` filter shape. **Regression coverage:** new `backend/tests/integration/test_proposals_study_filter.py` (5 cases mirroring `test_proposals_template_filter.py`) including the explicit regression assertion that a nonexistent study_id returns empty (pre-fix returned the global pending pool). **E2E test fixes:** both restored tests originally asserted `tooltip-trigger-digest.open_pr_*` testids that `InfoTooltip` in `asChild` mode explicitly does NOT set (per `info-tooltip.tsx:49-50` — a DOM node can carry only one `data-testid` and the caller's wins). Replaced with the canonical hover → `tooltip-body-*` visibility pattern from the existing `InfoTooltip reveals on hover and ESC dismisses` test in the same file. **Pre-existing PR #136 prettier drift in 2 unrelated UI files** (`entity-select.tsx`, `form-select-discipline.test.tsx`) surfaced when the pre-commit `prettier-ui` hook ran on a follow-up commit. Pure formatting, inline-fixed in a separate `style:` commit; root cause captured as new `chore_ci_prettier_check` idea — CI's frontend job lacks a `prettier --check` step, so drift survives any path that bypasses the pre-commit hook. **Gemini Code Assist:** zero line comments, "I have no feedback to provide." **CI fight:** 4 push cycles before green — (1) initial: backend coverage lane failed because the new integration test helper had stale repo signatures (used `optimizer_config` instead of `config`, hand-typed kwargs vs. canonical test pattern); (2) helper realigned to match `test_pagination.py:60-77`; (3) NOT NULL violation on `rubric` column — `test_pagination.py` predates the calibration column additions, but `test_fts_endpoints.py:104` had the up-to-date pattern (rubric + status + failed_reason + calibration); (4) green. **Smoke lane (the load-bearing check)** passed cleanly with the restored E2E tests on the second push onward — proves the original PR #130 failure is genuinely resolved. **Process gap:** the failure mode where the smoke runner's stale DB state caused divergent results between local (clean DB) and CI (polluted with prior pending proposals) only surfaces because of the proposals-filter bug — fix locks the divergence shut. **Follow-ups filed:** `chore_ci_prettier_check` (CI prettier-check gate missing from `pr.yml` frontend job).