Skip to content

🐛 fix: Allow dot (.) in rule names#28

Merged
heumsi merged 1 commit intomainfrom
fix/allow-dot-in-rule-names
Mar 31, 2026
Merged

🐛 fix: Allow dot (.) in rule names#28
heumsi merged 1 commit intomainfrom
fix/allow-dot-in-rule-names

Conversation

@heumsi
Copy link
Copy Markdown
Owner

@heumsi heumsi commented Mar 31, 2026

Summary

  • Expand rule name validation regex from [a-zA-Z0-9_-]+ to [a-zA-Z0-9_.-]+ to allow dot-separated rule names like shared.domain
  • Update error message and documentation
  • Add tests for dot-containing rule names

Closes #27

Test plan

  • All existing tests pass
  • Added tests for dot-containing rule names (shared.domain, context.adapters.inbound)

🤖 Generated with Claude Code

Expand rule name validation regex from [a-zA-Z0-9_-]+ to [a-zA-Z0-9_.-]+
so users can use module-path-style names like `shared.domain`.

Closes #27

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@heumsi heumsi merged commit 13c5cd4 into main Mar 31, 2026
5 checks passed
@heumsi heumsi deleted the fix/allow-dot-in-rule-names branch March 31, 2026 17:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Allow dot (.) in rule names

1 participant