Repository navigation
docs: restructure the README into a short landing page and topic guides - #70
Merged
Merged
Conversation
This was referenced Oct 3, 2026
Contributor
Author
scott-lowe-vapi
marked this pull request as ready for review
October 3, 2026 06:53
This was referenced Oct 3, 2026
1 of 2 tasks
scott-lowe-vapi
force-pushed
the
docs/public-release-scrub
branch
from
October 6, 2026 22:50
92656e0 to
b25f8d6
Compare
scott-lowe-vapi
force-pushed
the
docs/readme-restructure
branch
from
October 6, 2026 22:50
9a7ae67 to
3cd9c83
Compare
Contributor
Author
Merge activity
|
scott-lowe-vapi
changed the base branch from
docs/public-release-scrub
to
graphite-base/70
October 7, 2026 17:52
The README was 1,100 lines serving newcomers, daily operators, promotion and CI admins and coding agents at once, with its first-run path spread across three sections. It's now a 275-line landing page: what the repo does, a diagram, a five-step quick start, core concepts (including what is and isn't committed), a one-line command table, guides, the Vapi field guide, staying up to date, security, and contributing. Long-form sections move verbatim into docs/guides/ (commands, workflows, file formats, PR checks, promotion, how it works, configuration, troubleshooting); only heading levels and relative links change. Of the old README's 795 content lines, 755 moved verbatim; the rest are the sections rewritten in the new README. Editing the moved content is a separate change, so this one can be reviewed as a move. New README content: - getting a private copy: clone and repush (keeps upstream history), or Use this template, and why not a public fork; - staying up to date, including the template's unrelated-history merge; - the docs/learnings field guide surfaced as a section. References to old README sections (workflows, AGENTS.md, SECURITY.md, simulations.md, the starter example) now point at the guides. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
scott-lowe-vapi
force-pushed
the
docs/readme-restructure
branch
from
October 7, 2026 17:54
3cd9c83 to
453459c
Compare
scott-lowe-vapi
added a commit
that referenced
this pull request
Oct 7, 2026
## Value **V.A.L.U.E. tier:** small — docs only. It edits the guides that #70 moved, for readers who don't know the repo's history, and fixes inaccuracies. Not micro: more than 200 lines. - **Problem:** - **Insider wording:** the moved content read like a changelog ("used to spawn duplicates", "the CLI now refuses", "P0-1 regression suite"). - **Duplicates:** some content still repeated itself. - **Inaccuracies:** a few statements were wrong, and two troubleshooting tips could make things worse. - **What changes:** - **Workflows:** - one section each for deploying (including scoped deploys), starting fresh, creating new resources, pulling, testing, recovering and cleaning up; - the AI-agent callout becomes a note for the user, since `AGENTS.md` already instructs agents. - **Commands:** one sentence per command, and the false claim that every command is interactive is fixed. `migrate` moves to "Upgrading from an older version". - **Configuration:** documents every setting users set: per-org `.env` values and generated bindings, precedence, the promotion and PR check secrets and variables, config files, and the CI and debug variables. - **How it works:** the stale project tree (a third of `src/`, 5 of 68 tests, internal labels) becomes a short "Where things live" table. The guide now says `push` deletes only with `--force`. - **Troubleshooting:** - it no longer advises deleting a state entry, which made the next deploy treat the file as new; - it no longer suggests quietly editing engine code. It points at `audit`'s per-finding suggested fixes, and at opening an issue. ## Evidence of value Every changed claim was checked against the code: - `apply` forwards type and path arguments to `pull` and `push`, which share the parser that rejects bare IDs; - `setup`, `apply`, `pull`, `push`, `cleanup` and `call` are the only interactive commands; - environment variables beat `.env` files, and `.env.<org>.local` only fills gaps; - the EU base URL is `https://api.eu.vapi.ai`; - `push` prints "Deletions: Disabled (pass --force to enable)"; - `audit` reports `state-ghost` and `state-uuid-collision` with a suggested action. All links resolve and `npm test` passes, 496 tests. **Found while checking, not changed here:** `src/config.ts` comments `.env.<org>.local` as "local overrides", but the loader reads it after `.env.<org>` and only fills unset variables, so it can't override anything. The guide documents the actual behaviour. Fixing the code or the comment is a separate change. ## Testing plan - The link checker and `npm test`. - **Not tested:** the commands and workflows against a real org (this PR changes no commands). Stacked on #70. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
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
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.

Value
V.A.L.U.E. tier: small — docs only. This restructures the README for readers arriving from the Vapi docs. Not micro: more than 200 lines and 8 files.
docs/learnings/field guide was only mentioned in a file tree.--allow-unrelated-historiesonce. A public fork would publish the user's configuration.docs/guides/: commands, workflows, file formats, PR checks, promotion, how it works, configuration, troubleshooting. Only heading levels and relative links change, so this PR reviews as a move; docs: edit the guides for new readers and fix inaccuracies #71 edits the content.vapi-checks.yml,AGENTS.md,SECURITY.md,simulations.mdand the starter example) now point at the guides.Evidence of value
npm testpasses, 496 tests, including the snippet test, which now reads the file formats guide.Testing plan
Stacked on #69.
🤖 Generated with Claude Code