docs: edit the guides for new readers and fix inaccuracies - #71
Open
scott-lowe-vapi wants to merge 1 commit into
Open
scott-lowe-vapi wants to merge 1 commit into
scott-lowe-vapi wants to merge 1 commit into
Conversation
The previous change moved the README's long-form content into docs/guides/ verbatim; this one edits it. - workflows: merge the duplicated sections (pull without losing work appeared twice, pull by UUID three times, selective push separately from deploy), drop the old "How to Use This Repo" list now covered by the README, and turn the AI-agent callout into a note for the user (AGENTS.md already instructs agents). - commands: one sentence per command; fix the claim that every command is interactive; move migrate to an "Upgrading from an older version" section. - configuration: document every setting users actually set: per-org .env values and generated bindings, precedence, the promotion and PR check secrets and variables, config files, and CI/debug variables. - how it works: replace the stale project tree (a third of src/, 5 of 68 tests, internal labels) with a short "where things live" table; push deletes only with --force. - troubleshooting: don't advise deleting state entries (that makes the next deploy treat the file as new) or silently editing engine code; point at audit's suggested fixes and an issue. - Present-tense wording throughout (no "used to", "now refuses"). Co-Authored-By: Claude Opus 5.5 <[email protected]>
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
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. 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.
AGENTS.mdalready instructs agents.migratemoves to "Upgrading from an older version"..envvalues and generated bindings, precedence, the promotion and PR check secrets and variables, config files, and the CI and debug variables.src/, 5 of 68 tests, internal labels) becomes a short "Where things live" table. The guide now sayspushdeletes only with--force.audit's per-finding suggested fixes, and at opening an issue.Evidence of value
Every changed claim was checked against the code:
applyforwards type and path arguments topullandpush, which share the parser that rejects bare IDs;setup,apply,pull,push,cleanupandcallare the only interactive commands;.envfiles, and.env.<org>.localonly fills gaps;https://api.eu.vapi.ai;pushprints "Deletions: Disabled (pass --force to enable)";auditreportsstate-ghostandstate-uuid-collisionwith a suggested action.All links resolve and
npm testpasses, 496 tests.Found while checking, not changed here:
src/config.tscomments.env.<org>.localas "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
npm test.Stacked on #70.
🤖 Generated with Claude Code