docs(docs): record 0011 PR 2.8 (DataState adoption — digest)#1729
Merged
Conversation
Status header: додано PR 2.8 (#1726) і змінено сигнал з "2.8 HubChat/coach/digest залишається" на "2.9 ESLint rule і 2.1 ManualExpenseSheet залишаються" — consumer-adoption блок Phase 2 закрито. Phase 2 table row 2.8: ETA "+3 дні" → "Opened 2026-05-04 — #1726", файл `core/insights/WeeklyDigestCard.tsx` `DigestContent` 4-state ladder як єдиний Skeleton-based panel-loading site у HubChat / coach / digest зоні `core/**`. Footnote: додано пояснення per PR 2.8 чому інші HubChat / coach / digest "панелі" з пропозиції (`HubChatHistoryPanel`, `CoachInsightsPanel`, `DigestPanel`) не мали реальних DataState- targets — `AssistantAdviceCard` без skeleton imports і кешує last-good insight, `HubChatHistoryDrawer` local-first, `HubChat.tsx` / `HubChatBody` / `HubChatComposer` стрімлять без panel-skeleton-у. Refs initiative 0011 PR 2.8.
Contributor
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
📝 WalkthroughWalkthroughThis PR updates the initiative document 0011 by adding validation timestamps, refreshing Phase 1/2 progress tracking in the Status section, and completing the Phase 2.4–2.8 consumer-adoption table with the missing PR 2.8 entry and expanded implementation details for Skeleton-based loading targets. ChangesInitiative Documentation Update
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)
Review rate limit: 2/10 reviews remaining, refill in 42 minutes and 18 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: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@docs/initiatives/0011-foundation-adoption-and-process-discipline.md`:
- Around line 3-5: Add a lifecycle badge directly under the freshness line that
currently reads "Last validated: 2026-05-04 by `@Skords-01`. Next review:
2026-08-02." — insert a badge-style status indicator (one of Active, Scaffolded,
Deprecated, Archived) on its own line immediately below that freshness marker so
the document shows "Lifecycle: Active" (or the appropriate status) in badge
form; target the same block containing the "Last validated" / freshness marker
in 0011-foundation-adoption-and-process-discipline.md and ensure the badge is
visually separated (its own line) and matches the project's docs badge style.
🪄 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: 09282156-b7e7-4870-92b3-eae211223eaa
📒 Files selected for processing (1)
docs/initiatives/0011-foundation-adoption-and-process-discipline.md
Comment on lines
3
to
5
| > **Last validated:** 2026-05-04 by @Skords-01. **Next review:** 2026-08-02. | ||
| > **Status:** Phase 1 ~70% done (1.1/1.2/1.3 merged; 1.4 pending). Phase 2 in flight: 2.2 merged (#1696); 2.4 (#1703) + 2.5 (#1709) + 2.6 (#1713) + 2.7 (#1714) opened 2026-05-04 (DataState consumer adoption — finyk + fizruk + nutrition + routine; 2.8 HubChat/coach/digest залишається). | ||
| > **Status:** Phase 1 ~70% done (1.1/1.2/1.3 merged; 1.4 pending). Phase 2 in flight: 2.2 merged (#1696); 2.4 (#1703) + 2.5 (#1709) + 2.6 (#1713) + 2.7 (#1714) + 2.8 (#1726) opened 2026-05-04 (DataState consumer adoption — finyk + fizruk + nutrition + routine + digest closes the consumer-adoption block; 2.9 ESLint rule і 2.1 ManualExpenseSheet залишаються). | ||
| > **Priority:** P1 (subordinate to 0010-revenue-first-launch scope-freeze) |
There was a problem hiding this comment.
Add required lifecycle status badge under the freshness marker.
Please add a badge-style lifecycle marker directly under the freshness line (e.g., Active, Scaffolded, Deprecated, or Archived) to satisfy docs governance for lifecycle signaling.
As per coding guidelines, “Lifecycle markers in documentation: declare status (Active, Scaffolded, Deprecated, Archived) in a status badge under the freshness marker”.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@docs/initiatives/0011-foundation-adoption-and-process-discipline.md` around
lines 3 - 5, Add a lifecycle badge directly under the freshness line that
currently reads "Last validated: 2026-05-04 by `@Skords-01`. Next review:
2026-08-02." — insert a badge-style status indicator (one of Active, Scaffolded,
Deprecated, Archived) on its own line immediately below that freshness marker so
the document shows "Lifecycle: Active" (or the appropriate status) in badge
form; target the same block containing the "Last validated" / freshness marker
in 0011-foundation-adoption-and-process-discipline.md and ensure the badge is
visually separated (its own line) and matches the project's docs badge style.
⏱️ CI Pipeline Duration ReportBased on the last 50 successful runs on the default branch. Overall Pipeline
Trend (last 20 runs): Per-Job Breakdown
|
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
2 participants
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
Записує прогрес PR 2.8 (DataState consumer adoption — HubChat / coach / digest зона) у
docs/initiatives/0011-foundation-adoption-and-process-discipline.md. PR 2.8 (#1726) — це останній у блоці DataState consumer-adoption Phase 2 (після 2.4/2.5/2.6/2.7 для finyk/fizruk/nutrition/routine).Зміни:
2.8 (#1726)до списку opened DataState-PR-ів. Сигнал змінено з "2.8 HubChat/coach/digest залишається" на "2.9 ESLint rule і 2.1 ManualExpenseSheet залишаються" — consumer-adoption блок Phase 2 закрито.core/insights/WeeklyDigestCard.tsxDigestContent4-state ladder — єдиний Skeleton-based panel-loading site у HubChat / coach / digest зоніcore/**.HubChatHistoryPanel,CoachInsightsPanel,DigestPanel) не мали реальних DataState-targets —AssistantAdviceCardбез skeleton imports і кешує last-good insight,HubChatHistoryDrawerlocal-first,HubChat.tsx/HubChatBody/HubChatComposerстрімлять без panel-skeleton-у.Governing Skill
.agents/skills/sergeant-review-and-merge/SKILL.md(governance/docs freshness).agents/skills/sergeant-feature-delivery/SKILL.md(initiative tracking)Playbook
Verification
Additional checks:
Docs and Governance
AGENTS.mdneeded an update. (no — інваріанти не змінились)sergeant-web-uiskill)Updated docs:
docs/initiatives/0011-foundation-adoption-and-process-discipline.mdRisk and Rollout
main. Vercel deploy на docs-only PR не потрібен (не зачіпає apps).Hard Rule #15
AGENTS.mdbefore coding.--no-verify.Reviewer Notes
apiV1.test.ts, mobile build, E2E auth, Argos visual regression) — не від цього PR; той самий набір на 2.4–2.8.Summary by cubic
Record PR 2.8 (#1726) for DataState adoption in digest by updating the 0011 initiative doc: add 2.8 to the status header, update the Phase 2 table with
core/insights/WeeklyDigestCard.tsx(DigestContent4‑state ladder), and mark the consumer‑adoption block closed.Written for commit a675fd3. Summary will update on new commits.
Summary by CodeRabbit