Review: Compass environment, source-backed data and task verification - #84
Review: Compass environment, source-backed data and task verification#84jackjin1997 wants to merge 23 commits into
Conversation
Adds a Flask mirror of compass.com as the 16th WebHarbor site, with browse / search / filter, listing detail, agent directory, account flows (save, tour, inquiry, saved search, collection), and 18 WebVoyager-format benchmark tasks. sites/compass/: - app.py (1011 lines): 10 SQLAlchemy models, 35+ routes, token-overlap scored search with city/state/neighborhood boosts. User.check_password accepts both pbkdf2 and bcrypt prefixes so seed-time PBKDF2 hashes (deterministic) coexist with runtime Flask-Bcrypt writes. - seed_data.py (659 lines): idempotent function-level gates; PBKDF2 with fixed per-email salt to preserve byte-identical reset; Co-op pool backfilled to keep filter-based tasks at >=5 candidates. - 33 Jinja templates + 327-line hand-rolled CSS (white/black/serif to match the real Compass palette). - tasks.jsonl: 18 WebVoyager tasks (3 hard multi-step). - listings_clean.json: 524 normalized listings consumed by seed_data at build time (committed alongside the mirror, per the convention used by booking/, arxiv/, etc.). Registration (3 files, must stay in sync per AGENTS.md): - websyn_start.sh: compass appended to SITES, two ready-count 15s -> 16. - control_server.py: 'compass' appended to SITES. - Dockerfile: EXPOSE 8101 40000-40015. Heavy assets (instance_seed/compass.db, static/images/, ~129 MB packed) ship via the companion HuggingFace PR ChilleD/WebHarbor#3. .assets-revision already pins main, so once that merges this Just Works. Byte-identical reset verified: md5sum instance/compass.db instance_seed/compass.db -> 2a7458e3b6c3e3d0b39c32cca5d0f519 (both files). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Independent blind first-pass review (Compass, 16 tasks)Reviewer model: Claude Fable 5.1 ( Scope
Result: 16/16 PASS, 0 FAIL (Compass--0 to 7, 10 to 17). No failed task IDs. Method
Interpretive assumption worth cross-checking: listings with Not performed / not certified
Structured per-task rationale and evidence are in the frozen verdict file held by the reviewer. |
Independent-review reconciliationRechecked the frozen verdict hash and all 618 input-file hashes. All 16 task verdicts agree with deterministic scoring. The original Claude verdict remains unchanged.
Updated report and evidence boundaries. This follow-up changes documentation only and reuses the frozen review candidate |
|
Addressed the human-reported missing Sell section in Four original / before / after comparison groups and remaining differences · Checks and image/input hashes
Validation: 42 application/source tests pass, existing Buy/Rent and login/navigation work in the real browser, all 20 homepages return 200, and Compass reset is byte-identical. The incremental Docker build and owner runtime have the same six changed file hashes; updating the owner environment preserved its database and left the other 19 processes untouched. HF #53 now pins |
|
Homepage/navigation correction pushed in Current checks: 168 tests PASS, 91 local homepage URLs/assets, 20/20 site homepages, matching 29-file incremental image, and byte-identical Compass reset. Compass--17 was newly recorded at the final application commit: 8 steps / 16 screenshots, deterministic PASS and unchanged DB. Its refreshed independent review is pending; the older 16-task review is not relabeled as coverage of this UI. HF #53 now pins |
|
Follow-up: the actual 815px viewport exposed remaining structural defects in The updated source/before/after report adds 20 verified viewport images (52 retained total), final responsive and runtime checks, asset pin |
|
Concierge and Neighborhood Guides reconstruction is now in 26 source / before / after screenshots, measured sections and remaining boundaries · Validation and hashes Validation: 168 tests passed during the repair; final browser checks exercise the case controls and local CTA. The final incremental image matches 47 runtime files; all 20 homepages and 69 local routes/media URLs return 200, and Compass reset restores the seed byte-for-byte. HF #53 now pins Human visual/scope acceptance and the refreshed Compass--17 independent verdict remain pending. Existing frozen executions keep their original versions; these new page captures are UI QA evidence. PR remains Draft. |



Reviewer takeover of Compass contribution #25 by Peichun Hua (@sarendis56), with 16 candidate tasks/verifiers. Fixed source-backed listing data and local account workflows are retained. Environment, task and verifier audit.
The homepage/navigation now follows the source structure and responsive layout, including listing/promotion placement, original photographs, services, community banners and complete menu groups. The new Concierge page replaces its one-panel placeholder with the source video hero, marketing sections, eight case states, before/after photographs, services, process and FAQs. Neighborhood Guides now includes all 24 source regions in the original order and unequal grid layout.
Concierge/directory original–before–after report includes 26 versioned screenshots at 1440, 815 and 390px plus deeper sections. Homepage/navigation report retains its 52 versioned screenshots. The directory comparison below uses 1440×900, loaded fonts and scroll 0:
Current application:
25b8b3375aeb5a3b2c5823f8ed6d52a05eddeccf. During this repair, 168 application/source/verifier tests passed; subsequent typography, anchor and explicit-link refinements were checked through browser operations and an HTTP sweep. A fresh incremental image matched all 47 changed runtime files; 20/20 site homepages and 69 local routes/media URLs returned 200. Compass reset restored its runtime seed byte-for-byte in 0.722s. Eight case states, wrap, mobile controls and the local agent CTA were exercised. This does not repeat the clean full asset-fetch/reset-all check. Validation and screenshot manifest.Task/verifier evidence is reused by explicit impact/hash checks: 63 protected files remain byte-identical to 82b5915, and only the neighborhood directory handler changed. The original 16-task/268-screenshot execution packet and reconciled Claude verdict retain their earlier identities. Compass--17 was freshly executed at 9741284/e063074: 8 real steps, 16 screenshots, deterministic PASS and no DB changes. Its independent frozen-execution verdict is pending. The new landing-page QA does not count as new benchmark trajectories or current-UI Claude acceptance.
Assets: HF PR #53, revision
b09ad95ac9428d5f98e1dc6206f1fa5f87aec671. Archive 192,209,798 bytes, SHA-256ced1411f5d86992ee07a55b98c3b8660ec548f204e0602c1a90d0fe0436f4b98. This phase adds 38 original media/font/SVG files; all 2,430 prior archive files including seed and the other 19 site archives are unchanged.Draft: human visual/interaction acceptance, confirmation of the disclosed local-page boundaries, and the refreshed execution’s independent verdict remain pending. Full regional editorial guides are not reproduced: six existing regional links retain local listing snapshots, while 18 explicitly open the source. Concierge contact/financing and YouTube case films use explicit external links; its local CTA opens the existing agent directory. Sell remains an introduction without the original lead form/full workflow; live-map, New Development and corporate/financial services retain disclosed limits. These omissions are not answer-leak safeguards. Fixed data, synthetic accounts and anti-answer-leak differences are preserved separately. Maintainers coordinate final code/asset merges.