From ddeb130555182c4b3f5102cb8a1de13af6d17b92 Mon Sep 17 00:00:00 2001
From: Ryan Lamb <4955475+kinyoklion@users.noreply.github.com>
Date: Mon, 30 Sep 2024 12:37:39 -0700
Subject: [PATCH] Fix build.

---
 packages/shared/common/src/index.ts                       | 8 +++++++-
 .../sdk-client/src/datasource/DataSourceStatusManager.ts  | 2 ++
 .../shared/sdk-client/src/polling/PollingProcessor.ts     | 7 +++----
 .../shared/sdk-client/src/streaming/StreamingProcessor.ts | 5 ++---
 .../shared/sdk-server/src/data_sources/FileDataSource.ts  | 4 +---
 .../sdk-server/src/data_sources/PollingProcessor.ts       | 7 +++----
 packages/shared/sdk-server/src/data_sources/Requestor.ts  | 8 ++------
 7 files changed, 20 insertions(+), 21 deletions(-)

diff --git a/packages/shared/common/src/index.ts b/packages/shared/common/src/index.ts
index 79f7e6396..2d23590f0 100644
--- a/packages/shared/common/src/index.ts
+++ b/packages/shared/common/src/index.ts
@@ -1,7 +1,12 @@
 import AttributeReference from './AttributeReference';
 import Context from './Context';
 import ContextFilter from './ContextFilter';
-import { DataSourceErrorKind, LDPollingError, LDStreamingError } from './datasource';
+import {
+  DataSourceErrorKind,
+  LDFileDataSourceError,
+  LDPollingError,
+  LDStreamingError,
+} from './datasource';
 
 export * from './api';
 export * from './validators';
@@ -19,4 +24,5 @@ export {
   DataSourceErrorKind,
   LDPollingError,
   LDStreamingError,
+  LDFileDataSourceError,
 };
diff --git a/packages/shared/sdk-client/src/datasource/DataSourceStatusManager.ts b/packages/shared/sdk-client/src/datasource/DataSourceStatusManager.ts
index 4bbcc7f6d..c5335c5e6 100644
--- a/packages/shared/sdk-client/src/datasource/DataSourceStatusManager.ts
+++ b/packages/shared/sdk-client/src/datasource/DataSourceStatusManager.ts
@@ -1,3 +1,5 @@
+import { DataSourceErrorKind } from '@launchdarkly/js-sdk-common';
+
 import LDEmitter from '../LDEmitter';
 import DataSourceStatus, { DataSourceState } from './DataSourceStatus';
 import DataSourceStatusErrorInfo from './DataSourceStatusErrorInfo';
diff --git a/packages/shared/sdk-client/src/polling/PollingProcessor.ts b/packages/shared/sdk-client/src/polling/PollingProcessor.ts
index 85db0fcfe..fc25ec8b5 100644
--- a/packages/shared/sdk-client/src/polling/PollingProcessor.ts
+++ b/packages/shared/sdk-client/src/polling/PollingProcessor.ts
@@ -1,11 +1,12 @@
 import {
+  DataSourceErrorKind,
   Encoding,
   getPollingUri,
   httpErrorMessage,
   HttpErrorResponse,
-  internal,
   isHttpRecoverable,
   LDLogger,
+  LDPollingError,
   Requests,
   subsystem,
 } from '@launchdarkly/js-sdk-common';
@@ -14,9 +15,7 @@ import { PollingDataSourceConfig } from '../streaming/DataSourceConfig';
 import { Flags } from '../types';
 import Requestor, { LDRequestError } from './Requestor';
 
