Skip to content

Reduce plugin capability name size - #992

Merged
brsbl merged 2 commits into
bb/skill-detail-pagination-thr_cxz5rwzns3from
bb/capability-name-typography-thr_cxz5rwzns3
Aug 4, 2026
Merged

Reduce plugin capability name size#992
brsbl merged 2 commits into
bb/skill-detail-pagination-thr_cxz5rwzns3from
bb/capability-name-typography-thr_cxz5rwzns3

Conversation

@brsbl

@brsbl brsbl commented Aug 4, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • reduce capability names using the established text-xs typography token
  • keep scheduled jobs and other shared detail-row consumers unchanged
  • add focused regression coverage for capability name styling

Verification

  • pnpm exec turbo run test --filter=@bb/app -- --run src/components/tools/detail-page-recipes.test.tsx (38 passed)
  • pnpm exec turbo run typecheck --filter=@bb/app
  • git diff --check
  • visually verified Automations and Workflows plugin details in the branch dev app, including proportional, monospace, and long-description capability rows

Stack

Layer 2. Base: bb/skill-detail-pagination-thr_cxz5rwzns3 (PR #991).

@SawyerHood

Copy link
Copy Markdown
Collaborator

🚨 SLOP COP 🚨 · review

I am SlopCop. I am reviewing this pull request now.

The stack contains #992, #993, #994, #995, #996, #997, #1000, and #1001, from bottom to top.

I will review security, code quality, performance, architecture, and tests. I will run the end-to-end test on #1001.

@SawyerHood

Copy link
Copy Markdown
Collaborator

🚨 SLOP COP 🚨 · review

I am SlopCop. I am reviewing this stack under the review rule.

The stack contains these pull requests, from bottom to top:

I will run security, code quality and architecture, performance, and one end-to-end phase on top PR #1001.

I skipped no pull requests.

@SawyerHood

Copy link
Copy Markdown
Collaborator

🚨 SLOP COP 🚨 · review

Security phase complete.

I found no security issue in this PR after I reviewed the complete stack.

The changes do not add a trust-boundary change, unsafe render sink, data exposure, dependency risk, or protocol change.

@SawyerHood SawyerHood left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

🚨 SLOP COP 🚨 · review

Review complete.

I found no issues in this pull request after the complete stack lands.

The security, code-quality, architecture, performance, and final review phases found no issue here.

I used a comment-only review. I did not approve or request changes.

@SawyerHood

Copy link
Copy Markdown
Collaborator

🚨 SLOP COP 🚨 · review

Performance phase complete.

I found no material performance issue in this PR after I reviewed the complete stack.

The render scope, allocations, bundle inputs, and pagination behavior stay stable.

@SawyerHood

Copy link
Copy Markdown
Collaborator

🚨 SLOP COP 🚨 · review

Code-quality and architecture phase complete.

I found no actionable code-quality or architecture issue in this PR.

I checked the full stack and checked later PRs before I reported each issue.

@SawyerHood SawyerHood left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

🚨 SLOP COP 🚨 · review

Review result: no finding.

  • Security review found no issue.
  • Performance review found no material render, allocation, bundle, or pagination issue.
  • Turbo type checks passed for the app, shared UI, Automations, and Docs packages.
  • The affected tests passed after clean reruns. The latest Docs suite passed all 57 tests.

The browser test covered the full stack on the top PR.

## Summary
- reserve a two-line description area on every plugin Browse card
- apply the sanctioned border-under-cursor interaction to category
filter pills
- add focused regression coverage for both behaviors

## Verification
- `pnpm exec turbo run test --filter=@bb/app -- --run
src/components/plugin/management/BrowsePluginsTab.test.tsx
src/components/plugin/PluginsOverview.test.tsx` (16 tests passed)
- `pnpm exec turbo run typecheck --filter=@bb/app`
- branch-built desktop dev app: verified one-line and multi-line card
alignment plus default, hover, selected, and filtered category states

## Stack
- Parent: #992
- This PR: plugin Browse card/filter polish
@brsbl
brsbl merged commit 55491f3 into bb/skill-detail-pagination-thr_cxz5rwzns3 Aug 4, 2026
10 checks passed
@brsbl
brsbl deleted the bb/capability-name-typography-thr_cxz5rwzns3 branch August 4, 2026 23: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.

2 participants