Skip to content

refactor(config): remove unused service-info and consolidate on build-info (#297) - #305

Merged
David-patrick-chuks-02 merged 2 commits into
Lilly-Protocol:mainfrom
ghzhost:refactor/remove-unused-service-info-297
Sep 6, 2026
Merged

refactor(config): remove unused service-info and consolidate on build-info (#297)#305
David-patrick-chuks-02 merged 2 commits into
Lilly-Protocol:mainfrom
ghzhost:refactor/remove-unused-service-info-297

Conversation

@ghzhost

@ghzhost ghzhost commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Summary

Closes #297

  • Removes the redundant src/config/service-info.ts module in favor of src/config/build-info.ts as the single metadata source.
  • Updates tests/service-info.test.ts to test buildInfo directly alongside the existing health endpoint tests.
  • Translates the non-English comment in src/config/build-info.ts to English.
  • Preserves identical runtime behavior for GET /api/v1/health and server startup logging.

Verification

  • npx vitest run tests/service-info.test.ts tests/health.test.ts tests/health-payload.test.ts passes 100% (14 tests passed).
  • Checked for any remaining imports/usages across src/ and tests/.

@David-patrick-chuks-02
David-patrick-chuks-02 merged commit 16da80a into Lilly-Protocol:main Sep 6, 2026
1 check failed
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.

[Bounty: $80] Remove the unused service-info module and keep build-info as the single metadata source

3 participants