Skip to content

fix: preserve hermes hook indentation on re-bootstrap#16

Merged
cafitac merged 1 commit intomainfrom
fix/hermes-hook-approval-merge-regression
Apr 27, 2026
Merged

fix: preserve hermes hook indentation on re-bootstrap#16
cafitac merged 1 commit intomainfrom
fix/hermes-hook-approval-merge-regression

Conversation

@cafitac
Copy link
Copy Markdown
Owner

@cafitac cafitac commented Apr 27, 2026

요약

  • Hermes user-scope bootstrap 재실행 시 compact YAML hooks indentation을 보존하도록 수정했습니다
  • quoted/unquoted command 표기 차이에서도 hook 중복 삽입을 막도록 merge 조건을 보강했습니다
  • live에서 재현된 regression을 installer test로 고정했습니다

테스트

  • PYTHONPATH=src .venv/bin/python -m pytest -q
  • npm test
  • live ~/.hermes 기준 bootstrap --adapters hermes 재실행 검증
  • live ~/.hermes 기준 hermes hooks doctor healthy 확인
  • live ~/.hermes 기준 hermes chat --accept-hooks -q 'Say OK only' --quiet 확인

- preserve compact YAML hook list indentation during user-scope hook merge\n- avoid duplicate hook command insertion across quoted/unquoted command styles\n- cover the live regression with installer tests
@cafitac cafitac merged commit 0c4b4d8 into main Apr 27, 2026
6 checks passed
@cafitac cafitac deleted the fix/hermes-hook-approval-merge-regression branch April 27, 2026 16:33
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