Skip to content

[DRAFT] feat: add browserbase brand skill#85

Open
Kylejeong2 wants to merge 2 commits intomainfrom
kylejeong/gro-1274-create-a-brand-aligned-release-video-generation-skill
Open

[DRAFT] feat: add browserbase brand skill#85
Kylejeong2 wants to merge 2 commits intomainfrom
kylejeong/gro-1274-create-a-brand-aligned-release-video-generation-skill

Conversation

@Kylejeong2
Copy link
Copy Markdown
Member

@Kylejeong2 Kylejeong2 commented Apr 28, 2026

Summary

Browserbase Brand Skill to help us generate code, videos, slides, images, etc with our refreshed brand.

DRAFT


Note

Low Risk
Low risk since it primarily adds documentation and static brand assets (tokens and SVG logos) plus .gitignore rules, with no runtime logic changes. Main concern is repo size/asset management due to new binary-oriented directories and large markdown.

Overview
Adds a new browserbase-brand skill that codifies Browserbase’s brand guidelines (voice/tone, color and typography rules, logo usage, and do/don’t checks) for generating on-brand copy and visuals.

Includes reusable brand token files in both references/brand-tokens.css and references/brand-tokens.json, plus checked-in SVG logo assets and an assets/README.md describing how to use them.

Updates .gitignore to exclude the skill’s font binaries (assets/Fonts/) and a temporary references/positioning.md file from version control.

Reviewed by Cursor Bugbot for commit e037904. Bugbot is set up for automated code reviews on this repo. Configure here.

