Skip to content

Release 1.35.1: restore Toolkit catalogue loading - #145

Merged
cursor[bot] merged 2 commits into
masterfrom
dougrathbone/fix-toolkit-catalog-java17-8423
Sep 30, 2026
Merged

cursor[bot] merged 2 commits into
masterfrom
dougrathbone/fix-toolkit-catalog-java17-8423

Conversation

@dougrathbone

@dougrathbone dougrathbone commented Sep 30, 2026 •

Copy link
Copy Markdown
Owner

Summary

Fixes the managed-mode Toolkit failure reported in #122 where the secure C-Gate session opens but Toolkit eventually reports “No Catalog Available”. C-Gate 3.3.x needs access to a JDK-internal XML serializer when serving its database/catalogue response, which Java 17 blocks by default.

This also releases that fix as 1.35.1.

Changes

  • Open the required XML serializer package when starting C-Gate on modular JVMs.
  • Probe JVM support first so Java 8 remains usable on 32-bit add-on architectures.
  • Add regression coverage for both Java 17+ and Java 8 startup paths.
  • Bump the add-on to 1.35.1.

Test plan

  • npm test passes locally with no failures
  • New code has unit test coverage (aim for ≥ 60% on changed files)
  • Existing tests were not broken or removed without justification
  • npm run lint passes
  • npm run typecheck passes

Checklist

  • Version bumped in package.json and homeassistant-addon/config.yaml (if releasing)
  • CHANGELOG.md updated (if releasing)
  • No sensitive data or credentials included
Open in Web Open in Cursor 

C-Gate 3.3.x uses a JDK-internal XML serializer when Toolkit requests
its database and unit catalogue. Java 17 blocks that package by default,
so Toolkit establishes its secure command session but eventually reports
No Catalog Available. Open the serializer package on modular JVMs while
preserving Java 8 startup on the 32-bit add-on architectures.
@cursor cursor Bot changed the title Fix Toolkit catalogue loading against managed C-Gate Release 1.35.1: restore Toolkit catalogue loading Sep 30, 2026
@dougrathbone
dougrathbone marked this pull request as ready for review September 30, 2026 21:58
@cursor
cursor Bot merged commit 82640a0 into master Sep 30, 2026
25 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.

1 participant