@barefootjs/jinja 0.18.3 → 0.18.5

package/dist/index.js

@@ -187369,7 +187369,7 @@ function isAriaBooleanAttr(name) {
 }
 
 // src/adapter/jinja-adapter.ts
-import { BF_SLOT, BF_COND, BF_REGION } from "@barefootjs/shared";
+import { BF_SLOT, BF_COND, BF_REGION, escapeHtml } from "@barefootjs/shared";
 
 // src/adapter/lib/constants.ts
 var JINJA_TEMPLATE_PRIMITIVES = {
@@ -187378,7 +187378,10 @@ var JINJA_TEMPLATE_PRIMITIVES = {
   Number: { arity: 1, emit: (args) => `bf.number(${args[0]})` },
   "Math.floor": { arity: 1, emit: (args) => `bf.floor(${args[0]})` },
   "Math.ceil": { arity: 1, emit: (args) => `bf.ceil(${args[0]})` },
-  "Math.round": { arity: 1, emit: (args) => `bf.round(${args[0]})` }
+  "Math.round": { arity: 1, emit: (args) => `bf.round(${args[0]})` },
+  "Math.min": { arity: 2, emit: (args) => `bf.min(${args[0]}, ${args[1]})` },
+  "Math.max": { arity: 2, emit: (args) => `bf.max(${args[0]}, ${args[1]})` },
+  "Math.abs": { arity: 1, emit: (args) => `bf.abs(${args[0]})` }
 };
 var JINJA_PRIMITIVE_EMIT_MAP = Object.fromEntries(Object.entries(JINJA_TEMPLATE_PRIMITIVES).map(([k, v]) => [k, v.emit]));
 
@@ -187557,6 +187560,12 @@ function renderArrayMethod(method, object, args, emit) {
       const recv = emit(object);
       return `bf.trim(${recv})`;
     }
+    case "trimStart":
+    case "trimEnd": {
+      const fn = method === "trimStart" ? "trim_start" : "trim_end";
+      const recv = emit(object);
+      return `bf.${fn}(${recv})`;
+    }
     case "toFixed": {
       const recv = emit(object);
       const digits = args.length >= 1 ? emit(args[0]) : "0";
@@ -187590,6 +187599,12 @@ function renderArrayMethod(method, object, args, emit) {
       const newS = emit(args[1]);
       return `bf.replace(${recv}, ${oldS}, ${newS})`;
     }
+    case "replaceAll": {
+      const recv = emit(object);
+      const oldS = emit(args[0]);
+      const newS = emit(args[1]);
+      return `bf.replace_all(${recv}, ${oldS}, ${newS})`;
+    }
     case "repeat": {
       const recv = emit(object);
       const count = args.length === 0 ? "0" : emit(args[0]);
@@ -187686,6 +187701,7 @@ function renderFlatMethod(recv, depth, emit) {
 
 // src/adapter/expr/emitters.ts
 import {
+  groupBinaryOperand,
   identifierPath,
   matchSearchParamsMethodCall,
   sortComparatorFromArrow
@@ -187731,11 +187747,11 @@ class JinjaFilterEmitter {
       return "none";
     return String(value);
   }
-  member(object, property, _computed, emit) {
+  member(object, property, _computed, _optional, emit) {
     if (property === "length") {
       return `bf.length(${emit(object)})`;
     }
-    return `${emit(object)}.${property}`;
+    return `${emit(object)}['${escapeJinjaSingleQuoted(property)}']`;
   }
   indexAccess(object, index, emit) {
     return `${emit(object)}[${emit(index)}]`;
@@ -187754,8 +187770,8 @@ class JinjaFilterEmitter {
     return emit(argument);
   }
   binary(op, left, right, emit) {
-    const l = emit(left);
-    const r = emit(right);
+    const l = groupBinaryOperand(left, emit(left));
+    const r = groupBinaryOperand(right, emit(right));
     const opMap = {
       "===": "==",
       "!==": "!=",
@@ -187837,7 +187853,7 @@ class JinjaTopLevelEmitter {
       return "none";
     return String(value);
   }
-  member(object, property, _computed, emit) {
+  member(object, property, _computed, _optional, emit) {
     if (object.kind === "identifier" && object.name === "props") {
       return jinjaIdent(property);
     }
@@ -187849,7 +187865,7 @@ class JinjaTopLevelEmitter {
     const obj = emit(object);
     if (property === "length")
       return `bf.length(${obj})`;
-    return `${obj}.${property}`;
+    return `${obj}['${escapeJinjaSingleQuoted(property)}']`;
   }
   indexAccess(object, index, emit) {
     return `${emit(object)}[${emit(index)}]`;
@@ -187883,8 +187899,8 @@ class JinjaTopLevelEmitter {
     return emit(argument);
   }
   binary(op, left, right, emit) {
-    const l = emit(left);
-    const r = emit(right);
+    const l = groupBinaryOperand(left, emit(left));
+    const r = groupBinaryOperand(right, emit(right));
     const opMap = {
       "===": "==",
       "!==": "!=",
@@ -188256,6 +188272,7 @@ class JinjaAdapter extends BaseAdapter {
   options;
   errors = [];
   inLoop = false;
+  currentLoopKeyDepth = 0;
   propsObjectName = null;
   propsParams = [];
   booleanTypedProps = new Set;
@@ -188332,7 +188349,7 @@ class JinjaAdapter extends BaseAdapter {
     return this.renderElement(node);
   }
   emitText(node) {
-    return node.value;
+    return escapeHtml(node.value);
   }
   emitExpression(node) {
     return this.renderExpression(node);
@@ -188563,7 +188580,7 @@ ${whenTrue}
     const renderedChildren = this.renderChildren(loop.children);
     const loopVar = loop.iterationShape === "keys" ? "__bf_item" : supportableDestructure ? "__bf_item" : param;
     const indexLocalLines = [];
-    if (loop.iterationShape === "keys") {
+    if (loop.objectIteration) {} else if (loop.iterationShape === "keys") {
       indexLocalLines.push(`{% set ${jinjaIdent(param)} = loop.index0 %}`);
     } else if (loop.index) {
       indexLocalLines.push(`{% set ${jinjaIdent(loop.index)} = loop.index0 %}`);
@@ -188583,13 +188600,17 @@ ${whenTrue}
     }
     const prevInLoop = this.inLoop;
     this.inLoop = true;
+    const prevLoopKeyDepth = this.currentLoopKeyDepth;
+    this.currentLoopKeyDepth = loop.depth;
     const childrenUnderLoop = this.renderChildren(loop.children);
+    this.currentLoopKeyDepth = prevLoopKeyDepth;
     this.inLoop = prevInLoop;
     const bodyChildren = loop.bodyIsItemConditional && loop.key ? `{{ bf.comment("loop-i:" ~ bf.string(${this.convertExpressionToJinja(loop.key)})) | safe }}
 ${childrenUnderLoop}` : childrenUnderLoop;
     const lines = [];
     lines.push(`{{ bf.comment("loop:${loop.markerId}") | safe }}`);
-    lines.push(`{% for ${jinjaIdent(loopVar)} in ${array} %}`);
+    const forHeader = loop.objectIteration === "entries" ? `{% for ${jinjaIdent(loop.index ?? param)}, ${jinjaIdent(param)} in ${array}.items() %}` : loop.objectIteration === "keys" ? `{% for ${jinjaIdent(param)} in ${array}.keys() %}` : loop.objectIteration === "values" ? `{% for ${jinjaIdent(param)} in ${array}.values() %}` : `{% for ${jinjaIdent(loopVar)} in ${array} %}`;
+    lines.push(forHeader);
     for (const il of indexLocalLines)
       lines.push(il);
     if (loop.filterPredicate) {
@@ -188661,9 +188682,20 @@ ${childrenUnderLoop}` : childrenUnderLoop;
   renderComponent(comp) {
     const segments = [{ kind: "entries", parts: [] }];
     const currentEntries = () => this.componentPropSegmentEntries(segments);
+    const namedSlotSetBlocks = [];
     for (const p of comp.props) {
       if ((p.name.match(/^on[A-Z]/) || p.name === "ref") && p.value.kind === "expression")
         continue;
+      if (p.value.kind === "jsx-children" && p.name !== "children") {
+        const prevInLoop = this.inLoop;
+        this.inLoop = false;
+        const slotBody = this.renderChildren(p.value.children);
+        this.inLoop = prevInLoop;
+        const captureName = `bf_prop_${this.childrenCaptureCounter++}`;
+        namedSlotSetBlocks.push(`{% set ${captureName} %}${slotBody}{% endset %}`);
+        currentEntries().push(`${jinjaHashKey(p.name)}: ${captureName}`);
+        continue;
+      }
       if (p.value.kind === "spread") {
         const trimmed = p.value.expr.trim();
         if (this.propsObjectName && this.propsObjectName === trimmed) {
@@ -188692,11 +188724,11 @@ ${childrenUnderLoop}` : childrenUnderLoop;
       const captureName = `bf_children_${comp.slotId ?? "c" + this.childrenCaptureCounter++}`;
       currentEntries().push(`${jinjaHashKey("children")}: ${captureName}`);
       const dict = this.combineComponentPropSegments(segments);
-      return `{% set ${captureName} %}${childrenBody}{% endset %}{{ bf.render_child('${tplName}', ${dict}) | safe }}`;
+      return `${namedSlotSetBlocks.join("")}{% set ${captureName} %}${childrenBody}{% endset %}{{ bf.render_child('${tplName}', ${dict}) | safe }}`;
     }
     const isEmpty = segments.every((s) => s.kind === "entries" && s.parts.length === 0);
     const dictEntries = isEmpty ? "" : `, ${this.combineComponentPropSegments(segments)}`;
-    return `{{ bf.render_child('${tplName}'${dictEntries}) | safe }}`;
+    return `${namedSlotSetBlocks.join("")}{{ bf.render_child('${tplName}'${dictEntries}) | safe }}`;
   }
   childrenCaptureCounter = 0;
   presenceVarCounter = 0;
@@ -188741,7 +188773,7 @@ ${alternate}
 ${children}`;
   }
   elementAttrEmitter = {
-    emitLiteral: (value, name) => `${name}="${value.value}"`,
+    emitLiteral: (value, name) => `${name}="${escapeHtml(value.value)}"`,
     emitExpression: (value, name) => {
       if (name === "style") {
         const css = this.tryLowerStyleObject(value.expr);
@@ -188849,9 +188881,10 @@ ${name}="{{ bf.string(${val}) }}"
       let attrName;
       if (attr.name === "className")
         attrName = "class";
-      else if (attr.name === "key")
-        attrName = "data-key";
-      else
+      else if (attr.name === "key") {
+        const depth = this.currentLoopKeyDepth;
+        attrName = depth > 0 ? `data-key-${depth}` : "data-key";
+      } else
         attrName = attr.name;
       const lowered = emitAttrValue(attr.value, this.elementAttrEmitter, attrName);
       if (lowered)
@@ -189110,24 +189143,10 @@ var conformancePins = {
     { code: "BF101", severity: "error", issue: "https://github.com/piconic-ai/barefootjs/issues/2038" }
   ],
   "array-map-function-reference": [{ code: "BF101", severity: "error" }],
-  "dangerous-inner-html": [{ code: "BF101", severity: "error" }],
-  "string-replaceall": [{ code: "BF101", severity: "error" }]
+  "dangerous-inner-html": [{ code: "BF101", severity: "error" }]
 };
 // src/render-divergences.ts
-var renderDivergences = {
-  "arithmetic-text": "`(count() + 2) * 3` renders 10 instead of 18 — the parenthesised sub-expression loses its grouping (silent wrong arithmetic)",
-  "html-entity-text": "`©` in JSX literal text: Hono decodes to `©`, this adapter re-emits the raw entity — same DOM, different bytes",
-  "math-methods": "Math.min/max/abs over a signal render empty (only Math.floor is in the template-primitive registry)",
-  "boolean-attr-literals": 'camelCase boolean alias `readOnly`: Hono SSRs `readOnly="true"`, this adapter emits bare presence',
-  "camelcase-attributes": "`htmlFor` is not lowered to `for` (Hono maps it)",
-  "static-attr-escape": 'static attribute values are not HTML-escaped (`title="Fish & Chips"` emitted raw; Hono escapes)',
-  "svg-icon": "SVG camelCase presentation attrs (`strokeWidth`, `strokeLinecap`) pass through unmapped; Hono lowers to kebab-case",
-  "object-entries-map": "`Object.entries(prop).map(([k, v]) => …)` renders an EMPTY list — the object-shaped prop silently produces zero iterations",
-  "nested-loop-outer-binding": "nested-loop inner items carry `data-key` where the reference emits the depth-suffixed `data-key-1`",
-  "jsx-element-prop": "a JSX element passed as a NON-children prop renders an empty slot — the element value is silently dropped",
-  "string-slice": "`.slice()` on a STRING renders empty (array-slice helper misfires on strings)",
-  "string-trim-sided": "`.trimStart()` / `.trimEnd()` render empty (no lowering)"
-};
+var renderDivergences = {};
 export {
   renderDivergences,
   jinjaAdapter,

package/dist/render-divergences.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"render-divergences.d.ts","sourceRoot":"","sources":["../src/render-divergences.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;GAYG;AAEH,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,iBAAiB,CAAA;AAExD,eAAO,MAAM,iBAAiB,EAAE,iBAyB/B,CAAA"}
+{"version":3,"file":"render-divergences.d.ts","sourceRoot":"","sources":["../src/render-divergences.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;GAYG;AAEH,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,iBAAiB,CAAA;AAExD,eAAO,MAAM,iBAAiB,EAAE,iBAAsB,CAAA"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@barefootjs/jinja",
-  "version": "0.18.3",
+  "version": "0.18.5",
   "description": "Jinja2 adapter for BarefootJS — compiles IR to .jinja templates and ships the Python BarefootJS rendering runtime; runs under any Python web framework (Flask, etc.)",
   "type": "module",
   "main": "./dist/index.js",
@@ -53,14 +53,14 @@
     "directory": "packages/adapter-jinja"
   },
   "dependencies": {
-    "@barefootjs/shared": "0.18.3"
+    "@barefootjs/shared": "0.18.5"
   },
   "peerDependencies": {
     "@barefootjs/jsx": ">=0.2.0"
   },
   "devDependencies": {
     "@barefootjs/adapter-tests": "0.1.0",
-    "@barefootjs/jsx": "0.18.3",
+    "@barefootjs/jsx": "0.18.5",
     "typescript": "^5.0.0"
   }
 }

package/python/barefootjs/runtime.py

@@ -819,6 +819,29 @@ class BarefootJS:
         # is -1, not -2). `floor(n + 0.5)` reproduces that for both signs.
         return float(math.floor(n + 0.5))
 
+    def min(self, a: Any, b: Any) -> float:
+        """`Math.min(a, b)` -- two-arg form only (#2168 math-methods).
+        JS returns NaN if either operand is NaN."""
+        x, y = js_number(a), js_number(b)
+        if _is_nan(x):
+            return x
+        if _is_nan(y):
+            return y
+        return x if x < y else y
+
+    def max(self, a: Any, b: Any) -> float:
+        x, y = js_number(a), js_number(b)
+        if _is_nan(x):
+            return x
+        if _is_nan(y):
+            return y
+        return x if x > y else y
+
+    def abs(self, value: Any) -> float:
+        """`Math.abs()` (#2168 math-methods)."""
+        n = js_number(value)
+        return n if _is_nan(n) else abs(n)
+
     # -----------------------------------------------------------------
     # Array / String method helpers (#1448 Tier A)
     # -----------------------------------------------------------------
@@ -933,12 +956,20 @@ class BarefootJS:
             out.extend(b)
         return out
 
-    def slice(self, recv: Any, start: Any, end: Any) -> list:
-        if not isinstance(recv, list):
+    def slice(self, recv: Any, start: Any, end: Any) -> Any:
+        # `Array.prototype.slice(start, end?)` AND `String.prototype.slice`
+        # (the `string-slice` divergence, #2182) -- the adapter emits the
+        # same `bf.slice(recv, start, end)` call for both receiver shapes
+        # (it can't disambiguate string vs. array at compile time), so
+        # this dispatches on the Python type, mirroring `includes` above.
+        # `str` and `list` share slicing semantics (`recv[s:e]`) and
+        # `len()`, so one clamp computation serves both; `str` indexing
+        # is by Unicode code point already, matching JS except for
+        # astral-plane input (the same divergence boundary every other
+        # adapter's pad/trim helpers already accept).
+        if not isinstance(recv, (list, str)):
             return []
         length = len(recv)
-        if length == 0:
-            return []
         s = start if start is not None else 0
         if s < 0:
             s = length + s
@@ -950,8 +981,8 @@ class BarefootJS:
         e = max(e, 0)
         e = min(e, length)
         if s >= e:
-            return []
-        return list(recv[s:e])
+            return recv[0:0]
+        return recv[s:e] if isinstance(recv, str) else list(recv[s:e])
 
     def reverse(self, recv: Any) -> list:
         if not isinstance(recv, list):
@@ -1014,6 +1045,20 @@ class BarefootJS:
             return ""
         return js_string(recv).strip()
 
+    def trim_start(self, recv: Any) -> str:
+        """`String.prototype.trimStart()` -- the one-sided sibling of `trim`
+        above (#2183 follow-up)."""
+        if recv is None or isinstance(recv, (list, dict)):
+            return ""
+        return js_string(recv).lstrip()
+
+    def trim_end(self, recv: Any) -> str:
+        """`String.prototype.trimEnd()` -- the one-sided sibling of `trim`
+        above (#2183 follow-up)."""
+        if recv is None or isinstance(recv, (list, dict)):
+            return ""
+        return js_string(recv).rstrip()
+
     def to_fixed(self, value: Any, digits: int = 0) -> str:
         n = self.number(value)
         if _is_nan(n):
@@ -1076,6 +1121,19 @@ class BarefootJS:
             return s
         return s[:i] + n + s[i + len(o) :]
 
+    def replace_all(self, recv: Any, pattern: Any, replacement: Any) -> str:
+        """`String.prototype.replaceAll(pattern, replacement)`, string-pattern
+        form only (#2182) -- every occurrence, the all-occurrences sibling of
+        `replace` above. Python's own `str.replace(old, new)` (no count arg)
+        is already global by default, including the empty-pattern-inserts-
+        at-every-boundary edge case (`"abc".replace("", "X")` -> "XaXbXcX"),
+        so it needs no hand-rolled loop the way the other runtimes' first-
+        occurrence-only native replace does."""
+        s = _scalar_or_empty(recv)
+        o = js_string(pattern)
+        n = js_string(replacement)
+        return s.replace(o, n)
+
     def query(self, base: Any, *triples: Any) -> str:
         """`queryHref(base, {...})` (#2042) -- build `"$base?k=v&..."` from a
         flat list of (guard, key, value) triples. A pair is included iff its

package/python/tests/test_helper_vectors.py

@@ -117,13 +117,19 @@ BINDINGS = {
     "floor": bf.floor,
     "ceil": bf.ceil,
     "round": bf.round,
+    "min": bf.min,
+    "max": bf.max,
+    "abs": bf.abs,
     "to_fixed": lambda *a: bf.to_fixed(*a),
     "lower": bf.lc,
     "upper": bf.uc,
     "trim": bf.trim,
+    "trim_start": bf.trim_start,
+    "trim_end": bf.trim_end,
     "starts_with": lambda *a: bf.starts_with(*a),
     "ends_with": lambda *a: bf.ends_with(*a),
     "replace": lambda *a: bf.replace(*a),
+    "replace_all": lambda *a: bf.replace_all(*a),
     "repeat": lambda *a: bf.repeat(*a),
     "pad_start": lambda *a: bf.pad_start(*a),
     "pad_end": lambda *a: bf.pad_end(*a),

package/python/tests/test_template_primitives.py

@@ -74,6 +74,27 @@ class TemplatePrimitivesTest(unittest.TestCase):
         self.assertEqual(self.bf.round(-1.6), -2)
         self.assertTrue(_is_nan(self.bf.round("not")))
 
+    def test_min_max_abs(self):
+        # `Math.min(a, b)` / `Math.max(a, b)` (two-arg forms only) and
+        # `Math.abs()` (#2168 math-methods). JS returns NaN if EITHER
+        # min/max operand is NaN.
+        self.assertEqual(self.bf.min(3, 7), 3)
+        self.assertEqual(self.bf.min(7, 3), 3)
+        self.assertEqual(self.bf.min(-2, -5), -5)
+        self.assertTrue(_is_nan(self.bf.min("not", 5)))
+        self.assertTrue(_is_nan(self.bf.min(5, "not")))
+
+        self.assertEqual(self.bf.max(3, 7), 7)
+        self.assertEqual(self.bf.max(7, 3), 7)
+        self.assertEqual(self.bf.max(-2, -5), -2)
+        self.assertTrue(_is_nan(self.bf.max("not", 5)))
+        self.assertTrue(_is_nan(self.bf.max(5, "not")))
+
+        self.assertEqual(self.bf.abs(-7.6), 7.6)
+        self.assertEqual(self.bf.abs(7.6), 7.6)
+        self.assertEqual(self.bf.abs(0), 0)
+        self.assertTrue(_is_nan(self.bf.abs("not")))
+
     def test_includes_dispatch(self):
         # `Array.prototype.includes(x)` + `String.prototype.includes(sub)`
         # lower to the same `bf.includes(recv, elem)` shape -- see #1448
@@ -158,6 +179,18 @@ class TemplatePrimitivesTest(unittest.TestCase):
         out.append("mutated")
         self.assertEqual(src, ["a", "b", "c"])
 
+    def test_slice_string_receiver(self):
+        # The `string-slice` divergence (#2182): a string receiver used
+        # to fall through the array-only branch and return an empty
+        # list instead of a substring.
+        word = "barefootjs"
+        self.assertEqual(self.bf.slice(word, 0, 4), "bare")
+        self.assertEqual(self.bf.slice(word, -4, None), "otjs")
+        self.assertEqual(self.bf.slice(word, 4, None), "footjs")
+        self.assertEqual(self.bf.slice(word, 5, 2), "")
+        # Multi-byte: index by character, not byte.
+        self.assertEqual(self.bf.slice("héllo", 0, 2), "hé")
+
     def test_reverse_mutation_isolation(self):
         self.assertEqual(self.bf.reverse(["a", "b", "c"]), ["c", "b", "a"])
         self.assertEqual(self.bf.reverse([]), [])
@@ -175,6 +208,18 @@ class TemplatePrimitivesTest(unittest.TestCase):
         self.assertEqual(self.bf.trim({"a": 1}), "")
         self.assertEqual(self.bf.trim(42), "42")
 
+    def test_trim_start_and_trim_end(self):
+        # The one-sided siblings of `trim` above (#2183). Padding BOTH
+        # sides so a swapped side fails visibly.
+        self.assertEqual(self.bf.trim_start("   padded   "), "padded   ")
+        self.assertEqual(self.bf.trim_end("   padded   "), "   padded")
+        self.assertEqual(self.bf.trim_start(""), "")
+        self.assertEqual(self.bf.trim_end(""), "")
+        self.assertEqual(self.bf.trim_start(None), "")
+        self.assertEqual(self.bf.trim_end(None), "")
+        self.assertEqual(self.bf.trim_start({"a": 1}), "")
+        self.assertEqual(self.bf.trim_end([1, 2]), "")
+
     def test_split(self):
         self.assertEqual(self.bf.split("a,b,c", ","), ["a", "b", "c"])
         self.assertEqual(self.bf.split("a.b.c", "."), ["a", "b", "c"])

package/src/__tests__/jinja-adapter-unit.test.ts

@@ -383,6 +383,27 @@ export function C(props: { count: number }) {
   })
 })
 
+describe('JinjaAdapter - named-slot capture identifier safety (#2168 jsx-element-prop)', () => {
+  // A JSX-valued prop under a hyphenated name (`data-slot`, a valid JSX
+  // attribute name) must not leak into the `{% set %}` capture variable's
+  // identifier — Jinja variable names can't contain `-`. The capture
+  // identifier is purely counter-based (never derived from the prop name);
+  // the hash KEY passed to `render_child` still carries the real name,
+  // quoted via `jinjaHashKey`.
+  test('a hyphenated prop name does not appear in the capture variable', () => {
+    const { template } = compileAndGenerate(`
+function Card(props) { return null }
+export function Parent() {
+  return <Card data-slot={<strong>Title</strong>}>text</Card>
+}
+`)
+    expect(template).toContain('{% set bf_prop_0 %}')
+    expect(template).toContain("'data-slot': bf_prop_0")
+    expect(template).not.toContain('data-slot %}')
+    expect(template).not.toContain('data-slot_')
+  })
+})
+
 // #2038 nested-callback-predicate loudness is pinned at the shared
 // conformance layer (workstream C): `filter-nested-callback-predicate` /
 // `filter-nested-find-predicate` (BF101 via `expectedDiagnostics`) and

package/src/adapter/expr/array-method.ts

@@ -91,6 +91,15 @@ export function renderArrayMethod(
       const recv = emit(object)
       return `bf.trim(${recv})`
     }
+    case 'trimStart':
+    case 'trimEnd': {
+      // `.trimStart()` / `.trimEnd()` — the one-sided siblings of
+      // `.trim()` (#2183 follow-up). Dedicated `bf.trim_start` /
+      // `bf.trim_end` helpers, not `bf.trim` with a flag.
+      const fn = method === 'trimStart' ? 'trim_start' : 'trim_end'
+      const recv = emit(object)
+      return `bf.${fn}(${recv})`
+    }
     case 'toFixed': {
       // `.toFixed(digits?)` — `bf.to_fixed` mirrors JS rounding +
       // zero-padding (default 0 digits). #1897.
@@ -126,6 +135,16 @@ export function renderArrayMethod(
       const newS = emit(args[1])
       return `bf.replace(${recv}, ${oldS}, ${newS})`
     }
+    case 'replaceAll': {
+      // `.replaceAll(old, new)` — string-pattern form, EVERY occurrence,
+      // via the dedicated `bf.replace_all` helper (not `bf.replace` with
+      // a flag) — the regex-pattern form is refused upstream at the
+      // parser, same as `.replace`. See #2182.
+      const recv = emit(object)
+      const oldS = emit(args[0])
+      const newS = emit(args[1])
+      return `bf.replace_all(${recv}, ${oldS}, ${newS})`
+    }
     case 'repeat': {
       const recv = emit(object)
       const count = args.length === 0 ? '0' : emit(args[0])

package/src/adapter/expr/emitters.ts

@@ -48,7 +48,7 @@
  *      entry point `_renderJinjaFilterExprPublic`.
  */
 
-import {
+import { groupBinaryOperand,
   type ParsedExprEmitter,
   type HigherOrderMethod,
   type ArrayMethod,
@@ -141,7 +141,7 @@ export class JinjaFilterEmitter implements ParsedExprEmitter {
     return String(value)
   }
 
-  member(object: ParsedExpr, property: string, _computed: boolean, emit: (e: ParsedExpr) => string): string {
+  member(object: ParsedExpr, property: string, _computed: boolean, _optional: boolean, emit: (e: ParsedExpr) => string): string {
     // `.length` — route through `bf.length` (handles both array element
     // count and string char count, JS-compatibly). Jinja's builtin
     // `|length` filter also faults trying to match JS semantics for every
@@ -149,8 +149,16 @@ export class JinjaFilterEmitter implements ParsedExprEmitter {
     if (property === 'length') {
       return `bf.length(${emit(object)})`
     }
-    // Attribute / dict-key access — Jinja `.` resolves both transparently.
-    return `${emit(object)}.${property}`
+    // Bracket/item access, NOT `.` attribute access: Jinja's default
+    // `getattr` semantics try a Python ATTRIBUTE first, falling back to a
+    // dict key only if no such attribute exists — so `group.items` (a
+    // dict key from the JS object) instead resolves to the built-in bound
+    // method `dict.items`, raising "not iterable" downstream instead of
+    // returning the key's value. `[...]` compiles through Jinja's
+    // `getitem`, which tries the KEY first, sidestepping any built-in
+    // dict method name (items/keys/values/get/pop/update/...) that would
+    // otherwise shadow a same-named JS object field.
+    return `${emit(object)}['${escapeJinjaSingleQuoted(property)}']`
   }
 
   indexAccess(object: ParsedExpr, index: ParsedExpr, emit: (e: ParsedExpr) => string): string {
@@ -175,8 +183,11 @@ export class JinjaFilterEmitter implements ParsedExprEmitter {
   }
 
   binary(op: string, left: ParsedExpr, right: ParsedExpr, emit: (e: ParsedExpr) => string): string {
-    const l = emit(left)
-    const r = emit(right)
+    // Preserve source grouping: a compound operand re-emitted as infix
+    // text is otherwise re-parsed under THIS language's precedence —
+    // `(count() + 2) * 3` would silently become `count + 2 * 3` (#2173).
+    const l = groupBinaryOperand(left, emit(left))
+    const r = groupBinaryOperand(right, emit(right))
     // Jinja's `==` / `!=` are value-equality operators that compare strings
     // and numbers correctly — unlike Perl's numeric `==` (which the Mojo
     // adapter must steer around with `eq`/`ne`). Same reasoning as Kolon.
@@ -310,7 +321,7 @@ export class JinjaTopLevelEmitter implements ParsedExprEmitter {
     return String(value)
   }
 
-  member(object: ParsedExpr, property: string, _computed: boolean, emit: (e: ParsedExpr) => string): string {
+  member(object: ParsedExpr, property: string, _computed: boolean, _optional: boolean, emit: (e: ParsedExpr) => string): string {
     // `props.x` flattens to the bare context var the SSR caller binds each
     // prop to (props arrive as individual top-level context entries, not a
     // nested `props` dict).
@@ -328,8 +339,13 @@ export class JinjaTopLevelEmitter implements ParsedExprEmitter {
     const obj = emit(object)
     // `.length` → `bf.length` (array count or string char count, JS-compat).
     if (property === 'length') return `bf.length(${obj})`
-    // Jinja `.` access works for both dicts and objects.
-    return `${obj}.${property}`
+    // Bracket/item access, NOT `.` attribute access — see the sibling
+    // `JinjaFilterEmitter.member()` for why: Jinja's `.` tries a Python
+    // ATTRIBUTE first, so a dict key that happens to share a name with a
+    // built-in dict method (`items`, `keys`, `values`, `get`, ...) resolves
+    // to the bound method instead of the value. `[...]` (Jinja `getitem`)
+    // tries the key first.
+    return `${obj}['${escapeJinjaSingleQuoted(property)}']`
   }
 
   indexAccess(object: ParsedExpr, index: ParsedExpr, emit: (e: ParsedExpr) => string): string {
@@ -378,8 +394,11 @@ export class JinjaTopLevelEmitter implements ParsedExprEmitter {
   }
 
   binary(op: string, left: ParsedExpr, right: ParsedExpr, emit: (e: ParsedExpr) => string): string {
-    const l = emit(left)
-    const r = emit(right)
+    // Preserve source grouping: a compound operand re-emitted as infix
+    // text is otherwise re-parsed under THIS language's precedence —
+    // `(count() + 2) * 3` would silently become `count + 2 * 3` (#2173).
+    const l = groupBinaryOperand(left, emit(left))
+    const r = groupBinaryOperand(right, emit(right))
     // Jinja's `==` / `!=` handle both strings and numbers (unlike Perl's
     // numeric `==`), so all equality comparisons stay on `==` / `!=`.
     const opMap: Record<string, string> = {