NBizyaev/mqtt-ingest
1
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Wiki Activity

remove default env values. correct package json with env vars . #6

Merged
APervov merged 3 commits from edge5modbus into main 2026-06-19 08:18:08 +00:00
Conversation 5 Commits 3 Files Changed 14 +70 -111
APervov commented 2026-06-19 06:51:44 +00:00
Contributor
Copy Link
No description provided.
APervov added 1 commit 2026-06-19 06:51:45 +00:00
remove default env values. correct package json with env vars . cbe33500cf
NBizyaev reviewed 2026-06-19 06:58:47 +00:00
app/Dockerfile Outdated
@@ -11,3 +11,3 @@
COPY src ./src
CMD ["npm", "run", "dev"]
CMD ["npm", "run", "start"]
NBizyaev commented 2026-06-19 06:58:47 +00:00
Owner
Copy Link

CMD ["npm", "run", "start"] - это антипаттерн для prod.
npm запускает node как дочерний процесс (например, SIGTERM от Docker при docker stop до node не доходит — контейнер висит 10 секунд до SIGKILL)

prod надо запускать так:

CMD ["node", "dist/index.js"]

Скрипт

npm start

используется во время разработки, чтоб проверить билд после npm build

CMD ["npm", "run", "start"] - это антипаттерн для prod. npm запускает node как дочерний процесс (например, SIGTERM от Docker при docker stop до node не доходит — контейнер висит 10 секунд до SIGKILL) prod надо запускать так: ``` CMD ["node", "dist/index.js"] ``` Скрипт ``` npm start ``` используется во время разработки, чтоб проверить билд после `npm build`
NBizyaev marked this conversation as resolved
APervov added 1 commit 2026-06-19 07:30:21 +00:00
Убрал лишние проверки. Убрал дефолтные значения env переменных. Скорректировал dockerfile 9b53f602ce
NBizyaev reviewed 2026-06-19 07:36:47 +00:00
app/package.json Outdated
@@ -4,3 +4,3 @@
"type": "module",
"scripts": {
"dev": "tsx watch --env-file=.env src/index.ts",
"dev": "tsx watch src/index.ts",
NBizyaev commented 2026-06-19 07:36:47 +00:00
Owner
Copy Link

мы зациклились
"dev": "tsx watch --env-file=.env src/index.ts", - это корректная строчка
потому что это запуск проекта разработчиком
на его, разработчика, компьютере
и переменные окружения он, разработчик, положил в файл .env или .env.* и разумеется хочет чтобы при запуске проекта у него локально, эти переменные окружения из этого файла использовались в его, разработчика, коде.

мы зациклились "dev": "tsx watch --env-file=.env src/index.ts", - это корректная строчка потому что это запуск проекта разработчиком на его, разработчика, компьютере и переменные окружения он, разработчик, положил в файл .env или .env.* и разумеется хочет чтобы при запуске проекта у него локально, эти переменные окружения из этого файла использовались в его, разработчика, коде.
NBizyaev marked this conversation as resolved
NBizyaev reviewed 2026-06-19 07:39:50 +00:00
app/package.json Outdated
@@ -7,2 +6,3 @@
"dev": "tsx watch src/index.ts",
"build": "tsc",
"start": "node --env-file=.env dist/index.js"
"start": "node dist/index.js"
NBizyaev commented 2026-06-19 07:39:50 +00:00
Owner
Copy Link

мы зациклились
"node --env-file=.env dist/index.js" - это корректная строчка
потому что это запуск билда разработчиком
на его, разработчика, компьютере
и переменные окружения он, разработчик, положил в файл .env или .env.* и разумеется хочет чтобы при запуске билда у него локально, эти переменные окружения из этого файла использовались в его, разработчика, коде.

Для запуска билда в продакшне мы будем использовать
CMD ["node", "dist/index.js"]
без указания конкретного .env
так как там в продакшн окружении будут указаны свои собственные значения переменных продакшн окружения.

мы зациклились "node --env-file=.env dist/index.js" - это корректная строчка потому что это запуск билда разработчиком на его, разработчика, компьютере и переменные окружения он, разработчик, положил в файл .env или .env.* и разумеется хочет чтобы при запуске билда у него локально, эти переменные окружения из этого файла использовались в его, разработчика, коде. Для запуска билда в продакшне мы будем использовать CMD ["node", "dist/index.js"] без указания конкретного .env так как там в продакшн окружении будут указаны свои собственные значения переменных продакшн окружения.
NBizyaev marked this conversation as resolved
NBizyaev reviewed 2026-06-19 07:44:36 +00:00
app/Dockerfile
@@ -6,4 +7,4 @@
RUN npm ci
COPY tsconfig.json ./
COPY public ./public
NBizyaev commented 2026-06-19 07:44:36 +00:00
Owner
Copy Link

в проекте нет папки public

(обычно, такую или подобную папку (assets) создают в проекте тогда,
когда нужно использовать какие-то файлы, которые лежат прямо в проекте,
например, файлы маппингов можно было бы вынести в общую папку mappings
и копировать её в контейнер, чтобы быть уверенным, что когда приложению понадобится читать файл маппинга,
то он будет найдет в общей папке mappings. Рекомендую. )

в проекте нет папки public (обычно, такую или подобную папку (assets) создают в проекте тогда, когда нужно использовать какие-то файлы, которые лежат прямо в проекте, например, файлы маппингов можно было бы вынести в общую папку mappings и копировать её в контейнер, чтобы быть уверенным, что когда приложению понадобится читать файл маппинга, то он будет найдет в общей папке mappings. Рекомендую. )
NBizyaev marked this conversation as resolved
NBizyaev reviewed 2026-06-19 07:48:11 +00:00
app/src/handlers/edge5-modbus.ts Outdated
@@ -24,4 +16,0 @@
if (
Array.isArray(value) &&
value.length === REGISTER_COUNT &&
NBizyaev commented 2026-06-19 07:48:11 +00:00
Owner
Copy Link

Мы зациклились.
Я просил убрать бессмысленные перехваты try .. catch
но не саму валидацию вообще как понятие
просто в случае возникновения исключения, оно и так возникнет и мы узнаем, что возникло исключения
НО
валидацию на то, что в массиве действительно 125 элементов убирать не надо.
вдруг там придёт не 125 элементов - это проблема - и вот тут мы не узнаем, что возникло исключение.

Мы зациклились. Я просил убрать бессмысленные перехваты try .. catch но не саму валидацию вообще как понятие просто в случае возникновения исключения, оно и так возникнет и мы узнаем, что возникло исключения НО валидацию на то, что в массиве действительно 125 элементов убирать не надо. вдруг там придёт не 125 элементов - это проблема - и вот тут мы не узнаем, что возникло исключение.
NBizyaev marked this conversation as resolved
APervov added 1 commit 2026-06-19 08:11:14 +00:00
Убрал копирование public папки из dockerfile. Вернул валидацию на количество тегов, присылаемых с node-red. Вернул чтение .env переменной при локальном запуске. d00f79f4d3
APervov merged commit 34f24824e7 into main 2026-06-19 08:18:08 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
NBizyaev
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
NBizyaev
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: NBizyaev/mqtt-ingest#6
Delete Branch "edge5modbus"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?