archetech / archon / 23119891353

94%

push

github

feat: Add getVersion() to KeymasterClient (#219)

* feat: Add getVersion() to KeymasterClient and use in UI

Add getVersion() method to KeymasterClient that returns { version,
commit } from the server's /version endpoint. Fix GatekeeperClient
getVersion() return type from number to { version, commit }. Add
get_version() to Python SDK for parity.

Replace raw fetch('/api/v1/version') in KeymasterUI with
keymaster.getVersion(), decoupling the UI from URL details.

Closes #205

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* fix: Add getVersion() to Keymaster class via gatekeeper delegation

The Keymaster core class (used by gatekeeper-client) didn't have
getVersion(), only KeymasterClient did. Delegate to the gatekeeper's
getVersion() with a runtime check since GatekeeperInterface doesn't
include it (local Gatekeeper has no version endpoint).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* fix: Read Keymaster version from package.json instead of gatekeeper

The Keymaster class is a local library, not a remote client. Its
getVersion() should return its own package version, not delegate to
the gatekeeper.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* fix: Use hardcoded version constant instead of createRequire

createRequire is Node-only and breaks when Vite bundles keymaster for
browser clients. Use a simple version constant instead.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* fix: Remove getVersion from Keymaster class, use runtime check in UI

Keymaster is a library used in both Node and browser — importing
package.json or using createRequire breaks Vite builds. Remove
getVersion() from the Keymaster class. The UI now checks if
keymaster.getVersion exists (KeymasterClient has it), falling back
to a direct fetch with serverUrl for the local Keymaster case.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* fix: Display version i... (continued)

1755 of 1933 branches covered (90.79%)

Branch coverage included in aggregate %.

0 of 4 new or added lines in 1 file covered. (0.0%)

3751 of 3912 relevant lines covered (95.88%)

1176.37 hits per line