Skip to content

[-]: Promise API 추가#8

Merged
sjsjsj1246 merged 1 commit into
mainfrom
codex/add-promise-api
Mar 24, 2026
Merged

[-]: Promise API 추가#8
sjsjsj1246 merged 1 commit into
mainfrom
codex/add-promise-api

Conversation

@sjsjsj1246
Copy link
Copy Markdown
Owner

변경 사항

  • tutorial.open()Promise<{ reason: 'completed' | 'skipped' | 'closed' }>를 반환하도록 추가했습니다.
  • 마지막 step 완료, built-in 건너뛰기, 외부 close, 새 tutorial open 시 기존 pending 정리까지 resolve reason을 구분했습니다.
  • README, docs, 계획 문서를 async 사용 예제와 현재 상태에 맞게 업데이트했습니다.

테스트

  • pnpm -C packages/main test
  • pnpm build
  • pnpm -C packages/document build

남은 리스크

  • Promise 결과 타입은 tutorial.open() 반환형으로는 노출되지만 별도 named export 타입은 아직 제공하지 않습니다.
  • 새 tutorial open 시 이전 pending을 closed로 정리하는 정책을 택했으므로, 중첩 tutorial을 기대하는 소비자가 있다면 동작 변화를 확인해야 합니다.

@vercel
Copy link
Copy Markdown

vercel Bot commented Mar 24, 2026

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

Project Deployment Actions Updated (UTC)
react-tutorial-overlay Ready Ready Preview, Comment Mar 24, 2026 2:54pm

@github-actions
Copy link
Copy Markdown
Contributor

packages/main coverage

Metric Result
Lines 96.71% (206/213)
Statements 96.44% (217/225)
Functions 95.23% (40/42)
Branches 84.31% (129/153)

Workflow run

@sjsjsj1246 sjsjsj1246 self-assigned this Mar 24, 2026
@sjsjsj1246 sjsjsj1246 merged commit 882e3db into main Mar 24, 2026
3 checks passed
@sjsjsj1246 sjsjsj1246 deleted the codex/add-promise-api branch March 24, 2026 14:55
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