ci: Fix performance step in CI

[mtrezza]

Pull Request

• Report security issues confidentially.
• Any contribution is under this license.
• Link this pull request to an issue.

Issue

Various optimizations.

Summary by CodeRabbit

• Tests

  • Adjusted performance benchmark parameters to reduce execution time.

• Chores

  • Simplified CI workflow documentation by removing benchmark configuration details.

[parse-github-assistant]

🚀 Thanks for opening this pull request!

[coderabbitai]

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

The changes reduce benchmark configuration parameters: iteration count drops from 10,000 to 1,000, and cleanup wait time drops from 10 seconds to 1 second. The CI workflow comment is simplified to remove benchmark iteration and CPU affinity details.

Changes

Cohort / File(s)	Summary
CI Workflow Comment Update .github/workflows/ci-performance.yml	Removed PR-comment text describing benchmark iteration count (10,000) and CPU pinning details; comment now contains only thresholds note
Benchmark Configuration Reduction benchmark/performance.js	Reduced default ITERATIONS from 10000 to 1000; reduced final cleanup wait time from 10000 ms to 1000 ms

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

• Verify that reducing iterations from 10,000 to 1,000 maintains sufficient statistical significance for performance benchmarks
• Confirm the 10-fold reduction in cleanup wait time (10s → 1s) does not cause premature exit or resource cleanup issues
• Ensure the parameter changes are intentional and align with performance testing goals

Possibly related PRs

• ci: Fix performance step in CI #9922: Identical changes to benchmark iteration count (10,000 → 1,000) in both CI workflow comments and benchmark/performance.js
• ci: Add performance impact step to CI #9916: Modifies the same benchmark parameters and CI workflow comment in the same files with aligned changes
• ci: Fix performance step in CI #9921: Updates benchmarking defaults (ITERATIONS to 1000) with corresponding changes to benchmark/performance.js and CI performance workflow

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 3cc8c1a and ed89c76.

📒 Files selected for processing (2)

• .github/workflows/ci-performance.yml (1 hunks)
• benchmark/performance.js (2 hunks)

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[parseplatformorg]

✅ Snyk checks have passed. No issues have been found so far.

Status	Scanner	Critical	High	Medium	Low	Total (0)
✅	Open Source Security	0	0	0	0	0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.07%. Comparing base (3cc8c1a) to head (ed89c76).
⚠️ Report is 4 commits behind head on alpha.

Additional details and impacted files

@@           Coverage Diff           @@
##            alpha    #9927   +/-   ##
=======================================
  Coverage   93.07%   93.07%           
=======================================
  Files         187      187           
  Lines       15224    15224           
  Branches      177      177           
=======================================
  Hits        14170    14170           
  Misses       1042     1042           
  Partials       12       12           

☔ View full report in Codecov by Sentry.
📢 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.

[parseplatformorg]

🎉 This change has been released in version 8.5.0-alpha.8