feat: code polish on classifier

[yuluo-yx]

1. optimize semantic-router/util/classification

[netlify]

✅ Deploy Preview for vllm-semantic-router ready!

Name	Link
🔨 Latest commit	6b8adde
🔍 Latest deploy log	https://app.netlify.com/projects/vllm-semantic-router/deploys/68bc30f42372050008df00c0
😎 Deploy Preview	https://deploy-preview-67--vllm-semantic-router.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify project configuration.

[github-actions]

👥 vLLM Semantic Team Notification

The following members have been identified for the changed files in this PR and have been automatically assigned:

📁 src

Owners: @rootfs, @Xunzhuo, @wangchen615
Files changed:

• src/semantic-router/pkg/utils/classification/classifier.go
• src/semantic-router/pkg/utils/classification/classifier_test.go

---

This comment was automatically generated based on the OWNER files in the repository.

[Xunzhuo]

can you give it a simple conclusion of what you optimized for classifier?

[yuluo-yx]

can you give it a simple conclusion of what you optimized for classifier?

sure,

1. i delete the contains func, which the go slices library includes.
2. extracted the forEachModelScore func because the line for _, modelScore := range cat.ModelScores { appears four times in the file, which is not concise.
3. extracted updateBestModel func.

[Xunzhuo]

ok thanks! can you separate the ut to another pr which is easier to go ahead, and remember to sign your commit

[Xunzhuo]

and add ut around your code polish in this PR to make sure things go in expectations.

[yuluo-yx]

and add ut around your code polish in this PR to make sure things go in expectations.

hi I split ttft ut to #68, pls