From 524360db5fe13349ace928a76d61bad19beddcf5 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Tue, 6 Jan 2026 16:51:49 +0000 Subject: [PATCH] Phase 1: Fix require-await and await-thenable errors Co-authored-by: johndoe6345789 <224850594+johndoe6345789@users.noreply.github.com> --- frontends/nextjs/src/app/api/health/route.test.tsx | 2 +- .../nextjs/src/components/PackageStyleLoader.tsx | 2 +- frontends/nextjs/src/lib/auth/api/fetch-session.ts | 4 ++-- frontends/nextjs/src/lib/auth/api/login.ts | 4 ++-- frontends/nextjs/src/lib/auth/api/register.ts | 4 ++-- frontends/nextjs/src/lib/compiler/index.ts | 4 ++-- .../github/workflows/listing/list-workflow-runs.ts | 4 ++-- frontends/nextjs/src/lib/routing/index.ts | 12 ++++++------ .../nextjs/src/lib/ui-pages/load-page-from-db.ts | 4 ++-- .../src/lib/ui-pages/load-page-from-lua-packages.ts | 4 ++-- 10 files changed, 22 insertions(+), 22 deletions(-) diff --git a/frontends/nextjs/src/app/api/health/route.test.tsx b/frontends/nextjs/src/app/api/health/route.test.tsx index 6f3f14d67..debe644b8 100644 --- a/frontends/nextjs/src/app/api/health/route.test.tsx +++ b/frontends/nextjs/src/app/api/health/route.test.tsx @@ -5,7 +5,7 @@ import { GET } from './route' describe('GET /api/health', () => { it('returns OK status and permission level count', async () => { - const response = await GET(new NextRequest('http://example.com/api/health')) + const response = GET(new NextRequest('http://example.com/api/health')) const payload = await response.json() expect(payload.status).toBe('ok') diff --git a/frontends/nextjs/src/components/PackageStyleLoader.tsx b/frontends/nextjs/src/components/PackageStyleLoader.tsx index 69bb51609..5c80bb6c7 100644 --- a/frontends/nextjs/src/components/PackageStyleLoader.tsx +++ b/frontends/nextjs/src/components/PackageStyleLoader.tsx @@ -40,7 +40,7 @@ export function PackageStyleLoader({ packages }: PackageStyleLoaderProps) { } if (packages.length > 0) { - loadStyles() + void loadStyles() } }, [packages]) diff --git a/frontends/nextjs/src/lib/auth/api/fetch-session.ts b/frontends/nextjs/src/lib/auth/api/fetch-session.ts index fea030d20..2310b566d 100644 --- a/frontends/nextjs/src/lib/auth/api/fetch-session.ts +++ b/frontends/nextjs/src/lib/auth/api/fetch-session.ts @@ -4,7 +4,7 @@ import type { User } from '@/lib/types/level-types' -export async function fetchSession(): Promise { +export function fetchSession(): Promise { // TODO: Implement session fetching - return null + return Promise.resolve(null) } diff --git a/frontends/nextjs/src/lib/auth/api/login.ts b/frontends/nextjs/src/lib/auth/api/login.ts index 2e71b5fa2..c0389aae1 100644 --- a/frontends/nextjs/src/lib/auth/api/login.ts +++ b/frontends/nextjs/src/lib/auth/api/login.ts @@ -9,8 +9,8 @@ export interface LoginCredentials { password: string } -export async function login(_identifier: string, _password: string): Promise { +export function login(_identifier: string, _password: string): Promise { // TODO: Implement login // For now, throw an error to indicate not implemented - throw new Error('Login not implemented') + return Promise.reject(new Error('Login not implemented')) } diff --git a/frontends/nextjs/src/lib/auth/api/register.ts b/frontends/nextjs/src/lib/auth/api/register.ts index f32398614..dcd806096 100644 --- a/frontends/nextjs/src/lib/auth/api/register.ts +++ b/frontends/nextjs/src/lib/auth/api/register.ts @@ -10,7 +10,7 @@ export interface RegisterData { password: string } -export async function register(_username: string, _email: string, _password: string): Promise { +export function register(_username: string, _email: string, _password: string): Promise { // TODO: Implement registration - throw new Error('Registration not implemented') + return Promise.reject(new Error('Registration not implemented')) } diff --git a/frontends/nextjs/src/lib/compiler/index.ts b/frontends/nextjs/src/lib/compiler/index.ts index f66ffc9e6..e44de311f 100644 --- a/frontends/nextjs/src/lib/compiler/index.ts +++ b/frontends/nextjs/src/lib/compiler/index.ts @@ -17,7 +17,7 @@ export function compile(source: string, _options?: CompileOptions): CompileResul return { code: source } } -export function loadAndInjectStyles(_packageId: string): string { +export function loadAndInjectStyles(_packageId: string): Promise { // TODO: Implement style loading and injection - return '' + return Promise.resolve('') } diff --git a/frontends/nextjs/src/lib/github/workflows/listing/list-workflow-runs.ts b/frontends/nextjs/src/lib/github/workflows/listing/list-workflow-runs.ts index 3e885db54..67232379c 100644 --- a/frontends/nextjs/src/lib/github/workflows/listing/list-workflow-runs.ts +++ b/frontends/nextjs/src/lib/github/workflows/listing/list-workflow-runs.ts @@ -17,7 +17,7 @@ export interface ListWorkflowRunsOptions { perPage?: number } -export function listWorkflowRuns(_options: ListWorkflowRunsOptions): WorkflowRun[] { +export function listWorkflowRuns(_options: ListWorkflowRunsOptions): Promise { // TODO: Implement workflow runs listing - return [] + return Promise.resolve([]) } diff --git a/frontends/nextjs/src/lib/routing/index.ts b/frontends/nextjs/src/lib/routing/index.ts index d3cef4aa2..f0549bf25 100644 --- a/frontends/nextjs/src/lib/routing/index.ts +++ b/frontends/nextjs/src/lib/routing/index.ts @@ -65,9 +65,9 @@ export function parseRestfulRequest( export function executeDbalOperation( _op: unknown, _context?: unknown -): { success: boolean; data?: unknown; error?: string; meta?: unknown } { +): Promise<{ success: boolean; data?: unknown; error?: string; meta?: unknown }> { // TODO: Implement DBAL operation execution - return { success: false, error: 'Not implemented' } + return Promise.resolve({ success: false, error: 'Not implemented' }) } export function executePackageAction( @@ -76,9 +76,9 @@ export function executePackageAction( _action: unknown, _id: unknown, _context?: unknown -): { success: boolean; data?: unknown; error?: string } { +): Promise<{ success: boolean; data?: unknown; error?: string }> { // TODO: Implement package action execution - return { success: false, error: 'Not implemented' } + return Promise.resolve({ success: false, error: 'Not implemented' }) } export interface TenantValidationResult { @@ -91,9 +91,9 @@ export function validateTenantAccess( _user: unknown, _tenant: unknown, _minLevel: unknown -): TenantValidationResult { +): Promise { // TODO: Implement tenant access validation - return { allowed: false, reason: 'Not implemented' } + return Promise.resolve({ allowed: false, reason: 'Not implemented' }) } // Re-export auth functions diff --git a/frontends/nextjs/src/lib/ui-pages/load-page-from-db.ts b/frontends/nextjs/src/lib/ui-pages/load-page-from-db.ts index 4b82af5fb..9610d76f5 100644 --- a/frontends/nextjs/src/lib/ui-pages/load-page-from-db.ts +++ b/frontends/nextjs/src/lib/ui-pages/load-page-from-db.ts @@ -11,7 +11,7 @@ export interface UIPageData { actions?: Record } -export function loadPageFromDb(_path: string, _tenantId?: string): PageConfig | null { +export function loadPageFromDb(_path: string, _tenantId?: string): Promise { // TODO: Implement page loading from database - return null + return Promise.resolve(null) } diff --git a/frontends/nextjs/src/lib/ui-pages/load-page-from-lua-packages.ts b/frontends/nextjs/src/lib/ui-pages/load-page-from-lua-packages.ts index 79a67cfa4..67abc73cc 100644 --- a/frontends/nextjs/src/lib/ui-pages/load-page-from-lua-packages.ts +++ b/frontends/nextjs/src/lib/ui-pages/load-page-from-lua-packages.ts @@ -4,7 +4,7 @@ import type { PageConfig } from '../types/level-types' -export function loadPageFromLuaPackages(_b_path: string): PageConfig | null { +export function loadPageFromLuaPackages(_b_path: string): Promise { // TODO: Implement page loading from Lua packages - return null + return Promise.resolve(null) }