fix: address review feedback - signature stripping, TABLE/VIEW separation, view column privileges

- Strip function/procedure argument lists before matching ignore patterns
  (fixes exact-name patterns like "dblink_connect_u" vs "dblink_connect_u(text)")
- Separate TABLE and VIEW into distinct switch cases to prevent cross-contamination
- Also filter column privileges on ignored views, not just tables
- Fix cleanup comment in test

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

Author: tianzhou

cmd/ignore_integration_test.go

@@ -1321,7 +1321,7 @@ GRANT SELECT ON users TO app_user;
 		}
 	})
 
-	// Clean up roles from embedded PG
+	// Clean up roles from sharedEmbeddedPG (plan subtest may create roles there)
 	cleanupStatements := []string{
 		"REASSIGN OWNED BY app_user TO testuser",
 		"DROP OWNED BY app_user",

ir/ignore.go

@@ -5,6 +5,15 @@ import (
 	"strings"
 )
 
+// stripFunctionArgs removes the argument list from a function/procedure signature.
+// e.g. "dblink_connect_u(text, text)" -> "dblink_connect_u"
+func stripFunctionArgs(name string) string {
+	if idx := strings.Index(name, "("); idx != -1 {
+		return name[:idx]
+	}
+	return name
+}
+
 // IgnoreConfig represents the configuration for ignoring database objects
 type IgnoreConfig struct {
 	Tables            []string `toml:"tables,omitempty"`
@@ -73,13 +82,14 @@ func (c *IgnoreConfig) ShouldIgnorePrivilegeByObjectType(objectName string, obje
 		return false
 	}
 	switch objectType {
-	case "TABLE", "VIEW":
-		// Views use table-level privileges in PostgreSQL
-		return c.shouldIgnore(objectName, c.Tables) || c.shouldIgnore(objectName, c.Views)
+	case "TABLE":
+		return c.shouldIgnore(objectName, c.Tables)
+	case "VIEW":
+		return c.shouldIgnore(objectName, c.Views)
 	case "FUNCTION":
-		return c.shouldIgnore(objectName, c.Functions)
+		return c.shouldIgnore(stripFunctionArgs(objectName), c.Functions)
 	case "PROCEDURE":
-		return c.shouldIgnore(objectName, c.Procedures)
+		return c.shouldIgnore(stripFunctionArgs(objectName), c.Procedures)
 	case "SEQUENCE":
 		return c.shouldIgnore(objectName, c.Sequences)
 	case "TYPE":

ir/inspector.go

@@ -2480,8 +2480,8 @@ func (i *Inspector) buildColumnPrivileges(ctx context.Context, schema *IR, targe
 			continue
 		}
 
-		// Skip column privileges for ignored tables
-		if i.ignoreConfig != nil && i.ignoreConfig.ShouldIgnoreTable(tableName) {
+		// Skip column privileges for ignored tables/views
+		if i.ignoreConfig != nil && (i.ignoreConfig.ShouldIgnoreTable(tableName) || i.ignoreConfig.ShouldIgnoreView(tableName)) {
 			continue
 		}