🔧 chore: 되돌릴 근거가 사라진 @react-three/postprocessing 의존성 제거

[pcwadarong]

목적

#69에서 outline 렌더를 imperative composer로 옮기며 @react-three/postprocessing은 import 0건이 됐지만, "wrapper가 고쳐지면 되돌린다"는 전제로 의존성만 남아 있었다. 그 전제를 다시 확인해 되돌리지 않기로 확정하고, 남은 흔적을 정리한다.

	전	후
@react-three/postprocessing	dependencies에 있음 (import 0건)	제거 — n8ao, maath 트리도 같이 빠짐
코드 주석의 근거	"R3F v9 호환성 문제를 우회한다" (해소된 문제)	"composer 옵션을 직접 제어해야 해서 wrapper를 안 쓴다"
outline 렌더 경로	imperative composer	그대로

적용 범위는 의존성 1개와 주석 2줄. 렌더 동작은 건드리지 않았다.

설계

1. 되돌릴 근거였던 upstream TODO는 실제로 사라졌다

#69가 declarative wrapper를 버린 근거 중 하나가 이 주석이었다.

// @react-three/postprocessing v3.0.4 — src/EffectComposer.tsx L134
// TODO: rewrite all of this with R3F v9
const groupInstance = (group.current as Group & { __r3f: Instance<Group> }).__r3f

v3.1.1(3.1.0 2026-08-23, 3.1.1 2026-08-27)에서는 이렇게 바뀌었다.

// v3.1.1 — src/util.tsx
// Reads a <group>'s direct r3f children, filtered by type - transparent to
// non-host wrapper components, since r3f's instance tree already is.
export function readGroupChildren<T>(group, filter) { ... group.__r3f ... }

	v3.0.4 (기존 lock)	v3.1.1
__r3f 접근	인라인 + "v9로 다시 써야 함" TODO	여전히 하지만 문서화된 헬퍼
pass 등록	매 렌더 재구축	목록이 실제로 바뀔 때만 재구축
fiber peer	^9.0.0	>=9.7.0 (레포는 lock 9.5.0)

즉 "미완성이라 못 믿겠다"는 상태는 해소됐다. 근거가 유효한지부터 확인하고 결론을 낸 것이지, 결론을 먼저 정한 게 아니다.

2. 그럼에도 wrapper로 되돌리지 않는다

되돌리려면 fiber lock 9.5.0 → 9.7.0, wrapper 3.0.4 → 3.1.1, outline-effect.tsx와 그 테스트 재작성이 필요한데 사용자에게 보이는 변화는 없다. 반대로 잃는 건 명확하다.

지금 직접 잡고 있는 것	wrapper로 가면
gl.toneMapping 저장 → NoToneMapping → 언마운트 시 복구	라이브러리 기본 동작에 위임
Canvas의 antialias: true를 대체하는 multisampling: min(4, maxSamples)	기본값 8 (비용 판단이 우리 손을 떠남)
outline.dispose()를 composer.dispose()보다 먼저 호출해 object layer 복구	정리 순서를 라이브러리에 의존

v3.1.0은 "internal rewrite" 규모의 릴리스이고 나온 지 2주다. 갈아탈 이유가 없는 시점에 먼저 올라탈 이유도 없다.

3. 되돌릴 계획이 없으면 의존성도 근거도 남기지 않는다

import 0건인 패키지는 lock에서 n8ao, maath까지 끌고 온다. 되돌릴 계획이 없다고 결론 났으니 남길 이유가 없다. postprocessing(직접 의존, ^6.39.0)은 실제 사용처라 유지한다.

주석도 같이 고쳤다. "R3F v9 호환성 문제를 우회한다"를 그대로 두면 다음 사람이 이미 없는 제약을 믿고 판단하게 된다. 코드는 안 틀렸는데 근거만 낡은 자리가 제일 조용히 사람을 속인다.

Changes

해시	내용
f375957	🔧 chore: 되돌릴 근거가 사라진 @react-three/postprocessing 의존성 제거

유지 / 의도적 제외

• outline 렌더 경로: imperative composer 그대로. 위 2번 이유
• @react-three/fiber ^9.5.0: wrapper를 안 쓰기로 했으므로 9.7.0으로 올릴 이유가 이번 범위에 없다. 무관한 3D 회귀 위험만 얹는다
• postprocessing ^6.39.0: 최신 6.39.4지만 이번 결정과 무관한 범프라 섞지 않았다
• 락파일의 무관한 3줄: eslint·glob·@ungap/structured-clone의 deprecated 메시지가 갱신됐다. pnpm install --lockfile-only가 레지스트리 메타데이터를 새로 읽어와서 생긴 것으로 버전 변화는 없다

