Skip to content

chore(fmt): upgrade prettier & ruff and auto format missed files - #2884

Merged
asvishnyakov merged 8 commits into
mainfrom
pr-split/2-3-formatting
Apr 8, 2026
Merged

asvishnyakov merged 8 commits into
mainfrom
pr-split/2-3-formatting

Conversation

@asvishnyakov

@asvishnyakov asvishnyakov commented Apr 7, 2026 •

Copy link
Copy Markdown
Member
  • Prettier 3 — bump to ^3.8.1, add .prettierignore for pnpm-lock.yaml, run prettier --write across the repo
  • Ruff — require >=0.13.3, set target-version = "py310", fix chainlit/sync.py for UP036 (remove redundant sys.version_info gate).
  • Cypress — Ruff import fixes on cypress/e2e/**/main.py entrypoints.
  • Editor / Git config — expand .editorconfig, add .gitattributes (* text=auto), drop printWidth from .prettierrc so width comes from EditorConfig.

No changes beyond formatting hygiene.


Summary by cubic

Upgrade formatting tooling to Prettier 3 and Ruff 0.13+, align EditorConfig/Git settings, and reformat the repo for consistent styles across code, docs, and workflows. No behavior changes.

  • Dependencies

    • Bumped prettier to ^3.8.1; added .prettierignore to exclude pnpm-lock.yaml; removed printWidth from .prettierrc to defer to EditorConfig.
    • Added .gitattributes with * text=auto; expanded .editorconfig (utf-8, indent size 2, max_line_length=80, single quotes).
    • Required ruff >=0.13.3 and set target-version = "py310" in backend/pyproject.toml; refreshed uv.lock.
  • Refactors

    • Dropped redundant Python 3.10 gate in backend/chainlit/sync.py; normalized import order in Cypress Python entrypoints and TS/TSX files.
    • Ran Prettier v3 across HTML/CSS/JSON/YAML/MD/TS/TSX (e.g., <!doctype html>, quote/spacing fixes); normalized GitHub Actions YAML quoting; fixed markdown tables/newlines across docs (incl. ESLint v9 migration plan).

Written for commit 4d19d44. Summary will update on new commits.

asvishnyakov and others added 3 commits April 8, 2026 00:31
- Bump Prettier to ^3.8.1 and add .prettierignore for pnpm-lock.yaml
- Run Prettier across the repo (no per-package format script changes)
- Bump Ruff to >=0.13.3, target-version py310; refresh uv.lock
- Drop redundant Python 3.10 gate in chainlit/sync.py (UP036)
- Apply Ruff import fixes to Cypress Python entrypoints

Co-Authored-By: Cursor <noreply@cursor.com>
Made-with: Cursor
Prettier does not read quote_type; the block only affected editors that
implement that extension. YAML in this repo is formatted by Prettier.

Co-Authored-By: Cursor <noreply@cursor.com>
Made-with: Cursor
- Add * text=auto for consistent line endings in Git
- Remove printWidth from .prettierrc; use max_line_length from .editorconfig

Co-Authored-By: Cursor <noreply@cursor.com>
Made-with: Cursor
@dosubot dosubot Bot added size:XXL This PR changes 1000+ lines, ignoring generated files. dev-tooling Relating to developer/contributor toolings. labels Apr 7, 2026

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1 issue found across 93 files

Note: This PR contains a large number of files. cubic only reviews up to 75 files per PR, so some files may not have been reviewed. We prioritized the most important files first.

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="docs/plans/2026-03-31-eslint-v9-migration.md">

<violation number="1" location="docs/plans/2026-03-31-eslint-v9-migration.md:319">
P3: Fix the corrupted glob example in this note; it currently documents the wrong pattern (`_/` instead of `*/`) and malformed inline formatting.</violation>
</file>

Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.

Comment thread docs/plans/2026-03-31-eslint-v9-migration.md Outdated
@asvishnyakov
asvishnyakov enabled auto-merge April 8, 2026 00:04

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1 issue found across 1 file (changes from recent commits).

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="docs/plans/2026-03-31-eslint-v9-migration.md">

<violation number="1" location="docs/plans/2026-03-31-eslint-v9-migration.md:364">
P3: Remove the stray "why" at the end of this bullet; it reads as an accidental typo in the plan doc.</violation>
</file>

Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.

Comment thread docs/plans/2026-03-31-eslint-v9-migration.md Outdated
@asvishnyakov
asvishnyakov added this pull request to the merge queue Apr 8, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Apr 8, 2026
@asvishnyakov
asvishnyakov added this pull request to the merge queue Apr 8, 2026
Merged via the queue into main with commit 314a148 Apr 8, 2026
11 checks passed
@asvishnyakov
asvishnyakov deleted the pr-split/2-3-formatting branch April 8, 2026 01:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dev-tooling Relating to developer/contributor toolings. size:XXL This PR changes 1000+ lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants