N-05: Named mapping var in ERC7984ObserverAccess

[arr00]

Summary by CodeRabbit

• Refactor
  • Improved internal code readability and maintainability through enhanced declaration formatting.

[netlify]

✅ Deploy Preview for confidential-tokens ready!

Name	Link
🔨 Latest commit	e370f2e
🔍 Latest deploy log	https://app.netlify.com/projects/confidential-tokens/deploys/691494407231cf000815fa9d
😎 Deploy Preview	https://deploy-preview-251--confidential-tokens.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify project configuration.

[coderabbitai]

Important

Review skipped

Auto incremental reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Walkthrough

A mapping declaration in ERC7984ObserverAccess.sol was updated to use named parameters instead of anonymous key/value types. The mapping _observers now uses named types (account and observer) for improved readability while maintaining identical functionality.

Changes

Cohort / File(s)	Summary
Mapping Declaration Naming contracts/token/ERC7984/extensions/ERC7984ObserverAccess.sol	Updated private _observers mapping from mapping(address => address) to mapping(address account => address observer) for improved code clarity via named parameters.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A mapping once unnamed, obscure and plain,
Now wears its labels like a clear refrain—
Account and observer, names that shine,
Readability blooms—a touch divine!

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title directly and specifically describes the main change: renaming the internal _observers mapping to use named parameters (account and observer) for improved readability in ERC7984ObserverAccess.sol.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

---

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