From 8c4d69d0ae40ebb737eb8a89c38380192ca98fbe Mon Sep 17 00:00:00 2001
From: Steve Dignam <steve@dignam.xyz>
Date: Fri, 17 Oct 2025 22:35:00 -0400
Subject: [PATCH] linter: false positive with constraint-missing-not-valid

rel: https://github.com/sbdchd/squawk/issues/693
---
 .../src/rules/constraint_missing_not_valid.rs    | 16 ++++++++++++++++
 1 file changed, 16 insertions(+)

diff --git a/crates/squawk_linter/src/rules/constraint_missing_not_valid.rs b/crates/squawk_linter/src/rules/constraint_missing_not_valid.rs
index cc687035..b422d625 100644
--- a/crates/squawk_linter/src/rules/constraint_missing_not_valid.rs
+++ b/crates/squawk_linter/src/rules/constraint_missing_not_valid.rs
@@ -126,6 +126,13 @@ pub(crate) fn constraint_missing_not_valid(ctx: &mut Linter, parse: &Parse<Sourc
                                 continue;
                             }
                         }
+                        if let Some(ast::Constraint::PrimaryKeyConstraint(pk)) =
+                            add_constraint.constraint()
+                        {
+                            if pk.using_index().is_some() {
+                                continue;
+                            }
+                        }
 
                         ctx.report(Violation::for_node(
                             Rule::ConstraintMissingNotValid,
@@ -206,6 +213,15 @@ ALTER TABLE distributors VALIDATE CONSTRAINT distfk;
         assert_eq!(errors.len(), 0);
     }
 
+    #[test]
+    fn adding_using_index_ok() {
+        let sql = r#"
+ALTER TABLE account ADD CONSTRAINT account_pk PRIMARY KEY USING INDEX account_pk_idx;
+        "#;
+        let errors = lint(sql, Rule::ConstraintMissingNotValid);
+        assert_eq!(errors.len(), 0);
+    }
+
     #[test]
     fn adding_check_constraint_err() {
         let sql = r#"