feat: encoder

[mdydek]

Closes #1299, because it uses different resampler, which does not produce this issue

⚠️ Breaking changes ⚠️

• recorder.start() no longer takes options
• fileNamePrefix is renamed to fileName, with different meaning. The new field is the exact file name, with nothing appended.
• enableFileOutput now rejects names it used to accept. The name must be bare: no extension, no path separators, no .., and at most 128 characters. Violations return an error result from enableFileOutput rather than being passed through.

Introduced changes

• use system apis instead of ffmpeg to encode recorded buffers
• abstracted file writers and recorders, there is main c++ class, which only calls native classes, but majority of the implementation can be shared

Proposed architecture:

Checklist

• Linked relevant issue
• Updated relevant documentation
• Added/Conducted relevant tests
• Performed self-review of the code
• Updated Web Audio API coverage
• Added support for web
• Updated old arch android spec file

[github-actions]

WPT non-regression comparison

PASS — no regressions · 0 improved section(s) · overall 3420 → 3420 (0)

Unchanged sections (28)

Spec section	Base pass	Head pass	Delta
Processing model	0	0	0
Other	56	56	0
AnalyserNode	138	138	0
AudioBuffer	140	140	0
AudioBufferSourceNode	218	218	0
AudioContext	58	58	0
AudioNode	261	261	0
AudioParam	680	680	0
BiquadFilterNode	275	275	0
ChannelMergerNode	30	30	0
ChannelSplitterNode	7	7	0
ConstantSourceNode	64	64	0
ConvolverNode	203	203	0
DelayNode	104	104	0
DestinationNode	0	0	0
DynamicsCompressorNode	4	4	0
GainNode	15	15	0
IIRFilterNode	87	87	0
MediaElementAudioSourceNode	0	0	0
MediaStreamAudioDestinationNode	1	1	0
MediaStreamAudioSourceNode	0	0	0
OfflineAudioContext	35	35	0
OscillatorNode	104	104	0
PannerNode	723	723	0
PeriodicWave	33	33	0
ScriptProcessorNode	0	0	0
StereoPannerNode	102	102	0
WaveShaperNode	82	82	0

_{Baseline: 87cca81bff30ee15f3a407008cdc2ee30527ef89 · Candidate: a84156a7b787d27f59254ba7036c3d6e8a3ac675}

_{Workflow run · this comment is updated on every push.}

[closetcaiman]

Please add description to the PR for fast-access reference to the scope of these changes.

[maciejmakowski2003]

could you add an overview of refactor? would be great to get some diagram

[closetcaiman]

Shouldn't those be conditionally compiled per platform (like we used to do with decoding)? Now the common header actually considers both platforms.

[mdydek]

hmmm, don't you think that it would introduce some unnecessary noise, it's only option and the name directly suggest that where to use it. Maybe we could introduce some defaults there, to not use them directly in another system (ios properties in android), left for discussion

[closetcaiman]

I'd also consider this worker thread to be a separate class - worth to skim a bit, but if the shared state is too much I'd stick to offloader approach so that it doesn't get messy.

[mdydek]

you mean that runWriterTask should be in a different class?