pook/compliancebot
1
0
Fork 0
Code Issues 1.1k Pull requests 82 Projects Releases Packages Wiki Activity Actions

fix: exit with clear error on missing required env vars #195

Closed
pook wants to merge 2 commits from fix/env-var-validation into main
pull from: fix/env-var-validation
merge into: pook:main
Conversation 1 Commits 2 Files changed 6 +406
pook commented 2026-04-09 02:15:00 -04:00
Owner
Copy link

Summary

  • Adds startup validation for OPENAI_API_KEY and DATABASE_URL env vars
  • App exits with FATAL: Missing required environment variable: <name> if either is absent
  • Extracted into testable env-check module with 5 unit tests (all passing)

Closes #114 (refs #142, #190 as duplicates)

Test plan

  • bun test src/env-check.test.ts — 5/5 pass
  • Manual: start API without OPENAI_API_KEY → verify exit with clear message
  • Manual: start API without DATABASE_URL → verify exit with clear message
  • Manual: start API with both set → verify normal startup

🤖 Generated with Claude Code

## Summary - Adds startup validation for `OPENAI_API_KEY` and `DATABASE_URL` env vars - App exits with `FATAL: Missing required environment variable: <name>` if either is absent - Extracted into testable `env-check` module with 5 unit tests (all passing) Closes #114 (refs #142, #190 as duplicates) ## Test plan - [x] `bun test src/env-check.test.ts` — 5/5 pass - [ ] Manual: start API without `OPENAI_API_KEY` → verify exit with clear message - [ ] Manual: start API without `DATABASE_URL` → verify exit with clear message - [ ] Manual: start API with both set → verify normal startup 🤖 Generated with [Claude Code](https://claude.com/claude-code)
Validates required env vars at startup before server initialization.
Extracted into testable env-check module with full unit test coverage.

Closes #114 (refs #142, #190 as duplicates)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
pook commented 2026-04-10 15:06:06 -04:00
Author
Owner
Copy link

Closed 2026-04-10 during pipeline triage.

Merge conflicts with current main were blocking the CEO agent's backlog view. Reopen / resubmit against current main if the work is still relevant — CEO now injects open issues/PRs into its prompt and avoids duplicating.

Closed 2026-04-10 during pipeline triage. Merge conflicts with current main were blocking the CEO agent's backlog view. Reopen / resubmit against current main if the work is still relevant — CEO now injects open issues/PRs into its prompt and avoids duplicating.
pook closed this pull request 2026-04-10 15:06:06 -04:00

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
agent-task

Task for autonomous agent execution

  
agent-task

Paperclip autonomous agent task

No labels
agent-task
agent-task
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
pook/compliancebot!195
No description provided.