Skip to content

Commit 6fae041

Browse files
psteinroeclaude
andcommitted
fix: correct AnalyserRules reference in codegen
Changed pgls_analyse::AnalyserRules to pgls_analyser::LinterRules and fixed import statements to use the correct crate for RuleOptions. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
1 parent dc098c0 commit 6fae041

File tree

4 files changed

+7
-6
lines changed

4 files changed

+7
-6
lines changed

crates/pgls_analyser/src/registry.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ use pgls_analyse::{RegistryVisitor, RuleKey};
55
pub fn visit_registry<V: RegistryVisitor>(registry: &mut V) {
66
registry.record_category::<crate::lint::Lint>();
77
}
8-
#[doc = r" Maps rule keys to rule executors"]
8+
#[doc = r" Maps rule keys to rule executors (zero-cost abstraction)"]
99
#[doc = r" This function is generated by codegen and includes all linter rules"]
1010
pub fn get_linter_rule_executor(key: &RuleKey) -> Option<RegistryLinterRule> {
1111
match key . rule_name () { "addSerialColumn" => Some (crate :: linter_registry :: RegistryLinterRule :: new :: < crate :: lint :: safety :: add_serial_column :: AddSerialColumn > ()) , "addingFieldWithDefault" => Some (crate :: linter_registry :: RegistryLinterRule :: new :: < crate :: lint :: safety :: adding_field_with_default :: AddingFieldWithDefault > ()) , "addingForeignKeyConstraint" => Some (crate :: linter_registry :: RegistryLinterRule :: new :: < crate :: lint :: safety :: adding_foreign_key_constraint :: AddingForeignKeyConstraint > ()) , "addingNotNullField" => Some (crate :: linter_registry :: RegistryLinterRule :: new :: < crate :: lint :: safety :: adding_not_null_field :: AddingNotNullField > ()) , "addingPrimaryKeyConstraint" => Some (crate :: linter_registry :: RegistryLinterRule :: new :: < crate :: lint :: safety :: adding_primary_key_constraint :: AddingPrimaryKeyConstraint > ()) , "addingRequiredField" => Some (crate :: linter_registry :: RegistryLinterRule :: new :: < crate :: lint :: safety :: adding_required_field :: AddingRequiredField > ()) , "banCharField" => Some (crate :: linter_registry :: RegistryLinterRule :: new :: < crate :: lint :: safety :: ban_char_field :: BanCharField > ()) , "banConcurrentIndexCreationInTransaction" => Some (crate :: linter_registry :: RegistryLinterRule :: new :: < crate :: lint :: safety :: ban_concurrent_index_creation_in_transaction :: BanConcurrentIndexCreationInTransaction > ()) , "banDropColumn" => Some (crate :: linter_registry :: RegistryLinterRule :: new :: < crate :: lint :: safety :: ban_drop_column :: BanDropColumn > ()) , "banDropDatabase" => Some (crate :: linter_registry :: RegistryLinterRule :: new :: < crate :: lint :: safety :: ban_drop_database :: BanDropDatabase > ()) , "banDropNotNull" => Some (crate :: linter_registry :: RegistryLinterRule :: new :: < crate :: lint :: safety :: ban_drop_not_null :: BanDropNotNull > ()) , "banDropTable" => Some (crate :: linter_registry :: RegistryLinterRule :: new :: < crate :: lint :: safety :: ban_drop_table :: BanDropTable > ()) , "banTruncateCascade" => Some (crate :: linter_registry :: RegistryLinterRule :: new :: < crate :: lint :: safety :: ban_truncate_cascade :: BanTruncateCascade > ()) , "changingColumnType" => Some (crate :: linter_registry :: RegistryLinterRule :: new :: < crate :: lint :: safety :: changing_column_type :: ChangingColumnType > ()) , "constraintMissingNotValid" => Some (crate :: linter_registry :: RegistryLinterRule :: new :: < crate :: lint :: safety :: constraint_missing_not_valid :: ConstraintMissingNotValid > ()) , "creatingEnum" => Some (crate :: linter_registry :: RegistryLinterRule :: new :: < crate :: lint :: safety :: creating_enum :: CreatingEnum > ()) , "disallowUniqueConstraint" => Some (crate :: linter_registry :: RegistryLinterRule :: new :: < crate :: lint :: safety :: disallow_unique_constraint :: DisallowUniqueConstraint > ()) , "lockTimeoutWarning" => Some (crate :: linter_registry :: RegistryLinterRule :: new :: < crate :: lint :: safety :: lock_timeout_warning :: LockTimeoutWarning > ()) , "multipleAlterTable" => Some (crate :: linter_registry :: RegistryLinterRule :: new :: < crate :: lint :: safety :: multiple_alter_table :: MultipleAlterTable > ()) , "preferBigInt" => Some (crate :: linter_registry :: RegistryLinterRule :: new :: < crate :: lint :: safety :: prefer_big_int :: PreferBigInt > ()) , "preferBigintOverInt" => Some (crate :: linter_registry :: RegistryLinterRule :: new :: < crate :: lint :: safety :: prefer_bigint_over_int :: PreferBigintOverInt > ()) , "preferBigintOverSmallint" => Some (crate :: linter_registry :: RegistryLinterRule :: new :: < crate :: lint :: safety :: prefer_bigint_over_smallint :: PreferBigintOverSmallint > ()) , "preferIdentity" => Some (crate :: linter_registry :: RegistryLinterRule :: new :: < crate :: lint :: safety :: prefer_identity :: PreferIdentity > ()) , "preferJsonb" => Some (crate :: linter_registry :: RegistryLinterRule :: new :: < crate :: lint :: safety :: prefer_jsonb :: PreferJsonb > ()) , "preferRobustStmts" => Some (crate :: linter_registry :: RegistryLinterRule :: new :: < crate :: lint :: safety :: prefer_robust_stmts :: PreferRobustStmts > ()) , "preferTextField" => Some (crate :: linter_registry :: RegistryLinterRule :: new :: < crate :: lint :: safety :: prefer_text_field :: PreferTextField > ()) , "preferTimestamptz" => Some (crate :: linter_registry :: RegistryLinterRule :: new :: < crate :: lint :: safety :: prefer_timestamptz :: PreferTimestamptz > ()) , "renamingColumn" => Some (crate :: linter_registry :: RegistryLinterRule :: new :: < crate :: lint :: safety :: renaming_column :: RenamingColumn > ()) , "renamingTable" => Some (crate :: linter_registry :: RegistryLinterRule :: new :: < crate :: lint :: safety :: renaming_table :: RenamingTable > ()) , "requireConcurrentIndexCreation" => Some (crate :: linter_registry :: RegistryLinterRule :: new :: < crate :: lint :: safety :: require_concurrent_index_creation :: RequireConcurrentIndexCreation > ()) , "requireConcurrentIndexDeletion" => Some (crate :: linter_registry :: RegistryLinterRule :: new :: < crate :: lint :: safety :: require_concurrent_index_deletion :: RequireConcurrentIndexDeletion > ()) , "runningStatementWhileHoldingAccessExclusive" => Some (crate :: linter_registry :: RegistryLinterRule :: new :: < crate :: lint :: safety :: running_statement_while_holding_access_exclusive :: RunningStatementWhileHoldingAccessExclusive > ()) , "transactionNesting" => Some (crate :: linter_registry :: RegistryLinterRule :: new :: < crate :: lint :: safety :: transaction_nesting :: TransactionNesting > ()) , _ => None , }

crates/pgls_configuration/src/linter/rules.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -908,7 +908,7 @@ impl Safety {
908908
pub fn push_to_analyser_rules(
909909
rules: &Rules,
910910
metadata: &pgls_analyse::MetadataRegistry,
911-
analyser_rules: &mut pgls_analyse::AnalyserRules,
911+
analyser_rules: &mut pgls_analyser::LinterRules,
912912
) {
913913
if let Some(rules) = rules.safety.as_ref() {
914914
for rule_name in Safety::GROUP_RULES {

crates/pgls_configuration/src/rules/configuration.rs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
use biome_deserialize::Merge;
22
use biome_deserialize_macros::Deserializable;
3-
use pgls_analyse::RuleFilter;
43
use pgls_analyser::RuleOptions;
54
use pgls_diagnostics::Severity;
65
#[cfg(feature = "schema")]

xtask/codegen/src/generate_configuration.rs

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -287,7 +287,8 @@ fn generate_lint_rules_file(
287287
288288
use crate::rules::{RuleConfiguration, RulePlainConfiguration};
289289
use biome_deserialize_macros::Merge;
290-
use pgls_analyse::{RuleFilter, options::RuleOptions};
290+
use pgls_analyse::RuleFilter;
291+
use pgls_analyser::RuleOptions;
291292
use pgls_diagnostics::{Category, Severity};
292293
use rustc_hash::FxHashSet;
293294
#[cfg(feature = "schema")]
@@ -426,7 +427,7 @@ fn generate_lint_rules_file(
426427
pub fn push_to_analyser_rules(
427428
rules: &Rules,
428429
metadata: &pgls_analyse::MetadataRegistry,
429-
analyser_rules: &mut pgls_analyse::AnalyserRules,
430+
analyser_rules: &mut pgls_analyser::LinterRules,
430431
) {
431432
#(
432433
if let Some(rules) = rules.#group_idents.as_ref() {
@@ -789,7 +790,8 @@ fn generate_action_actions_file(
789790
790791
use crate::rules::{RuleAssistConfiguration, RuleAssistPlainConfiguration};
791792
use biome_deserialize_macros::{Deserializable, Merge};
792-
use pgls_analyse::{RuleFilter, options::RuleOptions};
793+
use pgls_analyse::RuleFilter;
794+
use pgls_analyser::RuleOptions;
793795
use pgls_diagnostics::{Category, Severity};
794796
use rustc_hash::FxHashSet;
795797
#[cfg(feature = "schema")]

0 commit comments

Comments
 (0)