chore(preview): fix pre-existing CI drift (web format + i18n types + oxlint warnings)

[JOBYINC]

Why

origin/preview had three categories of pre-existing CI drift that silently blocked any PR touching apps/web or @plane/i18n from going green — surfaced while trying to land PR #21 (sort by custom field):

1. check:format failed on 5 unformatted web files (none owned by a single PR; left-over whitespace drift).
2. check:types failed in @plane/i18n on a import.meta.env access with no ImportMeta.env type declaration anywhere in the repo.
3. pre-commit (lint-staged) hook failed on the same 5 files: CI tolerates 11957 oxlint warnings on web (--max-warnings), but the pre-commit hook uses --deny-warnings (0 tolerance). The moment any author runs oxfmt --write on these files, lint-staged exposes 16 pre-existing warnings and refuses to commit.

PR #22 (docs only) passed cleanly because turbo --affected correctly skipped web/i18n. Any other PR is blocked.

What

Fix 1 — oxfmt --write on 5 web files

Pure whitespace / line-wrapping. No behavior change.

apps/web/app/(all)/lark-quick-create/page.tsx
apps/web/ce/components/automations/root.tsx
apps/web/core/components/profile/overview/workload.tsx
apps/web/core/components/workspace/settings/lark-invite-modal.tsx
apps/web/core/services/project/project-automation.service.ts

Fix 2 — import.meta.env type in @plane/i18n

The package has no vite-env.d.ts and this is the only import.meta.env usage in it. Minimal inline typed cast — single line, no new file, no any:

// before
const envDefault = (import.meta.env?.VITE_DEFAULT_LANGUAGE ?? "") as string;

// after
const viteMeta = import.meta as { env?: { VITE_DEFAULT_LANGUAGE?: string } };
const envDefault = (viteMeta.env?.VITE_DEFAULT_LANGUAGE ?? "") as string;

Fix 3 — clear 16 pre-existing oxlint warnings exposed in fix 1

Categorized:

Category	#	Fix
Unused vars	8	Underscore-prefix (the rule's own suggested fix)
s.onload = …	2	s.addEventListener("load", …) (mechanical)
await inside sequential CDN-fallback loop	2	// eslint-disable-next-line no-await-in-loop + reason — Promise.all would change behavior (parallelize)
key={idx} on local-state lists (no stable id, no reorder UI)	2	// eslint-disable-next-line react/no-array-index-key + reason
<a> with no href on a non-link wrapper	1	Replaced with <div> (closest non-anchor element, zero behavior change)
.then() missing explicit return	1	Added return;

Total: +228 / −314 lines, of which most are whitespace re-wrap. Net real change is small.

Test plan

• CI green on this PR (proves the hygiene fix works)
• After merge, PR feat(custom-fields): sort the issue list by a custom field (List parity) #21 (feat/list-sort-by-custom-field) CI re-runs go green without touching its own diff
• No runtime behavior change: only formatting + type cast + lint disables/renames