]> Piment Noir Git Repositories - e-mobility-charging-stations-simulator.git/log
e-mobility-charging-stations-simulator.git
12 days agorefactor: eliminate explicit `any` from `once()` and `toHandler()` with proper generics
Jérôme Benoit [Fri, 27 Mar 2026 14:56:48 +0000 (15:56 +0100)] 
refactor: eliminate explicit `any` from `once()` and `toHandler()` with proper generics

12 days agorefactor: type buildTransactionEvent with OCPP20TransactionEventOptions
Jérôme Benoit [Fri, 27 Mar 2026 14:30:15 +0000 (15:30 +0100)] 
refactor: type buildTransactionEvent with OCPP20TransactionEventOptions

Replace the untyped Record<string, unknown> cast in buildTransactionEvent
with the proper OCPP20TransactionEventOptions parameter type. Add build
hint fields to the options interface. Remove 64 stale as-unknown-as casts
from transaction event tests.

12 days agorefactor: accept enum types in registry/manager signatures, remove 429 redundant...
Jérôme Benoit [Fri, 27 Mar 2026 13:39:32 +0000 (14:39 +0100)] 
refactor: accept enum types in registry/manager signatures, remove 429 redundant as-string casts

Widen buildRegistryKey, buildCaseInsensitiveCompositeKey, and
VariableMetadata.component to accept OCPP20ComponentName and VariableName
enums directly, eliminating 429 as-string casts across the registry and
variable manager. 8 casts retained on enum comparison lines required by
no-unsafe-enum-comparison lint rule.

