V3 - Tidy up and rewrite

.dockerignore

@@ -1,6 +1,10 @@
 node_modules
-npm-debug.log
+dist
+data
+tests
+coverage
 .env
 .git
 .github
-docs
+.claude
+docs

.editorconfig

@@ -0,0 +1,9 @@
+root = true
+
+[*]
+charset = utf-8
+end_of_line = lf
+indent_style = space
+indent_size = 2
+insert_final_newline = true
+trim_trailing_whitespace = true

.env.example

@@ -1,3 +1,23 @@
+# Required: bot token and application (client) ID from the Discord developer portal
 TOKEN=
 CLIENT_ID=
-JOB_SCHEDULE=0 9 * * *
+
+# Cron schedule (5 or 6 fields) for cleanup, birthday notifications and calendars
+JOB_SCHEDULE=0 9 * * *
+
+# IANA time zone used for scheduling and birthday calculations
+BOT_TIMEZONE=Europe/London
+
+# SQLite database file (directory is created if missing); use :memory: for throwaway runs
+DB_PATH=data/dingir.sqlite
+
+# Data for unreachable guilds is only purged after at least this many failed checks...
+PURGE_MIN_FAILURES=3
+# ...spanning at least this many days
+PURGE_GRACE_DAYS=7
+
+# Log verbosity: fatal, error, warn, info, debug or trace
+LOG_LEVEL=info
+
+# Optional: register slash commands to this guild only (instant updates during development)
+# DEV_GUILD_ID=

.github/dependabot.yml

@@ -5,27 +5,27 @@
 
 version: 2
 updates:
-  - package-ecosystem: "npm"
-    directory: "/"
+  - package-ecosystem: 'npm'
+    directory: '/'
     schedule:
-      interval: "daily"
+      interval: 'daily'
     labels:
-      - "dependencies"
+      - 'dependencies'
     groups:
       all-dependencies:
         patterns:
-          - "*"
+          - '*'
 
-  - package-ecosystem: "github-actions"
-    directory: "/"
+  - package-ecosystem: 'github-actions'
+    directory: '/'
     schedule:
-      interval: "daily"
+      interval: 'daily'
     labels:
-      - "dependencies"
+      - 'dependencies'
 
-  - package-ecosystem: "docker"
-    directory: "/"
+  - package-ecosystem: 'docker'
+    directory: '/'
     schedule:
-      interval: "daily"
+      interval: 'daily'
     labels:
-      - "dependencies"
+      - 'dependencies'

.github/workflows/build-and-publish.yml

@@ -3,15 +3,15 @@ name: Build on Tag
 on:
   push:
     tags:
-    - '*'
+      - '*'
 
 permissions:
   checks: write
   packages: write
-      
+
 jobs:
-  run-linters:
-    name: Run ESLint
+  verify:
+    name: Format, Lint, Typecheck and Test
     runs-on: ubuntu-latest
 
     steps:
@@ -24,18 +24,27 @@ jobs:
           node-version: 24
           cache: npm
 
-      # ESLint and Prettier must be in `package.json`
       - name: Install Node.js dependencies
         run: npm ci
 
       - name: Audit dependencies
         run: npm audit --audit-level=low
 
-      - name: Run linters
+      - name: Check formatting
+        run: npm run format:check
+
+      - name: Run linter
         run: npm run lint
 
+      - name: Type check
+        run: npm run typecheck
+
+      - name: Run tests
+        run: npm run test:coverage
+
   docker:
     name: Build Docker Image
+    needs: verify
     runs-on: ubuntu-latest
 
     steps:
@@ -58,6 +67,6 @@ jobs:
         with:
           push: true
           platforms: linux/amd64,linux/arm64
-          tags: | 
-              ghcr.io/brentoates/dingir:${{ github.ref_name }}
-              ghcr.io/brentoates/dingir:latest
+          tags: |
+            ghcr.io/brentoates/dingir:${{ github.ref_name }}
+            ghcr.io/brentoates/dingir:latest

.github/workflows/lint-and-build.yml

@@ -1,4 +1,4 @@
-name: Lint and Build
+name: Format, Lint, Typecheck and Test
 
 on:
   # Trigger the workflow on push or pull request,
@@ -15,7 +15,7 @@ permissions:
   checks: write
 
 jobs:
-  lint-and-build:
+  verify:
     name: Lint and Build
     runs-on: ubuntu-latest
 
@@ -35,8 +35,14 @@ jobs:
       - name: Audit dependencies
         run: npm audit --audit-level=low
 
-      - name: Run linters
+      - name: Check formatting
+        run: npm run format:check
+
+      - name: Run linter
         run: npm run lint
 
-      - name: Build TypeScript
-        run: npm run build
+      - name: Type check
+        run: npm run typecheck
+
+      - name: Run tests
+        run: npm run test:coverage

.npmrc

@@ -1 +1,2 @@
-registry=https://registry.npmjs.org/
+registry=https://registry.npmjs.org/
+ignore-scripts=true

.oxfmtrc.json

@@ -0,0 +1,11 @@
+{
+  "$schema": "./node_modules/oxfmt/configuration_schema.json",
+  "printWidth": 100,
+  "tabWidth": 2,
+  "useTabs": false,
+  "semi": true,
+  "singleQuote": true,
+  "trailingComma": "all",
+  "endOfLine": "lf",
+  "ignorePatterns": ["package-lock.json", "coverage", "data", "src/resources/**", ".claude"]
+}

.oxlintrc.json

@@ -0,0 +1,26 @@
+{
+  "$schema": "./node_modules/oxlint/configuration_schema.json",
+  "plugins": ["typescript", "unicorn", "import", "node", "oxc"],
+  "categories": { "correctness": "error" },
+  "env": { "node": true, "es2024": true },
+  "rules": {
+    "typescript/no-floating-promises": "error",
+    "typescript/no-misused-promises": "error",
+    "typescript/no-explicit-any": "error",
+    "typescript/consistent-type-imports": "error",
+    "eqeqeq": ["error", "always", { "null": "ignore" }],
+    "curly": ["error", "all"],
+    "no-console": "error"
+  },
+  "overrides": [
+    {
+      "files": ["tests/**/*.ts"],
+      "rules": { "typescript/no-floating-promises": "off" }
+    },
+    {
+      "files": ["src/scripts/**/*.ts"],
+      "rules": { "no-console": "off" }
+    }
+  ],
+  "ignorePatterns": ["node_modules", "coverage", "data", ".claude"]
+}

Dockerfile

@@ -1,17 +1,9 @@
-# Stage 1 - Build
-
-FROM node:24-trixie-slim AS build
+FROM node:24-trixie-slim
 WORKDIR /usr/src/app
-COPY package*.json ./
-RUN npm ci
-COPY . .
-RUN npm run build
-
-# Stage 2 - Dist Only
-
-FROM node:24-trixie-slim AS dist
-WORKDIR /usr/src/app
-COPY package*.json ./
+COPY package*.json .npmrc ./
 RUN npm ci --omit=dev
-COPY --from=build /usr/src/app/dist dist
-ENTRYPOINT ["npm", "start"]
+COPY src ./src
+RUN mkdir -p data && chown node:node data
+USER node
+VOLUME /usr/src/app/data
+CMD ["node", "src/index.ts"]