Skip to content

Rebrand: fast-forward chevron logo & violet theme - #4078

Merged
amitsingh-007 merged 1 commit into
mainfrom
f-rebranding
Jul 19, 2026
Merged

amitsingh-007 merged 1 commit into
mainfrom
f-rebranding

Conversation

@amitsingh-007

@amitsingh-007 amitsingh-007 commented Jul 19, 2026 •

Copy link
Copy Markdown
Owner

Check if the Pull Request fulfils these requirements

  • Does the extension require a version change?

🤖 Generated with Claude Code

Greptile Summary

This PR updates the product branding across the extension and web app.

  • New extension and web app icon assets.
  • Extension version bump to 25.0.0.
  • Updated extension and web manifest icon metadata.
  • Violet theme color updates for the web app and shared UI tokens.
  • Rounded header logo styling.

Confidence Score: 5/5

This looks mergeable after small manifest icon cleanups.

  • The changed code is mostly branding assets and color tokens.
  • The remaining issues affect extension and installed app icon presentation.
  • No auth, data, or runtime execution path changed.

apps/extension/src/constants/manifest.ts, apps/web/public/manifest.webmanifest.json

Prompt To Fix All With AI
Fix the following 2 code review issues. Work through them one at a time, proposing concise fixes.

---

### Issue 1 of 2
apps/extension/src/constants/manifest.ts:20
**Missing Extension App Icon Size**

The manifest now declares only a 128px app icon while the 48px asset was deleted. Chrome uses the top-level `icons` field for extension management and package validation surfaces, so those paths must scale the 128px image or can flag the package as missing the expected 48px icon.

### Issue 2 of 2
apps/web/public/manifest.webmanifest.json:9
**Installed Icon Loses Masking**

Both manifest icons now use only `any`, and these are the only web manifest icons. Android/Chrome no longer has a maskable-safe icon for installed app rendering, so launcher icons can be clipped and PWA checks can report the missing maskable icon.

Reviews (1): Last reviewed commit: "feat: rebrand with fast-forward chevron ..." | Re-trigger Greptile

Greptile also left 2 inline comments on this PR.

Context used (3)

- New three-chevron "Accelerate" mark across extension + web icons
  (on/off/pending states, favicon, apple-touch, PWA icons, all sizes)
- Squircle violet-gradient badge; supersampled PNGs for crisp small sizes
- Shift shared UI brand hue indigo -> violet (both light/dark) in @bypass/ui
- Sync theme-color meta to primary; round web header logo
- Manifest: 128 store icon + 32 toolbar action; drop unused 16/48
- Bump extension to 25.0.0

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@webext-bot

webext-bot Bot commented Jul 19, 2026

Copy link
Copy Markdown

Extension version is updated from 24.15.0 to 25.0.0

@sourcery-ai sourcery-ai 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.

