Skip to content

Conversation

@rileyseaburg
Copy link
Contributor

Summary

  • Adds SKIP_WIDGET_RENDERER: "true" to all manifest files (production, staging, demo)
  • Prevents CF from removing this env var during deployments, which was causing instances to crash when trying to load the Rust widget renderer library

Copilot AI review requested due to automatic review settings December 23, 2025 05:52
@rileyseaburg rileyseaburg merged commit 7779a14 into production Dec 23, 2025
13 of 14 checks passed
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds the SKIP_WIDGET_RENDERER: "true" environment variable to all three Cloud Foundry manifest files to prevent instance crashes during deployments. The environment variable prevents the application from attempting to load the Rust widget renderer library, which was being removed by CF during deployments and causing failures.

Key Changes

  • Added SKIP_WIDGET_RENDERER: "true" to production, staging, and demo manifests
  • Ensures consistent configuration across all deployment environments
  • Prevents the widget renderer initialization code from attempting to load the Rust library

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
touchpoints.yml Added SKIP_WIDGET_RENDERER environment variable to production manifest
touchpoints-staging.yml Added SKIP_WIDGET_RENDERER environment variable to staging manifest
touchpoints-demo.yml Added SKIP_WIDGET_RENDERER environment variable to demo manifest

After thoroughly reviewing the changes, I found no issues with the implementation. The changes are:

  • Consistent across all three manifest files
  • Properly formatted with correct indentation matching each file's style
  • Using the appropriate string value format ("true")
  • Logically placed within the env sections
  • Aligned with the existing code in config/initializers/widget_renderer.rb which checks for this environment variable

The PR successfully addresses the stated problem without introducing any bugs, naming issues, or other concerns.


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

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.

2 participants