feat: Add getVersion() to KeymasterClient

[macterra]

Summary

• Add getVersion() method to KeymasterClient returning { version, commit }
• Fix GatekeeperClient.getVersion() return type from number to { version: string, commit: string }
• Add get_version() to Python SDK for parity
• Replace raw fetch(serverUrl + '/api/v1/version') in KeymasterUI with keymaster.getVersion()

Closes #205

Test plan

• Verify server version displays correctly in Settings tab for gatekeeper-client
• Verify server version displays correctly in Settings tab for keymaster-client
• Verify keymaster.getVersion() works from browser console

🤖 Generated with Claude Code

[Copilot]

Pull request overview

Adds a first-class getVersion() call to the JS/TS Keymaster client and wires the web UIs (and Python SDK) to use it so version fetching is centralized in the client layer rather than ad-hoc fetch() calls.

Changes:

• Add KeymasterClient.getVersion() returning { version, commit }
• Fix GatekeeperClient.getVersion() typing to return { version, commit } instead of number
• Add Python SDK get_version() and show client/server versions in both wallet UIs

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 5 comments.

Show a summary per file

File	Description
python/keymaster_sdk/src/keymaster_sdk/keymaster_sdk.py	Adds get_version() wrapper for /api/v1/version in the Python SDK.
packages/keymaster/src/keymaster-client.ts	Adds getVersion() method calling ${API}/version and returning { version, commit }.
packages/gatekeeper/src/gatekeeper-client.ts	Updates getVersion() return type to { version, commit } to match server response.
apps/keymaster-client/src/KeymasterUI.jsx	Uses keymaster.getVersion() when available and displays client/server version in Settings.
apps/gatekeeper-client/src/KeymasterUI.jsx	Same UI change as keymaster-client: prefers keymaster.getVersion() and displays versions.

Comments suppressed due to low confidence (1)

packages/gatekeeper/src/gatekeeper-client.ts:165

• This return type change will break the existing Gatekeeper client test mocks (and any consumers) that still treat /api/v1/version as a numeric/string version. For example, tests/gatekeeper/client.test.ts currently mocks '1' and asserts toBe(1), but the Gatekeeper server actually returns { version, commit }. Update the tests/mocks (and any callers) to expect the object shape.

    async getVersion(): Promise<{ version: string; commit: string }> {
        try {
            const response = await this.axios.get(`${this.API}/version`);
            return response.data;
        }

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.