Skip to content

feat(book-gig): add surrounding town mappings for Pembroke, Pulaski, Wirtz, and Huddleston - #873

Draft
JoshuaVSherman wants to merge 1 commit into
devfrom
agy/surrounding-towns-mapping
Draft

feat(book-gig): add surrounding town mappings for Pembroke, Pulaski, Wirtz, and Huddleston#873
JoshuaVSherman wants to merge 1 commit into
devfrom
agy/surrounding-towns-mapping

Conversation

@JoshuaVSherman

Copy link
Copy Markdown
Contributor

Summary

  • Add Pembroke, Pulaski, Giles, Wirtz, Huddleston, and Moneta to METRO_SURROUNDING and KNOWN_METROS in src/book-gig/parser.ts.
  • Expands surrounding-town discovery for Blacksburg/NRV and Roanoke/Salem so candidate venues in Giles County (Mountain Lake Lodge in Pembroke), Pulaski County (Rock House Marina in Claytor Lake/Pulaski), and Franklin County / Smith Mountain Lake (Brooks Mill Winery in Wirtz) are resolved in regional candidate sweeps.
  • Bump version to 1.32.34 in deno.json.

Part of #484

How to test locally

  • Automated unit test suite:
    • Run deno task test to verify all 1947 tests pass 100% green.
    • Run deno task fmt:check, deno task lint, and deno task sast.
  • Behavioral verification of geographic candidate resolution:
    • Call parseLocation("Lynchburg, Blacksburg, Martinsville, Salem, Roanoke, and surrounding areas").
    • Verify candidate venues in Pembroke (24136), Pulaski (24301), and Wirtz (24184) are included in the resolved geographic perimeter.

Test evidence

Task fmt:check deno fmt --check src/ test/
Checked 202 files
Task lint deno lint src/ test/
Checked 166 files
Task sast bash scripts/sast.sh
Ran 214 rules on 63 files: 0 findings.
[sast] OK — no blocking findings.
ok | 1947 passed | 0 failed (59s)

🤖 Work by Antigravity — Gemini Flash (High)

@JoshuaVSherman
JoshuaVSherman force-pushed the agy/surrounding-towns-mapping branch from 829ace4 to ed9beae Compare August 28, 2026 21:47
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