tree-sitter-sh 0.15.0 → 0.16.0

package/Cargo.toml

@@ -1,30 +1,18 @@
 [package]
 name = "tree-sitter-sh"
-version = "0.15.0"
+version = "0.16.0"
 edition = "2024"
-description = "Tree-sitter grammar for POSIX sh."
-readme = "README.md"
-repository = "https://github.com/konomanoasa/tree-sitter-sh"
-license = "MIT"
-keywords = ["incremental", "parsing", "sh", "tree-sitter"]
-categories = ["parser-implementations", "parsing", "text-editors"]
 build = "bindings/rust/build.rs"
-include = [
-  "/LICENSE",
-  "/README.md",
-  "/bindings/rust/*.rs",
-  "/grammar.js",
-  "/queries/*.scm",
-  "/src/*.c",
-  "/src/*.h",
-  "/src/*.json",
-  "/src/tree_sitter/*.h",
-  "/tree-sitter.json",
-]
+links = "tree-sitter-sh"
+publish = false
 
 [lib]
 path = "bindings/rust/lib.rs"
 
+[[test]]
+name = "bindings"
+path = "test/bindings.test.rs"
+
 [dependencies]
 tree-sitter-language = "0.1.8"
 
@@ -32,4 +20,4 @@ tree-sitter-language = "0.1.8"
 tree-sitter = "0.27.0"
 
 [build-dependencies]
-cc = "1.4.6"
+cc = "1.4"

package/README.md

@@ -1,6 +1,6 @@
 # tree-sitter-sh
 
