Skip to content

[codex] Add Recreation.gov mirror#8

Open
Chubi-alt wants to merge 1 commit into
aiming-lab:mainfrom
Chubi-alt:feat/recreation-gov
Open

[codex] Add Recreation.gov mirror#8
Chubi-alt wants to merge 1 commit into
aiming-lab:mainfrom
Chubi-alt:feat/recreation-gov

Conversation

@Chubi-alt
Copy link
Copy Markdown

Summary

Adds a self-contained recreation_gov WebHarbor mirror for https://www.recreation.gov/.

  • Implements the Recreation.gov-style Flask app with search, category browsing, detail pages, OpenStreetMap-backed maps, auth, account creation, saved locations, cart, checkout, reservations, cancellation, reviews, Help Center, articles, and state/popular-location browsing.
  • Registers recreation_gov as the next site in websyn_start.sh, control_server.py, and Dockerfile on port 40015.
  • Adds 20 WebVoyager-schema tasks covering search/comparison, help/articles, account creation, authenticated state changes, checkout, cancellation, review submission, and booking/cart flows.
  • Adds GUIDANCE.md documenting the WebHarbor clone workflow and acceptance requirements.

Seed / Asset Summary

Validation

  • python3 -m py_compile sites/recreation_gov/app.py sites/recreation_gov/seed_data.py
  • sites/recreation_gov/tasks.jsonl parsed successfully: 20 tasks, unique IDs
  • Local task verification: all 20 tasks passed after per-task reset
  • Local reset: POST /reset/recreation_gov returned {"pid":3218,"ready":true,"site":"recreation_gov"}
  • Local DB md5 matched: e5e9ce886f044fe7e789b0b15ccecbd7 for both runtime and seed DB
  • ./scripts/build.sh webharbor:dev
  • Docker smoke test: all sites alive, recreation_gov pages returned 200
  • Docker reset: POST /reset/recreation_gov returned {"pid":172,"ready":true,"site":"recreation_gov"}
  • Docker DB md5 matched: e5e9ce886f044fe7e789b0b15ccecbd7 for both runtime and seed DB

Reviewer Notes

This PR is draft until the HF asset PR is merged and .assets-revision can be bumped to the merged dataset revision. Screenshot comparison assets can be attached after final review if required.

@Chubi-alt Chubi-alt marked this pull request as ready for review May 17, 2026 02:48
hqhq1025 pushed a commit to hqhq1025/WebHarbor that referenced this pull request May 26, 2026
Conflicts resolved:
- websyn_start.sh / control_server.py: append recreation_gov after imdb.
- Dockerfile EXPOSE 40000-40019 → 40000-40020.
- .gitignore: keep main's full set, accept PR's comment reformatting.

Note: recreation_gov ships with a pre-built db (HF refs/pr/8), so the
werkzeug generate_password_hash() in seed_data.py does NOT cycle on
reset. set_password is only hit on live registration. No bcrypt-pin
fix needed here.
hqhq1025 pushed a commit to hqhq1025/WebHarbor that referenced this pull request May 27, 2026
…tch learnings

verify-site-gui:
- PER_SLUG examples expanded with 5 P0 sites (eventbrite/smartasset/apartments_com/mayo_clinic/fandom)
  showing the _url_overrides pattern for multi-placeholder URLs (Avoids "amanda-dixon" pollution)
- Add 2 new false-positive categories (aiming-lab#8 template:null download endpoints, aiming-lab#9 designed-404 fallback pages)
  + corresponding harness filter logic
- Add 2 new real-bug categories: §5 byte-identical reset broken by bcrypt salt / hash() / utcnow non-determinism
  (fandom dc5b676 fix pattern); §6 circular import in python3 app.py standalone (apartments 850465d fix)
- Document max_pages CLI 3rd arg (python3 _verify_playwright.py mayo_clinic 50034 80 to cover full 77 pages)
- Document agent-dispatch traps: no SendMessage tool; worktree baseRef defaults to origin so misses unpushed work

document-site-gui:
- Add explicit "incremental Write/Edit mandatory" rule with 4-step procedure
- MD 50-130 KB + YAML 100-200 KB targets cannot fit a single Write tool call (32k output token cap)
- Prior agents hit this and stopped at "let me write the file" with nothing written
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.

1 participant