Skip to content

[Web|Lint] Format the files Biome flags on dev - #2165

Merged
CREDO23 merged 1 commit into
devfrom
fix/web-format-drift
Oct 3, 2026
Merged

CREDO23 merged 1 commit into
devfrom
fix/web-format-drift

Conversation

@CREDO23

@CREDO23 CREDO23 commented Oct 3, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

  • Web Lint runs biome check over all of surfsense_web, so any PR that touches the folder fails on 11 errors already on dev, in ten files: nine formatted by hand since late August and two with imports out of order (one file has both).
  • biome check --write on those ten files only: line wrapping and import order, nothing else. Biome now reports no errors across the folder, and the 92 web unit tests pass.
  • Merge before [CI|Node] Build the website and the installers on Node 24 #2162 and [Local|Release] Set the desktop version to 2.1.0 #2164, which both touch surfsense_web and fail Web Lint on these files.

@CREDO23
CREDO23 requested a review from AnishSarkar22 as a code owner October 3, 2026 05:48
@vercel

vercel Bot commented Oct 3, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
surf-sense-frontend Error Error Oct 3, 2026 5:48am UTC

Request Review

@coderabbitai

coderabbitai Bot commented Oct 3, 2026

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration
  • Configuration used: Repository: MODSetter/SurfSense/.coderabbit.yaml
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: da88345f-ee95-44ef-95a9-9ae5a5526d56
📥 Commits

Reviewing files that changed from the base of the PR and between a22c81e and f61a686.

📒 Files selected for processing (10)
  • surfsense_web/app/(home)/downloads/page.tsx
  • surfsense_web/app/(home)/private-ai-for-business/page.tsx
  • surfsense_web/app/layout.tsx
  • surfsense_web/components/assistant-ui/composer-add-menu-drawer.tsx
  • surfsense_web/components/assistant-ui/thread.tsx
  • surfsense_web/components/homepage/home/home-sections.tsx
  • surfsense_web/components/json-view.tsx
  • surfsense_web/components/settings/git-remote-settings.tsx
  • surfsense_web/components/site/site-nav.tsx
  • surfsense_web/lib/documents/git-synced-folders.ts
 ______________________________________________________________
< How would you like your code review? Scrambled or over-easy? >
 --------------------------------------------------------------
  \
   \   (\__/)
       (•ㅅ•)
       /   づ
✨ Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autopilot · 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

Comment @coderabbitai help to get the list of available commands.

@CREDO23
CREDO23 merged commit e1d30b2 into dev Oct 3, 2026
20 of 24 checks passed

This branch had an error being deployed

1 failed deployment
Preview — f61a6862 Deployed Oct 3, 2026 by vercel[bot]
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.

1 participant