Repository navigation
Document uncaught Nexus caller failure coverage - #197
Merged
Merged
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.
Follow-through for #122 and Server #489.
Describe the separate uncaught-failure cases in the existing published Nexus caller command: four PHP/Python directions, shared plus tenant namespaces, original remote/caller failure, one failed call and workflow, unchanged original identity/clocks and thirty-second budgets. Keep handled error results, legacy invocable-carrier cells, embedded execution and native qualification distinct.
Clarify that the existing 2 MiB retained-report bound may omit full reports and names those omissions in
outcomes.json. No generated evidence, manifest, runtime, CI or quality settings change.The exact Server candidate passes actual SDK execution 39113849803, feature/source, Sonar and boundary checks. All eight distinct uncaught originals pass, alongside the sixteen external/auth originals. Merged source
c38f8e34, identical reviewed treeca5d1df3, now also passes independent fresh-main 39114545513, feature/source 39114545566, Sonar and boundary checks. All nine complete outcome notices were reviewed; eight distinct uncaught originals pass under the original budgets, maximum 3.869 seconds. The original reader failure remains failed.Verification: full two-file diff review and
git diff --check; normal repository and Sonar checks qualify the documentation. No prose-matching tests are added.Delivered at
8f7e0f1a0f31a160ea5471df5663cf5d67e31a65, exact reviewed treee8f276d8a44d608dbcf0e31c38eafd67df00b23b. Independent main repository checks 39115303042 and main Sonar 39115303059 pass. The completed branch/worktree is removed. The owning delivery is .github #122; other inventory gaps remain open there.