Skip to content

feat(den): library logos + neutral chips + Ready-to-use tab - #3450

Merged
benjaminshafii merged 1 commit into
devfrom
feat/library-logos
Aug 3, 2026
Merged

feat(den): library logos + neutral chips + Ready-to-use tab#3450
benjaminshafii merged 1 commit into
devfrom
feat/library-logos

Conversation

@benjaminshafii

Copy link
Copy Markdown
Member

Summary

Follow-up to #3445 per canvas frame 16 (updated): color now carries meaning only.

  • Kind chips neutral (Skill/Plugin gray; Connection keeps blue) — amber appears ONLY on needs-your-action surfaces (sign-in row, its tab pill). Matches frame 13's chip neutrality.
  • Real logos in row tiles via the existing DenBrandMark ladder: native provider map (google/microsoft), MCP connections favicon off a new url field on /v1/me/library connection items, GitHub owner avatars from sourceRepositoryUrl; neutral monogram fallback.
  • Ready to use state tab (count pill, hidden at zero); per-tab countClassName on TabItem replaces the fragile nth-child styling.
  • den-api: POST /v1/plugins now accepts + persists sourceRepositoryUrl (create-with-components path previously dropped it); library connection items expose url.

Verification

  • pnpm --filter @openwork-ee/den-web typecheck — pass
  • bun test test/plugin-system-me-library.test.ts — 0 fail (url field + create passthrough)
  • stack spec library-view.slow.test.ts — passed at head SHA (re-pinned: Ready tab, GitHub-avatar img on the spec-controlled plugin row, neutral chip computed style, state-tab interactions; vision 3/3)
  • pnpm evals:typecheck — pre-existing den-trim-page-headers.flow.ts failures only

Evidence

Photo roll in follow-up comment.

…iles (provider map, MCP favicon via new connection url field, GitHub owner avatars), amber reserved for needs-action
@benjaminshafii

Copy link
Copy Markdown
Member Author

Photo roll — library-logos — 3/3 frames passed

3/3 frames passed · 4 expectations passed · 0 failed

✅ PASS — 1. A compact gradient header is titled Library, with its description immediately below and before the tabs

  • PASS A compact gradient header is titled Library, with its description immediately below and before the tabs — A blue gradient banner is titled “Library”; directly below it is “Everything you can use in chat — yours, shared with you, from your teams, and org-wide.” followed by the tabs.

A compact gradient header is titled Library, with its description immediately below and before the tabs

✅ PASS — 2. Rows show logo or monogram tiles in aligned lanes with neutral kind chips and a quiet right-hand source column

  • PASS Rows show logo or monogram tiles in aligned lanes with neutral kind chips and a quiet right-hand source column — Ready-to-use rows have aligned letter tiles (C, D, D, E), neutral “Skill” chips, and muted “Catalog” text in the right-hand column.
  • PASS An amber needs-sign-in row shows a dark Sign in button under a NEEDS YOUR SIGN-IN caption — A pale amber Acme Robotics row appears beneath “NEEDS YOUR SIGN-IN” and has a dark “Sign in” button on the right.

Rows show logo or monogram tiles in aligned lanes with neutral kind chips and a quiet right-hand source column

✅ PASS — 3. A narrow mobile layout keeps rows readable with wrapped chips

  • PASS A narrow mobile layout keeps rows readable with wrapped chips — The narrow single-column view remains readable; filter chips wrap across three lines, while library rows retain clear icons, labels, badges, and chevrons.

A narrow mobile layout keeps rows readable with wrapped chips


Roll created 2026-08-03T10:24:24.551Z · Source: evals/results/rolls/2026-08-03T10-24-24-551Z-library-logos/roll.json · Repro: pnpm fraimz:publish -- --pr 3450 --roll 2026-08-03T10-24-24-551Z-library-logos

@vercel

vercel Bot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
openwork-app Ready Ready Preview Aug 3, 2026 10:25am
openwork-den Ready Ready Preview Aug 3, 2026 10:25am
openwork-den-worker-proxy Ready Ready Preview Aug 3, 2026 10:25am
openwork-diagnostics Ready Ready Preview Aug 3, 2026 10:25am
openwork-landing Ready Ready Preview, v0 Aug 3, 2026 10:25am

@diff-warden diff-warden Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Warden security clearance: clear. No new security issues found in this diff (7517d7f2d2cbe0b6b242d6ce304e63e5738bd2df). Automated clearance satisfies the required-review gate only — a human still reviews and merges. Analysis run

@benjaminshafii
benjaminshafii merged commit f6cac28 into dev Aug 3, 2026
25 checks passed
@benjaminshafii
benjaminshafii deleted the feat/library-logos branch August 3, 2026 11:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant