fix(e2e): make the suite load again after the lint sweep (#590)

The lint pass left two shapes that eslint and prettier both call clean while
the Playwright suite fails to load. `development` collected ZERO tests.

A Playwright callback's destructuring is a FIXTURE REQUEST, not a binding list.
Renaming an unused `request` to `_request` asks for a fixture that does not
exist and the file never loads. The binding is renamed, the key kept:

    async ({ page, request: _request }) => {

And the type-import conversion added a second `import type { Page }` to a file
that already imported `Page` alongside `ConsoleMessage`, which is a
duplicate-identifier SyntaxError.

`npx playwright test --list` parses every spec without running one, and is the
check that catches both. A green linter cannot.

Author: rubenvdlinde

tests/e2e/workflows/team-folder-sharing.spec.ts

@@ -50,7 +50,7 @@ const API = `${APP_BASE.replace('/apps/keepiq', '')}/apps/keepiq/api/v1`
 test.describe('team folder sharing', () => {
 	test('owner shares a folder, fans out to a user, and revokes', async ({
 		page,
-		_request,
+		request: _request,
 	}) => {
 		await gotoLockSettled(page)
 		await unlockVault(page)