refactor: fix lint issues (#1316)

This PR is the result of running `yarn lint:fix` with the configurations
in
https://github.com/launchdarkly/js-core/tree/skz/sdk-2234/remove-prettier

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

---

<a href="https://app.devin.ai/review/launchdarkly/js-core/pull/1316"
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]
> **Medium Risk**
> Mostly mechanical lint/prettier autofixes (import/export ordering,
ternary formatting, string literal normalization), but it also changes a
few public entrypoints/exports ordering and adds `variationDetail` to
`sdk-client`’s `LDClientImpl`, which could affect consumers relying on
exact module shape or missing APIs.
> 
> **Overview**
> Applies repo-wide `yarn lint:fix` output: normalizes import ordering
(including `type`-only imports), reformats conditional/ternary
expressions, and replaces many template literals with plain string
literals.
> 
> Also adjusts several package entrypoints to reorder/re-export symbols
consistently (e.g. `index.ts` files across
browser/electron/react-native/server/edge/ai providers), and updates a
few runtime import positions (notably Shopify Oxygen timer polyfill).
> 
> Adds `variationDetail()` to `packages/shared/sdk-client`’s
`LDClientImpl`, enabling detail-returning evaluations alongside existing
`variation()`.
> 
> <sup>Reviewed by [Cursor Bugbot](https://cursor.com/bugbot) for commit
84065c1. Bugbot is set up for automated
code reviews on this repo. Configure
[here](https://www.cursor.com/dashboard/bugbot).</sup>
<!-- /CURSOR_SUMMARY -->

Author: joker23

packages/ai-providers/server-ai-langchain/src/LangChainProvider.ts

@@ -2,7 +2,6 @@ import { BaseChatModel } from '@langchain/core/language_models/chat_models';
 import { AIMessage, HumanMessage, SystemMessage } from '@langchain/core/messages';
 import { initChatModel } from 'langchain/chat_models/universal';
 
-import { AIProvider } from '@launchdarkly/server-sdk-ai';
 import type {
   ChatResponse,
   LDAIConfig,
@@ -12,6 +11,7 @@ import type {
   LDTokenUsage,
   StructuredResponse,
 } from '@launchdarkly/server-sdk-ai';
+import { AIProvider } from '@launchdarkly/server-sdk-ai';
 
 let instrumentPromise: Promise<void> | undefined;
 
@@ -61,7 +61,7 @@ export class LangChainProvider extends AIProvider {
       } catch {
         logger?.debug(
           'OpenTelemetry instrumentation not available for LangChain provider. ' +
-            'Install @traceloop/instrumentation-langchain to enable automatic tracing.',
+          'Install @traceloop/instrumentation-langchain to enable automatic tracing.',
         );
       }
     })();

packages/ai-providers/server-ai-openai/src/OpenAIProvider.ts

@@ -1,6 +1,5 @@
 import { OpenAI } from 'openai';
 
-import { AIProvider } from '@launchdarkly/server-sdk-ai';
 import type {
   ChatResponse,
   LDAIConfig,
@@ -10,6 +9,7 @@ import type {
   LDTokenUsage,
   StructuredResponse,
 } from '@launchdarkly/server-sdk-ai';
+import { AIProvider } from '@launchdarkly/server-sdk-ai';
 
 let instrumentPromise: Promise<void> | undefined;
 
@@ -71,7 +71,7 @@ export class OpenAIProvider extends AIProvider {
       } catch {
         logger?.debug(
           'OpenTelemetry instrumentation not available for OpenAI provider. ' +
-            'Install @traceloop/instrumentation-openai to enable automatic tracing.',
+          'Install @traceloop/instrumentation-openai to enable automatic tracing.',
         );
       }
     })();

packages/ai-providers/server-ai-vercel/src/VercelProvider.ts

@@ -1,6 +1,5 @@
 import { generateObject, generateText, jsonSchema, LanguageModel } from 'ai';
 
