Skip to content

Brand the logged-out front door as Micro - #1502

Merged
asim merged 1 commit into
mainfrom
fix/micro-landing-brand
Sep 5, 2026
Merged

Brand the logged-out front door as Micro#1502
asim merged 1 commit into
mainfrom
fix/micro-landing-brand

Conversation

@asim

@asim asim commented Sep 5, 2026

Copy link
Copy Markdown
Member

Summary

  • make the logged-out / front door identify as Micro
  • keep the caption A personal assistant
  • leave Mu as the runtime/project/system identity
  • add a focused regression test for both the browser title and visible wordmark

Scope

Landing only. The x402 routing and tools_*x402_* cleanup already landed in #1501 and are intentionally not part of this PR.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 5, 2026

Copy link
Copy Markdown

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-05T08:19:42.615181Z 2286cd4 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.

asim commented Sep 5, 2026

Copy link
Copy Markdown
Member Author

@codex please finish this PR with only the landing implementation change in home/index.go:

  • change the logged-out app.Index title from "Mu" to "Micro"
  • change the visible landing wordmark from <div class="lbrand">Mu</div> to Micro
  • keep A personal assistant unchanged
  • do not change README, About, manifest/PWA branding, x402 routing, or any other files except where strictly required by the focused regression test already in this PR

Run the focused home test and keep the diff minimal.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 2286cd4bfd

ℹ️ 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".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Index(w, r)
body := w.Body.String()

if !strings.Contains(body, "<title>Micro</title>") {

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Change the landing output before asserting Micro

This test fails for every logged-out request because Index still passes Title: "Mu" to app.RenderIndex, and indexBody still emits <div class="lbrand">Mu</div>. Consequently go test ./home (and therefore go test ./...) fails immediately, while the user-visible rename described by the commit is not implemented.

Useful? React with 👍 / 👎.

@chatgpt-codex-connector

Copy link
Copy Markdown

Summary

Testing

  • go test ./home -run '^TestLoggedOutRootIsMicro$' -count=1
  • test -z "$(gofmt -l home/index.go home/landing_brand_test.go)"
  • git diff --check

View task →

asim commented Sep 5, 2026

Copy link
Copy Markdown
Member Author

@codex address that feedback. Implement the two home/index.go substitutions exactly as described, keep the diff minimal, and run go test ./home.

@asim
asim merged commit f415406 into main Sep 5, 2026
1 of 2 checks passed
@chatgpt-codex-connector

Copy link
Copy Markdown

Summary

  • Updated the logged-out landing page’s browser title from Mu to Micro. home/index.goL79-L85
  • Updated the visible landing wordmark to Micro, while leaving A personal assistant unchanged. home/index.goL241-L243
  • Kept the diff limited to the two requested substitutions in home/index.go.
  • Committed the changes as 1759c80 (Finish Micro landing implementation) and prepared the follow-up PR titled Finish Micro landing implementation.

Testing

  • go test ./home
  • test -z "$(gofmt -l home/index.go home/landing_brand_test.go)"
  • git diff --check
  • ⚠️ Screenshot capture was unavailable because the environment does not include a browser or screenshot-capable renderer.

View task →

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