Straighten the method with guard clauses

[andrea-ligios]

For a better developer experience

Summary by CodeRabbit

• Refactor
  • Simplified API key resolution logic while keeping the same behavior.
  • API key is now used directly when available; otherwise, the fallback token is still supported with the same deprecation warning and validation.

[andrea-ligios]

Can you please merge this change @AmsterGet @Bam6ycha ?
It's just syntactic sugar, but still an improvement. Thanks

[maria-hambardzumian]

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

Status	Scan Engine	Critical	High	Medium	Low	Total (0)
✅	Open Source Security	0	0	0	0	0 issues
✅	Code Security	0	0	0	0	0 issues

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

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 7a059f4f-4128-4d2a-b661-8d9507209e7e

📥 Commits

Reviewing files that changed from the base of the PR and between f14c46d and f43d930.

📒 Files selected for processing (1)

• lib/commons/config.js

---

Walkthrough

getApiKey in lib/commons/config.js is refactored from a calculatedApiKey-based nested conditional into early-return logic: return apiKey if truthy, otherwise validate token, throw if missing, emit deprecation warning, and return token.

Changes

getApiKey simplification

Layer / File(s)	Summary
Early-return refactor in getApiKey lib/commons/config.js	Removes the calculatedApiKey temporary variable; uses an immediate return apiKey guard and a flat fallback path for token validation and deprecation warning.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

A rabbit once juggled a variable spare,
But decided one leap through the code was more fair.
Return early, dear apiKey, hop right out!
No nesting, no detours, no reason to doubt.
🐇✨

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title matches the refactor: the method was simplified using guard clauses and cleaner early returns.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 ESLint

If the error stems from missing dependencies, add them to the package.json file. For unrecoverable errors (e.g., due to private dependencies), disable the tool in the CodeRabbit configuration.

ESLint install failed: one or more packages not found in the registry.

---

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