Skip to content

Hygiene: fix stale comments, social meta, AQI no-data wording; add snow status#12

Merged
tylerpearson merged 1 commit into
mainfrom
advisor/006-hygiene-sweep
Jul 1, 2026
Merged

Hygiene: fix stale comments, social meta, AQI no-data wording; add snow status#12
tylerpearson merged 1 commit into
mainfrom
advisor/006-hygiene-sweep

Conversation

@tylerpearson

Copy link
Copy Markdown
Owner

Executes plans/006-hygiene-sweep.md. Advisor-reviewed: diff read in full, 32 vitest + 25 pytest re-run.

  • Stale in-memory-PMTiles comment corrected; dead OUT_FIELDS constant removed
  • og:url fixed (was example.com); og:image/twitter:image added (social.png = docs/screenshot.png)
  • AQI status no longer claims 'Good' when AirNow returns zero features
  • Snow overlay gets a status line like the other live layers

🤖 Generated with Claude Code

https://claude.ai/code/session_01HNxgKNjtfeKtxYvN34KbGs

…ow status

Fixes five small trust-eroding defects: the archiveReady comment in main.ts
described removed code, fetch_mvum.py carried a 22-line unused OUT_FIELDS
constant, index.html's social meta pointed at example.com with no share
image, the AQI status line claimed "Good" when AirNow returned zero data,
and snow was the only live overlay without a status line.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01HNxgKNjtfeKtxYvN34KbGs
@tylerpearson tylerpearson merged commit 0cf820d into main Jul 1, 2026
2 of 3 checks passed
@tylerpearson tylerpearson deleted the advisor/006-hygiene-sweep branch July 1, 2026 22:25
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