#54 fix:resolve eslint findings

.eslintignore

@@ -6,4 +6,11 @@ lib/user-interface/genai-newsletter-ui/src/graphql/*
 lib/user-interface/genai-newsletter-ui/vite.config.ts
 vite.config.ts
 misc/
-lib/api/functions/out/*
+lib/api/functions/out/*
+cdk.out/*
+node_modules/
+lib/api/functions/resolver/*/index.js
+lib/api/functions/pipeline/*/index.js
+
+scripts/*
+pages/*

.eslintrc.json

@@ -5,9 +5,7 @@
     "node": true
   },
   "extends": [
-    "plugin:react/jsx-runtime",
-    "standard-with-typescript",
-    "plugin:react/recommended",
+    "eslint:recommended",
     "plugin:@typescript-eslint/recommended"
   ],
   "parserOptions": {

.github/workflows/eslint.yml

@@ -0,0 +1,45 @@
+# This workflow uses actions that are not certified by GitHub.
+# They are provided by a third-party and are governed by
+# separate terms of service, privacy policy, and support
+# documentation.
+# ESLint is a tool for identifying and reporting on patterns
+# found in ECMAScript/JavaScript code.
+# More details at https://github.com/eslint/eslint
+# and https://eslint.org
+
+name: ESLint
+
+on:
+  push:
+    branches: [ "main" ]
+  pull_request:
+    # The branches below must be a subset of the branches above
+    branches: [ "main" ]
+  schedule:
+    - cron: '45 13 * * 4'
+
+jobs:
+  eslint:
+    name: Run eslint scanning
+    runs-on: ubuntu-latest
+    permissions:
+      contents: read
+      security-events: write
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v4
+      - name: Setup Node.js
+        uses: actions/setup-node@v4
+        with:
+          node-version: 20
+      - name: Install Node Packages
+        run: npm install --save-dev
+      - name: Run ESLint
+        run: npx eslint . --config .eslintrc.json --ext .js,.jsx,.ts,.tsx --format @microsoft/eslint-formatter-sarif --output-file eslint-results.sarif --debug
+        continue-on-error: false
+
+      - name: Upload analysis results to GitHub
+        uses: github/codeql-action/upload-sarif@v3
+        with:
+          sarif_file: eslint-results.sarif
+          wait-for-processing: true

.gitignore

@@ -35,4 +35,5 @@ lib/user-interface/genai-newsletter-ui/public/amplifyconfiguration.json
 lib/user-interface/genai-newsletter-ui/.DS_Store
 .DS_Store
 lib/api/functions/out
-pages/.vitepress/cache
+pages/.vitepress/cache
+eslint-results.sarif

bin/genai-newsletter-app.ts

@@ -5,7 +5,7 @@ import { GenAINewsletter } from '../lib'
 import getConfig from '../lib/config'
 import path from 'path'
 import { AwsSolutionsChecks } from 'cdk-nag'
-import { addNagSuppressions } from './cdk-nag-supressions'
+import { addNagSuppressions } from '../lib/cdk-nag-supressions'
 
 const app = new App()
 

cli/config-manage.ts

@@ -295,15 +295,18 @@ if (['UPDATE', 'NEW'].includes(configStyle)) {
       console.log(formatText('Requires a hostname & a pre-existing AWS Certificate Manager public cert ARN.' +
       'For more information, visit https://docs.aws.amazon.com/acm/latest/userguide/gs-acm-request-public.html',
       { italic: true }))
-      while (true) {
+      let loopA = true
+      while (loopA) {
         const response = prompter(
           formatText('Do you want to proceed? (y/N):', { bold: true, textColor: 'blue' }),
           'N'
         )
         if (response.toLowerCase() === 'y') {
           configHostname = true
+          loopA = false
           break
         } else if (response.toLowerCase() === 'n') {
+          loopA = false
           break
         } else {
           console.log(
@@ -316,7 +319,8 @@ if (['UPDATE', 'NEW'].includes(configStyle)) {
         }
       }
       if (configHostname) {
-        while (true) {
+        let loopB = true
+        while (loopB) {
           const existingHostname = ((config.ui?.hostName) != null) ? config.ui.hostName : ''
           const response = prompter(
             formatText(`Enter the hostname you want to use for the frontend:(${existingHostname})`, { textColor: 'blue' })
@@ -327,8 +331,10 @@ if (['UPDATE', 'NEW'].includes(configStyle)) {
               config.ui = {}
             }
             config.ui.hostName = response
+            loopB = false
             break
           } else if (response.length < 1 && config.ui?.hostName !== undefined && config.ui.hostName !== null) {
+            loopB = false
             break
           } else {
             console.log(
@@ -340,7 +346,8 @@ if (['UPDATE', 'NEW'].includes(configStyle)) {
             )
           }
         }
-        while (true) {
+        let loopC = true
+        while (loopC) {
           const existingAcmCert = ((config.ui?.acmCertificateArn) != null) ? config.ui.acmCertificateArn : ''
           const response = prompter(
             formatText(`Enter the ACM Certificate ARN you want to use for the frontend:(${existingAcmCert})`, { textColor: 'blue' })
@@ -351,8 +358,10 @@ if (['UPDATE', 'NEW'].includes(configStyle)) {
               config.ui = {}
             }
             config.ui.acmCertificateArn = response
+            loopC = false
             break
           } else if (response.length < 1 && config.ui?.acmCertificateArn !== undefined && config.ui.acmCertificateArn !== null) {
+            loopC = false;
             break
           } else {
             console.log(

lib/api/functions/resolver-helper.ts

@@ -54,7 +54,7 @@ export const convertFieldIdToObjectId = (obj: any, idFieldName: string): any =>
     return obj
   }
   obj.id = obj[idFieldName]
-  // eslint-disable-next-line @typescript-eslint/no-dynamic-delete
+   
   delete obj[idFieldName]
   return obj
 }
@@ -73,7 +73,7 @@ export const convertFieldIdToObject = (obj: any, fieldIdName: string, objectName
     __typename: objectName,
     id: obj[fieldIdName]
   }
-  // eslint-disable-next-line @typescript-eslint/no-dynamic-delete
+   
   delete obj[fieldIdName]
   return obj
 }

lib/authorization/authorization-helper.ts

@@ -22,12 +22,12 @@ export const getEntityItem = (schema: Record<string, any>, entityId: string, ent
     }
   }
   if (entityData !== undefined) {
-    item.attributes = getEntityAttributes(schema, entityType, entityData, { logger })
+    item.attributes = getEntityAttributes(schema, entityType, entityData)
   }
   return item
 }
 
-export const getEntityAttributes = (schema: Record<string, any>, entityType: string, entityData: Record<string, any>, optionals?: { logger?: Logger }): Record<string, AttributeValue> => {
+export const getEntityAttributes = (schema: Record<string, any>, entityType: string, entityData: Record<string, any>): Record<string, AttributeValue> => {
   const avpSchema = schema.GenAINewsletter
   const entityAttributes: Record<string, AttributeValue> = {}
   if (avpSchema !== undefined && avpSchema.entityTypes !== undefined) {

lib/authorization/index.action-authorization.ts

@@ -9,8 +9,7 @@ import { Logger, injectLambdaContext } from '@aws-lambda-powertools/logger'
 import { MetricUnits, Metrics } from '@aws-lambda-powertools/metrics'
 import { CognitoJwtVerifier } from 'aws-jwt-verify'
 import { CognitoIdentityProviderClient, GetUserCommand, type GetUserCommandInput } from '@aws-sdk/client-cognito-identity-provider' // ES Modules import
-import middy from '@middy/core'
-import { type Context } from 'aws-lambda'
+import middy from '@middy/core' 
 import { GetSchemaCommand, VerifiedPermissionsClient, type IsAuthorizedCommandInput, IsAuthorizedCommand, Decision } from '@aws-sdk/client-verifiedpermissions'
 import { queryToActionAuth } from './authorization-helper'
 
@@ -45,7 +44,7 @@ const cognitoIdp = tracer.captureAWSv3Client(new CognitoIdentityProviderClient()
 
 let schema: Record<string, unknown>
 
-const lambdaHandler = async (event: any, context: Context): Promise<any> => {
+const lambdaHandler = async (event: any): Promise<any> => {
   logger.debug('AuthorizationCheckEventTriggered', { event })
   if (schema === undefined || schema === null || Object.keys(schema).length === 0) {
     logger.debug('AVP Schema not yet cached. Retrieving AVP Schema')

lib/authorization/index.list-filter-authorization.ts

@@ -9,7 +9,6 @@ import { Logger, injectLambdaContext } from '@aws-lambda-powertools/logger'
 import { MetricUnits, Metrics } from '@aws-lambda-powertools/metrics'
 
 import middy from '@middy/core'
-import { type Context } from 'aws-lambda'
 import { GetSchemaCommand, VerifiedPermissionsClient, type IsAuthorizedCommandInput, IsAuthorizedCommand, Decision } from '@aws-sdk/client-verifiedpermissions'
 import { getEntityItem, lowercaseFirstLetter, queryToActionAuth, queryToResourcesEntity } from './authorization-helper'
 
@@ -29,7 +28,7 @@ const verifiedpermissions = tracer.captureAWSv3Client(new VerifiedPermissionsCli
 
 let schema: Record<string, unknown>
 
-const lambdaHandler = async (event: any, context: Context): Promise<any> => {
+const lambdaHandler = async (event: any): Promise<any> => {
   logger.debug('FilterAuthorizationCheckEventTriggered', { event })
   const { userId, accountId } = event
   if (schema === undefined || schema === null || Object.keys(schema).length === 0) {

lib/authorization/index.read-authorization.ts

@@ -10,7 +10,6 @@ import { MetricUnits, Metrics } from '@aws-lambda-powertools/metrics'
 
 // import { getEntityItem } from '../shared/api/schema-to-avp/permission-map'
 import middy from '@middy/core'
-import { type Context } from 'aws-lambda'
 import { GetSchemaCommand, VerifiedPermissionsClient, type IsAuthorizedCommandInput, IsAuthorizedCommand, Decision } from '@aws-sdk/client-verifiedpermissions'
 import { getEntityItem, lowercaseFirstLetter, queryToActionAuth, queryToResourceEntity } from './authorization-helper'
 
@@ -30,7 +29,7 @@ const verifiedpermissions = tracer.captureAWSv3Client(new VerifiedPermissionsCli
 
 let schema: Record<string, unknown>
 
-const lambdaHandler = async (event: any, context: Context): Promise<any> => {
+const lambdaHandler = async (event: any): Promise<any> => {
   logger.debug('AuthorizationCheckEventTriggered', { event })
   const root = event.root as string | undefined
   const contingentAction = event.contingentAction as string | undefined

lib/data-feed-ingestion/prompts.ts

@@ -3,7 +3,7 @@
  * Copyright Amazon.com, Inc. or its affiliates. All Rights Reserved.
  * SPDX-License-Identifier: MIT-0
  */
