Skip to content

feat: governor limit reporting#624

Merged
lcottercertinia merged 18 commits into
certinia:mainfrom
lukecotter:feat-governor-limits-in-cells
Jun 7, 2025
Merged

feat: governor limit reporting#624
lcottercertinia merged 18 commits into
certinia:mainfrom
lukecotter:feat-governor-limits-in-cells

Conversation

@lukecotter
Copy link
Copy Markdown
Collaborator

@lukecotter lukecotter commented May 28, 2025

📝 PR Overview

  • Add formatting for cumulative limit usage in the call tree.
  • Always show the cumulative limit usage rows (even though time is 0)
  • Add percentage bars in cells that use cumulative limit usage totals for the percent.
    • DML + SOQL rows, DML + SOQL statements
  • Show governor limits totals on timeline tooltip e.g 1000/50000 for row s
    -show tooltip on call tree and analysis grid to show Governor totals e.g 4000/10000 for dml rows.

Example: "Adds a flame chart view for CPU time analysis in Apex logs to improve log analysis performance."

🧩 Type of change (check all applicable)

  • 🐛 Bug fix - something not working as expected
  • ✨ New feature – adds new functionality
  • ♻️ Refactor - internal changes with no user impact
  • ⚡ Performance Improvement
  • 📝 Documentation - README or documentation site changes
  • 🔧 Chore - dev tooling, CI, config
  • 💥 Breaking change

📷 Screenshots / gifs / video [optional]

Show off your UI changes.

🔗 Related Issues

fixes #
resolves #308
resolves #309
closes #
related #

✅ Tests added?

  • 👍 yes
  • 🙅 no, not needed
  • 🙋 no, I need help

📚 Docs updated?

  • 🔖 README.md
  • 🔖 CHANGELOG.md
  • 📖 help site
  • 🙅 not needed

Anything else we need to know? [optional]

lukecotter added 18 commits May 23, 2025 17:18
Displays better in VSCode market place
Makes it much easier to see and understand the governor limits for each namespace
Previously the were hidden (as details) because the had 0 total time.
…e events

This displays better in the call tree
Makes it possible to filter by namespace and see only
the limit usage for that namespace.
new prop on `ApexLog` called `governorLimits` which has the
total governors used. This is a sum across all namesspaces.

There is also a map `limitsByNamespace` which has the limits used in each namespace.
- new styling
- better size to fit within timeline
- right align values to make them easier to read
e.g
100/1000 for dml
1500/10000 ms (self 500) for time
Make it easier to access and makes more sense.
@lcottercertinia lcottercertinia merged commit 4ad3f6c into certinia:main Jun 7, 2025
5 checks passed
@lukecotter lukecotter deleted the feat-governor-limits-in-cells branch June 7, 2025 23:41
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.

feat: better handing of cumulative usage on calltree feat: show max governor limit for each value in cell (calltree, analysis, db)

2 participants