Skip to content

chore(lint): Fix suppressed ESLint errors in base-controller package#7443

Merged
Mrtenz merged 1 commit into
mainfrom
mrtenz/fix-lint-base-controller
Dec 11, 2025
Merged

chore(lint): Fix suppressed ESLint errors in base-controller package#7443
Mrtenz merged 1 commit into
mainfrom
mrtenz/fix-lint-base-controller

Conversation

@Mrtenz

@Mrtenz Mrtenz commented Dec 11, 2025

Copy link
Copy Markdown
Member

Explanation

This fixes all suppressed ESLint errors in the base-controller package.

References

Closes #7344.

Checklist

  • I've updated the test suite for new or updated code as appropriate
  • I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
  • I've communicated my changes to consumers by updating changelogs for packages I've changed
  • I've introduced breaking changes in this PR and have prepared draft pull requests for clients and consumer packages to resolve them

Note

Refactors BaseController types and tests to satisfy ESLint by adding explicit return types, refining generics, and cleaning up imports; removes related suppressions.

  • Core (packages/base-controller/src/BaseController.ts)
    • Add explicit return types to state getter, applyPatches, destroy, and update result.
    • Rename generic/type params for clarity (TType), and tighten typings (StateChangeListener, StateDeriver, StateMetadata).
    • Minor callback param rename in produceWithPatches cast (callbackFn).
  • Tests (packages/base-controller/src/BaseController.test.ts)
    • Add explicit return types to controller overrides and helper functions; annotate arrow properties.
    • Switch sinon import to default; reorder imports; add // eslint-disable-next-line no-new where needed.
  • Lint config
    • Update eslint-suppressions.json to remove suppressions for base-controller files now addressed.

Written by Cursor Bugbot for commit 71469bd. This will update automatically on new commits. Configure here.

@Mrtenz
Mrtenz marked this pull request as ready for review December 11, 2025 13:08
@Mrtenz
Mrtenz requested a review from a team as a code owner December 11, 2025 13:08
@Mrtenz
Mrtenz added this pull request to the merge queue Dec 11, 2025
Merged via the queue into main with commit 0ed31aa Dec 11, 2025
286 checks passed
@Mrtenz
Mrtenz deleted the mrtenz/fix-lint-base-controller branch December 11, 2025 13:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fix suppressed lint errors in base-controller

2 participants