Skip to content

Conversation

@waleedlatif1
Copy link
Collaborator

Summary

revert tailwind back to v3 for main app, since v4 requires a lengthy refactor

Type of Change

  • Bug fix

Testing

N/A, reverting

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)

@vercel
Copy link

vercel bot commented Sep 26, 2025

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

Project Deployment Preview Comments Updated (UTC)
docs Building Building Preview Comment Sep 26, 2025 1:03am
sim Building Building Preview Comment Sep 26, 2025 1:03am

@waleedlatif1 waleedlatif1 merged commit 18599ac into staging Sep 26, 2025
3 of 5 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 Overview

Summary

This PR reverts Tailwind CSS from v4 back to v3.4.1 for the main app to avoid the extensive refactoring required by v4. The changes are clean and appropriate:

  • Removed @tailwindcss/postcss v4.1.13 dependency from devDependencies
  • Removed autoprefixer plugin from PostCSS configuration (not needed with Tailwind v3)
  • Added back @opentelemetry/exporter-jaeger and encoding dependencies that were removed during v4 migration
  • Updated bun.lock to reflect all dependency changes

The revert is well-executed with no breaking changes to the existing Tailwind v3 configuration, and the PostCSS setup is correctly simplified for v3 compatibility.

Confidence Score: 5/5

  • This PR is extremely safe to merge with no risk of breaking changes
  • Score reflects a clean revert to a known stable configuration with proper dependency management and no logic changes to application code
  • No files require special attention - all changes are straightforward dependency reverts

Important Files Changed

File Analysis

Filename        Score        Overview
apps/sim/package.json 5/5 Properly reverted from Tailwind v4 to v3 by removing @tailwindcss/postcss v4.1.13 dependency and adding back OpenTelemetry Jaeger exporter and encoding dependencies
apps/sim/postcss.config.mjs 5/5 Correctly removed autoprefixer plugin from PostCSS config to match Tailwind v3 setup
bun.lock 5/5 Lock file updated to reflect dependency changes - removed Tailwind v4 dependencies and added back required OpenTelemetry and encoding packages

Sequence Diagram

sequenceDiagram
    participant Dev as Developer
    participant Pkg as package.json
    participant Post as postcss.config.mjs  
    participant TW as Tailwind CSS
    participant Build as Build System
    
    Dev->>Pkg: Remove @tailwindcss/postcss v4.1.13
    Dev->>Post: Remove autoprefixer plugin
    Dev->>Pkg: Keep tailwindcss v3.4.1 in devDependencies
    Dev->>Pkg: Add back @opentelemetry/exporter-jaeger and encoding
    
    Note over TW: Tailwind v4 → v3 revert
    
    Build->>Post: Read PostCSS config
    Post->>TW: Initialize with tailwindcss plugin only
    TW->>Build: Generate CSS using v3 syntax
    Build->>Build: Complete build process
Loading

2 files reviewed, no comments

Edit Code Review Agent Settings | Greptile

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