Skip to content

docs: restyle README to match the CodeBoarding repo#4

Merged
brovatten merged 6 commits into
mainfrom
docs/readme-codeboarding-style
Jun 6, 2026
Merged

docs: restyle README to match the CodeBoarding repo#4
brovatten merged 6 commits into
mainfrom
docs/readme-codeboarding-style

Conversation

@brovatten
Copy link
Copy Markdown
Member

Formatting-only pass to align the action README with the CodeBoarding engine repo's README style.

  • Plain title + tagline + intro (drop the centered logo block).
  • ·-separated links line + the same language badges.
  • Sentence-case section headers, period-terminated bullets, lighter bold.
  • Remove the 🟩🟨🟥 / 👀 emoji (use plain color wording).

Content is unchanged — formatting only. Open questions for review: keeping the logo removed, and UsageQuick start.

Match the engine repo's README formatting: plain title + tagline + intro, a
`·`-separated links line, language badges, sentence-case section headers,
period-terminated bullets, lighter bold, and no emoji legend (drop the centered
logo block). Content is unchanged — formatting only.
@brovatten brovatten changed the base branch from feat/mermaid-architecture-diff to main June 6, 2026 14:58
brovatten added 5 commits June 6, 2026 17:02
Title -> 'CodeBoarding Review'. Replace the toy gateway/cache sample with
CodeBoarding's actual level-1 components (from .codeboarding/analysis.json),
colored as a diff: 1 added (green), 2 modified (yellow), 1 deleted (red dashed),
with edges colored to match.
Default OpenRouter runs to google/gemini-3-flash-preview (agent) and
google/gemini-3.1-flash-lite-preview (parsing) when the user doesn't pin a
model. Gated on provider==openrouter, so other providers still defer to their
own engine default (multi-provider intact). Both slugs verified to exist on
OpenRouter. Remove the 'Model format on OpenRouter ...' README paragraph and
update the Inputs table.
Add an early sticky-comment step (same 'codeboarding-architecture-diff' header)
right after the guard, before the multi-minute checkout/analysis, so the PR
shows an 'analyzing…' comment within seconds. The final post step edits that
same comment in place with the diagram (and the failure step replaces it on
error). best-effort (continue-on-error) so a placeholder hiccup can't fail the
run. Consumers get the Qodo/CodeRabbit-style placeholder for free.
The CI lint job (shellcheck) failed on three info-level findings in run_local.sh
(pre-existing since the run_engine rewrite): SC2030/SC2031 from re-exporting
OPENROUTER_API_KEY inside run_engine's subshell, and SC2015 from the
A && B && C || {…} precondition. Drop the redundant key re-export (it's inherited
from the env) and rewrite the precondition as an if. shellcheck now passes.
@brovatten brovatten merged commit 2927270 into main Jun 6, 2026
2 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.

1 participant