mirror of
https://github.com/openharmony/ability_ability_runtime.git
synced 2026-08-24 12:43:16 -04:00
0dc85f4e8c
JsUIAbility::ExecuteSkill and JsServiceExtension::ExecuteSkill had four silent early-return paths (null napi env, function not found, napi_call failure, and a previously uncaught pending exception) that left the AMS record in EXECUTING forever after the skill-timeout PR canceled the launch timer, hanging the caller. Reuse ERR_TIMED_OUT as the result code to keep CLI exitCode behavior identical to the pre-timeout baseline and avoid growing the public error-code surface; the per-path detail rides in SkillExecuteResult.result (WantParams) under a reserved key SKILL_ERROR_MSG_KEY. The CLI side (ToolUtil::BuildSkillSessionInfo) now extracts that message into ExecResult.errorText, which is already wired through IPC and JS but was never populated for skills. Also removes the stale "skill will time out" log comment that the timeout PR invalidated. Co-Authored-By: Agent Signed-off-by: RuiChen_01 <chenrui193@huawei.com> 🤖 AI[100%] 👌 AI Adopted[100%] 🧑 Human[0%] Co-authored-by: claude (glm-5.2) <ai@local>