Better strategy for #62409

Part of #65781

fixes #65758

fixes #31609

This reverts commit 5897b877.

Continues #60565 for converting tests. A few files used TypeStat to add `!`s but the following interface changes were manually inferred from implementing classes:
* `IActiveCodeEditor`'s methods can all return `null`
* `IEditor` needs to report `getControl(): IEdotorControl | null` because `BaseEditor`'s `getControl` can return `null`

See #65672

Not all of these files only import strict null check eligible files

* Converted another ~70 test files to strictNullChecks

These are the remaining test files that didn't require any changes to non-test file types. I'll send those in separately.

Most changes were automatic and small (`!`s).
The only non-trival change is that `extHostSearch.ts` needs an interface for its test data.

Again used [TypeStat](https://github.com/JoshuaKGoldberg/TypeStat) when possible per #65581. I'm happy to split this up into smaller PRs if you'd prefer!

* Added missing `!`s to `commands.test.ts`

Fixes #65183

Fix run active file in terminal on paths with spaces under Windows

Drop gconf dependency

Part of #63832

Fixes #62580

Fixes #65436

Support finding multiple terms on same line in terminal

fixes #64193

fixes #65012

Simplify start debug command handler

fixes #65271