Support case-insensitive model filter patterns#1725
Conversation
|
CI note: the initial checks failed before repository commands ran. The lint and test jobs failed while downloading
|
5afaae6 to
c53391a
Compare
Greptile SummaryThis PR adds
Confidence Score: 5/5Safe to merge — the change is a focused, well-scoped fix with no regressions on existing patterns. The refactor correctly replaces bare new RegExp(val) validation with isValidModelPattern, which translates the modifier into a proper JS i flag before compiling. All existing matching semantics for non-modifier patterns are preserved and the implementation is consistent across validation, matching, locale strings, and backend tests. No files require special attention. Important Files Changed
Reviews (3): Last reviewed commit: "Support case-insensitive model filter pa..." | Re-trigger Greptile |
c53391a to
f59cd71
Compare
|
有点冲突了 |
f59cd71 to
5b30302
Compare
Co-authored-by: userZ <spare.nets-0y@icloud.com>
Summary
(?i)in channel model filter preview and validation.Tests
go test ./internal/pkg/xregexppnpm exec tsc --noEmit --pretty falsegit diff --checkCloses #1724