Plan: add phase D3 — language wrappers + proxy-type renaming#91
Open
ptesavol wants to merge 1 commit into
Open
Plan: add phase D3 — language wrappers + proxy-type renaming#91ptesavol wants to merge 1 commit into
ptesavol wants to merge 1 commit into
Conversation
Resolves the naming question deferred from the D1 review (PR #90) as its own phase: the proxy-prefixed shared infrastructure (ProxyResult, Error, proxyClientResultDelete, the library lifecycle functions, struct Proxy) gets neutral streamr* names in the same breaking release that extends the C++/Swift/Kotlin/Python wrappers with the streamrNode* full-node functionality, so the wrappers are written against the final names once. Old proxyClient* names stay as deprecated aliases for one release; the definition of done is amended accordingly. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
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
Documentation-only change to
trackerless-network-completion-plan.md(pushed with--no-verify: no code touched, so the all-packages lint hook was skipped).Adds Phase D3 — Language wrappers and proxy-type renaming to Milestone D, resolving the naming question deferred from the D1 review (#90) as its own phase, combined with the wrapper work as requested:
streamrNode*full-node functionality: C++ (wrappers/cpp), Swift package (dist/ios-swift-package), Kotlin Android library module (dist/android-library-module), Python (wrappers/python); decide whether the Go wrapper follows or is dropped.ProxyResult,Error,proxyClientResultDelete,proxyClientInitLibrary/proxyClientCleanupLibrary,struct Proxy/StreamrEntryPoint) to neutralstreamr*names — so the wrappers are written against the final names once instead of being renamed later.proxyClient*names remain as deprecated aliases for one release; shared-library major version bump.StreamrNodeTest-style round-trips plus the existing proxy suites against the renamed types.Also amends definition-of-done point 3, which previously promised an "unchanged proxy API", to reflect the deliberate rename-with-aliases.
🤖 Generated with Claude Code