"grey": { "value": "#C5D3E8", "token": "--bb-grey", "pantone": "657 C", "cmyk": "C21 M11 Y2 K0", "usage": "Core grey. Interface/layout backgrounds, charts, diagrams." }
},
"secondary": {
"staghandGreen": { "value": "#00C851", "token": "--bb-stagehand-green", "pantone": "2420 C", "cmyk": "C73 M0 Y92 K0", "usage": "Stagehand product signature. Small accents only." },
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Typo in JSON key: staghandGreen missing letter

Medium Severity

The JSON key staghandGreen is missing the letter "e" — it needs to be stagehandGreen. Any code or tooling consuming this token file by key name (e.g., to look up the Stagehand product color) will fail to find it. The token field inside the value correctly says --bb-stagehand-green, making the mismatch easy to miss.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit e037904. Configure here.


## Identity & positioning

Full positioning detail — brand idea, mission, product archetypes, positioning statements, messaging ladder, taglines, and value props — lives in `references/positioning.md`. Read it when writing copy or building brand narratives.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Referenced positioning.md is gitignored and missing

High Severity

SKILL.md instructs the AI to "Read" references/positioning.md for brand copy and messaging, but that file is added to .gitignore and doesn't exist in the repository. Anyone cloning the repo and using this skill will hit a missing-file error whenever the agent tries to follow the skill's own instructions. The file needs to either be committed or the references to it removed.

Additional Locations (2)
Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit e037904. Configure here.

"blue": { "value": "#0000CD", "token": "--bb-blue", "pantone": "Blue 072 C", "cmyk": "C93 M86 Y0 K0", "usage": "Supporting accent." },
"magenta": { "value": "#FF00FF", "token": "--bb-magenta", "pantone": "Real Purple C", "cmyk": "C27 M81 Y0 K0", "note": "Does not translate easily to CMYK — recommend specialty inks.", "usage": "Expressive accent only." },
"pink": { "value": "#FFC0CB", "token": "--bb-pink", "pantone": "182 C", "cmyk": "C0 M34 Y7 K0", "usage": "Supporting accent." },
"paleYellow": { "value": "#FFFACD", "token": null, "pantone": "2001 C", "cmyk": "C1 M0 Y24 K0", "usage": "IMAGE TOOL ONLY — do not use in layouts." }
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

we should remove paleYellow globally to avoid usage


Official logo lockups. Pick variant by background:
- **Light bg** → `browserbase-logo-color.svg` or `browserbase-logo-black.svg`
- **Dark bg** → `Browserbase-logo-white.svg`
Copy link
Copy Markdown

@fosmond fosmond Apr 28, 2026

Choose a reason for hiding this comment

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

follow up from slack thread - let's delete all mentions of dark bg so we can reinforce light/red bg

Official logo lockups. Pick variant by background:
- **Light bg** → `browserbase-logo-color.svg` or `browserbase-logo-black.svg`
- **Dark bg** → `Browserbase-logo-white.svg`
- **Orange bg** → `Browserbase-logo-white.svg` or `browserbase-logo-black.svg`
Copy link
Copy Markdown

@fosmond fosmond Apr 28, 2026

Choose a reason for hiding this comment

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

inconsistent verbiage globally: change orange -> red


All assets are checked in. Reference by relative path — do not hotlink Notion S3 URLs.

## `logos/`
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

follow up from slack thread - let's delete black logo/logomark

--bb-leading-body: 1.5;

/* Shape */
--bb-radius-sm: 4px;
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

we can delete sm and md radius. all shapes have sharp corners, unless its a label which would use pill

--bb-radius-sm: 4px;
--bb-radius-md: 8px;
--bb-radius-pill: 999px;
--bb-hairline: 1px;
Copy link
Copy Markdown

@fosmond fosmond Apr 28, 2026

Choose a reason for hiding this comment

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

if no conflict, can we globally change "hairline" to "border" pls and ty

|---|---|
| GT Planar (Light, Regular, Medium) | Display, UI, headings, navigation |
| GT Standard Mono (Regular, Bold) | Code blocks, inline code, uppercase labels |
| GT Standard L/M/S (Regular, Bold) | Body text at optical sizes |
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

we can remove this line - no longer using GT standard L/M/S as body text

| GT Planar (Light, Regular, Medium) | Display, UI, headings, navigation |
| GT Standard Mono (Regular, Bold) | Code blocks, inline code, uppercase labels |
| GT Standard L/M/S (Regular, Bold) | Body text at optical sizes |
| Plain (Thin – Bold) | Role TBD |
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

replace "role TBD" with "Body text at optical sizes"


| Family | Role |
|---|---|
| GT Planar (Light, Regular, Medium) | Display, UI, headings, navigation |
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

remove "light" and "regular"

| Family | Role |
|---|---|
| GT Planar (Light, Regular, Medium) | Display, UI, headings, navigation |
| GT Standard Mono (Regular, Bold) | Code blocks, inline code, uppercase labels |
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

we can remove "bold," because were only really using regular

--bb-white: #FFFFFF;
--bb-grey: #C5D3E8; /* Grey 500 Core — Pantone 657 C */
/* Legacy alias */
--bb-orange: var(--bb-red);
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

this line looks extra

/* Type scale (tight display, 1.5 body) */
--bb-font-display: "GT Planar", ui-sans-serif, system-ui, sans-serif;
--bb-font-mono: "GT Standard Mono", ui-monospace, "JetBrains Mono", Menlo, monospace;
/* Legacy alias — use --bb-font-display for new work */
Copy link
Copy Markdown

@fosmond fosmond Apr 28, 2026

Choose a reason for hiding this comment

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

question - are lines in legacy alias important? i feel like the specifics are covered in typography in skill.md

Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 3 potential issues.

There are 5 total unresolved issues (including 2 from previous reviews).

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit a854493. Configure here.

.bb-surface {
background: var(--bb-bg);
color: var(--bb-fg);
font-family: var(--bb-font-display);
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

CSS tokens missing body font, applies wrong family

Medium Severity

brand-tokens.css defines --bb-font-display (GT Planar) and --bb-font-mono but has no --bb-font-body token for the Plain font. The .bb-surface class applies var(--bb-font-display) to all text, meaning body copy renders in GT Planar instead of Plain — contradicting SKILL.md's font roles table which specifies body copy uses Plain.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit a854493. Configure here.

.bb-surface h3 {
letter-spacing: var(--bb-tracking-display);
line-height: var(--bb-leading-display);
}
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

CSS heading styles omit h4 through h6 elements

Low Severity

The .bb-surface heading selector only targets h1, h2, h3 for display tracking and leading, but SKILL.md defines heading styles H1 through H7. Any h4h6 inside .bb-surface inherits body values (letter-spacing: 0, line-height: 1.5) instead of display values (-0.02em, 1.1), producing visually off-brand headings with incorrect spacing.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit a854493. Configure here.

<path d="M832.416 146.52C819.448 146.52 808.864 135.958 808.864 123.079V103.526C808.864 90.5088 819.476 80.0842 832.416 80.0842H842.501L853.363 87.3924V80.0842H866.441V146.52H853.363V139.212L842.501 146.52H832.416ZM821.942 123.079C821.942 128.815 826.652 133.503 832.416 133.503H838.428L853.335 123.465V103.14L838.428 93.1011H832.416C826.652 93.1011 821.942 97.7894 821.942 103.526V123.079Z" fill="black"/>
<path d="M901.504 146.52C888.865 146.52 879.483 138.302 879.483 127.243V124.651H892.508V127.243C892.508 131.407 895.902 134.413 901.504 134.413H913.37C919.496 134.413 922.504 131.931 922.504 128.153C922.504 113.426 880.394 122.941 880.394 97.6791C880.394 87.1166 888.865 80.0842 901.256 80.0842H912.459C924.187 80.0842 933.321 88.5507 933.321 99.4992V100.685H920.296V99.4992C920.296 95.3349 917.04 92.191 912.487 92.191H901.283C895.819 92.191 893.474 94.3973 893.474 97.6515C893.474 108.214 935.584 101.568 935.584 128.125C935.584 139.074 926.588 146.492 913.425 146.492H901.559L901.504 146.52Z" fill="black"/>
<path d="M969.369 146.52C956.457 146.52 945.918 135.847 945.918 122.941V103.526C945.918 90.5088 956.485 80.0842 969.369 80.0842H979.798C992.434 80.0842 1002.61 89.8469 1003.25 102.229V116.294H958.94V122.941C958.94 128.677 963.63 133.365 969.369 133.365H979.798C985.537 133.365 990.227 128.677 990.227 122.941V121.644H1003.25V122.941C1003.25 135.847 992.82 146.52 979.798 146.52H969.369ZM958.94 105.897H990.227V103.553C990.227 97.6791 985.675 93.1287 979.798 93.1287H969.369C963.492 93.1287 958.94 97.817 958.94 103.553V105.897Z" fill="black"/>
</svg>
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Black logo files committed but unreferenced everywhere

Low Severity

browserbase-logo-black.svg and Browserbase-logomark-black.svg are committed to the repo but are not listed in SKILL.md's logo table, assets/README.md, or brand-tokens.json's logo variants. These are orphaned assets with no documentation pointing to them, and the PR reviewer explicitly requested their deletion.

Additional Locations (1)
Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit a854493. Configure here.


## Voice & tone

**Goal:** Inspire more people to take on ambitious automation on the web.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

"automation" 👀 👀

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@bbclanker can you update this

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.

3 participants