🛡️ Sentinel: [CRITICAL] Fix Data Integrity Loss via Negative Grep

[seonghobae]

🚨 Severity: CRITICAL
💡 Vulnerability: R의 기본 grep() 함수는 검색 결과가 없을 때 integer(0)을 반환합니다. 이를 벡터 인덱싱에서 음수 기호(-)와 함께 사용하여 요소를 제외하려고 할 때(예: x[-c(grep(...))]), 전체 벡터가 삭제되어 character(0)이 반환되는 데이터 무결성 훼손 취약점이 존재했습니다. 이로 인해 IPDParmNames 벡터의 데이터가 유실되어 하위 로직이 오작동할 위험이 있습니다.
🎯 Impact: 특정 조건 하에서 매개변수 필터링을 수행할 때 데이터가 완전히 유실되어 추후 모형 추정이나 파라미터 매핑에 치명적인 영향을 미칠 수 있습니다 (무결성 손실/기능 고장).
🔧 Fix: 여러 개의 grep() 결합을 단일 grepl()을 이용한 논리 부정(!grepl(...))으로 교체하여, 매칭되는 패턴이 없을 때에도 벡터가 온전히 보존되도록 리팩토링했습니다.
✅ Verification: 변경 사항 적용 후 R 패키지 로컬 빌드(install.packages) 및 testthat 테스트 스위트(Rscript -e "testthat::test_dir('tests/testthat')")를 실행하여 모든 항목(총 20개 테스트)이 성공적으로 통과함을 검증했습니다. 추가로 관련된 보안 이슈 사항을 .jules/sentinel.md 저널에 상세히 기록했습니다.

---

PR created automatically by Jules for task 15331737750300903575 started by @seonghobae

[google-labs-jules]

👋 Jules, reporting for duty! I'm here to lend a hand with this pull request.

When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down.

I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job!

For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with @jules. You can find this option in the Pull Request section of your global Jules UI settings. You can always switch back!

New to Jules? Learn more at jules.google/docs.

---

For security, I will only act on instructions from the user who triggered this task.

[seonghobae]

Closing as superseded by lower-number PR #111, which carries the same negative-grep data-integrity fix for IPDParmNames. Keeping this later duplicate open would create duplicate review/check churn without adding a new code path or test surface.

[google-labs-jules]

Closing as superseded by lower-number PR #111, which carries the same negative-grep data-integrity fix for IPDParmNames. Keeping this later duplicate open would create duplicate review/check churn without adding a new code path or test surface.

Understood. Acknowledging that this work is now obsolete and superseded by PR #111, and stopping work on this task.