-/* eslint-disable @typescript-eslint/no-extraneous-class */
+ 
 export class ArticleIngestorPromptConfiguration {
   private static readonly BASE_PROMPT =
     '\n\nHuman: ' +

lib/newsletter-generator/index.newsletter-campaign-creator.ts

@@ -65,7 +65,7 @@ const lambdaHandler = async (
       ? await getEmailBodiesFromS3(emailKey)
       : await getEmailBodiesFromS3(await getEmailKey(newsletterId, emailId))
   await createEmailTemplate(emailId, html, text, title)
-  const campaignId = await createEmailCampaign(emailId, title)
+  const campaignId = await createEmailCampaign(emailId)
   await saveCampaignId(newsletterId, emailId, campaignId)
 }
 
@@ -196,8 +196,7 @@ const createEmailTemplate = async (
 }
 
 const createEmailCampaign = async (
-  emailId: string,
-  newsletterTitle: string
+  emailId: string
 ): Promise<string> => {
   logger.debug('Creating email campaign', { emailId })
   const input: CreateCampaignCommandInput = {

lib/newsletter-generator/pinpoint-app.pinpoint-campaign-hook.ts

@@ -44,8 +44,7 @@ interface PinpointCampaignHookInput {
 }
 
 const lambdaHandler = async (
-  event: PinpointCampaignHookInput,
-  context: any
+  event: PinpointCampaignHookInput
 ): Promise<any> => {
   logger.debug('Filter event called.', { event })
   const newsletterId = await getNewsletterForCampaign(event.CampaignId)

lib/shared/api/types.ts

@@ -1,4 +1,4 @@
-/* eslint-disable */
+ 
 export type Maybe<T> = T | null
 export type InputMaybe<T> = Maybe<T>
 export type Exact<T extends { [key: string]: unknown }> = {

lib/user-interface/genai-newsletter-ui/src/app.tsx

@@ -20,7 +20,7 @@ import DataFeedDetails from './pages/data-feeds/detail'
 import MyNewsletters from './pages/newsletters/my-newsletters'
 import MyNewsletterSubscriptions from './pages/newsletters/my-subscriptions'
 import EditDataFeed from './pages/data-feeds/edit'
-function App() {
+function App () {
   return (
     <div style={{ height: '100%' }}>
       <BrowserRouter>

lib/user-interface/genai-newsletter-ui/src/common/helpers/storage-helper.ts

@@ -11,14 +11,14 @@ const THEME_STORAGE_NAME = `${PREFIX}-theme`
 const NAVIGATION_PANEL_STATE_STORAGE_NAME = `${PREFIX}-navigation-panel-state`
 
 export abstract class StorageHelper {
-  static getTheme() {
+  static getTheme () {
     const value = localStorage.getItem(THEME_STORAGE_NAME) ?? Mode.Light
     const theme = value === Mode.Dark ? Mode.Dark : Mode.Light
 
     return theme
   }
 
-  static applyTheme(theme: Mode) {
+  static applyTheme (theme: Mode) {
     localStorage.setItem(THEME_STORAGE_NAME, theme)
     applyMode(theme)
 
@@ -30,7 +30,7 @@ export abstract class StorageHelper {
     return theme
   }
 
-  static getNavigationPanelState(): NavigationPanelState {
+  static getNavigationPanelState (): NavigationPanelState {
     const value =
       localStorage.getItem(NAVIGATION_PANEL_STATE_STORAGE_NAME) ??
       JSON.stringify({
@@ -47,7 +47,7 @@ export abstract class StorageHelper {
     return state ?? {}
   }
 
-  static setNavigationPanelState(state: Partial<NavigationPanelState>) {
+  static setNavigationPanelState (state: Partial<NavigationPanelState>) {
     const currentState = this.getNavigationPanelState()
     const newState = { ...currentState, ...state }
     const stateStr = JSON.stringify(newState)

lib/user-interface/genai-newsletter-ui/src/common/hooks/use-navigation-panel-state.ts

@@ -7,7 +7,7 @@ import { useState } from 'react'
 import { StorageHelper } from '../helpers/storage-helper'
 import { NavigationPanelState } from '../types'
 
-export function useNavigationPanelState(): [
+export function useNavigationPanelState (): [
   NavigationPanelState,
   (state: Partial<NavigationPanelState>) => void
 ] {

lib/user-interface/genai-newsletter-ui/src/common/hooks/use-on-follow.ts

@@ -11,7 +11,7 @@ interface FollowDetail {
   href?: string
 }
 
-export default function useOnFollow() {
+export default function useOnFollow () {
   const navigate = useNavigate()
 
   return useCallback(

lib/user-interface/genai-newsletter-ui/src/components/app-configured.tsx

@@ -14,7 +14,7 @@ import Authenticator from './auth/custom-authenticator'
 import { StorageHelper } from '../common/helpers/storage-helper'
 import { Mode } from '@cloudscape-design/global-styles'
 
-export default function AppConfigured() {
+export default function AppConfigured () {
   const [config, setConfig] = useState<AppConfig | null>(null)
   const [error, setError] = useState<boolean | null>(null)
   const [theme, setTheme] = useState(StorageHelper.getTheme());

lib/user-interface/genai-newsletter-ui/src/components/auth/authenticator-views.tsx

@@ -24,7 +24,7 @@ import {
 import { useCallback, useContext, useState } from 'react'
 import { AppContext } from '../../common/app-context'
 
-export function DefaultAuthenticator() {
+export function DefaultAuthenticator () {
   const [username, setUsername] = useState<string>('')
   const [password, setPassword] = useState<string>('')
   const [loading, setLoading] = useState<boolean>(false)
@@ -208,7 +208,7 @@ export function DefaultAuthenticator() {
   )
 }
 
-export function CustomAuthenticator() {
+export function CustomAuthenticator () {
   const appContext = useContext(AppContext)
   const handleFederateClick = () => {
     let provider = null

lib/user-interface/genai-newsletter-ui/src/components/auth/custom-authenticator.tsx

@@ -14,7 +14,7 @@ import {
 } from './authenticator-views'
 import { Container, SpaceBetween, Spinner } from '@cloudscape-design/components'
 
-export default function Authenticator(props: PropsWithChildren) {
+export default function Authenticator (props: PropsWithChildren) {
   const appContext = useContext(AppContext)
   const [isLoading, setIsLoading] = useState<boolean>(true)
   const [userId, setUserId] = useState(userContextDefault.userId)

lib/user-interface/genai-newsletter-ui/src/components/base-app-layout.tsx

@@ -7,7 +7,7 @@ import { AppLayout, AppLayoutProps } from '@cloudscape-design/components'
 import NavigationPanel from './navigation-panel'
 import { useNavigationPanelState } from '../common/hooks/use-navigation-panel-state'
 
-export default function BaseAppLayout(props: AppLayoutProps) {
+export default function BaseAppLayout (props: AppLayoutProps) {
   const [navigationPanelState, setNavigationPanelState] =
     useNavigationPanelState()
   return (