feat(utils): Resolve PR Comments

for the PR google#3673 & commit: 96e2f82

Author: Jainish-S

contributing/samples/nested_state_agent/__init__.py

@@ -0,0 +1 @@
+from . import agent

contributing/samples/nested_state_agent/agent.py

@@ -0,0 +1,33 @@
+import logging
+
+from google.adk.agents import Agent
+from google.adk.agents.callback_context import CallbackContext
+from google.adk.agents.readonly_context import ReadonlyContext
+from google.adk.utils.instructions_utils import inject_session_state
+
+
+def inject_nested_state(callback_context: CallbackContext):
+    callback_context.state["user"] = {
+        # "name": "Jainish",
+        # "profile": {"age": 24, "role": "Software Engineer"},
+    }
+    logging.info("State populated with nested user object.")
+
+
+async def build_instruction(readonly_context: ReadonlyContext) -> str:
+    print(readonly_context.state)
+    template = (
+        "Current user is {{user?.name?}} and {{user?.profile?.role?}}. Please greet"
+        " them by name and designation."
+    )
+    return await inject_session_state(template, readonly_context)
+
+
+agent = Agent(
+    name="nested_state_agent",
+    model="gemini-2.0-flash-lite",
+    instruction=build_instruction,
+    before_agent_callback=[inject_nested_state],
+)
+
+root_agent = agent

src/google/adk/utils/instructions_utils.py

@@ -93,7 +93,7 @@ def _get_nested_value(obj: Any, path: str) -> Any:
         return None
 
       optional = part.endswith('?')
-      key = part[:-1] if optional else part
+      key = part.removesuffix('?')
 
       # Try dictionary access first
       if hasattr(current, '__getitem__'):
@@ -127,7 +127,7 @@ async def _replace_match(match) -> str:
       var_name = full_path.removeprefix('artifact.')
       optional = var_name.endswith('?')
       if optional:
-        var_name = var_name[:-1]
+        var_name = var_name.removesuffix('?')
 
       if invocation_context.artifact_service is None:
         raise ValueError('Artifact service is not initialized.')