chore: merge main

Author: psteinroe

.github/workflows/pull_request.yml

@@ -272,6 +272,8 @@ jobs:
         run: cargo run -p xtask_codegen -- configuration
       - name: Run the bindings codegen
         run: cargo run -p xtask_codegen -- bindings
+      - name: Run the splinter codegen
+        run: cargo run -p xtask_codegen -- splinter
       - name: Run the docs codegen
         run: cargo run -p docs_codegen
       - name: Check for git diff -- run "just ready" if you see an error

.sqlx/query-02927e584e85871ba6f84c58e8b5e4454b2c36eaf034657d5d2d95633fb85bdb.json

@@ -144,6 +144,12 @@ cargo insta review
 
 ## Development Notes
 
+### Code Quality Guidelines
+**IMPORTANT**: Always run `cargo clippy --all-targets --all-features` and fix all warnings after making code changes. Clippy warnings must be resolved before committing code to maintain code quality standards.
+
+### Git Commit and PR Guidelines
+**IMPORTANT**: NEVER add "Claude" or any AI assistant name to commit messages or pull request descriptions. Commits and PRs should appear as authored by the human developer only.
+
 ### Code Generation
 Many parser structures are generated from PostgreSQL's protobuf definitions using procedural macros in `pgls_query_macros`. Run `just gen-lint` after modifying analyzer rules or configurations.
 

.sqlx/query-425fc6118e76cea42cf256b3b0f11046dc8b77d84c94314a0ed0716e5803df69.json

@@ -82,6 +82,7 @@ pgls_query                  = { path = "./crates/pgls_query", version = "0.0.0"
 pgls_query_ext              = { path = "./crates/pgls_query_ext", version = "0.0.0" }
 pgls_query_macros           = { path = "./crates/pgls_query_macros", version = "0.0.0" }
 pgls_schema_cache           = { path = "./crates/pgls_schema_cache", version = "0.0.0" }
+pgls_splinter               = { path = "./crates/pgls_splinter", version = "0.0.0" }
 pgls_statement_splitter     = { path = "./crates/pgls_statement_splitter", version = "0.0.0" }
 pgls_suppressions           = { path = "./crates/pgls_suppressions", version = "0.0.0" }
 pgls_text_edit              = { path = "./crates/pgls_text_edit", version = "0.0.0" }

.sqlx/query-b869d517301aaf69d382f092c09d5a53712d68afd273423f9310cd793586f532.json

@@ -12,6 +12,7 @@ Install: [instructions](https://pg-language-server.com/#installation)
 - [VSCode](https://marketplace.visualstudio.com/items?itemName=Supabase.postgrestools)
 - [Neovim](https://github.com/neovim/nvim-lspconfig/blob/master/doc/configs.md#postgres_lsp)
 - [Zed](https://github.com/LoamStudios/zed-postgres-language-server)
+- [Sublime Text](https://lsp.sublimetext.io/language_servers/#postgresql)
 
 ## Overview
 LSP Demo             |  CLI Demo

AGENTS.md

@@ -58,8 +58,7 @@ mod tests {
     use crate::{
         CompletionItem, CompletionItemKind, complete,
         test_helper::{
-            CompletionAssertion, assert_complete_results, assert_no_complete_results,
-            get_test_deps, get_test_params,
+            CompletionAssertion, assert_complete_results, get_test_deps, get_test_params,
         },
     };
 
@@ -717,18 +716,6 @@ mod tests {
             &pool,
         )
         .await;
-
-        // no completions in the values list!
-        assert_no_complete_results(
-            format!(
-                "insert into instruments (id, name) values ({})",
-                QueryWithCursorPosition::cursor_marker()
-            )
-            .as_str(),
-            None,
-            &pool,
-        )
-        .await;
     }
 
     #[sqlx::test(migrator = "pgls_test_utils::MIGRATIONS")]

Cargo.lock

@@ -1,4 +1,4 @@
-use pgls_text_size::{TextRange, TextSize};
+use pgls_text_size::TextRange;
 use pgls_treesitter::TreesitterContext;
 
 use crate::{is_sanitized_token_with_quote, remove_sanitized_token};
@@ -9,29 +9,25 @@ pub(crate) fn node_text_surrounded_by_quotes(ctx: &TreesitterContext) -> bool {
 }
 
 pub(crate) fn get_range_to_replace(ctx: &TreesitterContext) -> TextRange {
-    match ctx.node_under_cursor.as_ref() {
-        Some(node) => {
-            let content = ctx.get_node_under_cursor_content().unwrap_or("".into());
-            let content = content.as_str();
+    let node = &ctx.node_under_cursor;
+    let content = ctx.get_node_under_cursor_content().unwrap_or("".into());
+    let content = content.as_str();
 
-            let sanitized = remove_sanitized_token(content);
-            let length = sanitized.len();
+    let sanitized = remove_sanitized_token(content);
+    let length = sanitized.len();
 
-            let mut start = node.start_byte();
-            let mut end = start + length;
+    let mut start = node.start_byte();
+    let mut end = start + length;
 
-            if sanitized.starts_with('"') && sanitized.ends_with('"') {
-                start += 1;
+    if sanitized.starts_with('"') && sanitized.ends_with('"') {
+        start += 1;
 
-                if sanitized.len() > 1 {
-                    end -= 1;
-                }
-            }
-
-            TextRange::new(start.try_into().unwrap(), end.try_into().unwrap())
+        if sanitized.len() > 1 {
+            end -= 1;
         }
-        None => TextRange::empty(TextSize::new(0)),
     }
+
+    TextRange::new(start.try_into().unwrap(), end.try_into().unwrap())
 }
 
 pub(crate) fn only_leading_quote(ctx: &TreesitterContext) -> bool {
@@ -45,7 +41,7 @@ pub(crate) fn with_schema_or_alias(
     item_name: &str,
     schema_or_alias_name: Option<&str>,
 ) -> String {
-    let is_already_prefixed_with_schema_name = ctx.schema_or_alias_name.is_some();
+    let is_already_prefixed_with_schema_name = ctx.has_any_qualifier();
 
     let with_quotes = node_text_surrounded_by_quotes(ctx);
     let single_leading_quote = only_leading_quote(ctx);