feat(helm): add missing environment variables to chart

[asithade]

Summary

Add comprehensive environment variable configuration to the Helm chart including all missing variables required by the application.

Changes

• Added PCC_AUTH0_SECRET for Auth0 session management
• Added complete Snowflake analytics configuration (account, username, role, database, warehouse, API key)
• Added optional Snowflake configuration with sensible defaults (log level, lock strategy, connection pool settings)
• Added LOG_LEVEL and SUPABASE_STORAGE_BUCKET with defaults
• Updated README.md with comprehensive documentation for all new variables

Environment Variables Added

Required Secrets

• PCC_AUTH0_SECRET - Auth0 session secret
• SNOWFLAKE_ACCOUNT - Snowflake account identifier (org-account format)
• SNOWFLAKE_USERNAME - Service user for read-only queries
• SNOWFLAKE_ROLE - User role with SELECT-only permissions
• SNOWFLAKE_DATABASE - Analytics database name
• SNOWFLAKE_WAREHOUSE - Warehouse for query execution
• SNOWFLAKE_API_KEY - Private key for authentication

Optional Configuration

• LOG_LEVEL (default: "info")
• SUPABASE_STORAGE_BUCKET (default: "meeting-attachments")
• SNOWFLAKE_LOG_LEVEL (default: "ERROR")
• SNOWFLAKE_LOCK_STRATEGY (default: "memory")
• SNOWFLAKE_MIN_CONNECTIONS (default: "2")
• SNOWFLAKE_MAX_CONNECTIONS (default: "10")

Validation

✅ Helm chart lints successfully with no errors
✅ All files formatted with Prettier
✅ Documentation is complete and accurate
✅ Build passes successfully

JIRA Ticket

LFXV2-691

Generated with Claude Code

[coderabbitai]

Walkthrough

Configuration and documentation updates add Snowflake Analytics integration parameters and Auth0 session secret management to the Helm chart. Both README and values.yaml files updated with new environment variables, required fields, and defaults for Snowflake connectivity and optional logging/pooling settings.

Changes

Cohort / File(s)	Summary
Documentation charts/lfx-v2-ui/README.md	Added Snowflake Analytics Configuration section documenting required parameters (SNOWFLAKE_ACCOUNT, SNOWFLAKE_USERNAME, SNOWFLAKE_ROLE, SNOWFLAKE_DATABASE, SNOWFLAKE_WAREHOUSE, SNOWFLAKE_API_KEY) and optional settings (SNOWFLAKE_LOG_LEVEL, SNOWFLAKE_LOCK_STRATEGY, SNOWFLAKE_MIN_CONNECTIONS, SNOWFLAKE_MAX_CONNECTIONS). Added PCC_AUTH0_SECRET to Auth0 configuration table as required field.
Configuration Defaults charts/lfx-v2-ui/values.yaml	Added LOG_LEVEL environment variable (default: "info"). Introduced PCC_AUTH0_SECRET for session management. Added Snowflake configuration block with required variables (SNOWFLAKE_ACCOUNT, SNOWFLAKE_USERNAME, SNOWFLAKE_ROLE, SNOWFLAKE_DATABASE, SNOWFLAKE_WAREHOUSE, SNOWFLAKE_API_KEY) and optional variables with defaults (SNOWFLAKE_LOG_LEVEL, SNOWFLAKE_LOCK_STRATEGY, SNOWFLAKE_MIN_CONNECTIONS, SNOWFLAKE_MAX_CONNECTIONS).

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

• Verify Snowflake configuration parameter names and defaults align with Snowflake SDK expectations
• Confirm PCC_AUTH0_SECRET is properly integrated into Auth0 configuration flow in application code
• Check documentation accuracy for connection pooling defaults and log level semantics

Pre-merge checks and finishing touches

❌ Failed checks (2 warnings)

