Skip to content

docs: restructure README per README guidelines#14

Merged
Miyamura80 merged 10 commits into
mainfrom
claude/apply-readme-guidelines-danhqp
Jun 29, 2026
Merged

docs: restructure README per README guidelines#14
Miyamura80 merged 10 commits into
mainfrom
claude/apply-readme-guidelines-danhqp

Conversation

@Miyamura80

@Miyamura80 Miyamura80 commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

Summary

Restructures the README to follow the project's README guidelines (pitch first, progressive disclosure, TLDRs, tables, copy-pasteable commands), adapted for a source-consumed library rather than a CLI product.

What changed

  • Pragmatic masthead: kept the existing badges, added a one-line tagline and an inline -separated nav bar.
  • "What is this": a short bullet summary plus the public/auditability positioning statement.
  • Status callout: a > [!NOTE] for the 0.1.0, not-published-to-npm status and the "no secrets in this repo" rule.
  • Architecture diagram: a new committed docs/architecture.svg plus a matching ASCII diagram. Both are deliberately abstract (two capability domains, standard-module distribution, host runtime) and avoid folder names, subpaths, component lists, and specific build tooling, so they survive routine renames and refactoring.
  • Progressive disclosure: Quick start extras, Package entrypoints, Modules, Architecture, and Visual regression tests are behind <details>, each with a one-line TLDR:.
  • Reference tables and annotated config: package entrypoints table and an annotated EnvConfig interface (no live secret values).
  • Credits and a zero-maintenance contrib.rocks contributors footer.

CLI-only guideline sections that do not apply to this package (install one-liner, --help/doctor, agent quickstart, exit codes, leaderboard, API-key env vars) were dropped.

Notes

  • Prose avoids em dashes and contrastive "not just X, but Y" phrasing so it passes the repo's scripts/check-ai-writing.ts CI check.
  • The architecture SVG was rendered to PNG and visually verified for layout/clipping before committing.

🤖 Generated with Claude Code

https://claude.ai/code/session_01BSXhDvj3zDtbbqz8XzL7ke


Generated by Claude Code


Summary by cubic

Restructures the README to follow our guidelines with a pitch‑first layout, progressive disclosure, and copy‑paste commands. Adds a brand‑styled architecture SVG (two‑domain view, host runtime removed), clarifies config scope and the unpublished status for @edison/shared, and removes the duplicate Architecture section and ASCII diagram.

  • New Features

    • Masthead adds a one‑line tagline and inline nav (Architecture link removed).
    • Tight “What is this” with the public/auditability positioning.
    • Status callout: 0.1.0, not published to npm, and “no secrets in this repo”.
    • Progressive disclosure with TLDRs for Quick start, Entrypoints/Modules, and Visual tests.
    • Reference updates: entrypoints table, annotated EnvConfig, peer deps note, and clear config scope.
    • New brand‑styled docs/architecture.svg (simplified to design‑system + client‑services; host‑runtime layer and inbound arrow removed; alt text updated).
    • Credits and a contrib.rocks contributors footer.
  • Bug Fixes

    • Formatted README with Prettier to fix the format:check CI failure.
    • Removed repo‑convention notes from Contributing.

Written for commit dbe7dbc. Summary will update on new commits.

Review in cubic

claude added 5 commits June 29, 2026 11:29
Apply the project README guidelines to @edison/shared, adapting the
CLI-oriented patterns to a source-consumed library:

- Lead with a dense "what is this" paragraph and a one-line tagline
- Add an inline nav bar and a NOTE maturity callout (0.1.0, unpublished)
- Add a committed architecture diagram (docs/architecture.svg) plus an
  ASCII module/build layout in a collapsible Architecture section
- Use progressive disclosure with TLDR lines for Quick start, Visual
  regression tests, and Architecture
- Tabulate package entrypoints and link modules out to src/
- Show the EnvConfig schema as an annotated example
- Add Credits and a contrib.rocks contributors footer

Avoids em dashes and contrastive parallelism so the AI-writing CI check
passes.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01BSXhDvj3zDtbbqz8XzL7ke
Replace the prose paragraph with the prior four-bullet summary.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01BSXhDvj3zDtbbqz8XzL7ke
Redraw the SVG and ASCII diagrams around durable concepts (two
capability domains, standard-module distribution, host runtime) instead
of folder names, subpaths, component lists, and specific build tooling,
so the diagrams survive routine renames and refactoring.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01BSXhDvj3zDtbbqz8XzL7ke
Move both reference sections behind <details> with TLDR lines so the
top-level README scan stays short.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01BSXhDvj3zDtbbqz8XzL7ke
@qodo-code-review

Copy link
Copy Markdown

Qodo reviews are paused for this user.

Troubleshooting steps vary by plan Learn more →

On a Teams plan?
Reviews resume once this user has a paid seat and their Git account is linked in Qodo.
Link Git account →

Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center?
These require an Enterprise plan - Contact us
Contact us →

claude added 5 commits June 29, 2026 12:11
Fixes the CI format:check failure (aligned tables, comment spacing).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01BSXhDvj3zDtbbqz8XzL7ke
Redraw docs/architecture.svg per the Edison brand guide: void-black
field, electric-cyan wireframe boxes with solid corner tags (40% width,
varied placement for rhythm), white Archivo-stack typography, cyan grid
markers and arrows. Architecture content is unchanged.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01BSXhDvj3zDtbbqz8XzL7ke
The brand SVG near the top already conveys the architecture, so the
duplicate text diagram and its section are removed (and dropped from the
nav bar).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01BSXhDvj3zDtbbqz8XzL7ke
Drop the host-runtime layer (and the inbound arrow), tighten the canvas
height, and update the README alt text to match.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01BSXhDvj3zDtbbqz8XzL7ke
@Miyamura80 Miyamura80 merged commit bcb8274 into main Jun 29, 2026
4 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