Skip to content

2.8.0 - #167

Merged
bfoujols merged 16 commits into
mainfrom
2.x
Oct 20, 2026
Merged

2.8.0#167
bfoujols merged 16 commits into
mainfrom
2.x

Conversation

@bfoujols

Copy link
Copy Markdown
Contributor

No description provided.

bfoujols and others added 16 commits October 17, 2026 22:41
…s liens pour DbGate et phpMyAdmin

fix(debug-bar): correction de l'alignement des colonnes contenant des liens dans le panneau de la barre de debug
docs(changelog): ajout des corrections et améliorations dans le changelog
Profilage à la demande du code PHP exécuté par requête HTTP, avec
l'extension Excimer (échantillonnage 0,1 ms, compatible FrankenPHP).

- Docker : extension excimer installée (ARG EDU_PROFILER), variables
  EDU_PROFILER (off|trigger|on) et EDU_PROFILER_KEEP
- CodeProfiler démarré/arrêté par LoadCouchCore, profil rattaché au
  token de la requête et stocké dans serv_code_profiles
- Endpoint /edu-code-profile (résumé, full=1, export Speedscope)
- Bloc Code dans la barre de debug (cookie edu_profile) et onglet Code
  dans le profiler (temps propre/inclusif, appelants/appelés)
- Tests, CI (extension excimer), documentation et CHANGELOG

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_017anBiidLUwUBVStBQuAbVn
La décompression d'un profil corrompu convertit l'avertissement en
exception, l'envoie dans les logs PHP et retourne des données vides,
au lieu de le masquer avec l'opérateur @.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_017anBiidLUwUBVStBQuAbVn
…ocole

Reprend le changement de 2.x (texte des liens Mailpit, DbGate et
phpMyAdmin sans http://) et met à jour le test qui échouait sur la PR.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_017anBiidLUwUBVStBQuAbVn
- Temps profilé mesuré avant l'arrêt du driver (agrégation exclue)
- « Désactiver » retire aussi ?_profile=1 de l'URL
- Bloc Code : vrai bouton Désactiver hors du lien, panneau ouvert au focus
- Onglet Code : tri, sélection de fonction et graphe accessibles au clavier
  (boutons natifs)
- Réponses d'un profil qui n'est plus affiché ignorées
- Réouverture de page limitée aux URL du site, avec noopener

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_017anBiidLUwUBVStBQuAbVn
…D1002)

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_017anBiidLUwUBVStBQuAbVn
…PHP-D1002)

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_017anBiidLUwUBVStBQuAbVn
- DebugCodeProfileController::execute() découpé (export Speedscope et
  détail complet dans des méthodes privées) : complexité sous le seuil
- Variables courtes renommées dans CodeProfileAggregator
- Lecture des superglobales dans CodeProfiler::isRequested() justifiée
  (le profilage démarre avant le routage) et annotée pour PHPMD

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_017anBiidLUwUBVStBQuAbVn
…1076)

DeepSource ne voyait pas les appels à la méthode privée fakeDriver(),
qui retournait une classe anonyme. Les deux copies sont remplacées par
une classe FakeProfilerDriver chargée par le bootstrap des tests.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_017anBiidLUwUBVStBQuAbVn
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@deepsource-io

deepsource-io Bot commented Oct 20, 2026 •

Copy link
Copy Markdown
Contributor

DeepSource Code Review

We reviewed changes in dfabe4b...254b0da on this pull request. Below is the summary for the review, and you can see the individual issues we found as inline review comments.

See full review on DeepSource ↗

PR Report Card

Overall Grade   Security  

Reliability  

Complexity  

Hygiene  

Code Review Summary

Analyzer Status Updated (UTC) Details
PHP Oct 20, 2026 7:46p.m. Review ↗
Shell Oct 20, 2026 7:46p.m. Review ↗

Important

AI Review is run only on demand for your team. We're only showing results of static analysis review right now. To trigger AI Review, comment @deepsourcebot review on this thread.

@codacy-production

codacy-production Bot commented Oct 20, 2026 •

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 105 complexity · 0 duplication

Metric Results
Complexity 105
Duplication 0

View in Codacy

🟢 Coverage 94.54% diff coverage · +2.27% coverage variation

Metric Results
Coverage variation ✅ +2.27% coverage variation (-1.00%)
Diff coverage ✅ 94.54% diff coverage

View coverage diff in Codacy

Coverage variation details
Coverable lines Covered lines Coverage
Common ancestor commit (dfabe4b) 1530 1223 79.93%
Head commit (254b0da) 1815 (+285) 1492 (+269) 82.20% (+2.27%)

Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: <coverage of head commit> - <coverage of common ancestor commit>

Diff coverage details
Coverable lines Covered lines Diff coverage
Pull request (#167) 293 277 94.54%

Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: <covered lines added or modified>/<coverable lines added or modified> * 100%

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@bfoujols
bfoujols merged commit a5dc9f0 into main Oct 20, 2026
11 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.

2 participants