Skip to content

Config: Remove the GPT-5.4-Pro Model and UI Updates#215

Merged
Ayush8923 merged 4 commits into
mainfrom
feat/ui-updates
Jun 16, 2026
Merged

Config: Remove the GPT-5.4-Pro Model and UI Updates#215
Ayush8923 merged 4 commits into
mainfrom
feat/ui-updates

Conversation

@Ayush8923

@Ayush8923 Ayush8923 commented Jun 16, 2026

Copy link
Copy Markdown
Collaborator

Issue:

Closes: #214

Type:

type: fix

Summary:

  • Removed the GPT-5.4-Pro model from the config.
  • Added the maxLength limit in Name field on Dataset and Evals form.
  • Fixed the breaking UI in dataset and evals form while choosing the large name document.
  • Use the InfoTooltip component instead of hard-coding tooltip code.

@coderabbitai

coderabbitai Bot commented Jun 16, 2026

Copy link
Copy Markdown

Review Change Stack

Warning

Review limit reached

@Ayush8923, we couldn't start this review because you've reached your PR review rate limit.

More reviews will be available in 25 minutes and 48 seconds. Learn how PR review limits work.

Your organization has used up its prepaid credits, and credit purchases are no longer available. Enable the review add-on in the billing tab to keep reviews running — you're only billed for reviews past your plan's rate limits ($0.25/file).

⌛ How to resolve this issue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available.

Please see our Fair Usage Limits Policy for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 14b3cae2-0b3d-4c02-84e7-45c86c6e28d6

📥 Commits

Reviewing files that changed from the base of the PR and between 8746f93 and cb9955b.

📒 Files selected for processing (1)
  • app/components/evaluations/CreateDatasetForm.tsx
📝 Walkthrough

Walkthrough

Adds a MAX_NAME_LENGTH constant (64) to app/lib/constants.ts and applies it as a maxLength prop on the name fields in CreateDatasetForm and RunEvaluationForm. Both forms' success banners receive overflow/truncation CSS fixes. The gpt-5.4-pro model is removed from MODEL_OPTIONS.

Changes

Name constraints, UI overflow fixes, and model list cleanup

Layer / File(s) Summary
MAX_NAME_LENGTH constant
app/lib/constants.ts
Exports MAX_NAME_LENGTH = 64 after DEFAULT_PAGE_LIMIT.
Name field enforcement and success banner overflow
app/components/evaluations/CreateDatasetForm.tsx, app/components/evaluations/RunEvaluationForm.tsx
Both forms import MAX_NAME_LENGTH and pass it to their "Name *" Field. CreateDatasetForm success panel adds truncate + title tooltip on the filename and cursor-pointer shrink-0 on the remove button. RunEvaluationForm success panel gains overflow-hidden on its container with inner min-width/width and wrap-anywhere text constraints.
Remove gpt-5.4-pro model option
app/lib/models.ts
Deletes the "gpt-5.4-pro" entry from the openai array in MODEL_OPTIONS; all other model entries and filtering logic are unchanged.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Possibly related PRs

  • ProjectTech4DevAI/kaapi-frontend#87: Previously added GPT-5.4 model variants (including pro) to MODEL_OPTIONS and introduced isGpt5Model; this PR reverses the pro variant addition.

Suggested labels

enhancement

Suggested reviewers

  • AkhileshNegi
  • vprashrex

Poem

🐇 A rabbit tidied the field one day,
Snipped "gpt-5.4-pro" right away,
Capped the name at sixty-four,
Truncated filenames overflow no more,
Now the forms are neat — hooray! 🎉

🚥 Pre-merge checks | ✅ 4 | ❌ 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 (4 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately summarizes the two main changes: removing the GPT-5.4-Pro model and implementing UI updates related to name length restrictions.
Linked Issues check ✅ Passed All objectives from issue #214 are met: GPT-5.4-Pro model removed from config and max-length restrictions added to Name fields in both Dataset and Evals forms.
Out of Scope Changes check ✅ Passed All changes are directly related to issue #214 objectives: model removal, max-length constant definition, and UI adjustments for long names.
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 unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/ui-updates

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🧹 Nitpick comments (1)
app/components/evaluations/RunEvaluationForm.tsx (1)

122-141: ⚡ Quick win

Consider adding a tooltip for consistency with CreateDatasetForm.

The dataset name uses wrap-anywhere to prevent layout breaking, but unlike the file preview in CreateDatasetForm (lines 176-179), it lacks a title attribute. Adding title={selectedDataset.dataset_name} on line 126 would provide a tooltip for users to see the full name and maintain consistency across both forms.

📋 Suggested enhancement for consistency
             <div className="flex-1 min-w-0">
-              <div className="text-sm font-medium text-text-primary wrap-anywhere">
+              <div
+                className="text-sm font-medium text-text-primary wrap-anywhere"
+                title={selectedDataset.dataset_name}
+              >
                 {selectedDataset.dataset_name}
               </div>
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@app/components/evaluations/RunEvaluationForm.tsx` around lines 122 - 141, The
dataset name display in RunEvaluationForm lacks a tooltip attribute for
consistency with CreateDatasetForm. Add a title attribute with the value
selectedDataset.dataset_name to the div element that displays the dataset name
(the one with className containing "text-sm font-medium text-text-primary
wrap-anywhere"). This will provide users with a tooltip showing the full dataset
name and maintain consistent UX patterns across both forms.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Nitpick comments:
In `@app/components/evaluations/RunEvaluationForm.tsx`:
- Around line 122-141: The dataset name display in RunEvaluationForm lacks a
tooltip attribute for consistency with CreateDatasetForm. Add a title attribute
with the value selectedDataset.dataset_name to the div element that displays the
dataset name (the one with className containing "text-sm font-medium
text-text-primary wrap-anywhere"). This will provide users with a tooltip
showing the full dataset name and maintain consistent UX patterns across both
forms.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 3361a0e0-63d2-4c32-9b57-0a6cdff94675

📥 Commits

Reviewing files that changed from the base of the PR and between ad24900 and 8746f93.

📒 Files selected for processing (4)
  • app/components/evaluations/CreateDatasetForm.tsx
  • app/components/evaluations/RunEvaluationForm.tsx
  • app/lib/constants.ts
  • app/lib/models.ts
💤 Files with no reviewable changes (1)
  • app/lib/models.ts

@Ayush8923 Ayush8923 merged commit d521c7b into main Jun 16, 2026
2 checks passed
@Ayush8923 Ayush8923 deleted the feat/ui-updates branch June 16, 2026 15:05
@github-actions

Copy link
Copy Markdown

🎉 This PR is included in version 0.2.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@github-actions

Copy link
Copy Markdown

🎉 This PR is included in version 0.3.0-main.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Configuration: Remove costly model

2 participants