feat: add ASCA ECC bundle#4
Open
ecc-tools[bot] wants to merge 14 commits into
Open
Conversation
…A-instincts.yaml)
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
12 tasks
jlinsights
added a commit
that referenced
this pull request
May 1, 2026
…14) Group B (DataLoader mock 3건) + 부수 enum 1건 = +4 PASS. Root cause: - user query: production resolver가 DataLoader.userLoader.load() 사용, batchLoadFn은 db.query.users.findMany 호출. test mock은 findFirst만. - 부수 발견: UserRole enum이 'member' (lowercase) reject → 'MEMBER' - members query: MemberStatus enum 'active' → 'ACTIVE' - exhibitions query: ExhibitionStatus 'CURRENT' invalid → 'ONGOING' (실제 enum: UPCOMING/ONGOING/COMPLETED/CANCELLED) 검증 (mini-do): - 9 fail → 7 fail (Query Operations 4 PASS) - 27 tests 중 20 PASS / 7 fail 잔여 7건은 4 별 사이클 (Plan §2.1): - C tests-route-authz (1) - D tests-route-schema-mock (2 잔여, mutation) - E tests-route-auth-header (1) - F tests-route-error-policy (3, formatted error 추가) 부모 사이클: tests-stale-route-extras (archived) 계획서: docs/01-plan/features/tests-route-dataloader-mock.plan.md (rev β #4)
jlinsights
added a commit
that referenced
this pull request
May 1, 2026
본 사이클은 rev β 패턴의 4연속 적용. DataLoader mock 함정 + GraphQL enum casing 강제 fix로 Query Operations 3 GREEN. CI 효과는 mini-do 절반 (+4 vs +2) — cross-test isolation 가능성, 별 사이클 'tests-ci-isolation' 후보. 핵심 산출물: - 1 file 4 hunks (DataLoader findMany + 3 enum casing) - main 효과: passed 393 → 395 (+2) - 7 PDCA chain 누적: passed 228 → 395 (+167, +73%) 학습 (메모리 권장): - feedback_dataloader_mock: production이 DataLoader 사용 시 batchLoadFn은 findMany 호출 → test mock에 findFirst만 정의 시 silent fail - feedback_graphql_enum_casing: GraphQL enum UPPER_SNAKE_CASE 강제 - rev β #4 검증 (Match 평균 97.5%, ejection 0, mini-do→CI 첫 변동)
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
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
Auto-generated ECC bundle from repository analysis.
What This Does
Merging this PR adds repo-local ECC artifacts for both Claude Code and Codex. The generated bundle captures repository patterns, Codex baseline config, and reusable workflow scaffolds derived from git history analysis.
Analysis Scope
Files
.claude/ecc-tools.json.claude/skills/ASCA/SKILL.md.agents/skills/ASCA/SKILL.md.agents/skills/ASCA/agents/openai.yaml.claude/identity.json.codex/config.toml.codex/AGENTS.md.codex/agents/explorer.toml.codex/agents/reviewer.toml.codex/agents/docs-researcher.toml.claude/homunculus/instincts/inherited/ASCA-instincts.yaml.claude/commands/pdca-feature-delivery-cycle.md.claude/commands/security-fix-implementation.md.claude/commands/post-implementation-autoformat.mdOptional: Continuous Learning (29 instincts)
This PR also includes instincts for the continuous-learning-v2 skill. These are optional and only useful if you use that skill.
Import after merging:
Review Checklist
ECC Tools | Everything Claude Code