This reverts commit a1bb5ac2, reversing
changes made to 9a08a107.

When the core references `vscode`, we only want to import the types and never generate a real import (which will fail to load). Use `import type` to better enforce this

* debt - allow extHostSearch in common

* web - implement text search provider

* fix text search

* Update extensions/vscode-api-tests/src/extension.ts
Co-Authored-By: NRob Lourens <roblourens@gmail.com>

We compile using the alwaysStrict flag so these directives are not needed

related to #34404

There are many places in the code that do `import Event, { ...} from '.../event'`.

related to #10340

see https://basarat.gitbooks.io/typescript/content/docs/tips/nominalTyping.html and https://github.com/Microsoft/TypeScript/blob/7b48a182c05ea4dea81bab73ecbbe9e013a79e99/src/compiler/types.ts#L693-L698