Skip to content

chore: improve prettier lint#608

Draft
brojd wants to merge 4 commits into
elevenlabs:mainfrom
brojd:brojd/improve-prettier-lint
Draft

chore: improve prettier lint#608
brojd wants to merge 4 commits into
elevenlabs:mainfrom
brojd:brojd/improve-prettier-lint

Conversation

@brojd

@brojd brojd commented Mar 19, 2026

Copy link
Copy Markdown
Contributor

Closes #582

  • Changed all lint:prettier scripts from prettier 'src/**/*.ts' --check to prettier . --check so .tsx, .json,
    .md, and config files are covered
  • Fixed convai-widget-embed which was missing the --check flag
  • Added lint:prettier to @elevenlabs/types (previously had none)
  • Added format script (prettier . --write) to all packages and a pnpm format root command via turbo
  • Added .prettierignore files per package to exclude dist/, generated files, and worklets. This was needed because prettier doesn't respect root .gitignore when running inside pnpm workspaces
  • Ran pnpm format to apply prettier changes across all affected packages to fix existing formatting

Note

Medium Risk
Mostly formatting/tooling changes, but it rewrites a large generated types file and updates Prettier inputs/ignores across packages, which could cause noisy diffs or missed/extra files in CI if misconfigured.

Overview
Improves Prettier linting and formatting across the monorepo. lint:prettier is standardized to run prettier . --check (covering TSX/JSON/MD/configs) and a new format script (prettier . --write) is added in packages plus root-level pnpm format/pnpm lint:prettier via Turborepo.

Adds per-package .prettierignore files to exclude dist/ and generated/worklet outputs, and updates turbo.json so lint:prettier considers all files plus .prettierignore while format runs uncached. The PR also applies Prettier-driven formatting to docs/config/tests and regenerates/reformats packages/types/generated/types/asyncapi-types.ts.

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

@kraenhansen kraenhansen self-assigned this Mar 19, 2026
@kraenhansen

Copy link
Copy Markdown
Member

To reduce the amount of conflicts, I'd like for this to merge after we're merged next into main.

@kraenhansen kraenhansen marked this pull request as draft March 20, 2026 12:39
@brojd

brojd commented Mar 20, 2026

Copy link
Copy Markdown
Contributor Author

Sound good @kraenhansen 👍 I can solve the conflicts after the next -> main merge. That will be easy because we can just revert d001bd6 and rerun pnpm format against codebase after merge

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.

Prettier lint scripts don't cover .tsx files

2 participants