forst-lang / forst / 29165468284

78%

main: 76%

Build Type

Pull #136

github

Committed by haveyaseen

Commit Message

fix(node-runtime): keep bootstrap logs off stdout RPC channel

NodeRuntime.runMain injects Logger.prettyLoggerDefault onto stdout by
default, which corrupted length-prefixed proto frames and broke nodert
CallSync/CallAsync integration. Pass disablePrettyLogger at bootstrap
and indexer CLI entrypoints, and write structured logs via fd 2 in
ForstNodeRuntimeLayer so stdout stays reserved for RPC frames.

Restore runProtoLoop as an async stdin loop wrapped in Effect.promise
from startRpcServer. Add nodert regression coverage that bootstrap
stdout stays idle until an RPC response, and layer tests asserting
pretty logging never touches stdout.

chore(remix-serve): remove debug console.error from legacy TS modules

Drop per-function console.error tracing from remix-serve legacy
activity and todos helpers now that node-runtime logging is stable.

Pull Request Pull Request #136: feat(node-runtime)!: adopt Effect for RPC, logging, host runtime

Coverage Stats

82 of 100 new or added lines in 6 files covered. (82.0%)

1 existing line in 1 file now uncovered.

31894 of 41142 relevant lines covered (77.52%)

1343.88 hits per line