feat(dashboard): restore primitives grid, comparison table, API surface#356
Conversation
|
Warning You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again! |
|
Warning Review limit reached
Next review available in: 32 minutes Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (4)
✨ 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 |
PR #339 (shadcn redesign) dropped three content blocks from the landing page that existed nowhere else: the five-primitives grid, the memory-vs-AgentState comparison table, and the API-surface endpoint table. Recover and restyle them for the new shadcn token system instead of letting them stay lost in history. - Landing page: add a "Five primitives" section (state, lease, claim, token, conversation) restyled with bg-card/border-border/hover:bg-muted, since the current feature grid had silently stopped mentioning three of the five core primitives. - New /compare page: houses the memory-vs-AgentState comparison table, linked from header nav, footer, and the new primitives section. - docs.astro: add an "API surface" reference section reusing the existing endpoint-list pattern and methodColor() helper already in that file, matching its (pre-#339) token conventions. Closes #354 Co-Authored-By: Duyet Le <me@duyet.net> Co-Authored-By: duyetbot <bot@duyet.net>
Rebase fixup after #355 landed src/lib/site.ts (single source of truth for the site origin, replacing hardcoded https://agentstate.app). The new "API surface" section added by this branch had one hardcoded URL in its footer note; switch it to API_BASE_URL to match every other occurrence in docs.astro. Co-Authored-By: Duyet Le <me@duyet.net> Co-Authored-By: duyetbot <bot@duyet.net>
10e6dbb to
b2b417b
Compare
Summary
PR #339 (shadcn redesign) rebuilt the landing page on the landing-page11 structure and dropped three content blocks that weren't preserved anywhere else. All three were recovered from
ce9a1fa^:packages/dashboard/src/pages/index.astro(the pre-#339 tree) and restyled for the current shadcn token system rather than pasted back verbatim.Closes #354
Where each block landed, and why
permissionScopesin docs.astro:state:read,state:write,lease:write,claim:write). Restyled withbg-card/border-border/hover:bg-mutedto match the PR feat(dashboard): shadcn/ui redesign — landing page, dashboard spacing, chat components #339 system; kept--color-accentfor the CTA link per the "accent is brand vermilion, not shadcn hover" rule./comparepage. This is "why choose us" content that doesn't fit the trimmed shadcnblocks landing structure, but is valuable enough to deserve a permanent, linkable home rather than being buried. Linked from the header nav, footer "Product" list, and a callout in the new primitives section ("Why not just memory? → Compare").methodColor()helper and<ul>/<li>endpoint-list pattern already used by the Conversations/Analytics sections in that file, rather than reintroducing the old<table>markup — docs.astro wasn't touched by feat(dashboard): shadcn/ui redesign — landing page, dashboard spacing, chat components #339 and still uses the pre-shadcn token names (fg,edge,panel), so the new section matches that file's existing conventions instead of mixing token systems within one file.Not in scope: the dropped 6th "Workers AI Durable Object" code-tab example mentioned in the issue as context — the issue's actual "Fix:" only calls out the three content blocks, and re-adding a 6th CodeTabs example is a separate, larger change (needs its own copy review). Flagging it here in case it should be a follow-up.
Test plan
bunx biome check packages/dashboard/src/— clean (pre-existing single-file-check false positives onMarketingLayout.astro/brand.astroconfirmed present onmaintoo, not introduced by this change)bunx astro check --minimumSeverity error— 0 errors across 201 files