progress

Author: psteinroe

crates/pgls_diagnostics_categories/src/categories.rs

@@ -92,9 +92,6 @@ define_categories! {
     "lint",
     "lint/performance",
     "lint/safety",
-    // splinter groups start
-    "dblint",
-    "dblint/splinter",
-    // splinter groups end
     // Lint groups end
-}
+}
+

packages/@postgres-language-server/backend-jsonrpc/src/workspace.ts

@@ -95,6 +95,28 @@ export type Category =
 	| "lint/safety/requireConcurrentIndexDeletion"
 	| "lint/safety/runningStatementWhileHoldingAccessExclusive"
 	| "lint/safety/transactionNesting"
+	| "dblint/splinter/authRlsInitplan"
+	| "dblint/splinter/authUsersExposed"
+	| "dblint/splinter/duplicateIndex"
+	| "dblint/splinter/extensionInPublic"
+	| "dblint/splinter/extensionVersionsOutdated"
+	| "dblint/splinter/fkeyToAuthUnique"
+	| "dblint/splinter/foreignTableInApi"
+	| "dblint/splinter/functionSearchPathMutable"
+	| "dblint/splinter/insecureQueueExposedInApi"
+	| "dblint/splinter/materializedViewInApi"
+	| "dblint/splinter/multiplePermissivePolicies"
+	| "dblint/splinter/noPrimaryKey"
+	| "dblint/splinter/policyExistsRlsDisabled"
+	| "dblint/splinter/rlsDisabledInPublic"
+	| "dblint/splinter/rlsEnabledNoPolicy"
+	| "dblint/splinter/rlsReferencesUserMetadata"
+	| "dblint/splinter/securityDefinerView"
+	| "dblint/splinter/tableBloat"
+	| "dblint/splinter/unindexedForeignKeys"
+	| "dblint/splinter/unknown"
+	| "dblint/splinter/unsupportedRegTypes"
+	| "dblint/splinter/unusedIndex"
 	| "stdin"
 	| "check"
 	| "configuration"
@@ -111,7 +133,9 @@ export type Category =
 	| "dummy"
 	| "lint"
 	| "lint/performance"
-	| "lint/safety";
+	| "lint/safety"
+	| "dblint"
+	| "dblint/splinter";
 export interface Location {
 	path?: Resource_for_String;
 	sourceCode?: string;

packages/@postgrestools/backend-jsonrpc/src/workspace.ts

@@ -95,6 +95,28 @@ export type Category =
 	| "lint/safety/requireConcurrentIndexDeletion"
 	| "lint/safety/runningStatementWhileHoldingAccessExclusive"
 	| "lint/safety/transactionNesting"
+	| "dblint/splinter/authRlsInitplan"
+	| "dblint/splinter/authUsersExposed"
+	| "dblint/splinter/duplicateIndex"
+	| "dblint/splinter/extensionInPublic"
+	| "dblint/splinter/extensionVersionsOutdated"
+	| "dblint/splinter/fkeyToAuthUnique"
+	| "dblint/splinter/foreignTableInApi"
+	| "dblint/splinter/functionSearchPathMutable"
+	| "dblint/splinter/insecureQueueExposedInApi"
+	| "dblint/splinter/materializedViewInApi"
+	| "dblint/splinter/multiplePermissivePolicies"
+	| "dblint/splinter/noPrimaryKey"
+	| "dblint/splinter/policyExistsRlsDisabled"
+	| "dblint/splinter/rlsDisabledInPublic"
+	| "dblint/splinter/rlsEnabledNoPolicy"
+	| "dblint/splinter/rlsReferencesUserMetadata"
+	| "dblint/splinter/securityDefinerView"
+	| "dblint/splinter/tableBloat"
+	| "dblint/splinter/unindexedForeignKeys"
+	| "dblint/splinter/unknown"
+	| "dblint/splinter/unsupportedRegTypes"
+	| "dblint/splinter/unusedIndex"
 	| "stdin"
 	| "check"
 	| "configuration"
@@ -111,7 +133,9 @@ export type Category =
 	| "dummy"
 	| "lint"
 	| "lint/performance"
-	| "lint/safety";
+	| "lint/safety"
+	| "dblint"
+	| "dblint/splinter";
 export interface Location {
 	path?: Resource_for_String;
 	sourceCode?: string;

xtask/codegen/src/generate_splinter.rs

@@ -128,15 +128,6 @@ fn update_categories_file(rules: BTreeMap<String, RuleInfo>) -> Result<()> {
         &format!("\n{}\n    ", splinter_entries),
     )?;
 
-    // Replace content between splinter groups markers
-    let groups_start = "// splinter groups start";
-    let groups_end = "// splinter groups end";
-
-    let groups_content = "\n    \"dblint\",\n    \"dblint/splinter\",\n    ";
-
-    let new_content =
-        replace_between_markers(&new_content, groups_start, groups_end, groups_content)?;
-
     fs2::write(categories_path, new_content)?;
 
     Ok(())