docs(docs): lock owner decisions on revenue-first launch (initiative 0010)#1690
Merged
Merged
Conversation
…0010)
Apply 15 owner decisions on top of revenue-first launch plan (initiative
0010 + audit-source). Lock pricing v3, mobile-strategy, auth multi-provider,
A/B onboarding; defer public metrics, hero copy, brand rename, AI-insights
pick to open questions; document FOP track as fase 3.1 risk.
Changes:
- 0010-revenue-first-launch.md
- TL;DR + Skoпe rewritten: scope freeze removed (OpenClaw active parallel),
auth multi-provider added, public metrics deferred, hero copy as
open-question
- Phase 1: 2 ADRs instead of 3 (pricing v3 + mobile-strategy; OpenClaw
park ADR removed per owner decision)
- Phase 4: new PR 4.3 — Apple + Google + Email auth via Better Auth
- Phase 5.2: A/B test goal-first vs vibe_picks 2 weeks (instead of direct
replacement), winner stays
- Phase 6: PR 6.2 (public metrics) removed; landing + EN locale stays
with hero copy as placeholder until owner finalization pre-merge
- Risks table: FOP blocker for phase 3 with T-7 deadline + owner; OpenClaw
flagged ACTIVE PARALLEL; mobile-stack deprecate DEFERRED; hero/brand
rename/AI-insights as OPEN owner-decisions
- DONE criteria + Metrics + Phases graph updated
- 2026-05-04-revenue-and-marketing-roast.md
- Update block (post-TL;DR) with owner decisions: accepted/rejected/deferred
- Audit recommendations (§3) now flagged as 'as-of original audit', superseded
by Update block
- initiatives/README.md
- Row 0010 updated to reflect locked decisions + scope drift
Verification:
- pnpm exec prettier --check (clean)
- pnpm lint:governance-sync — Errors: 64 (identical baseline to origin/main)
- pnpm docs:check-freshness-coverage — 277/277 docs with freshness header
- pnpm lint:tech-debt-freshness — green
No code changes. Pure docs delivery.
Co-Authored-By: Сон Хер <dmytro.s.stakhov@gmail.com>
Contributor
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
📝 WalkthroughWalkthroughDocumentation update capturing owner decisions for a revenue-first launch initiative: finalizing pricing v3, multi-provider authentication, activation metrics, and restructuring the deployment phases while deferring openClaw freeze and public metrics dashboard. ChangesRevenue-First Launch Initiative Update
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Possibly related PRs
Suggested labels
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Review rate limit: 0/10 reviews remaining, refill in 59 minutes and 8 seconds. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@docs/initiatives/0010-revenue-first-launch.md`:
- Line 4: Replace the non‑allowed lifecycle value used in the docs badge: locate
the "Status:" badge string (the line that currently reads "Status: Proposed")
and change "Proposed" to one of the allowed enum values — "Active",
"Scaffolded", "Deprecated", or "Archived" (choose the one matching this
initiative's real lifecycle, e.g., "Scaffolded" if it's newly planned).
- Around line 344-368: The fenced diagram block is missing a language tag which
triggers MD040; update the triple-backtick fence surrounding the phases graph
(the ``` block shown) to include a language identifier (e.g., ```text) so the
Markdown linter treats it as plain text; locate the diagram's opening ``` and
change it to ```text (and keep the closing ``` unchanged).
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro Plus
Run ID: 7ac4022a-0aa6-4cfc-b73d-5227737a5c74
📒 Files selected for processing (3)
docs/audits/2026-05-04-revenue-and-marketing-roast.mddocs/initiatives/0010-revenue-first-launch.mddocs/initiatives/README.md
| > **Last validated:** 2026-05-04 by @sonher468. **Next review:** 2026-08-02. | ||
| > **Status:** Proposed (план PR-ів; перший PR — цей документ + аудит-сорс) | ||
| > **Last validated:** 2026-05-04 by @Skords-01. **Next review:** 2026-08-02. | ||
| > **Status:** Proposed (decisions locked, scope final; перший PR — цей документ + аудит-сорс + owner-decisions) |
There was a problem hiding this comment.
Use an allowed lifecycle status value in the badge.
Line 4 uses Status: Proposed, but this docs lifecycle badge is expected to use the defined enum values. Please switch it to one of Active, Scaffolded, Deprecated, or Archived.
Suggested fix
-> **Status:** Proposed (decisions locked, scope final; перший PR — цей документ + аудит-сорс + owner-decisions)
+> **Status:** Scaffolded (decisions locked, scope final; перший PR — цей документ + аудит-сорс + owner-decisions)As per coding guidelines, docs/**/*.md: Lifecycle markers in documentation: declare status (Active, Scaffolded, Deprecated, Archived) in a status badge under the freshness marker.
📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
Suggested change
| > **Status:** Proposed (decisions locked, scope final; перший PR — цей документ + аудит-сорс + owner-decisions) | |
| > **Status:** Scaffolded (decisions locked, scope final; перший PR — цей документ + аудит-сорс + owner-decisions) |
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@docs/initiatives/0010-revenue-first-launch.md` at line 4, Replace the
non‑allowed lifecycle value used in the docs badge: locate the "Status:" badge
string (the line that currently reads "Status: Proposed") and change "Proposed"
to one of the allowed enum values — "Active", "Scaffolded", "Deprecated", or
"Archived" (choose the one matching this initiative's real lifecycle, e.g.,
"Scaffolded" if it's newly planned).
Comment on lines
344
to
368
| ``` | ||
| Фаза 0 (docs) | ||
| Фаза 0 (docs — цей PR) | ||
| │ | ||
| ├─→ Фаза 1.1 (ADR pricing) ─┐ | ||
| ├─→ Фаза 1.2 (ADR openclaw) │ | ||
| └─→ Фаза 1.3 (ADR mobile) │ | ||
| ↓ | ||
| Фаза 2 (data layer) | ||
| │ | ||
| ↓ | ||
| Фаза 3 (stripe e2e) | ||
| │ | ||
| ↓ | ||
| Фаза 4 (paywall UI) | ||
| │ | ||
| Фаза 5.1 (activation v2) ───┤ | ||
| ↓ | ||
| Фаза 5.2 (onboarding pivot) | ||
| │ | ||
| ↓ | ||
| Фаза 6 (landing + metrics) | ||
| ├─→ Фаза 1.1 (ADR pricing v3) ─┐ | ||
| └─→ Фаза 1.2 (ADR mobile-strategy) │ | ||
| ↓ | ||
| Фаза 2 (data layer) | ||
| │ | ||
| ↓ | ||
| Фаза 3 (stripe e2e) | ||
| │ | ||
| ├─→ Фаза 4.1 (paywall + plan hooks) | ||
| ├─→ Фаза 4.2 (real /pricing) | ||
| └─→ Фаза 4.3 (Apple/Google/Email auth) ──┐ | ||
| │ | ||
| Фаза 5.1 (activation v2 metric) ─────────────────────────────────────┐ │ | ||
| ↓ ↓ | ||
| Фаза 5.2 (A/B onboarding) | ||
| │ | ||
| ↓ | ||
| Фаза 6 (landing + EN) | ||
|
|
||
| Ankle-PR (поза фазами 1–6, scope: chore): | ||
| chore-console-move-to-tools ← apps/console/ → tools/console/ | ||
| ``` |
There was a problem hiding this comment.
Add a language identifier to the phases graph code fence.
Line 344 opens a fenced block without a language, which triggers MD040 and adds avoidable lint noise.
Suggested fix
-```
+```text
Фаза 0 (docs — цей PR)
│
├─→ Фаза 1.1 (ADR pricing v3) ─┐
└─→ Фаза 1.2 (ADR mobile-strategy) │
↓
Фаза 2 (data layer)
│
↓
Фаза 3 (stripe e2e)
│
├─→ Фаза 4.1 (paywall + plan hooks)
├─→ Фаза 4.2 (real /pricing)
└─→ Фаза 4.3 (Apple/Google/Email auth) ──┐
│
Фаза 5.1 (activation v2 metric) ─────────────────────────────────────┐ │
↓ ↓
Фаза 5.2 (A/B onboarding)
│
↓
Фаза 6 (landing + EN)
Ankle-PR (поза фазами 1–6, scope: chore):
chore-console-move-to-tools ← apps/console/ → tools/console/</details>
<details>
<summary>🧰 Tools</summary>
<details>
<summary>🪛 markdownlint-cli2 (0.22.1)</summary>
[warning] 344-344: Fenced code blocks should have a language specified
(MD040, fenced-code-language)
</details>
</details>
<details>
<summary>🤖 Prompt for AI Agents</summary>
Verify each finding against the current code and only fix it if needed.
In @docs/initiatives/0010-revenue-first-launch.md around lines 344 - 368, The
fenced diagram block is missing a language tag which triggers MD040; update the
triple-backtick fence surrounding the phases graph (the block shown) to include a language identifier (e.g.,text) so the Markdown linter treats it
as plain text; locate the diagram's opening and change it totext (and
keep the closing ``` unchanged).
</details>
<!-- fingerprinting:phantom:triton:hawk:ec00939c-19bc-4e9d-b33a-bbb0aeaea4b8 -->
<!-- d98c2f50 -->
<!-- This is an auto-generated comment by CodeRabbit -->
⏱️ CI Pipeline Duration ReportBased on the last 50 successful runs on the default branch. Overall Pipeline
Trend (last 20 runs): Per-Job Breakdown
|
10 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Apply 15 owner decisions on top of revenue-first launch plan that landed in #1673. Lock pricing v3, mobile-strategy, auth multi-provider, A/B onboarding; defer public metrics, hero copy, brand rename, AI-insights pick to open questions; document ФОП track as фаза 3.1 risk with T-7 deadline.
Changes (docs-only, scope
docs):docs/initiatives/0010-revenue-first-launch.md— TL;DR + Скоуп rewritten; Phase 1 → 2 ADRs (pricing v3 + mobile-strategy, OpenClaw park ADR removed); Phase 4 → +PR 4.3 (Apple/Google/Email auth via Better Auth); Phase 5.2 → A/B test goal-first vs vibe_picks 2 weeks; Phase 6 → PR 6.2 (public metrics) removed; Risks table → ФОП blocker T-7, OpenClaw ACTIVE PARALLEL, mobile-stack deprecate DEFERRED, hero/rename/AI-insights as [OPEN].docs/audits/2026-05-04-revenue-and-marketing-roast.md— Update block (post-TL;DR) with owner decisions: accepted/rejected/deferred.docs/initiatives/README.md— Row 0010 updated.No code changes. Pure docs delivery.
Governing Skill
.agents/skills/sergeant-feature-delivery/SKILL.md.agents/skills/sergeant-review-and-merge/SKILL.mdPlaybook
Verification
Additional checks:
Docs and Governance
AGENTS.mdneeded an update — no, нові скоупи / hard rules не вводяться.Updated docs:
docs/initiatives/0010-revenue-first-launch.mddocs/audits/2026-05-04-revenue-and-marketing-roast.mddocs/initiatives/README.mdRisk and Rollout
git revert— initiative повернеться до v1 (state у docs(docs): plan revenue-first launch (initiative 0010 + roast audit) #1673).Hard Rule #15
AGENTS.mdbefore coding.--no-verify.Reviewer Notes
{HERO_PLACEHOLDER}+ кандидати у §Ризики. Owner вибирає фінал перед merge PR 6.1, не у цьому PR.subscriptionsmigration). Якщо не встигаємо — фази 3+ затримуються до оформлення..agents/skills/better-auth-best-practices/SKILL.md.vibe_picksна A/B 50/50 на 2 тижні; decision rule — winner stays, loser cleanup-PR./api/public/metricsможе бути доданий окремою ініціативою пізніше.apps/console→tools/console/— окремий ankle-PR (scopechore), поза фазами 1–6, ~1 година.Summary by cubic
Locks owner decisions for initiative 0010 and updates the revenue‑first launch plan: pricing v3, mobile strategy, Apple/Google/Email auth, and A/B onboarding; defers public metrics and clarifies risks. Docs‑only update; no code changes.
docs/initiatives/0010-revenue-first-launch.md: locks pricing v3 ($7/mo, $49/yr, ₴ UA‑only, trial without card), sets mobile strategy (Capacitor primary, Expo paralleled), adds Phase 4.3 for multi‑provider auth via Better Auth, shifts Phase 5.2 to a 2‑week A/B test, removes public metrics, adds ФОП blocker (T‑7) and marks OpenClaw as active parallel; updates DONE, metrics, and phase graph; notesapps/console→tools/console/ankle‑PR.docs/audits/2026-05-04-revenue-and-marketing-roast.md(accepted/rejected/deferred).docs/initiatives/README.mdto reflect the locked plan.Written for commit bf97ed0. Summary will update on new commits.
Summary by CodeRabbit