Merge pull request #374 from BhAem/fix/skill_tool

warm-wm · web-flow · commit a752bf739676 · 2025-12-16T15:04:00.000+08:00
fix: remove get_env when getting ak/sk
diff --git a/veadk/tools/builtin_tools/execute_skills.py b/veadk/tools/builtin_tools/execute_skills.py
@@ -106,8 +106,8 @@ def execute_skills(
         f"Execute skills in session_id={session_id}, tool_id={tool_id}, host={host}, service={service}, region={region}, timeout={timeout}"
     )
 
-    ak = getenv("VOLCENGINE_ACCESS_KEY")
-    sk = getenv("VOLCENGINE_SECRET_KEY")
+    ak = tool_context.state.get("VOLCENGINE_ACCESS_KEY")
+    sk = tool_context.state.get("VOLCENGINE_SECRET_KEY")
     header = {}
 
     if not (ak and sk):

Original file line number	Diff line number	Diff line change
`@@ -106,8 +106,8 @@ def execute_skills(`
`106`	`106`	`f"Execute skills in session_id={session_id}, tool_id={tool_id}, host={host}, service={service}, region={region}, timeout={timeout}"`
`107`	`107`	`)`
`108`	`108`
`109`		`- ak = getenv("VOLCENGINE_ACCESS_KEY")`
`110`		`- sk = getenv("VOLCENGINE_SECRET_KEY")`
	`109`	`+ ak = tool_context.state.get("VOLCENGINE_ACCESS_KEY")`
	`110`	`+ sk = tool_context.state.get("VOLCENGINE_SECRET_KEY")`
`111`	`111`	`header = {}`
`112`	`112`
`113`	`113`	`if not (ak and sk):`