Skip to content

refactor: make, scripts. and fix error handling#32

Merged
zulfikar-ditya merged 3 commits into
aolus-software:mainfrom
zulfikar-ditya:main
Feb 8, 2026
Merged

refactor: make, scripts. and fix error handling#32
zulfikar-ditya merged 3 commits into
aolus-software:mainfrom
zulfikar-ditya:main

Conversation

@zulfikar-ditya

@zulfikar-ditya zulfikar-ditya commented Feb 7, 2026

Copy link
Copy Markdown
Member

This pull request introduces a major refactor to improve import organization, streamline build and deployment scripts, and update environment/configuration handling. The changes focus on using granular path aliases for imports, simplifying build commands and workflows, and aligning environment variables with standard Node conventions. Additionally, some minor improvements and cleanups are included.

Import organization and path alias refactor

  • Updated all internal imports to use granular path aliases (e.g., @config, @database, @mailer, @plugins, @repositories, @types, @utils) instead of the generic @libs alias. This improves clarity and maintainability across the codebase. (src/base.ts, src/bull/queue/send-mail-queue.ts, src/bull/worker/send-mail-worker.ts, src/index.ts, src/libs/cache/cache.ts, src/libs/config/app.config.ts, src/libs/database/postgres/index.ts, .github/copilot-instructions.md) [1] [2] [3] [4] [5] [6] [7] [8]

  • Expanded and clarified import instructions in .github/copilot-instructions.md to document available path aliases and enforce a new grouping order for imports.

Build, deployment, and workflow simplification

  • Simplified build and start scripts in package.json, Makefile, and Dockerfile by removing multi-service commands and focusing on a single server entrypoint. Removed concurrently dependency and related scripts. [1] [2] [3] [4] [5] [6] [7]

  • Updated GitHub Actions workflow (.github/workflows/build.yml) to use Bun-native commands for installing and running Drizzle migrations/generations, and removed unused Docker Compose steps. [1] [2] [3]

Environment and configuration updates

  • Changed environment variable handling to use NODE_ENV instead of APP_ENV for consistency with Node.js standards, updating .env.example and src/libs/config/app.config.ts. [1] [2]

Minor improvements and cleanups

  • Improved ESLint configuration to allow unused variables/arguments prefixed with _, reducing unnecessary lint errors.
  • Cleaned up formatting in migration snapshot JSON files for readability. [1] [2] [3] [4] [5] [6] [7]

@zulfikar-ditya zulfikar-ditya merged commit cd9ed36 into aolus-software:main Feb 8, 2026
1 check passed
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.

1 participant