-
Notifications
You must be signed in to change notification settings - Fork 3.1k
v0.3.56: i18n, sharepoint operations & logs search improvements #1380
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
waleedlatif1
commented
Sep 18, 2025
- fix(migrations): upgrade drizzle-kit in migrations container (fix(migrations): upgrade drizzle-kit in migrations container #1374)
- fix(variables): remove quote stripping from short & long inputs (fix(variables): remove quote stripping from short & long inputs #1375)
- fix(actions): updated i18n gh action to use PAT instead of default token (fix(actions): updated i18n gh action to use PAT instead of default token #1377)
- fix(selectors): gdrive and slack selectors inf loops (fix(selectors): gdrive and slack selectors inf loops #1376)
- improvement(search): added more granular logs search, added logs export, improved overall search experience (improvement(search): added more granular logs search, added logs export, improved overall search experience #1378)
* fix(migrations): upgrade drizzle-kit in migrations container * fix comments * rm unused file
* fix(variables): remove quote stripping from short & long inputs * restore env * remove quote stripping everywhere * remove unused file
* fix(selectors): gdrive and slack selectors inf loops * remove comment
…rt, improved overall search experience (#1378) * improvement(search): added more granular logs search, added logs export, improved overall search experience * updated tests
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Greptile Summary
This PR includes several important fixes and improvements that enhance the platform's stability and user experience:
Database Migration Fix: Upgrades drizzle-kit from ^0.31.1 to ^0.31.4 and drizzle-orm from ^0.41.0 to ^0.44.5 across the monorepo to resolve compatibility issues in the migrations container. The Docker configuration was also simplified to install all dependencies including dev dependencies, ensuring the migration tooling has access to compatible versions.
Variable Input System Refactoring: Removes quote stripping functionality from input components and the VariableManager class. The system previously stripped quotes during variable interpolation but has evolved to handle values as-is without quote manipulation. This affects multiple components including short/long inputs, comboboxes, file selectors, and formatted text display. The formatDisplayText() function signature was updated to accept only a single string parameter, removing the boolean parameter that controlled quote stripping.
OAuth Selector Infinite Loop Fixes: Resolves infinite loop bugs in Google Drive and Slack channel selectors that occurred when credentials became unauthorized (401 errors). The fixes add proper error handling to clear selections and trigger re-authentication flows, preventing continuous retry attempts with invalid credentials.
GitHub Actions Authentication: Updates the i18n workflow to use a Personal Access Token (PAT) instead of the default GITHUB_TOKEN, enabling proper triggering of downstream workflows when creating automated translation pull requests.
Code Cleanup: Removes a temporary test file (test-self-hosting.ts) that was used for debugging API key encryption functionality during development.
These changes integrate well with the existing codebase architecture, maintaining the monorepo structure while addressing critical infrastructure issues and improving the overall user experience with more reliable selectors and cleaner variable handling.
Confidence score: 4/5
- This PR contains mostly safe fixes and improvements with minimal risk of breaking changes
- Score reflects thorough testing evidence and straightforward bug fixes, though the drizzle upgrade involves dependency changes that require monitoring
- Pay close attention to the drizzle-kit/drizzle-orm upgrades and variable formatting changes across UI components
16 files reviewed, no comments