Linter

.github/workflows/just_do_it.yml

@@ -7,7 +7,7 @@
   workflow_dispatch:
 
 jobs:
-  checkout_and_install:
+  lint:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v4
@@ -16,9 +16,38 @@
         with:
           node-version: lts/*
 
-      - name: Install dependencies
-        run: |
-          npm ci
-          npm run lint
-          npm run test
-          npm run build
+      - name: Clean Install
+        run: npm ci
+
+      - name: Lint
+        run: npm run lint
+
+  test:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+
+      - uses: actions/setup-node@v4
+        with:
+          node-version: lts/*
+
+      - name: Clean Install
+        run: npm ci
+
+      - name: Test
+        run: npm run test
+
+  build:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+
+      - uses: actions/setup-node@v4
+        with:
+          node-version: lts/*
+
+      - name: Clean Install
+        run: npm ci
+
+      - name: Build
+        run: npm run build

.husky/pre-commit

@@ -0,0 +1,3 @@
+npm run lint
+npm run build
+npm run test

app.ts

@@ -1,11 +1,9 @@
-import { installStore } from '@/lib/services';
-
 import { App } from '@slack/bolt';
 import { LogLevel } from '@slack/logger';
 
 import { config } from '@/config';
 import { healthEndpoint } from '@/lib/routes/health';
-import { logger as customLogger } from '@/logger';
+import { withNamespace } from '@/logger';
 
 import '@/lib/stringExtensions';
 import '@/lib/dateExtensions';
@@ -36,6 +34,7 @@ import { register as shortcutsRegister } from '@/shortcuts';
 import { installService } from '@/lib/services';
 import { pointdPalInstallationStore } from '@/lib/installationStore';
 
+const appLogger = withNamespace('pointd-pal');
 const app = new App({
 	signingSecret: config.get('slack.signingSecret'),
 	clientId: config.get('slack.clientId'),
@@ -78,23 +77,23 @@ const app = new App({
 	logger: {
 		getLevel: () => config.get('logLevel') as LogLevel,
 		setLevel: (level: LogLevel) => config.set('logLevel', level),
-		debug: (...msg) => customLogger.debug(msg),
-		info: (...msg) => customLogger.info(msg),
-		warn: (...msg) => customLogger.warn(msg),
-		error: (...msg) => customLogger.error(msg),
-		setName: (name: string) => customLogger.label(name),
+		debug: (...msg) => appLogger.debug(msg),
+		info: (...msg) => appLogger.info(msg),
+		warn: (...msg) => appLogger.warn(msg),
+		error: (...msg) => appLogger.error(msg),
+		setName: (name: string) => appLogger.label(name),
 	},
 });
 
 void (async () => {
-	customLogger.info(
+	appLogger.info(
 		'Before we start the apps we will validate that all installations are up to date by running migrations. Please hold on...',
 	);
 	await installService.migrateAll();
 	// Start your app
 	await app.start({ port: config.get('port') });
 
-	customLogger.info(`⚡️ Bolt app is running at localhost:${config.get('port')}!`);
+	appLogger.info(`⚡️ Bolt app is running at localhost:${config.get('port')}!`);
 })();
 
 messagesCryptoRegister(app);

docker-compose.yml

@@ -1,14 +1,9 @@
 services:
   pointd-pal:
     container_name: pointd-pal
-    build:
-      context: .
-      dockerfile: Dockerfile
+    image: ghcr.io/o-mutt/pointd-pal:latest
     ports:
       - 3000:3000
-    volumes:
-      - ./:/home/node/app
-      - /home/node/app/node_modules
     environment:
       - CHOKIDAR_USEPOLLING=true
     depends_on:

eslint.config.mjs

@@ -2,6 +2,7 @@ import eslintPluginPrettierRecommended from 'eslint-plugin-prettier/recommended'
 import jestPlugin from 'eslint-plugin-jest';
 import tseslint from 'typescript-eslint';
 import eslint from '@eslint/js';
+import globals from 'globals';
 
 export default tseslint.config(
 	{
@@ -30,8 +31,16 @@ export default tseslint.config(
 	},
 	{
 		// enable jest rules on test files
-		files: ['test/**', '**/*.test.ts', '**/*.spec.ts'],
+		files: ['test/**', '**/*.test.*', '**/*.spec.*'],
 		extends: [jestPlugin.configs['flat/recommended']],
+		rules: {
+			'@typescript-eslint/no-floating-promises': 'off',
+		},
+		languageOptions: {
+			globals: {
+				...globals.node,
+			},
+		},
 	},
 	{
 		// ignore unused vars if they are prefixed with _

lint-staged.config.mjs

@@ -0,0 +1,7 @@
+const config = {
+	// eslint-disable-next-line @typescript-eslint/no-unsafe-call, @typescript-eslint/no-unsafe-member-access
+	'*': (stagedFiles) => `eslint ${stagedFiles.join(' ')}`,
+	'*.ts': 'npm run build',
+};
+
+export default config;