squawk-cli 2.54.0__tar.gz → 2.56.0__tar.gz

{squawk_cli-2.54.0 → squawk_cli-2.56.0}/Cargo.lock

@@ -34,12 +34,13 @@ checksum = "683d7910e743518b0e34f1186f92494becacb047c7b6bf616c96772180fef923"
 
 [[package]]
 name = "annotate-snippets"
-version = "0.12.4"
+version = "0.12.16"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "a8ee2f071d418442e50c643c4e7a4051ce3abd9dba11713cc6cdf4f4a3f3cca5"
+checksum = "f211a51805bc641f3ad5b7664c77d2547af685cc33b4cd8d31964027a46f13f1"
 dependencies = [
  "anstyle",
- "unicode-width 0.2.1",
+ "memchr",
+ "unicode-width 0.2.2",
 ]
 
 [[package]]
@@ -59,9 +60,9 @@ dependencies = [
 
 [[package]]
 name = "anstyle"
-version = "1.0.11"
+version = "1.0.14"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "862ed96ca487e809f1c8e5a8447f6ee2cf102f846893800b20cebdf541fc6bbd"
+checksum = "940b3a0ca603d1eade50a4846a2afffd5ef57a9feac2c0e2ec2e14f9ead76000"
 
 [[package]]
 name = "anstyle-lossy"
@@ -100,7 +101,7 @@ dependencies = [
  "anstyle-lossy",
  "anstyle-parse",
  "html-escape",
- "unicode-width 0.2.1",
+ "unicode-width 0.2.2",
 ]
 
 [[package]]
@@ -1399,9 +1400,9 @@ dependencies = [
 
 [[package]]
 name = "memchr"
-version = "2.7.5"
+version = "2.8.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "32a282da65faaf38286cf3be983213fcf1d2e2a58700e808f83f4ea9a4804bc0"
+checksum = "f8ca58f447f06ed17d5fc4043ce1b10dd205e060fb3ce5b979b8ed8e59ff3f79"
 
 [[package]]
 name = "memoffset"
@@ -1679,7 +1680,7 @@ checksum = "d2b0f8def1f117e13c895f3eda65a7b5650688da29d6ad04635f61bc7b92eebd"
 dependencies = [
  "bytecount",
  "fnv",
- "unicode-width 0.2.1",
+ "unicode-width 0.2.2",
 ]
 
 [[package]]
@@ -2434,7 +2435,7 @@ dependencies = [
 
 [[package]]
 name = "squawk"
-version = "2.54.0"
+version = "2.56.0"
 dependencies = [
  "annotate-snippets",
  "anyhow",
@@ -2465,7 +2466,7 @@ dependencies = [
 
 [[package]]
 name = "squawk-fmt"
-version = "2.54.0"
+version = "2.56.0"
 dependencies = [
  "anyhow",
  "camino",
@@ -2481,7 +2482,7 @@ dependencies = [
 
 [[package]]
 name = "squawk-github"
-version = "2.54.0"
+version = "2.56.0"
 dependencies = [
  "jsonwebtoken",
  "log",
@@ -2492,7 +2493,7 @@ dependencies = [
 
 [[package]]
 name = "squawk-ide"
-version = "2.54.0"
+version = "2.56.0"
 dependencies = [
  "annotate-snippets",
  "etcetera",
@@ -2514,14 +2515,14 @@ dependencies = [
 
 [[package]]
 name = "squawk-lexer"
-version = "2.54.0"
+version = "2.56.0"
 dependencies = [
  "insta",
 ]
 
 [[package]]
 name = "squawk-linter"
-version = "2.54.0"
+version = "2.56.0"
 dependencies = [
  "annotate-snippets",
  "enum-iterator",
@@ -2536,7 +2537,7 @@ dependencies = [
 
 [[package]]
 name = "squawk-parser"
-version = "2.54.0"
+version = "2.56.0"
 dependencies = [
  "annotate-snippets",
  "camino",
@@ -2549,7 +2550,7 @@ dependencies = [
 
 [[package]]
 name = "squawk-server"
-version = "2.54.0"
+version = "2.56.0"
 dependencies = [
  "anyhow",
  "crossbeam-channel",
@@ -2575,7 +2576,7 @@ dependencies = [
 
 [[package]]
 name = "squawk-syntax"
-version = "2.54.0"
+version = "2.56.0"
 dependencies = [
  "annotate-snippets",
  "camino",
@@ -2589,7 +2590,7 @@ dependencies = [
 
 [[package]]
 name = "squawk-thread"
-version = "2.54.0"
+version = "2.56.0"
 dependencies = [
  "crossbeam-channel",
  "crossbeam-utils",
@@ -2601,7 +2602,7 @@ dependencies = [
 
 [[package]]
 name = "squawk-wasm"
-version = "2.54.0"
+version = "2.56.0"
 dependencies = [
  "console_error_panic_hook",
  "console_log",
@@ -2829,9 +2830,9 @@ dependencies = [
 
 [[package]]
 name = "tiny_pretty"
-version = "0.2.1"
+version = "0.4.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "650d82e943da333637be9f1567d33d605e76810a26464edfd7ae74f7ef181e95"
+checksum = "01ddaca72c626914165b5a0faf5fde225949dad80e52ff4c9f12ec5226fc36cc"
 
 [[package]]
 name = "tinystr"
@@ -2967,9 +2968,9 @@ checksum = "7dd6e30e90baa6f72411720665d41d89b9a3d039dc45b8faea1ddd07f617f6af"
 
 [[package]]
 name = "unicode-width"
-version = "0.2.1"
+version = "0.2.2"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "4a1a07cc7db3810833284e8d372ccdc6da29741639ecc70c9ec107df0fa6154c"
+checksum = "b4ac048d71ede7ee76d585517add45da530660ef4390e49b098733c6e897f254"
 
 [[package]]
 name = "url"
@@ -3477,7 +3478,7 @@ checksum = "32ac00cd3f8ec9c1d33fb3e7958a82df6989c42d747bd326c822b1d625283547"
 
 [[package]]
 name = "xtask"
-version = "2.54.0"
+version = "2.56.0"
 dependencies = [
  "anyhow",
  "camino",

{squawk_cli-2.54.0 → squawk_cli-2.56.0}/Cargo.toml

@@ -3,7 +3,7 @@ members = ["crates/*"]
 resolver = "2"
 
 [workspace.package]
-version = "2.54.0"
+version = "2.56.0"
 edition = "2024"
 rust-version = "1.94"
 authors = ["Squawk Team & Contributors"]
@@ -35,7 +35,7 @@ serde_repr = "0.1"
 regex = "1.11.1"
 simplelog = "0.12.0"
 tempfile = "3.21.0"
-tiny_pretty = "0.2.1"
+tiny_pretty = "0.4.0"
 toml = "0.5.9"
 dir-test = "0.4"
 drop_bomb = "0.1.5"
@@ -55,7 +55,7 @@ wasm-bindgen-test = "0.3.50"
 web-sys = "0.3.91"
 console_error_panic_hook = "0.1.7"
 console_log = "1.0.0"
-annotate-snippets = "0.12.4"
+annotate-snippets = "0.12.16"
 anyhow = "1.0.94"
 either = "1.15.0"
 convert_case = "0.7.1"
@@ -79,14 +79,14 @@ rustc-hash = "2.1.1"
 
 # local
 # we have to make the versions explicit otherwise `cargo publish` won't work
-squawk-github = { path = "./crates/squawk_github", version = "2.54.0" }
-squawk-ide = { path = "./crates/squawk_ide", version = "2.54.0" }
-squawk-lexer = { path = "./crates/squawk_lexer", version = "2.54.0" }
-squawk-parser = { path = "./crates/squawk_parser", version = "2.54.0" }
-squawk-syntax = { path = "./crates/squawk_syntax", version = "2.54.0" }
-squawk-linter = { path = "./crates/squawk_linter", version = "2.54.0" }
-squawk-server = { path = "./crates/squawk_server", version = "2.54.0" }
-squawk-thread = { path = "./crates/squawk_thread", version = "2.54.0" }
+squawk-github = { path = "./crates/squawk_github", version = "2.56.0" }
+squawk-ide = { path = "./crates/squawk_ide", version = "2.56.0" }
+squawk-lexer = { path = "./crates/squawk_lexer", version = "2.56.0" }
+squawk-parser = { path = "./crates/squawk_parser", version = "2.56.0" }
+squawk-syntax = { path = "./crates/squawk_syntax", version = "2.56.0" }
+squawk-linter = { path = "./crates/squawk_linter", version = "2.56.0" }
+squawk-server = { path = "./crates/squawk_server", version = "2.56.0" }
+squawk-thread = { path = "./crates/squawk_thread", version = "2.56.0" }
 
 [workspace.lints.clippy]
 collapsible_else_if = "allow"

{squawk_cli-2.54.0 → squawk_cli-2.56.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: squawk-cli
-Version: 2.54.0
+Version: 2.56.0
 Classifier: License :: OSI Approved :: Apache Software License
 Classifier: License :: OSI Approved :: MIT License
 Classifier: Operating System :: MacOS

{squawk_cli-2.54.0 → squawk_cli-2.56.0}/crates/squawk/README.md

@@ -265,7 +265,7 @@ to your project's `.pre-commit-config.yaml`:
 ```yaml
 repos:
   - repo: https://github.com/sbdchd/squawk
-    rev: 2.54.0
+    rev: v2.56.0
     hooks:
       - id: squawk
         files: path/to/postgres/migrations/written/in/sql

{squawk_cli-2.54.0 → squawk_cli-2.56.0}/crates/squawk/src/reporter.rs

@@ -115,9 +115,11 @@ fn render_lint_error<W: std::io::Write>(
         .fold(true)
         .annotation(AnnotationKind::Primary.span(err.range.into()));
 
+    let rule_url = format!("https://squawkhq.com/docs/{}", err.rule_name);
     let mut group = level
         .primary_title(&err.message)
         .id(&err.rule_name)
+        .id_url(&rule_url)
         .element(snippet);
 
     if let Some(help) = &err.help {

{squawk_cli-2.54.0 → squawk_cli-2.56.0}/crates/squawk/src/snapshots/example.svg

@@ -22,7 +22,7 @@
   <rect width="100%" height="100%" y="0" rx="4.5" class="bg" />
 
   <text xml:space="preserve" class="container fg">
-    <tspan x="10px" y="28px"><tspan class="fg-yellow bold">warning[prefer-bigint-over-int]</tspan><tspan class="bold">: Using 32-bit integer fields can result in hitting the max `int` limit.</tspan>
+    <tspan x="10px" y="28px"><tspan class="fg-yellow bold">warning[</tspan><tspan class="fg-yellow bold"><a href="https://squawkhq.com/docs/prefer-bigint-over-int">prefer-bigint-over-int</a></tspan><tspan class="fg-yellow bold">]</tspan><tspan class="bold">: Using 32-bit integer fields can result in hitting the max `int` limit.</tspan>
 </tspan>
     <tspan x="10px" y="46px"><tspan> </tspan><tspan class="fg-bright-blue bold"> ╭▸ </tspan><tspan>../../example.sql:6:10</tspan>
 </tspan>
@@ -42,7 +42,7 @@
 </tspan>
     <tspan x="10px" y="190px"><tspan>  </tspan><tspan class="fg-bright-blue bold">╰╴</tspan><tspan>         </tspan><tspan class="fg-bright-green">+++</tspan>
 </tspan>
-    <tspan x="10px" y="208px"><tspan class="fg-yellow bold">warning[prefer-identity]</tspan><tspan class="bold">: Serial types make schema, dependency, and permission management difficult.</tspan>
+    <tspan x="10px" y="208px"><tspan class="fg-yellow bold">warning[</tspan><tspan class="fg-yellow bold"><a href="https://squawkhq.com/docs/prefer-identity">prefer-identity</a></tspan><tspan class="fg-yellow bold">]</tspan><tspan class="bold">: Serial types make schema, dependency, and permission management difficult.</tspan>
 </tspan>
     <tspan x="10px" y="226px"><tspan> </tspan><tspan class="fg-bright-blue bold"> ╭▸ </tspan><tspan>../../example.sql:6:10</tspan>
 </tspan>
@@ -64,7 +64,7 @@
 </tspan>
     <tspan x="10px" y="388px"><tspan>  </tspan><tspan class="fg-bright-blue bold">╰╴</tspan>
 </tspan>
-    <tspan x="10px" y="406px"><tspan class="fg-yellow bold">warning[prefer-text-field]</tspan><tspan class="bold">: Changing the size of a `varchar` field requires an `ACCESS EXCLUSIVE` lock, that will prevent all reads and writes to the table.</tspan>
+    <tspan x="10px" y="406px"><tspan class="fg-yellow bold">warning[</tspan><tspan class="fg-yellow bold"><a href="https://squawkhq.com/docs/prefer-text-field">prefer-text-field</a></tspan><tspan class="fg-yellow bold">]</tspan><tspan class="bold">: Changing the size of a `varchar` field requires an `ACCESS EXCLUSIVE` lock, that will prevent all reads and writes to the table.</tspan>
 </tspan>
     <tspan x="10px" y="424px"><tspan> </tspan><tspan class="fg-bright-blue bold"> ╭▸ </tspan><tspan>../../example.sql:7:13</tspan>
 </tspan>
@@ -86,7 +86,7 @@
 </tspan>
     <tspan x="10px" y="586px"><tspan>  </tspan><tspan class="fg-bright-blue bold">╰╴</tspan>
 </tspan>
-    <tspan x="10px" y="604px"><tspan class="fg-yellow bold">warning[require-concurrent-index-creation]</tspan><tspan class="bold">: During normal index creation, table updates are blocked, but reads are still allowed.</tspan>
+    <tspan x="10px" y="604px"><tspan class="fg-yellow bold">warning[</tspan><tspan class="fg-yellow bold"><a href="https://squawkhq.com/docs/require-concurrent-index-creation">require-concurrent-index-creation</a></tspan><tspan class="fg-yellow bold">]</tspan><tspan class="bold">: During normal index creation, table updates are blocked, but reads are still allowed.</tspan>
 </tspan>
     <tspan x="10px" y="622px"><tspan>  </tspan><tspan class="fg-bright-blue bold"> ╭▸ </tspan><tspan>../../example.sql:10:1</tspan>
 </tspan>
@@ -106,7 +106,7 @@
 </tspan>
     <tspan x="10px" y="766px"><tspan>   </tspan><tspan class="fg-bright-blue bold">╰╴</tspan><tspan>             </tspan><tspan class="fg-bright-green">++++++++++++</tspan>
 </tspan>
-    <tspan x="10px" y="784px"><tspan class="fg-yellow bold">warning[require-timeout-settings]</tspan><tspan class="bold">: Missing `set lock_timeout` before potentially slow operations</tspan>
+    <tspan x="10px" y="784px"><tspan class="fg-yellow bold">warning[</tspan><tspan class="fg-yellow bold"><a href="https://squawkhq.com/docs/require-timeout-settings">require-timeout-settings</a></tspan><tspan class="fg-yellow bold">]</tspan><tspan class="bold">: Missing `set lock_timeout` before potentially slow operations</tspan>
 </tspan>
     <tspan x="10px" y="802px"><tspan>  </tspan><tspan class="fg-bright-blue bold"> ╭▸ </tspan><tspan>../../example.sql:10:1</tspan>
 </tspan>
@@ -126,7 +126,7 @@
 </tspan>
     <tspan x="10px" y="946px"><tspan>   </tspan><tspan class="fg-bright-blue bold">╰╴</tspan>
 </tspan>
-    <tspan x="10px" y="964px"><tspan class="fg-yellow bold">warning[require-timeout-settings]</tspan><tspan class="bold">: Missing `set statement_timeout` before potentially slow operations</tspan>
+    <tspan x="10px" y="964px"><tspan class="fg-yellow bold">warning[</tspan><tspan class="fg-yellow bold"><a href="https://squawkhq.com/docs/require-timeout-settings">require-timeout-settings</a></tspan><tspan class="fg-yellow bold">]</tspan><tspan class="bold">: Missing `set statement_timeout` before potentially slow operations</tspan>
 </tspan>
     <tspan x="10px" y="982px"><tspan>  </tspan><tspan class="fg-bright-blue bold"> ╭▸ </tspan><tspan>../../example.sql:10:1</tspan>
 </tspan>
@@ -146,7 +146,7 @@
 </tspan>
     <tspan x="10px" y="1126px"><tspan>   </tspan><tspan class="fg-bright-blue bold">╰╴</tspan>
 </tspan>
-    <tspan x="10px" y="1144px"><tspan class="fg-yellow bold">warning[constraint-missing-not-valid]</tspan><tspan class="bold">: By default new constraints require a table scan and block writes to the table while that scan occurs.</tspan>
+    <tspan x="10px" y="1144px"><tspan class="fg-yellow bold">warning[</tspan><tspan class="fg-yellow bold"><a href="https://squawkhq.com/docs/constraint-missing-not-valid">constraint-missing-not-valid</a></tspan><tspan class="fg-yellow bold">]</tspan><tspan class="bold">: By default new constraints require a table scan and block writes to the table while that scan occurs.</tspan>
 </tspan>
     <tspan x="10px" y="1162px"><tspan>  </tspan><tspan class="fg-bright-blue bold"> ╭▸ </tspan><tspan>../../example.sql:12:24</tspan>
 </tspan>
@@ -160,7 +160,7 @@
 </tspan>
     <tspan x="10px" y="1252px"><tspan>   </tspan><tspan class="fg-bright-blue bold">╰ </tspan><tspan class="bold">help</tspan><tspan>: Use `NOT VALID` with a later `VALIDATE CONSTRAINT` call.</tspan>
 </tspan>
-    <tspan x="10px" y="1270px"><tspan class="fg-yellow bold">warning[disallowed-unique-constraint]</tspan><tspan class="bold">: Adding a `UNIQUE` constraint requires an `ACCESS EXCLUSIVE` lock which blocks reads and writes to the table while the index is built.</tspan>
+    <tspan x="10px" y="1270px"><tspan class="fg-yellow bold">warning[</tspan><tspan class="fg-yellow bold"><a href="https://squawkhq.com/docs/disallowed-unique-constraint">disallowed-unique-constraint</a></tspan><tspan class="fg-yellow bold">]</tspan><tspan class="bold">: Adding a `UNIQUE` constraint requires an `ACCESS EXCLUSIVE` lock which blocks reads and writes to the table while the index is built.</tspan>
 </tspan>
     <tspan x="10px" y="1288px"><tspan>  </tspan><tspan class="fg-bright-blue bold"> ╭▸ </tspan><tspan>../../example.sql:12:28</tspan>
 </tspan>

{squawk_cli-2.54.0 → squawk_cli-2.56.0}/crates/squawk/src/snapshots/squawk__github__test_github_comment__generating_comment_multiple_files.snap

@@ -17,7 +17,7 @@ SELECT 1;
 
 
 ```
-warning[adding-not-nullable-field]: Adding a NOT NULL field requires exclusive locks and table rewrites.
+warning[]8;;https://squawkhq.com/docs/adding-not-nullable-field\adding-not-nullable-field]8;;\]: Adding a NOT NULL field requires exclusive locks and table rewrites.
   ╭▸ alpha.sql:1:8
   │
 1 │ SELECT 1;

{squawk_cli-2.54.0 → squawk_cli-2.56.0}/crates/squawk/src/snapshots/squawk__reporter__test_reporter__display_violations_tty.snap

@@ -2,7 +2,7 @@
 source: crates/squawk/src/reporter.rs
 expression: "strip_ansi_codes(&String::from_utf8_lossy(&buff))"
 ---
-warning[require-timeout-settings]: Missing `set lock_timeout` before potentially slow operations
+warning[]8;;https://squawkhq.com/docs/require-timeout-settings\require-timeout-settings]8;;\]: Missing `set lock_timeout` before potentially slow operations
   ╭▸ main.sql:2:4
   │
 2 │    ALTER TABLE "core_recipe" ADD COLUMN "foo" integer NOT NULL;
@@ -12,7 +12,7 @@ warning[require-timeout-settings]: Missing `set lock_timeout` before potentially
   ╭╴
 2 +    set lock_timeout = '1s';
   ╰╴
-warning[require-timeout-settings]: Missing `set statement_timeout` before potentially slow operations
+warning[]8;;https://squawkhq.com/docs/require-timeout-settings\require-timeout-settings]8;;\]: Missing `set statement_timeout` before potentially slow operations
   ╭▸ main.sql:2:4
   │
 2 │    ALTER TABLE "core_recipe" ADD COLUMN "foo" integer NOT NULL;
@@ -22,14 +22,14 @@ warning[require-timeout-settings]: Missing `set statement_timeout` before potent
   ╭╴
 2 +    set statement_timeout = '5s';
   ╰╴
-warning[adding-required-field]: Adding a new column that is `NOT NULL` and has no default value to an existing table effectively makes it required.
+warning[]8;;https://squawkhq.com/docs/adding-required-field\adding-required-field]8;;\]: Adding a new column that is `NOT NULL` and has no default value to an existing table effectively makes it required.
   ╭▸ main.sql:2:30
   │
 2 │    ALTER TABLE "core_recipe" ADD COLUMN "foo" integer NOT NULL;
   │                              ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
   │
   ╰ help: Make the field nullable or add a non-VOLATILE DEFAULT
-warning[prefer-robust-stmts]: Missing `IF NOT EXISTS`, the migration can't be rerun if it fails part way through.
+warning[]8;;https://squawkhq.com/docs/prefer-robust-stmts\prefer-robust-stmts]8;;\]: Missing `IF NOT EXISTS`, the migration can't be rerun if it fails part way through.
   ╭▸ main.sql:2:30
   │
 2 │    ALTER TABLE "core_recipe" ADD COLUMN "foo" integer NOT NULL;
@@ -37,7 +37,7 @@ warning[prefer-robust-stmts]: Missing `IF NOT EXISTS`, the migration can't be re
   ╭╴
 2 │    ALTER TABLE "core_recipe" ADD COLUMN if not exists "foo" integer NOT NULL;
   ╰╴                                        +++++++++++++
-warning[prefer-bigint-over-int]: Using 32-bit integer fields can result in hitting the max `int` limit.
+warning[]8;;https://squawkhq.com/docs/prefer-bigint-over-int\prefer-bigint-over-int]8;;\]: Using 32-bit integer fields can result in hitting the max `int` limit.
   ╭▸ main.sql:2:47
   │
 2 │    ALTER TABLE "core_recipe" ADD COLUMN "foo" integer NOT NULL;
@@ -48,14 +48,14 @@ warning[prefer-bigint-over-int]: Using 32-bit integer fields can result in hitti
 2 -    ALTER TABLE "core_recipe" ADD COLUMN "foo" integer NOT NULL;
 2 +    ALTER TABLE "core_recipe" ADD COLUMN "foo" bigint NOT NULL;
   ╰╴
-warning[adding-required-field]: Adding a new column that is `NOT NULL` and has no default value to an existing table effectively makes it required.
+warning[]8;;https://squawkhq.com/docs/adding-required-field\adding-required-field]8;;\]: Adding a new column that is `NOT NULL` and has no default value to an existing table effectively makes it required.
   ╭▸ main.sql:3:24
   │
 3 │ ALTER TABLE "core_foo" ADD COLUMN "bar" integer NOT NULL;
   │                        ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
   │
   ╰ help: Make the field nullable or add a non-VOLATILE DEFAULT
-warning[prefer-robust-stmts]: Missing `IF NOT EXISTS`, the migration can't be rerun if it fails part way through.
+warning[]8;;https://squawkhq.com/docs/prefer-robust-stmts\prefer-robust-stmts]8;;\]: Missing `IF NOT EXISTS`, the migration can't be rerun if it fails part way through.
   ╭▸ main.sql:3:24
   │
 3 │ ALTER TABLE "core_foo" ADD COLUMN "bar" integer NOT NULL;
@@ -63,7 +63,7 @@ warning[prefer-robust-stmts]: Missing `IF NOT EXISTS`, the migration can't be re
   ╭╴
 3 │ ALTER TABLE "core_foo" ADD COLUMN if not exists "bar" integer NOT NULL;
   ╰╴                                  +++++++++++++
-warning[prefer-bigint-over-int]: Using 32-bit integer fields can result in hitting the max `int` limit.
+warning[]8;;https://squawkhq.com/docs/prefer-bigint-over-int\prefer-bigint-over-int]8;;\]: Using 32-bit integer fields can result in hitting the max `int` limit.
   ╭▸ main.sql:3:41
   │
 3 │ ALTER TABLE "core_foo" ADD COLUMN "bar" integer NOT NULL;

{squawk_cli-2.54.0 → squawk_cli-2.56.0}/crates/squawk/src/snapshots/squawk__reporter__test_reporter__display_violations_tty_and_github_annotations.snap

@@ -10,7 +10,7 @@ expression: "strip_ansi_codes(&String::from_utf8_lossy(&buff))"
 ::warning file=main.sql,line=5,col=2,endLine=6,endColumn=20,title=adding-required-field::Adding a new column that is `NOT NULL` and has no default value to an existing table effectively makes it required.
 ::warning file=main.sql,line=5,col=2,endLine=6,endColumn=20,title=prefer-robust-stmts::Missing `IF NOT EXISTS`, the migration can't be rerun if it fails part way through.
 ::warning file=main.sql,line=6,col=4,endLine=6,endColumn=11,title=prefer-bigint-over-int::Using 32-bit integer fields can result in hitting the max `int` limit.
-warning[require-timeout-settings]: Missing `set lock_timeout` before potentially slow operations
+warning[]8;;https://squawkhq.com/docs/require-timeout-settings\require-timeout-settings]8;;\]: Missing `set lock_timeout` before potentially slow operations
   ╭▸ main.sql:2:4
   │
 2 │    ALTER TABLE "core_recipe" ADD COLUMN "foo" integer NOT NULL;
@@ -20,7 +20,7 @@ warning[require-timeout-settings]: Missing `set lock_timeout` before potentially
   ╭╴
 2 +    set lock_timeout = '1s';
   ╰╴
-warning[require-timeout-settings]: Missing `set statement_timeout` before potentially slow operations
+warning[]8;;https://squawkhq.com/docs/require-timeout-settings\require-timeout-settings]8;;\]: Missing `set statement_timeout` before potentially slow operations
   ╭▸ main.sql:2:4
   │
 2 │    ALTER TABLE "core_recipe" ADD COLUMN "foo" integer NOT NULL;
@@ -30,14 +30,14 @@ warning[require-timeout-settings]: Missing `set statement_timeout` before potent
   ╭╴
 2 +    set statement_timeout = '5s';
   ╰╴
-warning[adding-required-field]: Adding a new column that is `NOT NULL` and has no default value to an existing table effectively makes it required.
+warning[]8;;https://squawkhq.com/docs/adding-required-field\adding-required-field]8;;\]: Adding a new column that is `NOT NULL` and has no default value to an existing table effectively makes it required.
   ╭▸ main.sql:2:30
   │
 2 │    ALTER TABLE "core_recipe" ADD COLUMN "foo" integer NOT NULL;
   │                              ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
   │
   ╰ help: Make the field nullable or add a non-VOLATILE DEFAULT
-warning[prefer-robust-stmts]: Missing `IF NOT EXISTS`, the migration can't be rerun if it fails part way through.
+warning[]8;;https://squawkhq.com/docs/prefer-robust-stmts\prefer-robust-stmts]8;;\]: Missing `IF NOT EXISTS`, the migration can't be rerun if it fails part way through.
   ╭▸ main.sql:2:30
   │
 2 │    ALTER TABLE "core_recipe" ADD COLUMN "foo" integer NOT NULL;
@@ -45,7 +45,7 @@ warning[prefer-robust-stmts]: Missing `IF NOT EXISTS`, the migration can't be re
   ╭╴
 2 │    ALTER TABLE "core_recipe" ADD COLUMN if not exists "foo" integer NOT NULL;
   ╰╴                                        +++++++++++++
-warning[prefer-bigint-over-int]: Using 32-bit integer fields can result in hitting the max `int` limit.
+warning[]8;;https://squawkhq.com/docs/prefer-bigint-over-int\prefer-bigint-over-int]8;;\]: Using 32-bit integer fields can result in hitting the max `int` limit.
   ╭▸ main.sql:2:47
   │
 2 │    ALTER TABLE "core_recipe" ADD COLUMN "foo" integer NOT NULL;
@@ -56,7 +56,7 @@ warning[prefer-bigint-over-int]: Using 32-bit integer fields can result in hitti
 2 -    ALTER TABLE "core_recipe" ADD COLUMN "foo" integer NOT NULL;
 2 +    ALTER TABLE "core_recipe" ADD COLUMN "foo" bigint NOT NULL;
   ╰╴
-warning[adding-required-field]: Adding a new column that is `NOT NULL` and has no default value to an existing table effectively makes it required.
+warning[]8;;https://squawkhq.com/docs/adding-required-field\adding-required-field]8;;\]: Adding a new column that is `NOT NULL` and has no default value to an existing table effectively makes it required.
   ╭▸ main.sql:5:3
   │
 5 │ ┏   ADD COLUMN "bar" 
@@ -64,7 +64,7 @@ warning[adding-required-field]: Adding a new column that is `NOT NULL` and has n
   │ ┗━━━━━━━━━━━━━━━━━━━━┛
   │
   ╰ help: Make the field nullable or add a non-VOLATILE DEFAULT
-warning[prefer-robust-stmts]: Missing `IF NOT EXISTS`, the migration can't be rerun if it fails part way through.
+warning[]8;;https://squawkhq.com/docs/prefer-robust-stmts\prefer-robust-stmts]8;;\]: Missing `IF NOT EXISTS`, the migration can't be rerun if it fails part way through.
   ╭▸ main.sql:5:3
   │
 5 │ ┏   ADD COLUMN "bar" 
@@ -73,7 +73,7 @@ warning[prefer-robust-stmts]: Missing `IF NOT EXISTS`, the migration can't be re
   ╭╴
 5 │   ADD COLUMN if not exists "bar" 
   ╰╴             +++++++++++++
-warning[prefer-bigint-over-int]: Using 32-bit integer fields can result in hitting the max `int` limit.
+warning[]8;;https://squawkhq.com/docs/prefer-bigint-over-int\prefer-bigint-over-int]8;;\]: Using 32-bit integer fields can result in hitting the max `int` limit.
   ╭▸ main.sql:6:5
   │
 6 │     integer NOT NULL;

{squawk_cli-2.54.0 → squawk_cli-2.56.0}/crates/squawk_github/src/app.rs

@@ -11,7 +11,7 @@ use serde_json::Value;
 use std::time::Duration;
 use std::time::{SystemTime, UNIX_EPOCH};
 
-pub(crate) const SQUAWK_USER_AGENT: &str = "squawk/2.54.0";
+pub(crate) const SQUAWK_USER_AGENT: &str = "squawk/2.56.0";
 
 #[derive(Debug, Serialize)]
 struct CommentBody {

{squawk_cli-2.54.0 → squawk_cli-2.56.0}/crates/squawk_ide/src/ast_nav.rs

@@ -13,28 +13,30 @@ pub(crate) fn find_cte_with_table(
     name_ref: &ast::NameRef,
     cte_name: &Name,
 ) -> Option<ast::WithTable> {
-    let with_clause = name_ref
+    let ref_start = name_ref.syntax().text_range().start();
+
+    for with_clause in name_ref
         .syntax()
         .ancestors()
-        .find_map(|query| ast::WithQuery::cast(query)?.with_clause())?;
-
-    let is_recursive = with_clause.recursive_token().is_some();
-    for with_table in with_clause.with_tables() {
-        if let Some(name) = with_table.name()
-            && Name::from_node(&name) == *cte_name
+        .filter_map(|query| ast::WithQuery::cast(query)?.with_clause())
+    {
+        let is_recursive = with_clause.recursive_token().is_some();
+        if let Some(with_table) = with_clause
+            .with_tables()
+            // Without RECURSIVE, only CTEs before the reference are visible.
+            .filter(|with_table| {
+                is_recursive || with_table.syntax().text_range().end() <= ref_start
+            })
+            .find(|with_table| {
+                with_table
+                    .name()
+                    .is_some_and(|name| Name::from_node(&name) == *cte_name)
+            })
         {
-            // Skip if we're inside this CTE's definition (CTE doesn't shadow itself)
-            if !is_recursive
-                && with_table
-                    .syntax()
-                    .text_range()
-                    .contains_range(name_ref.syntax().text_range())
-            {
-                continue;
-            }
             return Some(with_table);
         }
     }
+
     None
 }
 
@@ -205,6 +207,7 @@ pub(crate) enum ParentSouce {
     CreateTableAs(ast::CreateTableAs),
     CreateView(ast::CreateViewLike),
     ParenSelect(ast::ParenSelect),
+    SelectInto(ast::SelectInto),
     WithTable(ast::WithTable),
 }
 
@@ -237,6 +240,10 @@ pub(crate) fn parent_source(node: &SyntaxNode) -> Option<ParentSouce> {
         if let Some(create_table) = ast::CreateTableLike::cast(ancestor.clone()) {
             return Some(ParentSouce::CreateTable(create_table));
         }
+
+        if let Some(select_into) = ast::SelectInto::cast(ancestor.clone()) {
+            return Some(ParentSouce::SelectInto(select_into));
+        }
     }
 
     None

{squawk_cli-2.54.0 → squawk_cli-2.56.0}/crates/squawk_ide/src/classify.rs

@@ -724,6 +724,13 @@ pub(crate) fn classify_name_ref(node: &SyntaxNode) -> Option<NameRefClass> {
             // (anything in SELECT except FROM clause)
             return Some(NameRefClass::SelectColumn);
         }
+        if ast::CompoundSelect::can_cast(ancestor.kind())
+            && in_order_by_clause
+            && let Some(parent) = node.parent()
+            && ast::SortBy::can_cast(parent.kind())
+        {
+            return Some(NameRefClass::SelectOrderByAliasOrColumn);
+        }
         if ast::ColumnList::can_cast(ancestor.kind()) {
             in_column_list = true;
         }
@@ -848,6 +855,9 @@ pub(crate) fn classify_def_node(def_node: &SyntaxNode) -> Option<LocationKind> {
         if ast::CreateTableAs::can_cast(ancestor.kind()) {
             return Some(LocationKind::Table);
         }
+        if ast::SelectInto::can_cast(ancestor.kind()) {
+            return Some(LocationKind::Table);
+        }
         if ast::CreateIndex::can_cast(ancestor.kind()) {
             return Some(LocationKind::Index);
         }