otel-cf-workers icon indicating copy to clipboard operation
otel-cf-workers copied to clipboard

WIP Core logic refactor

Open evanderkoogh opened this issue 1 year ago • 1 comments

Very much Work In Progress and not properly tested yet.. do not merge.

Fixes #150 Fixes #179

What this PR solves / how to test:

This PR is a major rewrite of the core logic.. right now there is a lot of duplication in the individual instrumentations around handling configuration, instrumentation of Env and ExecutionContext etc etc.. But because everything was extremely, but not quite the same, it was hard to abstract out.

But now that we have a few different implementations it became clear how to abstract it properly and that is what this PR is. It greatly reduces the surface areas for subtle bugs like Promise not completing to hide.

evanderkoogh avatar Dec 02 '24 03:12 evanderkoogh

@evanderkoogh I know you're busy, but any plans to revisit this? I'm working on a thing that could use support for the new RPC workers, which I believe is blocked on this :)

sinkingpoint avatar Mar 23 '25 12:03 sinkingpoint