Check name	Status	Explanation	Resolution
Linked Issues Check	⚠️ Warning	The pull request successfully implements most objectives from LFXV2-691: PCC_AUTH0_SECRET has been added for Auth0 session management, all required Snowflake analytics variables have been added with correct configuration, optional Snowflake settings have been configured with appropriate defaults, LOG_LEVEL has been added with the default "info", and documentation has been updated in README.md. However, SUPABASE_STORAGE_BUCKET (default: "meeting-attachments") is listed as an objective in the linked issue and mentioned in the PR description, but it does not appear in the raw_summary of the actual code changes made to values.yaml, indicating this variable may not have been added to the changeset.	Verify that SUPABASE_STORAGE_BUCKET with default value "meeting-attachments" has been added to the values.yaml file under the environment configuration. If this variable is missing from the actual code changes, it should be added to meet the linked issue objectives. Review the PR diff to confirm whether this variable was included in the changeset but not reflected in the raw_summary, or if it needs to be added.
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	You can run @coderabbitai generate docstrings to improve docstring coverage.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title Check	✅ Passed	The pull request title "feat(helm): add missing environment variables to chart" clearly and specifically summarizes the main change in the changeset. The title uses the conventional commit format (feat prefix) and is concise without noise. The raw_summary confirms that the primary changes involve adding environment variables to the Helm chart configuration files (README.md and values.yaml), which directly aligns with what the title describes.
Out of Scope Changes Check	✅ Passed	All changes in the pull request are directly related to the stated objective of adding missing environment variables to the Helm chart. The modifications to README.md introduce documentation for the new environment variables, while the changes to values.yaml add the configuration for these variables with appropriate defaults. No extraneous code modifications, unrelated features, or out-of-scope changes are evident in the raw_summary.

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch feat/LFXV2-691

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

Disabled knowledge base sources:

• Jira integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between edcd722 and 666ec68.

📒 Files selected for processing (2)

• charts/lfx-v2-ui/README.md (3 hunks)
• charts/lfx-v2-ui/values.yaml (3 hunks)

🔇 Additional comments (4)

charts/lfx-v2-ui/values.yaml (1)

171-241: Well-structured additions with sensible defaults. The environment variable additions are properly organized, consistently formatted, and include comprehensive documentation. The Snowflake configuration block is logically separated with clear comments, all required fields have empty values for deployment-time configuration, and optional fields provide conservative production defaults (e.g., ERROR log level, 2-10 connection pool). The placement of LOG_LEVEL and PCC_AUTH0_SECRET among existing configuration is consistent with the file's structure.

charts/lfx-v2-ui/README.md (3)

48-66: Clear and comprehensive required configuration examples. The Snowflake configuration example provides all required parameters with realistic placeholder values, and the addition of PCC_AUTH0_SECRET is well-positioned. Format and styling align with existing configuration examples in the document.

---

137-143: Auth0 table properly updated. The new PCC_AUTH0_SECRET row is correctly marked as required (**Yes**) and includes a descriptive note about the sufficiency requirement. The integration into the Auth0 configuration table is seamless and consistent with the table's existing structure.

---

177-192: Excellent Snowflake Analytics documentation. The new section provides comprehensive parameter documentation with clear descriptions that guide users (e.g., "org-account" format for SNOWFLAKE_ACCOUNT, "read-only queries" context for SNOWFLAKE_USERNAME, "SELECT-only permissions" for role). Required vs. optional distinction is explicit, all defaults are conservative and production-safe, and the introductory context (line 179) explains the purpose of these analytics endpoints. Table formatting matches existing documentation standards.

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[Copilot]

Pull Request Overview

This PR adds Snowflake analytics configuration and additional Auth0 secrets management to the lfx-v2-ui Helm chart. The changes enable integration with Snowflake for analytics endpoints (active-weeks-streak, pull-requests-merged, code-commits) and add the missing Auth0 session secret configuration.

Key changes:

• Added Snowflake environment variables for analytics integration with both required and optional configuration parameters
• Added PCC_AUTH0_SECRET for Auth0 session management
• Added LOG_LEVEL configuration with a default value of "info"

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
charts/lfx-v2-ui/values.yaml	Adds LOG_LEVEL, PCC_AUTH0_SECRET, and comprehensive Snowflake configuration variables (account, credentials, connection pooling)
charts/lfx-v2-ui/README.md	Documents the new Snowflake configuration section, updates Auth0 configuration table to include PCC_AUTH0_SECRET, and adds configuration examples

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.