refactor(sdk): Clean-up tsyringe setup (#3362)

## Summary

Centralizes the `tsyringe` dependency injection setup by introducing a
dedicated `setupTsyringe.ts` module that handles the `reflect-metadata`
import. This eliminates the need for scattered `import
'reflect-metadata'` statements across source and test files, making the
DI configuration more maintainable and ensuring consistent
initialization.

## Changes

- Created `packages/sdk/src/setupTsyringe.ts` to centralize the
`reflect-metadata` import
- Updated core SDK entry points (`StreamrClient.ts`, `Config.ts`,
`ConfigTypes.ts`, `PersistenceManager.ts`) to import the new setup
module instead of `reflect-metadata` directly
- Added `setupTsyringe.ts` to Jest's `setupFilesAfterEnv` configuration
for automatic test initialization
- Added `setupTsyringe.ts` import to `karma-setup.js` to fix the browser
test build
- Removed redundant `import 'reflect-metadata'` statements from 60+ test
files

## Limitations and future improvements

- The setup module currently only imports `reflect-metadata`, but could
be extended in the future to include other tsyringe configuration or
polyfills if needed

Author: mondoreale

packages/sdk/jest.config.ts

@@ -6,6 +6,7 @@ const config: Config.InitialOptions = {
     globalSetup: './jest.setup.ts',
     setupFilesAfterEnv: [
         ...defaultConfig.setupFilesAfterEnv,
+        './src/setupTsyringe.ts',
         './test/test-utils/customMatchers.ts',
         '@streamr/test-utils/setupCustomMatchers',
     ],

packages/sdk/jest.setup.ts

@@ -1,6 +1,5 @@
 // eslint-disable-next-line import/no-extraneous-dependencies
 import { GitRevisionPlugin } from 'git-revision-webpack-plugin'
-import 'reflect-metadata'
 import pkg from './package.json'
 
 export default async function setup(): Promise<void> {

packages/sdk/karma-setup.js

@@ -3,6 +3,7 @@
  * package. See karma-config.js in that package for more details.
  */
 
+import './src/setupTsyringe.ts'
 import './test/test-utils/customMatchers'
 import { customMatchers } from '@streamr/test-utils'
 

packages/sdk/src/Config.ts

@@ -1,4 +1,3 @@
-import 'reflect-metadata'
 import cloneDeep from 'lodash/cloneDeep'
 import { merge } from '@streamr/utils'
 import { generateClientId } from './utils/utils'

packages/sdk/src/ConfigTypes.ts

@@ -1,4 +1,3 @@
-import 'reflect-metadata'
 import type { Overrides, Eip1193Provider } from 'ethers'
 import type { DeepRequired, MarkOptional } from 'ts-essentials'
 import type { HexString, LogLevel, KeyType } from '@streamr/utils'

packages/sdk/src/PersistenceManager.ts

@@ -1,4 +1,4 @@
-import 'reflect-metadata'
+import './setupTsyringe'
 
 import { join } from 'path'
 import { inject, Lifecycle, scoped } from 'tsyringe'

packages/sdk/src/StreamrClient.ts

@@ -2,8 +2,8 @@
  * Importing 'timers' ensures `setImmediate` is available in browsers,
  * as it's polyfilled by `timers-browserify`. In Node.js, it's already global.
  */
-import 'reflect-metadata'
 import 'timers'
+import './setupTsyringe'
 import './utils/PatchTsyringe'
 
 import { DhtAddress } from '@streamr/dht'

packages/sdk/src/setupTsyringe.ts

@@ -0,0 +1 @@
+import 'reflect-metadata'

packages/sdk/test/end-to-end/MemoryLeaks.test.ts

@@ -1,7 +1,6 @@
 import { config as CHAIN_CONFIG } from '@streamr/config'
 import { createTestPrivateKey, describeOnlyInNodeJs } from '@streamr/test-utils'
 import { Defer, merge, TheGraphClient, wait } from '@streamr/utils'
-import 'reflect-metadata'
 import { DependencyContainer, container as rootContainer } from 'tsyringe'
 import { writeHeapSnapshot } from 'v8'
 import { IdentityInjectionToken } from '../../src/identity/Identity'

packages/sdk/test/end-to-end/StorageNodeRegistry.test.ts

@@ -1,5 +1,3 @@
-import 'reflect-metadata'
-
 import { createTestWallet, randomEthereumAddress } from '@streamr/test-utils'
 import { until } from '@streamr/utils'
 import { Wallet } from 'ethers'