Skip to content

Implement client-side logging with a new server endpoint, configurable settings, and dedicated tests.#923

Draft
Tim020 wants to merge 4 commits intodevfrom
feature/client-log-server
Draft

Implement client-side logging with a new server endpoint, configurable settings, and dedicated tests.#923
Tim020 wants to merge 4 commits intodevfrom
feature/client-log-server

Conversation

@Tim020
Copy link
Contributor

@Tim020 Tim020 commented Feb 18, 2026

No description provided.

@github-actions github-actions bot added client Pull requests changing front end code server Pull requests changing back end code medium-diff labels Feb 18, 2026
@github-actions
Copy link

github-actions bot commented Feb 18, 2026

Client Test Results

99 tests  ±0   99 ✅ ±0   0s ⏱️ ±0s
 4 suites ±0    0 💤 ±0 
 1 files   ±0    0 ❌ ±0 

Results for commit df0850a. ± Comparison against base commit 3de1161.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Feb 18, 2026

Python Test Results

  1 files  ±0    1 suites  ±0   54s ⏱️ -4s
511 tests +4  511 ✅ +4  0 💤 ±0  0 ❌ ±0 
516 runs  +4  516 ✅ +4  0 💤 ±0  0 ❌ ±0 

Results for commit df0850a. ± Comparison against base commit 3de1161.

♻️ This comment has been updated with latest results.

@Tim020 Tim020 linked an issue Feb 19, 2026 that may be closed by this pull request
Tim020 and others added 2 commits February 19, 2026 19:16
…o file settings

- `client_log_level` was defined without a callback_fn, so changing the
  level at runtime via the settings UI would update the frontend (which
  reads SETTINGS dynamically on each log call) but leave the Python
  client_logger frozen at its startup level, silently dropping messages.
  Added `self._application.regen_logging` as the 5th positional arg to
  match the pattern used by `log_level` and the other client_log_* settings.

- Added `help_text` to `client_log_path`, `client_max_log_mb`, and
  `client_log_backups` to match the style used by `client_log_enabled`
  and `client_log_level`.

Co-Authored-By: Claude Sonnet 4.6 <[email protected]>
@sonarqubecloud
Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

client Pull requests changing front end code medium-diff server Pull requests changing back end code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Send client side logs to backend

1 participant

Comments