Skip to content

Use the same tooling as crelte (eslint 9, prettier/editorconfig, fmt, shared tsconfig) - #8

Merged
soerenmeier merged 1 commit into
mainfrom
tooling
Oct 30, 2026
Merged

soerenmeier merged 1 commit into
mainfrom
tooling

Conversation

@noah-meierlabs

@noah-meierlabs noah-meierlabs commented Oct 30, 2026 •

Copy link
Copy Markdown
Contributor

Brings the repo tooling in line with crelte. Only tooling changes, no library code changes apart from one eslint comment.

  • eslint 9 (flat config) with typescript-eslint 8, eslint-config-prettier, browser + node globals, and .gitignore as the ignore list via @eslint/compat. Same as crelte's eslint.config.js without the svelte parts.
  • npm audit: 12 → 0 warnings. They all came from the eslint 8 / typescript-eslint 7 dependency tree.
  • .editorconfig and .gitignore from crelte (.prettierrc stays as it was). The old .editorconfig said tabs for yaml, but prettier formats yaml with spaces.
  • New .prettierignore.
  • fmt script (prettier -w .) in every package and the workspace, like crelte.
  • The four identical package tsconfig.jsons now extend a shared tsconfig.base.json. outDir stays in each package, because relative paths in an extended config resolve against the base file.
  • Removed an eslint-disable-next-line no-constant-condition in Stream.ts that eslint 9 reports as unused (it allows while (true)).

Formatting: I ran prettier -w over the whole repository. Nothing changed, everything was already formatted.

Not taken from crelte:

  • the svelte tooling (no .svelte files here)
  • vitest (there are no tests yet; adding vitest makes sense together with the first tests)

Checks: build, check-fmt, check, lint (0 warnings) and check-jsr pass. The build output still ends up in each package's dist/.

🤖 Generated with Claude Code

- eslint 9 flat config with typescript-eslint 8, eslint-config-prettier,
  browser and node globals and the .gitignore as ignore list, like
  crelte's eslint.config.js (without the svelte parts). This also
  removes all npm audit warnings, they came from the eslint 8 tree.
- .editorconfig and .gitignore from crelte, plus a
  .prettierignore. The old .editorconfig used tabs for yaml, which
  prettier formats with spaces.
- fmt script (prettier -w) in every package and the workspace.
- The four identical package tsconfigs extend a shared
  tsconfig.base.json.
- Drop an eslint-disable comment eslint 9 reports as unused.

A prettier run over the whole repository changed nothing.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@soerenmeier
soerenmeier merged commit 851d3a6 into main Oct 30, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants