[LiveComponent] Skip the bracket pipeline for plain model names

src/LiveComponent/assets/dist/live_controller.js

@@ -262,6 +262,7 @@ function trimAll(str) {
 	return str.replace(/[\s]+/g, " ").trim();
 }
 function normalizeModelName(model) {
+	if (!model.includes("[") && !model.includes("]")) return model;
 	return model.replace(/\[]$/, "").split("[").map((s) => s.replace("]", "")).join(".");
 }
 function getValueFromElement(element, valueStore) {

src/LiveComponent/assets/src/string_utils.ts

@@ -31,6 +31,12 @@ export function trimAll(str: string) {
  * For example: "user[firstName]" becomes "user.firstName"
  */
 export function normalizeModelName(model: string): string {
+    // Most model names are plain (e.g. "query"), and the pipeline below is a
+    // no-op for them. ValueStore also normalizes already-normalized names.
+    if (!model.includes('[') && !model.includes(']')) {
+        return model;
+    }
+
     return (
         model
             // Names ending in "[]" represent arrays in HTML.

src/LiveComponent/assets/test/unit/string_utils.test.ts

@@ -30,4 +30,8 @@ describe('normalizeModelName', () => {
     it('can normalize a string ending in []', () => {
         expect(normalizeModelName('user[mailing][]')).toEqual('user.mailing');
     });
+
+    it('strips a stray closing bracket', () => {
+        expect(normalizeModelName('weird]name')).toEqual('weirdname');
+    });
 });