[codex] Document current neo TUI specs#22
Closed
IYENTeam wants to merge 1 commit into
Closed
Conversation
The Rust native TUI docs referenced spec files that were never checked in and still described copied dist assets, nextest-only testing, ColorSupport detection, and leader dispatch that do not match the implementation. Add the missing specs and update the package README/AGENTS guidance to reflect the current parser and build behavior.\n\nConstraint: Keep this PR documentation-only; do not add new runtime features for parsed-but-unused metadata.\nRejected: Implement leader-key dispatch and ColorSupport detection | out of scope for correcting documentation drift.\nConfidence: high\nScope-risk: narrow\nTested: git diff --check\nNot-tested: Runtime tests; documentation-only change
Owner
|
[sisyphus-bot] Thanks for documenting the neo TUI specs here, IYEN. Unfortunately this PR no longer has a target: |
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
3 participants
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.
What changed
packages/neo-tui/docs/theme-spec.mdandpackages/neo-tui/docs/keymap-spec.mdfiles referenced by the package docs.packages/neo-tui/README.mdandpackages/neo-tui/AGENTS.mdto match the current Rust implementation:include_str!, not copied intodistcargo test;cargo nextestis optionalleadermetadata is parsed but not dispatchedWhy
The docs mixed current runtime facts with older scaffold-plan claims, which made the native Rust TUI harder to maintain safely.
Validation
git diff --checknpm run checkNot tested
Summary by cubic
Add
packages/neo-tui/docs/theme-spec.mdandpackages/neo-tui/docs/keymap-spec.md, and updatepackages/neo-tui/README.mdandpackages/neo-tui/AGENTS.mdto match the current Rust TUI. Docs now reflect embedded assets viainclude_str!, defaultcargo test, single-key dispatch (leader parsed but not dispatched), true‑color as a parsed hint, and correct keybindings (Ctrl+D, history); docs-only change.Written for commit 8f15dec. Summary will update on new commits. Review in cubic