Mask problematic --concurrency=1 setting

[amotl]

This might help with GH-201, @IgorOhrimenko.

[coderabbitai]

Warning

Rate limit exceeded

@amotl has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 20 minutes and 24 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📥 Commits

Reviewing files that changed from the base of the PR and between dfdea75 and 28cf63e.

📒 Files selected for processing (1)

• CHANGES.rst (1 hunks)

Summary by CodeRabbit

• Changes
  • Updated default concurrency setting to 1 for request handling.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

Walkthrough

Docs and CLI defaults updated to set concurrency to 1; core engine defaults and enable_concurrency behavior were modified (constructor default changed, type annotation and guard added). No other functional changes indicated.

Changes

Cohort / File(s)	Summary
Changelog CHANGES.rst	Added entry noting default concurrency change.
CLI grafana_wtf/commands.py	Changed CLI --concurrency default/help from 0 to 1.
Core engine grafana_wtf/core.py	Adjusted GrafanaEngine default concurrency (was 5, now 0), added concurrency: int annotation on enable_concurrency, and added a guard that resets concurrency to 0 if it was 1 before applying a new value.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

• Inspect all usages/readers of GrafanaEngine.concurrency for changed default semantics.
• Review enable_concurrency guard for transition edge cases (especially around value 1).
• Verify CLI help/options match runtime behavior and documentation.

Poem

🐰 I hopped through diffs with nimble feet,

One concurrency now makes rhythm neat.
I nudged the defaults, left docs in place,
A tiny change — a quieter race. 🥕

Pre-merge checks and finishing touches

❌ Failed checks (2 warnings, 1 inconclusive)

Check name	Status	Explanation	Resolution
Title check	⚠️ Warning	The title describes masking a problematic concurrency setting, but the actual changes implement --concurrency=1 as the default, which contradicts the 'masking' implication.	Update the title to accurately reflect the change, such as 'Set --concurrency=1 as default' or 'Change default concurrency to 1' to match the actual implementation.
Docstring Coverage	⚠️ Warning	Docstring coverage is 25.00% which is insufficient. The required threshold is 80.00%.	You can run @coderabbitai generate docstrings to improve docstring coverage.
Description check	❓ Inconclusive	The description is vague and references an external GitHub issue (GH-201) without explaining what problem is being solved or how the changes address it.	Expand the description to explain the purpose of changing the concurrency default and how it relates to GH-201, making it easier for reviewers to understand the rationale.

---

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.}

[codecov]

Codecov Report

❌ Patch coverage is 75.00000% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 84.47%. Comparing base (78ed502) to head (28cf63e).
⚠️ Report is 2 commits behind head on main.

Files with missing lines	Patch %	Lines
grafana_wtf/core.py	75.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #208      +/-   ##
==========================================
- Coverage   84.53%   84.47%   -0.07%     
==========================================
  Files           9        9              
  Lines        1138     1140       +2     
==========================================
+ Hits          962      963       +1     
- Misses        176      177       +1     

Flag	Coverage Δ
unittests	84.47% <75.00%> (-0.07%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ 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.