Skip to content

fix(evaluators)!: remove Luna2 evaluator#234

Merged
namrataghadi-galileo merged 2 commits into
mainfrom
feature/66366-remove-luna2
Jun 5, 2026
Merged

fix(evaluators)!: remove Luna2 evaluator#234
namrataghadi-galileo merged 2 commits into
mainfrom
feature/66366-remove-luna2

Conversation

@namrataghadi-galileo
Copy link
Copy Markdown
Contributor

@namrataghadi-galileo namrataghadi-galileo commented Jun 3, 2026

Remove legacy Luna2 evaluator and only support Luna evaluator with new API.

@namrataghadi-galileo namrataghadi-galileo changed the title Remove Luna2 evaluator fix(evaluator): Remove Luna2 evaluator Jun 3, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented Jun 3, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@namrataghadi-galileo namrataghadi-galileo changed the title fix(evaluator): Remove Luna2 evaluator fix(evaluators): Remove Luna2 evaluator Jun 3, 2026
@namrataghadi-galileo
Copy link
Copy Markdown
Contributor Author

namrataghadi-galileo commented Jun 4, 2026

Screenshot 2026-06-03 at 6 14 59 PM

@abhinav-galileo
Copy link
Copy Markdown
Collaborator

abhinav-galileo commented Jun 5, 2026

The galileo.luna create flow should seed the same defaults as the form. Right now it falls through to {} in DEFAULT_EVALUATOR_CONFIGS, so fromConfig({}) opens with a blank threshold even though the default operator is gte. Please add a galileo.luna default config or make fromConfig fall back to initialValues.

@abhinav-galileo
Copy link
Copy Markdown
Collaborator

abhinav-galileo commented Jun 5, 2026

Since galileo.luna2 is a shipped evaluator id, this hard removal should be treated as a breaking release. Please mark the merge/release commit accordingly (for example fix(evaluators)!: remove Luna2 evaluator or a BREAKING CHANGE: footer) and include migration guidance, such as migrating to galileo.luna or pinning agent-control-evaluator-galileo <8.

@namrataghadi-galileo namrataghadi-galileo changed the title fix(evaluators): Remove Luna2 evaluator fix(evaluators)!: remove Luna2 evaluator Jun 5, 2026
@namrataghadi-galileo
BREAKING CHANGE: galileo.luna2 has been removed. Existing controls us…
b21797b 
…ing galileo.luna2 should migrate to galileo.luna and update evaluator config to the direct Luna scorer fields. Pin agent-control-evaluator-galileo <8 to keep using the legacy evaluator.
@namrataghadi-galileo namrataghadi-galileo merged commit ccc4fc6 into main Jun 5, 2026
6 checks passed
@namrataghadi-galileo namrataghadi-galileo deleted the feature/66366-remove-luna2 branch June 5, 2026 12:26
galileo-automation pushed a commit that referenced this pull request Jun 5, 2026
@semantic-release-bot
chore(release): ts sdk v3.0.0 [skip ci]
09c5a54 
## [3.0.0](ts-sdk-v2.6.0...ts-sdk-v3.0.0) (2026-06-05)

### ⚠ BREAKING CHANGES

* **evaluators:** remove Luna2 evaluator (#234)

### Bug Fixes

* **evaluators:** remove Luna2 evaluator ([#234](#234)) ([ccc4fc6](ccc4fc6))
* **server:** harden auth upstream connection handling ([#235](#235)) ([16d0115](16d0115))
@galileo-automation
Copy link
Copy Markdown
Collaborator

galileo-automation commented Jun 5, 2026

🎉 This PR is included in version 3.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Reviewers

@abhinav-galileo abhinav-galileo abhinav-galileo approved these changes

Assignees

No one assigned

Labels

released

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@namrataghadi-galileo @abhinav-galileo @galileo-automation