Skip to content

fix: Harness hook 추론과 runtime surface 검증 강화#96

Merged
alexization merged 2 commits into
developfrom
fix/harness-hook-runtime-guards
Apr 16, 2026
Merged

fix: Harness hook 추론과 runtime surface 검증 강화#96
alexization merged 2 commits into
developfrom
fix/harness-hook-runtime-guards

Conversation

@alexization

Copy link
Copy Markdown
Owner

Summary

  • completed task를 닫은 뒤에도 unpushed diff가 해당 scope에 매핑되면 pre-push가 올바른 task/phase context로 latest verification gate를 평가하도록 수정했습니다.
  • init가 runtime surface를 source 기준으로 다시 동기화하고, doctor.githooks/*workflows/system/hooks.json drift를 실패로 감지하도록 보강했습니다.
  • no-op pre_phase_start와 미사용 dangerous_cmd_guard.allowlist를 제거하고 관련 문서, skills, 회귀 테스트를 정리했습니다.

Linked Issue

How

  • CLI hook 진입부에서 changed path를 먼저 수집한 뒤 pre_push 전용 task inference를 수행하도록 순서를 바꿨습니다.
  • pre_push는 active task뿐 아니라 completed task까지 scope 매핑 후보에 포함하고, 단일 task로 해석되지 않으면 fail-closed 하도록 했습니다.
  • .githooks/pre-push는 shell 단계에서 active task를 먼저 박아 넣지 않고 CLI inference에 맡기도록 단순화했습니다.
  • initdoctor의 책임을 분리해 source surface copy와 content drift validation을 명시적으로 추가했습니다.
  • dead hook surface 제거에 맞춰 docs/skills/tests를 함께 갱신했습니다.

Validation Summary

  • python3 -m unittest discover -s tests -v 통과
  • python3 scripts/workflow.py doctor 통과
  • python3 scripts/workflow.py status --all --check 통과
  • git diff --check 통과
  • completed task scope pre-push, unrelated active task 공존, runtime surface drift 검출/재동기화 회귀 테스트를 추가했습니다.

Reviewer Focus

  • pre_push task inference가 completed task와 active task가 섞인 상황에서도 의도대로 fail-closed 하는지
  • init/doctor 분리가 외부 workspace install 흐름과 충돌하지 않는지
  • dead surface 제거 후 docs와 runtime contract가 일치하는지

Impact / Risks

  • 새 라이브러리, 외부 서비스, 스키마, 설정, 환경 변수, 마이그레이션: 없음
  • 사용자나 운영에 영향이 있으면 적어 주세요: runtime surface drift가 있던 workspace는 init 재실행이 필요할 수 있습니다.
  • 배포, 롤백, 커뮤니케이션 시 주의점이 있으면 적어 주세요: multi-task diff branch는 이제 더 명시적인 task binding이 필요할 수 있습니다.
  • 남아 있는 리스크와 후속 작업: multi-task branch 정책은 별도 운영 규칙으로 정리할 여지가 있습니다.

- completed task scope 기준 pre-push 검증과 fail-closed task 추론 보강
- init과 doctor에 githook 및 hooks.json 동기화와 drift 검출 추가
- dead hook surface 제거와 문서 및 회귀 테스트 정리
@alexization alexization changed the title [codex] Harness hook 추론과 runtime surface 검증 강화 fix: Harness hook 추론과 runtime surface 검증 강화 Apr 16, 2026
@alexization alexization marked this pull request as ready for review April 16, 2026 04:14

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 0da1167012

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread scripts/workflow_runtime/engine.py Outdated
@alexization alexization merged commit 6b24ffc into develop Apr 16, 2026
1 check passed
@alexization alexization deleted the fix/harness-hook-runtime-guards branch April 16, 2026 04:25
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