docs(docs): refresh storage-roadmap.md Stage 7 status (PRs #054a/b/c/x landed)#2074
Merged
Skords-01 merged 1 commit intoMay 6, 2026
Merged
Conversation
…x landed) - Header 'Last validated': add #054a/b/c/x landed PRs (#2058, #2066, #2072, #2073) - Stage 7 status table: 4/4 → 7/9; outstanding = #53 + #54 final - §3 PR plans: add 4 landed cleanup subsections (#054a budget 10→6, #054b supersedes-edge + 12 dangling cloudSync v1 doc-refs, #054c mutation.md dangling refs, #054x ADR-0049 README parity fix-forward) - §3 PR #53 KVStore deprecate stub → detailed plan with 2026-05-06 audit (web 7 prod consumers, mobile 26 prod consumers, 6 storage primitives) + sub-PR breakdown (a/b/c) + done criteria - §3 PR #54 final stub → detailed path (PR #53 deprecate → SQLite warm-cache takes hub-layout/UI prefs → 6 storage primitives become OPFS+SQLite shim → allowlist = []) + done criteria + dependency on PR #53 Hard Rule #15 (docs alongside code) — closes the gap left after PRs #2058, #2066, #2072, #2073 landed without updating the canonical roadmap doc. No code changes; pure docs refresh. Co-Authored-By: dmytro.s.stakhov <dmytro.s.stakhov@gmail.com>
Contributor
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Caution Review failedPull request was closed or merged during review 📝 WalkthroughWalkthroughUpdates ChangesStorage Roadmap Documentation
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~5 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)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Skords-01
deleted the
devin/1778090981-storage-roadmap-stage7-status-refresh
branch
Contributor
⏱️ 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
Refresh
docs/planning/storage-roadmap.mdto reflect Stage 7 cleanup progress: 4 PRs landed since the last roadmap update on 2026-05-06 (PR #054a/#2058, #054b/#2066, #054c/#2072, #054x/#2073) but the canonical roadmap doc still showedStage 7: 4/4and 1-line stubs for #53 / #54 final. This PR brings the doc back in sync (Hard Rule #15) and expands the two outstanding PR plans with concrete audit numbers + done criteria. Pure docs change, no code surface touched.Governing Skill
managing-storage-migration(storage-roadmap canonical doc keeper)governance-sync(Hard Rule Claude/review project structure l2 ke1 #15 enforcement)Playbook
docs/playbooks/storage-roadmap-update.md(does not exist as a discrete playbook; this is the de-facto playbook for the storage migration owner)docs/planning/storage-roadmap.mdis the canonical source-of-truth for the SQLite migration; whenever a roadmap PR lands, the doc must be refreshed in the same release cycle (Hard Rule Claude/review project structure l2 ke1 #15). When the matching landed PRs are docs-only governance fixes (like chore(ci): drop stale cloudSync entries from localStorage allowlist #2058/docs(docs): prune dangling cloudSync v1 source refs (Hard Rule #15) #2066/docs(docs): prune dangling refs to retired docs/testing/mutation.md (PR #054c) #2072/docs(docs): add ADR-0049 row to ADR README index (Hard Rule #15 fix) #2073), the refresh is also docs-only.Verification
Additional checks:
Docs and Governance
AGENTS.mdneeded an update. (No change — Stage 7 cleanup is a no-op for AGENTS hard-rules.)Updated docs:
docs/planning/storage-roadmap.mdLast validated: now lists #054a/b/c/x landed PRs with commit hashes.4/4→7/9; outstanding =#053 + #054 final.Risk and Rollout
git revert 6e23628e— single-commit revert is safe (no schema / code coupling).Hard Rule #15
AGENTS.mdbefore coding.storage-roadmap.mdwhich itself is mixed; no English-only doc was switched to Ukrainian or vice versa.)--no-verify.Audit-freeze (until 2026-06-02)
Reviewer Notes
077c738f):Critical-flow E2E (Playwright)— pre-existing on main.Migration lint (AGENTS rule)—Duplicate migration numbers: 047(two047_*migrations on main:047_billing_subscriptions.sqlfrom PR docs(docs): prefix Done/Closed initiatives with_for ls-visibility #2061 stripe billing +047_tg_topic_archive.sqlfrom another in-flight branch). Pre-existing on main; this PR has zero migration files.Test coverage (vitest)/Visual regression (Argos)/check— running but unrelated to docs.eslint allowlist = []).