Skip to content

fix(automations): remove broken AI thread init, fix nav search param, remove rounded-full#683

Merged
Israeltheminer merged 1 commit into
mainfrom
fix/automations-bugs
Mar 5, 2026
Merged

fix(automations): remove broken AI thread init, fix nav search param, remove rounded-full#683
Israeltheminer merged 1 commit into
mainfrom
fix/automations-bugs

Conversation

@Israeltheminer
Copy link
Copy Markdown
Collaborator

@Israeltheminer Israeltheminer commented Mar 5, 2026

Summary

Closes #673, closes #674

Bugs fixed

Copy link
Copy Markdown

@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.

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Mar 5, 2026

📝 Walkthrough

Walkthrough

This PR removes AI chat thread initialization from the automation creation flow and adjusts related UI components. Specifically, it eliminates imports and logic for chat thread creation and metadata updates from the automation creation dialog, removes rounded styling from a Send button, and adds an "ai-chat" panel query parameter to the automations table row navigation handler. The automation creation process now focuses solely on creating the automation and navigating to it, without AI infrastructure setup.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically summarizes all three main changes in the pull request: removing broken AI thread initialization, fixing navigation search parameters, and removing the rounded-full CSS class.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/automations-bugs

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

@Israeltheminer Israeltheminer merged commit 69ba1fc into main Mar 5, 2026
16 checks passed
@Israeltheminer Israeltheminer deleted the fix/automations-bugs branch March 5, 2026 20:03
yannickmonney pushed a commit that referenced this pull request Apr 8, 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

1 participant