Skip to content

docs(readme): rewrite root README for developer onboarding - #17347

Merged
os-sales merged 1 commit into
mainfrom
claude/readme-optimization-75w14b
Sep 10, 2026
Merged

docs(readme): rewrite root README for developer onboarding#17347
os-sales merged 1 commit into
mainfrom
claude/readme-optimization-75w14b

Conversation

@os-sales

Copy link
Copy Markdown
Collaborator

Summary

Rewrites the root README.md with three goals from the maintainer: remove every count-based statistic, delete low-value sections, and lead with what attracts a developer.

Removed

  • The static "tests passing" badge, the package count, the Zod schema count, the CRM file/line/token counts and context-window percentage. Nothing maintained these numbers and several had drifted.
  • The forward-deployed-engineer pitch, the full monorepo scripts table, and the standalone Roadmap / Architecture / Contributing / Documentation stubs (folded into "Hack on the framework" and "Community").

Restructured into: hero → what's in this repo vs ObjectOS → five-minute loop → what one definition gives you (with the REST curl now targeting the same support_desk_ticket object the example defines) → why mistakes don't ship → MCP → ship it → hack on the framework (CLI, layers, collapsed package directory) → community → license.

Facts refreshed against the tree

  • Package directory now lists every published package in packages/ plus all five examples (the old table listed roughly half; app-showcase, which pnpm dev runs, was missing).
  • CLI list matches packages/cli/src/commands (adds verify, diff; points at os --help for the rest).
  • Toolchain line now says pnpm 10 (matches packageManager); upgrade link points at the current /docs/upgrading page instead of the 10→11 note.
  • Links LICENSING.md next to the "no open-core asterisks" claim.

Validation

  • node scripts/check-role-word.mjs — OK
  • node scripts/check-bash32-floor.mjs — OK
  • check-doc-anchors needs pnpm install (exit 3, prerequisite); the new README contains no #fragment links, so it has nothing to judge here.

Docs-only change; publishes nothing from any released package, so this PR carries no changeset (the skip-changeset label applies).

🤖 Generated with Claude Code

https://claude.ai/code/session_016gFSzdPX4324QMRKCjccTc


Generated by Claude Code

Drop every count-based statistic (test badge, package/schema/file/line/token
counts) and the sections that carried little value for a first-time reader
(FDE pitch, full scripts table, standalone roadmap/architecture/contributing
stubs). Lead with the five-minute loop, what one definition yields, why
mistakes don't ship, MCP, shipping, and a single "hack on the framework"
section.

Refresh the facts the old copy had drifted from: the package directory now
lists every published package and example in the workspace, the CLI list
matches packages/cli/src/commands, the toolchain line matches
packageManager, and the upgrade link points at the current docs page.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_016gFSzdPX4324QMRKCjccTc
@github-actions github-actions Bot added size/m documentation Improvements or additions to documentation labels Sep 10, 2026
@os-sales os-sales added the skip-changeset PR has no user-facing published change; bypasses the changeset gate label Sep 10, 2026 — with Claude
@os-sales
os-sales marked this pull request as ready for review September 10, 2026 10:35
@os-sales
os-sales enabled auto-merge September 10, 2026 10:35
@os-sales
os-sales added this pull request to the merge queue Sep 10, 2026
Merged via the queue into main with commit e1eee43 Sep 10, 2026
33 of 34 checks passed
@os-sales
os-sales deleted the claude/readme-optimization-75w14b branch September 10, 2026 10:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation size/m skip-changeset PR has no user-facing published change; bypasses the changeset gate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants