Add oxlint rule to catch thrown un-awaited redirect helpers #8303
pr_checks.yml
on: pull_request
Detect changes
7s
code-quality
/
code-quality
1m 42s
Matrix: e2e / cli-v3
Waiting for pending jobs
Matrix: internal / 🧪 Unit Tests: Internal
Waiting for pending jobs
Matrix: packages / 🧪 Unit Tests: Packages
Waiting for pending jobs
Matrix: sdk-compat / node-compat
Waiting for pending jobs
Matrix: webapp / 🧪 Unit Tests: Webapp
Waiting for pending jobs
typecheck
/
typecheck
3m 23s
e2e-webapp
/
🧪 E2E Tests: Webapp
sdk-compat
/
Bun Runtime
sdk-compat
/
Cloudflare Workers
sdk-compat
/
Deno Runtime
All PR Checks
6s
Annotations
11 errors
|
code-quality / code-quality
Process completed with exit code 1.
|
|
trigger(no-thrown-unawaited-redirect):
apps/webapp/app/routes/vercel.onboarding.tsx#L154
Throwing an un-awaited "redirectWithErrorMessage()" throws a pending Promise (Remix renders the error boundary instead of redirecting). Use "throw await redirectWithErrorMessage()" or "return redirectWithErrorMessage()".
|
|
trigger(no-thrown-unawaited-redirect):
apps/webapp/app/routes/vercel.onboarding.tsx#L92
Throwing an un-awaited "redirectWithErrorMessage()" throws a pending Promise (Remix renders the error boundary instead of redirecting). Use "throw await redirectWithErrorMessage()" or "return redirectWithErrorMessage()".
|
|
trigger(no-thrown-unawaited-redirect):
apps/webapp/app/routes/vercel.onboarding.tsx#L72
Throwing an un-awaited "redirectWithErrorMessage()" throws a pending Promise (Remix renders the error boundary instead of redirecting). Use "throw await redirectWithErrorMessage()" or "return redirectWithErrorMessage()".
|
|
trigger(no-thrown-unawaited-redirect):
apps/webapp/app/routes/resources.orgs.$organizationSlug.select-plan.tsx#L142
Throwing an un-awaited "redirectWithErrorMessage()" throws a pending Promise (Remix renders the error boundary instead of redirecting). Use "throw await redirectWithErrorMessage()" or "return redirectWithErrorMessage()".
|
|
trigger(no-thrown-unawaited-redirect):
apps/webapp/app/routes/resources.orgs.$organizationSlug.select-plan.tsx#L83
Throwing an un-awaited "redirectWithErrorMessage()" throws a pending Promise (Remix renders the error boundary instead of redirecting). Use "throw await redirectWithErrorMessage()" or "return redirectWithErrorMessage()".
|
|
trigger(no-thrown-unawaited-redirect):
apps/webapp/app/routes/_app.orgs.$organizationSlug.projects.$projectParam.env.$envParam.branches/route.tsx#L137
Throwing an un-awaited "redirectWithErrorMessage()" throws a pending Promise (Remix renders the error boundary instead of redirecting). Use "throw await redirectWithErrorMessage()" or "return redirectWithErrorMessage()".
|
|
trigger(no-thrown-unawaited-redirect):
apps/webapp/app/routes/_app.orgs.$organizationSlug.projects.$projectParam.env.$envParam.regions/route.tsx#L113
Throwing an un-awaited "redirectWithErrorMessage()" throws a pending Promise (Remix renders the error boundary instead of redirecting). Use "throw await redirectWithErrorMessage()" or "return redirectWithErrorMessage()".
|
|
trigger(no-thrown-unawaited-redirect):
apps/webapp/app/routes/_app.orgs.$organizationSlug.projects.$projectParam.env.$envParam.regions/route.tsx#L106
Throwing an un-awaited "redirectWithErrorMessage()" throws a pending Promise (Remix renders the error boundary instead of redirecting). Use "throw await redirectWithErrorMessage()" or "return redirectWithErrorMessage()".
|
|
trigger(no-thrown-unawaited-redirect):
apps/webapp/app/routes/_app.orgs.$organizationSlug.projects.$projectParam.env.$envParam.concurrency/route.tsx#L158
Throwing an un-awaited "redirectWithErrorMessage()" throws a pending Promise (Remix renders the error boundary instead of redirecting). Use "throw await redirectWithErrorMessage()" or "return redirectWithErrorMessage()".
|
|
All PR Checks
Process completed with exit code 1.
|