Convert Extract to TypeScript

[Zyie]

Description of change

Pre-Merge Checklist

• Tests and/or benchmarks are included
• Documentation is changed or added
• Lint process passed (npm run lint)
• Tests passed (npm run test)

[codecov-io]

Codecov Report

Merging #6381 into dev will increase coverage by 0.06%.
The diff coverage is 66.66%.

@@            Coverage Diff            @@
##             dev    #6381      +/-   ##
=========================================
+ Coverage   76.7%   76.76%   +0.06%     
=========================================
  Files         76       70       -6     
  Lines       4142     4028     -114     
=========================================
- Hits        3177     3092      -85     
+ Misses       965      936      -29

Impacted Files	Coverage Δ
...nvas/canvas-graphics/src/CanvasGraphicsRenderer.js	33.7% <56.75%> (+6.13%)	⬆️
packages/canvas/canvas-renderer/src/canvasUtils.js	40.2% <92.85%> (+37.85%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 41be0fe...d99c8b0. Read the comment docs.

[ivanpopelyshev]

ez.

[bigtimebuddy]

Do you think it makes sense to create a IRendererPlugin interface that Extract, Prepare, etc that they can all implement? This would basically just take renderer as a single constructor argument. Not sure if there are any other methods called by Renderer. I like hardening plugin types for users to implement to.

[Zyie]

Do you think it makes sense to create a IRendererPlugin interface that Extract, Prepare, etc that they can all implement? This would basically just take renderer as a single constructor argument. Not sure if there are any other methods called by Renderer. I like hardening plugin types for users to implement to.

I didn't think it was possible to implement an interface that defines a constructor? Could you point to an example for me.

We could create a base class that plugins extend from if an interface doesn't work?

[bigtimebuddy]

Would probably look something like this:

interface IRendererPluginConstructor {
  new (renderer: Renderer): IRendererPlugin;
}

interface IRendererPlugin {
  destroy(): void;
}

So the Renderer here would be:

static registerPlugin(pluginName: string, ctor: IRendererPluginConstructor): void

[bigtimebuddy]

Is this good to go, @Zyie? Was there something else you wanted to do?

[Zyie]

Is this good to go, @Zyie? Was there something else you wanted to do?

Sorry got super busy last week. Just updated this to add the IRendererPlugin like you suggested.

So this is now good to go