Skip to content

fix(repo): add missing env variables to .env.example#453

Merged
JeanMeijer merged 1 commit intomainfrom
fix/missing-entities-in-env-example
Mar 25, 2026
Merged

fix(repo): add missing env variables to .env.example#453
JeanMeijer merged 1 commit intomainfrom
fix/missing-entities-in-env-example

Conversation

@JeanMeijer
Copy link
Copy Markdown
Collaborator

@JeanMeijer JeanMeijer commented Mar 25, 2026

Description

Briefly describe what you did and why.

Screenshots / Recordings

Add screenshots or recordings here to help reviewers understand your changes.

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • UI/UX update
  • Docs update
  • Refactor / Cleanup

Related Areas

  • Authentication
  • Calendar UI
  • Data/API
  • Docs

Testing

  • Manual testing performed
  • Cross-browser testing (if UI changes)
  • Mobile responsiveness verified (if UI changes)

Checklist

  • I’ve read the CONTRIBUTING guide
  • My code works and is understandable and follows the project's style guidelines
  • I have performed a self-review of my code
  • I have commented my code, particularly in complex areas
  • I have updated the documentation
  • Any dependent changes are merged and published

Notes

(Optional) Add anything else you'd like to share.

By submitting, I confirm I understand and stand behind this code. If AI was used, I’ve reviewed and verified everything myself.


Summary by cubic

Filled in missing environment variables in .env.example and switched the web tRPC client to use the validated env object for logger toggling. This fixes local setup gaps and ensures consistent behavior across environments.

  • Bug Fixes
    • Added optional keys: Zoom OAuth, Google Maps public key (NEXT_PUBLIC_GOOGLE_MAPS_API_KEY), Vercel URLs (VERCEL_URL, NEXT_PUBLIC_VERCEL_URL, NEXT_PUBLIC_VERCEL_ENV), and AI integrations (COMPOSIO_API_KEY, FIRECRAWL_API_KEY, BROWSERBASE_API_KEY).
    • Updated tRPC client to use env.NEXT_PUBLIC_ENV instead of process.env.NEXT_PUBLIC_ENV for logger enablement.

Written for commit 78c4879. Summary will update on new commits.

@vercel
Copy link
Copy Markdown

vercel bot commented Mar 25, 2026

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

Project Deployment Actions Updated (UTC)
analog Ready Ready Preview, Comment Mar 25, 2026 0:33am

Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

2 issues found across 2 files

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name=".env.example">

<violation number="1" location=".env.example:14">
P2: Zoom variables are marked optional, but the runtime env schema requires them.</violation>

<violation number="2" location=".env.example:45">
P2: The AI integrations section is labeled optional, but FIRECRAWL and BROWSERBASE keys are required by env validation.</violation>
</file>

Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.

MARBLE_API_URL=https://api.marblecms.com

# AI integrations (optional)
COMPOSIO_API_KEY=
Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai bot Mar 25, 2026

Choose a reason for hiding this comment

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

P2: The AI integrations section is labeled optional, but FIRECRAWL and BROWSERBASE keys are required by env validation.

Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At .env.example, line 45:

<comment>The AI integrations section is labeled optional, but FIRECRAWL and BROWSERBASE keys are required by env validation.</comment>

<file context>
@@ -29,4 +39,9 @@ SRH_CONNECTION_STRING="redis://redis:6379"
+MARBLE_API_URL=https://api.marblecms.com
+
+# AI integrations (optional)
+COMPOSIO_API_KEY=
+FIRECRAWL_API_KEY=
+BROWSERBASE_API_KEY=
</file context>
Fix with Cubic

MICROSOFT_CLIENT_ID=
MICROSOFT_CLIENT_SECRET=

# Zoom OAuth (optional)
Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai bot Mar 25, 2026

Choose a reason for hiding this comment

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

P2: Zoom variables are marked optional, but the runtime env schema requires them.

Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At .env.example, line 14:

<comment>Zoom variables are marked optional, but the runtime env schema requires them.</comment>

<file context>
@@ -11,8 +11,18 @@ GOOGLE_CLIENT_SECRET=
 MICROSOFT_CLIENT_ID=
 MICROSOFT_CLIENT_SECRET=
 
+# Zoom OAuth (optional)
+ZOOM_CLIENT_ID=
+ZOOM_CLIENT_SECRET=
</file context>
Suggested change
# Zoom OAuth (optional)
# Zoom OAuth
Fix with Cubic

@JeanMeijer JeanMeijer changed the title fix(repo): resolve missing env variables in env.example fix(repo): add missing env variables in env.example Mar 25, 2026
@JeanMeijer JeanMeijer changed the title fix(repo): add missing env variables in env.example fix(repo): add missing env variables to env.example Mar 25, 2026
@JeanMeijer JeanMeijer changed the title fix(repo): add missing env variables to env.example fix(repo): add missing env variables to .env.example Mar 25, 2026
@JeanMeijer JeanMeijer merged commit 2c26cfb into main Mar 25, 2026
7 checks passed
@JeanMeijer JeanMeijer deleted the fix/missing-entities-in-env-example branch March 25, 2026 00:46
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