feat: support update node weight

[leavesster]

No description provided.

[coderabbitai]

Note

Other AI code review bot(s) detected

CodeRabbit has detected other AI code review bot(s) in this pull request and will avoid duplicating their findings in the review comments. This may lead to a less comprehensive review.

Summary by CodeRabbit

• 新功能
  • 支持动态调整节点权重，提升资源分配灵活性。
• 重构
  • 引入统一的内部接口以简化上下文交互。
  • 集中化随机ID生成，减少重复实现。
• 可靠性
  • 增强参数校验，避免无效权重导致的请求失败。

Walkthrough

在 oocana/oocana/internal.py 新增 InternalAPI 类与 random_string 函数；InternalAPI.update_node_weight(node_id, weight) 校验权重为非负且为有限数，生成 request_id，构建 action 为 "UpdateNodeWeight" 的 BlockRequest 并通过 Mainframe 客户端发送；在 oocana/oocana/context.py 引入 InternalAPI 与 random_string，并在 Context 初始化时挂载 self.internal: InternalAPI。

Changes

Cohort / File(s)	Summary
Internal API 与辅助函数 oocana/oocana/internal.py	新增 InternalAPI 类（接收 Mainframe 客户端与 JobDict）；新增异步方法 `update_node_weight(node_id: str, weight: int
Context 集成 oocana/oocana/context.py	移除模块内本地 random_string 实现，改为从 .internal 导入 random_string 与 InternalAPI；在 Context 构造中创建并挂载 self.internal: InternalAPI = InternalAPI(mainframe, blockInfo.job_info())。

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Pre-merge checks

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Description Check	❓ Inconclusive	PR 描述为空，未提供关于更改内容或目的的任何说明，因此无法判断描述是否充分或与变更集相关，属于信息不足的情况。	请补充简短描述，说明主要改动（如新增 InternalAPI、update_node_weight、random_string 以及 Context.internal 的初始化）和变更目的或使用场景，以便审查者快速理解并完成代码审查。

✅ Passed checks (1 passed)

Check name	Status	Explanation
Title Check	✅ Passed	标题使用允许的格式（type: subject），仅用英文且准确概括了主要改动——为节点权重更新添加支持（新增 InternalAPI.update_node_weight），因此与变更集一致并通过命名规范检查。

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between c75296d and dc7d4ca.

📒 Files selected for processing (1)

• oocana/oocana/internal.py (1 hunks)

🧰 Additional context used

🧬 Code graph analysis (1)

oocana/oocana/internal.py (2)

oocana/oocana/mainframe.py (1)

• send (89-92)

oocana/oocana/data.py (1)

• JobDict (28-30)

🪛 Ruff (0.13.1)

oocana/oocana/internal.py

21-21: Avoid specifying long messages outside the exception class

(TRY003)

---

32-32: Standard pseudo-random generators are not suitable for cryptographic purposes

(S311)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)

• GitHub Check: check
• GitHub Check: check

🔇 Additional comments (5)

oocana/oocana/internal.py (5)

1-6: 导入结构清晰，模块依赖合理。

导入的模块都是必要的，依赖关系也很合理。

---

7-12: 类设计合理，初始化逻辑正确。

InternalAPI 类的设计很好地封装了内部 API 功能，构造函数参数类型正确。

---

13-22: 权重验证逻辑完整且正确。

验证逻辑正确地检查了权重的类型、有限性和非负性，错误消息也很清晰。异步方法设计为未来扩展做好了准备。

---

23-29: 确认 send 方法的同步/异步调用是否正确。

根据之前的审查意见，需要确认 self._client.send() 是否为异步方法。如果是异步方法，应该使用 await 调用。

请运行以下脚本验证 send 方法的定义：

#!/bin/bash
# 检查 Mainframe.send 方法的定义，确认是否为异步方法
ast-grep --pattern $'class Mainframe {
  $$$
  def send($$$) {
    $$$
  }
  $$$
}'

ast-grep --pattern $'class Mainframe {
  $$$
  async def send($$$) {
    $$$
  }
  $$$
}'

---

31-32: 考虑使用加密安全的随机数生成器。

random_string 函数使用 random.choices，这不是密码学安全的。对于可能用于安全敏感上下文的请求 ID，建议使用 secrets.token_urlsafe() 或 secrets.choice()。

+import secrets
+
-def random_string(length=8):
-    return ''.join(random.choices(string.ascii_letters + string.digits, k=length))
+def random_string(length=8):
+    return ''.join(secrets.choice(string.ascii_letters + string.digits) for _ in range(length))

或者更简单的方式：

+import secrets
+
-def random_string(length=8):
-    return ''.join(random.choices(string.ascii_letters + string.digits, k=length))
+def random_string(length=8):
+    return secrets.token_urlsafe(length)[:length]

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[Copilot]

Pull Request Overview

This PR introduces a new InternalAPI class that provides functionality to update node weights in the system. The implementation includes input validation and generates random request IDs for operations.

• Adds new InternalAPI class with update_node_weight method
• Implements weight validation to ensure non-negative numeric values
• Includes utility function for generating random string IDs

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}