Skip to content

feat: 튜토리얼 후속 개선 9건 — 카피 사실화·데모 깨짐 수정·/가이드 튜토리얼 연결 - #54

Merged
mindongdong merged 3 commits into
mainfrom
feat/tutorial-copy-fixes
Jul 3, 2026
Merged

feat: 튜토리얼 후속 개선 9건 — 카피 사실화·데모 깨짐 수정·/가이드 튜토리얼 연결#54
mindongdong merged 3 commits into
mainfrom
feat/tutorial-copy-fixes

Conversation

@mindongdong

@mindongdong mindongdong commented Jul 3, 2026

Copy link
Copy Markdown
Owner

요약

PR #53 튜토리얼 라이브 후 접수된 개선 요청 9건 반영. 봇 코드를 직접 검증해 실동작과 다른 카피를 전부 사실 기준으로 재작성했다.

스크린별 변경

스크린 변경
1 인트로 디스코드 데모 깨짐 수정 — 본문 text-align:center 상속으로 명령 칩이 중앙 정렬되던 것 차단 + 400px 고정높이에서 임베드가 제목째 잘리던 것을 높이 460px·상단 페이드 마스크로 해소 (랜딩 히어로는 무변경)
2 초대 "관리자 권한은 안 받아요" → "권한은 봇이 메시지를 보내기 위한 최소한의 권한만 설정했어요"
4 등록 없이 /가이드 언급·칩 제거, 알림 3종 설명 실동작 정정(공지=일정 시간마다 확인 후 발송 · 경험치=매일 오전 10시 · 썬데이=매주 금요일 오전 10시 10분, notification/scheduler.py 검증)
5 캐릭터 등록 데모 캐릭터명 무기콤보 → 메이트
6·9 열리는 문 스샷 콜라주 → video 슬롯(src null·콜라주 폴백). 영상 제공 시 media.src만 채우면 무한루프 자동재생(기존 재생 규격: autoPlay·muted·loop·playsInline)
7 API 키 폼 목업 3행 → getting-started와 동일 7행, 카피 교체 + openapi.nexon.com 하이퍼링크
10 사용 기조 "안 적으면 내 기준" 오류 정정 → "레벨 상위 최대 10명과 비교" (후속 커밋에서 팬아웃 상한 K=10 확정 반영 — 상한 구현은 별도 PR, 대상 지정은 최대 5명·/스펙 지정 필수) · 예시 /스펙 → /유니온
11 총정리 나만 봐요에 알림 토글 3종(켜기/끄기 label), 다같이 봐요에 발송물 3종(broadcasts 종 칩) 추가

  • /가이드 링크 버튼: 명령어 페이지 → 5분 튜토리얼(maplemate.site/tutorial), 라벨 "🎓 5분 튜토리얼 보기"

가드

  • 드리프트 가드(mjs·pytest) 정규식에 label 필드 허용, _EXPECTED_VISIBILITY에 알림 토글 3종(private) 추가, 작업지시서 §3.1 정본 갱신

테스트

  • pytest 전체 799 passed (가이드·드리프트 가드 포함)
  • npm run build 그린 · check-command-drift.mjs 30개 참조 검증 통과
  • 헤드리스 크롬: 스크린 1 애니 단계별(0.8s/4.2s/8s) + 변경 스크린 8종 데스크톱 + 1·7·11 모바일(500px) 눈확인
  • ruff check·format 통과
  • 배포 후 /가이드 버튼 → 튜토리얼 이동 라이브 1회 확인

가이드 명령이 사실상 링크 안내만 하므로 목적지를 온보딩 효과가 큰
인터랙티브 튜토리얼(maplemate.site/tutorial)로 바꾼다.
COMMANDS_URL → TUTORIAL_URL, 버튼 라벨 '📖 명령어 전체 보기' → '🎓 5분 튜토리얼 보기'.
- S1 디스코드 데모 깨짐 수정: text-align 중앙 상속 차단 + 높이 460px·상단
  페이드 마스크(임베드가 제목째 잘리던 것을 지나간 대화 연출로)
