| author |  Julius Marminge <julius0216@outlook.com> 2026-10-05 21:48:08-07:00 |
| committer |  GitHub <noreply@github.com> 2026-10-05 21:48:08-07:00 |
| log | fad673225650193fad2a85487ecffc903f8ff30a |
| tree | 1d10aae53c706f0c754cfc70d240004177d29908 |
| parent | 2d162d157dfb0eea7e205c508f47d663ab4e8bf9 |
| signature | Signed by PGP key B5690EEEBB952194 |
fix(server): main's settings, keybindings and session tests compile again (#16363)
Co-authored-by: Claude Opus 5.5 (1M context) <noreply@anthropic.com>3 files changed, 4 insertions(+), 4 deletions(-)
apps
server
src
apps/server/src/keybindings.test.ts+1-1
| ... | @@ -620,7 +620,7 @@ it.layer(NodeServices.layer)("keybindings", (it) => { | ... | @@ -620,7 +620,7 @@ it.layer(NodeServices.layer)("keybindings", (it) => { |
| 620 | const configState = yield* keybindings.loadConfigState; | 620 | const configState = yield* keybindings.loadConfigState; |
| 621 | assert.deepEqual(configState.issues, []); | 621 | assert.deepEqual(configState.issues, []); |
| 622 | assert.isTrue(configState.keybindings.some((entry) => entry.command === "terminal.toggle")); | 622 | assert.isTrue(configState.keybindings.some((entry) => entry.command === "terminal.toggle")); |
| 623 | }).pipe(Effect.provide(makeKeybindingsLayer())), | 623 | }).pipe(Effect.provide(layerKeybindings())), |
| 624 | ); | 624 | ); |
| 625 | 625 | ||
| 626 | it.effect("updates cached resolved config after upsert", () => | 626 | it.effect("updates cached resolved config after upsert", () => |
apps/server/src/orchestration-v2/ProviderSessionManager.test.ts+1-1
| ... | @@ -930,7 +930,7 @@ it.effect("ProviderSessionManagerV2 records provider session and turn metrics", | ... | @@ -930,7 +930,7 @@ it.effect("ProviderSessionManagerV2 records provider session and turn metrics", |
| 930 | }); | 930 | }); |
| 931 | 931 | ||
| 932 | yield* effect.pipe( | 932 | yield* effect.pipe( |
| 933 | Effect.provide(makeTestLayer({ state, idleTimeoutMs: 60_000 })), | 933 | Effect.provide(layerTest({ state, idleTimeoutMs: 60_000 })), |
| 934 | // A private registry keeps other tests' provider metrics out of the assertions. | 934 | // A private registry keeps other tests' provider metrics out of the assertions. |
| 935 | Effect.provideService(Metric.MetricRegistry, new Map()), | 935 | Effect.provideService(Metric.MetricRegistry, new Map()), |
| 936 | ); | 936 | ); |
apps/server/src/serverSettings.test.ts+2-2
| ... | @@ -49,7 +49,7 @@ const layerServerSettings = () => | ... | @@ -49,7 +49,7 @@ const layerServerSettings = () => |
| 49 | ), | 49 | ), |
| 50 | ); | 50 | ); |
| 51 | 51 | ||
| 52 | /** Like `makeServerSettingsLayer`, but also exposes the secret store for assertions. */ | 52 | /** Like `layerServerSettings`, but also exposes the secret store for assertions. */ |
| 53 | const layerServerSettingsWithSecrets = () => | 53 | const layerServerSettingsWithSecrets = () => |
| 54 | ServerSettingsModule.layer.pipe( | 54 | ServerSettingsModule.layer.pipe( |
| 55 | Layer.provideMerge(ServerSecretStore.layer), | 55 | Layer.provideMerge(ServerSecretStore.layer), |
| ... | @@ -322,7 +322,7 @@ it.layer(NodeServices.layer)("server settings", (it) => { | ... | @@ -322,7 +322,7 @@ it.layer(NodeServices.layer)("server settings", (it) => { |
| 322 | 322 | ||
| 323 | const settings = yield* serverSettings.getSettings; | 323 | const settings = yield* serverSettings.getSettings; |
| 324 | assert.equal(settings.responseStreamingMode, "turn"); | 324 | assert.equal(settings.responseStreamingMode, "turn"); |
| 325 | }).pipe(Effect.provide(makeServerSettingsLayer())), | 325 | }).pipe(Effect.provide(layerServerSettings())), |
| 326 | ); | 326 | ); |
| 327 | 327 | ||
| 328 | it.effect("decodes nested settings patches", () => | 328 | it.effect("decodes nested settings patches", () => |