Skip to content

Added "info" API endpoint to retrieve app info and configuration (to be used by the client)#125

Merged
SteRiccio merged 14 commits intomasterfrom
feat/api-info
Mar 3, 2026
Merged

Added "info" API endpoint to retrieve app info and configuration (to be used by the client)#125
SteRiccio merged 14 commits intomasterfrom
feat/api-info

Conversation

@SteRiccio
Copy link
Member

No description provided.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds a new info API endpoint intended to expose application identity/version and selected runtime configuration for client consumption, and centralizes the server app identifier constant.

Changes:

  • Introduces /api/info endpoint (endpoint wiring + handler) and adds an API test for it.
  • Adds EXPERIMENTAL_FEATURES env flag (ProcessEnv.experimentalFeatures) and updates .env.template.
  • Adds ArenaServerConstants.appId and reuses it for 2FA otpauth issuer generation.

Reviewed changes

Copilot reviewed 12 out of 13 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
src/service/user2FA/user2FAService.ts Uses ArenaServerConstants.appId instead of a local APP_NAME constant when generating the otpauth URL.
src/processEnv/index.ts Adds ProcessEnv.experimentalFeatures sourced from EXPERIMENTAL_FEATURES.
src/model/index.ts Re-exports ArenaServerConstants.
src/model/ArenaServerConstants.ts Defines a central appId constant.
src/api/info/read.ts Implements GET /api/info returning { appInfo, config }.
src/api/info/index.ts Registers the info read route via InfoApi.
src/api/endpoint/info.ts Adds endpoint builder for /api/info.
src/api/endpoint/index.ts Exposes ApiEndpoint.info.
src/api/api.ts Wires InfoApi into overall API initialization.
src/api/tests/info/index.ts Adds a test for the info endpoint response.
src/api/tests/api.test.ts Registers the new info test suite.
.env.template Documents EXPERIMENTAL_FEATURES.

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

@sonarqubecloud
Copy link

sonarqubecloud bot commented Mar 3, 2026

@SteRiccio SteRiccio merged commit 64d1417 into master Mar 3, 2026
5 checks passed
@mergify mergify bot deleted the feat/api-info branch March 3, 2026 11:12
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.

2 participants