chore: remove stale eslint-disable comments (#1300)

## Summary
- Remove ~65 stale `eslint-disable` comments for rules that no longer
exist after replacing airbnb config
- Rules removed: `no-param-reassign` (43), `no-bitwise` (10),
`no-restricted-syntax` for ForOfStatement (11), `no-plusplus` (4),
`no-continue` (1)
- Retains a targeted `no-restricted-syntax` disable in vendor
`TraceKit.ts` for a legitimate `for...in` loop

## Test plan
- [ ] `yarn workspaces foreach -p run lint` passes with 0 errors
- [ ] No functional code changes (comment removal only)
- [ ] `grep -rn
"eslint-disable.*no-(param-reassign|bitwise|plusplus|continue)"
packages/` returns zero results

Depends on #1299.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

<!-- CURSOR_SUMMARY -->
---

> [!NOTE]
> **Low Risk**
> Primarily removes lint suppression comments with no runtime behavior
changes; minor risk is limited to new `no-param-reassign` enforcement
surfacing previously ignored lint violations.
> 
> **Overview**
> Cleans up stale `eslint-disable` directives across SDKs/tests
(bitwise, plusplus, restricted-syntax, etc.) after the eslint config
change, leaving code behavior unchanged.
> 
> Updates `.eslintrc.js` to explicitly enforce `no-param-reassign`, and
keeps a single targeted `no-restricted-syntax` suppression in vendor
`TraceKit.ts` for a necessary `for...in` loop.
> 
> <sup>Reviewed by [Cursor Bugbot](https://cursor.com/bugbot) for commit
7695809. Bugbot is set up for automated
code reviews on this repo. Configure
[here](https://www.cursor.com/dashboard/bugbot).</sup>
<!-- /CURSOR_SUMMARY -->

---------

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: joker23

.eslintrc.js

@@ -24,6 +24,7 @@ module.exports = {
     '**/fromExternal/**',
   ],
   rules: {
+    'no-param-reassign': 'error',
     'eqeqeq': ['error', 'always', { null: 'ignore' }],
     'prefer-const': ['error', { ignoreReadBeforeAssign: true }],
     'no-var': 'error',

packages/sdk/browser/__tests__/platform/randomUuidV4.test.ts

@@ -1,4 +1,3 @@
-/* eslint-disable no-bitwise */
 import { fallbackUuidV4, formatDataAsUuidV4 } from '../../src/platform/randomUuidV4';
 
 it('formats conformant UUID', () => {

packages/sdk/electron/__tests__/platform/HeaderWrapper.test.ts

@@ -1,5 +1,3 @@
-/* eslint-disable no-restricted-syntax */
-// The header interface uses generators, so we are using restricted-syntax.
 import * as http from 'http';
 
 import HeaderWrapper from '../../src/platform/HeaderWrapper';

packages/sdk/electron/src/platform/HeaderWrapper.ts

@@ -35,7 +35,6 @@ export default class HeaderWrapper implements platform.Headers {
   // We want to use generators here for the simplicity of maintaining
   // this interface. Also they aren't expected to be high frequency usage.
   *values(): Iterable<string> {
-    // eslint-disable-next-line no-restricted-syntax
     for (const key of this.keys()) {
       const val = this.get(key);
       if (val !== null) {
@@ -45,7 +44,6 @@ export default class HeaderWrapper implements platform.Headers {
   }
 
   *entries(): Iterable<[string, string]> {
-    // eslint-disable-next-line no-restricted-syntax
     for (const key of this.keys()) {
       const val = this.get(key);
       if (val !== null) {

packages/sdk/react-native/src/fromExternal/react-native-sse/EventSource.ts

@@ -278,7 +278,6 @@ export default class EventSource<E extends string = never> {
     let retry = 0;
     let line = '';
 
-    // eslint-disable-next-line no-plusplus
     for (let i = 0; i < parts.length; i++) {
       line = parts[i].replace(/^(\s|\u00A0)+|(\s|\u00A0)+$/g, '');
       if (line.indexOf('event') === 0) {

packages/sdk/react-native/src/platform/crypto/index.ts

@@ -3,7 +3,6 @@ import type { Crypto, Hmac } from '@launchdarkly/js-client-sdk-common';
 import PlatformHasher from './PlatformHasher';
 import { SupportedHashAlgorithm } from './types';
 
-/* eslint-disable no-bitwise */
 /**
  * To avoid dependencies on uuid, this is good enough for now.
  * Ripped from the react-native repo:

packages/sdk/react-native/src/polyfills/btoa.ts

@@ -1,4 +1,3 @@
-/* eslint-disable no-bitwise */
 import { fromByteArray } from 'base64-js';
 
 import toUtf8Array from './toUtf8Array';

packages/sdk/react-native/src/polyfills/toUtf8Array.ts

@@ -1,6 +1,3 @@
-/* eslint-disable no-plusplus */
-/* eslint-disable no-bitwise */
-
 // Originally from: https://github.com/google/closure-library/blob/a1f5a029c1b32eb4793a2d920aa52abc085e1bf7/closure/goog/crypt/crypt.js
 
 // Once React Native versions uniformly support TextEncoder this code can be removed.

packages/sdk/react-native/src/polyfills/uuid.ts

@@ -1,4 +1,3 @@
-/* eslint-disable no-bitwise */
 /**
  * To avoid dependencies on uuid, this is good enough for now.
  * Ripped from the react-native repo:

packages/sdk/server-ai/examples/vercel-ai/src/index.ts

@@ -106,7 +106,6 @@ async function main() {
     );
 
     // Consume the stream immediately - no await needed before this!
-    // eslint-disable-next-line no-restricted-syntax
     for await (const textPart of streamResult.textStream) {
       process.stdout.write(textPart);
     }