Add FedEx WebHarbor mirror - #50
Conversation
Review — Add FedEx WebHarbor mirror (#50)Verdict: Request changes (one blocker), then Approve. The code is correct and the full Mechanical
Visual fidelity: PASSCorrect FedEx purple branding, nav, and a full tracking timeline (screenshot: Functional depth: PASSTracking shows the delay reason; support search resolves the exception article; Task quality: 18/18 PASSAll entities verified present (5 tracking numbers, 4 seeded users, named locations/articles). Required before approval: upload the FedEx asset tarball to HF + bump |
|
Claiming this PR for the end-to-end WebHarbor review completion pass. I will build from this branch, rerun the human environment and task checks, resolve the HF asset and revision blocker, update the tasks where needed, add deterministic verifiers and judge rubrics for all 18 tasks, validate them against agent trajectories and an independent fresh strong-model review, then open a new Review: fedex PR and link it here. Please avoid duplicating this review while I work on it. |
Summary
Implementation summary
This PR adds a self-contained FedEx-style logistics mirror under
sites/fedex/with Flask, SQLAlchemy, SQLite, Jinja templates, local CSS/JS, deterministic SVG visuals, seeded tracking timelines, rate estimates, shipment creation, pickup scheduling, invoices, claims, support content, and account flows.Selected WebSyn port:
40015Seed row counts
Benchmark users
TestPass123!Task count and categories
sites/fedex/tasks.jsonlKey routes
/,/home/track,/track/results,/tracking/<tracking_number>/rate-estimate/ship,/ship/service,/ship/review,/ship/confirmation/pickup/locations,/locations/<location_slug>/support,/support/<article_slug>/search/login,/register,/logout/account,/account/edit,/account/shipments,/invoices,/claimsVerification commands and results
py -3 -m py_compile sites/fedex/app.py sites/fedex/seed_data.py sites/fedex/_health.py-> passedpy -3 sites/fedex/seed_data.py-> generated deterministicinstance_seed/fedex.dband local SVG assetsDocker result
docker build -t webharbor:fedex-pr .-> passedwebharbor-fedex-submit8301 -> 8101,43000-43015 -> 40000-40015Smoke test result
200on/,/home,/track,/track/results,/rate-estimate,/locations,/support,/search?q=delay,/loginSH-260061, trackingFDX260000061, pickupPU-2609Reset + MD5 result
POST http://localhost:8301/reset/fedex->ready: truedocker exec webharbor-fedex-submit md5sum /opt/WebSyn/fedex/instance/fedex.db /opt/WebSyn/fedex/instance_seed/fedex.db-> both hashes matchedea65ea7d0ec6f17af8229806ed8c4317200Screenshot paths
Local review-only screenshots were captured at:
sites/fedex/scraped_data/submission_review/home.pngsites/fedex/scraped_data/submission_review/track.pngsites/fedex/scraped_data/submission_review/track_results.pngsites/fedex/scraped_data/submission_review/tracking_detail.pngsites/fedex/scraped_data/submission_review/rate_estimate.pngsites/fedex/scraped_data/submission_review/locations.pngsites/fedex/scraped_data/submission_review/support.pngsites/fedex/scraped_data/submission_review/login.pngsites/fedex/scraped_data/submission_review/mobile_home.pngHF asset status
hf upload --create-pr:.assets-revisionwas not modified in this PR and should stay unchanged until the HF asset change is merged/pinned.Known limitations
scripts/check_assets.sh/scripts/fetch_assets.shuse CRLF-sensitive bash entrypoints in this Windows checkout, so I followed the documented asset workflow with equivalent manualhf/tarcommands in PowerShell when needed.Manual review notes