Skip to content

fix(worker): read Supabase credentials from env vars in e2e tests#5669

Open
hobostay wants to merge 1 commit into
Helicone:mainfrom
hobostay:fix/hardcoded-supabase-key
Open

fix(worker): read Supabase credentials from env vars in e2e tests#5669
hobostay wants to merge 1 commit into
Helicone:mainfrom
hobostay:fix/hardcoded-supabase-key

Conversation

@hobostay

Copy link
Copy Markdown

Summary

  • Replace hardcoded Supabase service role key and URL in worker/e2e_test.py with os.environ.get() calls
  • Keep the existing default values for local development convenience while allowing CI/production overrides
  • Also make openai.api_base configurable via OPENAI_API_BASE env var

Test plan

  • Run e2e tests locally with default values — should work as before
  • Run with custom env vars (SUPABASE_SERVICE_ROLE_KEY, SUPABASE_URL, OPENAI_API_BASE) — should use those values

Generated with Claude Code (https://claude.com/claude-code)

…e2e tests

Replace hardcoded Supabase service role key and URL with os.environ.get()
calls. The hardcoded values are kept as defaults for local development
convenience, but production or CI environments can now override them via
environment variables.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.

@vercel

vercel Bot commented May 10, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

3 Skipped Deployments
Project Deployment Actions Updated (UTC)
helicone Skipped Skipped May 10, 2026 10:23am
helicone-bifrost Skipped Skipped May 10, 2026 10:23am
helicone-eu Skipped Skipped May 10, 2026 10:23am

Request Review

@vercel vercel Bot temporarily deployed to Preview – helicone-eu May 10, 2026 10:23 Inactive
@vercel vercel Bot temporarily deployed to Preview – helicone-bifrost May 10, 2026 10:23 Inactive
@vercel vercel Bot temporarily deployed to Preview – helicone May 10, 2026 10:23 Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant