fix: 同步master分支代码

[Mianhuatang8]

No description provided.

[github-actions]

Code Review Summary

本 PR 将 master 分支代码同步过来，包含多个功能新增：模板级别 Webhook 支持、子流程循环迭代、OTEL 自定义 Span 属性、权限模型扩展、Mako 模板白名单等。整体代码质量良好，以下是发现的需要关注的问题：

Critical Issues

1. broadcast_task_events 新增 data["template_id"] / data["task_id"] 访问导致 KeyError

• 文件：bkflow/space/views.py:317-319
• broadcast_task_events 被修改为访问 data["template_id"] 和 data["task_id"]，但现有调用方 bkflow/task/views.py:285 在 pause/resume/revoke 操作时发送的 data 不包含这两个字段，将导致 KeyError 使回调完全失败。
• 建议：使用 data.get("template_id") 并在值为 None 时跳过模板级别回调，或修改所有调用方统一传入 template_id。

2. get_plugin_outputs 使用 json.loads(request.body) 无异常处理

• 文件：bkflow/apigw/views/get_plugin_outputs.py:42-43
• 直接 json.loads(request.body) 后通过 data["plugin_id"] 取值，如果请求体非合法 JSON 或缺少 plugin_id，会抛出未捕获的 JSONDecodeError 或 KeyError，返回 500 而非友好错误。
• 建议：定义 Request Serializer 进行参数验证，保持与项目其他 API 一致的风格。

Important Issues

3. delete_template 事务中 raise 裸 Exception

• 文件：bkflow/apigw/views/delete_template.py:59
• 在 transaction.atomic() 中使用 raise Exception(...) 来触发回滚。虽然功能正确，但裸 Exception 会被外层 return_json_response 装饰器捕获为通用 500 错误。
• 建议：定义或复用一个业务异常类（如 ValidationError），使得错误响应更可控。

4. validate_task_id_list 在 validate 方法中做数据转换

• 文件：bkflow/apigw/serializers/task.py:231-232
• validate_task_id_list 将 List[int] 转为逗号分隔字符串 "1,2,3"。这种在校验阶段做格式转换的做法违反了 DRF 的设计原则（validate 应只做校验），使得 validated_data["task_id_list"] 的类型与字段声明 (ListField) 不一致，容易误导后续开发者。
• 建议：在 view 层或 queryset 构建时做转换，或使用 to_internal_value 方法。

5. broadcast_task_events 直接修改了入参 extra_info

• 文件：bkflow/space/views.py:318-319
• extra_info = data.get("extra_info") or 后直接 extra_info["delivery_id"] = data["task_id"]。当 data.get("extra_info") 返回的是 request.data 中的原始字典时，这会修改该可变对象，影响上面空间级别回调已使用的 extra_info。虽然 signal.send 是同步调用（空间级别先完成），但这仍是一个隐性副作用。
• 建议：extra_info = {**(data.get("extra_info") or {}), "delivery_id": data["task_id"]}

6. clear_scope_webhooks 异常处理吞掉错误仅返回 dict

• 文件：bkflow/utils/webhook.py:87-96
• 该函数在异常时返回 {"result": False, ...} 而非抛异常。但在 delete_template 中通过 if not result.get("result"): raise Exception(...) 重新触发。这种 "返回错误 dict → 检查并重新抛异常" 的模式增加了遗忘检查返回值的风险。
• 建议：统一为抛异常方式，由调用方的事务块自然处理回滚。

7. subprocess_plugin 循环逻辑中 self.inner_loop 越界风险

• 文件：bkflow/pipeline_plugins/components/collections/subprocess_plugin/v1_0_0.py:148,152
• loop_item_value = list(inputs)[self.inner_loop - 1] 和 items[self.inner_loop - 1] — 如果 self.inner_loop 大于实际列表长度，会触发 IndexError。虽然 min_loop_times 后续用来限制循环次数，但 min_loop_times 是在遍历所有 params 过程中渐进计算的，先遇到的短列表不能保护后续长列表以外的索引。
• 建议：在索引前增加边界检查，或确保 inner_loop 不超过当前参数值列表长度。

Minor Notes

• validate_loop_variables 返回 {"has_loop": True/False} 命名有歧义，has_loop=True 实际含义是 "validation passed"，建议改为 {"valid": True} 之类。
• send_task_message 新增 node_id 参数放在 task_id 和 msg_type 之间，所有现有调用方使用 kwargs 调用所以兼容性OK。
• 项目 CI workflow 安全加固（PR 作者和 sender 权限校验）是好的实践。