Repository navigation
fix(branding): finish NeoCode and NeoPi display identity - #108
Merged
Merged
Conversation
Finish the user-visible identity cutover: NeoPi provider labels and diagnostics no longer use the combined NeoPi/OMP name, and local CLI, project-schema and triage guidance identify NeoCode. Preserve explicit instance names, OMP executable compatibility, command names, saved state paths, protocol identifiers and external T3 Connect attribution. Route triage playbook refresh and source fallback through this fork's nightly branch. Directly reading the current main playbook showed it still redirects to upstream, so fetching main would undo the fork identity change before promotion. Remove incidental wording and prompt-copy assertions while retaining structured validation, lifecycle and argv-safety coverage. Record the upstream touches. Verified twelve focused files: 120 tests passed initially and one prior-brand regex failed. Replacing that wording assertion with the validation error tag passed the single-case rerun; the triage argv test and throwaway canonical/embedded playbook consistency check also passed after the branch correction. All changed files were formatted. GUI after-proof follows the coordinated installed desktop build; no native mobile execution is claimed. Refs #59 Model: openai-codex/gpt-6-astra Co-authored-by: GPT-6 Astra <noreply@openai.com> Co-authored-by: GPT-6.1 Sol <noreply@openai.com>
1 of 3 tasks
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.
What
Finish the visible NeoCode/NeoPi identity cutover across CLI guidance, provider settings, usage presentation, shared display fallbacks and provider diagnostics. The harness is named NeoPi, not NeoPi/OMP. Existing explicitly named instances remain authoritative, including names saved by older versions.
Project-schema guidance names NeoCode without changing
t3.jsonfields or its compatibility URL. Triage uses this fork's repository and nightly playbook; upstream attribution remains explicit.Why
Addresses #59. NeoCode is NeoPi's graphical frontend, not an upstream T3 Code instance with an optional provider. Earlier changes already established application branding, README guidance, NeoPi-first ordering and fresh-settings defaults; this removes the remaining bundled-app naming gaps.
A direct read of this fork's current main playbook showed upstream instructions. Refreshing from main would therefore undo the new fork routing before promotion. The playbook now refreshes from nightly and uses nightly as the fallback source branch.
No command names, provider IDs, data directories, saved choices or protocol identifiers change. OMP executable compatibility and other harnesses remain supported. Actual upstream services, legal attribution, quoted testimonials and upstream distribution guidance are not relabeled as fork services. CLI release provenance is separately repaired by merged #107; its stable publisher follow-up is #106.
Testing
bin.test.ts; CLIapp.test.ts,pair.test.ts,service.test.ts,triagePrompt.test.ts; providerNeoPiAdapter.test.ts,neopiUsageLimits.test.ts,mappers/ui.test.ts,NeoPiDriver.test.ts; shared/contractst3ProjectFile.test.ts; client-runtimeproviderInstanceDisplay.test.ts.c9f91e00a43096e5f94e050627738b26bc7a0c9b, version0.0.42-nightly.20261020.c9f91e00, includes this PR and fix(releases): download CLI runtimes from the NeoCode fork #107. Real launch and Settings → Providers passed in private Xvfb:10, with normal shell hydration. NeoPi is first, enabled/authenticated, and the savednpi-nightlypath survived. The six other harnesses remain disabled in this fixture. Before/after screenshots were saved and viewed; no owner database/wallet was used. Installed receipt.c9f91e00a43096e5f94e050627738b26bc7a0c9b. The separate CI AppImage is not claimed byte-identical.--helpalso identifies NeoCode. No full suite or package typecheck; no Windows/native mobile execution or repeated session-restoration/model-turn acceptance run claimed.vp checkandvpr typecheckpass (CI runs both; locally run only the checks for the touched scope, perAGENTS.md› Verifying) — Focused regressions and formatting only.Integration: openai-codex/gpt-6-astra. Implementation: openai-codex/gpt-6.1-sol. Focused verification: openai-codex/gpt-6-luna. Harness: NeoPi.