feat: chat

[mishraomp]

No description provided.

[Copilot]

Pull Request Overview

This pull request removes database-related functionality from the application, including PostgreSQL integration, Flyway migrations, and user management features. The change appears to be refactoring the application to remove database dependencies.

Key changes include:

• Complete removal of PostgreSQL database and Flyway migration components
• Removal of user management functionality from the backend
• Addition of Keycloak authentication to the frontend
• Update to Azure storage configuration for better security

Reviewed Changes

Copilot reviewed 42 out of 47 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
terragrunt/terragrunt.hcl	Removes flyway_image and db_master_password variables
migrations/*	Deletes all Flyway migration files and Docker configuration
initial-azure-setup.sh	Changes blob public access to false for better security
infra/outputs.tf	Removes CloudBeaver app service URL output
infra/modules/postgresql/*	Completely removes PostgreSQL module
infra/modules/flyway/*	Completely removes Flyway module
infra/modules/backend/*	Removes database-related variables and CloudBeaver sidecar
infra/main.tf	Removes PostgreSQL and Flyway module instantiations
frontend/src/services/*	Adds Keycloak authentication services
frontend/src/service/api-service.ts	Updates to use new authenticated HTTP client
frontend/src/main.tsx	Integrates Keycloak initialization
frontend/src/components/Layout.tsx	Adds authentication UI elements
frontend/package.json	Adds Keycloak dependency and updates Node.js version
backend/src/users/*	Removes entire user management module
backend/src/*	Removes Prisma service and database health checks
backend/prisma/schema.prisma	Removes Prisma schema
backend/package.json	Removes Prisma dependencies
.github/workflows/*	Updates CI/CD to exclude migration builds

Files not reviewed (1)

• backend/package-lock.json: Language not supported

Comments suppressed due to low confidence (2)

frontend/package.json:28

• The axios version 1.11.0 does not exist. The latest version in the 1.x series is 1.7.7 as shown in the removed line. Consider using a valid version like "^1.7.7".

    "axios": "^1.11.0",

frontend/package.json:7

• Node.js version 22.17.1 does not exist. The Node.js 22.x series uses different versioning (e.g., 22.0.0, 22.1.0). Consider using a valid Node.js version like ">=22.0.0".

    "node": ">=22.17.1"

[Copilot]

The error interceptor now has two return statements. The second return Promise.reject(error) is redundant since the first interceptor already handles error rejection. Consider removing this duplicate error handling.

[Copilot]

The parameter roles is typed as any, which defeats the purpose of TypeScript's type safety. Consider using a more specific type like string | string[] to better describe the expected input.

Suggested change

	const hasRole = (roles: any) => {
	const jwt = _kc.tokenParsed
	const userroles = jwt?.client_roles
	const includesRoles =
	typeof roles === 'string'
	? userroles?.includes(roles)
	: roles.some((r: any) => userroles?.includes(r))
	const hasRole = (roles: string | string[]) => {
	const jwt = _kc.tokenParsed
	const userroles = jwt?.client_roles
	const includesRoles =
	typeof roles === 'string'
	? userroles?.includes(roles)
	: roles.some((r: string) => userroles?.includes(r))

[Copilot]

The variable r is typed as any. If roles is properly typed as string[], this explicit type annotation is unnecessary and should be removed for cleaner code.

Suggested change

	const hasRole = (roles: any) => {
	const jwt = _kc.tokenParsed
	const userroles = jwt?.client_roles
	const includesRoles =
	typeof roles === 'string'
	? userroles?.includes(roles)
	: roles.some((r: any) => userroles?.includes(r))
	const hasRole = (roles: string | string[]) => {
	const jwt = _kc.tokenParsed
	const userroles = jwt?.client_roles
	const includesRoles =
	typeof roles === 'string'
	? userroles?.includes(roles)
	: roles.some((r) => userroles?.includes(r))