Skip to content

docs: rebrand Cordari to RoveNotes in README#37

Merged
rsteckler merged 1 commit into
mainfrom
agent/engineer/4b7d1d29
Jul 12, 2026
Merged

docs: rebrand Cordari to RoveNotes in README#37
rsteckler merged 1 commit into
mainfrom
agent/engineer/4b7d1d29

Conversation

@rsteckler

Copy link
Copy Markdown
Owner

Rebrands the managed-version callout in the README from Cordari to RoveNotes (cordari.ai → rovenotes.com).

  • Point the managed-version link to rovenotes.com
  • Swap the Cordari logo for the RoveNotes icon (assets/rovenotes-logo.png, sourced from the rovenotes-cloud repo)
  • Remove the now-unused assets/cordari-logo.png

Resolves APP-6.

Point the managed-version callout to rovenotes.com and swap the
Cordari logo for the RoveNotes icon.

Co-authored-by: multica-agent <github@multica.ai>
@github-actions

Copy link
Copy Markdown

Code Review: PR #37 — docs: rebrand Cordari to RoveNotes in README

Reviewed at commit 6f4a686bc5a36c76ebd6aabfb774a2cab73905b8.

Overview

Pure docs/asset rebrand: swaps the "managed version" callout in README.md from Cordari (cordari.ai) to RoveNotes (rovenotes.com), replaces assets/cordari-logo.png with assets/rovenotes-logo.png, and deletes the old logo. No source code, config, or CI is touched. Resolves APP-6.

Strengths

  • Old asset is deleted rather than left orphaned — no dead file lingering in assets/.
  • Link text, alt text, and image path are all updated consistently in the same hunk.
  • New logo is a valid 400×400 RGBA PNG, reasonably sized (~17KB), consistent with typical README badge/icon usage.
  • Repo-wide grep confirms no other references to "Cordari" remain anywhere outside this diff — the rebrand is complete, not partial.

Issues

No issues found. The diff is minimal, self-contained, and fully accomplishes what the PR description states.

Test coverage

N/A — this is a documentation/asset-only change with no executable code paths. Nothing to test beyond visually confirming the README renders correctly on GitHub, which is low-risk for a two-line text swap plus an image replacement.

Security

No issues. No credentials, executable code, or user input involved; the only external-facing change is a plain outbound link (rovenotes.com) in prose.

Verdict

ship-as-is

Minimal, correct, fully self-contained rebrand with no loose ends (old asset removed, no stray references left behind).

@rsteckler
rsteckler merged commit a5e0751 into main Jul 12, 2026
2 checks passed
@rsteckler
rsteckler deleted the agent/engineer/4b7d1d29 branch July 12, 2026 21:48
TimHayward added a commit to TimHayward/applaud that referenced this pull request Jul 17, 2026
…CI, v0.5.11)

Integrates all outstanding upstream (rsteckler/applaud) commits into this fork
while preserving local customizations (UI refactor, webhooks, multi-region auth,
tooling). Brings in:

  fc8410b CI: trusted auto-review for same-repo PRs + maintainer /review gate (rsteckler#33)
  50a64ab Plaud first-party (pld_ut/pld_urt) auth with tokenstr fallback (rsteckler#32)
  82f15e2 Windows: skip auto-detect, route straight to manual paste (rsteckler#34)
  de6002a Mimic rovenotes manual-paste instructions (rsteckler#35)
  57bb643 Auth instructions: real anchor links + clearer legacy lead-in (rsteckler#36)
  a5e0751 docs: rebrand Cordari to RoveNotes in README (rsteckler#37)
  7b1b504 Release v0.5.11

Conflict resolutions:
  - install.ps1: keep TimHayward fork URLs, take upstream v0.5.11 ref.
  - config-helpers.ts: redact lanToken (local) + ut/urt (upstream).
  - auth.ts: keep local multi-region validateToken loop, layer in upstream's
    first-party acceptFirstParty flow and session-based watch handler.
  - client.ts: re-add resolveRegionFromDomain (used by upstream parseRegionRedirect)
    alongside the local region-map helpers.
  - test fixtures: add authMode/ut/utExp/urt/urtExp/plaudWorkspaceId fields.

Verified: pnpm typecheck, pnpm test (256 pass), pnpm build all green.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
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