Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

Add inserted_at to lightning_columns

Aperta Adatta ai principianti
  • Qualcosa non va?
#559 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 2 giorni

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
84/100
Tipo di issue
Funzionalità
Chiarezza
Specificata chiaramente
Stato di attività
Tranquilla
Stack tecnologico
bun, sql, typescript
Ambito
database, testing

Direzione di ricerca

Inizia da platform/src/db/migrate.ts:40-47 e platform/test/db.test.ts:36-39, quindi esamina il formato di migrazione esistente in platform/migrations/0001_lightning_clients.sql. Esegui bun run migrate su un database in cui 0001 è registrata ed esegui i test del database. Il lavoro è completato quando 0002 viene applicata una volta, le righe esistenti ricevono inserted_at e l’asserzione sulla colonna ha esito positivo senza modifiche a store.ts.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

platform
Summary

lightning_clients doesn't record when a client was provisioned, so we can't tell when a client was added, and a future client list command would have nothing to sort or display by.

The runner records each applied filename in _migrations and only runs files it hasn't recorded yet (platform/src/db/migrate.ts:40-45); that table is the source of truth, not the IF NOT EXISTS guards in the DDL (platform/src/db/migrate.ts:5-7).

Proposed change
  • Add platform/migrations/0002_lightning_clients_inserted_at.sql with: 
    ALTER TABLE lightning_clients
  ADD COLUMN IF NOT EXISTS inserted_at TIMESTAMPTZ NOT NULL DEFAULT now();
    Existing rows backfill to the migration time, which is fine for a provisioning timestamp.
  • No change needed in store.ts: insertClient names its columns explicitly (name, auth_token_hash, anthropic_api_key), and DEFAULT now() fills in inserted_at on insert (platform/src/auth/client/store.ts:34-37). Verified.
  • Extend the column assertions in db.test.ts to expect inserted_at (platform/test/db.test.ts:36-39).
Acceptance criteria
  • platform/migrations/0002_lightning_clients_inserted_at.sql adds inserted_at TIMESTAMPTZ NOT NULL DEFAULT now() via ALTER TABLE ... ADD COLUMN IF NOT EXISTS
  • 0001_lightning_clients.sql is left untouched
  • bun run migrate applies 0002 against a DB that already has 0001 recorded, and is a no-op on a second run (platform/src/db/migrate.ts:42-47)
  • Existing rows have inserted_at backfilled to the migration time
  • db.test.ts asserts inserted_at is present in lightning_clients (platform/test/db.test.ts:36-39)
  • No change required to insertClient in store.ts (confirmed the default populates it)
Lingua principale
Jupyter Notebook
Stelle
5
Fork
10
Merge medio
2g 6h
PR unite (30g)
16

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di OpenFn/apollo

Tutte le issue di OpenFn/apollo

Issue simili

Altre issue su Databases

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.