Skip to content

chore: drop analytics-controller from lint:tsc - #9697

Merged
cryptodev-2s merged 1 commit into
mainfrom
chore/drop-analytics-controller-lint-tsc
Jul 29, 2026
Merged

chore: drop analytics-controller from lint:tsc#9697
cryptodev-2s merged 1 commit into
mainfrom
chore/drop-analytics-controller-lint-tsc

Conversation

@cryptodev-2s

@cryptodev-2s cryptodev-2s commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Explanation

analytics-controller is enrolled in lint:tsc (it has a tsconfig.lint.json and is listed in the root tsconfig.lint.json). Once it starts importing geolocation-controller and controller-utils (the geo enrichment work in #9691), and neither of those is a lint:tsc project yet, tsc pulls their source straight into the analytics-controller project and fails with rootDir errors:

error TS6059: File '.../controller-utils/src/types.ts' is not under 'rootDir' '.../analytics-controller'
error TS6307: File '.../controller-utils/src/types.ts' is not listed within the file list of project '.../analytics-controller/tsconfig.lint.json'

This turns the lint:tsc job red on #9691 and blocks the work.

This PR removes packages/analytics-controller/tsconfig.lint.json and its entry in the root tsconfig.lint.json, so analytics-controller is no longer type checked by lint:tsc. yarn lint:tsc is green with this change.

This is a stopgap to unblock the current work, not the end state. The proper fix is to make controller-utils and geolocation-controller lint:tsc projects (give them their own tsconfig.lint.json) and enroll analytics-controller again. That is a bigger change because it also turns on type checking for those packages' test files, which surfaces separate pre existing type errors that need their own cleanup.

References

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

Once analytics-controller imports geolocation-controller and controller-utils, neither of which is a lint:tsc project yet, tsc pulls their source into the analytics-controller project and fails with rootDir errors (TS6059 / TS6307). That is currently red on the geo enrichment PR.

Pull analytics-controller out of lint:tsc as a stopgap so the work is unblocked. We can enroll it again once controller-utils and geolocation-controller are lint:tsc projects.
@cryptodev-2s
cryptodev-2s requested review from a team as code owners July 29, 2026 15:38
@cryptodev-2s cryptodev-2s self-assigned this Jul 29, 2026
@cryptodev-2s
cryptodev-2s enabled auto-merge July 29, 2026 15:42
@cryptodev-2s
cryptodev-2s added this pull request to the merge queue Jul 29, 2026
Merged via the queue into main with commit 8b708ab Jul 29, 2026
432 of 433 checks passed
@cryptodev-2s
cryptodev-2s deleted the chore/drop-analytics-controller-lint-tsc branch July 29, 2026 16:53
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.

2 participants