docs(tui): unify help, anatomy, and preset operations#691
Merged
Conversation
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.
Summary
This is the single combined documentation/skills PR requested for review.
Help and runtime anatomy
lingtai-tui-helpinto a thin discoverable umbrella for the stable TUI mental model and machine-readable + human feature routinglaunchdlingtai-tui-anatomywith the repo-rootANATOMY.mdas normative owner instead of duplicating the conventionPreset guidance
lingtai-preset-skillas one thin dual-axis router: exactly 12 source-derived provider children plus 5 operation children (saved-presets,endpoint-capabilities,availability-save-gate,activation-session-refresh,troubleshooting-migration)codex-poolwith v1/v2 pool semantics, sticky weighted selection, safe non-secret inspection, authorization/backup/compare/atomic-replace/parser-validation boundaries, and the explicit refresh/new-session activation stepinitialize→account/rateLimits/readwithparams: null), safe per-window fields, and the dated/model/route-qualified context-window evidence without treating it as timelessValidation
Parent final gate:
cd tui && go test ./... -timeout 300s— PASS, all 14 packagesgo build ./...— PASSgo vet ./...— PASSgit diff --check— PASSFrozen independent final review:
1dda8f4bb0cf0d80d6469f2d58557e3b37bb3d8554ec5c88928716c53494b3ad375e2c998b989e22550e45f28251a3073ff40e35gpt-5.6-terrago test ./...: PASS, all 14 packagesThe three r1 findings are fixed and independently verified: OpenRouter vision is conditional on a user-owned saved preset advertising
vision; no false canonical template-order promise remains; emptytieris accepted as source does.Current
mainadvanced after the frozen base only through Windows PowerShell installer PR #690 (3 upstream paths, zero overlap with this PR's 28 paths).Scope
Documentation, bundled skills, Anatomy, and focused contract tests only. No runtime behavior, provider auto-routing, release, deploy, config, auth, cleanup, or lifecycle action.