Hey - I've left some high level feedback:

  • The new theme color (#6d33d2) in the web layout isn't reflected in the global OKLCH variables, which may lead to visual inconsistencies between the app shell and UI components; consider aligning the primary/accent tokens with this hex value or vice versa.
  • In the dev extension manifest, action.default_icon and icons now only define 32 and 128 sizes respectively; if the browser or tooling expects 16/48 variants, consider adding those here to avoid relying on implicit scaling.
Prompt for AI Agents
Please address the comments from this code review:

## Overall Comments
- The new theme color (#6d33d2) in the web layout isn't reflected in the global OKLCH variables, which may lead to visual inconsistencies between the app shell and UI components; consider aligning the primary/accent tokens with this hex value or vice versa.
- In the dev extension manifest, `action.default_icon` and `icons` now only define 32 and 128 sizes respectively; if the browser or tooling expects 16/48 variants, consider adding those here to avoid relying on implicit scaling.

Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

@amitsingh-007 amitsingh-007 linked an issue Jul 19, 2026 that may be closed by this pull request
@webext-bot

webext-bot Bot commented Jul 19, 2026

Copy link
Copy Markdown
Extension Size Change:   8.11 KB 🔺
Commit ec4901c
Latest release size 280.72 KB
Current size 288.83 KB
Percent change 2.89 %

This commit looks good, cheers 👏

Comment thread apps/extension/src/constants/manifest.ts
Comment thread apps/web/public/manifest.webmanifest.json
@coderabbitai

coderabbitai Bot commented Jul 19, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 6ca9a858-885a-4382-bc52-f7849fc21078

📥 Commits

Reviewing files that changed from the base of the PR and between 13521fd and ec4901c.

⛔ Files ignored due to path filters (9)
  • apps/extension/public/assets/bypass_link_off_32.png is excluded by !**/*.png
  • apps/extension/public/assets/bypass_link_on_128.png is excluded by !**/*.png
  • apps/extension/public/assets/bypass_link_on_32.png is excluded by !**/*.png
  • apps/extension/public/assets/bypass_link_on_48.png is excluded by !**/*.png
  • apps/extension/public/assets/bypass_link_pending_32.png is excluded by !**/*.png
  • apps/web/public/apple-touch-icon.png is excluded by !**/*.png
  • apps/web/public/bypass_link_192.png is excluded by !**/*.png
  • apps/web/public/bypass_link_512.png is excluded by !**/*.png
  • apps/web/public/favicon.ico is excluded by !**/*.ico
📒 Files selected for processing (6)
  • apps/extension/package.json
  • apps/extension/src/constants/manifest.ts
  • apps/web/public/manifest.webmanifest.json
  • apps/web/src/app/components/AppHeader.tsx
  • apps/web/src/app/layout.tsx
  • packages/ui/src/styles/globals.css

📝 Walkthrough

Walkthrough

Updated the extension version and development manifest icon structure. Adjusted web manifest icon purposes and theme colors, synchronized the viewport theme color, and added rounded styling to the header icon. Replaced light and dark theme color tokens, including primary, accent, ring, chart, and sidebar values.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes the rebrand and theme update reflected in the changeset.
Description check ✅ Passed The description matches the template by answering the required extension version-change question.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@qodo-code-review

Copy link
Copy Markdown

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (1) 📜 Skill insights (0)

Context used
✅ Compliance rules (platform): 24 rules

Grey Divider


Informational

1. Unapproved globals.css theme changes 📘 Rule violation § Compliance
Description
A pre-existing file under packages/ui was modified (primary/accent color variables changed)
without an explicit approval marker (e.g., UI-APPROVED: ...) in code or PR description. This
violates the rule restricting changes in packages/ui to new components or explicitly approved
modifications.
Code

packages/ui/src/styles/globals.css[R17-24]

+  --primary: oklch(0.495 0.225 293);
+  --primary-foreground: oklch(0.96 0.02 293);
  --secondary: oklch(0.967 0.001 286.375);
  --secondary-foreground: oklch(0.21 0.006 285.885);
  --muted: oklch(0.967 0.001 286.375);
  --muted-foreground: oklch(0.552 0.016 285.938);
-  --accent: oklch(0.51 0.23 277);
-  --accent-foreground: oklch(0.96 0.02 272);
+  --accent: oklch(0.495 0.225 293);
+  --accent-foreground: oklch(0.96 0.02 293);
Relevance

⭐ Low

PR #3908 changed packages/ui globals.css without approval marker; similar ui edits merged, rule not
enforced.

PR-#3908
PR-#3946

ⓘ Recommendations generated based on similar findings in past PRs

Evidence
The compliance rule requires explicit approval for modifications to pre-existing packages/ui files
unless the change is only adding a new component. The PR changes existing global theme variables in
packages/ui/src/styles/globals.css without any approval tag shown in the modified lines.

Rule 1805904: Restrict modifications in packages/ui to new components or approved changes
packages/ui/src/styles/globals.css[17-24]

Agent prompt
The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

## Issue description
`packages/ui/src/styles/globals.css` (an existing file) changes UI theme tokens without an explicit approval reference.

## Issue Context
Compliance requires that modifications to pre-existing files under `packages/ui` be limited to adding new components or be explicitly marked as approved (e.g., via a `UI-APPROVED: ...` tag in code or an approval link/ID in the PR description).

## Fix Focus Areas
- packages/ui/src/styles/globals.css[17-24]

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools


Grey Divider

Qodo Logo

@amitsingh-007
amitsingh-007 merged commit 96698cc into main Jul 19, 2026
5 checks passed
@amitsingh-007
amitsingh-007 deleted the f-rebranding branch July 19, 2026 14:27
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.

Redesign logo

1 participant