@homebridge-plugins/homebridge-ewelink 12.18.0 → 12.18.1

package/CHANGELOG.md

@@ -2,6 +2,19 @@
 
 All notable changes to `@homebridge-plugins/homebridge-ewelink` will be documented in this file.
 
+## v12.18.1 (2026-08-02)
+
+### Changed
+
+- chore: add a test setup with tests for the shared helper functions
+- chore(github): run the build and tests in ci, on node 22, 24 and 26
+- chore: use the same lint setup across every plugin
+- chore: add a changelog:sync script to populate the pending section from the commits
+- chore: count a repeated commit subject once when syncing the changelog
+- chore(github): check the changelog against the commits in ci
+- chore(deps): dependency updates
+- docs: add node 26 to the supported node versions
+
 ## v12.18.0 (2026-07-27)
 
 ### Changed

package/README.md

@@ -27,7 +27,7 @@ Homebridge plugin to integrate eWeLink devices into HomeKit
 ### Prerequisites
 
 - To use this plugin, you will need to already have:
-  - [Node](https://nodejs.org): latest version of `v22` or `v24` - any other major version is not supported.
+  - [Node](https://nodejs.org): latest version of `v22`, `v24` or `v26` - any other major version is not supported.
   - [Homebridge](https://homebridge.io): `v1.6` or above - refer to link for more information and installation instructions.
 
 ### Setup

package/lib/homebridge-ui/public/index.html

@@ -218,7 +218,7 @@
 <ul>
   <li>To use this plugin, you will need to already have:</li>
   <ul>
-    <li><a href="https://nodejs.org" target="_blank">Node</a>: latest version of <code>v22</code> or <code>v24</code> - any other major version is not supported.</li>
+    <li><a href="https://nodejs.org" target="_blank">Node</a>: latest version of <code>v22</code>, <code>v24</code> or <code>v26</code> - any other major version is not supported.</li>
     <li><a href="https://homebridge.io" target="_blank">Homebridge</a>: <code>v1.6</code> or above - refer to link for more information and installation instructions.</li>
   </ul>
 </ul>

package/lib/utils/functions.test.js

@@ -0,0 +1,77 @@
+import { describe, expect, it } from 'vitest'
+
+import { generateRandomString, hasProperty, parseDeviceId, parseError } from './functions.js'
+
+describe('parseDeviceId', () => {
+  it('strips punctuation and spacing users paste in from the app', () => {
+    expect(parseDeviceId('10:00:ab:cd')).toBe('1000abcd')
+    expect(parseDeviceId(' 1000abcd ')).toBe('1000abcd')
+    expect(parseDeviceId('1000-abcd')).toBe('1000abcd')
+  })
+
+  it('lowercases the id, so matching is not case sensitive', () => {
+    expect(parseDeviceId('1000ABCD')).toBe('1000abcd')
+  })
+
+  it('restores the SW marker, which is uppercase in a real device id', () => {
+    // ⚠️ Deliberately documenting the current behaviour rather than asserting
+    // it is ideal: the id is lowercased first, then the FIRST "sw" is put back
+    // to uppercase. A second occurrence is left lowercase.
+    expect(parseDeviceId('SW123')).toBe('SW123')
+    expect(parseDeviceId('sw123')).toBe('SW123')
+    expect(parseDeviceId('swsw')).toBe('SWsw')
+  })
+
+  it('accepts an id that is already clean', () => {
+    expect(parseDeviceId('1000abcd')).toBe('1000abcd')
+  })
+})
+
+describe('generateRandomString', () => {
+  it('returns the requested length', () => {
+    expect(generateRandomString(16)).toHaveLength(16)
+  })
+
+  it('uses only lowercase letters and digits', () => {
+    expect(generateRandomString(200)).toMatch(/^[a-z0-9]+$/)
+  })
+
+  it('does not return the same value twice in a row', () => {
+    expect(generateRandomString(32)).not.toBe(generateRandomString(32))
+  })
+})
+
+describe('hasProperty', () => {
+  it('detects own properties, including ones set to undefined', () => {
+    // Device payloads regularly carry a key with no value, and that is
+    // different from the key being absent.
+    expect(hasProperty({ switch: undefined }, 'switch')).toBe(true)
+    expect(hasProperty({}, 'switch')).toBe(false)
+  })
+
+  it('ignores inherited properties', () => {
+    expect(hasProperty({}, 'toString')).toBe(false)
+  })
+})
+
+describe('parseError', () => {
+  it('appends the first stack frame to the message', () => {
+    const err = new Error('boom')
+    err.stack = 'Error: boom\n    at thing (/a.js:1:1)'
+    expect(parseError(err)).toBe('boom at thing (/a.js:1:1)')
+  })
+
+  it('hides the stack for errors the caller expects', () => {
+    // The ACK timeout family (#773/#779) is noisy and routine, so those are
+    // logged without a stack.
+    const err = new Error('timeout')
+    err.stack = 'Error: timeout\n    at thing (/a.js:1:1)'
+    expect(parseError(err, ['timeout'])).toBe('timeout')
+  })
+
+  it('returns the message when there is no stack', () => {
+    const err = new Error('nostack')
+    err.stack = ''
+    expect(parseError(err)).toBe('nostack')
+  })
+})

package/package.json

@@ -3,7 +3,7 @@
   "displayName": "Homebridge eWeLink",
   "alias": "eWeLink",
   "type": "module",
-  "version": "12.18.0",
+  "version": "12.18.1",
   "description": "Homebridge plugin to integrate eWeLink devices into HomeKit.",
   "author": {
     "name": "bwp91",
@@ -55,11 +55,15 @@
   "main": "lib/index.js",
   "engines": {
     "homebridge": "^1.6.0 || ^2.0.0",
-    "node": "^22.12.0 || ^24.0.0"
+    "node": "^22.12.0 || ^24.0.0 || ^26.0.0"
   },
   "scripts": {
     "lint": "eslint . --max-warnings=0",
-    "lint:fix": "npm run lint -- --fix"
+    "lint:fix": "npm run lint -- --fix",
+    "test": "vitest run",
+    "test:watch": "vitest watch",
+    "test-coverage": "npm run test -- --coverage",
+    "changelog:sync": "node .github/scripts/sync-changelog.mjs"
   },
   "dependencies": {
     "@homebridge/plugin-ui-utils": "^2.2.5",
@@ -70,6 +74,8 @@
     "ws": "^8.21.1"
   },
   "devDependencies": {
-    "@antfu/eslint-config": "9.2.0"
+    "@antfu/eslint-config": "^9.2.0",
+    "@vitest/coverage-v8": "^4.1.10",
+    "vitest": "^4.1.10"
   }
 }