Merge pull request #17 from ncdcdev/refactor/extract-src-and-add-tests

refactor: ソースコードをsrcに移動し疎結合化・テスト追加

Author: eai04191

.github/workflows/_reusable_check.yml

@@ -25,6 +25,9 @@ jobs:
       - name: Biome Check
         run: npm run check
 
+      - name: Test
+        run: npm test
+
       - name: TypeCheck
         run: npm run typecheck
 

.github/workflows/_reusable_lambda_rie_test.yml

@@ -18,7 +18,7 @@ jobs:
       - run: npm ci
 
       - name: Bundle handler
-        run: npx esbuild index.ts --bundle --platform=node --target=node24 --external:@slack/bolt --outfile=dist/index.js
+        run: npx esbuild src/handler.ts --bundle --platform=node --target=node24 --external:@slack/bolt --outfile=dist/index.js
 
       - name: Prepare Lambda artifact
         run: cp -R node_modules dist/

index.ts

@@ -6,33 +6,26 @@ import * as iam from 'aws-cdk-lib/aws-iam';
 import * as runtime from 'aws-cdk-lib/aws-lambda';
 import * as lambda from 'aws-cdk-lib/aws-lambda-nodejs';
 import type { Construct } from 'constructs';
+import { loadEnv } from '../../src/env.ts';
 
 export class TimesAllBotStack extends cdk.Stack {
 	constructor(scope: Construct, id: string, props?: cdk.StackProps) {
 		super(scope, id, props);
 
-		const slackBotToken = process.env['SLACK_BOT_TOKEN'];
-		if (!slackBotToken) throw new Error('SLACK_BOT_TOKEN is required');
-		const slackSigningSecret = process.env['SLACK_SIGNING_SECRET'];
-		if (!slackSigningSecret)
-			throw new Error('SLACK_SIGNING_SECRET is required');
-		const timesAllChannelId = process.env['TIMES_ALL_CHANNEL_ID'];
-		if (!timesAllChannelId) throw new Error('TIMES_ALL_CHANNEL_ID is required');
-		const workspaceUrl = process.env['WORKSPACE_URL'];
-		if (!workspaceUrl) throw new Error('WORKSPACE_URL is required');
+		const env = loadEnv();
 
 		const projectRoot = path.join(import.meta.dirname, '../..');
 
 		const fn = new lambda.NodejsFunction(this, 'SlackHandler', {
-			entry: path.join(projectRoot, 'index.ts'),
+			entry: path.join(projectRoot, 'src/handler.ts'),
 			handler: 'handler',
 			runtime: runtime.Runtime.NODEJS_24_X,
 			timeout: cdk.Duration.seconds(30),
 			environment: {
-				SLACK_BOT_TOKEN: slackBotToken,
-				SLACK_SIGNING_SECRET: slackSigningSecret,
-				TIMES_ALL_CHANNEL_ID: timesAllChannelId,
-				WORKSPACE_URL: workspaceUrl,
+				SLACK_BOT_TOKEN: env.slackBotToken,
+				SLACK_SIGNING_SECRET: env.slackSigningSecret,
+				TIMES_ALL_CHANNEL_ID: env.timesAllChannelId,
+				WORKSPACE_URL: env.workspaceUrl,
 			},
 			bundling: {
 				nodeModules: ['@slack/bolt'],

infra/lib/times-all-bot-stack.ts

@@ -11,7 +11,8 @@
 	},
 	"dependencies": {
 		"aws-cdk-lib": "^2.245.0",
-		"constructs": "^10.6.0"
+		"constructs": "^10.6.0",
+		"valibot": "^1.3.1"
 	},
 	"devDependencies": {
 		"@tsconfig/strictest": "^2.0.8",

infra/package-lock.json

@@ -9,5 +9,5 @@
 		"types": ["node"],
 		"resolveJsonModule": true
 	},
-	"include": ["bin/**/*.ts", "lib/**/*.ts"]
+	"include": ["bin/**/*.ts", "lib/**/*.ts", "../src/env.ts"]
 }

infra/package.json

@@ -4,17 +4,19 @@
 	"description": "repost times-* channel to \"time-all\"",
 	"type": "module",
 	"scripts": {
-		"dev": "node --env-file=.env index.ts",
+		"dev": "node --env-file=.env src/handler.ts",
 		"lint": "biome check --write .",
 		"format": "biome format .",
 		"check": "biome check .",
-		"typecheck": "tsc --noEmit"
+		"typecheck": "tsc",
+		"test": "node --test 'src/**/*.test.ts'"
 	},
 	"keywords": [],
 	"author": "",
 	"license": "ISC",
 	"dependencies": {
-		"@slack/bolt": "^4.6.0"
+		"@slack/bolt": "^4.6.0",
+		"valibot": "^1.3.1"
 	},
 	"devDependencies": {
 		"@biomejs/biome": "2.4.9",