Skip to content

Review: URL leak realism (issue #13, takeover of #7) - #8

Open
Lxr-max wants to merge 300 commits into
mainfrom
cursor/fix-url-leak-realism-7f54
Open

Lxr-max wants to merge 300 commits into
mainfrom
cursor/fix-url-leak-realism-7f54

Conversation

@Lxr-max

@Lxr-max Lxr-max commented Sep 19, 2026

Copy link
Copy Markdown
Owner

Superseded by the upstream PR: aiming-lab#140

This fork-local PR was opened by mistake (origin is Lxr-max/WebHarbor). The review takeover for issue aiming-lab#13 lives on aiming-lab#140.

Open in Web Open in Cursor 

boyugou and others added 30 commits May 13, 2026 16:49
Remove instance/drugs_com.db, instance/.gitkeep, scraped_data/.gitkeep,
download_pill_images.py, and .claude/ralph-loop.local.md which should
not be tracked in git per .gitignore rules.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…, professional sidebar

- New GET /drugs/pro/<slug> route with FDA-style professional monograph template
- Interaction checker: support comma-separated ?drugs=drug1,drug2 single-param format
- Drug detail sidebar: Professional resources box linking to pro monograph and sub-pages
- CSS: .professional-resources and .drug-pro-monograph styles in main.css

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…ed drugs row

- News thumbnails: category-specific colored icon tiles (pill, warning, heart, flask, +)
- Drug detail sidebar: pregnancy category badges (A-X), CSA schedule labels, availability badges
- Drug detail: related medications as horizontal scroll-snap card row with rating badges
- My Med List: top stats bar (total/Rx/OTC counts)
- Pill identifier: inline imprint in results count header

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…ref, pricing, homepage

- app.py: DRUG_CONTENT_OVERRIDES before_taking content for 12 key drugs (ibuprofen,
  metformin, alprazolam, amoxicillin, warfarin, atorvastatin, lisinopril, amlodipine,
  sertraline, levothyroxine, naproxen, gabapentin)
- drug_detail.html: show before_taking detail + dosage quick-reference block
- drug_prices.html: CSS classes replacing inline styles for price trend bars
- index.html: Popular searches quick-chip strip (12 common drug names)
- main.css: .before-taking-detail, .dosage-quick-ref, .quick-browse-section, .quick-chip,
  pricing trend bar classes

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Add metformin to _ALCOHOL_BY_GENERIC with moderate lactic-acidosis warning
- When 'alcohol' is entered explicitly, fold lifestyle alcohol interactions into
  the main interactions list (counted in summary, shown in severity groups)
- Template: handle drug_b=None (alcohol interactions) without crashing on slug

Fixes benchmark task 20: metformin+alcohol now returns moderate interaction.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…bar, reviews

- nav: More... dropdown shows on hover with proper z-index and box-shadow
- drug detail: conditions-treated-box sidebar showing linked condition names
- interaction checker: semantic CSS classes for severity groups/rows, summary
  header with prominent count + breakdown badges, colored left-border strips
- drug reviews page: color-coded rating badges, star visualization, Write Review
  button at top, structured review byline
- main.css: ~250 lines of new component styles

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Add SEARCH_ALIASES dict so searching 'antibiotics' also matches
  'antibacterial'/'antimicrobial' — makes fluoroquinolones appear in results
- Include drug class description in score_drug text for better relevance
- Polish drug detail sidebar: status row icons with text glyphs, CSA
  'Not a controlled drug' guard, .status-link/.status-arrow styles
- Expand Pro Edition with 8 resource cards, SVG icons, and a
  'What's Included' feature grid covering monographs/FDA inserts/dosing calculators

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…ases

- Add BLACK BOX WARNING text for lisinopril (fetal toxicity), sertraline
  (suicidal ideation), and warfarin (bleeding risk) in DRUG_CONTENT_OVERRIDES
- Extend SYMPTOM_CONDITION_MAP with General body system (fever/chills)
  and map cough+fever to influenza/bacterial_infections
- Move symptom-checker chip inline styles to .symptom-chip CSS class

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
The pill-identifier.html route now processes imprint/shape/color GET params
directly (same as pill-identifier-results), so ?imprint=I-2 works without
a redirect. Updated form action to submit back to the same canonical URL.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…ts strip

- Add PREGNANCY_CATEGORIES dict mapping ~50 drugs + class fallbacks;
  drug_pregnancy page now shows FDA category badge, risk summary,
  clinical considerations, and trimester-by-trimester guidance
- Rebuild drug_side_effects template: frequency-at-a-glance table,
  color-coded Common/Less-common/Rare/Serious sections, FDA MedWatch
  reporting channels
- Add Quick Facts compact strip on drug detail page (availability,
  pregnancy category, CSA, drug class, rating)
- Limit reviews preview to first 3 with 'See all N reviews' CTA
- Fix similar-drug cards to always show rating or 'No ratings' pill

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Add Similarities/Differences side-by-side block below comparison table
- Update Popular Comparisons to benchmark-relevant drug pairs
- CSS for new .compare-simdiff section (green/red accent columns)

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…, footer

- Add pricing CTA callout block with 'Compare prices' link
- Upgrade inline interaction checker widget to gradient card with icon
- Expand 'Further Information' to 5-item list with poison-control number
  and extended medical disclaimer
- New CSS: .pricing-cta, .interaction-mini-card, .further-info-list

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Add DRUG_CLASS_DESCRIPTIONS_EXTENDED with overview/uses/side-effects
  for benzodiazepines, SSRIs, statins, ACE inhibitors
- drug_class_page route passes class_overview/uses/side_effects to template
- drug_class.html restructured with What-are/What-used-for/Side-effects sections
- Hero search autocomplete dropdown wired to /api/autocomplete
- Featured drug cards render pill SVG image with brand/rating
- Dosage sub-page gains section-tabs row with active 'Dosage' tab

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Add /<slug>/reviews/new dedicated write-review page with star selector,
  sub-ratings (effectiveness/ease-of-use/satisfaction), condition datalist
- Add 'Reviews by condition' chip filter to reviews listing page
- Route all 'Write a Review' buttons to the new dedicated page
- Add 'Seek immediate medical help' amber callout to drug_warnings page
- Navy-bordered 'Before taking' card styling on warnings page

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Remove inline style= from sev_badge macro in drug_interactions_page.html
  (rely on existing .sev-badge.sev-{major,moderate,minor} CSS rules)
- Update dosage page breadcrumb from 'Dosage' to 'Dosage Guide'

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Add enalapril, ramipril, captopril, benazepril to ACE inhibitors drug class.
Previously had only lisinopril; now 5 ACE inhibitors give a richer class page.
New instance_seed DB md5: ecc823a29cab1f7170d0dcbdc918adbb (uploaded to HF).

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Fill DMARDs, Tetracyclines, Macrolides, Triptans, SNRIs, Thyroid hormones,
Bronchodilators, Diuretics, Sleep aids classes to 3+ drugs each.
instance_seed updated (md5: 34dd2ed272f05841cedfc8573987010d).

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…ection

Move conditions from bottom of Uses section to a prominent callout box
at the top with .condition-chip pill links. Rename heading to
'What conditions is X used to treat?'.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…ements

- Pill identifier: replace dropdowns with visual shape chips (SVG icons)
  and color swatches; add instructions text; add clear/reset button
- Pill results: add 'Identify this pill' CTA button per card
- Search: add matched condition/class card with 'View all results' link;
  add 'Popular for query' sidebar with top drugs for matched entity
- Search: restructured drug cards with badge chip row

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…ent overrides

- Add extended descriptions for NSAIDs, Beta blockers, PPIs, Fluoroquinolones, Opioids
  drug class pages, matching depth and quality of existing benzo/SSRI/statin/ACE entries
- Fix ciprofloxacin page: OpenFDA had seeded ophthalmic formulation data; add runtime
  override with correct oral systemic content (indications, boxed warning, dosage)
- Add levofloxacin runtime content override with systemic formulation data
- Extend runtime override mechanism: drug_detail route now applies DRUG_CONTENT_OVERRIDES
  for uses/warnings/dosage/side_effects/description at render time (not just before_taking),
  so DB fixes no longer require re-seeding
- Fix drug class slug alias map: aliases now point to actual DB slugs (statins, ssris, etc.)
  instead of non-existent long-form slugs; NSAID alias still maps short → full as needed
- Add /compare/<slug1>-vs-<slug2> route that redirects to /compare?drug1=&drug2= format

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Adds specific before_taking guidance for: semaglutide, omeprazole, aspirin,
acetaminophen, metoprolol, tramadol, hydrocodone, oxycodone, furosemide,
levofloxacin, and fluoxetine. Also adds a complete semaglutide entry with
accurate description, uses, warnings, side_effects, and dosage reflecting
all three brand forms (Ozempic, Wegovy, Rybelsus).

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…ental()

Adds 16 new conditions (OCD, PTSD, panic disorder, social anxiety, PMDD,
neuropathic pain, UTI, pneumonia, edema, etc.) and maps them to relevant
drugs. Also patches conditions_json on Drug rows so the drug detail page
reflects all mapped conditions. seed_supplemental() is idempotent — runs
every boot outside the seed_database() gate so it backfills data without
breaking byte-identity when the DB is already fully populated.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Rewrite score_drug() to prioritize exact generic name matches (score +20),
partial name matches (+10), brand name matches (+8), and body text mentions
(+1). Previously all matches had equal weight of 1, causing drugs that
merely mention another drug's name in their description to outrank the
target drug itself.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Adds detailed overview, uses, and side_effects entries in
DRUG_CLASS_DESCRIPTIONS_EXTENDED for: SNRIs, GLP-1 receptor agonists,
SGLT2 inhibitors, ARBs, and Anticonvulsants. Each drug class page now
renders comprehensive clinical information matching the real drugs.com
drug class pages.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Add _COND_DISPLAY dict and _humanize_cond() to map slug keys (heart_disease,
  high_cholesterol, etc.) to readable names for use in review body templates
- Apply humanization in both seed_reviews and seed_extra_reviews
- Offset seed_extra_reviews template index by 5 to avoid producing reviews
  that duplicate benchmark-user reviews for popular drugs

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
QianhuiWu and others added 30 commits September 18, 2026 11:49
Integrate Cookpad PRs aiming-lab#52 and aiming-lab#78 with authentic recipes, reviewed fixes, and merged HF assets
…igslist

Integrate Craigslist #5aiming-lab#60aiming-lab#103 with authentic assets and reviewed fixes
Integrate Drugs.com #9/aiming-lab#71: sourced DailyMed assets, task and grading fixes
Review: Add Versus mirror + task verifiers (site by @Sun-sunshine06, verifiers by reviewer) (aiming-lab#41)
…binator

Review: rebuild Y Combinator mirror from PR aiming-lab#31
Salvage stale PR #7 against current main for issue aiming-lab#13. Share/copy boxes,
hidden next fields, and GitHub host recovery no longer leak localhost or
example.com placeholders. Relative next redirects are validated, and a
regression checker plus unit tests cover the known leak classes.

Co-authored-by: Xuanrui Li <xuanrui.li@se24.qmul.ac.uk>
Sign up for free to 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.