Skip to content

Catalog: clef 27b and pplx decider decision models - #184

Merged
sanchitmonga22 merged 2 commits into
mainfrom
siddhesh/decision-models-more
Oct 8, 2026
Merged

sanchitmonga22 merged 2 commits into
mainfrom
siddhesh/decision-models-more

Conversation

@Siddhesh2377

@Siddhesh2377 Siddhesh2377 commented Oct 8, 2026 •

Copy link
Copy Markdown
Collaborator

Adds three decision checkpoints with original upstream links: Clef 27B GGUF Q4, Clef 27B MLX 4bit, and pplx decider 27B MLX 4bit. Sizes verified against the Hub.

Made with Cursor


Summary by cubic

Adds three decision-model checkpoints to the catalog: Clef 27B GGUF Q4_K_M, Clef 27B MLX 4-bit, and pplx decider v1.1 27B MLX 4-bit, with file sizes verified against the Hugging Face Hub. Golden test expectations are regenerated to include the new models.

  • The Clef and pplx decider entries use placeholder url: None and rely on the file lists to select files, matching the existing Flash row pattern.
  • merge_key values clef-27b and pplx-decider-27b let the two Clef variants share a merge slot.

Written for commit 3bcfd56. Summary will update on new commits.

View guided diff Turn on auto-fix

Summary by CodeRabbit

  • New Features
    • Added Clef 27B decision models in GGUF and MLX formats, along with the Pplx Decider 27B model in MLX format, to the catalog. These options are now available for users browsing supported decision models.

Co-authored-by: Cursor <cursoragent@cursor.com>
@Siddhesh2377
Siddhesh2377 requested a review from a team as a code owner October 8, 2026 17:06
@coderabbitai

coderabbitai Bot commented Oct 8, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: aad75082-cc30-4449-b0d9-4fea8c704817
📥 Commits

Reviewing files that changed from the base of the PR and between 557446e and 3bcfd56.

📒 Files selected for processing (2)
  • tests/golden/expected/local__models__list__--all.stdout
  • tests/golden/expected/local__models__list__--all__--json.stdout

Included review availability: This review used your included allowance. Your plan provides up to 4 included reviews per hour; 3 remain after this review.


📝 Walkthrough

Walkthrough

The catalog adds file inventories and decision-category entries for Clef 27B GGUF, Clef 27B MLX, and Pplx Decider 27B MLX. The expected text and JSON model-list outputs include the new models.

Changes

Decision model catalog

Layer / File(s) Summary
Decision model file inventories
src/catalog/catalog.rs
Adds file inventories for the Clef GGUF and MLX variants and the Pplx Decider MLX variant. The inventories specify required and optional files.
Decision catalog entries and model-list expectations
src/catalog/catalog.rs, tests/golden/expected/local__models__list__--all.stdout, tests/golden/expected/local__models__list__--all__--json.stdout
Adds decision entries with model sizes and merge keys. The Clef variants share clef-27b; Pplx uses pplx-decider-27b. Updates the expected text and JSON model lists with the new entries.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Feature

Suggested reviewers: sanchitmonga22

Merge Risk: ⚪ Minimal · up to 3bcfd

This change adds three decision models to the catalog and updates the expected model-list output. No merge-blocking risk was found.

🚥 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 clearly and concisely identifies the main change: adding Clef 27B and Pplx Decider decision models to the catalog.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 1…
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.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

Co-authored-by: Cursor <cursoragent@cursor.com>
@Siddhesh2377
Siddhesh2377 enabled auto-merge (squash) October 8, 2026 18:28
@sanchitmonga22 sanchitmonga22 added the release:minor Auto-tag a minor product release on merge to main label Oct 8, 2026
@sanchitmonga22
sanchitmonga22 merged commit 2d6aa70 into main Oct 8, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release:minor Auto-tag a minor product release on merge to main

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants