Skip to content

Dev. CI. Automatic endpoint performance monitoring js_keys__get - #923

Merged
AntonV1211 merged 4 commits into
devfrom
ci_kg_av
Oct 30, 2026
Merged

AntonV1211 merged 4 commits into
devfrom
ci_kg_av

Conversation

@AntonV1211

Copy link
Copy Markdown
Contributor

@AntonV1211 AntonV1211 changed the title Ci kg av Dev. CI. Automatic endpoint performance monitoring js_keys__get Oct 29, 2026
@codecov

codecov Bot commented Oct 29, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 23 lines in your changes missing coverage. Please review.
✅ Project coverage is 34.70%. Comparing base (899f33b) to head (19a99b5).

Files with missing lines Patch % Lines
lib/Cleantalk/ApbctWP/AJAXService.php 0.00% 19 Missing ⚠️
lib/Cleantalk/ApbctWP/Constant.php 0.00% 4 Missing ⚠️

❌ Your patch check has failed because the patch coverage (0.00%) is below the target coverage (70.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files
@@             Coverage Diff              @@
##                dev     #923      +/-   ##
============================================
- Coverage     34.73%   34.70%   -0.03%     
- Complexity     7085     7091       +6     
============================================
  Files           298      298              
  Lines         27500    27522      +22     
============================================
  Hits           9551     9551              
- Misses        17949    17971      +22     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟡 Changes recommended

A missing constant type annotation breaks the beta workflow and blocks release ZIP generation, while nonpositive request counts silently disable performance monitoring without a warning.

2 open issues
What changed in this PR

Adds automated performance monitoring for the js_keys__get endpoint using opt-in server metrics and a dedicated GitHub Actions workflow.

Changes:

  • Adds configurable Server-Timing headers with timing, SQL query count, and peak memory metrics.
  • Introduces a beta/manual workflow that provisions WordPress and runs measurements.
  • Reports measurements and threshold violations through job summaries and warnings.
File Description
lib/​Cleantalk/​ApbctWP/​Constant.php Registers the performance-header configuration flag.
lib/​Cleantalk/​ApbctWP/​AJAXService.php Emits optional performance metrics for js_keys__get.
cleantalk.php Records plugin startup time.
.gitignore Ignores Python cache files.
.github/​workflows/​perf-js-keys.yml Provisions WordPress and runs endpoint measurements.
.github/​perf/​js_keys_perf.py Collects metrics, checks thresholds, and generates reports.

Review effort: Balanced

Comment thread .github/perf/js_keys_perf.py Resolved
Comment thread lib/Cleantalk/ApbctWP/Constant.php Resolved
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@AntonV1211
AntonV1211 requested review from Glomberg and svedge October 29, 2026 11:20
@AntonV1211
AntonV1211 merged commit 5af7dbe into dev Oct 30, 2026
9 of 10 checks passed
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.

4 participants