Skip to content

Commit 0fb4731

Browse files
committed
fix(logger): default value is undefined
1 parent 504c843 commit 0fb4731

File tree

5 files changed

+17
-17
lines changed

5 files changed

+17
-17
lines changed

packages/browser/src/adapter/WebStorage.ts

+3-3
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ import type { DataTransformer, EntityTransformer } from '@stenodb/utils'
66
export class BrowserStorage<T> {
77
name: string
88
storage: Storage
9-
logger: BaseLogger
9+
logger: BaseLogger | undefined
1010

1111
entityTransformer: EntityTransformer<T>
1212
dataTransformer: DataTransformer<T>
@@ -37,12 +37,12 @@ export class BrowserStorage<T> {
3737
read(): void {
3838
const data = this.storage.getItem(this.name)
3939
this.data = this.dataTransformer.toJSON(data)
40-
this.logger.info('Read data:', this.data)
40+
this.logger?.info('Read data:', this.data)
4141
}
4242

4343
write(): void {
4444
this.storage.setItem(this.name, this.dataTransformer.toString(this.data))
45-
this.logger.info('Write data:', this.data)
45+
this.logger?.info('Write data:', this.data)
4646
}
4747

4848
reset(): void {

packages/browser/src/provider/BrowserProvider.ts

+3-3
Original file line numberDiff line numberDiff line change
@@ -3,12 +3,12 @@ import type { Steno } from '../types.js'
33
import type { CreateLogger } from '@stenodb/logger'
44

55
export class BrowserProvider {
6-
#options: Steno.BrowserProviderOptions
6+
#options: Steno.BrowserProviderOptions | undefined
77
#logger: CreateLogger
88

9-
constructor(options: Steno.BrowserProviderOptions) {
9+
constructor(options?: Steno.BrowserProviderOptions) {
1010
this.#options = options
11-
this.#logger = options.logger
11+
this.#logger = options?.logger
1212
}
1313

1414
private registerAdapterModules<T>(adapter: Steno.BrowserAdapter<T>) {

packages/node/src/adapter/AsyncAdapter.ts

+5-5
Original file line numberDiff line numberDiff line change
@@ -11,26 +11,26 @@ export class AsyncAdapter<T> extends BaseAdapter<T> {
1111
try {
1212
const file = await readFile(this.filePath, 'utf-8')
1313
this.data = this.dataTransformer.toJSON(file)
14-
this.logger.info('Read data from file', this.data)
14+
this.logger?.info('Read data from file', this.data)
1515
} catch (err) {
1616
if (!this.data) {
1717
await this.reset()
1818
}
1919

2020
if ((err as NodeJS.ErrnoException).code !== 'ENOENT') {
21-
this.logger.error('Failed to read data from file', err)
21+
this.logger?.error('Failed to read data from file', err)
2222
}
2323
}
2424
}
2525

2626
async write(): Promise<void> {
2727
await this.writer.write(this.dataTransformer.toString(this.data))
28-
this.logger.info('Write data to file', this.data)
28+
this.logger?.info('Write data to file', this.data)
2929
}
3030

3131
async reset(): Promise<void> {
3232
if (!this.initialData) {
33-
this.logger.warn('No initial data to reset to')
33+
this.logger?.warn('No initial data to reset to')
3434
return
3535
}
3636

@@ -45,7 +45,7 @@ export class AsyncAdapter<T> extends BaseAdapter<T> {
4545
await this.write()
4646
} catch (err) {
4747
if ((err as NodeJS.ErrnoException).code !== 'ENOENT') {
48-
this.logger.error('Failed to read data from file', err)
48+
this.logger?.error('Failed to read data from file', err)
4949
}
5050
}
5151
}

packages/node/src/adapter/BaseAdapter.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ export class BaseAdapter<T> {
1313
dataTransformer: DataTransformer<T>
1414

1515
directory: DirectoryProvider
16-
logger: BaseLogger
16+
logger: BaseLogger | undefined
1717
writer: Writer
1818

1919
data: T | null = null

packages/node/src/adapter/SyncAdapter.ts

+5-5
Original file line numberDiff line numberDiff line change
@@ -11,26 +11,26 @@ export class SyncAdapter<T> extends BaseAdapter<T> {
1111
try {
1212
const file = readFileSync(this.filePath, 'utf-8')
1313
this.data = this.dataTransformer.toJSON(file)
14-
this.logger.info('Read data from file', this.data)
14+
this.logger?.info('Read data from file', this.data)
1515
} catch (err) {
1616
if (!this.data) {
1717
this.reset()
1818
}
1919

2020
if ((err as NodeJS.ErrnoException).code !== 'ENOENT') {
21-
this.logger.error('Failed to read data from file', err)
21+
this.logger?.error('Failed to read data from file', err)
2222
}
2323
}
2424
}
2525

2626
write(): void {
2727
this.writer.write(this.dataTransformer.toString(this.data))
28-
this.logger.info('Write data to file', this.data)
28+
this.logger?.info('Write data to file', this.data)
2929
}
3030

3131
reset(): void {
3232
if (!this.initialData) {
33-
this.logger.warn('No initial data to reset to')
33+
this.logger?.warn('No initial data to reset to')
3434
return
3535
}
3636

@@ -45,7 +45,7 @@ export class SyncAdapter<T> extends BaseAdapter<T> {
4545
this.write()
4646
} catch (err) {
4747
if ((err as NodeJS.ErrnoException).code !== 'ENOENT') {
48-
this.logger.error('Failed to read data from file', err)
48+
this.logger?.error('Failed to read data from file', err)
4949
}
5050
}
5151
}

0 commit comments

Comments
 (0)