Skip to content

fix(branding): use Claude across public labels - #331

Merged
777genius merged 1 commit into
mainfrom
docs/claude-branding
Oct 4, 2026
Merged

777genius merged 1 commit into
mainfrom
docs/claude-branding

Conversation

@777genius

@777genius 777genius commented Oct 4, 2026 •

Copy link
Copy Markdown
Owner

Use Claude consistently in current documentation, landing translations, installer prompts, CLI help and webhook labels. Simplify the README installer link and remove the preview caption while keeping the three labeled screenshots in one row.

Keep compatibility identifiers, Windows notification registration and historical evidence unchanged. The Windows troubleshooting guide retains the exact legacy OS entry.

Validation:

  • Independent GPT-6.1 xhigh review accepted dc1796e859f391872c97750ac5cc6962f2853100; all actionable findings fixed.
  • Focused Go webhook, setupwizard, summary and CLI tests passed in an isolated server TEST workspace. Exact Git blob comparisons bind the tested source to the final commit.
  • Product selector, setup loader and native OpenCode bootstrap lifecycle tests passed on the final commit with isolated TEST profiles.
  • Landing CI passed typecheck, 12 unit tests, release-loader contract, generation and 19 browser tests.
  • GitHub GFM render preserves the three ordered 31% screenshot images and confirms the requested caption/link wording changes.
  • 27 completed checks passed, including Ubuntu Go 1.25/1.26, macOS/Windows Go 1.25, native Claude/Gemini/OpenCode OS matrices and installer recovery.
  • Windows Go 1.26 failed twice on the unchanged baseline observation-cache deadline issue Investigate intermittent Windows observation-cache qualification timeouts #320. Its CLI package passed. This PR does not fix that separate availability defect.
  • macOS Go 1.26 initially failed dependency downloads because proxy.golang.org DNS resolution failed. Its full-suite retry remains queued; that full suite is unverified. macOS Go 1.25 and native Go 1.26 candidate matrices passed.
  • Independent review accepted merge with these disclosed limits after Ubuntu Go 1.26 passed. There are no repository-required checks or branch protection bypasses. The clean composition with main 7cfaa7d was independently reviewed: tree 9bdf20c82a070fc2d55bffd05fcd4c916b55f926; the three incoming diagnostics paths do not overlap the branding changes.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@coderabbitai

coderabbitai Bot commented Oct 4, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

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

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: b386ba0e-244e-43bf-8e04-03b22fe231e8
📥 Commits

Reviewing files that changed from the base of the PR and between c497dda and dc1796e.

📒 Files selected for processing (59)
  • .claude-plugin/marketplace.json
  • .claude-plugin/plugin.json
  • .github/ISSUE_TEMPLATE/bug_report.yml
  • CONTRIBUTING.md
  • README.md
  • bin/bootstrap.sh
  • bin/bootstrap_opencode_test.sh
  • bin/bootstrap_product_test.sh
  • bin/install.sh
  • bin/setup_test.sh
  • cmd/claude-notifications/main.go
  • cmd/claude-notifications/setup_products_args.go
  • cmd/claude-notifications/setup_products_presence_pty_test.go
  • cmd/claude-notifications/setup_products_test.go
  • cmd/claude-notifications/setup_wizard_cli_contract_test.go
  • cmd/claude-notifications/setup_wizard_output.go
  • docs/ARCHITECTURE.md
  • docs/CLAUDE_PLUGIN_IDENTITY.md
  • docs/CLICK_TO_FOCUS.md
  • docs/CODEX.md
  • docs/CONFIGURATION.md
  • docs/INSTALLATION.md
  • docs/LOCAL_DEVELOPMENT.md
  • docs/PLUGIN_COMPATIBILITY.md
  • docs/RELEASE.md
  • docs/opencode-notifications.md
  • docs/troubleshooting.md
  • docs/volume-control.md
  • docs/webhooks/README.md
  • docs/webhooks/custom.md
  • docs/webhooks/discord.md
  • docs/webhooks/lark.md
  • docs/webhooks/slack.md
  • docs/webhooks/telegram.md
  • docs/webhooks/troubleshooting.md
  • internal/agentnotify/setupwizard/prompt.go
  • internal/agentnotify/setupwizard/prompt_test.go
  • internal/agentnotify/setupwizard/public_prompt.go
  • internal/summary/summary.go
  • internal/webhook/formatters.go
  • internal/webhook/formatters_test.go
  • landing/components/AgentLogo.vue
  • landing/components/NotificationPreview.vue
  • landing/data/install.ts
  • landing/locales/ar.json
  • landing/locales/de.json
  • landing/locales/en.json
  • landing/locales/es.json
  • landing/locales/fr.json
  • landing/locales/hi.json
  • landing/locales/it.json
  • landing/locales/ja.json
  • landing/locales/ko.json
  • landing/locales/pt.json
  • landing/locales/ru.json
  • landing/locales/zh.json
  • landing/tests/browser/install.spec.ts
  • scripts/dev-real-plugin.sh
  • setup.sh

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


