Skip to content

fix(platform): improve auth token handling and env loading#253

Merged
Israeltheminer merged 2 commits into
mainfrom
update-ui
Jan 20, 2026
Merged

fix(platform): improve auth token handling and env loading#253
Israeltheminer merged 2 commits into
mainfrom
update-ui

Conversation

@Israeltheminer

@Israeltheminer Israeltheminer commented Jan 20, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Enable JWKS rotation on token generation errors in auth config to improve authentication reliability
  • Fix Vite env loading by using loadEnv in configResolved hook instead of process.env
  • Simplify workflow output schema to description-based approach for better flexibility

Test plan

  • Verify authentication works correctly with the JWKS rotation enabled
  • Confirm environment variables are properly loaded in Vite build
  • Test workflow execution with the simplified output schema

Summary by CodeRabbit

  • Bug Fixes

    • Enhanced authentication token handling with improved error recovery.
  • Changes

    • Simplified product recommendation data structure format.
  • Chores

    • Optimized environment configuration loading during build process.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai

coderabbitai Bot commented Jan 20, 2026

Copy link
Copy Markdown
Contributor
📝 Walkthrough

Walkthrough

This pull request makes three targeted changes across the platform services: (1) enables JWKS rotation on token generation errors in the Convex authentication plugin, (2) removes explicit item-level schema constraints from the recommendations array in the Loopi product recommendation workflow, loosening the schema structure, and (3) refactors the Vite inject-env plugin to load environment variables during the config resolution phase rather than at transform time, precomputing the environment object for HTML injection.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes


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

@Israeltheminer Israeltheminer merged commit 0f3f6fb into main Jan 20, 2026
1 check passed
@Israeltheminer Israeltheminer deleted the update-ui branch January 20, 2026 13:43
@coderabbitai coderabbitai Bot mentioned this pull request Mar 26, 2026
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