Skip to content

fix(scripts): clear effect-lint errors breaking CI on main - #1282

Merged
Makisuo merged 1 commit into
mainfrom
fix/main-effect-lint
Oct 7, 2026
Merged

Makisuo merged 1 commit into
mainfrom
fix/main-effect-lint

fix(scripts): clear effect-lint errors in seed-demo and screenshot sc…

5710deb
Select commit
Loading
Failed to load commit list.
Maple Review Bot / Maple / review succeeded Oct 7, 2026 in 19s

Confidence – · Nothing to review

Nothing to review

Lint-only cleanup in three CLI scripts: adds oxlint-disable-next-line effecttsgo/strict-effect-provide above the top-level Effect.provide and return yield*s the never-ending follow loop in seed-demo.ts. No runtime behavior changes.

What was checked
  • All three changed files are tooling/scripts, which this review does not cover
  • return yield* at scripts/seed-demo.ts:336 is on an Effect.forever loop, so the function never falls through differently
  • The new disable comments match the repository's existing pattern (e.g. packages/query-engine/src/local.ts:135, apps/cli/src/bin.ts:79)

5710deb · Updated on every push. Reply "won't fix" to dismiss a finding, or mention @maple-review-bot to ask about one.