refactor: remove prettier dependency (#1320)

This PR just removes our `prettier` dependency and its usage

NOTE: if we merge this then we will have a period of time where we would
not have a code formatter, this PR is isolated to break up our larger
change to introduce `stylistic` and modernize our eslint implementation

<!-- devin-review-badge-begin -->

---

<a href="https://app.devin.ai/review/launchdarkly/js-core/pull/1320"
target="_blank">
  <picture>
<source media="(prefers-color-scheme: dark)"
srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1">
<img
src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1"
alt="Open in Devin Review">
  </picture>
</a>
<!-- devin-review-badge-end -->

<!-- CURSOR_SUMMARY -->
---

> [!NOTE]
> **Low Risk**
> Tooling-only changes; main risk is CI/dev workflows no longer
enforcing formatting until a replacement is added.
> 
> **Overview**
> Removes Prettier from the repo’s tooling: deletes
`.prettierrc`/`.prettierignore`, drops `prettier`-related ESLint
integration (config/plugin/rule) from `.eslintrc.js` and the Svelte
example ESLint config, and removes
`prettier`/`eslint-plugin-prettier`/`eslint-config-prettier`/import-sorting
plugin deps across workspaces.
> 
> Updates workspace `scripts` to stop running `prettier` (removes
`prettier` scripts and trims `check` pipelines accordingly), plus a
small cleanup removing a now-unneeded `prettier` disable comment in the
browser contract-test entrypoint.
> 
> <sup>Reviewed by [Cursor Bugbot](https://cursor.com/bugbot) for commit
4ecfaa9. Bugbot is set up for automated
code reviews on this repo. Configure
[here](https://www.cursor.com/dashboard/bugbot).</sup>
<!-- /CURSOR_SUMMARY -->

Author: joker23

.eslintrc.js

@@ -3,7 +3,7 @@ module.exports = {
     node: true,
     'jest/globals': true,
   },
-  extends: ['eslint:recommended', 'plugin:@typescript-eslint/recommended', 'prettier'],
+  extends: ['eslint:recommended', 'plugin:@typescript-eslint/recommended'],
   parser: '@typescript-eslint/parser',
   parserOptions: {
     project: [
@@ -12,7 +12,7 @@ module.exports = {
     ],
     tsconfigRootDir: __dirname,
   },
-  plugins: ['@typescript-eslint', 'import', 'prettier', 'jest'],
+  plugins: ['@typescript-eslint', 'import', 'jest'],
   ignorePatterns: [
     '**/dist/**',
     '**/vercel/examples/**',
@@ -42,7 +42,6 @@ module.exports = {
       'error',
       { ignoreRestSiblings: true, argsIgnorePattern: '^_', varsIgnorePattern: '^__' },
     ],
-    'prettier/prettier': ['error'],
     'class-methods-use-this': 'off',
     'import/no-extraneous-dependencies': [
       'error',

.prettierignore

@@ -70,19 +70,14 @@
     "test": "echo Please run tests for individual packages.",
     "coverage": "npm run test -- --coverage",
     "contract-test-harness": "curl -s https://raw.githubusercontent.com/launchdarkly/sdk-test-harness/master/downloader/run.sh \\ | VERSION=v2 PARAMS=\"-url http://localhost:8000 -debug -stop-service-at-end $TEST_HARNESS_PARAMS\" sh",
-    "prettier": "npx prettier --write \"**/*.{js,ts,tsx,json,yaml,yml,md}\" --log-level warn",
-    "check": "yarn && yarn prettier && yarn lint && tsc && yarn build"
+    "check": "yarn && yarn lint && tsc && yarn build"
   },
   "devDependencies": {
-    "@trivago/prettier-plugin-sort-imports": "^4.1.1",
     "@typescript-eslint/eslint-plugin": "^6.20.0",
     "@typescript-eslint/parser": "^6.20.0",
     "eslint": "^8.56.0",
-    "eslint-config-prettier": "^8.8.0",
     "eslint-plugin-import": "^2.27.5",
     "eslint-plugin-jest": "^27.6.3",
-    "eslint-plugin-prettier": "^5.0.0",
-    "prettier": "^3.0.0",
     "typedoc": "0.25.0",
     "typescript": "5.1.6"
   },

.prettierrc

@@ -26,9 +26,8 @@
   "scripts": {
     "build": "tsup-node",
     "lint": "npx eslint . --ext .ts",
-    "prettier": "prettier --write '**/*.@(js|ts|tsx|json|css)' --ignore-path ../../../.prettierignore",
     "lint:fix": "yarn run lint --fix",
-    "check": "yarn prettier && yarn lint && yarn build && yarn test",
+    "check": "yarn lint && yarn build && yarn test",
     "test": "jest"
   },
   "files": [
@@ -47,18 +46,14 @@
     "@launchdarkly/server-sdk-ai": "^0.18.0",
     "@opentelemetry/api": "^1.9.0",
     "@traceloop/instrumentation-langchain": "^0.26.0",
-    "@trivago/prettier-plugin-sort-imports": "^4.1.1",
     "@types/jest": "^29.5.3",
     "@typescript-eslint/eslint-plugin": "^6.20.0",
     "@typescript-eslint/parser": "^6.20.0",
     "eslint": "^8.45.0",
-    "eslint-config-prettier": "^8.8.0",
     "eslint-plugin-import": "^2.27.5",
     "eslint-plugin-jest": "^27.6.3",
-    "eslint-plugin-prettier": "^5.0.0",
     "jest": "^29.6.1",
     "langchain": "^1.3.4",
-    "prettier": "^3.0.0",
     "ts-jest": "^29.1.1",
     "tsup": "^8.5.1",
     "typescript": "5.1.6"

package.json

@@ -26,9 +26,8 @@
   "scripts": {
     "build": "tsup-node",
     "lint": "npx eslint . --ext .ts",
-    "prettier": "prettier --write '**/*.@(js|ts|tsx|json|css)' --ignore-path ../../../.prettierignore",
     "lint:fix": "yarn run lint --fix",
-    "check": "yarn prettier && yarn lint && yarn build && yarn test",
+    "check": "yarn lint && yarn build && yarn test",
     "test": "jest"
   },
   "files": [
@@ -47,18 +46,14 @@
     "@launchdarkly/server-sdk-ai": "^0.18.0",
     "@opentelemetry/api": "^1.9.0",
     "@traceloop/instrumentation-openai": "^0.22.0",
-    "@trivago/prettier-plugin-sort-imports": "^4.1.1",
     "@types/jest": "^29.5.3",
     "@typescript-eslint/eslint-plugin": "^6.20.0",
     "@typescript-eslint/parser": "^6.20.0",
     "eslint": "^8.45.0",
-    "eslint-config-prettier": "^8.8.0",
     "eslint-plugin-import": "^2.27.5",
     "eslint-plugin-jest": "^27.6.3",
-    "eslint-plugin-prettier": "^5.0.0",
     "jest": "^29.6.1",
     "openai": "^5.0.0",
-    "prettier": "^3.0.0",
     "ts-jest": "^29.1.1",
     "tsup": "^8.5.1",
     "typescript": "5.1.6"

packages/ai-providers/server-ai-langchain/package.json

@@ -26,9 +26,8 @@
   "scripts": {
     "build": "tsup-node",
     "lint": "npx eslint . --ext .ts",
-    "prettier": "prettier --write '**/*.@(js|ts|tsx|json|css)' --ignore-path ../../../.prettierignore",
     "lint:fix": "yarn run lint --fix",
-    "check": "yarn prettier && yarn lint && yarn build && yarn test",
+    "check": "yarn lint && yarn build && yarn test",
     "test": "jest"
   },
   "files": [
@@ -49,18 +48,14 @@
     "@ai-sdk/mistral": "^2.0.0",
     "@ai-sdk/openai": "^2.0.0",
     "@launchdarkly/server-sdk-ai": "^0.18.0",
-    "@trivago/prettier-plugin-sort-imports": "^4.1.1",
     "@types/jest": "^29.5.3",
     "@typescript-eslint/eslint-plugin": "^6.20.0",
     "@typescript-eslint/parser": "^6.20.0",
     "ai": "^5.0.0",
     "eslint": "^8.45.0",
-    "eslint-config-prettier": "^8.8.0",
     "eslint-plugin-import": "^2.27.5",
     "eslint-plugin-jest": "^27.6.3",
-    "eslint-plugin-prettier": "^5.0.0",
     "jest": "^29.6.1",
-    "prettier": "^3.0.0",
     "ts-jest": "^29.1.1",
     "tsup": "^8.5.1",
     "typescript": "5.1.6"

packages/ai-providers/server-ai-openai/package.json

@@ -40,29 +40,24 @@
     "tsw": "yarn tsc --watch",
     "start": "rimraf dist && yarn tsw",
     "lint": "eslint . --ext .ts",
-    "prettier": "prettier --write '**/*.@(js|ts|tsx|json|css)' --ignore-path ../../../.prettierignore",
     "test": "NODE_OPTIONS=\"--experimental-vm-modules --no-warnings\" jest --ci --runInBand",
     "coverage": "yarn test --coverage",
-    "check": "yarn prettier && yarn lint && yarn build && yarn test && yarn doc"
+    "check": "yarn lint && yarn build && yarn test && yarn doc"
   },
   "devDependencies": {
     "@rollup/plugin-commonjs": "^25.0.0",
     "@rollup/plugin-node-resolve": "^15.0.2",
     "@rollup/plugin-terser": "^0.4.3",
     "@rollup/plugin-typescript": "^11.1.1",
-    "@trivago/prettier-plugin-sort-imports": "^4.1.1",
     "@types/crypto-js": "^4.1.1",
     "@types/jest": "^29.5.1",
     "@typescript-eslint/eslint-plugin": "^6.20.0",
     "@typescript-eslint/parser": "^6.20.0",
     "eslint": "^8.45.0",
-    "eslint-config-prettier": "^8.8.0",
     "eslint-plugin-import": "^2.27.5",
     "eslint-plugin-jest": "^27.6.3",
-    "eslint-plugin-prettier": "^5.0.0",
     "jest": "^29.5.0",
     "launchdarkly-js-test-helpers": "^2.2.0",
-    "prettier": "^3.0.0",
     "rimraf": "^5.0.0",
     "rollup": "^3.23.0",
     "rollup-plugin-generate-package-json": "^3.2.0",

packages/ai-providers/server-ai-vercel/package.json

@@ -40,29 +40,24 @@
     "tsw": "yarn tsc --watch",
     "start": "rimraf dist && yarn tsw",
     "lint": "eslint . --ext .ts",
-    "prettier": "prettier --write '**/*.@(js|ts|tsx|json|css)' --ignore-path ../../../.prettierignore",
     "test": "NODE_OPTIONS=\"--experimental-vm-modules --no-warnings\" jest --ci --runInBand",
     "coverage": "yarn test --coverage",
-    "check": "yarn prettier && yarn lint && yarn build && yarn test && yarn doc"
+    "check": "yarn lint && yarn build && yarn test && yarn doc"
   },
   "devDependencies": {
     "@rollup/plugin-commonjs": "^25.0.0",
     "@rollup/plugin-node-resolve": "^15.0.2",
     "@rollup/plugin-terser": "^0.4.3",
     "@rollup/plugin-typescript": "^11.1.1",
-    "@trivago/prettier-plugin-sort-imports": "^4.1.1",
     "@types/crypto-js": "^4.1.1",
     "@types/jest": "^29.5.1",
     "@typescript-eslint/eslint-plugin": "^6.20.0",
     "@typescript-eslint/parser": "^6.20.0",
     "eslint": "^8.45.0",
-    "eslint-config-prettier": "^8.8.0",
     "eslint-plugin-import": "^2.27.5",
     "eslint-plugin-jest": "^27.6.3",
-    "eslint-plugin-prettier": "^5.0.0",
     "jest": "^29.5.0",
     "launchdarkly-js-test-helpers": "^2.2.0",
-    "prettier": "^3.0.0",
     "rimraf": "^5.0.0",
     "rollup": "^3.23.0",
     "rollup-plugin-generate-package-json": "^3.2.0",

packages/sdk/akamai-base/package.json

@@ -10,24 +10,19 @@
     "start:headless": "tsc --noEmit && vite",
     "build": "tsc --noEmit && vite build",
     "lint": "eslint ./src",
-    "start:adapter": "sdk-testharness-server adapter",
-    "prettier": "prettier --write '**/*.@(js|ts|tsx|json|css)' --ignore-path ../../../../.prettierignore"
+    "start:adapter": "sdk-testharness-server adapter"
   },
   "dependencies": {
     "@launchdarkly/js-client-sdk": "workspace:^",
     "@launchdarkly/js-contract-test-utils": "workspace:^"
   },
   "devDependencies": {
-    "@trivago/prettier-plugin-sort-imports": "^4.1.1",
     "@typescript-eslint/eslint-plugin": "^6.20.0",
     "@typescript-eslint/parser": "^6.20.0",
     "eslint": "^8.45.0",
-    "eslint-config-prettier": "^8.8.0",
     "eslint-plugin-import": "^2.27.5",
     "eslint-plugin-jest": "^27.6.3",
-    "eslint-plugin-prettier": "^5.0.0",
     "playwright": "^1.49.1",
-    "prettier": "^3.0.0",
     "typescript": "^5.5.3",
     "vite": "^5.4.1"
   }