Optimize snapshot property access using propertyAccess utility

- Updated all snapshot property access patterns to use propertyAccess() function
- Applied consistent property access style throughout fast-instantiator.ts
- Addresses GitHub comment from airhorns for performance consistency
- Changes snapshot?.["${key}"] to snapshot?${propertyAccess(key)}

Co-Authored-By: Harry Brundage <[email protected]>

Author: devin-ai-integration[bot]

src/fast-instantiator.ts

@@ -115,7 +115,7 @@ export class InstantiatorBuilder<T extends IClassModelType<Record<string, IAnyTy
         segments.push(`
           // instantiate fallback for ${key} of type ${safeTypeName(type)}
           this${propertyAccess(key)} = ${this.alias(`model.properties["${key}"]`)}.instantiate(
-            snapshot?.["${key}"],
+            snapshot?${propertyAccess(key)},
             context,
             this
           );
@@ -262,7 +262,7 @@ export class InstantiatorBuilder<T extends IClassModelType<Record<string, IAnyTy
     }
   }
 
-  private expressionForDirectlyAssignableType(key: string, type: DirectlyAssignableType, valueExpression = `snapshot?.["${key}"]`) {
+  private expressionForDirectlyAssignableType(key: string, type: DirectlyAssignableType, valueExpression = `snapshot?${propertyAccess(key)}`) {
     return type instanceof DateType ? `new Date(${valueExpression})` : valueExpression;
   }
 
@@ -290,7 +290,7 @@ export class InstantiatorBuilder<T extends IClassModelType<Record<string, IAnyTy
 
     return `
       // setup reference for ${key}
-      const ${identifierVarName} = snapshot?.["${key}"];
+      const ${identifierVarName} = snapshot?${propertyAccess(key)};
 
       // eager resolve path: check the reference cache immediately for the identifier
       const ${instanceVarName} = context.referenceCache.get(${identifierVarName});
@@ -344,7 +344,7 @@ export class InstantiatorBuilder<T extends IClassModelType<Record<string, IAnyTy
 
     return `
       // optional type for ${key}
-      let ${varName} = snapshot?.["${key}"];
+      let ${varName} = snapshot?${propertyAccess(key)};
       if (${comparisonsToUndefinedValues.join(" || ")}) {
         ${varName} = ${defaultValueExpression}
       }
@@ -357,7 +357,7 @@ export class InstantiatorBuilder<T extends IClassModelType<Record<string, IAnyTy
       return `
         // instantiate fallback for ${key} of type ${safeTypeName(type)}
         this${propertyAccess(key)} = ${this.alias(`model.properties["${key}"]`)}.instantiate(
-          snapshot?.["${key}"],
+          snapshot?${propertyAccess(key)},
           context,
           this
         );
@@ -367,7 +367,7 @@ export class InstantiatorBuilder<T extends IClassModelType<Record<string, IAnyTy
     // Directly assignable types are primitives so we don't need to worry about setting parent/env/etc. Hence, we just
     // pass the snapshot straight through to the constructor.
     return `
-      const arrayData = snapshot?.["${key}"];
+      const arrayData = snapshot?${propertyAccess(key)};
       if (arrayData && arrayData.length > 0) {
         this${propertyAccess(key)} = new QuickArray(
           ${this.alias(`model.properties["${key}"]`)},
@@ -396,7 +396,7 @@ export class InstantiatorBuilder<T extends IClassModelType<Record<string, IAnyTy
     return `
       const ${mapVarName} = new QuickMap(${this.alias(`model.properties["${key}"]`)}, this, context);
       this${propertyAccess(key)} = ${mapVarName};
-      const ${snapshotVarName} = snapshot?.["${key}"];
+      const ${snapshotVarName} = snapshot?${propertyAccess(key)};
       if (${snapshotVarName}) {
         for (const key in ${snapshotVarName}) {
           const item = ${this.alias(`model.properties["${key}"].childrenType`)}.instantiate(
@@ -422,7 +422,7 @@ export class InstantiatorBuilder<T extends IClassModelType<Record<string, IAnyTy
       destinationProp = this.alias(`Symbol.for("${this.getters.memoSymbolName(snapshottedView.property)}")`);
     }
 
-    const valueExpression = `snapshot?.["${snapshottedView.property}"]`;
+    const valueExpression = `snapshot?${propertyAccess(snapshottedView.property)}`;
     return `
       // setup snapshotted view for ${snapshottedView.property}
       const ${varName} = ${valueExpression};