chore: add ESLint v9+ config, clean up code, remove unused deps, upda…

[NotoriousArnav]

…te docs for linting and code style

[NotoriousArnav]

@coderabbitai review

[coderabbitai]

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

[coderabbitai]

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Summary by CodeRabbit

• New Features

  • Added ESLint v9+ configuration and enforcement in CI/CD pipeline for consistent code quality.

• Documentation

  • Expanded development guidelines with linting instructions and coding standards.
  • Updated repository documentation with linting and code style best practices.

• Chores

  • Removed unused dependencies to streamline project packages.
  • Standardized code formatting and style across the codebase.

Walkthrough

This PR introduces ESLint v9+ configuration with flat-config setup, integrates linting into CI via GitHub Actions, adds linting documentation, updates test and source files with trailing commas and quote normalization per ESLint rules, removes unused dependencies from package.json, and extends credentials structure in mongoAuthState.js with accountSettings.

Changes

Cohort / File(s)	Summary
ESLint Configuration & CI .github/workflows/eslint.yml, eslint.config.js	Added new ESLint v9+ flat config with rules for quotes, semicolons, indentation, and comma-dangle; replaced npm install with npm ci and consolidated ESLint run step in GitHub Actions workflow with SARIF formatting
Documentation AGENTS.md, README.md, docs/DEVELOPMENT.md	Updated linting guidance and added "Linting & Code Style" section with ESLint usage commands; added example code structure with error handling; removed mongo-baileys from dependencies list
Test Files __tests__/index.test.js, __tests__/logger.test.js, __tests__/managementRoutes.test.js, __tests__/routes.test.js, __tests__/server.test.js, __tests__/socketManager.test.js	Added trailing commas to object literals, arrays, and function arguments per ESLint rules; no behavioral changes
Jest & Build Configuration jest.config.js	Added trailing commas to configuration arrays and objects; normalized indentation; no semantic changes to test behavior
Package Management package.json	Removed unused dependencies (@adiwajshing/keyed-db, @hapi/boom, mongo-baileys, uuid); added knip as devDependency; minor newline formatting
Source Code Formatting accountManager.js, accountRouter.js, audioConverter.js, db.js, index.js, managementRoutes.js, mediaHandler.js, routes.js, scripts/cli.js, socketManager.js	Added trailing commas to object literals and function arguments; normalized string quotes from double to single in db.js; converted template literals to single-quoted strings in scripts/cli.js; no functional logic changes
Credentials & Authentication mongoAuthState.js	Added accountSettings: { unarchiveChats: false } to credentials; normalized string quotes; updated BufferJSON serialization/deserialization; refactored eslint-disable directives; retained public API signatures

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

• mongoAuthState.js: Contains functional changes to credentials structure and Buffer JSON handling; requires verification that accountSettings behavior is intentional and doesn't break existing authentication flows
• ESLint configuration: New eslint.config.js and GitHub Actions workflow integration should be validated for correctness and rule coverage
• Package.json dependency removals: Verify that removed dependencies (@adiwajshing/keyed-db, @hapi/boom, mongo-baileys, uuid) are not imported elsewhere in the codebase or are intentionally being replaced

Possibly related PRs

• CodeRabbit Generated Unit Tests: Add Jest configuration and 155 unit tests #10: Overlaps on test file modifications (tests/*.test.js) and jest.config.js formatting changes for consistent lint compliance
• Feature/logging cleanup Add Jest unit test suite with coverage thresholds #11: Related through shared jest configuration and test suite formatting applied consistently across test files

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch chore/eslint-cleanup-and-docs

---

📜 Recent review details

Configuration used: Path: .coderabbit.yml

Review profile: ASSERTIVE

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 3910cb7 and 8a28a68.

⛔ Files ignored due to path filters (1)

• package-lock.json is excluded by !**/package-lock.json

📒 Files selected for processing (24)

• .github/workflows/eslint.yml (1 hunks)
• AGENTS.md (1 hunks)
• README.md (1 hunks)
• __tests__/index.test.js (3 hunks)
• __tests__/logger.test.js (24 hunks)
• __tests__/managementRoutes.test.js (22 hunks)
• __tests__/routes.test.js (24 hunks)
• __tests__/server.test.js (7 hunks)
• __tests__/socketManager.test.js (20 hunks)
• accountManager.js (2 hunks)
• accountRouter.js (5 hunks)
• audioConverter.js (6 hunks)
• db.js (1 hunks)
• docs/DEVELOPMENT.md (2 hunks)
• eslint.config.js (1 hunks)
• index.js (2 hunks)
• jest.config.js (1 hunks)
• managementRoutes.js (1 hunks)
• mediaHandler.js (8 hunks)
• mongoAuthState.js (8 hunks)
• package.json (1 hunks)
• routes.js (1 hunks)
• scripts/cli.js (2 hunks)
• socketManager.js (3 hunks)

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}