daemon: add identity metadata to health response#7
Open
MKV21 wants to merge 2 commits into
Open
Conversation
MKV21
force-pushed
the
codex-ask-health-diagnostics
branch
3 times, most recently
from
ce8af38 to
dca4c45
Compare
MKV21
force-pushed
the
codex-ask-health-diagnostics
branch
from
dca4c45 to
fa5e037
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Extends
openvide-daemon healthwith identity metadata so the app/CLI can detect when it's talking to an older globally-installed daemon than the one it was built against.openvide-daemon versionbehavior is unchanged.New fields on the
healthIPC response:version— daemon package version (sourced frombuildInfo.ts)daemonPath— absolute path of the running daemon entry (process.argv[1])nodeVersion—process.versioncapabilities— static feature flags (codexAppServer,codexAskMode)Existing fields (
pid,name,activeSessions,totalSessions,tools) are preserved. All new fields are optional onIpcResponse, so existing clients are unaffected.Changes
apps/daemon/src/buildInfo.ts— new module exportingDAEMON_VERSIONandDAEMON_CAPABILITIES.DAEMON_VERSIONis commented to stay in sync withpackage.json.apps/daemon/src/cli.ts— importsDAEMON_VERSIONfrombuildInfo.tsinstead of a local constant.apps/daemon/src/ipc.ts— extends thehealthhandler with the four new fields.apps/daemon/src/types.ts— adds optional fields toIpcResponse.apps/daemon/test/health.test.mjs— newnode --testcase covering the health shape.apps/daemon/package.json— addstestscript (npm run build && node --test test/*.test.mjs).Test plan
npm test --workspace @openvide/daemon./node_modules/.bin/tsc -p apps/daemon/tsconfig.jsonopenvide-daemon health, confirm new fields are present anddaemonPathpoints at the freshly built binary.openvide-daemon versionstill prints the package version only.Notes
DAEMON_VERSIONis still a string literal — keep it in sync withpackage.jsonuntil we add a build-time codegen step.capabilitiesare intentionally static booleans. New capabilities are added by new keys; CLI clients can feature-detect by key presence.AI disclosure
Parts of this change were drafted by Codex (AI coder) and reviewed by Claude Code (AI reviewer). A human (@MKV21) reviewed, tested, and approved the final result before submission.