The last change was from vorbis to flac in 8e5f8d73, unfortunately
without noting any reasoning. Arguably opus is a more reasonable
default - compressed, with a modern format. Additionally, there are
several known cases where using opus provides much better audio quality
due to driver issues with flac.

Christopher Davis authored Jun 15, 2023 and Christopher Davis committed Jun 15, 2023

Also runs a formatting pass on all files.

Christopher Davis authored Jun 15, 2023 and Christopher Davis committed Jun 15, 2023

Since we use the pre-generated type definitions, we don't actually
need this.

Christopher Davis authored Jun 15, 2023 and Christopher Davis committed Jun 15, 2023

Also fixes the errors reported by eslint.

Allows us to clean up all of the yarn-related configuration.

JS's async/await syntax provides a nicer way for us
to use asynchronous functions without needing to use
cumbersome callbacks.

We're past when this was originally scheduled for removal,
and we shouldn't copy whatever the user places in the host
Recordings folder anymore.

This commit does the following:

* Adds a gi-types submodules that tracks repository with generated
type definitions
* Adds a manually written gettext type definition file, allowing us
to properly check the types for the provided functions
* Removes the old generated types from types/
* Adds ambient types to tsconfig.json, allowing us to remove our
tripple slash reference.

`bind_model ()` *needs* a GObject. Before we ignored this and
told the TypeScript type checker to assume the item is
a `Recording`.

This commit takes `item` as a `GObject.Object` and casts it
to a `Recording` in the closure. This allows us to remove another
`ts-expect-error` comment.

Instead of wrapping our class definitions in
`GObject.registerClass()`, we can use class static initalization
blocks to register the class. This means that we can get
rid of the extra *Class types.

Use the latest fedora image and yarn to get everything we need.

This commit sets up the infrastructure required to build
Sound Recorder as a flatpak while trying to keep similar
code to the non-flatpak build process. To do this we rely
on the features of yarn to allow offline builds from a
specific mirror directory.

Previously we had a convention of using underscores to denote
that a field or function was private. However, this convention
was merely a soft guide that wasn't enforced by anything.
In practice this meant that some fields that should have been
marked as private weren't, and fields that were marked as private
were used outside of their intended files.

As part of the shift to TypeScript, which aims to use hard rules
to enforce conventions, we can adopt these access modifiers and
use them to clean up our code a bit. Where appropriate we've removed
the underscores on fields and functions and prepended access
modifiers appropriate for their use. The addition of these modifiers
allow us to know if a private field isn't in use, meaning that
we can clean up some dead fields.

We still use underscores in two scenarios:

* Fields generated by GObject.registerClass() use the underscore
convention. In order for us to align our TypeScript code we need
to continue using underscores for them when we declare them.
* Private fields that have public accessors still use underscores.

Errors in catch blocks can either be labeled `any` or `unknown`.
The semantics make a big difference. `any` here means
that a value can be anything, thus any method call or field
access is valid - essentially it turns off type checking for
the value. `unknown` however means that it's *something*,
but we need to figure out what.

We pair `unknown` with the `instanceof` operator in order
to narrow down the type to something specific. This allows
us to preserve TypeScript's type checking for error values
and their fields and methods.

See
https://www.typescriptlang.org/docs/handbook/2/narrowing.html#instanceof-narrowing

Enables strict type checking on our TypeScript code.

Move from checked JavaScript to TypeScript. For now this
uses the regular TypeScript settings.

NOTE: The flatpak manifest does not work yet. Please use
`toolbox` or your host OS to build and test.

Closes #126