Files
MoFin/scripts/cleanup_disabled_cron.py
T
hmo 4f83ee8a01 feat(hygiene): anti-redundancy enforcement — spec rules + weekly audit
Root cause analysis of the 2026-07-20 redundancy incident:
1. No single-source-of-truth rule -> same file legitimately lived in 4+
   locations, diverging silently
2. Relative path resolution (Path(__file__).parent/'data') -> each
   hardlinked copy of mofin_db.py pointed to a DIFFERENT database
3. 'Backup habit' left .bak/legacy files in production dirs, which
   monitoring then scanned and reported as false alarms
4. Half-done migrations: DB tables created but old JSON writers/readers
   stayed (price_events), old files stayed
5. Dead modules never got buried: xiaoguo 'dead' but bot ran 8 days
   as root eating 2.5GB
6. Monitoring checked 'does it exist' not 'is it alive' -> stale file
   mtime reported as 'pipeline stalled 14 days' (false alarm)
7. No 'system hygiene' as a check category at all

Prevention implemented:
- dev-spec.md v2.0: 五条红线 -> 十条红线
  #6 single source of truth (hardlink only, no independent copies)
  #7 absolute data paths only (no __file__-relative data resolution)
  #8 no backups/legacy in production data dirs (archive immediately)
  #9 dead module burial checklist (6 mandatory steps)
  #10 monitor liveness (DB table freshness) not existence
- File Location Constitution: canonical location per content type
- NEW system_hygiene_audit.py: weekly Monday 07:30 cron checking
  diverged copies / broken hardlinks / zombie processes / orphan data
  files / dead cron scripts / DB freshness -> hygiene_report.json + XMPP
- specs/hygiene.json: module spec per red line #1
- Verified: audit found 5 real issues on first run, all fixed, re-run clean
2026-07-20 19:04:05 +08:00

30 lines
1.2 KiB
Python

import json, shutil
REMOVE = {
'pa': ['MoFin 盘前中监控', 'MoFin 午后监控', '自选买入区提醒', '候选股自动推广-盘中'],
'default': ['持仓情报-盘中', '快速盯盘-15分钟', '价格监控-1分钟', 'PM-项目跟进'],
}
FILES = {
'pa': '/home/hmo/.hermes/profiles/position-analyst/cron/jobs.json',
'default': '/home/hmo/.hermes/cron/jobs.json',
}
for prof, jf in FILES.items():
shutil.copy(jf, jf + '.bak-20260720-disabled-cleanup')
d = json.load(open(jf))
is_list = isinstance(d, list)
jobs = d if is_list else d.get('jobs', [])
removed = [j.get('name') for j in jobs if j.get('name') in REMOVE[prof]]
kept = [j for j in jobs if j.get('name') not in REMOVE[prof]]
if removed:
if is_list:
json.dump(kept, open(jf, 'w'), ensure_ascii=False, indent=2)
else:
d['jobs'] = kept
json.dump(d, open(jf, 'w'), ensure_ascii=False, indent=2)
print(f'{prof}: removed {removed}')
# 剩余禁用任务
still_disabled = [j.get('name') for j in kept if not j.get('enabled', True)]
print(f' remaining disabled: {still_disabled}')
total = len(kept)
print(f' total jobs now: {total}')