[jan-kubica]

Moves to @stll/oxlint-config 4.0.0. Its library() preset enables more rules outside oxlint's default correctness category (stella/tooling#265). The new type-aware no-unsafe-* rules (stella/tooling#267) do not run here because this config keeps typeAware off.

Fixes for the new rules

• import/no-duplicates: merged split imports in 9 files.
• no-case-declarations: braces around two case bodies.
• array-callback-return: two exhaustive switches inside array callbacks now end with default: return x satisfies never, which keeps the exhaustiveness check.
• no-empty-object-type: four extension signatures use Partial<Record<never, never>> instead of Partial<{}> (the same type).
• preserve-caught-error: errors thrown from catch blocks carry { cause }.
• prefer-const, prefer-array-flat-map, no-useless-concat: mechanical.
• Disable comments with reasons for two intentional cases: the VS Code webview postMessage (which takes no target origin) and a test that temporarily defines a property on Object.prototype.
• import/no-empty-named-blocks is off: import type {} from is how tests and fixtures load a module's global declarations without running it.
• typescript/no-require-imports is off for test/__fixtures__/**, where boundary fixtures model CommonJS imports in TypeScript.

.cjs files keep require; the preset turns no-require-imports off for them.

[coderabbitai]

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 35 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

Run configuration

• Source mode: Auto
• Repository: stella/folio
• Branch: main
• File: .coderabbit.yaml
• Commit: eb49cc4
• Layers: Repository YAML
• Review profile: CHILL
• Plan: Advanced
• Run ID: 5d1347d7-fd93-4850-bbd5-ba224b80eefe

Commits

Reviewing files that changed from the base of the PR and between eb49cc4 and 8e845ee.

Files ignored due to path filters: 1

• bun.lock is excluded by !**/*.lock

Files selected for processing: 30

• .changeset/oxlint-config-4-cleanup.md
• oxlint.config.ts
• package.json
• packages/core/src/ai-edits/headless.ts
• packages/core/src/compare/section-boundary-properties.test.ts
• packages/core/src/controller/__tests__/freshRenderHarness.ts
• packages/core/src/controller/canonicalInlineShapes.test.ts
• packages/core/src/controller/canonicalStructure.test.ts
• packages/core/src/docx/rezip.ts
• packages/core/src/docx/server/materializeYjsDocx.test.ts
• packages/core/src/docx/tableRowChildSurvival.test.ts
• packages/core/src/layout-engine/measure/measureParagraph.ts
• packages/core/src/prosemirror/commands/comments.ts
• packages/core/src/prosemirror/extensions/features/AutoBidiDetectionExtension.ts
• packages/core/src/prosemirror/extensions/features/JoinedRunStyleExtension.ts
• packages/core/src/prosemirror/extensions/features/ParaIdAllocatorExtension.ts
• packages/core/src/prosemirror/extensions/features/ParagraphChangeTrackerExtension.ts
• packages/core/src/prosemirror/extensions/marks/markUtils.ts
• packages/docx-core/src/ops/apply.ts
• packages/docx-core/src/ops/review.ts
• packages/editor-web/src/vscode.tsx
• scripts/corpus-edit-fuzz.ts
• scripts/fuzz-failure-issues.test.ts
• scripts/lib/container-survival/canonicalSpellings.ts
• scripts/lib/corpus-extended.ts
• scripts/lib/corpus-family-census.ts
• scripts/lib/nested-object.test.ts
• scripts/workflow-cache-policy.test.ts
• test/consumer-scenarios/scenarios/numbering-oracle.test.ts
• test/consumer-scenarios/support/oracle.ts

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

• X
• Mastodon
• Reddit
• LinkedIn