- S2 초대 카피: 관리자 권한 문구 → 메시지 발송 최소 권한 안내
- S4 /가이드 언급 제거(링크 안내 명령이라 '되는 것' 목록에서 부적절),
  알림 3종 설명을 실제 발송 방식으로 정정(공지=주기 확인·경험치=매일
  10시·썬데이=금요일 10시 10분, notification/scheduler.py 기준)
- S5 등록 데모 캐릭터명 무기콤보 → 메이트
- S6·S9 열리는 문 스샷 → video 슬롯(src null·기존 콜라주 폴백, 영상
  제공 시 무한루프 자동재생은 기존 Media 규격 그대로)
- S7 넥슨 폼 목업 3행 → getting-started와 동일 7행, openapi.nexon.com
  하이퍼링크(sub를 ReactNode로 확장)
- S10 무인자 기조 정정: '내 기준' → '서버 등록자 전원 비교'(resolve_targets
  실동작, 상한 미구현·/스펙만 1~5명), 예시 /스펙 → /유니온
- S11 총정리: 알림 토글 3종을 나만 봐요(label '켜기/끄기'), 발송물 3종을
  다같이 봐요(broadcasts 종 칩)로 추가
- 드리프트 가드 2곳 label 필드 허용 + _EXPECTED_VISIBILITY 알림 3종,
  작업지시서 §3.1 정본 갱신
@vercel

vercel Bot commented Jul 3, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
maple-mate Ready Ready Preview, Comment Jul 3, 2026 6:50pm

무인자 전원 비교는 가독성·조회 속도상 위험해 상한 K=10 구현이 확정됨(별도 PR).
카피가 '전원·제한 없음'으로 배포되지 않도록 선보정.
@mindongdong
mindongdong merged commit 8533708 into main Jul 3, 2026
5 checks passed
@mindongdong
mindongdong deleted the feat/tutorial-copy-fixes branch July 3, 2026 19:13
mindongdong added a commit that referenced this pull request Jul 3, 2026
* feat: 무인자 비교 팬아웃 경계 — 대표 레벨 상위 10명 상한 (ADR-0008)

무인자 실행(등록자/키등록자 전원)은 PNG 가 인원만큼 세로로 자라고 넥슨
전역 스로틀(4req/s) 아래서 응답이 인원에 비례해 느려진다 → 경계 도입.

- Target·HistoryTarget 에 level 필드(대표 레벨, 쿼리 무변경 — 대표 해석
  시 이미 메모리에 있던 값)
- comparison.cap_by_level: 레벨 내림차순 상위 10명, 동레벨·미상(None=-1)은
  닉네임 가나다 타이브레이크. MAX_UNSPECIFIED_TARGETS=10 단일 소스
- /아이템·/유니온·/스타포스·/잠재 무인자에 적용 + 잘리면 푸터 안내 1줄
  (10명 이하 서버는 화면 무변경). 대상 지정은 파라미터 5슬롯이 기존 상한
- 이력류 무인자는 '키 미등록' 행 숨김(상한 슬롯 잠식 방지) — 지정 시 유지
- 수집 경로(get_targets/get_history_targets 자체)는 무변경 — /경험치
  스냅샷·백필 무영향
- 명령 description·파라미터 설명 문구 동기화, 테스트 11건(tests/test_fanout_cap.py)

* docs: ADR-0008 팬아웃 경계 기록 + 사이트 명령 요약 동기화

- ADR-0008: K=10·레벨 정렬·이력류 키 미등록 숨김·수집 경로 무변경 결정 기록
  (과거 결정이 미머지 브랜치에서 유실됐던 번호를 정식 채움 — ADR-0009 각주 참조)
- commands.json 아이템·유니온·스타포스·잠재 요약에 상한 문구 추가
  (튜토리얼 S10 은 PR #54 에서 선보정)
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