Skip to content

fix(marketplace): owner.email를 valid 형식으로 채움 (closes #7)#14

Open
hongsw wants to merge 1 commit intorevfactory:mainfrom
hongsw:fix/marketplace-owner-email
Open

fix(marketplace): owner.email를 valid 형식으로 채움 (closes #7)#14
hongsw wants to merge 1 commit intorevfactory:mainfrom
hongsw:fix/marketplace-owner-email

Conversation

@hongsw
Copy link
Copy Markdown

@hongsw hongsw commented May 1, 2026

Closes #7

문제

marketplace.jsonowner.email이 빈 문자열(\"\")이라 일부 런타임이 마켓플레이스 추가를 거부합니다. #7 보고대로 Copilot CLI는 다음 오류로 실패:

```
오류: Invalid marketplace.json: owner.email: Invalid email
```

Claude Code는 빈 문자열을 허용해 영향이 가시화되지 않았지만, 양 런타임 호환을 위해 valid한 이메일 형식이 필요합니다.

변경

```diff

  • "email": "",

GitHub 표준 noreply 형식 (`+@users.noreply.github.com`)을 사용했습니다 — 개인 이메일을 노출하지 않으면서 valid한 형식. 메인테이너님께서 다른 이메일을 선호하시면 같은 위치만 교체하시면 됩니다.

검증

  • `python3 -m json.tool` 통과
  • RFC-style email regex 매치
  • `@users.noreply.github.com` 표준 패턴

영향

  • Copilot CLI 등 strict-validation 런타임에서 마켓플레이스 추가 가능
  • Claude Code: 무영향 (이미 동작 중이던 케이스)
  • 기능/플러그인 동작: 무변경

Copilot CLI 등 일부 런타임은 marketplace.json의 `owner.email`이 비어
있으면 'Invalid email' 검증 오류로 마켓플레이스 추가에 실패한다.
Claude Code는 빈 문자열을 허용하지만 양 런타임 호환을 위해 valid한
형식이 필요하다.

revfactory 개인 이메일을 노출하지 않도록 GitHub 표준 noreply 형식
(`<id>+<login>@users.noreply.github.com`)을 채워둔다. 메인테이너가
다른 이메일로 바꾸길 원하면 같은 위치만 교체하면 된다.

검증:
- python3 -m json.tool 통과
- RFC-style email regex 매치
- @users.noreply.github.com 패턴

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
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.

[feat] Add Owner Email

1 participant