리뷰 포인트

1. 의존성 제거가 정말 안전한가 — grep -rn "react-three/postprocessing" 결과가 package.json 한 줄과 주석 한 줄뿐이었다. 빌드 설정(next.config.ts의 transpile 목록 등)에도 없다
2. wrapper 복귀를 지금 안 하는 판단 — 설계 2번의 표가 근거다. 반대 의견이 있으면 여기서
3. 바뀐 주석의 표현 — "왜 wrapper를 안 쓰는가"가 3개월 뒤에도 읽히는지
4. 락파일 diff의 메타데이터 노이즈 3줄 — 그대로 둘지, 해당 hunk만 되돌릴지

Test plan

• pnpm install --frozen-lockfile — 통과
• pnpm run check-types — 에러 없음
• pnpm run test:vitest — 전 버킷 통과 (src/features/interaction/ui/outline-effect.test.tsx 4 tests 포함)
• eslint + prettier --check (변경 파일) — 통과
• pnpm test:browser — 미실행. import 0건 패키지 제거라 런타임 경로 변화가 없다고 판단

후속

• wrapper 복귀 재검토: 지금은 안 한다. 다시 볼 조건은 (a) @react-three/fiber를 다른 이유로 9.7.0 이상으로 올리게 되고, (b) v3.1.x가 몇 릴리스 더 안정화됐을 때. 그 전엔 재검토 비용만 든다
• 참고로 남기는 upstream 발견: v3.1.0이 Outline requires <EffectComposer autoClear={false}> to render correctly. 콘솔 경고를 추가했다. wrapper의 autoClear 기본값은 지금도 true고, 이 조건에서 outline은 에러 없이 아무것도 안 그려진다. 홈 HERO 인터렉션 (카메라, 노트북) #69 당시 declarative 경로에서 outline이 안 보이던 증상과 조건이 일치한다. 당시 코드가 남아 있지 않아 단정할 수는 없지만, 언젠가 wrapper를 다시 볼 때 가장 먼저 확인할 값이다

Summary by CodeRabbit

• 문서

  • OutlineEffect 컴포넌트의 주석을 현재 구현 방식에 맞게 수정했습니다.

• 정리

  • 사용하지 않는 @react-three/postprocessing 의존성을 제거했습니다.

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated
chaen	Ready	Preview	Sep 6, 2026 2:47am UTC

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Team

Run ID: 969dca07-e57f-4d8a-be68-fc40c249f527

📥 Commits

Reviewing files that changed from the base of the PR and between dacc93c and f375957.

⛔ Files ignored due to path filters (1)

• pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml

📒 Files selected for processing (2)

• package.json
• src/features/interaction/ui/outline-effect.tsx

💤 Files with no reviewable changes (1)

• package.json

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.

---

Walkthrough

@react-three/postprocessing 의존성을 제거했습니다. OutlineEffect의 JSDoc은 toneMapping, multisampling, selection 갱신 시점을 직접 제어하는 이유를 설명하도록 변경했습니다. 런타임 동작은 변경하지 않았습니다.

Changes

OutlineEffect 유지보수

Layer / File(s)	Summary
의존성 및 JSDoc 갱신 package.json, src/features/interaction/ui/outline-effect.tsx	@react-three/postprocessing 의존성을 제거했습니다. OutlineEffect의 직접 구현 사유를 설명하는 JSDoc을 갱신했습니다.

Estimated code review effort: 1 (Trivial) | ~2분

Merge Risk: ⚪ Minimal · up to f3759

This change removes an unused postprocessing wrapper dependency and updates its rationale comment without altering the outline rendering path. No current merge-readiness risk remains.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	제목은 사용하지 않는 @react-three/postprocessing 의존성을 제거한다는 PR의 핵심 변경을 정확하고 구체적으로 요약합니다.
Description check	✅ Passed	설명은 목적, 설계 근거, 변경 범위, 유지 항목, 테스트 결과, 미실행한 브라우저 테스트, 후속 계획을 충분히 설명합니다. 템플릿의 일부 제목인 '작업 내용'과 '참고 문서/ADR'은 없고 공식 문서 링크도 없지만, 관련 정보는 대부분 다른 섹션에 포함되어 있어 설명은 대체로 완전합니다.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 1…
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch review/pr-69-outline-imperative-composer

Warning

Some tools did not complete. Review the errors below.

🔧 ESLint

If the error stems from missing dependencies, add them to the package.json file. For unrecoverable errors (e.g., due to private dependencies), disable the tool in the CodeRabbit configuration.

src/features/interaction/ui/outline-effect.tsx

ESLint skipped: missing config or dependency (missing-dependency). The ESLint configuration references a package that is not available in the sandbox.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}