feat(core+wizard): wire GonkaProvider end-to-end with native wizard branch (#3613, #3603)#3632
Merged
feat(core+wizard): wire GonkaProvider end-to-end with native wizard branch (#3613, #3603)#3632
Conversation
…ranch (#3613, #3603) - Add AnyProvider::Gonka variant with full delegate_provider! coverage, status_tx forwarding, and variant-level unit tests (#[cfg(feature = "gonka")]) - Implement build_gonka_provider factory: vault key resolution, RequestSigner construction, optional address mismatch check (lowercase-normalized), EndpointPool from GonkaNode list, tracing span; Zeroizing<String> for private_key_hex and ProviderConfigSnapshot.gonka_private_key - Add mandatory address: String field to GonkaNode config struct - Add native Gonka wizard branch in --init: inferenced CLI detection, key selection/export with manual hex fallback, node configuration, age vault enforcement; WizardState.gonka_private_key: Zeroizing<String> - Add config migration step 45: advisory comment inserted before [[llm.providers]] for GonkaGate entries with idempotency guard - Add insta snapshot test for Gonka provider TOML output - Add GonkaProvider Clone impl, with_generation_overrides, and set_status_tx - Add Gonka integration guide at book/src/guides/gonka.md covering GonkaGate and native paths with troubleshooting section (#3603) Closes #3613 Closes #3603
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
1 participant
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
GonkaProviderend-to-end:AnyProvider::Gonkavariant,build_gonka_providerfactory, native wizard branch, config migration step, and integration docsaddress: Stringfield toGonkaNodeconfig struct (required for ECDSA signature target)Zeroizing<String>across factory, wizard state, and provider snapshotGonkaProvidergainsstatus_txsupport — TUI status indicators now work during native Gonka inferencebook/src/guides/gonka.mdcovering both GonkaGate and native pathsChanges
AnyProvider::Gonkavariant + testscrates/zeph-llm/src/any.rsGonkaProviderClone,status_tx, overridescrates/zeph-llm/src/gonka/provider.rsbuild_gonka_providerfactory (Zeroizing, tracing span)crates/zeph-core/src/provider_factory.rsGonkaNode.addressfieldcrates/zeph-config/src/providers.rssrc/init/llm.rs,src/init/mod.rscrates/zeph-config/src/migrate/crates/zeph-core/src/agent/state/mod.rsbook/src/guides/gonka.md,book/src/SUMMARY.mdsrc/init/snapshots/zeph__init__tests__gonka_provider_entry_toml.snapTest plan
cargo +nightly fmt --check— PASScargo clippy --workspace --features full -- -D warnings— PASScargo nextest run --workspace --features full --lib --bins— 9284/9284 PASScargo insta test --workspace --features full --check --lib --bins— no new snapshotsZEPH_GONKA_PRIVATE_KEYin age vault + GNK staking — tracked in CI playbook.local/testing/playbooks/gonka-native.mdCloses #3613
Closes #3603