Skip to content

Log unrecognized SolisCloud product codes for reporting (#17) - #23

Merged
fboundy merged 1 commit into
mainfrom
patch
Jul 3, 2026
Merged

fboundy merged 1 commit into
mainfrom
patch

Conversation

@fboundy

@fboundy fboundy commented Jul 3, 2026

Copy link
Copy Markdown
Owner

Summary

Issue #17 asked for (1) more confirmed PRODUCT_CODE_TO_MODEL entries and (2) a lightweight mechanism to surface unrecognized product codes. (1) isn't actionable without new user-reported data; this PR implements (2):

  • infer_model_from_cloud_record() now logs a warning with the full cloud metadata whenever detection fails (no textual model match, no product-code match), so users can report it and grow the table.
  • No log when detection succeeds.

Test plan

  • tests/test_cloud_model_detection.py: unrecognized metadata logs the expected warning; a known product code logs nothing
  • Full suite: 296 passed
  • ruff check / ruff format --check clean

Closes #17

infer_model_from_cloud_record now warns with the full cloud metadata
when neither a textual model match nor PRODUCT_CODE_TO_MODEL resolves
a model, so users can report the code and grow the table. The table
itself stays a manual, community-driven expansion; there's no new
data to add without a live-verified sample.

Fixes #17
@github-actions

github-actions Bot commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

Failed to generate code suggestions for PR

@fboundy
fboundy merged commit eb29e80 into main Jul 3, 2026
6 checks passed
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.

Expand SolisCloud product-code to model detection table

1 participant