Skip to content

fix: merge _get_hermes_home() dynamic resolution and feishu receive_id_type detection#13725

Closed
dengtaoyuan450-a11y wants to merge 1 commit into
NousResearch:mainfrom
dengtaoyuan450-a11y:hermes-merge-fix
Closed

fix: merge _get_hermes_home() dynamic resolution and feishu receive_id_type detection#13725
dengtaoyuan450-a11y wants to merge 1 commit into
NousResearch:mainfrom
dengtaoyuan450-a11y:hermes-merge-fix

Conversation

@dengtaoyuan450-a11y

Copy link
Copy Markdown

修复内容

scheduler.py 动态解析

  • 模块级 _hermes_home → 动态函数 _get_hermes_home()
  • 模块级 _LOCK_DIR/_LOCK_FILE → 动态函数 _get_lock_paths()
  • 所有引用点已更新(scripts_dir、.env、config.yaml、prefill_file)

feishu.py receive_id_type 检测

  • 添加 receive_id_type 动态检测(oc_/ou_open_id,其他 → chat_id
  • 修复飞书 API [230001] ext=invalid receive_id 错误

关联任务

  • P0: 拉取远程最新代码 + 合并本地修改
  • P1: webhook direct delivery + wakeAgent gate (已就绪)

@alt-glitch alt-glitch added type/bug Something isn't working comp/cron Cron scheduler and job management platform/feishu Feishu / Lark adapter labels Apr 21, 2026
…d_type detection

- scheduler.py: Replace static _hermes_home with dynamic _get_hermes_home() function
  to support profile switching at runtime (HERMES_HOME override)
- scheduler.py: Replace static _LOCK_DIR/_LOCK_FILE with _get_lock_paths() function
  for profile-aware lock path resolution
- feishu.py: Add receive_id_type detection (oc_/ou_ -> open_id, else chat_id)
  to fix Feishu API '[230001] ext=invalid receive_id' error for user DMs
teknium1 added a commit that referenced this pull request May 5, 2026
After PR #13725 replaced the module-level _LOCK_DIR/_LOCK_FILE constants
with a dynamic _get_lock_paths() helper, the xdist-isolation fixture
needs to patch the function instead of the removed constants.
RationallyPrime pushed a commit to RationallyPrime/hermes-agent that referenced this pull request May 8, 2026
After PR NousResearch#13725 replaced the module-level _LOCK_DIR/_LOCK_FILE constants
with a dynamic _get_lock_paths() helper, the xdist-isolation fixture
needs to patch the function instead of the removed constants.
rmulligan pushed a commit to rmulligan/hermes-agent that referenced this pull request May 11, 2026
After PR NousResearch#13725 replaced the module-level _LOCK_DIR/_LOCK_FILE constants
with a dynamic _get_lock_paths() helper, the xdist-isolation fixture
needs to patch the function instead of the removed constants.
JinyuID pushed a commit to JinyuID/hermes-agent that referenced this pull request May 11, 2026
After PR NousResearch#13725 replaced the module-level _LOCK_DIR/_LOCK_FILE constants
with a dynamic _get_lock_paths() helper, the xdist-isolation fixture
needs to patch the function instead of the removed constants.
jsboige pushed a commit to jsboige/hermes-agent that referenced this pull request May 14, 2026
After PR NousResearch#13725 replaced the module-level _LOCK_DIR/_LOCK_FILE constants
with a dynamic _get_lock_paths() helper, the xdist-isolation fixture
needs to patch the function instead of the removed constants.
gweeteve pushed a commit to gweeteve/hermes-agent that referenced this pull request Jun 2, 2026
After PR NousResearch#13725 replaced the module-level _LOCK_DIR/_LOCK_FILE constants
with a dynamic _get_lock_paths() helper, the xdist-isolation fixture
needs to patch the function instead of the removed constants.
Egavasyug pushed a commit to Egavasyug/hermes-agent that referenced this pull request Jun 10, 2026
After PR NousResearch#13725 replaced the module-level _LOCK_DIR/_LOCK_FILE constants
with a dynamic _get_lock_paths() helper, the xdist-isolation fixture
needs to patch the function instead of the removed constants.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/cron Cron scheduler and job management platform/feishu Feishu / Lark adapter type/bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants