Skip to content

Format AIML representative model list#967

Open
PeterDaveHello wants to merge 1 commit intoChatGPTBox-dev:masterfrom
PeterDaveHello:fix-aiml-representative-model-format
Open

Format AIML representative model list#967
PeterDaveHello wants to merge 1 commit intoChatGPTBox-dev:masterfrom
PeterDaveHello:fix-aiml-representative-model-format

Conversation

@PeterDaveHello
Copy link
Copy Markdown
Member

@PeterDaveHello PeterDaveHello commented May 1, 2026

Collapse the AIML representative model list to Prettier's single-line form so later commits do not pick up unrelated formatting churn.

The drift came from 0eb64f7, which refreshed the AIML model IDs but left the older multiline layout from ea210a9.

Reference:

  • 0eb64f7 Refresh AIML default model presets
  • ea210a9 Expand OpenAI model regression coverage

Open in Devin Review

Summary by CodeRabbit

Release Notes

  • Tests
    • Reformatted test fixture for improved code consistency.

Collapse the AIML representative model list to Prettier's single-line
form so later commits do not pick up unrelated formatting churn.

The drift came from 0eb64f7, which refreshed the AIML model IDs but
left the older multiline layout from ea210a9.

Reference:
- 0eb64f7 Refresh AIML default model presets
- ea210a9 Expand OpenAI model regression coverage
@PeterDaveHello PeterDaveHello requested a review from Copilot May 1, 2026 15:53
@qodo-code-review
Copy link
Copy Markdown
Contributor

Review Summary by Qodo

Format AIML representative model list to single-line

Grey Divider

Walkthroughs

Description
• Collapse AIML representative model list to single-line format
• Align with Prettier's formatting standard
• Prevent unrelated formatting churn in future commits
Diagram
flowchart LR
  multiline["Multiline AIML model list"] -- "collapse to single-line" --> singleline["Single-line AIML model list"]
Loading

Grey Divider

File Changes

1. tests/unit/config/config-predicates.test.mjs Formatting +1/-4

Reformat AIML model names array to single-line

• Reformatted representativeAimlApiModelNames array from multiline to single-line format
• Maintains identical array content with two model identifiers
• Aligns with Prettier's code formatting standards

tests/unit/config/config-predicates.test.mjs


Grey Divider

Qodo Logo

@qodo-code-review
Copy link
Copy Markdown
Contributor

qodo-code-review Bot commented May 1, 2026

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Grey Divider

Qodo Logo

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 1, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 233f91c6-27bf-4d7b-a22c-b5019fc8aaf6

📥 Commits

Reviewing files that changed from the base of the PR and between c236a4b and a0120f3.

📒 Files selected for processing (1)
  • tests/unit/config/config-predicates.test.mjs

📝 Walkthrough

Walkthrough

A test fixture in the config predicates test file was reformatted from a multi-line array declaration to a single-line format. No logic, assertions, or functionality was altered.

Changes

Cohort / File(s) Summary
Test Fixture Formatting
tests/unit/config/config-predicates.test.mjs
Reformatted representativeAimlApiModelNames array from multi-line to single-line declaration while preserving identical content.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~1 minute

Poem

🐰 A line grew long, so we made it neat,
One string per row became one row complete,
The test stays strong, no logic bent,
Just formatting cleaner—time well-spent! ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the primary change: reformatting a test fixture array from multi-line to single-line format for the AIML representative model list.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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
Review rate limit: 7/8 reviews remaining, refill in 7 minutes and 30 seconds.

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

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request reformats the representativeAimlApiModelNames array in tests/unit/config/config-predicates.test.mjs from a multi-line to a single-line declaration. I have no feedback to provide as there are no review comments.

Copy link
Copy Markdown

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Open in Devin Review

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Normalizes formatting of the AIML representative model list in the config predicate unit tests to match the repo’s Prettier output, reducing future formatting-only churn.

Changes:

  • Reformat representativeAimlApiModelNames from a multiline array to a single-line array (fits within configured printWidth: 100).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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