Skip to content

Commit face879

Browse files
authored
fix: disable default Effect console logger (#21963)
1 parent 605559b commit face879

2 files changed

Lines changed: 3 additions & 4 deletions

File tree

packages/opencode/src/effect/logger.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@ export namespace EffectLogger {
5555
}
5656
})
5757

58-
export const layer = Logger.layer([logger], { mergeWithExisting: true })
58+
export const layer = Logger.layer([logger], { mergeWithExisting: false })
5959

6060
export const create = (base: Fields = {}): Handle => ({
6161
debug: (msg, extra) => call((item) => Effect.logDebug(item), base, msg, extra),

packages/opencode/src/effect/oltp.ts

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -6,9 +6,8 @@ import { Flag } from "@/flag/flag"
66
import { CHANNEL, VERSION } from "@/installation/meta"
77

88
export namespace Observability {
9-
export const enabled = !!Flag.OTEL_EXPORTER_OTLP_ENDPOINT
10-
119
const base = Flag.OTEL_EXPORTER_OTLP_ENDPOINT
10+
export const enabled = !!base
1211

1312
const resource = {
1413
serviceName: "opencode",
@@ -36,7 +35,7 @@ export namespace Observability {
3635
EffectLogger.layer,
3736
Otlp.layerJson({
3837
baseUrl: base,
39-
loggerExportInterval: Duration.seconds(5),
38+
loggerExportInterval: Duration.seconds(1),
4039
loggerMergeWithExisting: true,
4140
resource,
4241
headers,

0 commit comments

Comments
 (0)