[KagamiChan]

Summary

• run the shared db:migrate command from the deploy hook after dependency installation and before pruning or service restart
• make db:migrate apply Drizzle migrations for PostgreSQL and explicitly no-op for MongoDB
• abort deployment on migration failure while preserving the existing MongoDB deployment path
• add timestamped deployment stages, commit IDs, elapsed time, warnings, and failed-stage exit codes without shell tracing
• sanitize migration failures to exclude stack traces, configured database URLs, and machine-specific paths
• keep reusable migration error formatting in a side-effect-free module
• document the deployment-hook lifecycle and PostgreSQL provisioning without exposing machine filesystem paths
• add regression coverage for backend-aware migrations, hook ordering, safe diagnostics, and error redaction

Validation

• npm run test:unit
• npm run type-check
• npm run lint -- --quiet
• bash -n github-master-hook
• MongoDB npm run db:migrate no-op execution

Review

• CI passes
• all Copilot review threads are resolved
• latest Copilot review generated no new comments

[Copilot]

Pull request overview

This pull request wires the shared db:migrate command into the github-master-hook deploy lifecycle so PostgreSQL migrations run before pruning dependencies and restarting services, while MongoDB deployments explicitly no-op. It also adds structured deploy-stage logging and documents the hook + PostgreSQL provisioning flow.

Changes:

• Add migrateDatabase() to apply Drizzle migrations for PostgreSQL and no-op for MongoDB, with unit coverage.
• Update github-master-hook to run npm run db:migrate after npm ci and before npm prune --omit=dev, with stage/timing/commit logging and failure trapping.
• Document the deploy hook lifecycle and PostgreSQL operational expectations in README + docs.

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
tests/hooks.test.ts	Adds regression tests asserting hook ordering (migrate before prune/restart) and presence of stage/failure logging without shell tracing.
tests/database-migrations.test.ts	Adds unit tests for backend-aware migration behavior (MongoDB no-op, PostgreSQL applies, errors propagate).
src/db/postgres/migrations.ts	Introduces migrateDatabase() wrapper to route migrations based on resolved backend.
scripts/postgres-migrate.ts	Switches db:migrate entrypoint to call migrateDatabase() and print backend-appropriate completion messaging.
README.md	Documents deploy-hook lifecycle, ordering, and log expectations at a high level.
github-master-hook	Adds deploy stages, trap-based failure reporting, and runs db:migrate before pruning and restart steps.
docs/postgresql-deployment.md	Documents PostgreSQL provisioning/deploy behavior with the updated hook lifecycle and operational checks.

---

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

[Copilot]

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.

[Copilot]

Pull request overview

Copilot reviewed 8 out of 8 changed files in this pull request and generated 1 comment.

[Copilot]

Pull request overview

Copilot reviewed 8 out of 8 changed files in this pull request and generated 1 comment.

[Copilot]

Pull request overview

Copilot reviewed 8 out of 8 changed files in this pull request and generated no new comments.