新增市場情報 AI summary persistence run package preview
All checks were successful
CD Pipeline / deploy (push) Successful in 1m3s
All checks were successful
CD Pipeline / deploy (push) Successful in 1m3s
This commit is contained in:
@@ -1063,6 +1063,10 @@ def test_market_intel_preview_template_uses_safe_fetch_false_endpoint():
|
||||
"market_intel_review.market_intel_manual_sample_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
in template
|
||||
)
|
||||
assert (
|
||||
"market_intel_review.market_intel_manual_sample_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
in template
|
||||
)
|
||||
assert (
|
||||
"data-market-intel-sample-candidate-queue-review-decision-writer"
|
||||
in template
|
||||
@@ -1131,6 +1135,10 @@ def test_market_intel_preview_template_uses_safe_fetch_false_endpoint():
|
||||
"data-market-intel-sample-candidate-queue-review-ai-summary-persistence-writer-preflight"
|
||||
in template
|
||||
)
|
||||
assert (
|
||||
"data-market-intel-sample-candidate-queue-review-ai-summary-persistence-run-package"
|
||||
in template
|
||||
)
|
||||
assert "X-CSRFToken" in template
|
||||
assert "market_intel.market_intel_scheduler_plan" in template
|
||||
assert "market_intel.market_intel_match_review_plan" in template
|
||||
@@ -1167,7 +1175,7 @@ def test_legacy_source_bridge_default_is_planned_only():
|
||||
bridge = MarketIntelService().build_legacy_source_bridge()
|
||||
|
||||
assert bridge["mode"] == "legacy_source_bridge_planned"
|
||||
assert bridge["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert bridge["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert bridge["execute_requested"] is False
|
||||
assert bridge["read_only_query_executed"] is False
|
||||
assert bridge["database_connection_opened"] is False
|
||||
@@ -1325,7 +1333,7 @@ def test_mcp_tool_contract_preview_is_read_only_and_whitelisted():
|
||||
contract = MarketIntelService().build_mcp_tool_contract()
|
||||
|
||||
assert contract["mode"] == "mcp_tool_contract_preview"
|
||||
assert contract["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert contract["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert contract["caller"] == "market_intel"
|
||||
assert contract["contract_ready"] is True
|
||||
assert contract["blocked_reasons"] == []
|
||||
@@ -1458,7 +1466,7 @@ def test_mcp_activation_runbook_route_is_preview_only():
|
||||
|
||||
assert response.status_code == 200
|
||||
assert data["mode"] == "mcp_activation_runbook_preview"
|
||||
assert data["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert data["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert data["deployment_actions_executed"] is False
|
||||
assert data["docker_command_executed"] is False
|
||||
assert data["ssh_command_executed"] is False
|
||||
@@ -1471,7 +1479,7 @@ def test_mcp_fetch_gate_default_blocks_external_fetch():
|
||||
gate = MarketIntelService().build_mcp_fetch_gate(fetch_requested=True)
|
||||
|
||||
assert gate["mode"] == "mcp_fetch_gate_planned"
|
||||
assert gate["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert gate["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert gate["fetch_requested"] is True
|
||||
assert gate["manual_fetch_gate_open"] is False
|
||||
assert gate["network_request_allowed"] is False
|
||||
@@ -1541,7 +1549,7 @@ def test_mcp_fetch_gate_route_is_preview_only():
|
||||
|
||||
assert response.status_code == 200
|
||||
assert data["mode"] == "mcp_fetch_gate_planned"
|
||||
assert data["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert data["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert data["fetch_requested"] is False
|
||||
assert data["network_request_allowed"] is False
|
||||
assert data["external_network_executed"] is False
|
||||
@@ -1553,7 +1561,7 @@ def test_manual_sample_plan_preview_blocks_fetch_and_write():
|
||||
plan = MarketIntelService().build_manual_sample_plan()
|
||||
|
||||
assert plan["mode"] == "manual_sample_fetch_plan_preview"
|
||||
assert plan["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert plan["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert plan["ready_for_manual_sample_fetch"] is False
|
||||
assert plan["sample_fetch_executed"] is False
|
||||
assert plan["external_network_executed"] is False
|
||||
@@ -1601,7 +1609,7 @@ def test_manual_sample_plan_route_is_preview_only():
|
||||
|
||||
assert response.status_code == 200
|
||||
assert data["mode"] == "manual_sample_fetch_plan_preview"
|
||||
assert data["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert data["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert data["sample_fetch_executed"] is False
|
||||
assert data["external_network_executed"] is False
|
||||
assert data["database_write_executed"] is False
|
||||
@@ -1612,7 +1620,7 @@ def test_manual_sample_acceptance_preview_blocks_candidate_import():
|
||||
acceptance = MarketIntelService().build_manual_sample_acceptance()
|
||||
|
||||
assert acceptance["mode"] == "manual_sample_acceptance_preview"
|
||||
assert acceptance["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert acceptance["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert acceptance["contract_ready"] is True
|
||||
assert acceptance["sample_result_loaded"] is False
|
||||
assert acceptance["sample_result_accepted"] is False
|
||||
@@ -1654,7 +1662,7 @@ def test_manual_sample_acceptance_route_is_preview_only():
|
||||
|
||||
assert response.status_code == 200
|
||||
assert data["mode"] == "manual_sample_acceptance_preview"
|
||||
assert data["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert data["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert data["sample_result_loaded"] is False
|
||||
assert data["candidate_import_allowed"] is False
|
||||
assert data["external_network_executed"] is False
|
||||
@@ -1666,7 +1674,7 @@ def test_manual_sample_review_preview_is_planned_until_result_loaded():
|
||||
review = MarketIntelService().build_manual_sample_review()
|
||||
|
||||
assert review["mode"] == "manual_sample_review_preview"
|
||||
assert review["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert review["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert review["contract_ready"] is True
|
||||
assert review["sample_result_loaded"] is False
|
||||
assert review["sample_result_reviewed"] is False
|
||||
@@ -1777,7 +1785,7 @@ def test_manual_sample_review_route_is_preview_only():
|
||||
|
||||
assert response.status_code == 200
|
||||
assert data["mode"] == "manual_sample_review_preview"
|
||||
assert data["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert data["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert data["sample_result_loaded"] is False
|
||||
assert data["sample_result_reviewed"] is False
|
||||
assert data["candidate_import_allowed"] is False
|
||||
@@ -1816,7 +1824,7 @@ def test_manual_sample_review_evaluation_preview_accepts_payload_without_persist
|
||||
)
|
||||
|
||||
assert review["mode"] == "manual_sample_review_evaluation_preview"
|
||||
assert review["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert review["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert review["review_request_type"] == "operator_posted_json"
|
||||
assert review["payload_received"] is True
|
||||
assert review["payload_valid_json_object"] is True
|
||||
@@ -1878,7 +1886,7 @@ def test_manual_sample_review_evaluate_route_is_post_only_and_no_write():
|
||||
|
||||
assert response.status_code == 200
|
||||
assert data["mode"] == "manual_sample_review_evaluation_preview"
|
||||
assert data["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert data["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert data["payload_received"] is True
|
||||
assert data["payload_valid_json_object"] is True
|
||||
assert data["payload_persisted"] is False
|
||||
@@ -1958,7 +1966,7 @@ def test_manual_sample_candidate_handoff_preview_creates_candidates_without_pers
|
||||
)
|
||||
|
||||
assert handoff["mode"] == "manual_sample_candidate_handoff_preview"
|
||||
assert handoff["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert handoff["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert handoff["payload_received"] is True
|
||||
assert handoff["payload_valid_json_object"] is True
|
||||
assert handoff["payload_persisted"] is False
|
||||
@@ -2022,7 +2030,7 @@ def test_manual_sample_candidate_handoff_route_is_post_only_and_no_write():
|
||||
|
||||
assert response.status_code == 200
|
||||
assert data["mode"] == "manual_sample_candidate_handoff_preview"
|
||||
assert data["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert data["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert data["payload_received"] is True
|
||||
assert data["handoff_ready"] is True
|
||||
assert data["candidate_handoff_created"] is True
|
||||
@@ -2081,7 +2089,7 @@ def test_manual_sample_candidate_queue_draft_preview_builds_review_items_without
|
||||
)
|
||||
|
||||
assert queue_draft["mode"] == "manual_sample_candidate_queue_draft_preview"
|
||||
assert queue_draft["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert queue_draft["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert queue_draft["payload_received"] is True
|
||||
assert queue_draft["payload_valid_json_object"] is True
|
||||
assert queue_draft["payload_persisted"] is False
|
||||
@@ -2155,7 +2163,7 @@ def test_manual_sample_candidate_queue_draft_route_is_post_only_and_no_write():
|
||||
|
||||
assert response.status_code == 200
|
||||
assert data["mode"] == "manual_sample_candidate_queue_draft_preview"
|
||||
assert data["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert data["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert data["payload_received"] is True
|
||||
assert data["handoff_ready"] is True
|
||||
assert data["queue_draft_ready"] is True
|
||||
@@ -2218,7 +2226,7 @@ def test_manual_sample_candidate_queue_approval_preview_blocks_write_and_maps_ro
|
||||
)
|
||||
|
||||
assert approval["mode"] == "manual_sample_candidate_queue_approval_preview"
|
||||
assert approval["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert approval["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert approval["payload_received"] is True
|
||||
assert approval["payload_valid_json_object"] is True
|
||||
assert approval["payload_persisted"] is False
|
||||
@@ -2296,7 +2304,7 @@ def test_manual_sample_candidate_queue_approval_route_is_post_only_and_no_write(
|
||||
|
||||
assert response.status_code == 200
|
||||
assert data["mode"] == "manual_sample_candidate_queue_approval_preview"
|
||||
assert data["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert data["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert data["payload_received"] is True
|
||||
assert data["approval_preview_created"] is True
|
||||
assert data["approval_request_created"] is False
|
||||
@@ -2359,7 +2367,7 @@ def test_manual_sample_candidate_queue_transaction_preview_blocks_execution():
|
||||
)
|
||||
|
||||
assert transaction["mode"] == "manual_sample_candidate_queue_transaction_preview"
|
||||
assert transaction["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert transaction["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert transaction["payload_received"] is True
|
||||
assert transaction["payload_valid_json_object"] is True
|
||||
assert transaction["payload_persisted"] is False
|
||||
@@ -2439,7 +2447,7 @@ def test_manual_sample_candidate_queue_transaction_route_is_post_only_and_no_wri
|
||||
|
||||
assert response.status_code == 200
|
||||
assert data["mode"] == "manual_sample_candidate_queue_transaction_preview"
|
||||
assert data["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert data["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert data["payload_received"] is True
|
||||
assert data["transaction_preview_created"] is True
|
||||
assert data["transaction_ready"] is False
|
||||
@@ -6300,6 +6308,146 @@ def test_candidate_queue_review_ai_summary_persistence_writer_preflight_gates_cl
|
||||
)
|
||||
|
||||
|
||||
def test_candidate_queue_review_ai_summary_persistence_run_package_previews_cli_bundle():
|
||||
from services.market_intel.candidate_queue_review_ai_summary_persistence_run_package import (
|
||||
build_candidate_queue_review_ai_summary_persistence_run_package,
|
||||
)
|
||||
|
||||
expected_key = "market-alert-review:sample-batch:sku-001"
|
||||
writer_preflight = {
|
||||
"mode": "candidate_queue_review_ai_summary_persistence_writer_preflight_preview",
|
||||
"summary_persistence_writer_preflight_ready": True,
|
||||
"ready_for_summary_persistence_run_package": True,
|
||||
"statement_count": 1,
|
||||
"invalid_statement_count": 0,
|
||||
"expected_dedupe_keys": [expected_key],
|
||||
"summary_payload_hash": "f" * 64,
|
||||
"statement_payloads": [
|
||||
{
|
||||
"statement_type": "UPDATE",
|
||||
"table": "market_alert_review_queue",
|
||||
"target_column": "metadata_json",
|
||||
"dedupe_key": expected_key,
|
||||
"metadata_key": "ai_summary_review",
|
||||
"payload_hash": "f" * 64,
|
||||
"execute_in_api": False,
|
||||
}
|
||||
],
|
||||
"writer_preflight_contract": {
|
||||
"target_table": "market_alert_review_queue",
|
||||
"target_column": "metadata_json",
|
||||
"target_json_path": ["ai_summary_review"],
|
||||
"manual_cli_required": True,
|
||||
"api_write_allowed": False,
|
||||
"requires_metadata_json_backup": True,
|
||||
"requires_postwrite_smoke": True,
|
||||
},
|
||||
"database_connection_opened": False,
|
||||
"database_write_executed": False,
|
||||
"database_commit_executed": False,
|
||||
"llm_call_executed": False,
|
||||
"telegram_dispatched": False,
|
||||
"scheduler_attached": False,
|
||||
"writes_executed": False,
|
||||
"would_write_database": False,
|
||||
}
|
||||
operator_evidence = {
|
||||
"ai_summary_persistence_transaction_artifact_path": (
|
||||
"artifacts/market_intel/ai-summary-persistence-transaction.json"
|
||||
),
|
||||
"ai_summary_persistence_writer_preflight_artifact_path": (
|
||||
"artifacts/market_intel/ai-summary-persistence-writer-preflight.json"
|
||||
),
|
||||
"ai_summary_persistence_run_package_artifact_path": (
|
||||
"artifacts/market_intel/ai-summary-persistence-run-package.json"
|
||||
),
|
||||
"metadata_json_backup_artifact_path": (
|
||||
"artifacts/market_intel/metadata-json-backup.json"
|
||||
),
|
||||
"operator_confirmed_ai_summary_persistence_run_package": True,
|
||||
"operator_confirmed_payload_manifest_reviewed": True,
|
||||
"operator_confirmed_cli_command_reviewed": True,
|
||||
"operator_confirmed_summary_persistence_is_cli_only": True,
|
||||
"operator_confirmed_no_api_db_write": True,
|
||||
"operator_confirmed_no_telegram_dispatch": True,
|
||||
"operator_confirmed_no_scheduler_attach": True,
|
||||
}
|
||||
package = build_candidate_queue_review_ai_summary_persistence_run_package(
|
||||
ai_summary_persistence_writer_preflight=writer_preflight,
|
||||
operator_evidence=operator_evidence,
|
||||
execute_requested=True,
|
||||
)
|
||||
token_leak = build_candidate_queue_review_ai_summary_persistence_run_package(
|
||||
ai_summary_persistence_writer_preflight=writer_preflight,
|
||||
operator_evidence={
|
||||
**operator_evidence,
|
||||
"approval_token": TEST_APPROVAL_TOKEN,
|
||||
},
|
||||
execute_requested=True,
|
||||
)
|
||||
apply_write = build_candidate_queue_review_ai_summary_persistence_run_package(
|
||||
ai_summary_persistence_writer_preflight=writer_preflight,
|
||||
operator_evidence=operator_evidence,
|
||||
execute_requested=True,
|
||||
apply_real_write=True,
|
||||
)
|
||||
|
||||
assert package["mode"] == (
|
||||
"candidate_queue_review_ai_summary_persistence_run_package_preview"
|
||||
)
|
||||
assert package["target_operation"] == "persist_ai_summary_metadata_json"
|
||||
assert package["package_ready"] is True
|
||||
assert package["ready_for_summary_persistence_run_readiness"] is True
|
||||
assert package["ready_for_summary_persistence_cli_run"] is False
|
||||
assert package["ready_for_real_write"] is False
|
||||
assert package["ready_for_api_database_write"] is False
|
||||
assert package["ready_for_telegram_dispatch"] is False
|
||||
assert package["package_artifact_created"] is False
|
||||
assert package["summary_persistence_run_package_file_written"] is False
|
||||
assert package["run_package_file_written"] is False
|
||||
assert package["api_executes_cli"] is False
|
||||
assert package["api_reads_approval_token"] is False
|
||||
assert package["api_writes_file"] is False
|
||||
assert package["api_writes_database"] is False
|
||||
assert package["database_connection_opened"] is False
|
||||
assert package["database_write_executed"] is False
|
||||
assert package["database_commit_executed"] is False
|
||||
assert package["llm_call_executed"] is False
|
||||
assert package["telegram_dispatched"] is False
|
||||
assert package["scheduler_attached"] is False
|
||||
assert package["writes_executed"] is False
|
||||
assert package["would_write_database"] is False
|
||||
assert package["blocked_reasons"] == []
|
||||
assert package["statement_count"] == 1
|
||||
assert package["payload_manifest"]["payload_count"] == 1
|
||||
assert package["payload_manifest"]["dedupe_keys"] == [expected_key]
|
||||
assert package["payload_manifest"]["payloads"][0]["execute_in_api"] is False
|
||||
assert len(package["required_artifacts"]) == 5
|
||||
assert len(package["command_bundle"]) == 5
|
||||
assert package["command_bundle"][3]["executes_database"] is True
|
||||
assert package["command_bundle"][3]["executed"] is False
|
||||
assert "future_summary_persistence_write_must_use_cli_only_gate" in package[
|
||||
"safe_boundaries"
|
||||
]
|
||||
assert token_leak["package_ready"] is False
|
||||
assert token_leak["operator_ai_summary_persistence_run_package"][
|
||||
"approval_token_submitted_to_api"
|
||||
] is True
|
||||
assert (
|
||||
"summary_persistence_run_package_no_approval_token_submitted_to_api"
|
||||
in token_leak["blocked_reasons"]
|
||||
)
|
||||
assert (
|
||||
"summary_persistence_run_package_apply_real_write_not_requested_from_api"
|
||||
in apply_write["blocked_reasons"]
|
||||
)
|
||||
assert TEST_APPROVAL_TOKEN not in json.dumps(
|
||||
token_leak,
|
||||
ensure_ascii=False,
|
||||
sort_keys=True,
|
||||
)
|
||||
|
||||
|
||||
def test_candidate_queue_writer_preflight_route_is_post_only_and_no_write():
|
||||
from routes.market_intel_routes import market_intel_bp
|
||||
|
||||
@@ -6342,7 +6490,7 @@ def test_candidate_queue_writer_preflight_route_is_post_only_and_no_write():
|
||||
|
||||
assert response.status_code == 200
|
||||
assert data["mode"] == "candidate_queue_writer_preflight_planned"
|
||||
assert data["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert data["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert data["execute_requested"] is False
|
||||
assert data["read_only_query_executed"] is False
|
||||
assert data["database_connection_opened"] is False
|
||||
@@ -6399,7 +6547,7 @@ def test_candidate_queue_writer_status_route_never_leaks_approval_token(monkeypa
|
||||
|
||||
assert response.status_code == 200
|
||||
assert data["mode"] == "candidate_queue_writer_cli_blocked"
|
||||
assert data["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert data["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert data["execute_requested"] is True
|
||||
assert data["apply_real_write_requested"] is True
|
||||
assert data["approval_token_present"] is False
|
||||
@@ -6488,7 +6636,7 @@ def test_candidate_queue_writer_postwrite_smoke_route_is_post_only_and_no_write(
|
||||
|
||||
assert response.status_code == 200
|
||||
assert data["mode"] == "candidate_queue_writer_postwrite_smoke_planned"
|
||||
assert data["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert data["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert data["execute_requested"] is False
|
||||
assert data["read_only_query_executed"] is False
|
||||
assert data["database_connection_opened"] is False
|
||||
@@ -6542,7 +6690,7 @@ def test_candidate_queue_writer_operator_drill_route_is_post_only_and_no_write()
|
||||
|
||||
assert response.status_code == 200
|
||||
assert data["mode"] == "candidate_queue_writer_operator_drill_preview"
|
||||
assert data["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert data["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert data["operator_drill_ready"] is True
|
||||
assert data["api_executes_cli"] is False
|
||||
assert data["api_reads_approval_token"] is False
|
||||
@@ -6598,7 +6746,7 @@ def test_candidate_queue_writer_run_package_route_is_post_only_and_no_write():
|
||||
|
||||
assert response.status_code == 200
|
||||
assert data["mode"] == "candidate_queue_writer_run_package_preview"
|
||||
assert data["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert data["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert data["package_ready"] is True
|
||||
assert data["package_artifact_created"] is False
|
||||
assert data["api_writes_file"] is False
|
||||
@@ -6664,7 +6812,7 @@ def test_candidate_queue_writer_run_readiness_route_is_post_only_and_no_write():
|
||||
|
||||
assert response.status_code == 200
|
||||
assert data["mode"] == "candidate_queue_writer_run_readiness_preview"
|
||||
assert data["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert data["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert data["ready_for_cli_operator_run"] is True
|
||||
assert data["ready_for_api_database_write"] is False
|
||||
assert data["api_executes_cli"] is False
|
||||
@@ -6966,7 +7114,7 @@ def test_candidate_queue_writer_run_receipt_route_accepts_inline_payload_no_writ
|
||||
|
||||
assert response.status_code == 200
|
||||
assert data["mode"] == "candidate_queue_writer_run_receipt_preview"
|
||||
assert data["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert data["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert data["receipt_passed"] is True
|
||||
assert data["ready_for_api_database_write"] is False
|
||||
assert data["ready_for_scheduler_attach"] is False
|
||||
@@ -7014,7 +7162,7 @@ def test_candidate_queue_writer_run_closeout_route_is_post_only_and_no_write():
|
||||
|
||||
assert response.status_code == 200
|
||||
assert data["mode"] == "candidate_queue_writer_run_closeout_preview"
|
||||
assert data["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert data["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert data["closeout_passed"] is True
|
||||
assert data["ready_for_next_manual_phase"] is True
|
||||
assert data["ready_for_api_database_write"] is False
|
||||
@@ -7063,7 +7211,7 @@ def test_candidate_queue_review_handoff_route_is_post_only_and_no_write():
|
||||
|
||||
assert response.status_code == 200
|
||||
assert data["mode"] == "candidate_queue_review_handoff_preview"
|
||||
assert data["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert data["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert data["handoff_ready"] is True
|
||||
assert data["ready_for_manual_queue_review"] is True
|
||||
assert data["ready_for_api_database_write"] is False
|
||||
@@ -7121,7 +7269,7 @@ def test_candidate_queue_review_inventory_route_is_post_only_and_no_write():
|
||||
assert get_response.status_code == 405
|
||||
assert response.status_code == 200
|
||||
assert data["mode"] == "candidate_queue_review_inventory_preview"
|
||||
assert data["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert data["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert data["execute_requested"] is False
|
||||
assert data["review_inventory_ready"] is False
|
||||
assert data["ready_for_human_decision_review"] is False
|
||||
@@ -7187,7 +7335,7 @@ def test_candidate_queue_review_decision_route_is_post_only_and_no_write():
|
||||
assert get_response.status_code == 405
|
||||
assert response.status_code == 200
|
||||
assert data["mode"] == "candidate_queue_review_decision_preview"
|
||||
assert data["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert data["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert data["decision_ready"] is False
|
||||
assert data["ready_for_human_decision_record"] is False
|
||||
assert data["ready_for_api_review_state_update"] is False
|
||||
@@ -7258,7 +7406,7 @@ def test_candidate_queue_review_decision_approval_route_is_post_only_and_no_writ
|
||||
assert get_response.status_code == 405
|
||||
assert response.status_code == 200
|
||||
assert data["mode"] == "candidate_queue_review_decision_approval_preview"
|
||||
assert data["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert data["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert data["approval_ready"] is False
|
||||
assert data["ready_for_review_state_transaction_preview"] is False
|
||||
assert data["ready_for_cli_decision_writer"] is False
|
||||
@@ -7334,7 +7482,7 @@ def test_candidate_queue_review_decision_transaction_route_is_post_only_and_no_w
|
||||
assert get_response.status_code == 405
|
||||
assert response.status_code == 200
|
||||
assert data["mode"] == "candidate_queue_review_decision_transaction_preview"
|
||||
assert data["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert data["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert data["transaction_preview_created"] is False
|
||||
assert data["transaction_ready"] is False
|
||||
assert data["ready_for_manual_shell_update_window"] is False
|
||||
@@ -7416,7 +7564,7 @@ def test_candidate_queue_review_decision_writer_status_route_is_post_only_and_no
|
||||
assert get_response.status_code == 405
|
||||
assert response.status_code == 200
|
||||
assert data["mode"] == "candidate_queue_review_decision_writer_cli_blocked"
|
||||
assert data["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert data["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert data["execute_requested"] is True
|
||||
assert data["apply_real_write_requested"] is True
|
||||
assert data["approval_token_present"] is False
|
||||
@@ -7502,7 +7650,7 @@ def test_candidate_queue_review_decision_writer_preflight_route_is_post_only_and
|
||||
assert data["mode"] == (
|
||||
"candidate_queue_review_decision_writer_preflight_preview"
|
||||
)
|
||||
assert data["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert data["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert data["execute_requested"] is True
|
||||
assert data["apply_real_write_requested"] is True
|
||||
assert data["read_only_query_executed"] is False
|
||||
@@ -7585,7 +7733,7 @@ def test_candidate_queue_review_decision_writer_postwrite_smoke_route_is_post_on
|
||||
assert data["mode"] == (
|
||||
"candidate_queue_review_decision_writer_postwrite_smoke_planned"
|
||||
)
|
||||
assert data["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert data["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert data["execute_requested"] is False
|
||||
assert data["read_only_query_executed"] is False
|
||||
assert data["database_connection_opened"] is False
|
||||
@@ -7668,7 +7816,7 @@ def test_candidate_queue_review_decision_writer_operator_drill_route_is_post_onl
|
||||
assert data["mode"] == (
|
||||
"candidate_queue_review_decision_writer_operator_drill_preview"
|
||||
)
|
||||
assert data["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert data["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert data["operator_drill_ready"] is False
|
||||
assert data["ready_for_api_review_state_update"] is False
|
||||
assert data["ready_for_api_database_write"] is False
|
||||
@@ -7754,7 +7902,7 @@ def test_candidate_queue_review_decision_writer_run_package_route_is_post_only_a
|
||||
assert data["mode"] == (
|
||||
"candidate_queue_review_decision_writer_run_package_preview"
|
||||
)
|
||||
assert data["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert data["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert data["package_ready"] is False
|
||||
assert data["package_artifact_created"] is False
|
||||
assert data["ready_for_api_review_state_update"] is False
|
||||
@@ -7845,7 +7993,7 @@ def test_candidate_queue_review_decision_writer_run_readiness_route_is_post_only
|
||||
"candidate_queue_review_decision_writer_run_readiness_preview"
|
||||
)
|
||||
assert data["phase"] == (
|
||||
"phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
"phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
)
|
||||
assert data["ready_for_cli_operator_run"] is False
|
||||
assert data["ready_for_api_review_state_update"] is False
|
||||
@@ -7955,7 +8103,7 @@ def test_candidate_queue_review_decision_writer_run_receipt_route_is_post_only_a
|
||||
"candidate_queue_review_decision_writer_run_receipt_preview"
|
||||
)
|
||||
assert data["phase"] == (
|
||||
"phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
"phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
)
|
||||
assert data["receipt_passed"] is False
|
||||
assert data["ready_for_api_review_state_update"] is False
|
||||
@@ -8041,7 +8189,7 @@ def test_candidate_queue_review_decision_writer_run_closeout_route_is_post_only_
|
||||
"candidate_queue_review_decision_writer_run_closeout_preview"
|
||||
)
|
||||
assert data["phase"] == (
|
||||
"phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
"phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
)
|
||||
assert data["closeout_passed"] is False
|
||||
assert data["ready_for_api_review_state_update"] is False
|
||||
@@ -8098,7 +8246,7 @@ def test_candidate_queue_review_decision_post_closeout_inventory_route_is_post_o
|
||||
"candidate_queue_review_decision_post_closeout_inventory_preview"
|
||||
)
|
||||
assert data["phase"] == (
|
||||
"phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
"phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
)
|
||||
assert data["post_closeout_inventory_ready"] is False
|
||||
assert data["ready_for_api_review_state_update"] is False
|
||||
@@ -8151,7 +8299,7 @@ def test_candidate_queue_review_completion_archive_route_is_post_only_and_no_wri
|
||||
assert get_response.status_code == 405
|
||||
assert response.status_code == 200
|
||||
assert data["mode"] == "candidate_queue_review_completion_archive_preview"
|
||||
assert data["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert data["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert data["review_completion_archive_ready"] is False
|
||||
assert data["archive_manifest_ready"] is False
|
||||
assert data["ready_for_api_review_state_update"] is False
|
||||
@@ -8204,7 +8352,7 @@ def test_candidate_queue_review_archive_summary_route_is_post_only_and_no_write(
|
||||
assert get_response.status_code == 405
|
||||
assert response.status_code == 200
|
||||
assert data["mode"] == "candidate_queue_review_archive_summary_preview"
|
||||
assert data["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert data["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert data["archive_summary_ready"] is False
|
||||
assert data["summary_input_ready"] is False
|
||||
assert data["ready_for_ai_summary_generation"] is False
|
||||
@@ -8265,7 +8413,7 @@ def test_candidate_queue_review_ai_summary_preflight_route_is_post_only_and_no_w
|
||||
assert get_response.status_code == 405
|
||||
assert response.status_code == 200
|
||||
assert data["mode"] == "candidate_queue_review_ai_summary_preflight_preview"
|
||||
assert data["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert data["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert data["ai_summary_preflight_ready"] is False
|
||||
assert data["ready_for_manual_ollama_summary_run"] is False
|
||||
assert data["ready_for_ai_summary_generation"] is False
|
||||
@@ -8334,7 +8482,7 @@ def test_candidate_queue_review_ai_summary_run_package_route_is_post_only_and_no
|
||||
assert get_response.status_code == 405
|
||||
assert response.status_code == 200
|
||||
assert data["mode"] == "candidate_queue_review_ai_summary_run_package_preview"
|
||||
assert data["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert data["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert data["ai_summary_run_package_ready"] is False
|
||||
assert data["ready_for_manual_ollama_summary_run"] is False
|
||||
assert data["ready_for_ai_summary_generation"] is False
|
||||
@@ -8406,7 +8554,7 @@ def test_candidate_queue_review_ai_summary_output_receipt_route_is_post_only_and
|
||||
assert get_response.status_code == 405
|
||||
assert response.status_code == 200
|
||||
assert data["mode"] == "candidate_queue_review_ai_summary_output_receipt_preview"
|
||||
assert data["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert data["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert data["ai_summary_output_receipt_ready"] is False
|
||||
assert data["ready_for_summary_persistence_review"] is False
|
||||
assert data["manual_ai_summary_output_provided"] is False
|
||||
@@ -8479,7 +8627,7 @@ def test_candidate_queue_review_ai_summary_persistence_preflight_route_is_post_o
|
||||
"candidate_queue_review_ai_summary_persistence_preflight_preview"
|
||||
)
|
||||
assert data["phase"] == (
|
||||
"phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
"phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
)
|
||||
assert data["summary_persistence_preflight_ready"] is False
|
||||
assert data["ready_for_summary_transaction_preview"] is False
|
||||
@@ -8550,7 +8698,7 @@ def test_candidate_queue_review_ai_summary_persistence_transaction_route_is_post
|
||||
"candidate_queue_review_ai_summary_persistence_transaction_preview"
|
||||
)
|
||||
assert data["phase"] == (
|
||||
"phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
"phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
)
|
||||
assert data["summary_persistence_transaction_ready"] is False
|
||||
assert data["ready_for_summary_persistence_writer_gate"] is False
|
||||
@@ -8615,7 +8763,7 @@ def test_candidate_queue_review_ai_summary_persistence_writer_preflight_route_is
|
||||
"candidate_queue_review_ai_summary_persistence_writer_preflight_preview"
|
||||
)
|
||||
assert data["phase"] == (
|
||||
"phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
"phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
)
|
||||
assert data["summary_persistence_writer_preflight_ready"] is False
|
||||
assert data["ready_for_summary_persistence_run_package"] is False
|
||||
@@ -8655,6 +8803,79 @@ def test_candidate_queue_review_ai_summary_persistence_writer_preflight_route_is
|
||||
assert TEST_APPROVAL_TOKEN not in payload
|
||||
|
||||
|
||||
def test_candidate_queue_review_ai_summary_persistence_run_package_route_is_post_only_and_no_write():
|
||||
from routes.market_intel_routes import market_intel_bp
|
||||
from routes.market_intel_review_routes import market_intel_review_bp
|
||||
|
||||
app = Flask(__name__)
|
||||
app.secret_key = "test-secret"
|
||||
app.register_blueprint(market_intel_bp)
|
||||
app.register_blueprint(market_intel_review_bp)
|
||||
client = app.test_client()
|
||||
with client.session_transaction() as session:
|
||||
session["logged_in"] = True
|
||||
|
||||
get_response = client.get(
|
||||
"/api/market_intel/manual_sample_review/"
|
||||
"candidate_queue_review_ai_summary_persistence_run_package"
|
||||
)
|
||||
response = client.post(
|
||||
"/api/market_intel/manual_sample_review/"
|
||||
"candidate_queue_review_ai_summary_persistence_run_package"
|
||||
"?execute=true&apply_real_write=true",
|
||||
json={"sample_result": {}},
|
||||
)
|
||||
data = response.get_json()
|
||||
payload = json.dumps(data, ensure_ascii=False, sort_keys=True)
|
||||
|
||||
assert get_response.status_code == 405
|
||||
assert response.status_code == 200
|
||||
assert data["mode"] == (
|
||||
"candidate_queue_review_ai_summary_persistence_run_package_preview"
|
||||
)
|
||||
assert data["phase"] == (
|
||||
"phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
)
|
||||
assert data["package_ready"] is False
|
||||
assert data["ready_for_summary_persistence_run_readiness"] is False
|
||||
assert data["ready_for_summary_persistence_cli_run"] is False
|
||||
assert data["ready_for_real_write"] is False
|
||||
assert data["ready_for_api_database_write"] is False
|
||||
assert data["ready_for_telegram_dispatch"] is False
|
||||
assert data["package_artifact_created"] is False
|
||||
assert data["summary_persistence_run_package_file_written"] is False
|
||||
assert data["run_package_file_written"] is False
|
||||
assert data["api_executes_llm"] is False
|
||||
assert data["api_executes_cli"] is False
|
||||
assert data["api_reads_approval_token"] is False
|
||||
assert data["api_writes_file"] is False
|
||||
assert data["api_writes_database"] is False
|
||||
assert data["api_updates_review_state"] is False
|
||||
assert data["summary_persistence_record_written"] is False
|
||||
assert data["metadata_patch_written"] is False
|
||||
assert data["transaction_file_written"] is False
|
||||
assert data["ai_summary_generated"] is False
|
||||
assert data["llm_call_executed"] is False
|
||||
assert data["telegram_dispatched"] is False
|
||||
assert data["database_connection_opened"] is False
|
||||
assert data["database_write_executed"] is False
|
||||
assert data["database_commit_executed"] is False
|
||||
assert data["review_state_update_executed"] is False
|
||||
assert data["scheduler_attached"] is False
|
||||
assert data["statement_count"] == 0
|
||||
assert data["payload_manifest"]["payload_count"] == 0
|
||||
assert "summary_persistence_writer_preflight_ready" in data["blocked_reasons"]
|
||||
assert "summary_persistence_payload_manifest_ready" in data["blocked_reasons"]
|
||||
assert (
|
||||
"summary_persistence_run_package_apply_real_write_not_requested_from_api"
|
||||
in data["blocked_reasons"]
|
||||
)
|
||||
assert "do_not_open_database_connection_from_summary_persistence_run_package" in data[
|
||||
"safe_boundaries"
|
||||
]
|
||||
assert TEST_APPROVAL_TOKEN not in payload
|
||||
|
||||
|
||||
def test_candidate_queue_writer_run_receipt_route_is_post_only_and_no_write():
|
||||
from routes.market_intel_routes import market_intel_bp
|
||||
|
||||
@@ -8683,7 +8904,7 @@ def test_candidate_queue_writer_run_receipt_route_is_post_only_and_no_write():
|
||||
|
||||
assert response.status_code == 200
|
||||
assert data["mode"] == "candidate_queue_writer_run_receipt_preview"
|
||||
assert data["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert data["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert data["receipt_passed"] is True
|
||||
assert data["ready_for_next_manual_review"] is True
|
||||
assert data["ready_for_api_database_write"] is False
|
||||
@@ -8708,7 +8929,7 @@ def test_scheduler_plan_preview_blocks_job_attachment():
|
||||
plan = MarketIntelService().build_scheduler_plan()
|
||||
|
||||
assert plan["mode"] == "scheduler_attach_plan_preview"
|
||||
assert plan["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert plan["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert plan["ready_to_attach_scheduler"] is False
|
||||
assert plan["scheduler_attached"] is False
|
||||
assert plan["scheduler_registration_executed"] is False
|
||||
@@ -8746,7 +8967,7 @@ def test_scheduler_plan_route_is_preview_only():
|
||||
|
||||
assert response.status_code == 200
|
||||
assert data["mode"] == "scheduler_attach_plan_preview"
|
||||
assert data["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert data["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert data["scheduler_registration_executed"] is False
|
||||
assert data["crawler_job_started"] is False
|
||||
assert data["external_network_executed"] is False
|
||||
@@ -8757,7 +8978,7 @@ def test_match_review_plan_preview_blocks_auto_confirm():
|
||||
plan = MarketIntelService().build_match_review_plan()
|
||||
|
||||
assert plan["mode"] == "match_review_plan_preview"
|
||||
assert plan["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert plan["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert plan["ready_for_review_queue"] is False
|
||||
assert plan["review_queue_created"] is False
|
||||
assert plan["auto_match_executed"] is False
|
||||
@@ -8793,7 +9014,7 @@ def test_match_review_plan_route_is_preview_only():
|
||||
|
||||
assert response.status_code == 200
|
||||
assert data["mode"] == "match_review_plan_preview"
|
||||
assert data["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert data["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert data["review_queue_created"] is False
|
||||
assert data["auto_confirm_executed"] is False
|
||||
assert data["external_network_executed"] is False
|
||||
@@ -8804,7 +9025,7 @@ def test_opportunity_plan_preview_blocks_alerts_and_ai_summary():
|
||||
plan = MarketIntelService().build_opportunity_plan()
|
||||
|
||||
assert plan["mode"] == "opportunity_plan_preview"
|
||||
assert plan["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert plan["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert plan["ready_for_opportunity_queue"] is False
|
||||
assert plan["opportunity_queue_created"] is False
|
||||
assert plan["threat_alert_dispatched"] is False
|
||||
@@ -8845,7 +9066,7 @@ def test_opportunity_plan_route_is_preview_only():
|
||||
|
||||
assert response.status_code == 200
|
||||
assert data["mode"] == "opportunity_plan_preview"
|
||||
assert data["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert data["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert data["opportunity_queue_created"] is False
|
||||
assert data["threat_alert_dispatched"] is False
|
||||
assert data["ai_summary_generated"] is False
|
||||
@@ -8856,7 +9077,7 @@ def test_opportunity_scoring_plan_preview_blocks_scoring_and_alerts():
|
||||
plan = MarketIntelService().build_opportunity_scoring_plan()
|
||||
|
||||
assert plan["mode"] == "opportunity_scoring_plan_preview"
|
||||
assert plan["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert plan["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert plan["ready_for_scoring_job"] is False
|
||||
assert plan["scoring_job_created"] is False
|
||||
assert plan["score_calculation_executed"] is False
|
||||
@@ -8904,7 +9125,7 @@ def test_opportunity_scoring_plan_route_is_preview_only():
|
||||
|
||||
assert response.status_code == 200
|
||||
assert data["mode"] == "opportunity_scoring_plan_preview"
|
||||
assert data["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert data["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert data["scoring_job_created"] is False
|
||||
assert data["score_calculation_executed"] is False
|
||||
assert data["sample_scores_generated"] is False
|
||||
@@ -8916,7 +9137,7 @@ def test_opportunity_evidence_plan_preview_blocks_queries_and_alerts():
|
||||
plan = MarketIntelService().build_opportunity_evidence_plan()
|
||||
|
||||
assert plan["mode"] == "opportunity_evidence_plan_preview"
|
||||
assert plan["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert plan["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert plan["ready_for_evidence_bundle"] is False
|
||||
assert plan["evidence_bundle_created"] is False
|
||||
assert plan["evidence_query_executed"] is False
|
||||
@@ -8962,7 +9183,7 @@ def test_opportunity_evidence_plan_route_is_preview_only():
|
||||
|
||||
assert response.status_code == 200
|
||||
assert data["mode"] == "opportunity_evidence_plan_preview"
|
||||
assert data["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert data["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert data["evidence_bundle_created"] is False
|
||||
assert data["evidence_query_executed"] is False
|
||||
assert data["sample_evidence_generated"] is False
|
||||
@@ -8975,7 +9196,7 @@ def test_opportunity_alert_plan_preview_blocks_dispatch_and_llm_calls():
|
||||
plan = MarketIntelService().build_opportunity_alert_plan()
|
||||
|
||||
assert plan["mode"] == "opportunity_alert_plan_preview"
|
||||
assert plan["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert plan["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert plan["ready_for_alert_candidates"] is False
|
||||
assert plan["alert_candidate_created"] is False
|
||||
assert plan["alert_queue_created"] is False
|
||||
@@ -9060,7 +9281,7 @@ def test_opportunity_alert_plan_route_is_preview_only():
|
||||
|
||||
assert response.status_code == 200
|
||||
assert data["mode"] == "opportunity_alert_plan_preview"
|
||||
assert data["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert data["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert data["alert_candidate_created"] is False
|
||||
assert data["alert_queue_created"] is False
|
||||
assert data["review_queue_created"] is False
|
||||
@@ -9138,7 +9359,7 @@ def test_mcp_deploy_preflight_route_is_preview_only():
|
||||
|
||||
assert response.status_code == 200
|
||||
assert data["mode"] == "mcp_external_deploy_preflight_preview"
|
||||
assert data["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert data["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert data["deployment_actions_executed"] is False
|
||||
assert data["docker_command_executed"] is False
|
||||
assert data["ssh_command_executed"] is False
|
||||
@@ -9153,7 +9374,7 @@ def test_mcp_readiness_default_is_planned_only(monkeypatch):
|
||||
readiness = MarketIntelService().build_mcp_readiness()
|
||||
|
||||
assert readiness["mode"] == "mcp_readiness_planned"
|
||||
assert readiness["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert readiness["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert readiness["execute_requested"] is False
|
||||
assert readiness["router_enabled"] is False
|
||||
assert readiness["external_mcp_complete"] is False
|
||||
@@ -9695,6 +9916,12 @@ def test_deployment_readiness_reports_app_only_release_gate():
|
||||
]
|
||||
is True
|
||||
)
|
||||
assert (
|
||||
readiness["checks"][
|
||||
"candidate_queue_review_ai_summary_persistence_run_package_preview_safe"
|
||||
]
|
||||
is True
|
||||
)
|
||||
assert (
|
||||
readiness["checks"][
|
||||
"candidate_queue_review_decision_writer_cli_status_safe"
|
||||
@@ -9869,6 +10096,11 @@ def test_deployment_readiness_reports_app_only_release_gate():
|
||||
"candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
in readiness["production_smoke_targets"]
|
||||
)
|
||||
assert (
|
||||
"/api/market_intel/manual_sample_review/"
|
||||
"candidate_queue_review_ai_summary_persistence_run_package"
|
||||
in readiness["production_smoke_targets"]
|
||||
)
|
||||
assert (
|
||||
"/api/market_intel/manual_sample_review/"
|
||||
"candidate_queue_review_decision_writer_status"
|
||||
@@ -11748,6 +11980,84 @@ def test_deployment_readiness_reports_app_only_release_gate():
|
||||
]
|
||||
is False
|
||||
)
|
||||
assert (
|
||||
readiness["candidate_queue_review_ai_summary_persistence_run_package"][
|
||||
"mode"
|
||||
]
|
||||
== "candidate_queue_review_ai_summary_persistence_run_package_preview"
|
||||
)
|
||||
assert (
|
||||
readiness["candidate_queue_review_ai_summary_persistence_run_package"][
|
||||
"package_ready"
|
||||
]
|
||||
is False
|
||||
)
|
||||
assert (
|
||||
readiness["candidate_queue_review_ai_summary_persistence_run_package"][
|
||||
"ready_for_summary_persistence_run_readiness"
|
||||
]
|
||||
is False
|
||||
)
|
||||
assert (
|
||||
readiness["candidate_queue_review_ai_summary_persistence_run_package"][
|
||||
"ready_for_summary_persistence_cli_run"
|
||||
]
|
||||
is False
|
||||
)
|
||||
assert (
|
||||
readiness["candidate_queue_review_ai_summary_persistence_run_package"][
|
||||
"statement_count"
|
||||
]
|
||||
== 0
|
||||
)
|
||||
assert (
|
||||
readiness["candidate_queue_review_ai_summary_persistence_run_package"][
|
||||
"payload_manifest"
|
||||
]["payload_count"]
|
||||
== 0
|
||||
)
|
||||
assert (
|
||||
readiness["candidate_queue_review_ai_summary_persistence_run_package"][
|
||||
"api_executes_llm"
|
||||
]
|
||||
is False
|
||||
)
|
||||
assert (
|
||||
readiness["candidate_queue_review_ai_summary_persistence_run_package"][
|
||||
"package_artifact_created"
|
||||
]
|
||||
is False
|
||||
)
|
||||
assert (
|
||||
readiness["candidate_queue_review_ai_summary_persistence_run_package"][
|
||||
"summary_persistence_run_package_file_written"
|
||||
]
|
||||
is False
|
||||
)
|
||||
assert (
|
||||
readiness["candidate_queue_review_ai_summary_persistence_run_package"][
|
||||
"telegram_dispatched"
|
||||
]
|
||||
is False
|
||||
)
|
||||
assert (
|
||||
readiness["candidate_queue_review_ai_summary_persistence_run_package"][
|
||||
"database_connection_opened"
|
||||
]
|
||||
is False
|
||||
)
|
||||
assert (
|
||||
readiness["candidate_queue_review_ai_summary_persistence_run_package"][
|
||||
"database_write_executed"
|
||||
]
|
||||
is False
|
||||
)
|
||||
assert (
|
||||
readiness["candidate_queue_review_ai_summary_persistence_run_package"][
|
||||
"scheduler_attached"
|
||||
]
|
||||
is False
|
||||
)
|
||||
assert (
|
||||
readiness["candidate_queue_review_decision_writer_status"]["mode"]
|
||||
== "candidate_queue_review_decision_writer_cli_blocked"
|
||||
@@ -11898,7 +12208,7 @@ def test_migration_apply_drill_planned_is_safe_and_manual_only():
|
||||
drill = MarketIntelService().build_migration_apply_drill()
|
||||
|
||||
assert drill["mode"] == "migration_apply_drill_preview"
|
||||
assert drill["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert drill["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert drill["execute_requested"] is False
|
||||
assert drill["schema_state"] == "planned_no_db_probe"
|
||||
assert drill["drill_ready_for_operator_review"] is True
|
||||
@@ -12013,7 +12323,7 @@ def test_migration_apply_drill_route_is_preview_only():
|
||||
|
||||
assert response.status_code == 200
|
||||
assert data["mode"] == "migration_apply_drill_preview"
|
||||
assert data["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert data["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert data["execute_requested"] is False
|
||||
assert data["migration_executed"] is False
|
||||
assert data["rollback_executed"] is False
|
||||
@@ -12025,7 +12335,7 @@ def test_migration_catalog_review_planned_is_safe_and_diagnostic():
|
||||
review = MarketIntelService().build_migration_catalog_review()
|
||||
|
||||
assert review["mode"] == "migration_catalog_review_preview"
|
||||
assert review["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert review["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert review["execute_requested"] is False
|
||||
assert review["catalog_state"] == "planned_no_probe"
|
||||
assert review["seed_state"] == "planned_no_probe"
|
||||
@@ -12140,7 +12450,7 @@ def test_migration_catalog_review_route_is_preview_only():
|
||||
|
||||
assert response.status_code == 200
|
||||
assert data["mode"] == "migration_catalog_review_preview"
|
||||
assert data["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert data["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert data["execute_requested"] is False
|
||||
assert data["catalog_state"] == "planned_no_probe"
|
||||
assert data["migration_executed"] is False
|
||||
@@ -12153,7 +12463,7 @@ def test_migration_live_smoke_planned_is_preview_only():
|
||||
smoke = MarketIntelService().build_migration_live_smoke()
|
||||
|
||||
assert smoke["mode"] == "migration_live_smoke_preview"
|
||||
assert smoke["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert smoke["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert smoke["execute_requested"] is False
|
||||
assert smoke["smoke_result"] == "planned_no_execution"
|
||||
assert smoke["live_smoke_passed"] is False
|
||||
@@ -12215,7 +12525,7 @@ def test_migration_live_smoke_route_is_preview_only():
|
||||
|
||||
assert response.status_code == 200
|
||||
assert data["mode"] == "migration_live_smoke_preview"
|
||||
assert data["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert data["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert data["execute_requested"] is False
|
||||
assert data["smoke_result"] == "planned_no_execution"
|
||||
assert data["migration_executed"] is False
|
||||
@@ -12228,7 +12538,7 @@ def test_live_db_inventory_planned_is_preview_only():
|
||||
inventory = MarketIntelService().build_live_db_inventory()
|
||||
|
||||
assert inventory["mode"] == "live_db_inventory_planned"
|
||||
assert inventory["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert inventory["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert inventory["execute_requested"] is False
|
||||
assert inventory["read_only_query_executed"] is False
|
||||
assert inventory["database_connection_opened"] is False
|
||||
@@ -12372,7 +12682,7 @@ def test_live_db_inventory_route_is_preview_only():
|
||||
|
||||
assert response.status_code == 200
|
||||
assert data["mode"] == "live_db_inventory_planned"
|
||||
assert data["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert data["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert data["execute_requested"] is False
|
||||
assert data["read_only_query_executed"] is False
|
||||
assert data["database_write_executed"] is False
|
||||
@@ -12599,7 +12909,7 @@ def test_candidate_queue_writer_cli_script_outputs_blocked_gate(tmp_path):
|
||||
|
||||
assert result.returncode == 0
|
||||
assert data["mode"] == "candidate_queue_writer_cli_blocked"
|
||||
assert data["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert data["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert data["execute_requested"] is False
|
||||
assert data["apply_real_write_requested"] is False
|
||||
assert data["writes_executed"] is False
|
||||
@@ -12628,7 +12938,7 @@ def test_review_decision_writer_cli_script_outputs_blocked_gate_without_login_en
|
||||
|
||||
assert result.returncode == 0
|
||||
assert data["mode"] == "candidate_queue_review_decision_writer_cli_blocked"
|
||||
assert data["phase"] == "phase_85_candidate_queue_review_ai_summary_persistence_writer_preflight"
|
||||
assert data["phase"] == "phase_86_candidate_queue_review_ai_summary_persistence_run_package"
|
||||
assert data["execute_requested"] is False
|
||||
assert data["apply_real_write_requested"] is False
|
||||
assert data["approval_token_present"] is False
|
||||
|
||||
Reference in New Issue
Block a user