docs(devlog): close the brand icon unit with its on-screen and CI record

[lidge-jun]

Summary

• Closes the brand icon unit. The app icon and menu bar mark (feat(desktop): draw the app icon and menu bar mark from the brand artwork #5355), the alpha-channel fix the bundle build caught (fix(desktop): keep the alpha channel every generated icon needs #5356) and the two favicons (feat(brand): render the dashboard and documentation favicons from the app icon vector #5361) are all on dev, so the unit moves from devlog/_plan/ to devlog/_fin/ with a terminal record.
• The record states what was actually observed rather than what was intended: Finder and the Dock showing the mark on the system rounded rectangle, the menu bar status item rendering the template with both prompt glyphs as holes, and the two favicons read in a real browser tab rather than in a composite. The two screenshots that back those claims are committed alongside.

• Exact-head CI at 917d690ecb is green, including the aggregate ci check. The first attempt failed one job, macos 1/2, on tests/server/memory-watchdog.test.ts — serializes only an allowlisted Bun runtime provenance ([Bug]: doctor repeats OPENCODEX_BUN_PATH guidance when a Bun canary override is already active #848) — at 47.4s against its own 20s timeout. That test makes eight full /api/system/memory route calls and its own comment records the route costing roughly 600ms per read on shared runners. It passed at 64b0eca2b0, which already carried both icon commits, and failed at 07e2ac9b41 before any of this landed; a focused local run finishes in 377ms. Re-running the job at the same SHA turned it and ci green. The flake is recorded rather than papered over — the fix is to stop that test paying for eight route snapshots, not to widen the timeout again, and that is separate scope.
• Also corrects the icon check command in the plan: icons:check is a desktop package script and does not run from the repository root.

Verification

• bun test tests/ci-workflows/brand-favicons.test.ts tests/ci-workflows/build-desktop-icon-set.test.ts tests/test-layout.test.ts — 14 pass, 0 fail.
• Exact-head check-runs for 917d690ecb read at the SHA, distinguishing requested jobs from skipped ones: every requested job green, macos control / structure gate / the Windows shard placeholder skipped and named as such.
• This change is documentation and two screenshots; no source or generated asset moves.

Checklist

• Scope stays focused and avoids unrelated cleanup.
• Docs or release notes were updated when needed.
• Security-sensitive changes were reviewed for secrets, auth, and unsafe defaults.

[chatgpt-codex-connector]

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review	Status	Commit	Review trigger
📝 Code Review	✅ Completed 2026-09-20T17:44:19.411178Z	287321c	PR opened

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

[github-actions]

✅ Deterministic PR hygiene checks passed.

[coderabbitai]

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration

Configuration used: Repository: lidge-jun/opencodex/.coderabbit.yaml

Review profile: ASSERTIVE

Plan: Advanced

Run ID: dab7027c-bfcd-4e24-97fe-e23b17b6f556

📥 Commits

Reviewing files that changed from the base of the PR and between 917d690 and 287321c.

⛔ Files ignored due to path filters (2)

• assets/pr-screenshots/app-icon-finder-menubar.png is excluded by !**/*.png
• assets/pr-screenshots/favicon-browser-tab.png is excluded by !**/*.png

📒 Files selected for processing (3)

• devlog/_fin/260921_brand_icon_and_menu_bar_mark/000_plan.md
• devlog/_fin/260921_brand_icon_and_menu_bar_mark/010_favicons.md
• devlog/_fin/260921_brand_icon_and_menu_bar_mark/020_closure.md

 ______________________________________________________________________________________________
< Rabbits with frickin' laser beams attached to their heads. And they're pointed at your bugs. >
 ----------------------------------------------------------------------------------------------
  \
   \   (\__/)
       (•ㅅ•)
       / 　 づ

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Commit to this branch
• Create a new PR

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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