Hacktoberfest 2026:メンテナが10月に向けて印を付けた、オープンで初心者向けの issue。 Hacktoberfest の issue を見る

Startup does not verify the database schema matches the code

オープン
#352 コメント 3 件 リアクション 0 件 担当者 1 名 GitHub で見る

メンテナーはふだん 1 日以内に返信

@titankrypt-gif がすでに取り組んでいます。

2026年10月16日 から。

  • #463 @sparkysolutions10-pixel による — オープン
  • #537 @sparkysolutions10-pixel による — オープン
  • #553 @titankrypt-gif による — オープン

評価

この issue はまだ評価されていません。

説明

area:database difficulty:advanced help wanted points:200 Stellar Wave
Background

prisma/ contains only schema.prisma — there is no migrations/ directory,
so every schema change this project has made has been a prisma db push. That
means there is no record of which schema version is deployed and no check at
startup that the database matches what the code expects.

The failure mode is a process that starts cleanly and then fails on the first
query touching a column that does not exist — at request time, in production,
on one instance of several if a deploy is partial.

What to build
  • A startup check that the schema the code expects is actually present, failing
    fast and naming what is missing.
  • Report schema readiness through /ready (L168) so an instance with a stale
    database never receives traffic.
  • Write down, in docs/, why there are no migrations and what the intended path
    is — adopting Prisma Migrate is a larger decision than this issue, but the
    current state should be a recorded choice rather than an accident.
Acceptance criteria
  • Startup fails fast and names the problem when the database does not match the schema
  • /ready reflects schema readiness
  • The check is cheap enough to run on every boot
  • The absence of a migrations directory is documented with its rationale
  • A test covers the mismatch case

Drips Wave · Complexity: Advanced · 200 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

主要言語
TypeScript
スター
4
フォーク
71
平均マージ
1日 11時間
マージ済み PR(30日)
64

環境構築

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

Divinity-Labs/Lens のほかの issue

Divinity-Labs/Lens の issue をすべて見る

似ている issue

TypeScript の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。