Repository navigation
Fix native workflow alerts in chat proposals and AI Assist - #1742
Merged
Paul Lizer (paullizer) merged 1 commit intoOct 9, 2026
Conversation
Expose conditional alert rules and acknowledgment options through closed authoring schemas, preserve existing-rule edits, and disclose attention options before proposal approval. Improve model capability guidance without keyword routing, add regression coverage, and bump the app to 0.261.315. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Paul Lizer (paullizer)
merged commit Oct 9, 2026
f886f7c
into
microsoft:paullizer-react-v2-ui
11 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
Linked issue
N/A. No issue is associated with this work.
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?
Release-note updates were offered but not confirmed, so release notes remain unchanged.
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.314->0.261.315. No deployer changes.Testing / validation
python -m pytest -q functional_tests\test_workflow_ai_alert_capability_parity.py functional_tests\test_workflow_assist_scenarios.py functional_tests\test_workflow_assist_operations_security.py functional_tests\test_workflow_assist_field_parity.py functional_tests\test_workflow_assist_candidate_parity.py functional_tests\test_workflow_draft_service.py functional_tests\test_workflow_draft_save_parity.py functional_tests\test_workflow_draft_v2_round_trip.py functional_tests\test_workflow_alert_rules.py functional_tests\test_workflow_alert_acknowledgment.py- 381 passed. Covers targeted rule edits, invalid options, native conditions, real dry-run/create and production editor/save round trips.python -m pytest -q ui_tests\test_v2_orchestration_workflow_proposal_card.py- 49 passed, including desktop/light and mobile/dark disclosure and inert planner text. Uses the shared local/Azure Playwright connection fixture, run locally.node --test functional_tests\test_v2_workflow_proposal_alerts.mjs functional_tests\test_v2_workflow_proposal_merge.mjs- 6 passed.npm --prefix application\v2_ui run typecheck- passed.npm --prefix application\v2_ui run build -- --outDir ..\..\ui_tests\artifacts\orchestration-plan-editor- passed, with the existing large-chunk warning.python .\functional_tests\test_docs_app_surface_coverage.py- 7/7 passed, inventory remains current.python .\functional_tests\test_docs_site_quality.py- 6/6 passed.PYTHONPATH,python -m pytest -q functional_tests\test_orchestration_workflow_propose_capability.py::test_the_proposal_module_creates_nothing_and_never_imports_flask- 1 passed.git diff --cached --check- passed before committing.Validation limits: Broader proposal-route and hand-off integration suites were attempted but blocked by their fixture's pre-existing
functions_public_workspaces/functions_searchimport cycle. An existing round-trip test reproduced the same failure when changed application modules were loaded from originalHEADsource. Bootstrapping authentication first exposed an unrelated installed OpenSSL/cryptography incompatibility (GEN_EMAILmissing). Neither issue was changed or suppressed. Independent real builder/editor/save tests pass. Live provider-backed model-selection evaluation was not run; scripted model tests verify contracts, not selection quality on every paraphrase.Documentation
Updated AI Assist, chat proposal, draft-service and workflow creation guidance. Added
WORKFLOW_AI_ALERT_CAPABILITY_PARITY_FIX.mdwith root causes, behavior, coverage and validation limitations. Release notes await confirmation.Security checklist
@swagger_route(security=get_auth_security())sanitize_settings_for_user()No new routes or settings payloads. New schemas are closed and bounded, reject raw IDs/group audience/recipients, and use existing native validation. Proposal text renders through escaped React text; local test build artifacts are not committed.