-import { AIProvider } from '@launchdarkly/server-sdk-ai';
 import type {
   ChatResponse,
   LDAIConfig,
@@ -10,6 +9,7 @@ import type {
   LDTokenUsage,
   StructuredResponse,
 } from '@launchdarkly/server-sdk-ai';
+import { AIProvider } from '@launchdarkly/server-sdk-ai';
 
 import type {
   ModelUsageTokens,

packages/ai-providers/server-ai-vercel/src/index.ts

@@ -1,7 +1,7 @@
-export { VercelProvider } from './VercelProvider';
 export type {
   VercelAIModelParameters,
   VercelAISDKConfig,
   VercelAISDKMapOptions,
   VercelAISDKProvider,
 } from './types';
+export { VercelProvider } from './VercelProvider';

packages/sdk/akamai-base/example/ldClient.ts

@@ -1,6 +1,6 @@
 import { EdgeProvider, init, LDContext } from '@launchdarkly/akamai-server-base-sdk';
 
-export type { LDContext, EdgeProvider };
+export type { EdgeProvider, LDContext };
 
 const flagData = `
 {

packages/sdk/browser/__tests__/MockHasher.ts

@@ -4,9 +4,11 @@ export class MockHasher implements Hasher {
   update(_data: string): Hasher {
     return this;
   }
+
   digest?(_encoding: string): string {
     return 'hashed';
   }
+
   async asyncDigest?(_encoding: string): Promise<string> {
     return 'hashed';
   }

packages/sdk/browser/__tests__/bootstrap.test.ts

@@ -121,7 +121,7 @@ it('can read old bootstrap data', () => {
   expect(logger.info).not.toHaveBeenCalled();
   expect(logger.warn).toHaveBeenCalledWith(
     'LaunchDarkly client was initialized with bootstrap data that did not' +
-      ' include flag metadata. Events may not be sent correctly.',
+    ' include flag metadata. Events may not be sent correctly.',
   );
   expect(logger.warn).toHaveBeenCalledTimes(1);
   expect(logger.error).not.toHaveBeenCalled();
@@ -143,7 +143,7 @@ it('can handle invalid bootstrap data', () => {
   expect(logger.info).not.toHaveBeenCalled();
   expect(logger.warn).toHaveBeenCalledWith(
     'LaunchDarkly bootstrap data is not available because the back end' +
-      ' could not read the flags.',
+    ' could not read the flags.',
   );
   expect(logger.warn).toHaveBeenCalledTimes(1);
   expect(logger.error).not.toHaveBeenCalled();

packages/sdk/browser/contract-tests/entity/src/ClientEntity.ts

@@ -8,6 +8,7 @@ import {
   SynchronizerEntry,
 } from '@launchdarkly/js-client-sdk';
 import {
+  ClientSideTestHook as TestHook,
   CommandParams,
   CommandType,
   CreateInstanceParams,
@@ -16,7 +17,6 @@ import {
   SDKConfigDataSynchronizer,
   SDKConfigModeDefinition,
   SDKConfigParams,
-  ClientSideTestHook as TestHook,
   ValueType,
 } from '@launchdarkly/js-contract-test-utils/client';
 
@@ -123,9 +123,9 @@ function makeSdkConfig(options: SDKConfigParams, tag: string) {
 
     if (options.dataSystem.connectionModeConfig) {
       const connMode = options.dataSystem.connectionModeConfig;
-      dataSystem.automaticModeSwitching = connMode.initialConnectionMode
-        ? { type: 'manual', initialConnectionMode: connMode.initialConnectionMode }
-        : false;
+      dataSystem.automaticModeSwitching = connMode.initialConnectionMode ?
+          { type: 'manual', initialConnectionMode: connMode.initialConnectionMode } :
+        false;
 
       if (connMode.customConnectionModes) {
         const connectionModes: Record<string, any> = {};
@@ -323,9 +323,9 @@ export async function newSdkClientEntity(options: CreateInstanceParams) {
 
   const timeout =
     options.configuration.startWaitTimeMs !== null &&
-    options.configuration.startWaitTimeMs !== undefined
-      ? options.configuration.startWaitTimeMs
-      : 5000;
+    options.configuration.startWaitTimeMs !== undefined ?
+      options.configuration.startWaitTimeMs :
+      5000;
   const sdkConfig = makeSdkConfig(options.configuration, options.tag);
   const initialContext =
     options.configuration.clientSide?.initialUser ||

packages/sdk/browser/contract-tests/entity/src/TestHarnessWebSocket.ts

@@ -12,7 +12,7 @@ export default class TestHarnessWebSocket {
   constructor(private readonly _url: string) {}
 
   connect() {
-    this._logger.info(`Connecting to web socket.`);
+    this._logger.info('Connecting to web socket.');
     this._ws = new WebSocket(this._url, ['v1']);
     this._ws.onopen = () => {
       this._logger.info('Connected to websocket.');
@@ -24,7 +24,7 @@ export default class TestHarnessWebSocket {
       }, 1000);
     };
     this._ws.onerror = (err) => {
-      this._logger.info(`error:`, err);
+      this._logger.info('error:', err);
     };
 
     this._ws.onmessage = async (msg) => {

packages/sdk/browser/contract-tests/entity/src/main.ts

@@ -1,5 +1,5 @@
-// eslint-disable-next-line prettier/prettier
 import './style.css';
+
 import TestHarnessWebSocket from './TestHarnessWebSocket';
 
 async function runContractTests() {