fix: 단어 마지막에 괄호가 있다면 괄호 앞의 단어로 조사를 결정한다

[jw-r]

Overview

issue #32

• josa, josaPicker가 완전한 괄호로 끝나는 문자열을 첫 번째 매개변수로 받는다면 괄호 앞의 단어로 조사를 결정하도록 수정
• 테스트 케이스를 작성

PR Checklist

• I read and included theses actions below

1. I have read the Contributing Guide
2. I have written documents and tests, if needed.

[changeset-bot]

⚠️ No Changeset found

Latest commit: 4c4714a

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

💥 An error occurred when fetching the changed packages and changesets in this PR

Some errors occurred when validating the changesets config:
The package or glob expression "es-hangul" specified in the `fixed` option does not match any package in the project. You may have misspelled the package name or provided an invalid glob expression. Note that glob expressions must be defined according to https://www.npmjs.com/package/micromatch.

[netlify]

✅ Deploy Preview for es-hangul ready!

Name	Link
🔨 Latest commit	4c4714a
🔍 Latest deploy log	https://app.netlify.com/sites/es-hangul/deploys/661a392dd0721d0009e80876
😎 Deploy Preview	https://deploy-preview-34--es-hangul.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[evan-moon]

@jw-r 소중한 첫 기여 감사합니다! ☺️
혹시 괄호에 대한 처리가 josa 함수의 관심사라고 판단하신 이유가 있을까요???

개인적으로 josa 함수는 명사와의 문법 관계를 다루는 것에만, 괄호와 같은 특별한 맥락에 대한 처리는 함수 바깥에서 별도로 처리해주는 것이 이 함수의 관심사에 더 적합하지 않을까...하는 생각이 들었습니다!

const text = '샴푸(보습)';
const josaForText = josa(text.replace(/(.*\)/, ''); // '가';
console.log(`${text}${josaForText}`); // '샴푸(보습)가'

이 PR을 확인하신 다른 분들의 의견도 궁금합니다.

[jw-r]

@jw-r 소중한 첫 기여 감사합니다! ☺️ 혹시 괄호에 대한 처리가 josa 함수의 관심사라고 판단하신 이유가 있을까요???

개인적으로 josa 함수는 명사와의 문법 관계를 다루는 것에만, 괄호와 같은 특별한 맥락에 대한 처리는 함수 바깥에서 별도로 처리해주는 것이 이 함수의 관심사에 더 적합하지 않을까...하는 생각이 들었습니다!

const text = '샴푸(보습)';
const josaForText = josa(text.replace(/(.*\)/, ''); // '가';
console.log(`${text}${josaForText}`); // '샴푸(보습)가'

이 PR을 확인하신 다른 분들의 의견도 궁금합니다.

안녕하세요!

리뷰 감사합니다🤩

josa 함수 사용에 있어 보다 사용자의 기대에 부합하는 결과를 반환할 수 있겠다고 생각했습니다.

하지만 말씀하신대로 josa 함수의 역할을 고려할 때 괄호 처리를 내부에서 수행하는 것은 함수의 관심사에서 벗어나는 것 같습니다.

특히 불완전한 괄호에 대한 처리 등 다양한 엣지케이스가 생겨 @raon0211 의 우려가 현실이 될 것 같네요🤣

제시해주신 것처럼 사용하는 측에서 괄호를 관리하도록 유도하는 것이 더 적합할 수 있다는 점에 동의합니다.

리뷰어분들의 판단에 따라 이번 PR은 close해도 좋을 것 같습니다!

감사합니다!

[evan-moon]

@jw-r

의사결정 감사합니다! 그렇다면 이번 PR은 Close 하도록 하겠습니다! 추후 이슈레이징해주신 사례가 많이 늘어나게 되면 다시 한번 논의해봐요 😄