Check for NULL instead of matching a numeric pattern

The crash this fixes is a widget with no position values, so a NULL check
is enough to skip those rows. The numeric pattern also skipped values that
are not base-10 integers, which the original statement did not do.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

Author: ekanshul

migrations/versions/db0aca1ebd32_12_column_dashboard_layout.py

@@ -17,26 +17,26 @@
 
 
 def upgrade():
-    # Skip widgets whose position values are missing or not numeric, e.g. rows
-    # written through the API with options = '{}'. Without the filter the cast
-    # yields NULL, jsonb_set() returns NULL for a NULL new_value, and the update
-    # fails the NOT NULL constraint on widgets.options.
+    # Skip widgets that have no position values, e.g. rows written through the
+    # API with options = '{}'. Without the filter the cast yields NULL,
+    # jsonb_set() returns NULL for a NULL new_value, and the update fails the
+    # NOT NULL constraint on widgets.options.
     op.execute("""
     UPDATE widgets
     SET options = jsonb_set(options, '{position,col}', to_json((options->'position'->>'col')::int * 2)::jsonb)
-    WHERE options->'position'->>'col' ~ '^-?[0-9]+$';
+    WHERE options->'position'->>'col' IS NOT NULL;
     UPDATE widgets
     SET options = jsonb_set(options, '{position,sizeX}', to_json((options->'position'->>'sizeX')::int * 2)::jsonb)
-    WHERE options->'position'->>'sizeX' ~ '^-?[0-9]+$';
+    WHERE options->'position'->>'sizeX' IS NOT NULL;
     """)
 
 
 def downgrade():
     op.execute("""
     UPDATE widgets
     SET options = jsonb_set(options, '{position,col}', to_json((options->'position'->>'col')::int / 2)::jsonb)
-    WHERE options->'position'->>'col' ~ '^-?[0-9]+$';
+    WHERE options->'position'->>'col' IS NOT NULL;
     UPDATE widgets
     SET options = jsonb_set(options, '{position,sizeX}', to_json((options->'position'->>'sizeX')::int / 2)::jsonb)
-    WHERE options->'position'->>'sizeX' ~ '^-?[0-9]+$';
+    WHERE options->'position'->>'sizeX' IS NOT NULL;
     """)