fix(e2e): the lint pass renamed Playwright fixtures, which killed the suite

Playwright resolves a fixture BY NAME from the destructured parameter, so
renaming an unused one to `_page` asks for a fixture that does not exist. The
e2e leg dies before a single test runs and the report contains no test
entries at all.

Introduced by the pass that let the linter see tests/. The linter was right
that the parameters were unused; underscore-prefixing is the fix for an
ordinary unused argument and the wrong fix for a Playwright fixture, because
the name is the lookup key.

Same defect as buildiq#642, found by a fleet sweep.

Author: Conduction Release Bot

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

@@ -48,10 +48,7 @@ import {
 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,
-	}) => {
+	test('owner shares a folder, fans out to a user, and revokes', async ({ page }) => {
 		await gotoLockSettled(page)
 		await unlockVault(page)
 		await openVault(page)