Skip to content

Centralize OK Code brand constants and unify identity#265

Merged
BunsDev merged 2 commits into
mainfrom
okcode/ok-code-brand-refresh
Apr 5, 2026
Merged

Centralize OK Code brand constants and unify identity#265
BunsDev merged 2 commits into
mainfrom
okcode/ok-code-brand-refresh

Conversation

@BunsDev

@BunsDev BunsDev commented Apr 5, 2026

Copy link
Copy Markdown
Member

Summary

  • Centralized canonical brand values in @okcode/shared/brand so the app name and git identity come from one shared source.
  • Replaced hard-coded OK Code strings across desktop, web, server, marketing, and build scripts with shared branding constants.
  • Updated packaging and desktop identifiers to the new com.openknots.okcode app/user model ID.
  • Simplified brand asset generation so all platform icons use the same OK Code mark.
  • Cleaned up branding copy in BRANDING.md and corrected the copyright holder in LICENSE.

Testing

  • Not run
  • Not run: bun fmt
  • Not run: bun lint
  • Not run: bun typecheck

BunsDev and others added 2 commits April 5, 2026 02:32
- Standardize wordmark casing to "OK Code" (remove CSS uppercase)
- Replace hardcoded brand strings with APP_BASE_NAME constant
- Fix app bundle ID from com.okcode.okcode to com.openknots.okcode
- Unify Windows icon source to use same mark as all other platforms
- Fix LICENSE copyright from "OpenKnot" to "OpenKnots"
- Fix marketing site "OK CODE API" casing to "OK Code API"
- Fill in BRANDING.md TBDs: tagline, one-liner, emoji policy,
  error voice, WCAG AA target, reduced motion support

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Create a single source of truth for the product name by introducing
packages/shared/src/brand.ts with APP_BASE_NAME and GIT_IDENTITY_NAME.
Re-export from apps/web/src/branding.ts and import in all workspace
packages instead of hardcoding "OK Code" strings.

- New: @okcode/shared/brand module with canonical constants
- Web: branding.ts re-exports APP_BASE_NAME from shared package
- Web: ChatHomeEmptyState uses APP_BASE_NAME instead of hardcoded string
- Desktop: main.ts imports APP_BASE_NAME for display name construction
- Desktop: electron-launcher.mjs extracts APP_BASE_NAME with sync comment
- Server: CheckpointStore uses GIT_IDENTITY_NAME for git author/committer
- Server: codexCliVersion uses APP_BASE_NAME in upgrade message
- Scripts: build-desktop-artifact uses APP_BASE_NAME for product name fallback

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@vercel

vercel Bot commented Apr 5, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
okcode-marketing Ready Ready Preview Apr 5, 2026 7:43am
v0-compute-the-platform-to-build Ready Ready Preview, Open in v0 Apr 5, 2026 7:43am

@github-actions github-actions Bot added size:L vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. labels Apr 5, 2026
@BunsDev BunsDev merged commit a3c6ccb into main Apr 5, 2026
10 of 11 checks passed
@BunsDev BunsDev deleted the okcode/ok-code-brand-refresh branch April 6, 2026 21:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:L vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant