Skip to content

refactor(forges): read settings store inside the forge layer#3035

Merged
afonsojramos merged 2 commits into
mainfrom
refactor/forge-settings-store
Jul 20, 2026
Merged

refactor(forges): read settings store inside the forge layer#3035
afonsojramos merged 2 commits into
mainfrom
refactor/forge-settings-store

Conversation

@afonsojramos

Copy link
Copy Markdown
Member

Summary

Split from #3034. Pushes the settings-store boundary into the forge layer:

  • ForgeAdapter.listNotifications / enrichNotifications, both forge clients, and the notification-type handlers no longer thread a settings parameter through every call; they read useSettingsStore where the values are consumed.
  • The handlers' universally unused _settings parameter is removed.
  • Forge clients import the settings store leaf module directly (not the stores barrel), keeping the module graph free of a stores -> forges -> stores import cycle.
  • Forge tests seed the settings store instead of passing settings objects.

@sonarqubecloud

Copy link
Copy Markdown

@afonsojramos
afonsojramos merged commit e183b59 into main Jul 20, 2026
17 checks passed
@afonsojramos
afonsojramos deleted the refactor/forge-settings-store branch July 20, 2026 17:32
@github-actions github-actions Bot added this to the Release 7.0.0 milestone Jul 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

refactor Refactoring of existing feature

Development

Successfully merging this pull request may close these issues.

2 participants