fix: change z.enum(toolNames) to z.string() in model-settings

[kiloconnect]

Changes

This PR updates the model settings schema to use z.string() instead of z.enum(toolNames) for tool name validation.

What changed:

• Replaced z.enum(toolNames) with z.string() for included_tools and excluded_tools fields
• Removed unused toolNames import from @roo-code/types

Why:

This change allows for more flexible tool name validation, removing the strict enum constraint and allowing any string value for tool names.

Files modified:

• src/api/providers/kilocode/model-settings.ts

[changeset-bot]

⚠️ No Changeset found

Latest commit: 46359f4

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[kiloconnect]

✅ No Issues Found

1 file reviewed | Confidence: 95% | Recommendation: Merge

Review Details

Files: src/api/providers/kilocode/model-settings.ts

Checked: Security, bugs, type compatibility, schema consistency

Analysis:

• The change from z.enum(toolNames) to z.string() aligns ModelSettingsSchema with the existing ModelInfo type in packages/types/src/model.ts, which already defines includedTools and excludedTools as z.array(z.string())
• The parseModelSettings function correctly maps these fields to Partial<ModelInfo>, so type compatibility is maintained
• Removing the unused toolNames import is appropriate cleanup
• This is a valid relaxation of validation that provides more flexibility for tool name configuration

[chrarnoldus]

This is necessary for tool aliases to be accepted.