feat!: introduce more field input types to HumanInputNode

.editorconfig

@@ -0,0 +1,18 @@
+root = true
+
+[*]
+end_of_line = lf
+insert_final_newline = true
+trim_trailing_whitespace = true
+charset = utf-8
+
+[*.py]
+indent_style = space
+indent_size = 4
+
+[*.toml]
+indent_style = space
+indent_size = 4
+
+[*.md]
+trim_trailing_whitespace = false

.gitignore

@@ -16,3 +16,17 @@ dist/
 
 # Plugin Daemon Slim
 .slim
+# Unit test / coverage reports
+htmlcov/
+.tox/
+.nox/
+.coverage
+.coverage.*
+.cache
+nosetests.xml
+coverage.xml
+*.cover
+*.py,cover
+.hypothesis/
+.pytest_cache/
+cover/

pyproject.toml

@@ -80,6 +80,11 @@ ignore-one-line-docstrings = true
     'S101',    # Assert statements used for pytest.
     'PLR2004', # Magic value used in test cases.
     'PLC2701', # Allow import private members from graphon module.
+    'PLC1901', # be more precise about assertion in tests.
+
+    # permit calls of private methods until we separate
+    # public interface tests from implementation tests.
+    'SLF001', # allow calls of private class methods in tests
 ]
 
 [tool.ty.environment]

src/graphon/entities/pause_reason.py

@@ -4,7 +4,7 @@
 
 from pydantic import BaseModel, Field
 
-from graphon.nodes.human_input.entities import FormInput, UserAction
+from graphon.nodes.human_input.entities import FormInputConfig, UserActionConfig
 
 
 class PauseReasonType(StrEnum):
@@ -18,8 +18,8 @@ class HumanInputRequired(BaseModel):
     )
     form_id: str
     form_content: str
-    inputs: list[FormInput] = Field(default_factory=list)
-    actions: list[UserAction] = Field(default_factory=list)
+    inputs: list[FormInputConfig] = Field(default_factory=list[FormInputConfig])
+    actions: list[UserActionConfig] = Field(default_factory=list[UserActionConfig])
     node_id: str
     node_title: str
 

src/graphon/graph_events/node.py

@@ -4,6 +4,7 @@
 from pydantic import Field
 
 from graphon.entities.pause_reason import PauseReason
+from graphon.variables.segments import Segment
 from graphon.variables.variables import Variable
 
 from .base import GraphNodeEventBase
@@ -91,6 +92,10 @@ class NodeRunHumanInputFormFilledEvent(GraphNodeEventBase):
         ...,
         description="Display text of the chosen action button.",
     )
+    submitted_data: Mapping[str, Segment] = Field(
+        default_factory=dict,
+        description="Runtime submitted values keyed by form output variable name.",
+    )
 
 
 class NodeRunHumanInputFormTimeoutEvent(GraphNodeEventBase):

src/graphon/node_events/node.py

@@ -8,6 +8,7 @@
 from graphon.file.models import File
 from graphon.model_runtime.entities.llm_entities import LLMUsage
 from graphon.node_events.base import NodeRunResult
+from graphon.variables.segments import Segment
 from graphon.variables.variables import Variable
 
 from .base import NodeEventBase
@@ -73,6 +74,11 @@ class HumanInputFormFilledEvent(NodeEventBase):
     action_id: str
     action_text: str
 
+    # submitted_data records the data user submitted in the form inputs.
+    # It is a mapping from FormInput.output_variable_name to
+    # their runtime values.
+    submitted_data: Mapping[str, Segment] = Field(default_factory=dict)
+
 
 class HumanInputFormTimeoutEvent(NodeEventBase):
     """Event emitted when a human input form times out."""

src/graphon/nodes/base/node.py

@@ -884,6 +884,7 @@ def _(self, event: HumanInputFormFilledEvent) -> NodeRunHumanInputFormFilledEven
             rendered_content=event.rendered_content,
             action_id=event.action_id,
             action_text=event.action_text,
+            submitted_data=event.submitted_data,
         )
 
     @_dispatch.register

src/graphon/nodes/human_input/_exc.py

@@ -0,0 +1,22 @@
+from graphon.file.enums import FileTransferMethod
+
+
+class InvalidConfigError(Exception):
+    pass
+
+
+class InvalidSubmittedDataError(Exception):
+    pass
+
+
+class InvalidTransferMethodError(InvalidConfigError):
+    transfer_method: FileTransferMethod
+
+    def __init__(self, transfer_method: FileTransferMethod) -> None:
+        self.transfer_method = transfer_method
+        super().__init__(f"invalid file transfer method: {transfer_method}")
+
+
+class ExtensionsNotSetErrorValueError(InvalidConfigError):
+    def __init__(self) -> None:
+        super().__init__("allowed_file_extensions not set")