📝 Walkthrough

Walkthrough

The product label changes from “Claude Code” to “Claude” across application output, setup prompts, installation guidance, documentation, webhook examples, and landing-page translations. Tests update their expected labels. The README also changes two installation-section details.

Changes

Claude naming

Layer / File(s) Summary
Plugin identity metadata
.claude-plugin/*, docs/CLAUDE_PLUGIN_IDENTITY.md, docs/PLUGIN_COMPATIBILITY.md, docs/RELEASE.md
Plugin descriptions and identity guidance use “Claude” for the retained install identity. The instruction not to rename that identity remains.
Application and installer labels
bin/*, cmd/claude-notifications/*, internal/agentnotify/setupwizard/*, internal/summary/summary.go, internal/webhook/*
Installer messages, CLI and setup-wizard labels, and notification output use “Claude.” Related test expectations change to match.
Repository and integration guidance
.github/ISSUE_TEMPLATE/bug_report.yml, CONTRIBUTING.md, README.md, docs/*, scripts/dev-real-plugin.sh, setup.sh
Instructions and documentation use “Claude” in product references. The README removes the notification-preview disclaimer and the “stable” qualifier from its guided-installer link.
Landing-page labels and translations
landing/components/*, landing/data/install.ts, landing/locales/*, landing/tests/browser/install.spec.ts
Landing-page labels, translations, and browser-test expectations use “Claude.” The Chinese locale also states that click-to-return support depends on the terminal and operating system.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Other

Merge Risk: ⚪ Minimal · up to dc179

The product labels change without changing installation or selection behavior. No material user-facing or operational risk is established; the PR is mergeable after normal checks.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 21.74% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 23 functions across 21 files. (38 skipped… Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
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.
Title check ✅ Passed The title clearly and concisely describes the main change: using Claude across public-facing labels.
Description check ✅ Passed The description gives a clear summary and detailed validation results. It does not include the template’s explicit Test plan or CLA sections, but the summary and test information are substantially com…
Full details: Docstring Coverage

Explanation

Docstring coverage is 21.74% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 23 functions across 21 files. (38 skipped: 38 unsupported.)

  • Fix all pre-merge checks with AI
✨ Finishing Touches 💡 2
📝 Generate docstrings 💡
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
🛠️ Fix failing CI checks 💡
  • Commit to this branch
  • Create a new PR
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

A rabbit read the labels bright,
“Claude” now hops across the page.
Prompts and guides all match the name,
Tests keep pace from stage to stage.
The rabbit thumps a pleased hooray.

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

@777genius
777genius force-pushed the docs/claude-branding branch from 6a47173 to 7ac224b Compare October 4, 2026 10:25
@777genius
777genius force-pushed the docs/claude-branding branch from 7ac224b to dc1796e Compare October 4, 2026 10:27
@codecov

codecov Bot commented Oct 4, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 71.42857% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 65.95%. Comparing base (c497dda) to head (dc1796e).
⚠️ Report is 4 commits behind head on main.

Files with missing lines Patch % Lines
internal/agentnotify/setupwizard/public_prompt.go 0.00% 1 Missing ⚠️
internal/summary/summary.go 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #331      +/-   ##
==========================================
- Coverage   65.96%   65.95%   -0.01%     
==========================================
  Files         264      264              
  Lines       29929    29929              
==========================================
- Hits        19742    19740       -2     
- Misses       7236     7239       +3     
+ Partials     2951     2950       -1     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@777genius
777genius merged commit 8d5f814 into main Oct 4, 2026
36 of 40 checks passed
@777genius
777genius deleted the docs/claude-branding branch October 4, 2026 11:03
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