Skip to content

chore(drivers): move bundled pin to f4ab266e - #863

Merged
frahlg merged 1 commit into
masterfrom
agent/update-bundled-driver-pin
Aug 7, 2026
Merged

chore(drivers): move bundled pin to f4ab266e#863
frahlg merged 1 commit into
masterfrom
agent/update-bundled-driver-pin

Conversation

@frahlg

@frahlg frahlg commented Aug 7, 2026

Copy link
Copy Markdown
Member

Change

  • Move the bundled-driver pin to device-drivers@f4ab266e84e16f202deb468ce5198dfcadaa33df.
  • Bring in version bumps for CTEK API v1 and v2 (0.3.2) and Sungrow (1.5.8).
  • Keep telemetry, control, and register code unchanged across all 37 bundled drivers.
  • Add a patch changeset.

Checks

  • scripts/sync-bundled-drivers.sh --behind
  • make driver-versions-across-pin DRIVER_BASE=origin/master
  • make driver-repository-validate
  • go test ./internal/drivers/... ./internal/driverinventory/... ./internal/driverrepo/...
  • make verify
  • pre-push make verify-all

Fixes #862

@frahlg
frahlg marked this pull request as ready for review August 7, 2026 06:47

frahlg commented Aug 7, 2026

Copy link
Copy Markdown
Member Author

Merge check for head 924159b248abf069353e1f421f76d6fa746f0bd4:

  • The PR changes only drivers/BUNDLED_SOURCE.json and one patch changeset. No open PR touches either file.
  • Comparing the old and new pins shows only version-line changes in bundled Lua files: CTEK API v1 and v2 move to 0.3.2; Sungrow moves to 1.5.8. Telemetry, control, and register code does not change.
  • GitHub passes core, full stack, Device Support contract, changeset, policy, and final go test + vet checks.
  • Local checks also pass pin freshness, versions across the pin, publication of all 37 unsigned recovery drivers, the driver Go packages, make verify, and pre-push make verify-all.

The workflow's path classifier currently counts drivers/BUNDLED_SOURCE.json as core before it can count it as drivers, so the two driver-named CI jobs show as skipped. Core fetches this exact pin and tests it. I did not change test.yml here because open PRs #735, #736, and #785 already touch that file.

/drivers/ is owned by @frahlg; this PR is also authored by @frahlg, so GitHub cannot record a self-approval. There are no review comments or open threads.

@frahlg
frahlg merged commit 49f6369 into master Aug 7, 2026
13 checks passed
@frahlg
frahlg deleted the agent/update-bundled-driver-pin branch August 8, 2026 05:22
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.

Bundled driver pin is behind device-drivers

1 participant