conductor-oss / conductor-oss/python-sdk
TaskType.NOOP missing — NOOP system task has no enum value or builder class
- Langage dominant
- Python
- Étoiles
- 104
- Forks
- 42
- Merge moyen
- 2 j 1 h
- PR mergées (30 j)
- 3
Description
## Summary
Tested against: **Conductor OSS 3.32.0-rc.9**
The Conductor server has a `NOOP` system task type (`TaskType.NOOP`) that
completes immediately with no inputParameters. The Python SDK has neither an
enum value nor a builder class for it.
## Details
```python
from conductor.client.workflow.task.task_type import TaskType
hasattr(TaskType, "NOOP") # False
```
There is no `noop_task.py` and no `NoopTask` class anywhere in the SDK.
NOOP is useful as an explicit no-op placeholder in workflow branches (e.g.
the default branch of a SWITCH where nothing should happen). Without SDK support,
users must use raw dicts.
## Suggested fix
Add to `task_type.py`:
```python
NOOP = "NOOP"
```
Add `src/conductor/client/workflow/task/noop_task.py`:
```python
class NoopTask(TaskInterface):
def __init__(self, task_ref_name: str):
super().__init__(task_reference_name=task_ref_name,
task_type=TaskType.NOOP)
```
## Verified against
Conductor server **3.32.0-rc.9**. A raw-dict NOOP workflow registered and ran
to `COMPLETED` on the server. Only the SDK support is missing.
Guide de contribution
Aucun guide de contribution indexé pour ce dépôt
Piste de recherche
Commencez par src/conductor/client/workflow/task/task_type.py et le TaskInterface utilisé par les constructeurs de tâches. Ajoutez la valeur d’énumération NOOP et créez noop_task.py avec NoopTask en utilisant le nom et le type de référence de tâche spécifiés ; le travail est terminé lorsque le SDK expose à la fois TaskType.NOOP et NoopTask pour construire des workflows NOOP.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- python
- Domaine
- backend
- Type d'issue
- Fonctionnalité
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Activité
- Calme
- Clarté
- Clairement spécifiée
- Accessibilité débutants
- 88/100