feat: add tempest-cli skill — hyper-local weather from Tempest station #3
No reviewers
Labels
No labels
community-feedback
enhancement
skill-upgrade
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set
Reference
magnus/agent-skills!3
Loading…
Reference in a new issue
No description provided.
Delete branch "feat/tempest-cli-skill"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Summary
CLI wrapper for the WeatherFlow Tempest API: current conditions, forecast, historical observations, and real-time UDP broadcasts. Demonstrates all cli-builder patterns in a working, testable project.
Files
Commands
current— latest observations (auto-selects sensor, skips hub)forecast— current + daily + hourly (with --days N)stations— list stations and attached devicesobs— historical observations (--days N)udp listen— real-time broadcasts from hub on same LANFeatures
Setup
Token is free from weatherflow.com (Account → API Tokens).
Signed-off-by: Jasper magnus@groktop.us
Code Review: tempest-cli
Overall, this is a well-structured CLI that follows cli-builder patterns. Clean error handling, proper unit conversions, good use of argparse. A few issues to address before merging.
Issues found:
Bug: OBS_SKY_FIELDS missing 3 fields
OBS_SKY_FIELDSdefines 14 fields butreferences/tempest-api-field-layouts.mddocuments 17. Missing:nc_rain,local_day_nc_rain,precip_analysis_type. Thedecode_obsfunction forobs_skywill silently drop these values.Missing feature: ~/.tempest.env fallback
SKILL.md says "falls back to reading
~/.tempest.envif the env var is not set" but the code only doesos.getenv("TEMPEST_TOKEN", ""). No.tempest.envfile reading is implemented.Doc: SKILL.md obs_st table truncated
SKILL.md shows indices 0-18 for obs_st but the code defines 22 fields (indices 19-21 for nc_rain_accumulation, local_day_nc_rain_accumulation, precip_analysis_type are in both the code and reference doc). The SKILL.md should either show the complete table or note that the last 3 indices are covered in the reference doc.
Reviewed-by: Jasper (automated review)