diff --git a/docs/open-wiki-plan.md b/docs/open-wiki-plan.md index 0af4045..9c8198e 100644 --- a/docs/open-wiki-plan.md +++ b/docs/open-wiki-plan.md @@ -152,16 +152,14 @@ Owner/admin-only (`canManage`): 7. **Tests + guardrails** (rate limits, moderation hooks). 8. **E2E on this instance** — verify with real accounts on an open test wiki. -## 9. Open questions / decisions to confirm - -1. **Envelope vs. immediate-note:** recommend the submitter writes note+revision immediately plus - a `contribution` envelope (content stays on their repo). Alternative: store proposed content - only in the contribution record and materialize notes at approval — but then only the submitter - could host those records, so immediate-write is cleaner. **Confirm.** -2. **Cross-instance consistency:** pending→live is appview-local; the `contributionApproval` record - makes it portable to other instances. Scope is v1 on this instance. **Confirm acceptable.** -3. **Promotion role:** "make contributor" grants `contributor` (not admin). **Confirm.** -4. **Scope of "open":** only **public** wikis (per the issue). **Confirm.** -5. **Approvers:** owner + members with `admin` role (per the issue). **Confirm.** -6. **OAuth scope / lexicons:** confirm no extra `wiki.lichen.permissions` scope change is needed - for the two new record types (likely not, but verify with `discover_permission_sets`). \ No newline at end of file +## 9. Decisions (confirmed) + +1. **Envelope vs. immediate-note:** submitter writes note+revision to their PDS immediately plus + a `contribution` envelope. **Confirmed.** +2. **Cross-instance consistency:** v1 scoped to this instance; `contributionApproval` records are + written for future portability (other appviews may not honor pending/approval yet). **Confirmed.** +3. **Promotion role:** "Approve & make contributor" grants `contributor`. **Confirmed.** +4. **Scope of "open":** only **public** wikis. **Confirmed.** +5. **Approvers:** owner + members with `admin` role (i.e. existing `canManage`). **Confirmed.** +6. **OAuth scope / lexicons:** verify no extra `wiki.lichen.permissions` scope change is needed + for the two new record types (check with `discover_permission_sets` before the lexicon work). \ No newline at end of file