Skip to content

Remove redundant section header from differentials Presentations/Diagnoses streams - #2049

Merged
BigSimmo merged 2 commits into
mainfrom
claude/remove-clinical-entries-text-0vtmot
Aug 17, 2026
Merged

Remove redundant section header from differentials Presentations/Diagnoses streams#2049
BigSimmo merged 2 commits into
mainfrom
claude/remove-clinical-entries-text-0vtmot

Conversation

@BigSimmo

Copy link
Copy Markdown
Owner

Summary

  • Removed the "Presentation pathways" / "Clinical entries" <h2> heading and its subtitle from DifferentialStreamWorkspace (src/components/differentials/differential-stream-workspace.tsx) so result cards render directly below the search results band on both /differentials/presentations and /differentials/diagnoses, instead of behind a duplicate label.
  • Dropped the now-unused entriesLabel field from the per-stream copy table.
  • Updated the DOM test and the Playwright route-coverage spec that asserted on the removed heading text.

Verification

  • npm run typecheck — clean
  • npx eslint src/components/differentials/differential-stream-workspace.tsx tests/differential-stream-page.dom.test.tsx tests/ui-route-coverage.spec.ts — clean
  • npx prettier --check <changed files> — "All matched files use Prettier code style!"
  • node scripts/run-vitest.mjs run tests/differential-stream-page.dom.test.tsx — 7 passed (7)
  • npm run verify:pr-local — not run: this is a localized, non-behavioural text/heading removal confined to one component's section header (no shared chrome/composer/routing/data change); the focused DOM test plus typecheck/lint above cover the plausible regression surface for this diff.
  • npm run verify:ui — not run for the same reason; no shared search-chrome, composer, or cross-page UI foundation was touched, only a page-local <h2>/subtitle removed. Playwright's ui-route-coverage.spec.ts assertion on this heading was updated to match, but the full Chromium gate was not run.

Risk and rollout

  • Risk: Very low — pure UI copy/markup removal (a redundant duplicate section label) with no logic, data, or routing change.
  • Rollback: Revert this commit; the removed heading/subtitle strings are preserved in the diff.
  • Provider or production effects: None.

Notes

  • This diff does not touch ingestion, answer generation, search/ranking, source rendering, document access, privacy, production env, or clinical output, so the Clinical Governance Preflight section is not applicable.

Generated by Claude Code

…noses streams
Drop the "Presentation pathways"/"Clinical entries" heading and its
subtitle so result cards render directly below the search results
band instead of behind a duplicate label.
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

@supabase

supabaseBot commented Aug 17, 2026

Copy link
Copy Markdown

This pull request has been ignored for the connected project sjrfecxgysukkwxsowpy because there are no changes detected in supabase directory. You can change this behaviour in Project Integrations Settings ↗︎.


Preview Branches by Supabase.
Learn more about Supabase Branching ↗︎.

@coderabbitai

Copy link
Copy Markdown
Contributor

Caution

Review failed

An error occurred during the review process. Please try again later.


Comment @coderabbitai help to get the list of available commands.

@github-actions

Copy link
Copy Markdown
Contributor

CI triage

CI failed on this PR. Automated classification of the 2 failed job(s):

  • Safety and config checksneeds investigation: inspect the failing step and uploaded diagnostics; rerun only after classifying the cause.
  • PR requiredneeds investigation: inspect the failing step and uploaded diagnostics; rerun only after classifying the cause.

Compared with main CI run #11455 (success).

Classification is evidence routing, not permission to ignore a failure. Exact quarantined Playwright identities remain governed by the flake ledger.

@BigSimmo
BigSimmo merged commit de00524 into mainAug 17, 2026
43 of 45 checks passed
@BigSimmo
BigSimmo deleted the claude/remove-clinical-entries-text-0vtmot branch August 17, 2026 16:48
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@BigSimmo@claude