12 days agofix(deps): update all non-major dependencies (#1759)
renovate[bot] [Fri, 27 Mar 2026 13:26:42 +0000 (14:26 +0100)] 
fix(deps): update all non-major dependencies (#1759)

* fix(deps): update all non-major dependencies

* [autofix.ci] apply automated fixes

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
12 days agorefactor: consolidate enforceMessageLimits types with generic R and RejectionReason
Jérôme Benoit [Fri, 27 Mar 2026 13:23:14 +0000 (14:23 +0100)] 
refactor: consolidate enforceMessageLimits types with generic R and RejectionReason

- Add generic type parameter R to enforceMessageLimits and
  enforcePostCalculationBytesLimit, eliminating 4 'as typeof' casts
- Extract RejectionReason interface (additionalInfo + reasonCode) to
  replace inline anonymous types, aligned with StatusInfoType fields
- Remove all enum→string→enum round-trips in callers

12 days agofix: type buildRejected reasonCode as ReasonCodeEnumType instead of string
Jérôme Benoit [Fri, 27 Mar 2026 12:55:08 +0000 (13:55 +0100)] 
fix: type buildRejected reasonCode as ReasonCodeEnumType instead of string

Eliminates the pointless enum→string→enum round-trip: the callback
parameter is now typed as ReasonCodeEnumType directly, removing the
as-string casts at call sites and the reverse keyof-typeof reconversion
in callers.

12 days agofeat: enrich OCPP 2.0 template with OCPP 1.6 equivalent config keys
Jérôme Benoit [Fri, 27 Mar 2026 12:46:34 +0000 (13:46 +0100)] 
feat: enrich OCPP 2.0 template with OCPP 1.6 equivalent config keys

Add LocalAuthListCtrlr.Enabled, ReservationCtrlr.Enabled/NonEvseSpecific,
TxCtrlr.EVConnectionTimeOut, AuthCtrlr.LocalAuthorizationOffline to
keba-ocpp2 template — previously impossible due to key collisions.

- Add LocalAuthListEnabled → LocalAuthListCtrlr.Enabled key remapping
- Add ReserveConnectorZeroSupported → ReservationCtrlr.NonEvseSpecific remapping
- Remove AuthCtrlr.Enabled from template (no application consumer)
- Add NonEvseSpecific, MaxEnergyOnInvalidId to OCPP20OptionalVariableName enum
- Replace string literals with enum refs in registry and service utils
- Add tests for the 2 new key remappings

12 days ago[autofix.ci] apply automated fixes
autofix-ci[bot] [Fri, 27 Mar 2026 11:34:09 +0000 (11:34 +0000)] 
[autofix.ci] apply automated fixes

12 days agofix: use Component.Variable[.Instance] key format for OCPP 2.0 variable persistence
Jérôme Benoit [Fri, 27 Mar 2026 11:31:06 +0000 (12:31 +0100)] 
fix: use Component.Variable[.Instance] key format for OCPP 2.0 variable persistence

The previous format used bare variable names (e.g., 'Enabled'), causing
collisions across components (9 components define Enabled, 8+ define
Available, etc.). Now uses the spec's (Component, Variable, Instance)
triplet as the persisted configuration key.

- Rewrite computeConfigurationKeyName to return Component.Variable format
- Remove shouldFlattenInstance (no longer needed)
- Extract buildConfigKey helper in ConfigurationKeyUtils, export via barrel
- Add MaxCertificateChainSize to OCPP20OptionalVariableName enum
- Update all direct config key lookups with component prefix
- Update keba-ocpp2 template keys to new format
- Update OCPP2_PARAMETER_KEY_MAP resolved values
- Replace all enum string literals in log messages with enum references
- Update all tests to use new key format

12 days agofeat: log once per key when OCPP 1.6 configuration key is remapped to OCPP 2.0 variable
Jérôme Benoit [Thu, 26 Mar 2026 23:09:30 +0000 (00:09 +0100)] 
feat: log once per key when OCPP 1.6 configuration key is remapped to OCPP 2.0 variable

12 days agofix: remove OrganizationName from ISO15118Ctrlr (spec says SecurityCtrlr only)
Jérôme Benoit [Thu, 26 Mar 2026 22:55:28 +0000 (23:55 +0100)] 
fix: remove OrganizationName from ISO15118Ctrlr (spec says SecurityCtrlr only)

Per OCPP 2.0.1 appendix section 3.1.15, OrganizationName belongs to
SecurityCtrlr only. The duplicate entry under ISO15118Ctrlr was not
in the spec and shadowed the SecurityCtrlr default value.

12 days agorefactor: add Enabled to OCPP20RequiredVariableName enum and fix LocalAuthListCtrlr...
Jérôme Benoit [Thu, 26 Mar 2026 22:25:20 +0000 (23:25 +0100)] 
refactor: add Enabled to OCPP20RequiredVariableName enum and fix LocalAuthListCtrlr.Enabled mapping

Replace all 'Enabled' string literals with OCPP20RequiredVariableName.Enabled
in the variable registry and auth adapter. Move LocalAuthListEnabled
default from AuthCtrlr to LocalAuthListCtrlr component per OCPP 2.0.1 spec.

12 days agofix: align OCPP 2.0 variable names to spec (LocalPreAuthorization, LocalAuthorization...
Jérôme Benoit [Thu, 26 Mar 2026 22:09:05 +0000 (23:09 +0100)] 
fix: align OCPP 2.0 variable names to spec (LocalPreAuthorization, LocalAuthorizationOffline)

Rename LocalPreAuthorize → LocalPreAuthorization and
LocalAuthorizeOffline → LocalAuthorizationOffline to match the OCPP
2.0.1 appendix. Add key resolution mapping for both. Replace string
literals with enum references in OCPP20AuthAdapter.

12 days agochore: remove test plan from git tracking
Jérôme Benoit [Thu, 26 Mar 2026 21:43:28 +0000 (22:43 +0100)] 
chore: remove test plan from git tracking

12 days agofeat: resolve OCPP 1.6 configuration keys to OCPP 2.0 equivalents transparently
Jérôme Benoit [Thu, 26 Mar 2026 21:42:53 +0000 (22:42 +0100)] 
feat: resolve OCPP 1.6 configuration keys to OCPP 2.0 equivalents transparently

Add resolveKey() inside ConfigurationKeyUtils that maps OCPP 1.6 key
names to their OCPP 2.0 equivalents (MeterValuesSampledData →
TxUpdatedMeasurands, ConnectionTimeOut → EVConnectionTimeOut, etc.)
transparently in get/add/set/delete operations. Remap keba-ocpp2
template keys and add missing OCPP 2.0 variables. Add exhaustive
tests for key resolution across all OCPP versions.

13 days agofix: use OCPP version-specific parameter keys for meter value measurands
Jérôme Benoit [Thu, 26 Mar 2026 20:35:50 +0000 (21:35 +0100)] 
fix: use OCPP version-specific parameter keys for meter value measurands

Remap keba-ocpp2 template configuration keys from OCPP 1.6 names to
OCPP 2.0 variable names (MeterValuesSampledData → TxUpdatedMeasurands,
MeterValueSampleInterval → TxUpdatedInterval, AuthorizeRemoteTxRequests
→ AuthorizeRemoteStart). Remove OCPP 1.6-only keys with no 2.0
equivalent. Use StandardParametersKey union to resolve the correct
key at runtime based on OCPP version.

13 days agofix(docs): use bold instead of italic for vendor-specific markers to avoid underscore...
Jérôme Benoit [Thu, 26 Mar 2026 20:03:00 +0000 (21:03 +0100)] 
fix(docs): use bold instead of italic for vendor-specific markers to avoid underscore conflicts

13 days agodocs: list OCPP 2.0 device model variables by component in README
Jérôme Benoit [Thu, 26 Mar 2026 20:00:03 +0000 (21:00 +0100)] 
docs: list OCPP 2.0 device model variables by component in README

Replace the incorrect GetVariables/SetVariables command listing with
141 device model variables across 19 components extracted from
OCPP20VariableRegistry.ts. Vendor-specific variables are marked.
Matches the structure used for OCPP 1.6 configuration keys.

13 days agochore: release main (#1751) ocpp-server@v3.4.0 simulator@v3.4.0 v3 v3.4 webui@v3.4.0
Jérôme Benoit [Thu, 26 Mar 2026 19:31:47 +0000 (20:31 +0100)] 
chore: release main (#1751)

* chore: release main

* [autofix.ci] apply automated fixes

---------

Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
13 days agofeat(ocpp2): add configurable firmware signature verification simulation (L01.FR...
Jérôme Benoit [Thu, 26 Mar 2026 19:06:53 +0000 (20:06 +0100)] 
feat(ocpp2): add configurable firmware signature verification simulation (L01.FR.03/L01.FR.04) (#1757)

* feat(ocpp2): add FirmwareCtrlr component and signature verification variable

* test(ocpp2): add firmware signature verification failure tests

- Add test for InvalidSignature path when SimulateSignatureVerificationFailure is true
- Add test for normal path when SimulateSignatureVerificationFailure is false (TDD red phase)
- Tests verify lifecycle termination and SecurityEventNotification type correctness
- Evidence: typecheck & lint pass; tests fail as expected (awaiting Task 3 implementation)

* feat(ocpp2): implement configurable firmware signature verification simulation

* docs: update README firmware signature verification note

* fix(ocpp2): address PR review feedback on firmware signature verification

- Await sendSecurityEventNotification in failure path for consistency with success path
- Reorder test assertions: check array length before index access
- Add tick-forward verification that lifecycle stops after InvalidSignature
- Remove redundant standardCleanup() calls already handled by afterEach
- Remove firmware signature verification note from README

* fix(ocpp2): improve firmware update spec conformance and test quality

- L01.FR.07: Move EVSE unavailability check inside wait loop for continuous monitoring
- L01.FR.30: Honor retries/retryInterval with simulated download retry cycle
- Merge retry test into existing DownloadFailed test (DRY)
- Add EVSE continuous monitoring lifecycle test (L01.FR.07)
- Fix assertion ordering: length checks before index access
- Remove as-unknown-as-string cast in favor of String()
- Tighten assert.ok to assert.notStrictEqual where applicable

* fix(tests): type CapturedOCPPRequest.command as OCPP20RequestCommand

- Replace string type with OCPP20RequestCommand enum on CapturedOCPPRequest.command
- Remove 6 unnecessary 'as string' casts across UpdateFirmware and TransactionEvent tests
- Replace 'Unavailable' string literal with OCPP20ConnectorStatusEnumType.Unavailable
- Remove unnecessary optional chaining on non-nullable payload
- Complete @param JSDoc for retries/retryInterval on simulateFirmwareUpdateLifecycle

* [autofix.ci] apply automated fixes

* [autofix.ci] apply automated fixes (attempt 2/3)

* fix(tests): replace flaky sentRequests assertions with mock.method spy in EVSE monitoring test

Mock sendEvseStatusNotifications directly instead of asserting on fire-and-forget
side effects captured in sentRequests. The internal async chain (sendAndSetConnectorStatus
→ dynamic import → requestHandler) needed non-deterministic flushMicrotasks() stacking.
The spy makes assertions synchronous and CI-deterministic.

---------

Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
13 days agofix: respect elementAddDelay by using sequential startup when configured
Jérôme Benoit [Thu, 26 Mar 2026 16:42:21 +0000 (17:42 +0100)] 
fix: respect elementAddDelay by using sequential startup when configured

The perf commit 218fd550 switched to parallel Promise.allSettled for
startup speed, but broke elementAddDelay since all stations launched
simultaneously. Now uses sequential await loop when elementAddDelay > 0
and parallel allSettled when 0 or unset. Removes redundant outer
try/catch — config errors propagate as fatal, station errors are
handled per-station in both paths.

13 days agofix(deps): update all non-major dependencies (#1755)
renovate[bot] [Thu, 26 Mar 2026 11:42:37 +0000 (12:42 +0100)] 
fix(deps): update all non-major dependencies (#1755)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
13 days agochore: remove test plan from git tracking
Jérôme Benoit [Thu, 26 Mar 2026 00:03:45 +0000 (01:03 +0100)] 
chore: remove test plan from git tracking

13 days agofix: cleanup connector after queued TransactionEvent.Ended replay
Jérôme Benoit [Thu, 26 Mar 2026 00:02:01 +0000 (01:02 +0100)] 
fix: cleanup connector after queued TransactionEvent.Ended replay

Extract cleanupEndedTransaction() in OCPP20ServiceUtils to consolidate
stopPeriodicMeterValues + resetConnectorStatus + unlock + Available
status. Called from both stopTransaction20 (normal flow) and
sendQueuedTransactionEvents (offline replay). Fixes stale Occupied
status and orphaned meter value interval after queued Ended events.

13 days agotest(webui): harmonize test stubs, remove AAA comments, add composable and StateButto...
Jérôme Benoit [Wed, 25 Mar 2026 22:41:36 +0000 (23:41 +0100)] 
test(webui): harmonize test stubs, remove AAA comments, add composable and StateButton tests

13 days agofeat: add server-side refresh notification over WebSocket
Jérôme Benoit [Wed, 25 Mar 2026 22:15:51 +0000 (23:15 +0100)] 
feat: add server-side refresh notification over WebSocket

Bootstrap calls scheduleClientNotification() after each cache mutation.
A 500ms debounce collapses rapid updates into a single broadcast of
ProtocolNotification [ServerNotification.REFRESH] to all connected WS
clients. setChargingStationData/deleteChargingStationData return boolean
so callers only notify on actual change. Frontend responseHandler uses
isServerNotification/isProtocolResponse guards to dispatch messages.

13 days agofix: post updated message on connectorStatusChanged to fix stale UI cache
Jérôme Benoit [Wed, 25 Mar 2026 21:20:43 +0000 (22:20 +0100)] 
fix: post updated message on connectorStatusChanged to fix stale UI cache

The .finally() in internalSendMessage emits the updated event BEFORE
sendAndSetConnectorStatus updates connectorStatus.status, causing the
UI cache to snapshot the old status. The connectorStatusChanged event
fires AFTER the update, so posting an updated message on it ensures
the cache receives the correct connector status.

2 weeks agorefactor(webui): extract useExecuteAction composable to eliminate duplication
Jérôme Benoit [Wed, 25 Mar 2026 20:55:02 +0000 (21:55 +0100)] 
refactor(webui): extract useExecuteAction composable to eliminate duplication

2 weeks agorefactor(webui): use useChargingStations composable in ChargingStationsView
Jérôme Benoit [Wed, 25 Mar 2026 20:36:53 +0000 (21:36 +0100)] 
refactor(webui): use useChargingStations composable in ChargingStationsView

2 weeks agorefactor(webui): add StateButton, centralize active style, fix refresh lifecycle
Jérôme Benoit [Wed, 25 Mar 2026 20:30:03 +0000 (21:30 +0100)] 
refactor(webui): add StateButton, centralize active style, fix refresh lifecycle

- StateButton: stateless toggle driven by server state (active prop)
- Button: owns .button--active CSS, ToggleButton/StateButton pass :active
- Conditional buttons: show Start/Stop, Open/Close, Lock/Unlock based on state
- refreshChargingStations(): shared composable eliminates 4x duplicated refresh
- executeAction() helper in CSData/CSConnector: DRY action/toast/refresh
- Fix: trigger getChargingStations() on need-refresh event
- Fix: refresh after AddChargingStations, StartTransaction, SetSupervisionUrl
- table-layout: fixed on connectors table to prevent overflow

2 weeks agochore(deps): update dependency typescript to v6 (#1750)
renovate[bot] [Wed, 25 Mar 2026 18:59:13 +0000 (19:59 +0100)] 
chore(deps): update dependency typescript to v6 (#1750)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
2 weeks agofix(webui): add table-layout fixed to connectors table to prevent overflow on action...
Jérôme Benoit [Wed, 25 Mar 2026 18:56:25 +0000 (19:56 +0100)] 
fix(webui): add table-layout fixed to connectors table to prevent overflow on action panel open

2 weeks agofeat: add connector cable retention lock/unlock simulation (#1754)
Jérôme Benoit [Wed, 25 Mar 2026 18:20:36 +0000 (19:20 +0100)] 
feat: add connector cable retention lock/unlock simulation (#1754)

* feat(types): add connector lock simulation types and UI protocol enums

* feat(core): add connector lock/unlock simulation methods

* feat(ocpp): set lock state on UnlockConnector command for both OCPP stacks

* feat(ui-server): add lock/unlock broadcast channel commands and MCP schemas

* feat(ui): add lock/unlock buttons to web UI

* test: add connector lock/unlock simulation tests

* docs: add connector lock/unlock simulation documentation

* feat(ocpp): auto-lock connector on transaction start, auto-unlock on normal termination

* fix(ui): reorder Locked column before Transaction in connector table

* fix: guard lock unlock on accepted stop, emit updated event, add null status warnings, fix resetConnectorStatus test

* [autofix.ci] apply automated fixes

* fix(ocpp): only auto-lock connector on accepted transaction start in OCPP 2.0.1

* fix(core): emit connectorStatusChanged instead of updated on lock state change

* test: add idempotency tests for lockConnector and unlockConnector

---------

Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
2 weeks agofix(deps): update all non-major dependencies (#1753)
renovate[bot] [Wed, 25 Mar 2026 13:51:33 +0000 (14:51 +0100)] 
fix(deps): update all non-major dependencies (#1753)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
2 weeks agofeat(ocpp-server): enhance OCPP 2.0.1 mock server for comprehensive E2E testing ...
Jérôme Benoit [Wed, 25 Mar 2026 11:38:55 +0000 (12:38 +0100)] 
feat(ocpp-server): enhance OCPP 2.0.1 mock server for comprehensive E2E testing (#1752)

* feat(ocpp-server): add boot status sequence for Pending→Accepted transition testing

* feat(ocpp-server): add configurable TriggerMessage type

* feat(ocpp-server): add configurable Reset type and ChangeAvailability status

* feat(ocpp-server): add command sequencing for multi-step CSMS workflows

* feat(ocpp-server): add groupIdToken and cacheExpiry to Authorize response

* feat(ocpp-server): add multi-variable SetVariables/GetVariables CLI support

* fix(ocpp-server): share boot_index across reconnections so sequence advances correctly

* fix(ocpp-server): schedule send_commands as background task so it runs after cp.start()

* fix(ocpp-server): correct CLI help text casing and validate boot-status-sequence

* chore: remove E2E test plan from PR branch

* refactor(ocpp-server): fix misplaced tests and harmonize style

* docs(ocpp-server): document all new CLI flags and transaction tracking

* fix(ocpp-server): harden input validation and connection cleanup

* fix(ocpp-server): clean up auth-mode choices and CLI edge cases

* fix(ocpp-server): validate cache-expiry, skip trailing commas, document yield semantics

* fix(ocpp-server): harmonize empty-entry handling and wrap CLI parsers in error handler

2 weeks agorefactor: use try/catch instead of .catch() for stopMessageSequence error handling
Jérôme Benoit [Tue, 24 Mar 2026 21:21:21 +0000 (22:21 +0100)] 
refactor: use try/catch instead of .catch() for stopMessageSequence error handling

2 weeks agofix: add OCPP 2.0 DataTransfer outgoing support and B03 boot retry in Rejected state
Jérôme Benoit [Tue, 24 Mar 2026 19:40:46 +0000 (20:40 +0100)] 
fix: add OCPP 2.0 DataTransfer outgoing support and B03 boot retry in Rejected state

Add DATA_TRANSFER to OCPP20RequestCommand enum, buildRequestPayload
passthrough, schema registration, and response handler (P02.FR.02).

Allow BootNotification retry when station is in Rejected state by
adding inRejectedState() to the internalSendMessage gate (B03.FR.06).

Fix mock server: use OCPPCommCtrlr component for HeartbeatInterval
in SetVariables, add customerIdentifier in CustomerInformation.

Add unit tests for promiseWithTimeout and DataTransfer outgoing.

2 weeks agofix: prevent shutdown timeout with promiseWithTimeout helper
Jérôme Benoit [Tue, 24 Mar 2026 18:08:34 +0000 (19:08 +0100)] 
fix: prevent shutdown timeout with promiseWithTimeout helper

Extract a shared promiseWithTimeout<T>() utility from 3 divergent
Promise.race+timeout implementations. Use it in ChargingStation.stop()
to cap stopMessageSequence at 30s so the stopped event is always
emitted even when the OCPP server is unreachable, preventing the
60s Bootstrap shutdown timeout.

Also fix mock server SecurityEventNotification handler parameter name
and add OCPP 2.0.1 E2E test plan.

2 weeks agodocs: fix inaccurate @returns in getChargingProfilesLimit
Jérôme Benoit [Tue, 24 Mar 2026 17:01:10 +0000 (18:01 +0100)] 
docs: fix inaccurate @returns in getChargingProfilesLimit

Return type can be undefined but JSDoc only documented the non-null case.

2 weeks agodocs: harmonize JSDoc @param/@returns across entire codebase
Jérôme Benoit [Tue, 24 Mar 2026 16:54:43 +0000 (17:54 +0100)] 
docs: harmonize JSDoc @param/@returns across entire codebase

- Add missing dash separator to @param tags in OCPP20VariableRegistry
  and OCPPServiceUtils (35 occurrences)
- Fill empty @param descriptions in Helpers, IdTagsCache,
  OCPPRequestService, and Worker modules (23 occurrences)
- Replace type-only @returns with meaningful descriptions

2 weeks agodocs: add JSDoc to ChargingStation public API methods
Jérôme Benoit [Tue, 24 Mar 2026 16:45:25 +0000 (17:45 +0100)] 
docs: add JSDoc to ChargingStation public API methods

Add concise JSDoc with @param and @returns tags to 29 public methods
following the project's established pattern (param - description).
Resolves all 45 jsdoc/require-param and jsdoc/require-returns warnings.

2 weeks agorefactor: eliminate remaining non-null assertions in ChargingStation and OCPP16Servic...
Jérôme Benoit [Tue, 24 Mar 2026 16:26:31 +0000 (17:26 +0100)] 
refactor: eliminate remaining non-null assertions in ChargingStation and OCPP16ServiceUtils

- ChargingStation: replace 8 non-null assertions with optional chaining,
  nullish coalescing defaults, and null guards
- OCPP16ServiceUtils: add type narrowing guards for composite charging
  schedules, extract connector status variables with null checks,
  replace all 14 non-null assertions with safe access patterns

Zero non-null assertions remain across the entire src/ directory.

2 weeks agorefactor: eliminate non-null assertions in 8 files (31 of 53 total)
Jérôme Benoit [Tue, 24 Mar 2026 16:19:37 +0000 (17:19 +0100)] 
refactor: eliminate non-null assertions in 8 files (31 of 53 total)

- AutomaticTransactionGenerator: extract connector status variables with
  null guards, use optional chaining for stationInfo and ATG config
- ChargingStation: replace stationInfo! with null guards and ?./?? defaults
- Helpers: replace stationInfo!.maximumPower! with ?./?? defaults
- IdTagsCache: replace stationInfo!.hashId with optional chaining
- ChargingStationWorkerBroadcastChannel: replace stationInfo!.hashId
- OCPP16IncomingRequestService: wrap firmware/diagnostics status in
  null guards, replace resetTime!/configurationKey! with safe access
- UIServerFactory: replace options!.host! with optional chaining
- Configuration: replace getConfigurationData()! with null guards

2 weeks agorefactor: audit-driven improvements — use BaseError, fix barrel bypass, remove redund...
Jérôme Benoit [Tue, 24 Mar 2026 14:51:21 +0000 (15:51 +0100)] 
refactor: audit-driven improvements — use BaseError, fix barrel bypass, remove redundant console

- Replace throw new Error with BaseError in OCPP20CertificateManager and
  OCPP20VariableManager (3 occurrences)
- Move getMessageTypeString from OCPPServiceUtils to Helpers, resolving
  the barrel bypass in ErrorUtils — consumers import via charging-station
  barrel, tests moved to Helpers.test.ts
- Remove redundant console.warn in Helpers (logger.warn already called)
- Remove unused chalk import from Helpers

2 weeks agorefactor: audit-driven quick wins — fix assert.ok misuse, naming conventions, and...
Jérôme Benoit [Tue, 24 Mar 2026 13:46:17 +0000 (14:46 +0100)] 
refactor: audit-driven quick wins — fix assert.ok misuse, naming conventions, and hardcoded defaults

- Replace 71 improper assert.ok(comparison) with assert.strictEqual/notStrictEqual
  in 10 test files per TEST_STYLE_GUIDE (assert.ok for boolean/existence only)
- Rename lastUpdated → lastUpdatedDate in auth interfaces and strategies
  to follow Date suffix naming convention
- Extract hardcoded 'ws://localhost' to OCPP20Constants.DEFAULT_CONNECTION_URL
- Replace setTimeout(resolve, 50) with proper httpServer.close() await
  in UIMCPServer integration test

2 weeks agofix(webui): remove deprecated baseUrl for TypeScript 6 compatibility
Jérôme Benoit [Tue, 24 Mar 2026 11:47:53 +0000 (12:47 +0100)] 
fix(webui): remove deprecated baseUrl for TypeScript 6 compatibility

TS6 deprecated baseUrl (TS5101). Since paths entries already use './'
prefix, they work standalone without baseUrl. Vite resolve.alias handles
runtime resolution independently.

2 weeks agorefactor(ocpp-server): audit-driven test improvements
Jérôme Benoit [Tue, 24 Mar 2026 11:20:08 +0000 (12:20 +0100)] 
refactor(ocpp-server): audit-driven test improvements

- Extract mock_valid_ws and rate_limit_charge_point fixtures (DRY)
- Replace inline blacklist AuthConfig with existing fixture
- Strengthen log assertions using caplog.records with level checks
- Add edge case tests: empty token in whitelist/blacklist modes
- Add test for unknown transaction event type (case _ branch)
Coverage: 90.18% → 90.98%, 159 → 162 tests.

2 weeks agorefactor(ocpp-server): harmonize shutdown tests with project conventions
Jérôme Benoit [Tue, 24 Mar 2026 11:07:03 +0000 (12:07 +0100)] 
refactor(ocpp-server): harmonize shutdown tests with project conventions

Move fixture near other fixtures, replace _patch_main tuple return with
contextmanager, parametrize SIGINT/SIGTERM, remove dead code helpers,
merge two test classes into one.

2 weeks agotest(ocpp-server): add graceful shutdown tests covering signal handling
Jérôme Benoit [Tue, 24 Mar 2026 11:03:17 +0000 (12:03 +0100)] 
test(ocpp-server): add graceful shutdown tests covering signal handling

Cover SIGINT/SIGTERM handlers, double-signal force quit, Windows
fallback via call_soon_threadsafe, and shutdown timeout warning.
Coverage: 80% → 90%.

2 weeks agofeat(ocpp-server): add graceful shutdown with signal handling
Jérôme Benoit [Tue, 24 Mar 2026 10:44:28 +0000 (11:44 +0100)] 
feat(ocpp-server): add graceful shutdown with signal handling

Handle SIGINT/SIGTERM for clean WebSocket close (code 1001) to all
connected charge points. Includes shutdown drain timeout, double-signal
force quit, and cross-platform Windows fallback via call_soon_threadsafe.

2 weeks agotest(mcp): add OCPP schema mapping consistency tests
Jérôme Benoit [Tue, 24 Mar 2026 01:21:55 +0000 (02:21 +0100)] 
test(mcp): add OCPP schema mapping consistency tests

Verify that every broadcast channel OCPP command has a corresponding
MCP tool schema and OCPP schema mapping entry. Ensures schema coverage
stays in sync with supported commands as a single source of truth guard.

2 weeks agochore: release main (#1748) ocpp-server@v3.3.0 simulator@v3.3.0 v3.3 webui@v3.3.0
Jérôme Benoit [Tue, 24 Mar 2026 01:09:52 +0000 (02:09 +0100)] 
chore: release main (#1748)

* chore: release main

* [autofix.ci] apply automated fixes

---------

Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
2 weeks agofix(deps): update all non-major dependencies (#1749)
renovate[bot] [Tue, 24 Mar 2026 01:05:31 +0000 (02:05 +0100)] 
fix(deps): update all non-major dependencies (#1749)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
2 weeks agorefactor(mcp): split MCP definitions by spec concern — Resources, Tools, ToolSchemas
Jérôme Benoit [Tue, 24 Mar 2026 00:58:12 +0000 (01:58 +0100)] 
refactor(mcp): split MCP definitions by spec concern — Resources, Tools, ToolSchemas

Rename MCPResourceHandlers.ts to MCPResources.ts (contains only MCP
Resources: station-list, station-by-id, template-list, OCPP schemas).

Extract log tools (readCombinedLog, readErrorLog) and their helpers
(getLogFilePath, tailFile) into MCPTools.ts (MCP Tools per spec).

MCPToolSchemas.ts unchanged (tool schema definitions).

Directory now reflects MCP spec separation of concerns:
- MCPResources.ts  — MCP Resources (spec §Resources)
- MCPTools.ts      — MCP Tools with custom logic (spec §Tools)
- MCPToolSchemas.ts — Tool schema definitions for OCPP commands

2 weeks agorefactor(broadcast-channel): extract response status registries from switch/case
Jérôme Benoit [Tue, 24 Mar 2026 00:43:04 +0000 (01:43 +0100)] 
refactor(broadcast-channel): extract response status registries from switch/case

Replace 11 repetitive switch/case blocks in commandResponseToResponseStatus
with two static registries:
- emptyResponseCommands: Set of commands where empty response = success
- acceptedStatusCommands: Map of commands to status check predicates

Custom cases (AUTHORIZE, START/STOP_TRANSACTION, HEARTBEAT,
TRANSACTION_EVENT) remain in switch for version-specific logic.

Adding a new command now requires 1 line in the registry instead of
a 5-line case block.

2 weeks agorefactor(broadcast-channel): extract passthrough() factory, remove 20 identical handlers
Jérôme Benoit [Tue, 24 Mar 2026 00:39:40 +0000 (01:39 +0100)] 
refactor(broadcast-channel): extract passthrough() factory, remove 20 identical handlers

Replace 20 identical BroadcastChannel handler methods (handleAuthorize,
handleDataTransfer, handleHeartbeat, handleStatusNotification, etc.)
with a single passthrough(RequestCommand) factory method that creates
a CommandHandler forwarding the request payload to requestHandler.

Retain only handlers with custom logic: handleBootNotification (merges
bootNotificationRequest), handleMeterValues (version-specific meter
value building), handleStopTransaction (adds meterStop).

Net reduction: -208 lines.

2 weeks agorefactor(ocpp): extract buildEmptyMeterValue helper for empty MeterValue construction
Jérôme Benoit [Tue, 24 Mar 2026 00:16:06 +0000 (01:16 +0100)] 
refactor(ocpp): extract buildEmptyMeterValue helper for empty MeterValue construction

Replace 4 inline { sampledValue: [], timestamp: new Date() } constructions
with buildEmptyMeterValue() helper in OCPPServiceUtils and OCPP16ServiceUtils.

2 weeks agofeat(ui-server): add MCP transport and deprecate HTTP (#1746)
Jérôme Benoit [Mon, 23 Mar 2026 23:59:59 +0000 (00:59 +0100)] 
feat(ui-server): add MCP transport and deprecate HTTP (#1746)

* feat(types): add MCP application protocol type

* chore(deps): add @modelcontextprotocol/sdk and zod

* refactor(ui-server): deprecate UIHttpServer in favor of MCP transport

* feat(ui-server): add MCP tool zod schemas for all procedures

* feat(ui-server): add UIMCPServer transport and MCP resource handlers

* feat(ui-server): integrate UIMCPServer into UIServerFactory

* test(ui-server): add MCP factory integration tests

* test(ui-server): add UIMCPServer unit tests

* test(ui-server): add MCP endpoint integration tests and fix lint

* refactor(ui-server): extract HttpMethod enum to shared UIServerUtils

* docs: restructure UI Protocol section — MCP first, remove redundant API listing

* fix(ui-server): address PR review — security hardening and schema corrections

- Send 404 + destroy for non-/mcp requests instead of hanging socket
- Add rate limiting (429) matching UIHttpServer behavior
- Add body size limit (createBodySizeLimiter) to prevent DoS
- Register res.on('close') before handleRequest to prevent race
- Return 400 for JSON parse errors, 500 for server errors
- Fix setSupervisionUrl schema: supervisionUrl → url
- Fix stopTransaction schema: transactionIds string[] → transactionId number

* refactor(tests): use HttpMethod enum instead of string literals

* docs: remove redundant MCP discoverability statement

* docs: restructure MCP section for agent config, update uiServer type in config table

* feat(ui-server): log deprecation warning when HTTP transport is used

* fix(ui-server): harmonize log levels and message patterns with existing transports

* test(ui-server): align MCP tests with style guide

- Use createLoggerMocks() for log assertions instead of trivial checks
- Add server.stop() to factory fallback tests to prevent resource leaks
- Replace fragile globalThis.clearTimeout override with t.mock.method

* docs: restore PDU acronym definition in WebSocket Protocol section

* fix(ui-server): use UI protocol version for MCP server version

* fix(ui-server): derive log file paths from Configuration instead of hardcoding

* perf(ui-server): tail-read log files instead of loading entire file into memory

* refactor(ui-server): convert log resources to tools with tail parameter

Follow MCP best practices: resources are for static snapshots,
tools are for parameterized on-demand data.

- Replace log://combined and log://error resources with
  readCombinedLog and readErrorLog tools
- Add tail parameter (default 200, max 5000) following the
  filesystem server head/tail pattern
- Add readOnlyHint annotation per MCP tool spec
- Return totalLines metadata in response for context
- Keep station and template resources (small, static data)

* feat(ui-server): expose OCPP JSON schemas as MCP resource templates

Add schema://ocpp/{version} listing and schema://ocpp/{version}/{command}
resource templates that serve the existing JSON schema files from
src/assets/json-schemas/ocpp/ on demand.

LLM agents can discover available OCPP commands per version and read
the full JSON schema for any command without bloating tools/list.
Schemas are read from disk on demand — no memory duplication with
the AJV validators already loaded by OCPP services.

* feat(ui-server): add version-aware OCPP tool schemas with JSON Schema injection

- Add ocppSchemaMapping linking ProcedureName to JSON Schema files
- 16 OCPP tools now have ocpp16Payload/ocpp20Payload fields with
  version affinity in descriptions
- Intercept tools/list to inject full JSON Schemas from disk
  into inputSchema.properties (no Zod duplication, DRY)
- Add payload flattening: extract versioned payload, spread flat
- Add mutual exclusivity: reject if both payloads provided
- Add pre-flight version check: clear error on version mismatch
- Runtime guard: graceful fallback if SDK internal API changes

* refactor(ui-server): use OCPPVersion enum instead of string literals

* refactor(ui-server): improve MCP code elegance and project rule compliance

- Replace as-unknown-as cast with Reflect.get() for SDK handler access
- Replace generic Error with BaseError in stop/readRequestBody
- Extract createToolErrorResponse/createToolResponse static helpers (DRY)
- Extract closeTransportSafely for duplicate transport cleanup
- Extract registerLogReadTool factory for duplicate log tools (DRY)
- Remove unnecessary String() conversions in resource handlers

* fix(ui-server): fix body size double-counting and schema resource path

- readRequestBody: pass chunk.length (not accumulated total) to
  createBodySizeLimiter — matches UIHttpServer pattern
- getSchemaBaseDir: use correct relative path instead of broken
  resolve().includes('assets') condition (always true)

* refactor(ui-server): consolidate version fallback guards and clarify README version docs

* fix(ui-server): harden MCP schema resources against path traversal and fix transport leak

- Add path traversal guard in schema resource handlers to validate resolved
  paths stay within the OCPP schema base directory
- Close MCP transport on 405 Method Not Allowed responses to prevent leak
- Fix log file path to use direct path when rotation is disabled
- Fix misleading Heartbeat tool description that referenced unsupported
  ocpp payload fields

* fix(ui-server): address remaining review findings and lint warnings

- Reorder stop() to delete-before-reject preventing re-entrant issues
- Use ephemeral port in integration test instead of fixed port 18999
- Document synchronous authenticate() assumption and SDK pattern deviation
- Add 'modelcontextprotocol' to cspell wordlist
- Convert JSDoc to inline comment to fix jsdoc lint warnings

* fix(ui-server): exact MCP path match, pin SDK version, log schema injection

- Use exact pathname match (url.pathname !== '/mcp') instead of
  startsWith to prevent unintended paths from reaching the MCP handler
- Pin @modelcontextprotocol/sdk to ~1.27.1 (patch-only) since the
  OCPP schema injection relies on SDK internals pinned to 1.27.x
- Add startup log confirming successful OCPP JSON schema injection

* test(ui-server): add comprehensive UIMCPServer unit tests

Cover private method logic missing from initial test suite:
- invokeProcedure: service null, dual payload, version mismatch,
  direct response, service error, 30s timeout (mock timers)
- checkVersionCompatibility: OCPP 1.6/2.0/2.0.1 matching, cross-version
  errors, hashIds filtering, all-stations fallback
- readRequestBody: valid JSON, payload too large, invalid JSON, stream error
- loadOcppSchemas: disk loading, cache entry validation

Refactor for elegance and guide compliance:
- Centralize Reflect.get wrappers in TestableUIMCPServer class
- Extract assertToolError() helper to eliminate repeated triplet
- Move createMockChargingStationDataWithVersion() to UIServerTestUtils
- Replace magic number with DEFAULT_MAX_PAYLOAD_SIZE import

* fix(ui-server): per-request McpServer factory, async iterator body reader, transport leak

- Create a new McpServer per HTTP request via createMcpServer() factory
  to fix concurrency bug: McpServer.connect() overwrites a single
  internal _transport field, causing cross-talk under concurrent requests.
  OCPP schema cache and UI service are pre-loaded at startup; tool
  registration is ~12µs per request (negligible).

- Replace event-listener readRequestBody with for-await-of async
  iterator pattern. Single settlement guaranteed by language semantics,
  eliminating the double-reject bug on payload-too-large.

- Close transport on mcpServer.connect() failure to prevent resource leak.

- Clean up both transport and McpServer on response close via unified
  cleanup callback.

* fix: align StatusNotification field name with OCPP 2.0.1 schema, fix asset paths

- Use 'connectorStatus' (OCPP 2.0.1 schema field name) instead of
  'status' (non-standard) in OCPP20RequestService.buildRequestPayload
  and both call sites in OCPP20IncomingRequestService

- Fix esbuild-relative asset paths for OCPP JSON schemas and MCP
  resource handlers — import.meta.url resolves to dist/start.js,
  not the original source tree depth

* refactor(ocpp): align payload builders to object-param API, fix StatusNotification field names

- Refactor buildStatusNotificationRequest from positional params to
  object-based StatusNotificationRequest input. Builder accepts both
  'status' (internal/OCPP 1.6) and 'connectorStatus' (OCPP 2.0.1)
  with consistent fallback order (connectorStatus ?? status).

- Refactor buildTransactionEvent from 3 overloads to single
  object-based OCPP20TransactionEventRequest input, removing
  positional param complexity and non-null assertions.

- Refactor sendAndSetConnectorStatus from positional params to
  object-based StatusNotificationRequest input. Callers pass
  version-correct field names: 'status' in OCPP 1.6 code,
  'connectorStatus' in OCPP 2.0 code, either in common code.

- Extract getSchemaBaseDir() as protected method on UIMCPServer
  for test overridability, replacing fragile symlink hack.

- Migrate all call sites (28 sendAndSetConnectorStatus, 60+
  buildTransactionEvent) and their corresponding tests.

* build: externalize @modelcontextprotocol/sdk and zod from esbuild bundle

Add @modelcontextprotocol/* and zod to esbuild external list for
consistency with all other production dependencies (ajv, ws, winston,
etc.) which are already externalized. Reduces bundle size and avoids
bundling the SDK's transitive dependency tree.

* fix(ui-server): fix SonarCloud quality gate — localeCompare sort, reduce cognitive complexity

- Add localeCompare to schema file sort in MCPResourceHandlers to
  ensure locale-aware alphabetical ordering (SonarCloud bug)

- Extract sendErrorResponse helper from handleMcpRequest to reduce
  cognitive complexity from 16 to below threshold (SonarCloud smell)

* chore: update .serena/project.yml with new default config fields

* refactor(ocpp): buildMeterValue resolves connectorId/evseId from transactionId

- Refactor buildMeterValue signature from (station, connectorId,
  transactionId, interval) to (station, transactionId, interval).
  connectorId and evseId are resolved internally from the unique
  transactionId via getConnectorIdByTransactionId/getEvseIdByTransactionId.

- Refactor getSampledValueTemplate to accept optional evseId. When
  provided, EVSE-level MeterValues templates take priority; falls back
  to merging connector-level templates from all EVSE connectors.

- Add MeterValues field to EvseStatus and EvseTemplate types to support
  EVSE-level meter value template definitions.

- Refactor handleMeterValues in BroadcastChannel from if/else to
  switch/case on OCPP version with explicit default error.

- Simplify OCPP20IncomingRequestService fallback MeterValues (no
  transaction = static empty meter value, skip buildMeterValue).

- Add buildMeterValue unit tests covering OCPP 1.6/2.0 resolution,
  unknown transactionId throw, EVSE-level template priority, and
  connector template merge fallback.

- Add TEST_TRANSACTION_ID_STRING constant for OCPP 2.0 tests, fix
  BroadcastChannel test to use string transactionId for OCPP 2.0.1.

* docs: add MCP Protocol to ToC, fix HTTP Protocol link, document EVSE-level MeterValues

- Add MCP Protocol entry to table of contents
- Fix HTTP Protocol anchor to match 'deprecated' heading
- Document EVSE-level MeterValues templates in Evses section examples
  with priority explanation (EVSE-level overrides connector-level)

* fix(ocpp): version-aware Authorize response mapping, complete cross-version types

- Fix commandResponseToResponseStatus for Authorize: use switch/case
  on OCPP version. OCPP 1.6 checks idTagInfo.status, OCPP 2.0 checks
  idTokenInfo.status. Previously only checked idTagInfo (1.6 field),
  causing MCP to report 'failure' for successful OCPP 2.0 authorizations.

- Make AuthorizeResponse and AuthorizeRequest cross-version union types
  (OCPP16 | OCPP20) consistent with AuthorizationStatus pattern.

- Make isIdTagRemoteAuthorized version-aware: sends idTag for OCPP 1.6,
  idToken for OCPP 2.0, and checks the version-correct response field.

- Use version-specific casts (OCPP16AuthorizeResponse,
  OCPP20AuthorizeResponse) in version-switched code paths instead of
  agnostic union type.

* refactor(ocpp): harmonize version switch/case in auth functions

- Refactor isIdTagAuthorizedUnified and isIdTagAuthorized from if/else
  on OCPP version to switch/case with explicit VERSION_16, VERSION_20/201
  cases and separate default fallback.

- Remove unnecessary optional chaining on stationInfo inside
  version-switched cases where stationInfo is guaranteed non-null.

- Separate VERSION_16 from default in all auth switch/case blocks
  for spec-compliant version handling.

* fix(ocpp): buildMeterValue from transactionId, log date locale fix, MeterValues EVSE templates

- Refactor buildMeterValue to resolve connectorId/evseId from
  transactionId internally. Returns empty MeterValue when transactionId
  is undefined (MCP pass-through with provided meterValue).

- Fix log file date resolution: use local date instead of UTC to match
  Winston DailyRotateFile naming convention.

- Add optional date parameter to readCombinedLog/readErrorLog MCP tools
  for accessing rotated log files by date (YYYY-MM-DD).

- Add MeterValues field to EvseStatus/EvseTemplate types for EVSE-level
  meter value template definitions. getSampledValueTemplate checks
  EVSE-level templates first, falls back to merging connector templates.

- Refactor handleMeterValues to switch/case on OCPP version with
  explicit default error.

- Make AuthorizeRequest/AuthorizeResponse cross-version union types.
  Fix commandResponseToResponseStatus and isIdTagRemoteAuthorized with
  version-aware switch/case for OCPP 1.6 (idTagInfo) vs 2.0 (idTokenInfo).

- Add integration tests for readCombinedLog tool (default date, explicit
  date, missing file error). Add buildMeterValue unit tests.

2 weeks agochore(deps): lock file maintenance (#1747)
renovate[bot] [Mon, 23 Mar 2026 12:04:34 +0000 (13:04 +0100)] 
chore(deps): lock file maintenance (#1747)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
2 weeks agochore(deps): update dependency vite to v8 (#1717)
renovate[bot] [Mon, 23 Mar 2026 11:15:16 +0000 (12:15 +0100)] 
chore(deps): update dependency vite to v8 (#1717)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
2 weeks agochore: release main (#1743) ocpp-server@v3.2.1 simulator@v3.2.1 v3.2 webui@v3.2.1
Jérôme Benoit [Sun, 22 Mar 2026 18:35:32 +0000 (19:35 +0100)] 
chore: release main (#1743)

* chore: release main

* [autofix.ci] apply automated fixes

---------

Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
2 weeks agofix(ocpp2): implement StopTxOnInvalidId and MaxEnergyOnInvalidId per E05 (#1745)
Jérôme Benoit [Sun, 22 Mar 2026 18:26:00 +0000 (19:26 +0100)] 
fix(ocpp2): implement StopTxOnInvalidId and MaxEnergyOnInvalidId per E05 (#1745)

* fix(ocpp2): implement StopTxOnInvalidId and MaxEnergyOnInvalidId per E05

Implement the full deauthorization decision tree per OCPP 2.0.1 spec:
- StopTxOnInvalidId=false: transaction continues, no termination
- StopTxOnInvalidId=true + MaxEnergyOnInvalidId>0: defer termination
  until energy limit reached (checked in periodic meter values callback)
- StopTxOnInvalidId=true + MaxEnergyOnInvalidId=0: immediate suspension
  and termination

Also includes:
- Deduplicate transactionSetInterval/transactionTxUpdatedSetInterval
  into single transactionMeterValuesSetInterval field
- Add transactionDeauthorized/transactionDeauthorizedEnergyWh to
  ConnectorStatus for deauth state tracking (cleared in resetConnectorStatus)
- Extract readVariableValue/readVariableAsBoolean/readVariableAsInteger
  helpers using convertToInt and convertToBoolean utilities
- Fix buildEvsesStatus test that was missing evse-to-connector wiring

* refactor(ocpp2): address PR review comments

- Add logger.warn in readVariableAsInteger catch for diagnostic
- Replace unsafe 'as string' cast with .toString() in periodic callback
- Add tests for StopTxOnInvalidId=false and deauth state tracking

* chore: update webui.png

Signed-off-by: Jérôme Benoit <jerome.benoit@sap.com>
* test: add resetConnectorStatus unit tests to Helpers.test.ts

Verify all 18 transaction fields are properly cleaned, TX_PROFILE
charging profiles matching the transaction are removed, and
non-transaction fields are preserved.

---------

Signed-off-by: Jérôme Benoit <jerome.benoit@sap.com>
2 weeks agofix(ocpp2): align MeterValues implementation with OCPP 2.0.1 spec (#1744)
Jérôme Benoit [Sun, 22 Mar 2026 15:36:32 +0000 (16:36 +0100)] 
fix(ocpp2): align MeterValues implementation with OCPP 2.0.1 spec (#1744)

* fix(ocpp2): use AlignedDataInterval for standalone MeterValues

Add getAlignedDataInterval() helper to OCPP20ServiceUtils that reads
AlignedDataCtrlr.Interval from the variable registry (default 900s).

Replace getTxUpdatedInterval() with getAlignedDataInterval() in the
broadcast channel handleMeterValues OCPP 2.0 branch. Standalone
MeterValuesRequest is non-transaction data per OCPP 2.0.1 spec and
should use the aligned data interval, not the tx-updated interval.

* fix(ocpp2): include meter value in TransactionEvent Started

Add buildTransactionBeginMeterValues() to OCPP20ServiceUtils following
the buildFinalMeterValues() pattern. Builds an OCPP20MeterValue with
Transaction.Begin context and Energy.Active.Import.Register measurand.

Wire it into both TransactionEvent(Started) call sites:
- OCPPServiceUtils.startTransactionOnConnector (ATG/broadcast channel)
- OCPP20IncomingRequestService RequestStartTransaction event listener

This aligns OCPP 2.0 with the OCPP 1.6 beginEndMeterValues behavior
per OCPP 2.0.1 spec SampledDataTxStartedMeasurands requirement.

* test(ocpp2): add tests for buildTransactionBeginMeterValues

Test Transaction.Begin context, energy register value, default to 0
when undefined, and empty array when energy is negative.

* refactor(ocpp2): address PR review comments

- Extract buildEnergyMeterValues private helper to eliminate DRY
  violation between buildTransactionBeginMeterValues and
  buildFinalMeterValues
- Add AlignedDataInterval to OCPP20RequiredVariableName enum replacing
  raw string literal in getAlignedDataInterval
- Clarify zero-energy test name to document that 0 Wh is a valid
  Transaction.Begin reading
- Add meterValue assertions to RequestStartTransaction test verifying
  Transaction.Begin context and Energy.Active.Import.Register measurand

* refactor(ocpp2): extract readVariableAsIntervalMs to eliminate DRY

getAlignedDataInterval and getTxUpdatedInterval were structurally
identical. Extract the shared variable-reading logic into a private
readVariableAsIntervalMs helper that accepts component name, variable
name, and default seconds. Both public methods become one-liner
delegates.

* refactor(ocpp2): extract terminateTransaction and resolveActiveTransaction

requestDeauthorizeTransaction and requestStopTransaction shared
identical transaction termination logic (build final meter values,
send TransactionEvent Ended, stop periodic, reset connector status).

Extract resolveActiveTransaction for the shared precondition check
and transactionId string resolution, and terminateTransaction for
the shared Ended event + cleanup workflow. Both public methods now
focus only on their unique behavior.

* fix(test): use enum constants instead of string literals in RequestStartTransaction test

Replace 'Transaction.Begin' and 'Energy.Active.Import.Register' string
literals with OCPP20ReadingContextEnumType.TRANSACTION_BEGIN and
OCPP20MeasurandEnumType.ENERGY_ACTIVE_IMPORT_REGISTER to match
codebase conventions.

* refactor(test): replace string literals with enum constants in OCPP 2.0 tests

Replace hardcoded string literals with their corresponding OCPP 2.0.1
enum values across test files for type safety and consistency:

- RequestStopTransaction: Transaction.End, Energy.Active.Import.Register
- SchemaValidation: Immediate, OnIdle, Heartbeat, BootNotification
- enforceMessageLimits: TooManyElements, TooLargeElement
- CertificateManager: Accepted, Failed, NotFound
- ChangeAvailability: UnknownEvse
- GetBaseReport: Accepted (SetVariableStatusEnumType)
- CertificateSigned: InternalError
- ServiceUtils-TransactionEvent: TransactionEvent command name

* fix(test): replace bogus mock return values with correctly typed empty responses

requestDeauthorizeTransaction mocks returned { status: 'Accepted' }
but OCPP20TransactionEventResponse has no status field. Replace with
properly typed empty response objects since the return value is not
asserted in these tests.

* refactor(ocpp2): align method names with OCPP 2.0.1 spec terminology

Rename methods to match OCPP 2.0.1 TransactionEvent terminology:
- buildTransactionBeginMeterValues -> buildTransactionStartedMeterValues
- buildFinalMeterValues -> buildTransactionEndedMeterValues
- beginMeterValues variable -> startedMeterValues
- finalMeterValues variable -> endedMeterValues

The spec uses Started/Updated/Ended for TransactionEvent types, not
Begin/Final.

* refactor(test): replace 'Operative' string literal with enum constants

Use OCPP20OperationalStatusEnumType.Operative in OCPP 2.0 test files
and the cross-version AvailabilityType.Operative in MessageChannelUtils
test to match ConnectorStatus.availability typed field.

2 weeks agofix(deps): update all non-major dependencies (#1742)
renovate[bot] [Sun, 22 Mar 2026 13:52:31 +0000 (14:52 +0100)] 
fix(deps): update all non-major dependencies (#1742)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
2 weeks agofix(ocpp2): build meter values payload in broadcast channel for OCPP 2.0.x
Jérôme Benoit [Sun, 22 Mar 2026 11:40:27 +0000 (12:40 +0100)] 
fix(ocpp2): build meter values payload in broadcast channel for OCPP 2.0.x

The OCPP 2.0 branch in handleMeterValues was passing the raw broadcast
channel payload through to requestHandler without constructing the
required evseId and meterValue fields. Build the payload using
buildMeterValue and resolve evseId from connectorId, matching the
pattern used by the OCPP 1.6 branch.

2 weeks ago[autofix.ci] apply automated fixes
autofix-ci[bot] [Sat, 21 Mar 2026 20:35:45 +0000 (20:35 +0000)] 
[autofix.ci] apply automated fixes

2 weeks agochore: release main (#1733) ocpp-server@v3.2.0 simulator@v3.2.0 webui@v3.2.0
Jérôme Benoit [Sat, 21 Mar 2026 20:32:53 +0000 (21:32 +0100)] 
chore: release main (#1733)

2 weeks agorefactor(ocpp2): use enum values instead of string literals in variable registry
Jérôme Benoit [Sat, 21 Mar 2026 20:29:59 +0000 (21:29 +0100)] 
refactor(ocpp2): use enum values instead of string literals in variable registry

Replace all string literals that have corresponding TypeScript enums:
- OCPP20OperationalStatusEnumType for AvailabilityState defaultValue/enumeration
- OCPP20ChargingRateUnitEnumType for ChargingScheduleAllowedChargingRateUnit
  defaultValue, enumeration, and description

2 weeks agorevert(ocpp2): re-comment Available→Available self-transitions
Jérôme Benoit [Sat, 21 Mar 2026 20:20:07 +0000 (21:20 +0100)] 
revert(ocpp2): re-comment Available→Available self-transitions

Self-transitions are not allowed per OCPP 2.0.1 spec. The boot
sequence warning needs a different fix (skip StatusNotification
when connector status unchanged).

2 weeks agofix(ocpp2): add AvailabilityState defaults and allow self-transitions
Jérôme Benoit [Sat, 21 Mar 2026 20:15:55 +0000 (21:15 +0100)] 
fix(ocpp2): add AvailabilityState defaults and allow self-transitions

- Add defaultValue using OCPP20OperationalStatusEnumType.Operative
  enum for ChargingStation AvailabilityState (was missing, causing
  'Missing required configuration key mapping' error at auth time)
- Replace string literals with OCPP20OperationalStatusEnumType enum
  values in AvailabilityState enumeration arrays
- Allow Available→Available self-transitions in OCPP20Constants for
  both ChargingStation and Connector status tables (required by
  boot sequence StatusNotification per spec Part 2 §4.1 step 5)

2 weeks agofix(lifecycle): clear auth service on template reload and station reset
Jérôme Benoit [Sat, 21 Mar 2026 19:26:19 +0000 (20:26 +0100)] 
fix(lifecycle): clear auth service on template reload and station reset

Auth service instance was persisting across template reloads and
station resets, causing stale adapters if OCPP version changed.

- Add OCPPAuthServiceFactory.clearInstance() before initialize()
  in template file change handler
- Add OCPPAuthServiceFactory.clearInstance() before initialize()
  in reset() method

2 weeks agofix(auth): replace phantom spec references in source code comments
Jérôme Benoit [Sat, 21 Mar 2026 19:18:54 +0000 (20:18 +0100)] 
fix(auth): replace phantom spec references in source code comments

- G03.FR.01 → remove (was Change Availability, not auth)
- R10 → C10.FR.08 (cache expiration behavior)
- R16, R5 → C10.FR.08 (TTL reset 'since last used')
- R2 → C10.FR.07 (evict non-valid entries first)
- R17 → C13.FR.01 (local auth list priority over cache)

2 weeks agotest(auth): fix OCPP spec references and remove duplicated tests
Jérôme Benoit [Sat, 21 Mar 2026 19:11:34 +0000 (20:11 +0100)] 
test(auth): fix OCPP spec references and remove duplicated tests

Replace incorrect G03/G04 (Change Availability) prefixes with correct
OCPP 2.0.1 Authorization section codes (C10/C11/C12/C13/C15).
Remove spec prefixes from internal tests with no direct spec mapping.

- C10.FR.01: cache all statuses (was G03.FR.01.T4)
- C10.FR.07: eviction mechanism (was G03.FR.01.T5)
- C10.FR.08: TTL/expiry behavior (was G03.FR.01.T6/T7)
- C11: cache invalidation (was G03.FR.01.004)
- C12: cache miss behavior (was G03.FR.01.002)
- C13.FR.01: local auth list priority (was G03.FR.01.T8)
- C15: offline authorization (was G03.FR.02)
- Remove 5 duplicated cache unit tests from integration suite
- Update TEST_STYLE_GUIDE.md examples

2 weeks agotest(auth): strengthen assertion quality and edge case coverage
Jérôme Benoit [Sat, 21 Mar 2026 18:41:46 +0000 (19:41 +0100)] 
test(auth): strengthen assertion quality and edge case coverage

- G03.FR.01 cache tests now verify status and TTL, not just cache key
- Add stats increment verification after success and failure auth
- Add cache error resilience test (cache.set throws)
- Add idTag boundary tests for exactly 19, 20, 21 characters
- Rename misleading 'concurrent operations' test to 'sequential batch'

2 weeks agorefactor(auth): remove ocppVersion from UnifiedIdentifier
Jérôme Benoit [Sat, 21 Mar 2026 18:29:50 +0000 (19:29 +0100)] 
refactor(auth): remove ocppVersion from UnifiedIdentifier

Version info is now sourced from adapter.ocppVersion since each
station has a single adapter. Eliminates redundant version stamping
on every identifier.

- Remove ocppVersion field from UnifiedIdentifier interface
- CertificateAuthStrategy uses this.adapter.ocppVersion
- Adapters no longer stamp version on created identifiers
- Fix CertificateAuthStrategy OCPP 1.6 test to use 1.6 adapter
  instead of relying on missing certificateHashData

2 weeks agorefactor(auth): replace adapter Map with single adapter per station
Jérôme Benoit [Sat, 21 Mar 2026 18:01:48 +0000 (19:01 +0100)] 
refactor(auth): replace adapter Map with single adapter per station

Each charging station has exactly one OCPP version, making the
Map<OCPPVersion, OCPPAuthAdapter> unnecessary indirection. Simplify
to a single adapter field across the auth module.

- AuthComponentFactory: createAdapter() returns single OCPPAuthAdapter
- RemoteAuthStrategy: setAdapter()/clearAdapter() replace map ops
- CertificateAuthStrategy: readonly adapter field, no map
- OCPPAuthServiceImpl: single adapter, initializeAdapter() singular
- Tests updated to match new single-adapter API

2 weeks agofix(auth): address audit findings in OCPP auth abstraction layer
Jérôme Benoit [Sat, 21 Mar 2026 17:39:09 +0000 (18:39 +0100)] 
fix(auth): address audit findings in OCPP auth abstraction layer

- Read AuthCtrlr.LocalAuthorizeOffline variable instead of hardcoding true
- Delegate testConnectivity() to actual adapter availability checks
- Remove duplicate mapOCPP20AuthStatus(), use canonical mapOCPP20AuthorizationStatus
- Map strategies by name instead of fragile priority numbers
- Make connectorId truly optional in OCPP20AuthAdapter.authorizeRemote()
- Use ES2022 Error cause chain in AuthenticationError

2 weeks agofix(webui): use table-layout fixed so table shrinks with container when action panel...
Jérôme Benoit [Sat, 21 Mar 2026 17:03:40 +0000 (18:03 +0100)] 
fix(webui): use table-layout fixed so table shrinks with container when action panel opens

Replace min-width: 1280px + table-layout: auto with table-layout: fixed.
Auto layout ignores width: 100% and expands based on content, causing
the table to overflow behind the action panel. Fixed layout respects
the container width and distributes columns equally. Add overflow:
hidden on charging-stations-container as belt-and-suspenders clip.

2 weeks agofix(webui): remove white-space nowrap on buttons to prevent table overflow
Jérôme Benoit [Sat, 21 Mar 2026 16:56:43 +0000 (17:56 +0100)] 
fix(webui): remove white-space nowrap on buttons to prevent table overflow

The nowrap prevented button text from wrapping in narrow table columns,
causing the table to overflow its container when the action panel was
open. Buttons in the toolbar still fit without wrapping due to flex
sizing.

2 weeks agofix(webui): prevent action panel flex grow and allow table container to shrink
Jérôme Benoit [Sat, 21 Mar 2026 16:23:56 +0000 (17:23 +0100)] 
fix(webui): prevent action panel flex grow and allow table container to shrink

Use ID selector on #action-container for higher specificity (1-1-0)
over Container.vue scoped .container (0-2-0), add flex: none to prevent
growing. Add min-width: 0 and remove width: 100% on charging stations
container to allow flex shrink below table content width.

2 weeks agorefactor(webui): css quality overhaul with scoped styles, class selectors, design...
Jérôme Benoit [Sat, 21 Mar 2026 15:46:55 +0000 (16:46 +0100)] 
refactor(webui): css quality overhaul with scoped styles, class selectors, design tokens

- add scoped to all 12 Vue component style blocks
- move body/app global styles to index.html
- replace all 33 ID CSS selectors with class selectors
- add :deep() for parent-child table style propagation
- add spacing scale and typography tokens to all themes
- replace magic number percentages with spacing variables
- add :focus-visible styles for buttons and select
- replace hardcoded form widths with max-width constraints
- use gap for flex toolbar spacing
- remove dead CSS

2 weeks agofix(ocpp2.0): add proper response handlers for certificate commands + handler coverag...
Jérôme Benoit [Fri, 20 Mar 2026 20:39:36 +0000 (21:39 +0100)] 
fix(ocpp2.0): add proper response handlers for certificate commands + handler coverage tests

Replace emptyResponseHandler with proper logging handlers for
Get15118EVCertificate, GetCertificateStatus, and SignCertificate
responses. Add handler registration coverage tests for both OCPP 1.6
and 2.0: every RequestCommand must have a response handler, every
IncomingRequestCommand must have an incoming request handler.

2 weeks agofeat(ocpp2.0): implement Authorize request, fix double TransactionEvent(Started)
Jérôme Benoit [Fri, 20 Mar 2026 20:07:05 +0000 (21:07 +0100)] 
feat(ocpp2.0): implement Authorize request, fix double TransactionEvent(Started)

OCPP20AuthAdapter.authorizeRemote() was incorrectly sending a
TransactionEvent(Started) as a proxy for authorization, causing a
duplicate Started event when ATG subsequently called
startTransactionOnConnector(). Implement the proper OCPP 2.0.1
Authorize request/response flow, harmonizing the auth+start sequence
with OCPP 1.6: Authorize first, then TransactionEvent(Started).

2 weeks agofix(ocpp2.0): guard BroadcastChannel MeterValues for both VERSION_20 and VERSION_201
Jérôme Benoit [Fri, 20 Mar 2026 19:10:38 +0000 (20:10 +0100)] 
fix(ocpp2.0): guard BroadcastChannel MeterValues for both VERSION_20 and VERSION_201

The early return in handleMeterValues only checked VERSION_201, missing
VERSION_20. A station configured with ocppVersion '2.0' would fall
through to the OCPP 1.6 path and call convertToInt on a UUID string.

2 weeks agofix(ocpp2.0): pass real transactionId to buildMeterValue for energy accumulation
Jérôme Benoit [Fri, 20 Mar 2026 19:05:22 +0000 (20:05 +0100)] 
fix(ocpp2.0): pass real transactionId to buildMeterValue for energy accumulation

buildMeterValue was called with transactionId=0 for OCPP 2.0 stations,
but transaction IDs are UUID strings in 2.0.1. The energy lookup
compared 'uuid-string' === 0, always returning 0 Wh. Pass the actual
connectorStatus.transactionId and widen the parameter type to
number | string | undefined.

2 weeks agorefactor(webui): restore native table layout to fix column overlap on action panel...
Jérôme Benoit [Fri, 20 Mar 2026 19:00:23 +0000 (20:00 +0100)] 
refactor(webui): restore native table layout to fix column overlap on action panel open

Replace display:flex overrides on <table>/<tr>/<td> with native HTML table
layout. Add overflow-x:auto wrapper with min-width floor so columns scroll
horizontally instead of compressing and overlapping when the action panel
reduces available width.

2 weeks agofix(auth): log actual station OCPP version in RemoteAuthStrategy
Jérôme Benoit [Fri, 20 Mar 2026 18:49:23 +0000 (19:49 +0100)] 
fix(auth): log actual station OCPP version in RemoteAuthStrategy

Deduplicate adapter validation in initialize() to avoid logging the
same adapter twice for VERSION_20/VERSION_201 aliases. Add ocppVersion
to AuthConfiguration so strategies log the station's configured version
instead of internal map keys.

2 weeks agofix(ocpp2.0): align VARIABLE_REGISTRY spec conformance and log severity
Jérôme Benoit [Fri, 20 Mar 2026 18:23:38 +0000 (19:23 +0100)] 
fix(ocpp2.0): align VARIABLE_REGISTRY spec conformance and log severity

Distinguish required vs optional variables per OCPP 2.0.1 spec
(dm_components_vars.csv). Missing required variables log as error,
missing optional ones as warning. Tag 58 spec-required entries with
required: true and 6 non-spec entries with vendorSpecific: true.

2 weeks agofix(deps): update all non-major dependencies (#1739)
renovate[bot] [Fri, 20 Mar 2026 17:10:41 +0000 (18:10 +0100)] 
fix(deps): update all non-major dependencies (#1739)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
2 weeks agorefactor: build version-agnostic OCPP transaction primitives in service layer (#1741)
Jérôme Benoit [Fri, 20 Mar 2026 17:07:17 +0000 (18:07 +0100)] 
refactor: build version-agnostic OCPP transaction primitives in service layer (#1741)

* refactor: add StopTransactionResult type and OCPP 2.0 reason mapping

* refactor: extract OCPP 1.6 stopTransactionOnConnector to OCPP16ServiceUtils

* refactor: add unified stopTransactionOnConnector and stopRunningTransactions to OCPPServiceUtils

Add two version-dispatching functions to OCPPServiceUtils:

- stopTransactionOnConnector: dispatches to OCPP16ServiceUtils or
  OCPP20ServiceUtils via dynamic import, returns StopTransactionResult
- stopRunningTransactions: sequential for OCPP 1.6, parallel (Promise.all)
  for OCPP 2.0, includes transactionPending check for OCPP 2.0

Both use dynamic imports to avoid circular dependencies. stopRunningTransactions
is exposed as OCPPServiceUtils.stopRunningTransactions class member.
stopTransactionOnConnector is exported as standalone function only (class member
omitted due to OCPP16ServiceUtils override type conflict).

Includes 6 tests covering both OCPP versions and error cases.

* refactor: simplify ChargingStation and ATG to use OCPPServiceUtils unified methods

- ChargingStation.stopRunningTransactions: delegate to standalone function from OCPPServiceUtils
- Remove ChargingStation.stopRunningTransactionsOCPP20 private method (logic now in standalone function)
- Remove OCPP20ReasonEnumType import from ChargingStation (no longer needed)
- AutomaticTransactionGenerator.stopTransaction: call stopTransactionOnConnector standalone function
- Update ATG return type to Promise<StopTransactionResult | undefined>
- Use result.accepted instead of stopResponse.idTagInfo?.status === AuthorizationStatus.ACCEPTED
- Remove TODO comment from ATG.stopTransaction

* refactor: remove stopTransactionOnConnector from ChargingStation public API

* refactor: remove stopTransactionOnConnector from ChargingStation public API

- Remove ChargingStation.stopTransactionOnConnector public method
- Remove unused StopTransactionRequest, StopTransactionResponse, buildTransactionEndMeterValue, OCPP16ServiceUtils imports from ChargingStation
- Update OCPP16IncomingRequestService.handleRequestUnlockConnector to call OCPP16ServiceUtils.stopTransactionOnConnector directly
- Update test mocks in RemoteStopUnlock tests to target OCPP16ServiceUtils
- Fix OCPPServiceUtils-StopTransaction test: use args-based pattern for command detection, add proper JSDoc, fix type issues

* refactor(ocpp2.0): deduplicate stop pattern via stopAllTransactions

- Add OCPP20ServiceUtils.stopAllTransactions() — single factored function for parallel
  EVSE iteration + requestStopTransaction, supports optional evseId for single-EVSE scope
- terminateAllTransactions delegates to stopAllTransactions(station, ResetCommand, reason)
- terminateEvseTransactions delegates to stopAllTransactions(station, ResetCommand, reason, evseId)
- stopRunningTransactions OCPP 2.0 path delegates to stopAllTransactions(station, trigger, stopped)
- Eliminates 3x duplicated iteration+parallel-stop pattern

* refactor: add startTransactionOnConnector and flushQueuedTransactionMessages abstractions

- Add startTransactionOnConnector to OCPPServiceUtils (version dispatch via dynamic imports)
  OCPP 1.6: sends START_TRANSACTION via new OCPP16ServiceUtils.startTransactionOnConnector
  OCPP 2.0: sends TransactionEvent(Started) via OCPP20ServiceUtils.sendTransactionEvent
  Returns StartTransactionResult { accepted: boolean }
- Add flushQueuedTransactionMessages to OCPPServiceUtils (OCPP 1.6: no-op, OCPP 2.0: flushes queue)
- Migrate ATG startTransaction to use startTransactionOnConnector (fixes OCPP 2.0 ATG start)
- Migrate ATG handleStartTransactionResponse to handleStartTransactionResult (uses unified type)
- Remove ATG dependency on AuthorizationStatus, RequestCommand, StartTransactionRequest/Response
- Remove ChargingStation.flushQueuedTransactionEvents private method
- Remove version check in ChargingStation boot handler (line 2304)
- Add StartTransactionResult type to types barrel

* refactor: extract periodic meter values to OCPP service layer

- Add OCPP16ServiceUtils.startPeriodicMeterValues/stopPeriodicMeterValues
- Add OCPP20ServiceUtils.startPeriodicMeterValues/stopPeriodicMeterValues
- Add OCPPServiceUtils.startPeriodicMeterValues/stopPeriodicMeterValues (version dispatch)
- Remove startMeterValues, stopMeterValues, startTxUpdatedInterval, stopTxUpdatedInterval,
  restartMeterValues from ChargingStation.ts
- Migrate all callers to use versioned ServiceUtils methods
- Fix test referencing removed startTxUpdatedInterval method
- ChargingStation.ts: -130 lines of version-specific meter values logic removed

* fix: update tests for renamed APIs and removed ChargingStation methods

- AutomaticTransactionGenerator.test.ts: handleStartTransactionResponse -> handleStartTransactionResult, use StartTransactionResult { accepted } instead of StartTransactionResponse
- ChargingStation-Transactions.test.ts: test already uses OCPP16ServiceUtils.stopPeriodicMeterValues
- OCPP20ServiceUtils-TransactionEvent.test.ts: startTxUpdatedInterval -> startPeriodicMeterValues
- StationHelpers.ts: remove stopMeterValues, startMeterValues, startTxUpdatedInterval, stopTxUpdatedInterval, restartMeterValues from mock (no longer on ChargingStation)
- OCPP16ServiceUtils.stopPeriodicMeterValues: add missing delete after clearInterval

* fix: correct startPeriodicMeterValues test to verify no-transaction guard instead of version guard

* fix: guard undefined evseId and avoid throw in shutdown paths

- stopTransactionOnConnector: guard getEvseIdByConnectorId returning undefined, return { accepted: false } with warn log
- stopRunningTransactions: replace throw with warn log in default branch (shutdown path must not crash)

* fix: deduplicate test coverage and address review findings

- Remove ChargingStation-StopRunningTransactions.test.ts (duplicated coverage with OCPPServiceUtils-StopTransaction.test.ts)
- Move error handling test to OCPPServiceUtils-StopTransaction.test.ts
- Guard undefined evseId in stopTransactionOnConnector (review finding)
- Replace throw with warn in stopRunningTransactions default branch (review finding)

* fix: generate transactionId for OCPP 2.0 start and align offline stop acceptance

- startTransactionOnConnector OCPP 2.0: generate UUID transactionId and reset seqNo when
  connector has no transactionId (ATG path)
- stopTransactionOnConnector OCPP 2.0: treat missing idTokenInfo as accepted (offline queued
  events return undefined idTokenInfo, consistent with startTransactionOnConnector)

* fix: map Other reason correctly and use static generateUUID import

- mapStopReasonToOCPP20: add explicit Other -> Other/AbnormalCondition mapping
  instead of falling through to Local/StopAuthorized
- startTransactionOnConnector: use statically imported generateUUID instead of
  unnecessary dynamic import (utils/index.js already imported at top of file)

* test: add coverage for startTransactionOnConnector, flushQueuedTransactionMessages, and mapStopReasonToOCPP20

- startTransactionOnConnector: OCPP 1.6 accepted, OCPP 2.0 accepted, OCPP 2.0 UUID generation
- flushQueuedTransactionMessages: OCPP 1.6 no-op, OCPP 2.0 flush queued events
- mapStopReasonToOCPP20: Other, undefined, Remote mappings

* refactor: remove stopRunningTransactions indirection in ChargingStation

Call stopRunningTransactions(this, reason) directly instead of through
a private method that only delegates.

2 weeks agofix(ocpp2.0): make stopRunningTransactions version-aware for OCPP 2.0
Jérôme Benoit [Thu, 19 Mar 2026 17:59:42 +0000 (18:59 +0100)] 
fix(ocpp2.0): make stopRunningTransactions version-aware for OCPP 2.0

2 weeks agorefactor(ocpp2.0): parameterize requestStopTransaction triggerReason and stoppedReason
Jérôme Benoit [Thu, 19 Mar 2026 17:49:02 +0000 (18:49 +0100)] 
refactor(ocpp2.0): parameterize requestStopTransaction triggerReason and stoppedReason

2 weeks agofix(ocpp2.0): reset transaction sequence number before new transaction per E01.FR.07
Jérôme Benoit [Thu, 19 Mar 2026 17:41:30 +0000 (18:41 +0100)] 
fix(ocpp2.0): reset transaction sequence number before new transaction per E01.FR.07

2 weeks agofix(ocpp2.0): inject offline:true in sendTransactionEvent offline path per E04.FR.03
Jérôme Benoit [Thu, 19 Mar 2026 17:29:44 +0000 (18:29 +0100)] 
fix(ocpp2.0): inject offline:true in sendTransactionEvent offline path per E04.FR.03

2 weeks agorefactor(webui): harmonize barrel usage in types and composables
Jérôme Benoit [Thu, 19 Mar 2026 15:27:49 +0000 (16:27 +0100)] 
refactor(webui): harmonize barrel usage in types and composables

Add OCPP16AvailabilityType, OCPP16ChargePointStatus, and
OCPP16RegistrationStatus to the types barrel. Redirect all direct
module imports in tests to use barrels (types/, composables/).

2 weeks agofix: merge duplicate node:process imports in Utils.test.ts
Jérôme Benoit [Thu, 19 Mar 2026 15:21:43 +0000 (16:21 +0100)] 
fix: merge duplicate node:process imports in Utils.test.ts

2 weeks agofix: merge duplicate type imports from same barrel module
Jérôme Benoit [Thu, 19 Mar 2026 15:18:42 +0000 (16:18 +0100)] 
fix: merge duplicate type imports from same barrel module

Consolidate split import type statements from types/index.js into
single imports in OCPP16TestUtils.ts and OCPP20IncomingRequestService-
MasterPass.test.ts.

2 weeks agorefactor: harmonize barrel usage across all test files
Jérôme Benoit [Thu, 19 Mar 2026 15:11:03 +0000 (16:11 +0100)] 
refactor: harmonize barrel usage across all test files

Redirect all direct module imports (utils/Logger, utils/Constants,
worker/WorkerTypes, types/*, charging-station/ChargingStation) to
their respective barrel index.js in test files. Unit tests of the
module itself retain direct imports per convention.

2 weeks agofix: use exception barrel instead of direct module imports
Jérôme Benoit [Thu, 19 Mar 2026 14:58:59 +0000 (15:58 +0100)] 
fix: use exception barrel instead of direct module imports

Redirect OCPPError imports from exception/OCPPError.js to the barrel
exception/index.js in auth services and OCPP20 test utils.

2 weeks agofix(ocpp): keep StopTransactionRequest as 1.6 wire type, narrow reason via indexed...
Jérôme Benoit [Thu, 19 Mar 2026 14:53:24 +0000 (15:53 +0100)] 
fix(ocpp): keep StopTransactionRequest as 1.6 wire type, narrow reason via indexed access

Revert the Omit-based widening of StopTransactionRequest — it's a 1.6
wire protocol message and should not accept OCPP 2.0 reason values.
Use StopTransactionRequest['reason'] indexed access type for narrowing
in ChargingStation, avoiding any version-specific OCPP16* import.