refactor(graphify): upstream /graphify 단일 진입점 전환 — 레거시 Graphify 레이어 제거#35
Open
pko89403 wants to merge 6 commits into
Open
refactor(graphify): upstream /graphify 단일 진입점 전환 — 레거시 Graphify 레이어 제거#35pko89403 wants to merge 6 commits into
pko89403 wants to merge 6 commits into
Conversation
Remove the entire repo-local Graphify orchestration layer: Scripts removed: - graphify_code_refresh.sh, graphify_prepare_corpus.sh - graphify_full_refresh.py, graphify_semantic_adapter.py - graphify_verify_full_refresh.py, graphify_sync_staged.sh - graphify_ci_candidate.sh Hooks removed: - graphify-auto-refresh.sh, graphify-pretool.sh, graphify-mode-note.sh - Claude/Codex hook registrations for Graphify CI removed: - graphify-full-refresh-candidate job Tests removed: - test_graphify_auto_refresh.py, test_graphify_producer.py - test_graphify_workflow.py Artifacts removed: - graphify-out/BUILD_INFO.json - build_info_full_refresh.json, full_refresh_graph.json fixtures Docs updated: - AGENTS.md, CLAUDE.md, README.md, README.ko.md now upstream-only - .graphifyignore is the canonical corpus boundary - Eval fixtures migrated from full_refresh to document_context - Graph artifacts regenerated (883 nodes, 2349 edges, 33 communities) All 148 tests pass. ruff/mypy clean. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Delete ephemeral artifacts from git: VERIFY_FULL_REFRESH.json, manifest.json, cost.json, .graphify_chunk_01.json - Add graphify-out ephemeral files to .gitignore - Remove dangling --ignore=test_graphify_auto_refresh.py from pre-push.sh - Replace "staged Graphify full-refresh" with upstream wording in SPEC.md Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…eminder - graphify hook install: AST rebuild on every commit (no LLM cost) - post-checkout: rebuild on branch switch - Custom raw/ reminder: prints update suggestion when docs/papers change Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Codex hook API only supports Bash matcher, so upstream graphify emits awareness on every Bash call. This script parses stdin JSON and fires only for grep/rg/find/fd-class commands, matching Claude's Glob|Grep semantics while staying within Codex's API constraint. Reverts English upstream section appended to CLAUDE.md by `graphify codex install`; folds only the new rule (graphify query/path/explain) into the existing Korean 레이어 2 section. Warns against re-running the upstream installer in local-adaptation.md since it would overwrite the repo-local script reference. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Post-commit hook rebuilt graph.json/graph.html/GRAPH_REPORT.md after the Codex awareness hook commit. 875 nodes, 1946 edges, 34 communities. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
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.
한 줄 요약
/graphify스킬을 유일한 Graphify 모델로 전환합니다.배경 / 문제
/graphify와 중복·충돌/graphify를 single source of truth으로,.graphifyignore로 corpus boundary 제어PR 대시보드
main변경 묶음별 리뷰 가이드
.agents/skills/graphify/SKILL.md.githooks/post-commit,.githooks/post-checkoutgraphify hook install+raw/리마인더 커스텀scripts/graphify_*.{sh,py},scripts/hooks/graphify-*.shgit diff --stat.agents/skills/graphify-{manager,full}/CLAUDE.md,AGENTS.md,README.md,README.ko.md등.claude/settings.json,.codex/hooks.json,.github/workflows/ci.ymlpretool-block-graphify-out.sh) 유지 확인tests/test_graphify_harness.pyscripts/graphify_eval.py,tests/test_graphify_eval_*.py.codex/hooks.json,scripts/hooks/pretool-graphify-awareness.sh,CLAUDE.md,.agents/policies/local-adaptation.mdgraphify codex install의 inline 명령을 repo-local 스크립트로 추출. stdin JSON을 파싱해grep|rg|find|fd계열 Bash에만 awareness emitgraphify codex install재실행 금지 안내graphify-out/graph.json,GRAPH_REPORT.md,graph.htmlBefore / After
Before
code_update/full_refresh모드 분리, BUILD_INFO trust signal, staged verify → sync 흐름After
/graphify단일 진입점 (/graphify .,/graphify . --update).graphifyignore로 corpus boundary 제어pretool-block-graphify-out.shguard hook만 유지 (에이전트 직접 편집 방지, Claude 전용 — Codex는apply_patch경로가 hook API 밖이라 원천 불가)graphify hook install기반 자동화:post-commit: 코드 변경 → AST 자동 리빌드 (LLM 비용 없음) +raw/변경 시 리마인더 출력post-checkout: 브랜치 전환 → 자동 리빌드.codex/hooks.jsonPreToolUse Bash matcher →scripts/hooks/pretool-graphify-awareness.shgrep/rg/find/fd계열만 필터링해 Claude의Glob|Grepmatcher와 기능적 시맨틱 parity 확보 (Codex API가 Bash-only로 제한됨을 우회)local-adaptation.md에 "graphify codex install재실행 금지" 운영 규칙 명시 — 재실행 시 repo-local 스크립트 경로가 upstream inline command로 덮어써짐.gitignore에 추가검증
uv run pytest— 148 passed (최초 143, eval + awareness 추가 후 148)uv run ruff check .uv run mypy src— 30 filesraw/변경 커밋 시 리마인더 출력 확인grep/rg/find/fd/subshell/pipe → EMIT,ls/git/uv run pytest/ungrep/invalid JSON → SILENT (12/12)scripts/hooks/pre-push.sh내 143 test gate 통과남은 리스크 / 후속 작업
--update필요 (hook이 리마인더로 안내)CONFLICTING)graphify codex install재실행 금지. upstream awareness 메시지 내용 변경은scripts/hooks/pretool-graphify-awareness.sh를 수동으로 갱신해 반영.apply_patch/Write/Edit은 hook 범위 밖. 따라서 Claude의pretool-block-graphify-out.sh와 동등한 파일 편집 차단은 Codex 측에서 불가. AGENTS.md(=CLAUDE.md) 지시 준수에 의존.graphify-out/graph.jsondiff가 크지만 upstream 재생성 결과이므로 내용 리뷰 불필요raw/리마인더 로직, guard hook 유지 여부, awareness hook의 정규식 word-boundary 처리