Hello @deluan, I'm Gemini Code Assist¹! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

Summary of Changes

As gemini-code-assist, I've reviewed this pull request which aims to enhance the 'About' dialog in the UI by adding a new tab to display the server's current configuration. This involves creating a new API endpoint on the server to expose the configuration (only to admin users), flattening the potentially nested configuration structure for easier display, and updating the frontend About dialog to fetch and present this data in a dedicated 'Configuration' tab, dynamically adjusting the dialog's size.

Highlights

• New API Endpoint for Configuration: A new native API endpoint /config has been added to expose the server's current configuration settings. This endpoint is protected and only accessible to admin users.
• Configuration Flattening Logic: The server-side implementation includes a flatten function using reflection to traverse the nested configuration structure and present it as a flat list of key-value pairs, making it suitable for display in the UI.
• UI Integration in About Dialog: The frontend 'About' dialog has been modified to include a new 'Configuration' tab. This tab fetches the data from the new /config endpoint (if the user is an admin and the feature is enabled) and displays the flattened configuration settings in a table.
• Feature Toggle and UI Responsiveness: A new development configuration option, DevUIShowConfig, has been introduced to control the visibility of the 'Configuration' tab in the UI. The About dialog's width is also dynamically adjusted when switching between the 'About' and 'Configuration' tabs for a smoother user experience.

Changelog

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Config flies unseen,
Now a tab, a list appears,
Settings brought to light.

Code Review

This pull request successfully introduces a new configuration tab in the About dialog, which is a valuable addition for administrators to inspect server settings. The backend work includes a new /config endpoint that flattens the server configuration using reflection, and the frontend integrates this data into a new tab with a smooth UI transition for dialog width.

Overall, the implementation is solid, with good test coverage for the new backend endpoint and appropriate UI updates. I've identified a couple of areas in the backend error handling that could be improved for robustness. No explicit style guide was provided, so feedback is based on common Go and React/JavaScript best practices.

Summary of Findings

• Error Handling in server/nativeapi/config.go: Errors from json.Marshal (when serializing complex config values) and json.NewEncoder().Encode() (when sending the HTTP response) are currently ignored. This could mask issues and make debugging more difficult. It's recommended to log these errors.

Merge Readiness

The pull request is well-structured and the new feature is implemented effectively. However, there are a couple of medium-severity issues related to error handling in the new server/nativeapi/config.go file that should be addressed to improve robustness and debuggability. Once these are considered, the PR should be in good shape for merging. As a reviewer, I am not authorized to approve pull requests; please ensure further review and approval by authorized team members.

Pull Request Overview

This PR introduces a configuration tab in the About dialog while also flattening and exposing the server configuration via the /config endpoint. Key changes include updates to the UI translations and dialogs, additions to the server configuration and endpoints, and new tests ensuring that configuration properties (including the new DevUIShowConfig flag) are correctly exposed.

Reviewed Changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 1 comment.

Pull Request Overview

This PR introduces a new “Configuration” tab in the About dialog for admin users, driven by a new devUIShowConfig flag, and implements a secured /config endpoint to expose application settings with redaction.

• Add devUIShowConfig flag in server and UI defaults and expose it in the injected app config
• Implement a native API getConfig handler with flattening, sorting, redaction, and comprehensive tests
• Update the AboutDialog UI to support tabs (About + Configuration), fetch config data, and include new translations

Reviewed Changes

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

const AboutDialog = ({ open, onClose }) => {

Pull Request Overview

Adds a new “Configuration” tab in the About dialog, backed by a server‐side /config endpoint with admin‐only access and redacted sensitive values, and allows exporting the current config as TOML.

• Introduce devUIShowConfig feature flag (default true) throughout server and client
• Register new /config API route and implement handler to return flattened, redacted config
• Build a React tab in AboutDialog (admin only) showing config values, with TOML export and translations

Reviewed Changes

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

"Prometheus.MetricsPath",