Skip to content

fix(deps): revert dependencies to before pg block was added#1189

Merged
icecrasher321 merged 2 commits intomainfrom
staging
Aug 29, 2025
Merged

fix(deps): revert dependencies to before pg block was added#1189
icecrasher321 merged 2 commits intomainfrom
staging

Conversation

@icecrasher321
Copy link
Collaborator

Summary

Brief description of what this PR does and why.

Fixes #(issue)

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation
  • Other: ___________

Testing

How has this been tested? What should reviewers focus on?

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

Screenshots/Videos

@vercel
Copy link

vercel bot commented Aug 29, 2025

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

Project Deployment Preview Comments Updated (UTC)
docs Ready Ready Preview Comment Aug 29, 2025 9:28pm
sim (staging) Building Building Aug 29, 2025 9:28pm

@icecrasher321 icecrasher321 merged commit 316c970 into main Aug 29, 2025
6 of 8 checks passed
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 performs a comprehensive rollback of database integration functionality that was previously added to the SimStudio platform. The changes systematically remove PostgreSQL, MySQL, and Parallel AI integrations across the entire codebase, including:

Database Tools Removed:

  • Complete PostgreSQL integration (query, insert, update, delete, execute operations)
  • Complete MySQL integration (query, insert, update, delete, execute operations)
  • Parallel AI search functionality

Code Changes:

  • Dependencies: Removed @types/pg, mysql2, and pg packages from package.json while retaining the postgres package
  • API Routes: Deleted all PostgreSQL and MySQL API endpoints (/api/tools/postgresql/* and /api/tools/mysql/*)
  • Tools: Removed all database tool configurations and utility functions, cleared tool index files
  • Blocks: Removed PostgreSQL, MySQL, and Parallel blocks from the registry and deleted their configuration files
  • Types: Removed database-specific TypeScript interfaces and reverted the array type from PrimitiveValueType
  • UI Components: Removed PostgresIcon, MySQLIcon, and ParallelIcon from the icons library
  • Documentation: Deleted all documentation files for the removed tools

The PR title indicates this reversion addresses dependency issues introduced when the PostgreSQL block was originally added. The changes restore the codebase to use the existing postgres package for core database operations while removing the additional database tool integrations that were layered on top.

Confidence score: 1/5

  • This PR contains critical issues that will cause immediate build failures and runtime errors
  • Score reflects incomplete cleanup that leaves broken import statements and orphaned registry references
  • Multiple files require immediate attention, particularly the MySQL and PostgreSQL tool index files that are empty but still referenced

41 files reviewed, no comments

Edit Code Review Bot Settings | Greptile

arenadeveloper02 pushed a commit to arenadeveloper02/p2-sim that referenced this pull request Sep 19, 2025
fix(deps): revert dependencies to before pg block was added
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