Expand Extensions detail descriptions to full width - #1116
Conversation
SawyerHood
left a comment
There was a problem hiding this comment.
🚨 SLOP COP 🚨 · review
I am the SlopCop. I am reviewing this pull request under the review rule.
The stack contains only #1116. A second scan after two minutes found no later pull request. I skipped no pull requests.
I started parallel security, code-quality, architecture, and performance reviews. I will test the top pull request through the app when the local route permits it.
SawyerHood
left a comment
There was a problem hiding this comment.
🚨 SLOP COP 🚨 · review
Security review complete. I found no security issues.
The change only replaces static width classes. React still escapes both description values as text. The diff adds no trust boundary, authorization, dependency, or data-exposure change. git diff --check passed.
SawyerHood
left a comment
There was a problem hiding this comment.
🚨 SLOP COP 🚨 · review
Performance review complete. I found no performance issues.
The parent page limits content to max-w-5xl. The new class adds no render work, dependency, or meaningful bundle size. The changed tests passed through Turbo: 70 tests passed.
SawyerHood
left a comment
There was a problem hiding this comment.
🚨 SLOP COP 🚨 · review
Code-quality and architecture review complete. I found no actionable issues.
The shared resource layout already owns the page and section structure. The catalog and installed plugin descriptions use two parallel render paths. Their small class and fallback duplication does not justify another abstraction. Both changed test files passed, and GitHub reports all required checks as successful.
SawyerHood
left a comment
There was a problem hiding this comment.
🚨 SLOP COP 🚨 · review
End-to-end review complete on the top pull request. I found no visual or route issue.
I started the source app and enabled the Extensions experiment. I opened the GitHub catalog detail and the installed Remote access detail. In both paths, the description measured 984 pixels, equal to the About section width. The computed maximum width was none. I also inspected the GitHub detail screenshot.
SawyerHood
left a comment
There was a problem hiding this comment.
🚨 SLOP COP 🚨 · review
Final review complete. I found no issues.
The stack contains only #1116, and I skipped no pull requests. The security, code-quality, architecture, and performance reviews found no actionable problem. The focused Turbo run passed 70 tests. The browser test covered both catalog and installed plugin detail routes. Both descriptions filled their 984-pixel About sections. All GitHub checks passed. The final GPT-5.6 gate found no issue.
I used a comment-only review. I did not use GitHub's approval or change-request action.
Summary
Verification
pnpm exec vitest run apps/app/src/views/ToolsView.plugin-detail.test.tsx apps/app/src/components/tools/detail-page-recipes.test.tsx(70 passed)pnpm exec turbo run typecheck --filter=@bb/apppnpm exec turbo run lint --filter=@bb/app(0 errors; existing warnings only)