-[![CI](https://github.com/konomanoasa/tree-sitter-sh/actions/workflows/ci.yml/badge.svg)](https://github.com/konomanoasa/tree-sitter-sh/actions/workflows/ci.yml)
+[![CI](https://github.com/konomanoasa/tree-sitter-sh/actions/workflows/ci.yaml/badge.svg)](https://github.com/konomanoasa/tree-sitter-sh/actions/workflows/ci.yaml)
 [![npm](https://img.shields.io/npm/v/tree-sitter-sh)](https://www.npmjs.com/package/tree-sitter-sh)
 
 [Tree-sitter](https://tree-sitter.github.io/tree-sitter/) grammar for the
@@ -12,11 +12,20 @@ POSIX.1-2024 Shell Command Language.
 npm install tree-sitter-sh
 ```
 
+## Grammar
+
+| Grammar | Description | Rust constant |
+| --- | --- | --- |
+| `sh` | POSIX.1-2024 Shell Command Language | `LANGUAGE` |
+
 ## Development
 
+Development requires Node.js 24.2.0 or later.
+
 ```sh
 npm install
-npm run parse -- script.sh
+npm run build
+npm test
 ```
 
 ## Specification

package/bindings/rust/build.rs

@@ -1,9 +1,23 @@
+use std::{env, path::Path};
+
 fn main() {
-    println!("cargo::rerun-if-changed=src");
+  let mut build = cc::Build::new();
+  build.std("c17");
+  if build.get_compiler().is_like_msvc() {
+    build.flag("-utf-8");
+  }
+  if env::var("TARGET").expect("Cargo must provide TARGET")
+    == "wasm32-unknown-unknown"
+  {
+    let headers = env::var_os("DEP_TREE_SITTER_LANGUAGE_WASM_HEADERS")
+      .expect("tree-sitter-language must provide WebAssembly headers");
+    build.include(&headers);
+    println!("cargo::rerun-if-changed={}", Path::new(&headers).display());
+  }
 
-    cc::Build::new()
-        .std("c17")
-        .include("src")
-        .files(["src/parser.c", "src/scanner.c"])
-        .compile("tree-sitter-sh");
+  println!("cargo::rerun-if-changed=src");
+  build
+    .include("src")
+    .files(["src/parser.c", "src/scanner.c"])
+    .compile("tree-sitter-sh");
 }

package/bindings/rust/lib.rs

@@ -1,56 +1,12 @@
-//! POSIX.1-2024 Shell Command Language support for Tree-sitter.
-//!
-//! ```
-//! let mut parser = tree_sitter::Parser::new();
-//! parser.set_language(&tree_sitter_sh::LANGUAGE.into()).unwrap();
-//! let tree = parser.parse("echo hello\n", None).unwrap();
-//! assert!(!tree.root_node().has_error());
-//! ```
-
 use tree_sitter_language::LanguageFn;
 
 unsafe extern "C" {
-    fn tree_sitter_sh() -> *const ();
+  fn tree_sitter_sh() -> *const ();
 }
 
-/// Grammar entry point for `tree_sitter::Parser::set_language`.
-pub const LANGUAGE: LanguageFn = unsafe { LanguageFn::from_raw(tree_sitter_sh) };
+pub const LANGUAGE: LanguageFn =
+  unsafe { LanguageFn::from_raw(tree_sitter_sh) };
 
-/// Public node types and fields in Tree-sitter's JSON format.
 pub const NODE_TYPES: &str = include_str!("../../src/node-types.json");
 
-/// Highlight query using standard Tree-sitter and Neovim captures.
 pub const HIGHLIGHTS_QUERY: &str = include_str!("../../queries/highlights.scm");
-
-#[cfg(test)]
-mod tests {
-    use tree_sitter::{Parser, Query, QueryCursor, StreamingIterator};
-
-    #[test]
-    fn parses_quoted_here_document_and_highlights_its_terminator() {
-        let source = "cat <<'EOF'\n$HOME\nEOF\n";
-        let language = super::LANGUAGE.into();
-        let mut parser = Parser::new();
-        parser.set_language(&language).unwrap();
-        let tree = parser.parse(source, None).unwrap();
-        let root = tree.root_node();
-        assert_eq!(root.kind(), "program");
-        assert_eq!(root.byte_range(), 0..source.len());
-        assert!(!root.has_error(), "{}", root.to_sexp());
-
-        let query = Query::new(&language, super::HIGHLIGHTS_QUERY).unwrap();
-        let mut cursor = QueryCursor::new();
-        let mut captures = cursor.captures(&query, root, source.as_bytes());
-        let mut terminators = 0;
-        while let Some((matched, index)) = captures.next() {
-            let capture = matched.captures()[*index];
-            if capture.node.kind() == "here_document_end_text" {
-                assert_eq!(query.capture_names()[capture.index as usize], "label");
-                assert_eq!(capture.node.byte_range(), 18..21);
-                assert_eq!(capture.node.utf8_text(source.as_bytes()).unwrap(), "EOF");
-                terminators += 1;
-            }
-        }
-        assert_eq!(terminators, 1);
-    }
-}

package/grammar.js

@@ -76,16 +76,50 @@ const lexical = ($, begin) =>
 const wholeLexical = ($, begin, decorate = (source) => source) =>
   seq(optional(begin), decorate(sourceToken($, begin, "lexical", "whole")));
 
-const keywordSource = ($, name) =>
+// Alias the token directly so inserted MISSING tokens stay visible.
+const wholeSource = ($, begin, lexeme, visible) =>
   choice(
-    wholeLexical($, $[`_${name}_keyword_begin`], (source) =>
-      alias(source, $[`${name}_keyword`]),
-    ),
-    alias($[`_${name}_keyword_lexeme`], $[`${name}_keyword`]),
+    wholeLexical($, begin, (source) => alias(source, visible)),
+    alias(lexeme, visible),
+  );
+
+const KEYWORDS = [
+  "if",
+  "then",
+  "elif",
+  "else",
+  "fi",
+  "for",
+  "in",
+  "do",
+  "done",
+  "case",
+  "esac",
+  "while",
+  "until",
+];
+
+const keywordRules = () =>
+  Object.fromEntries(
+    KEYWORDS.flatMap((name) => [
+      [
+        `_${name}_keyword_lexeme`,
+        ($) => lexical($, $[`_${name}_keyword_begin`]),
+      ],
+      [
+        `_${name}_keyword_source`,
+        ($) =>
+          wholeSource(
+            $,
+            $[`_${name}_keyword_begin`],
+            $[`_${name}_keyword_lexeme`],
+            $[`${name}_keyword`],
+          ),
+      ],
+    ]),
   );
 
-// Each delimiter reduces to one hidden node instead of two stack entries, so
-// error recovery can still reach the enclosing command.
+// Reduce each delimiter to one stack entry so recovery can reach the command.
 const PHYSICAL_RULES = {
   _punct_left_parenthesis: "(",
   _punct_right_parenthesis: ")",
@@ -488,7 +522,7 @@ const parenthesizedArithmetic = ($, expression) =>
 const arithmeticExpansionStart = (
   $,
   marker,
-  start = $._command_or_arithmetic_substitution_start,
+  start = $._command_substitution_start,
 ) => seq(start, marker, physical($, $._punct_left_parenthesis, "("));
 const closedArithmeticExpansion = ($, start, expression, closing) =>
   seq(
@@ -569,6 +603,8 @@ const dollarSingleQuoted = ($, dollar) =>
     repeat(choice($.dollar_single_quote_text, $.dollar_single_quote_escape)),
     physical($, $._dollar_sq_close, "'"),
   );
+const hereDocumentEndSource = ($, begin, text) =>
+  seq(begin, repeat(choice($._here_document_end_leading_tabs, text)));
 const backquoteSubstitution = ($, start) =>
   seq(
     backquoteDelimiter($, start),
@@ -743,8 +779,7 @@ const conditionalThenBranch = ($, header, tail) =>
     tail,
   );
 
-// Only the consequence owns closing layout; a competing owner before fi
-// lets elif reduce its empty alternative before reaching else.
+// Layout before fi lets elif reduce its empty alternative before else.
 const ifClause = ($) =>
   conditionalThenBranch(
     $,
@@ -1159,31 +1194,7 @@ export default grammar({
 
     _or_if_lexeme: ($) => lexical($, $._or_if_begin),
 
-    _if_keyword_lexeme: ($) => lexical($, $._if_keyword_begin),
-
-    _then_keyword_lexeme: ($) => lexical($, $._then_keyword_begin),
-
-    _elif_keyword_lexeme: ($) => lexical($, $._elif_keyword_begin),
-
-    _else_keyword_lexeme: ($) => lexical($, $._else_keyword_begin),
-
-    _fi_keyword_lexeme: ($) => lexical($, $._fi_keyword_begin),
-
-    _for_keyword_lexeme: ($) => lexical($, $._for_keyword_begin),
-
-    _in_keyword_lexeme: ($) => lexical($, $._in_keyword_begin),
-
-    _do_keyword_lexeme: ($) => lexical($, $._do_keyword_begin),
-
-    _done_keyword_lexeme: ($) => lexical($, $._done_keyword_begin),
-
-    _case_keyword_lexeme: ($) => lexical($, $._case_keyword_begin),
-
-    _esac_keyword_lexeme: ($) => lexical($, $._esac_keyword_begin),
-
-    _while_keyword_lexeme: ($) => lexical($, $._while_keyword_begin),
-
-    _until_keyword_lexeme: ($) => lexical($, $._until_keyword_begin),
+    ...keywordRules(),
 
     _dsemi_lexeme: ($) => lexical($, $._dsemi_begin),
 
@@ -1407,19 +1418,6 @@ export default grammar({
     _and_if: ($) => alias($._and_if_lexeme, $.and_if),
     _or_if: ($) => alias($._or_if_lexeme, $.or_if),
     bang: ($) => lexical($, $._bang_begin),
-    _if_keyword_source: ($) => keywordSource($, "if"),
-    _then_keyword_source: ($) => keywordSource($, "then"),
-    _elif_keyword_source: ($) => keywordSource($, "elif"),
-    _else_keyword_source: ($) => keywordSource($, "else"),
-    _fi_keyword_source: ($) => keywordSource($, "fi"),
-    _for_keyword_source: ($) => keywordSource($, "for"),
-    _in_keyword_source: ($) => keywordSource($, "in"),
-    _do_keyword_source: ($) => keywordSource($, "do"),
-    _done_keyword_source: ($) => keywordSource($, "done"),
-    _case_keyword_source: ($) => keywordSource($, "case"),
-    _esac_keyword_source: ($) => keywordSource($, "esac"),
-    _while_keyword_source: ($) => keywordSource($, "while"),
-    _until_keyword_source: ($) => keywordSource($, "until"),
     function_definition: ($) =>
       seq(
         $._function_definition_header,
@@ -1429,8 +1427,6 @@ export default grammar({
         field("body", $.function_body),
       ),
 
-    // The header reduces before the body so its pieces do not stay on the
-    // stack while the body is parsed.
     _function_definition_header: ($) =>
       seq(
         field("name", $.fname),
@@ -1724,26 +1720,18 @@ export default grammar({
     here_document_end: ($) =>
       seq(
         choice(
-          seq(
+          hereDocumentEndSource(
+            $,
             $._quoted_here_document_end_begin,
-            repeat(
-              choice(
-                $._here_document_end_leading_tabs,
-                alias(
-                  $._quoted_here_document_end_text_lexeme,
-                  $.here_document_end_text,
-                ),
-              ),
+            alias(
+              $._quoted_here_document_end_text_lexeme,
+              $.here_document_end_text,
             ),
           ),
-          seq(
+          hereDocumentEndSource(
+            $,
             $._here_document_end_begin,
-            repeat(
-              choice(
-                $._here_document_end_leading_tabs,
-                $.here_document_end_text,
-              ),
-            ),
+            $.here_document_end_text,
           ),
         ),
         optional($._here_document_end_line_end),
@@ -1800,7 +1788,7 @@ export default grammar({
         ),
       ),
 
-    _assignment_non_delimiter_part: ($) =>
+    _assignment_word_part: ($) =>
       choice(
         $._assignment_bracket_fallback,
         $.pattern_bracket_source,
@@ -1809,7 +1797,6 @@ export default grammar({
         $.pattern_question_source,
         $._word_structured_part,
       ),
-    _assignment_word_part: ($) => $._assignment_non_delimiter_part,
 
     _assignment_literal: ($) => prec.right(assignmentPlainChunk($)),
 
@@ -1838,10 +1825,10 @@ export default grammar({
         $._word_tilde_end,
       ),
 
-    tilde_user: ($) => repeat1($._word_non_slash_part),
-    _assignment_tilde_user: ($) => repeat1($._assignment_non_delimiter_part),
-    _parameter_tilde_user: ($) => repeat1($._parameter_non_slash_part),
-    _word_non_slash_part: ($) =>
+    tilde_user: ($) => repeat1($._word_part),
+    _assignment_tilde_user: ($) => repeat1($._assignment_word_part),
+    _parameter_tilde_user: ($) => repeat1($._parameter_pattern_part),
+    _word_part: ($) =>
       choice(
         $._word_bracket_fallback,
         $.pattern_bracket_source,
@@ -1851,14 +1838,13 @@ export default grammar({
         $._word_structured_part,
       ),
 
-    _word_part: ($) => $._word_non_slash_part,
     _word_structured_part: ($) => choice(...structuredSourceParts($)),
 
     _word_initial_part: ($) =>
       choice(
         $._word_initial_bracket_fallback,
         alias($._word_initial_pattern_bracket, $.pattern_bracket_source),
-        alias($._word_initial_literal, $.literal),
+        $._word_initial_literal,
         alias($._word_initial_pattern_star, $.pattern_star_source),
         alias($._word_initial_pattern_question, $.pattern_question_source),
         alias($._word_initial_escaped_character, $.escaped_character),
@@ -1871,10 +1857,14 @@ export default grammar({
         alias($._word_initial_backquote_substitution, $.backquote_substitution),
       ),
     _word_initial_literal: ($) =>
-      choice(
-        lexical($, $._word_initial_literal_begin),
-        wholeLexical($, $._word_initial_literal_begin),
+      wholeSource(
+        $,
+        $._word_initial_literal_begin,
+        $._word_initial_literal_lexeme,
+        $.literal,
       ),
+    _word_initial_literal_lexeme: ($) =>
+      lexical($, $._word_initial_literal_begin),
     _word_initial_pattern_star: ($) =>
       lexical($, $._word_initial_pattern_star_begin),
     _word_initial_pattern_question: ($) =>
@@ -2256,7 +2246,7 @@ export default grammar({
 
     _parameter_tilde_source: ($) =>
       alias($._parameter_tilde_expansion, $.tilde_expansion),
-    _parameter_non_slash_part: ($) =>
+    _parameter_pattern_part: ($) =>
       choice(
         $._parameter_bracket_fallback,
         alias(
@@ -2268,8 +2258,6 @@ export default grammar({
         $.pattern_question_source,
         $._word_structured_part,
       ),
-    _parameter_pattern_part: ($) => $._parameter_non_slash_part,
-
     _parameter_pattern_literal: ($) => prec.right(parameterPlainChunk($)),
 
     _double_quoted_parameter_text: ($) =>
@@ -2278,12 +2266,10 @@ export default grammar({
     _double_quoted_parameter_escape: ($) =>
       lexical($, $._double_quoted_parameter_escape_begin),
     command_substitution: ($) =>
-      commandSubstitution($, $._command_or_arithmetic_substitution_start),
+      commandSubstitution($, $._command_substitution_start),
 
     _command_substitution_start: ($) =>
       dollarExpansionStart($, $._command_open, "("),
-    _command_or_arithmetic_substitution_start: ($) =>
-      $._command_substitution_start,
     command_substitution_body: ($) =>
       prec.left(substitutionCommandsBody($, $._closing_layout)),
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "tree-sitter-sh",
-  "version": "0.15.0",
+  "version": "0.16.0",
   "type": "module",
   "description": "Tree-sitter grammar for POSIX sh.",
   "license": "MIT",
@@ -14,14 +14,19 @@
     "src/*.h",
     "src/*.json",
     "src/tree_sitter/*.h",
+    "test/bindings.test.rs",
     "tree-sitter.json"
   ],
+  "publishConfig": {
+    "access": "public"
+  },
   "scripts": {
+    "build": "npm run tree-sitter -- build-all",
     "check": "biome check . && node scripts/check-c.js && node scripts/check-generated.js && npm test",
     "format": "biome check --write . && node scripts/check-c.js --write",
     "generate": "npm run tree-sitter -- generate-all",
-    "parse": "npm run tree-sitter -- parse --scope source.sh",
     "test": "npm run tree-sitter -- test-corpus --rebuild && node --test --test-concurrency=1 \"test/*.test.js\"",
+    "test:bindings": "cargo test --locked",
     "test:fuzz": "node --test --test-name-pattern=\"fixed-seed generated histories\" test/incremental.test.js",
     "test:fuzz:corpus": "npm run tree-sitter -- fuzz-all --iterations 100 --edits 5",
     "test:highlights": "node --test test/highlight.test.js",
@@ -32,7 +37,7 @@
     "tree-sitter": "node scripts/tree-sitter.js"
   },
   "devDependencies": {
-    "@biomejs/biome": "2.5.11",
+    "@biomejs/biome": "2.5.14",
     "@types/node": "26.5.1",
     "tree-sitter-cli": "0.27.0"
   },

package/src/arithmetic.h

@@ -1,6 +1,7 @@
 #ifndef TREE_SITTER_SH_ARITHMETIC_H_
 #define TREE_SITTER_SH_ARITHMETIC_H_
 
+#include "source.h"
 #include "tree_sitter/alloc.h"
 
 #include <stdbool.h>
@@ -110,31 +111,6 @@ struct DynamicArithmeticGraph {
   bool valid;
 };
 
-static void *dynamic_arithmetic_grow(
-  void *buffer,
-  size_t *capacity,
-  size_t length,
-  size_t width
-) {
-  if (length <= *capacity) {
-    return buffer;
-  }
-  size_t maximum = SIZE_MAX / width;
-  if (length > maximum) {
-    return NULL;
-  }
-  size_t next = *capacity == 0 ? 16 : *capacity;
-  while (next < length) {
-    next = next > maximum / 2 ? maximum : next * 2;
-  }
-  void *grown = ts_realloc(buffer, next * width);
-  if (grown == NULL) {
-    return NULL;
-  }
-  *capacity = next;
-  return grown;
-}
-
 static bool arithmetic_whitespace(int32_t character) {
   return character ==
     ' ' ||
@@ -147,25 +123,29 @@ static bool arithmetic_whitespace(int32_t character) {
     character == '\f';
 }
 
-static bool dynamic_arithmetic_name_start(int32_t character) {
+static bool is_decimal_digit(int32_t character) {
+  return character >= '0' && character <= '9';
+}
+
+static bool is_hexadecimal_digit(int32_t character) {
+  return is_decimal_digit(character) ||
+    (character >= 'A' && character <= 'F') ||
+    (character >= 'a' && character <= 'f');
+}
+
+static bool is_name_start_character(int32_t character) {
   return (character >= 'A' && character <= 'Z') ||
     (character >= 'a' && character <= 'z') ||
     character == '_';
 }
 
-static bool dynamic_arithmetic_digit(int32_t character) {
-  return character >= '0' && character <= '9';
-}
-
-static bool dynamic_arithmetic_hex(int32_t character) {
-  return dynamic_arithmetic_digit(character) ||
-    (character >= 'a' && character <= 'f') ||
-    (character >= 'A' && character <= 'F');
+static bool is_name_character(int32_t character) {
+  return is_name_start_character(character) || is_decimal_digit(character);
 }
 
 static int dynamic_arithmetic_lex(uint8_t state, int32_t character) {
-  bool name = dynamic_arithmetic_name_start(character);
-  bool digit = dynamic_arithmetic_digit(character);
+  bool name = is_name_start_character(character);
+  bool digit = is_decimal_digit(character);
   switch (state) {
   case DYNAMIC_LEX_START:
     if (arithmetic_whitespace(character)) {
@@ -236,10 +216,10 @@ static int dynamic_arithmetic_lex(uint8_t state, int32_t character) {
     }
     return name ? DYNAMIC_LEX_INVALID : DYNAMIC_LEX_STOP;
   case DYNAMIC_LEX_HEX_PREFIX:
-    return dynamic_arithmetic_hex(character) ? DYNAMIC_LEX_HEX
-                                             : DYNAMIC_LEX_INVALID;
+    return is_hexadecimal_digit(character) ? DYNAMIC_LEX_HEX
+                                           : DYNAMIC_LEX_INVALID;
   case DYNAMIC_LEX_HEX:
-    if (dynamic_arithmetic_hex(character)) {
+    if (is_hexadecimal_digit(character)) {
       return DYNAMIC_LEX_HEX;
     }
     return name || digit ? DYNAMIC_LEX_INVALID : DYNAMIC_LEX_STOP;
@@ -369,7 +349,7 @@ static bool dynamic_arithmetic_vertex(
     if (graph->vertex_count == SIZE_MAX) {
       return false;
     }
-    struct DynamicArithmeticVertex *vertices = dynamic_arithmetic_grow(
+    struct DynamicArithmeticVertex *vertices = source_grow(
       graph->vertices,
       &graph->vertex_capacity,
       graph->vertex_count + 1,
@@ -400,7 +380,7 @@ static bool dynamic_arithmetic_edge(
   ) {
     return false;
   }
-  struct DynamicArithmeticEdge *edges = dynamic_arithmetic_grow(
+  struct DynamicArithmeticEdge *edges = source_grow(
     graph->edges,
     &graph->edge_capacity,
     graph->edge_count + 1,
@@ -434,7 +414,7 @@ static bool dynamic_arithmetic_enqueue(
   if (graph->pending_count == SIZE_MAX) {
     return false;
   }
-  struct DynamicArithmeticLexItem *pending = dynamic_arithmetic_grow(
+  struct DynamicArithmeticLexItem *pending = source_grow(
     graph->pending,
     &graph->pending_capacity,
     graph->pending_count + 1,
@@ -477,7 +457,7 @@ static bool dynamic_arithmetic_emit(
     if (graph->endpoint_count == SIZE_MAX) {
       return false;
     }
-    size_t *endpoints = dynamic_arithmetic_grow(
+    size_t *endpoints = source_grow(
       graph->endpoints,
       &graph->endpoint_capacity,
       graph->endpoint_count + 1,
@@ -669,7 +649,7 @@ static bool dynamic_arithmetic_project_enqueue(
   if (projection->count == SIZE_MAX) {
     return false;
   }
-  size_t *pending = dynamic_arithmetic_grow(
+  size_t *pending = source_grow(
     projection->pending,
     &projection->capacity,
     projection->count + 1,
@@ -954,7 +934,7 @@ static bool dynamic_arithmetic_add_item(
   if (chart->item_count == SIZE_MAX) {
     return false;
   }
-  struct DynamicArithmeticItem *items = dynamic_arithmetic_grow(
+  struct DynamicArithmeticItem *items = source_grow(
     chart->items,
     &chart->item_capacity,
     chart->item_count + 1,
@@ -964,7 +944,7 @@ static bool dynamic_arithmetic_add_item(
     return false;
   }
   chart->items = items;
-  size_t *pending = dynamic_arithmetic_grow(
+  size_t *pending = source_grow(
     chart->pending,
     &chart->pending_capacity,
     chart->pending_count + 1,
@@ -1059,7 +1039,7 @@ static bool dynamic_arithmetic_complete(
   if (chart->completion_count == SIZE_MAX) {
     return false;
   }
-  struct DynamicArithmeticCompletion *completions = dynamic_arithmetic_grow(
+  struct DynamicArithmeticCompletion *completions = source_grow(
     chart->completions,
     &chart->completion_capacity,
     chart->completion_count + 1,