feat: 하네스 control plane을 구조화된 SDD 런타임으로 고도화#94
Merged
Conversation
- spec 승인 시 structured intake와 소크라테스 Q/A/Decision 검증 추가 - init, doctor, hook runtime으로 자동 검증 경로와 guard 강제력 보강 - AGENTS, docs, skills, tests를 새 workflow 계약에 맞춰 정리
- .githooks/pre-push를 실행 가능 상태로 수정 - git push 시 local hook이 실제로 동작하도록 정리
- pre-push wrapper에서 pre_command를 항상 먼저 검사 - active task가 없으면 verification guard를 생략하고 종료
- git-ranker 포인터를 fc85948b까지 반영 - backend unit test baseline과 서비스 계층 테스트 도입 커밋을 연결
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 6d23d05dad
ℹ️ 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".
- git push -f와 short force flag를 dangerous command guard에서 차단 - task_id 경로 검증을 추가해 task 디렉터리 path traversal을 방지 - 관련 CLI 회귀 테스트를 추가
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
approve -> plan -> run -> verify -> review흐름의 강제력을 높였습니다.spec.md의 소크라테스 질문 로그를Q/A/Decision형식으로 검증하고, 승인 시task.json.intake에 구조화된 intake를 잠그도록 바꿨습니다..githooks, workflow runtime, doctor, docs, skills, tests를 함께 정리해 로컬 자동 검증과 문서 계약이 같은 규칙을 가리키도록 맞췄고,git-ranker서브모듈 포인터도fc85948b까지 올렸습니다.Linked Issue
How
scripts/workflow_runtime/를 추가해 CLI, state machine, validation, guard, doctor 로직을 분리하고workflow.py를 안정적인 entrypoint로 고정했습니다.approve,plan,status에 structured intake, spec readiness, spec/intake sync 검증을 추가했습니다.init가core.hooksPath=.githooks를 맞추고,doctor가 hooksPath와 핵심 문서 marker를 검사하도록 강화했습니다.AGENTS.md,docs/*, workflow-local skills, GitHub template, ignore rule을 새 contract에 맞게 재작성하고 불필요한 legacy 문서/skill/spec tree를 제거했습니다..githooks/pre-push를 보정해 active task가 없을 때는 dangerous command check만 수행하고 일반 브랜치 push를 막지 않도록 했습니다.git-rankergitlink를 backendorigin/develop의fc85948b로 갱신해 최신 backend baseline을 함께 가리키도록 맞췄습니다.Validation Summary
python3 scripts/workflow.py initpython3 scripts/workflow.py doctorpython3 scripts/workflow.py status --checkpython3 -m unittest discover -s tests -v.githooks/pre-push origin feat/93-harness-runtime-hardeninggit diff --cached --checkgit diff --cached --submodule=log -- git-rankerReviewer Focus
Q/A/Decision검증이 사용자가 원하는 SDD/Socratic 승인 흐름을 충분히 강제하는지 봐 주세요.AGENTS.md,docs/runtime.md,docs/runbook.md, skill 문서가 같은 workflow 계약을 일관되게 설명하는지 봐 주세요..githooks/pre-push,init,doctor조합이 로컬 자동 검증 경로를 실제로 보장하는지 봐 주세요.git-ranker서브모듈 포인터가 의도한 backend 커밋fc85948b를 정확히 가리키는지 봐 주세요.Impact / Risks
docs/specs,docs/operations,docs/architecture, legacy skill 경로를 참조하던 개인 메모나 외부 문서는 업데이트가 필요하고, submodule checkout 사용자도git submodule update --init --recursive가 필요할 수 있습니다.doctor의 문서 marker 기반 검증은 최소 정합성 검증이므로, 향후 runtime contract가 더 복잡해지면 더 구체적인 문서/skill drift 테스트가 필요할 수 있습니다.