Skip to content

Conversation

@waleedlatif1
Copy link
Collaborator

waleedlatif1 and others added 5 commits September 18, 2025 11:04
* 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
@vercel
Copy link

vercel bot commented Sep 18, 2025

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

Project Deployment Preview Comments Updated (UTC)
docs Ready Ready Preview Comment Sep 18, 2025 9:25pm
sim (staging) Ready Ready Comment Sep 18, 2025 9:25pm

Copy link
Contributor

@greptile-apps greptile-apps bot left a 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

Edit Code Review Bot Settings | Greptile

@waleedlatif1 waleedlatif1 changed the title v0.3.56: i18n, platform & logs search improvements v0.3.56: i18n, sharepoint operations & logs search improvements Sep 18, 2025
…t from create list (#1379)

* added add list items

(cherry picked from commit df6ea35)

* bun run lint

* minor changes

---------

Co-authored-by: Adam Gough <adamgough@Mac.attlocal.net>
Co-authored-by: Adam Gough <adamgough@Adams-MacBook-Pro.local>
@railway-app railway-app bot temporarily deployed to sim (sim / staging) September 18, 2025 21:19 Inactive
@icecrasher321 icecrasher321 merged commit bff1852 into main Sep 18, 2025
13 of 16 checks passed
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.

4 participants