ADC-681: resolve Reflux through the public AMR provider protocol - #654
Draft
wolf75222 wants to merge 3 commits into
Draft
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.
Stack
This draft is stacked on #638, which is itself stacked on #617. All existing code, test, documentation, and merge commits are preserved; this branch adds exactly three thematic commits.
Strict audit result
Current
masterdoes not contain the #617/#638 commits. Acrossmaster + #617 + #638, the generated Reflux ABI, prepared adapter, local execution, PoPS-owned collective publication, and internal pybind installation seam exist. The first confirmed remaining source gap was the explicit one documented by #638: Reflux was not part of normalized public AMR provider resolution, so onlyclusteringandtaggercrossed resolve/compile/bind.What changed
RefluxProvider(component=...)authoring and builtinFluxRegisterReflux;AMR(..., reflux=...)with the builtin as its explicit default authority;component_installerfrom the native-interface protocol rather than branching on a concrete provider class;Validation
git diff --checkpassed.The native introspection test was intentionally not run: this isolated worktree has no locally built
_popsextension and the root native build was left untouched.Deliberate non-claims / remaining ADC-681 work