fix(nano): Strip quotes from state variable default string values

The parser was keeping the raw quotes around string default values,
causing empty strings like  to be rendered literally as "" in the UI.

Changes:
- Update parseStateBlock() to strip surrounding quotes from string values
- Add tests for state variable parsing with empty and non-empty strings
- Verify IR conversion preserves the unquoted string values

This fixes the LoginForm rendering issue where email and password inputs
showed "" instead of being empty.

Author: phodal

mpp-idea/build.gradle.kts

@@ -346,6 +346,10 @@ project(":") {
         }
 
         testImplementation(kotlin("test"))
+        testImplementation("cc.unitmesh:mpp-core:${prop("mppVersion")}") {
+            exclude(group = "org.jetbrains.kotlinx", module = "kotlinx-coroutines-core")
+            exclude(group = "org.jetbrains.kotlinx", module = "kotlinx-coroutines-core-jvm")
+        }
     }
 
     tasks {

mpp-idea/src/main/kotlin/cc/unitmesh/devins/idea/renderer/JewelRenderer.kt

@@ -6,6 +6,8 @@ import cc.unitmesh.agent.render.BaseRenderer
 import cc.unitmesh.devins.llm.MessageRole
 import cc.unitmesh.agent.render.RendererUtils
 
+import cc.unitmesh.agent.render.TaskInfo
+import cc.unitmesh.agent.render.TaskStatus
 import cc.unitmesh.agent.render.TimelineItem
 import cc.unitmesh.agent.render.ToolCallDisplayInfo
 import cc.unitmesh.agent.render.ToolCallInfo
@@ -82,6 +84,10 @@ class JewelRenderer : BaseRenderer() {
     private val _currentPlan = MutableStateFlow<AgentPlan?>(null)
     val currentPlan: StateFlow<AgentPlan?> = _currentPlan.asStateFlow()
 
+    // Task tracking (from task-boundary tool)
+    private val _tasks = MutableStateFlow<List<TaskInfo>>(emptyList())
+    val tasks: StateFlow<List<TaskInfo>> = _tasks.asStateFlow()
+
     /**
      * Set the current plan directly.
      * Used to sync with PlanStateService from CodingAgent.
@@ -152,6 +158,11 @@ class JewelRenderer : BaseRenderer() {
 
         jewelRendererLogger.info("renderToolCall: parsed params keys=${params.keys}")
 
+        // Handle task-boundary tool - update task list
+        if (toolName == "task-boundary") {
+            updateTaskFromToolCall(params)
+        }
+
         // Handle plan management tool - update plan state
         if (toolName == "plan") {
             jewelRendererLogger.info("renderToolCall: detected plan tool, calling updatePlanFromToolCall")
@@ -178,6 +189,11 @@ class JewelRenderer : BaseRenderer() {
         // Convert Map<String, Any> to Map<String, String> for internal use
         val stringParams = params.mapValues { it.value.toString() }
 
+        // Handle task-boundary tool - update task list
+        if (toolName == "task-boundary") {
+            updateTaskFromToolCall(stringParams)
+        }
+
         // Handle plan management tool - update plan state with original params
         if (toolName == "plan") {
             updatePlanFromToolCallWithAnyParams(params)
@@ -322,6 +338,37 @@ class JewelRenderer : BaseRenderer() {
         }
     }
 
+    /**
+     * Update task list from task-boundary tool call
+     */
+    private fun updateTaskFromToolCall(params: Map<String, String>) {
+        val taskName = params["taskName"] ?: return
+        val statusStr = params["status"] ?: "WORKING"
+        val summary = params["summary"] ?: ""
+        val status = TaskStatus.fromString(statusStr)
+
+        _tasks.update { currentTasks ->
+            val existingIndex = currentTasks.indexOfFirst { it.taskName == taskName }
+            if (existingIndex >= 0) {
+                // Update existing task
+                currentTasks.toMutableList().apply {
+                    this[existingIndex] = currentTasks[existingIndex].copy(
+                        status = status,
+                        summary = summary,
+                        timestamp = System.currentTimeMillis()
+                    )
+                }
+            } else {
+                // Add new task
+                currentTasks + TaskInfo(
+                    taskName = taskName,
+                    status = status,
+                    summary = summary
+                )
+            }
+        }
+    }
+
     override fun renderToolResult(
         toolName: String,
         success: Boolean,

xuiper-ui/src/main/kotlin/cc/unitmesh/xuiper/parser/IndentParser.kt

@@ -194,11 +194,18 @@ class IndentParser(
 
             val match = STATE_VAR_REGEX.matchEntire(line)
             if (match != null) {
+                val rawValue = match.groupValues[3].trim()
+                // Strip quotes from string values (e.g., "" -> empty string, "hello" -> hello)
+                val defaultValue = if (rawValue.startsWith("\"") && rawValue.endsWith("\"")) {
+                    rawValue.substring(1, rawValue.length - 1)
+                } else {
+                    rawValue
+                }
                 variables.add(
                     NanoNode.StateVariable(
                         name = match.groupValues[1],
                         type = match.groupValues[2],
-                        defaultValue = match.groupValues[3].trim()
+                        defaultValue = defaultValue
                     )
                 )
             }

xuiper-ui/src/test/kotlin/cc/unitmesh/xuiper/dsl/NanoDSLParserTest.kt

@@ -3,6 +3,7 @@ package cc.unitmesh.xuiper.dsl
 import cc.unitmesh.xuiper.action.HttpMethod
 import cc.unitmesh.xuiper.action.NanoAction
 import cc.unitmesh.xuiper.ast.NanoNode
+import kotlinx.serialization.json.jsonPrimitive
 import org.junit.jupiter.api.Test
 import kotlin.test.assertEquals
 import kotlin.test.assertNotNull
@@ -390,4 +391,74 @@ component ContactForm:
         assertEquals("/api/contact", fetchAction!!.url)
         assertEquals(HttpMethod.POST, fetchAction.method)
     }
+
+    @Test
+    fun `should parse state variables with empty string default values correctly`() {
+        val source = """
+component LoginForm:
+    state:
+        email: str = ""
+        password: str = ""
+        username: str = "default"
+        loading: bool = False
+        count: int = 0
+
+    VStack:
+        Text("Login")
+        """.trimIndent()
+
+        val result = NanoDSL.parse(source)
+
+        // Check state was parsed
+        assertNotNull(result.state)
+        val state = result.state!!
+
+        // Check that empty string default values are correctly stripped of quotes
+        val emailVar = state.variables.find { it.name == "email" }
+        assertNotNull(emailVar)
+        assertEquals("str", emailVar!!.type)
+        assertEquals("", emailVar.defaultValue, "Empty string should have quotes stripped")
+
+        val passwordVar = state.variables.find { it.name == "password" }
+        assertNotNull(passwordVar)
+        assertEquals("", passwordVar!!.defaultValue, "Empty string should have quotes stripped")
+
+        // Check that non-empty string values are also correctly stripped
+        val usernameVar = state.variables.find { it.name == "username" }
+        assertNotNull(usernameVar)
+        assertEquals("default", usernameVar!!.defaultValue, "String value should have quotes stripped")
+
+        // Check that non-string values are preserved as-is
+        val loadingVar = state.variables.find { it.name == "loading" }
+        assertNotNull(loadingVar)
+        assertEquals("False", loadingVar!!.defaultValue)
+
+        val countVar = state.variables.find { it.name == "count" }
+        assertNotNull(countVar)
+        assertEquals("0", countVar!!.defaultValue)
+    }
+
+    @Test
+    fun `should render state with empty string correctly in IR`() {
+        val source = """
+component LoginForm:
+    state:
+        email: str = ""
+        password: str = ""
+
+    Card:
+        Input(value:=state.email, placeholder="Email")
+        """.trimIndent()
+
+        val result = NanoDSL.parse(source)
+        val ir = NanoDSL.toIR(result)
+
+        // Check state in IR
+        assertNotNull(ir.state)
+        val emailState = ir.state!!.variables["email"]
+        assertNotNull(emailState)
+        assertEquals("str", emailState!!.type)
+        // The default value should be an empty string, not ""
+        assertEquals("", emailState.defaultValue?.jsonPrimitive?.content)
+    }
 }