Fourteen files read process.env directly, and nothing validates it

Đang mở
  • Có gì không ổn?
#338 5 bình luận 0 reaction 1 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 4 ngày

@Jossy-ossy đang làm issue này rồi.

Từ ngày 16/10/2026.

  • #387 của @Jossy-ossy — đang mở
  • #481 của @sparkysolutions10-pixel — đã đóng, không merge

Đánh giá

Độ khó
4/5
Thời gian dự kiến
3-5 ngày
Mức phù hợp với người mới
35/100
Loại issue
Tái cấu trúc
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
nodejs, postgresql, prometheus, redis, typescript
Lĩnh vực
backend

Hướng nghiên cứu

Start with src/config.ts, which already reads process.env 34 times, then find the other direct reads in src/db.ts, src/app.ts, src/index.ts, src/api/admin.ts and src/bazaar/rateLimit.ts. Move parsing into one zod-validated module that fails at startup and names the bad variable, and check the result against .env.example with the drift guard, run through npm test (vitest). Done means process.env is read in exactly one module and a non-numeric numeric variable is rejected.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

area:config difficulty:intermediate help wanted points:150 Stellar Wave
Background

src/config.ts exists and is 333 lines, reading process.env 34 times. It does
no validation — no schema, no zod, despite zod already being a dependency used
for route validation. And 14 other files under src/ read process.env
directly, bypassing it entirely, among them src/db.ts, src/app.ts,
src/index.ts, src/api/admin.ts and src/bazaar/rateLimit.ts.

The failure mode is a misspelled or missing variable becoming undefined, then
a default, then a service that starts cleanly and behaves wrongly. A numeric
limit read as undefined silently becomes NaN in arithmetic. Nothing fails
until a price is wrong.

What to build
  • One validated config module as the single source: parse and validate the whole
    environment once at startup with zod, and fail to start on anything invalid.
  • Types derived from the schema, so a typo in a config key is a compile error.
  • Replace the 14 direct reads with imports from it.
  • Every variable documented in .env.example. The repo has a drift guard that
    checks this — run it.
  • Required-in-production variables must be required, not silently defaulted.
Acceptance criteria
  • process.env is read in exactly one module
  • Invalid or missing required configuration fails at startup with a message naming the variable
  • Config values are typed from the schema
  • .env.example documents every variable and the drift guard passes
  • A numeric variable that is not a number is rejected rather than becoming NaN

Drips Wave · Complexity: Intermediate · 150 points

Repo: Divinity-Labs/Lens. Node + TypeScript, Fastify, Prisma on Postgres,
Redis for queues, Prometheus via prom-client.

Run the tests: npm test. The tests are vitest, not jest. Running jest
against this repo reports "66 suites, 0 tests" and means nothing. A single file
is npx vitest run src/__tests__/yourfile.test.ts.

Load test (prices) is already failing on main and has been for some
time. It is not your PR that broke it. Every other check should be green.

This service prices real assets on Stellar mainnet. A number that is wrong
is worse than a number that is missing, because a caller cannot tell the
difference. Prefer failing loudly over returning a plausible value.

Never elide an address, a contract id or a key in code, a comment or a test
fixture. A previous batch wrote a shortened contract id in an issue text and
five contributors invented plausible-looking ids to fill the gap. Write the
whole thing or do not write it at all.

The house pattern for an outbound deadline already exists — see
src/alerts/delivery.ts, which passes signal: AbortSignal.timeout(10_000) to
fetch. Where an issue below asks for a timeout, copy that, do not invent a
second convention.

Required: Before submitting, join the contributor Telegram so your work can be tracked and counted toward the Stellar Wave: https://t.me/+fxHXq8f1SwlkZDBk

Ngôn ngữ chính
TypeScript
Star
4
Fork
71
Merge trung bình
3 ngày 22 giờ
Pull request đã merge (30 ngày)
142

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của Divinity-Labs/Lens

Tất cả issue của Divinity-Labs/Lens

Issue tương tự

Thêm issue về TypeScript

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.