fix(db): reconcile live database drift and harden security without RAG regression - #281
Closed
BigSimmo wants to merge 0 commit into
Closed
fix(db): reconcile live database drift and harden security without RAG regression#281BigSimmo wants to merge 0 commit into
BigSimmo wants to merge 0 commit into
Conversation
BigSimmo
marked this pull request as ready for review
July 5, 2026 17:14
Contributor
Bugbot is not enabled for your account, so this pull request was not reviewed. Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs. |
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
BigSimmoforce-pushed
the
cursor/fix-all-db-issues-5f13
branch
from
July 5, 2026 17:41
ef6f9ff to
6f1d525CompareBigSimmo
commented
Jul 5, 2026
OwnerAuthor
Closing as largely superseded by #274, #277, and #287 on \main. This branch is also merge-conflicted. If still needed, salvage only the unique pieces onto fresh branches from current \main:
Child PRs #284 and #285 should be retargeted/reopened against \main, not this branch. |
Updates to Preview Branch (cursor/fix-all-db-issues-5f13) ↗︎
Tasks are run on every commit but only new migration files are pushed.
❌ Branch Error • Sun, 05 Jul 2026 18:44:24 UTC View logs for this Workflow Run ↗︎. |
BigSimmoforce-pushed
the
cursor/fix-all-db-issues-5f13
branch
from
July 5, 2026 18:41
60613a3 to
6201117CompareBigSimmo added a commit
that referenced
this pull request
Aug 18, 2026
… favourites timestamps, and answer notice (#309, #281, #339, #165) (#2074) * feat(ui): implement filter density tiers, mobile route deduplication, favourites timestamps, and answer notice (#309, #281, #339, #165) * fix(ui): repair CI blockers on filter density tiers (PR #2074) * fix(ui): keep document-viewer retry panel ids unique under density tiers (PR #2074) * fix(favourites): fix O(N²) storage callbacks and hydration mismatch in useSyncExternalStore Co-authored-by: BigSimmo <87357024+BigSimmo@users.noreply.github.com> * style: fix prettier formatting in favourites command library page CI's format:changed check was failing on this file's useSyncExternalStore formatting. * fix(ui): keep document rail high-yield summary visible when printing `max-sm:hidden` (added to dedupe the summary panel on phone viewports) also suppressed it during print emulation at narrow viewport widths, since the print stylesheet only restores overflow/box-shadow and never overrides display:none. Add `print:block`, the pattern already used elsewhere in this codebase (verification-notice.tsx, medication-considerations.tsx) to keep a responsively-hidden element printable. Fixes the "document viewer content disclosures are naturally closed and mutually exclusive by default" Production UI failure, which set a 390px viewport and asserted the high-yield-summary content becomes visible under print media. * fix(test): update high-yield summary nav assertion for mobile dedup The exclusive-accordion assertions after clicking the "High-yield summary" section-nav row assumed the rail's <details id="source-summary"> still opens on a 390px viewport. Since the mobile route-dedup change hides that rail copy (max-sm:hidden) in favour of the always-visible in-flow DocumentClinicalSummary card, jumpToDocumentSection's existing "displayed copy" resolution (use-section-spy.ts's source-summary -> source-summary-card alias) now scrolls to the card instead of toggling the hidden accordion — so `summary.open` never becomes true at this viewport. Assert the visible copy scrolls into view instead, matching the pattern already used for the "Indexed source text" row above it. --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: Claude <noreply@anthropic.com>
7 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
CI green on clean branch
Branch head:
09b3567c8Issues debugged and fixed
verify6f1d525e3— formatted 4 filesallowGlobalSearchon document lookup args09b3567c8— added optional field tofetchBestDocumentLookupChunksargs typeAuto-hide answer support chips…(caused merge conflicts +desktopBackdropClassNametype error)CI run (manual dispatch)
https://github.com/BigSimmo/Database/actions/runs/28750309235
Live database (already applied)
20260705220000onsjrfecxgysukkwxsowpyindexing-v3-agentedge function deployedMerge note
Branch may still conflict with
mainwhere overlapping access-rollout work landed via other PRs (#291 etc.). Resolve at merge without re-importing unrelated UI commits onto this branch.