Skip to content

chore: apply BRAND.md copy and remove duplicated brand guidelines - #267

Merged
boojack merged 3 commits into
mainfrom
chore/apply-brand-copy
Oct 2, 2026
Merged

boojack merged 3 commits into
mainfrom
chore/apply-brand-copy

Conversation

@boojack

@boojack boojack commented Oct 2, 2026

Copy link
Copy Markdown
Member

Before: docs/brand-guidelines.md called itself the single source of truth for brand copy, and the site used its copy: tagline "Catch a thought. Keep it yours." and description "Write short memos without titles or folders. Find them later by search, tag, or date."

After: BRAND.md in usememos/.github is the only brand definition. The duplicated guide is deleted, and the site uses BRAND.md's copy verbatim:

Surface Copy
Homepage hero headline, default title, social image Tagline: "Your thoughts, your data, shared on your terms."
Homepage hero text Short
Default and homepage metadata, JSON-LD One-liner: "Your own timeline for quick notes, private by default."
Docs introduction, FAQ "What is Memos?", llms.txt summary Standard

How:

  • src/shared/lib/branding.ts holds the four BRAND.md strings (BRAND_TAGLINE, BRAND_DESCRIPTION = One-liner, new BRAND_SHORT, new BRAND_STANDARD).
  • docs/brand-guidelines.md deleted; AGENTS.md, README.md, DESIGN_SYSTEM.md and code comments now point to BRAND.md.
  • The social image renders the tagline as one string, so it wraps to three lines instead of being cut off after two.
  • The getting-started page drops the old intro sentence and starts with the install path.
  • Kept as is: the homepage proof points and the homepage SEO title "Memos - Open-Source, Self-Hosted Note-Taking App" and keywords.
  • Tests updated to the new strings. pnpm lint and pnpm test pass (290 tests). Checked the hero at 1440px and 390px and the social image locally.

Generated by Claude Code

BRAND.md in usememos/.github is now the single source of truth for brand
copy. Delete docs/brand-guidelines.md, point references to BRAND.md, and
update the tagline, descriptions, docs introduction, llms.txt summary and
the homepage social image to the approved copy.
@coderabbitai

coderabbitai Bot commented Oct 2, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 52 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 03a625ab-dc08-4a3e-833d-ffea47fe5446

📥 Commits

Reviewing files that changed from the base of the PR and between 360798d and 3e7a19b.

📒 Files selected for processing (1)
  • AGENTS.md

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 6bb0fd4d-540b-4ec2-8925-deb3b454778a

📥 Commits

Reviewing files that changed from the base of the PR and between 39f854d and 360798d.

📒 Files selected for processing (22)
  • AGENTS.md
  • DESIGN_SYSTEM.md
  • README.md
  • content/docs/faq.mdx
  • content/docs/getting-started.mdx
  • content/docs/index.mdx
  • docs/brand-guidelines.md
  • scripts/smoke-test.mjs
  • src/app/(public)/(site)/brand/page.tsx
  • src/app/(public)/(site)/page.test.tsx
  • src/app/(public)/(site)/page.tsx
  • src/app/(public)/(site)/sponsors/page.tsx
  • src/app/layout.tsx
  • src/app/og/home/image.png/route.tsx
  • src/features/ai-discovery/lib/llms-content.test.ts
  • src/features/ai-discovery/lib/llms-content.ts
  • src/features/editorial/lib/social-preview.ts
  • src/features/marketing/components/hero-accent.test.tsx
  • src/features/marketing/components/home-own-section.tsx
  • src/features/marketing/data/features/data.ts
  • src/shared/lib/branding.test.ts
  • src/shared/lib/branding.ts
💤 Files with no reviewable changes (1)
  • docs/brand-guidelines.md

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.


Walkthrough

The change updates shared brand constants and applies revised copy across the homepage, metadata, documentation, and content summaries. It removes the local brand-guidelines document and directs contributors to the external BRAND.md. It also adds contributor instructions to consult installed Next.js guides and notes that next dev regenerates the agent-rules block.

Priority: ⬇️ Low

Merge Risk: ⚪ Minimal · up to 36079

The reviewed changes update brand messaging without an established functional or operational regression. No actionable issue remains before merge.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 4 functions across 15 files. (6 skipped: 6… Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes the main changes: applying the centralized BRAND.md copy and removing the duplicated local brand guidelines.
Description check ✅ Passed The description directly explains the branding source change, deleted file, updated copy surfaces, implementation details, and validation results.
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.
Full details: Docstring Coverage

Explanation

Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 4 functions across 15 files. (6 skipped: 6 unsupported.)

  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Autopilot is currently an internal CodeRabbit preview.


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.

@boojack boojack self-assigned this Oct 2, 2026
@boojack
boojack merged commit 40feae0 into main Oct 2, 2026
7 checks passed
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.

1 participant