-// TODO: revisit usage of internal and figure out best practice
-const { DataSourceErrorKind, LDPollingError } = internal;
-export type PollingErrorHandler = (err: internal.LDPollingError) => void;
+export type PollingErrorHandler = (err: LDPollingError) => void;
 
 /**
  * @internal
diff --git a/packages/shared/sdk-client/src/streaming/StreamingProcessor.ts b/packages/shared/sdk-client/src/streaming/StreamingProcessor.ts
index 7753ad91c..03b95637f 100644
--- a/packages/shared/sdk-client/src/streaming/StreamingProcessor.ts
+++ b/packages/shared/sdk-client/src/streaming/StreamingProcessor.ts
@@ -1,4 +1,5 @@
 import {
+  DataSourceErrorKind,
   Encoding,
   EventName,
   EventSource,
@@ -7,6 +8,7 @@ import {
   HttpErrorResponse,
   internal,
   LDLogger,
+  LDStreamingError,
   ProcessStreamResponse,
   Requests,
   shouldRetry,
@@ -15,9 +17,6 @@ import {
 
 import { StreamingDataSourceConfig } from './DataSourceConfig';
 
-// TODO: revisit usage of internal and figure out best practice
-const { DataSourceErrorKind, LDStreamingError } = internal;
-
 const reportJsonError = (
   type: string,
   data: string,
diff --git a/packages/shared/sdk-server/src/data_sources/FileDataSource.ts b/packages/shared/sdk-server/src/data_sources/FileDataSource.ts
index b649a321c..87d896e71 100644
--- a/packages/shared/sdk-server/src/data_sources/FileDataSource.ts
+++ b/packages/shared/sdk-server/src/data_sources/FileDataSource.ts
@@ -1,6 +1,6 @@
 import {
   Filesystem,
-  internal,
+  LDFileDataSourceError,
   LDLogger,
   subsystem,
   VoidFunction,
@@ -14,8 +14,6 @@ import { processFlag, processSegment } from '../store/serialization';
 import VersionedDataKinds from '../store/VersionedDataKinds';
 import FileLoader from './FileLoader';
 
-// TODO: revisit usage of internal and figure out best practice
-type LDFileDataSourceError = internal.LDFileDataSourceError;
 export type FileDataSourceErrorHandler = (err: LDFileDataSourceError) => void;
 
 function makeFlagWithValue(key: string, value: any, version: number): Flag {
diff --git a/packages/shared/sdk-server/src/data_sources/PollingProcessor.ts b/packages/shared/sdk-server/src/data_sources/PollingProcessor.ts
index 5b53466b2..5014b8eed 100644
--- a/packages/shared/sdk-server/src/data_sources/PollingProcessor.ts
+++ b/packages/shared/sdk-server/src/data_sources/PollingProcessor.ts
@@ -1,8 +1,9 @@
 import {
+  DataSourceErrorKind,
   httpErrorMessage,
-  internal,
   isHttpRecoverable,
   LDLogger,
+  LDPollingError,
   subsystem,
   VoidFunction,
 } from '@launchdarkly/js-sdk-common';
@@ -13,9 +14,7 @@ import { deserializePoll } from '../store';
 import VersionedDataKinds from '../store/VersionedDataKinds';
 import Requestor from './Requestor';
 
-// TODO: revisit usage of internal and figure out best practice
-const { DataSourceErrorKind, LDPollingError } = internal;
-export type PollingErrorHandler = (err: internal.LDPollingError) => void;
+export type PollingErrorHandler = (err: LDPollingError) => void;
 
 /**
  * @internal
diff --git a/packages/shared/sdk-server/src/data_sources/Requestor.ts b/packages/shared/sdk-server/src/data_sources/Requestor.ts
index 9fa5a3ef4..9c8aa5c10 100644
--- a/packages/shared/sdk-server/src/data_sources/Requestor.ts
+++ b/packages/shared/sdk-server/src/data_sources/Requestor.ts
@@ -1,7 +1,8 @@
 import {
+  DataSourceErrorKind,
   getPollingUri,
-  internal,
   LDHeaders,
+  LDPollingError,
   Options,
   Requests,
   Response,
@@ -10,11 +11,6 @@ import {
 import { LDFeatureRequestor } from '../api/subsystems';
 import Configuration from '../options/Configuration';
 
-const { DataSourceErrorKind } = internal;
-
-// TODO: revisit usage of internal and figure out best practice
-const { LDPollingError } = internal;
-
 /**
  * @internal
  */