Skip to content

docs(readme): refine benchmark and architecture visuals - #32

Merged
Shuyi-zsy merged 4 commits into
mainfrom
docs/readme-post-merge-refinements
Aug 27, 2026
Merged

docs(readme): refine benchmark and architecture visuals#32
Shuyi-zsy merged 4 commits into
mainfrom
docs/readme-post-merge-refinements

Conversation

@Yofuria

@Yofuria Yofuria commented Aug 27, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • replace Efficient Context with Less Tokens in the benchmark headline
  • replace the Architecture diagram with the refined compact version
  • fix label overlap and text overflow in the updated Architecture SVG

Validation

  • ruff check .
  • ruff format --check .
  • Architecture and benchmark SVG files parsed successfully
  • git diff --check origin/main...HEAD

@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: e7d108b9f7

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

Comment thread assert/architecture.svg Outdated
<path d="M380 545V548Q380 560 392 560H608Q620 560 620 566" class="arrow"/>
<path d="M1020 545V548Q1020 560 1008 560H792Q780 560 780 566" class="arrow"/>
<path d="M645 299V329Q645 341 631 341H394Q380 341 380 353V356" class="arrowPurple"/>
<path d="M805 299V329Q805 341 819 341H1006Q1020 341 1020 353V356" class="arrowPurple"/>

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Route the context manager only to the HTTP service

In the README architecture diagram, this new branch runs from the BCG Context Manager directly into the Python SDK, even though the manager always constructs BcgClient and sends evicted turns to the HTTP /turns endpoint (agent-cli/src/core/context/bcg-context.ts:588, agent-cli/src/core/context/bcg-client.ts:39-49). The SDK is a separate direct-backend integration, as the SVG description itself states, so readers following this diagram may incorrectly expect the bundled Agent context manager to support an SDK/in-process path; remove this branch or originate it from a separate SDK consumer.

Useful? React with 👍 / 👎.

@Yofuria
Yofuria requested a review from Shuyi-zsy August 27, 2026 07:04
@Shuyi-zsy
Shuyi-zsy merged commit 283a2aa into main Aug 27, 2026
5 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.

2 participants