Skip to content

Commit

Permalink
Change StructureRunTask parent to PromptTask
Browse files Browse the repository at this point in the history
  • Loading branch information
collindutter committed Oct 7, 2024
1 parent 2f5c871 commit bbbfbc8
Show file tree
Hide file tree
Showing 6 changed files with 8 additions and 131 deletions.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -63,6 +63,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
- **BREAKING**: `BaseExtractionEngine` no longer catches exceptions and returns `ErrorArtifact`s.
- **BREAKING**: `JsonExtractionEngine.template_schema` is now required.
- **BREAKING**: `CsvExtractionEngine.column_names` is now required.
- `StructureRunTask` now inherits from `PromptTask`.
- `JsonExtractionEngine.extract_artifacts` now returns a `ListArtifact[JsonArtifact]`.
- `CsvExtractionEngine.extract_artifacts` now returns a `ListArtifact[CsvRowArtifact]`.
- Remove `manifest.yml` requirements for custom tool creation.
Expand Down
2 changes: 0 additions & 2 deletions griptape/tasks/__init__.py
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
from .base_task import BaseTask
from .base_text_input_task import BaseTextInputTask
from .base_multi_text_input_task import BaseMultiTextInputTask
from .prompt_task import PromptTask
from .actions_subtask import ActionsSubtask
from .toolkit_task import ToolkitTask
Expand All @@ -23,7 +22,6 @@
__all__ = [
"BaseTask",
"BaseTextInputTask",
"BaseMultiTextInputTask",
"PromptTask",
"ActionsSubtask",
"ToolkitTask",
Expand Down
60 changes: 0 additions & 60 deletions griptape/tasks/base_multi_text_input_task.py

This file was deleted.

10 changes: 7 additions & 3 deletions griptape/tasks/structure_run_task.py
Original file line number Diff line number Diff line change
Expand Up @@ -4,15 +4,16 @@

from attrs import define, field

from griptape.tasks import BaseMultiTextInputTask
from griptape.artifacts.list_artifact import ListArtifact
from griptape.tasks.prompt_task import PromptTask

if TYPE_CHECKING:
from griptape.artifacts import BaseArtifact
from griptape.drivers.structure_run.base_structure_run_driver import BaseStructureRunDriver


@define
class StructureRunTask(BaseMultiTextInputTask):
class StructureRunTask(PromptTask):
"""Task to run a Structure.
Attributes:
Expand All @@ -22,4 +23,7 @@ class StructureRunTask(BaseMultiTextInputTask):
driver: BaseStructureRunDriver = field(kw_only=True)

def run(self) -> BaseArtifact:
return self.driver.run(*self.input)
if isinstance(self.input, ListArtifact):
return self.driver.run(*self.input.value)

Check warning on line 27 in griptape/tasks/structure_run_task.py

View check run for this annotation

Codecov / codecov/patch

griptape/tasks/structure_run_task.py#L27

Added line #L27 was not covered by tests
else:
return self.driver.run(self.input)
10 changes: 0 additions & 10 deletions tests/mocks/mock_multi_text_input_task.py

This file was deleted.

56 changes: 0 additions & 56 deletions tests/unit/tasks/test_base_multi_text_input_task.py

This file was deleted.

0 comments on commit bbbfbc8

Please sign in to comment.