Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Fetch branding#75

Merged
AHarmlessPyro merged 7 commits into
mainhyperbrowserai/python-sdk:mainfrom
fetch_brandinghyperbrowserai/python-sdk:fetch_brandingCopy head branch name to clipboard
Apr 21, 2026
Merged

Fetch branding#75
AHarmlessPyro merged 7 commits into
mainhyperbrowserai/python-sdk:mainfrom
fetch_brandinghyperbrowserai/python-sdk:fetch_brandingCopy head branch name to clipboard

Conversation

@AHarmlessPyro
Copy link
Copy Markdown
Contributor

@AHarmlessPyro AHarmlessPyro commented Apr 21, 2026


Open in Devin Review

Note

Medium Risk
Public SDK response/request models change to include a new branding output, which could affect downstream type expectations/serialization. No runtime logic changes beyond model/schema expansion.

Overview
Adds first-class support for a new branding fetch output. This introduces a BrandingProfile (and nested typography/color/component/etc. models) and exposes it as branding on PageData/FetchResponseData, plus a new FetchOutputBranding option and branding entry in FetchOutputFormat.

Exports the new branding types from hyperbrowser.models and bumps the package version to 0.90.5.

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

Copy link
Copy Markdown

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 2 additional findings.

Open in Devin Review

devin-ai-integration[bot]

This comment was marked as resolved.

cursor[bot]

This comment was marked as resolved.

devin-ai-integration[bot]

This comment was marked as resolved.

cursor[bot]

This comment was marked as resolved.

cursor[bot]

This comment was marked as resolved.

@AHarmlessPyro AHarmlessPyro merged commit b7137c5 into main Apr 21, 2026
1 of 2 checks passed
@AHarmlessPyro AHarmlessPyro deleted the fetch_branding branch April 21, 2026 18:21
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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.