Skip to content

Library upgrade - #70

Merged
janishar merged 2 commits into
mainfrom
library-upgrade
Jun 7, 2026
Merged

janishar merged 2 commits into
mainfrom
library-upgrade

Conversation

@janishar

@janishar janishar commented Jun 7, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

Copilot AI review requested due to automatic review settings June 7, 2026 10:31
@janishar
janishar merged commit 7c89462 into main Jun 7, 2026
2 checks passed
@janishar
janishar deleted the library-upgrade branch June 7, 2026 10:32

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR upgrades a number of runtime/dev dependencies and adjusts TypeScript + Docker tooling to match the new build/runtime layout, while also adding a basic /health endpoint for container health checks.

Changes:

  • Upgraded multiple dependencies/devDependencies (Express, Mongoose, Jest, TypeScript, etc.) and adjusted scripts (notably the new build output path for serve).
  • Updated TypeScript configuration (rootDir/outDir, include tests) and removed tsconfig.build.json.
  • Added a new health route and introduced Docker Compose healthchecks + service_healthy dependencies.

Reviewed changes

Copilot reviewed 14 out of 17 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
tsconfig.json Updates TS compiler settings/output layout and includes tests in compilation.
tsconfig.build.json Removed build-specific TS config file.
tests/routes/blog/index/unit.test.ts Fixes Jest matcher usage (toHaveReturnedTimes).
src/routes/index.ts Registers the new health router before auth middleware.
src/routes/health/index.ts Adds a new health endpoint returning a timestamp.
package.json Upgrades deps and updates runtime/test scripts (serve path, explicit jest config).
Dockerfile.test Updates test image base to node:24-alpine.
Dockerfile Updates app image base to node:24-alpine.
docker-compose.yml Adds healthchecks, health-based dependency ordering, network, and bumps Mongo/Redis images.
addons/init-project.ts Disables TS checking for the init script and fixes a missing semicolon.
.vscode/tasks.json Renames route template generation task label.
.vscode/settings.json Updates VS Code JS/TS settings keys.
.vscode/launch.json Renames debug config (but still points at old build output path).
.templates/route/index.ts Adjusts relative import paths in the route template file.
.gitignore Adds tsconfig.tsbuildinfo to ignores.
.env.example Renames Mongo admin env vars to MONGO_INITDB_* keys.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/routes/index.ts
Comment on lines +17 to 19
/*---------------------------------------------------------*/
router.use('health', health);
/*---------------------------------------------------------*/
Comment thread docker-compose.yml
Comment on lines +35 to +37
healthcheck:
test: ["CMD", "curl", "-f", "http://localhost:3000/health"]
interval: 5s
Comment thread .vscode/launch.json
Comment on lines +10 to 12
"name": "Debug Server",
"program": "${workspaceFolder}/build/server.js",
"preLaunchTask": "tsc: build - tsconfig.json",
Comment thread addons/init-project.ts
Comment on lines +1 to 3
// @ts-nocheck

import { existsSync, readFileSync, writeFileSync } from 'fs';
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants