Onboarding polish: on-brand architecture diagram, README refresh, notification mockup - #35
Merged
Merged
Conversation
Add .github/assets/architecture.svg — a self-contained diagram (phone mockup running the Start/Stop app, an outbound wss arrow to a hosted gateway, and a cloud-agent node) in the SealGate dark/cyan style — and reference it from the README Architecture section in place of the ASCII art. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01K129pdFr14dSupQYYVCwzd
Redesign .github/assets/architecture.svg to carry the two key ideas as pictures rather than prose: - Outbound-only transport: one bold wss arrow out, paired with a dashed inbound arrow stopped by a no-entry sign (was a text sub-label). - Local stdio MCP servers drawn as terminal-window glyphs instead of 'stdio processes / stdin / stdout' text. - Cloud agents shown by the ChatGPT and Claude logos; dropped the 'ChatGPT / Claude / ...' caption. - 'Data & logins stay on the device' reduced to a lock badge. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01K129pdFr14dSupQYYVCwzd
Apply the sealgate-brand system to .github/assets/architecture.svg: - Represent the hosted Gateway with the SealGate 'Hackbox' mark: a wireframe square with a solid cyan tag flush to the top-left corner (40% x 10%), labelled SEALGATE / HOSTED. - Switch to the brand palette: Core Cyan #C3FFFD on Baseline Black #000000, Graphene Grey #9BA4A6 for secondary boxes and arrows, Circuit Green #00C781 for the active state, Infra Red #FF3B4D for the blocked inbound state. - Drop the arrow glow, square off the container corners, add a blueprint grid with sparse cyan markers, and render the cloud-agent logos monochrome so the diagram stays monochromatic. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01K129pdFr14dSupQYYVCwzd
- Render the Claude mark in its brand terracotta (#D97757) again instead of monochrome white. - Add the Devin (Cognition) logo, sourced from eito.me/icons, as a third cloud-agent icon beside ChatGPT and Claude. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01K129pdFr14dSupQYYVCwzd
Replace the hand-drawn Hackbox placeholder with the official SealGate wordmark lockup (Cyan_White variant), pulled verbatim from Edison-Watch/sealgate-landing-page (frontend/public/logos/brand/Wordmark). The lockup already carries the Hackbox mark plus the 'SealGate' lettering, so it identifies the hosted gateway as SealGate directly. Reworked the middle node into a horizontal container to fit the wide lockup and adjusted the flanking arrows. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01K129pdFr14dSupQYYVCwzd
Remove labels the pictures already carry: the header eyebrow and title, the GATEWAY / HOSTED eyebrow, and the 'no inbound port' caption (the no-entry sign and one-way arrow already show it). Center the SealGate wordmark in its node and crop the now-empty top band. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01K129pdFr14dSupQYYVCwzd
The filled control still reads as the app's action button without the text. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01K129pdFr14dSupQYYVCwzd
- Add the Android logo (svgl.app) inline next to the lede's Android mention; vendor it at .github/assets/android.svg. - Cut contrastive 'a starter template, not a finished product' down to 'a starter template' and trim the trailing clause. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01K129pdFr14dSupQYYVCwzd
…ments - Remove the Project layout tree (redundant with the repo itself). - Add the foreground-service notification mockup under Getting started; vendor it at .github/assets/notification.svg. - Put the Android logo on the Requirements heading. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01K129pdFr14dSupQYYVCwzd
|
ⓘ Qodo reviews are paused because the subscription is no longer active. Ask your workspace admin to reactivate the subscription to resume reviews. Manage billing |
…tunnel-onboarding-km2ip3 # Conflicts: # README.md
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01K129pdFr14dSupQYYVCwzd
![cubic-dev-ai[bot]](https://avatars.githubusercontent.com/in/1082092?s=60&v=4){kind=small}
There was a problem hiding this comment.
All reported issues were addressed across 1 file (changes from recent commits).
Reply with feedback, questions, or to request a fix.
Re-trigger cubic
…l docs Fixes a dangling CLAUDE.md pointer to the removed README worked example and generalizes the bt_gatt tool descriptions/comment. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01K129pdFr14dSupQYYVCwzd
…am and project-layout tree Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01K129pdFr14dSupQYYVCwzd
…tunnel-onboarding-km2ip3 # Conflicts: # CLAUDE.md
github-actions
Bot
deleted the
claude/android-stdio-tunnel-onboarding-km2ip3
branch
August 28, 2026 18:10
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Polishes the first-run/onboarding presentation of the template — the architecture diagram and README — so the repo reads clearly on-brand at a glance. No app code changes; the tunnel transport stays stubbed.
Changes
Architecture diagram (
.github/assets/architecture.svg)README (
README.md).github/assets/notification.svg) under Getting started so the ongoing-notification behavior is visible.New assets
.github/assets/android.svg— Android glyph..github/assets/notification.svg— Android lock-screen notification mockup, strings sourced fromapp/src/main/res/values/strings.xml.Notes
All embedded SVGs are GitHub-safe (no scripts, external refs, or web fonts). The app builds, installs, and runs exactly as before — this is documentation and presentation only.
🤖 Generated with Claude Code
Generated by Claude Code
Summary by cubic
Refreshes the onboarding docs and architecture diagram so the repo reads on-brand at a glance, and drops the Flipper-specific worked example in favor of general BLE/serial-RPC guidance.
bt_gatt_*tool descriptions inCLAUDE.mdandBluetoothModule.ktso they cover any UART-over-BLE or serial-RPC profile..github/assets/android.svgand.github/assets/notification.svg; notification strings match the app'sstrings.xml.Written for commit ab888ad. Summary will update on new commits.