refactor: 생성되지 않던 TEST_RULES.md 연결, 프레임워크 규칙 중복과 미사용 코드 제거 - #4
Closed
intelligent04 wants to merge 1 commit into
Closed
refactor: 생성되지 않던 TEST_RULES.md 연결, 프레임워크 규칙 중복과 미사용 코드 제거#4intelligent04 wants to merge 1 commit into
intelligent04 wants to merge 1 commit into
Conversation
세 가지를 정리한다. 1. TEST_RULES.md가 생성되지 않았다. _render_test_rules()와 프로필별 test_rules 데이터가 이미 있었지만 render()의 출력 목록에 연결되어 있지 않아 실제 생성물에는 테스트 규칙이 없었다. 출력에 추가하고 AGENTS.md 라우팅 표에도 넣었다. 2. 프레임워크별 규칙이 세 문서에 그대로 중복됐다. 동일한 문장이 GLOBAL_RULES.md, CODE_STYLE_RULES.md, SERVICE_LAYER_RULES.md에 세 번 들어갔다. 같은 문장을 반복해도 준수율은 오르지 않고 컨텍스트 예산만 쓴다. GLOBAL_RULES.md를 단일 출처로 두고 나머지 두 문서에는 참조 문구를 남긴다. 덤으로 한국어 SERVICE_LAYER_RULES.md에 영어 라벨(Service Notes)이 섞여 있던 문제도 사라진다. 3. 호출되지 않는 렌더러 메서드 5종(208줄)을 제거했다. 결과적으로 규칙 문서가 하나 늘었는데도 총량은 줄었다. fastapi: 8파일 21,824B -> 9파일 21,772B fastapi,react: 8파일 24,636B -> 9파일 24,192B - tests/test_rule_documents.py 4건 추가. 규칙 문서 간 중복 문장 0을 강제하고 생성된 모든 규칙 문서가 AGENTS.md 라우팅 표에서 도달 가능한지 검사한다. - 기존 테스트 중 SERVICE_LAYER_RULES.md에서 프로필명을 확인하던 단정은 단일 출처인 GLOBAL_RULES.md를 보도록 옮겼다. - CODE_STYLE_RULES.md의 공백 규칙은 유지보수자 의견에 따라 그대로 둔다.
This was referenced Aug 4, 2026
Owner
|
TEST 규칙은 의도적으로 사용하지 않고 있습니다. agent의 테스트 품질이 만족스럽지 않아 별도의 프롬프트나 수기 작성으로 사용중이라 의도적인 비활성화가 맞습니다. PR은 닫아 두겠습니다 |
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
2 participants
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.
관련 이슈: #1 (3번 항목)
작업 중 발견한 것
TEST_RULES.md가 생성되지 않고 있습니다._render_test_rules()와profiles.py의 프로필별test_rules데이터는 이미 작성되어 있는데,render()가 반환하는 출력 목록에 연결이 빠져 있습니다. 그래서 생성된.codex/ai_rule_developer/에는 테스트 규칙이 전혀 들어가지 않습니다. 내용 품질은 이미 충분해서 출력에 연결만 했고,AGENTS.md라우팅 표에도테스트 → TEST_RULES.md행을 추가했습니다.의도적으로 비활성화하신 것이라면 되돌리겠습니다.
함께 정리한 것
프레임워크 규칙 중복 제거. 프로필의
framework_rules()결과가GLOBAL_RULES.md,CODE_STYLE_RULES.md,SERVICE_LAYER_RULES.md에 동일한 문장으로 세 번 들어갑니다. 같은 문장을 반복해도 준수율이 오르지는 않고 컨텍스트만 소모하므로,GLOBAL_RULES.md를 단일 출처로 두고 나머지 두 문서에는 참조 문구 한 줄을 남겼습니다.부수 효과로 한국어 출력의
SERVICE_LAYER_RULES.md에 영어 라벨[FastAPI Service Notes]가 섞이던 문제도 사라집니다.미사용 렌더러 제거. 어디서도 호출되지 않는 메서드 5종 208줄을 제거했습니다:
_render_framework_rules,_render_project_overview,_render_feature_spec,_render_api_spec,_render_db_spec. 이 중_render_framework_rules는 위 중복 문제와 같은 내용을 또 렌더링하던 것입니다. 계획 중인 용도가 있으면 알려주세요.결과
규칙 문서가 하나 늘었는데 총량은 오히려 줄었습니다.
fastapifastapi,react정정
이슈 #1의 3번 항목에서 "프로필을 조합하면 규칙이 중복 제거 없이 연결된다"고 적었는데, 실제로 측정해 보니 틀린 지적이었습니다.
fastapi와react프로필의 규칙 교집합은 0건입니다. 프로필 간 중복은 없고, 위에서 다룬 문서 간 중복만 존재합니다.검증
pytest43건 통과.tests/test_rule_documents.py4건 추가 — 규칙 문서 간 중복 문장 0을 강제하고, 생성된 모든 규칙 문서가AGENTS.md라우팅 표에서 도달 가능한지 검사합니다.SERVICE_LAYER_RULES.md에서 프로필명을 확인하던 단정은 단일 출처인GLOBAL_RULES.md를 보도록 옮겼습니다.CODE_STYLE_RULES.md의 공백 규칙과 해당 테스트는 의도된 사항이라는 답변에 따라 그대로 유지했습니다.