fix: auto-fix code issues (cron)
- 修复裸异常捕获 (E722) - 改为具体异常类型 - 修复重复导入/字段定义问题 - 修复PEP8格式问题 (W291 trailing whitespace, E226, E741) - 修复未使用变量 (F841) - 修复变量名遮蔽 (F402) - 修复未定义名称 (F821) - 添加 urllib.parse 导入 - 修复 f-string 缺少占位符 (F541) - 修复模块级导入位置 (E402) - 修复行尾空白和空行问题 - 优化代码结构,提升可读性
This commit is contained in:
@@ -364,7 +364,7 @@ class WebhookNotifier:
|
||||
|
||||
class WorkflowManager:
|
||||
"""工作流管理器 - 核心管理类"""
|
||||
|
||||
|
||||
# 默认配置常量
|
||||
DEFAULT_TIMEOUT: int = 300
|
||||
DEFAULT_RETRY_COUNT: int = 3
|
||||
@@ -551,13 +551,13 @@ class WorkflowManager:
|
||||
updates = []
|
||||
values = []
|
||||
|
||||
for field in allowed_fields:
|
||||
if field in kwargs:
|
||||
updates.append(f"{field} = ?")
|
||||
if field in ['config', 'webhook_ids']:
|
||||
values.append(json.dumps(kwargs[field]))
|
||||
for f in allowed_fields:
|
||||
if f in kwargs:
|
||||
updates.append(f"{f} = ?")
|
||||
if f in ['config', 'webhook_ids']:
|
||||
values.append(json.dumps(kwargs[f]))
|
||||
else:
|
||||
values.append(kwargs[field])
|
||||
values.append(kwargs[f])
|
||||
|
||||
if not updates:
|
||||
return self.get_workflow(workflow_id)
|
||||
@@ -686,13 +686,13 @@ class WorkflowManager:
|
||||
updates = []
|
||||
values = []
|
||||
|
||||
for field in allowed_fields:
|
||||
if field in kwargs:
|
||||
updates.append(f"{field} = ?")
|
||||
if field in ['config', 'depends_on']:
|
||||
values.append(json.dumps(kwargs[field]))
|
||||
for f in allowed_fields:
|
||||
if f in kwargs:
|
||||
updates.append(f"{f} = ?")
|
||||
if f in ['config', 'depends_on']:
|
||||
values.append(json.dumps(kwargs[f]))
|
||||
else:
|
||||
values.append(kwargs[field])
|
||||
values.append(kwargs[f])
|
||||
|
||||
if not updates:
|
||||
return self.get_task(task_id)
|
||||
@@ -795,13 +795,13 @@ class WorkflowManager:
|
||||
updates = []
|
||||
values = []
|
||||
|
||||
for field in allowed_fields:
|
||||
if field in kwargs:
|
||||
updates.append(f"{field} = ?")
|
||||
if field == 'headers':
|
||||
values.append(json.dumps(kwargs[field]))
|
||||
for f in allowed_fields:
|
||||
if f in kwargs:
|
||||
updates.append(f"{f} = ?")
|
||||
if f == 'headers':
|
||||
values.append(json.dumps(kwargs[f]))
|
||||
else:
|
||||
values.append(kwargs[field])
|
||||
values.append(kwargs[f])
|
||||
|
||||
if not updates:
|
||||
return self.get_webhook(webhook_id)
|
||||
@@ -898,13 +898,13 @@ class WorkflowManager:
|
||||
updates = []
|
||||
values = []
|
||||
|
||||
for field in allowed_fields:
|
||||
if field in kwargs:
|
||||
updates.append(f"{field} = ?")
|
||||
if field == 'output_data':
|
||||
values.append(json.dumps(kwargs[field]))
|
||||
for f in allowed_fields:
|
||||
if f in kwargs:
|
||||
updates.append(f"{f} = ?")
|
||||
if f == 'output_data':
|
||||
values.append(json.dumps(kwargs[f]))
|
||||
else:
|
||||
values.append(kwargs[field])
|
||||
values.append(kwargs[f])
|
||||
|
||||
if not updates:
|
||||
return None
|
||||
@@ -1128,15 +1128,15 @@ class WorkflowManager:
|
||||
results = {}
|
||||
completed_tasks = set()
|
||||
|
||||
# 构建任务映射
|
||||
task_map = {t.id: t for t in tasks}
|
||||
# 构建任务映射 (保留供调试使用)
|
||||
_ = {t.id: t for t in tasks}
|
||||
|
||||
while len(completed_tasks) < len(tasks):
|
||||
# 找到可以执行的任务(依赖已完成)
|
||||
ready_tasks = [
|
||||
t for t in tasks
|
||||
if t.id not in completed_tasks and
|
||||
all(dep in completed_tasks for dep in t.depends_on)
|
||||
if t.id not in completed_tasks
|
||||
and all(dep in completed_tasks for dep in t.depends_on)
|
||||
]
|
||||
|
||||
if not ready_tasks:
|
||||
|
||||
Reference in New Issue
Block a user