Skip to content

fix: 修复 Binance 心跳通知 + 运行时崩溃问题#84

Merged
Pigbibi merged 1 commit into
mainfrom
fix/heartbeat-imports-and-event-filter
Jun 29, 2026
Merged

fix: 修复 Binance 心跳通知 + 运行时崩溃问题#84
Pigbibi merged 1 commit into
mainfrom
fix/heartbeat-imports-and-event-filter

Conversation

@Pigbibi

@Pigbibi Pigbibi commented Jun 29, 2026

Copy link
Copy Markdown
Collaborator

问题

Binance 平台心跳好几天没有正常通知,根因是两个问题叠加:

1. 运行时崩溃(6/29 引入)

cd4b99a 新增的 enrich_btc_snapshot_with_cycle_indicators 函数使用了 jsonPath 但没有 import,导致运行时 NameError 崩溃。

2. 心跳无法检测运行时(6/27 引入)

3036c42 将 Runtime workflow 的触发方式从纯 workflow_dispatch 改成了 workflow_run(CI完成后触发),但心跳脚本的 GitHub API 查询仍然只过滤 event=workflow_dispatch,导致心跳永远找不到 CI 触发的 runtime runs。

修复

  • main.py: 添加 import jsonfrom pathlib import Path
  • scripts/runtime_workflow_heartbeat.py: 移除两处 GitHub API 查询中的 event=workflow_dispatch 过滤,让心跳能检测到所有事件类型(workflow_dispatch + workflow_run)的 runtime runs

🤖 Generated with Claude Code

…ilter in heartbeat

- main.py: add  and  that were
  missing after the BTC cycle indicators feature (cd4b99a), causing
  NameError crash in enrich_btc_snapshot_with_cycle_indicators
- runtime_workflow_heartbeat.py: remove  filter
  from both GitHub API calls so the heartbeat can detect Runtime
  workflow runs triggered by workflow_run (CI completion) in addition
  to manual workflow_dispatch

Co-Authored-By: Claude <noreply@anthropic.com>
@Pigbibi Pigbibi force-pushed the fix/heartbeat-imports-and-event-filter branch from 9ce6e31 to c827234 Compare June 29, 2026 00:34
@Pigbibi Pigbibi merged commit 7e18721 into main Jun 29, 2026
1 check failed
@Pigbibi Pigbibi deleted the fix/heartbeat-imports-and-event-filter branch June 29, 2026 00:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant