Skip to content

fix: reduce output noise and improve behavior summary prioritization#13

Merged
3rdflr merged 1 commit intomainfrom
feat/noise-reduction
Feb 23, 2026
Merged

fix: reduce output noise and improve behavior summary prioritization#13
3rdflr merged 1 commit intomainfrom
feat/noise-reduction

Conversation

@3rdflr
Copy link
Owner

@3rdflr 3rdflr commented Feb 23, 2026

Summary

  • 불완전 라인 필터링 강화: ) / ] 로 시작하는 조각 라인 제거, 닫힌 괄호 불균형도 필터링
  • 빈 섹션 제거: 내용이 없는 ✏️ 변경됨 심볼 섹션 숨김
  • 동작 요약 우선순위 버킷: 단순 캡 → 카테고리별 제한으로 교체
    • Tier 1 (state) / (API): 최대 4개 — 핵심 데이터 흐름
    • Tier 2 (guard) / (catch): 최대 2개 — 안전/에러 경계
    • Tier 3 (cond): 최대 2개 — 분기 로직
    • Tier 4 (setState) / useEffect / (return): 최대 2개 — 부수효과
  • 일반 함수 호출 제거: 의미 없는 레이블 없는 foo() 항목 제거

Test plan

  • npx github-mobile-reader --repo owner/repo --pr N 실행 후 불완전 라인 미출력 확인
  • 내용 없는 ✏️ 변경됨 섹션 미출력 확인
  • 긴 함수에서 항목이 카테고리별로 적절히 제한되는지 확인

- isSyntacticallyIncomplete: reject lines starting with ) or ] (continuation fragments)
  and treat both open and close unbalanced parens as incomplete
- generateSymbolSections: skip modified symbols with no content lines (empty sections)
- buildBehaviorSummary: replace flat 8-item cap with priority bucket system
  Tier 1 (state/API): max 4 — core data flow, always shown first
  Tier 2 (guard/catch): max 2 — safety/error boundaries
  Tier 3 (cond): max 2 — branching logic
  Tier 4 (setState/useEffect/return): max 2 — side effects
- Remove generic function call capture (callMatch) — was producing noisy unlabeled entries
@github-actions
Copy link
Contributor

PR #13 — fix: reduce output noise and improve behavior summary prioritization

3rdflr/github-mobile-reader · 1f450a3 · JS/TS 1개 파일 변경


src/parser.ts

✏️ isSyntacticallyIncomplete (Function) — 변경됨

  • (guard) parens !== 0 || brackets !== 0 → early return
  • (guard) parens !== 0 || brackets < 0 → early return

✏️ buildBehaviorSummary (Function) — 변경됨

generateSymbolSections (Function) — 새로 추가
변수: normalized, init, initLabel, argLabel, hookMatch 외 9개

  • (cond) lines.length === 0 && sym.status === "modified"

🛠 Auto-generated by github-mobile-reader. Do not edit manually.

@3rdflr 3rdflr merged commit 9695f20 into main Feb 23, 2026
1 check passed
3rdflr added a commit that referenced this pull request Feb 24, 2026
…13)

- isSyntacticallyIncomplete: reject lines starting with ) or ] (continuation fragments)
  and treat both open and close unbalanced parens as incomplete
- generateSymbolSections: skip modified symbols with no content lines (empty sections)
- buildBehaviorSummary: replace flat 8-item cap with priority bucket system
  Tier 1 (state/API): max 4 — core data flow, always shown first
  Tier 2 (guard/catch): max 2 — safety/error boundaries
  Tier 3 (cond): max 2 — branching logic
  Tier 4 (setState/useEffect/return): max 2 — side effects
- Remove generic function call capture (callMatch) — was producing noisy unlabeled entries
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