Merge pull request #56 from DEVxNetwork/sam/fix-55

Fix invalid regex with hyphens in ranges

Author: samholmes

app/setup/page.tsx

@@ -117,7 +117,7 @@ export default function Setup() {
 			}
 
 			// Validate handle format: lowercase alphanumeric with underscores/hyphens, 3-30 chars
-			const handleRegex = /^[a-z0-9_-]+$/
+			const handleRegex = /^(?:[a-z0-9_]|-)+$/
 			if (!handleRegex.test(handle) || handle.length < 3 || handle.length > 30) {
 				setHandleAvailable(false)
 				return
@@ -305,7 +305,7 @@ export default function Setup() {
 									onChange={(e) => setHandle(e.target.value.toLowerCase())}
 									placeholder="your-handle"
 									required
-									pattern="[a-z0-9_-]{3,30}"
+									pattern="(?:[a-z0-9_]|-){3,30}"
 									minLength={3}
 									maxLength={30}
 								/>

scripts/update-events.ts

@@ -563,7 +563,7 @@ async function scrapeEventPage(
 
 					if (normalizedText) {
 						domLocationText = normalizedText
-						const cityStateRegex = /([A-Za-z][A-Za-z\s.'-]+?),\s*([A-Z]{2})(?:\s|,|$)/
+						const cityStateRegex = /([A-Za-z](?:[A-Za-z\s.']|-)+?),\s*([A-Z]{2})(?:\s|,|$)/
 						const cityStateMatch = normalizedText.match(cityStateRegex)
 
 						if (cityStateMatch) {

supabase/migrations/20250122000016_update_handle_regex_pattern.sql

@@ -0,0 +1,14 @@
+-- Update handle format constraint to use clearer alternation pattern
+-- This avoids ambiguity with hyphens in character classes
+ALTER TABLE profiles
+DROP CONSTRAINT IF EXISTS handle_format_check;
+
+ALTER TABLE profiles
+ADD CONSTRAINT handle_format_check CHECK (
+	handle IS NULL OR (
+		handle ~ '^(?:[a-z0-9_]|-)+$' AND
+		LENGTH(handle) >= 3 AND
+		LENGTH(handle) <= 30
+	)
+);
+