Repository navigation
Fix memory-aware orchestration request resolution - #1786
Merged
Paul Lizer (paullizer) merged 2 commits intoOct 19, 2026
Merged
Conversation
Resolve ambiguous current-message sources contextually and supply authorized read-only memory before clarification decisions. Preserve retrieval and audience safeguards, add private diagnostics and regressions, and document separate harness repairs. Refs #1785 Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Use explicit string addition for both multiline parametrized prompts while preserving all three request inputs. Update required version and validation documentation. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Paul Lizer (paullizer)
merged commit Oct 19, 2026
beb26d2
into
paullizer-react-v2-ui
12 checks passed
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.
Summary
The eight supplied UI scenarios all stopped on turn one. Six failures concern transient pending-message selectors and one concerns incidental wording; these harness issues are documented separately from the application fix. This PR neither changes the external harness nor certifies the previously unexecuted later turns.
Linked issue
Refs #1785
Release Notes & Latest Features
Is this visible to end users?
Is this admin-facing (Admin Settings, governance, deployment, config)?
Should this become a Latest Feature card?
Screenshot needed for the card?
Version bump
application/single_app/config.pyVERSIONthird segment bumped, or not needed because this is docs-onlydeployers/version.txtbumped, or not needed becausedeployers/was not changedApplication version:
0.261.348 -> 0.261.349. No deployer changes.Testing / validation
Commands ran from the repository root with
PYTHONPATHcontainingapplication\single_appandfunctional_tests. Below,pythondenotes the isolated session interpreter. Repository-pinned Flask/Werkzeug, Cosmos SDK, and the missing YAMCS client were restored there; an isolated compatible cryptography package resolved the shared installation's OpenSSL incompatibility. Global packages and application requirements were not changed.python functional_tests\route_tests\test_route_blueprint_policy_inventory.py- passed, 15 checks.python functional_tests\route_tests\test_route_unauthenticated_policy_contract.py- passed, 10 checks.python functional_tests\route_tests\test_route_policy_test_coverage.py- passed, 3 checks.python functional_tests\test_docs_app_surface_coverage.py- passed, 7 checks; generated application-surface inventory remains current.python functional_tests\test_docs_site_quality.py- passed, 6 checks.git diff --checkand staged whitespace validation - passed.Deterministic completions verify context delivery and state transitions, not live model compliance. No production deployment or full FED-31 through FED-38 rerun was performed. The harness owner still needs to repair selectors/assertions and rerun all complete sequences, including mixed modes, reloads, and updated-memory/stale-value checks.
Documentation
Release notes updated, or not needed
Feature documentation updated, or not needed
Fix documentation updated, or not needed
Durable application and harness-owner guidance:
docs/explanation/fixes/ORCHESTRATION_REQUEST_MEMORY_RESOLUTION_FIX.md.Updated
CHAT_ORCHESTRATION.mdanddocs/reference/logging-tags.mddescribe early memory resolution, recall lifecycle, and safe diagnostics.Release-note changes were offered but not approved while the user was unavailable, so
docs/explanation/release_notes.mdis unchanged. No additional issue status comment was posted.Security checklist
@swagger_route(security=get_auth_security())- N/A: no new routes; existing route policy checks pass.sanitize_settings_for_user()- N/A: no settings response changes.