@@ -574,10 +592,11 @@
const mcpActivationRoot = document.querySelector('[data-market-intel-mcp-activation]');
const mcpFetchGateRoot = document.querySelector('[data-market-intel-mcp-fetch-gate]');
const schedulerRoot = document.querySelector('[data-market-intel-scheduler]');
+ const matchReviewRoot = document.querySelector('[data-market-intel-match-review]');
const migrationRoot = document.querySelector('[data-market-intel-migration]');
const approvalRoot = document.querySelector('[data-market-intel-approval]');
const deployRoot = document.querySelector('[data-market-intel-deploy]');
- if (!root && !writerRoot && !cliRoot && !dbProbeRoot && !seedDiffRoot && !legacyBridgeRoot && !mcpReadinessRoot && !mcpPreflightRoot && !mcpActivationRoot && !mcpFetchGateRoot && !schedulerRoot && !migrationRoot && !approvalRoot && !deployRoot) return;
+ if (!root && !writerRoot && !cliRoot && !dbProbeRoot && !seedDiffRoot && !legacyBridgeRoot && !mcpReadinessRoot && !mcpPreflightRoot && !mcpActivationRoot && !mcpFetchGateRoot && !schedulerRoot && !matchReviewRoot && !migrationRoot && !approvalRoot && !deployRoot) return;
const meta = root ? root.querySelector('[data-market-intel-preview-meta]') : null;
const body = root ? root.querySelector('[data-market-intel-preview-body]') : null;
@@ -623,6 +642,10 @@
const schedulerBody = schedulerRoot ? schedulerRoot.querySelector('[data-market-intel-scheduler-body]') : null;
const schedulerRefresh = schedulerRoot ? schedulerRoot.querySelector('[data-market-intel-scheduler-refresh]') : null;
const schedulerEndpoint = "{{ url_for('market_intel.market_intel_scheduler_plan') }}";
+ const matchReviewMeta = matchReviewRoot ? matchReviewRoot.querySelector('[data-market-intel-match-review-meta]') : null;
+ const matchReviewBody = matchReviewRoot ? matchReviewRoot.querySelector('[data-market-intel-match-review-body]') : null;
+ const matchReviewRefresh = matchReviewRoot ? matchReviewRoot.querySelector('[data-market-intel-match-review-refresh]') : null;
+ const matchReviewEndpoint = "{{ url_for('market_intel.market_intel_match_review_plan') }}";
const migrationMeta = migrationRoot ? migrationRoot.querySelector('[data-market-intel-migration-meta]') : null;
const migrationBody = migrationRoot ? migrationRoot.querySelector('[data-market-intel-migration-body]') : null;
const migrationRefresh = migrationRoot ? migrationRoot.querySelector('[data-market-intel-migration-refresh]') : null;
@@ -1423,6 +1446,109 @@
}
};
+ const renderMatchReviewMeta = data => {
+ const thresholds = data.thresholds || {};
+ matchReviewMeta.innerHTML = [
+ `mode=${data.mode || 'unknown'}`,
+ `ready=${data.ready_for_review_queue ? 'yes' : 'no'}`,
+ `signals=${(data.scoring_signals || []).length}`,
+ `auto_confirm=${thresholds.auto_confirm_enabled ? 'on' : 'off'}`,
+ `writes=${data.writes_executed ? 'executed' : 'blocked'}`
+ ].map(item => `
${escapeHtml(item)}`).join('');
+ };
+
+ const renderMatchReviewBody = data => {
+ const blockers = (data.blocked_reasons || []).join(' / ');
+ const checks = Object.entries(data.gate_checks || {});
+ const signals = data.scoring_signals || [];
+ const actions = data.review_actions || [];
+ const sequence = data.operator_sequence || [];
+ const renderCheck = ([name, passed]) => `
+
+
+ ${escapeHtml(name)}
+
+
${passed ? 'PASS' : 'BLOCK'}
+
+ `;
+ const renderSignal = item => `
+
+ ${escapeHtml(item.key)}
+ ${escapeHtml(item.label)}
+ weight=${escapeHtml(item.weight)} / ${escapeHtml(item.source)}
+
+ `;
+ const renderAction = item => `
+
+
+ ${escapeHtml(item.key)}
+ ${escapeHtml(item.label)}
+
+
${item.requires_operator ? 'HITL' : 'AUTO'}
+
+ `;
+ matchReviewBody.innerHTML = `
+
商品比對目前只建立審核規則與 HITL 流程；不建立 queue、不自動確認、不寫 DB、不呼叫外部 MCP。${blockers ? `阻擋：${escapeHtml(blockers)}` : ''}
+
+
+
MATCH GATES
+
${
+ checks.length
+ ? checks.map(renderCheck).join('')
+ : '
尚未提供比對 gate。
'
+ }
+
+
+
SCORING SIGNALS
+
${
+ signals.length
+ ? signals.map(renderSignal).join('')
+ : '
尚未提供比對訊號。
'
+ }
+
+
+
REVIEW ACTIONS
+
${
+ actions.length
+ ? actions.map(renderAction).join('')
+ : '
尚未提供審核動作。
'
+ }
+
+
+
OPERATOR SEQUENCE
+
${
+ sequence.length
+ ? sequence.map((item, index) => `
+
+
+ ${escapeHtml(`step_${index + 1}`)}
+ ${escapeHtml(item)}
+
+
REQUIRED
+
+ `).join('')
+ : '
尚未提供操作順序。
'
+ }
+
+
+ `;
+ };
+
+ const loadMatchReview = async () => {
+ if (!matchReviewMeta || !matchReviewBody) return;
+ matchReviewBody.innerHTML = '
讀取商品比對審核計畫中...
';
+ try {
+ const response = await fetch(matchReviewEndpoint, { credentials: 'same-origin' });
+ if (!response.ok) throw new Error(`HTTP ${response.status}`);
+ const data = await response.json();
+ renderMatchReviewMeta(data);
+ renderMatchReviewBody(data);
+ } catch (error) {
+ matchReviewMeta.innerHTML = '
error';
+ matchReviewBody.innerHTML = `
商品比對審核計畫讀取失敗：${escapeHtml(error.message)}
`;
+ }
+ };
+
const renderMigrationMeta = data => {
const seedWriter = data.command_plan && data.command_plan.seed_writer_command
? data.command_plan.seed_writer_command
@@ -1690,6 +1816,9 @@
if (schedulerRefresh) {
schedulerRefresh.addEventListener('click', loadScheduler);
}
+ if (matchReviewRefresh) {
+ matchReviewRefresh.addEventListener('click', loadMatchReview);
+ }
if (migrationRefresh) {
migrationRefresh.addEventListener('click', loadMigration);
}
@@ -1710,6 +1839,7 @@
loadMcpActivation();
loadMcpFetchGate();
loadScheduler();
+ loadMatchReview();
loadMigration();
loadApproval();
loadDeploy();
diff --git a/tests/test_market_intel_skeleton.py b/tests/test_market_intel_skeleton.py
index abf8cb9..99c4aa2 100644
--- a/tests/test_market_intel_skeleton.py
+++ b/tests/test_market_intel_skeleton.py
@@ -448,6 +448,11 @@ def test_market_intel_preview_template_uses_safe_fetch_false_endpoint():
assert "data-market-intel-scheduler-jobs" in template
assert "data-market-intel-scheduler-sequence" in template
assert "data-market-intel-scheduler-fallback" in template
+ assert "data-market-intel-match-review" in template
+ assert "data-market-intel-match-review-checks" in template
+ assert "data-market-intel-match-review-signals" in template
+ assert "data-market-intel-match-review-actions" in template
+ assert "data-market-intel-match-review-sequence" in template
assert "data-market-intel-migration" in template
assert "data-market-intel-migration-tables" in template
assert "data-market-intel-approval" in template
@@ -466,6 +471,7 @@ def test_market_intel_preview_template_uses_safe_fetch_false_endpoint():
assert "market_intel.market_intel_mcp_activation_runbook" in template
assert "market_intel.market_intel_mcp_fetch_gate" in template
assert "market_intel.market_intel_scheduler_plan" in template
+ assert "market_intel.market_intel_match_review_plan" in template
assert "market_intel.market_intel_migration_blueprint" in template
assert "market_intel.market_intel_write_approval_runbook" in template
assert "market_intel.market_intel_deployment_readiness" in template
@@ -491,7 +497,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_33_scheduler_plan_preview"
+ assert bridge["phase"] == "phase_34_match_review_plan_preview"
assert bridge["execute_requested"] is False
assert bridge["read_only_query_executed"] is False
assert bridge["database_connection_opened"] is False
@@ -649,7 +655,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_33_scheduler_plan_preview"
+ assert contract["phase"] == "phase_34_match_review_plan_preview"
assert contract["caller"] == "market_intel"
assert contract["contract_ready"] is True
assert contract["blocked_reasons"] == []
@@ -782,7 +788,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_33_scheduler_plan_preview"
+ assert data["phase"] == "phase_34_match_review_plan_preview"
assert data["deployment_actions_executed"] is False
assert data["docker_command_executed"] is False
assert data["ssh_command_executed"] is False
@@ -795,7 +801,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_33_scheduler_plan_preview"
+ assert gate["phase"] == "phase_34_match_review_plan_preview"
assert gate["fetch_requested"] is True
assert gate["manual_fetch_gate_open"] is False
assert gate["network_request_allowed"] is False
@@ -865,7 +871,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_33_scheduler_plan_preview"
+ assert data["phase"] == "phase_34_match_review_plan_preview"
assert data["fetch_requested"] is False
assert data["network_request_allowed"] is False
assert data["external_network_executed"] is False
@@ -877,7 +883,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_33_scheduler_plan_preview"
+ assert plan["phase"] == "phase_34_match_review_plan_preview"
assert plan["ready_to_attach_scheduler"] is False
assert plan["scheduler_attached"] is False
assert plan["scheduler_registration_executed"] is False
@@ -915,13 +921,60 @@ def test_scheduler_plan_route_is_preview_only():
assert response.status_code == 200
assert data["mode"] == "scheduler_attach_plan_preview"
- assert data["phase"] == "phase_33_scheduler_plan_preview"
+ assert data["phase"] == "phase_34_match_review_plan_preview"
assert data["scheduler_registration_executed"] is False
assert data["crawler_job_started"] is False
assert data["external_network_executed"] is False
assert data["database_write_executed"] is False
+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_34_match_review_plan_preview"
+ assert plan["ready_for_review_queue"] is False
+ assert plan["review_queue_created"] is False
+ assert plan["auto_match_executed"] is False
+ assert plan["auto_confirm_executed"] is False
+ assert plan["database_session_created"] is False
+ assert plan["database_write_executed"] is False
+ assert plan["database_commit_executed"] is False
+ assert plan["external_network_executed"] is False
+ assert plan["scheduler_attached"] is False
+ assert plan["writes_executed"] is False
+ assert plan["would_write_database"] is False
+ assert plan["thresholds"]["auto_confirm_enabled"] is False
+ assert len(plan["scoring_signals"]) >= 5
+ assert "needs_review" in plan["planned_status_flow"]
+ assert "confirmed" in plan["planned_status_flow"]
+ assert "manual_operator_approval" in plan["blocked_reasons"]
+ assert "do_not_auto_confirm_matches" in plan["safe_boundaries"]
+ assert all(item["requires_operator"] for item in plan["review_actions"])
+
+
+def test_match_review_plan_route_is_preview_only():
+ from routes.market_intel_routes import market_intel_bp
+
+ app = Flask(__name__)
+ app.secret_key = "test-secret"
+ app.register_blueprint(market_intel_bp)
+ client = app.test_client()
+ with client.session_transaction() as session:
+ session["logged_in"] = True
+
+ response = client.get("/api/market_intel/match_review_plan")
+ data = response.get_json()
+
+ assert response.status_code == 200
+ assert data["mode"] == "match_review_plan_preview"
+ assert data["phase"] == "phase_34_match_review_plan_preview"
+ assert data["review_queue_created"] is False
+ assert data["auto_confirm_executed"] is False
+ assert data["external_network_executed"] is False
+ assert data["database_write_executed"] is False
+
+
def test_mcp_deploy_preflight_blocks_without_required_env():
preflight = build_mcp_deploy_preflight_plan(env={})
@@ -985,7 +1038,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_33_scheduler_plan_preview"
+ assert data["phase"] == "phase_34_match_review_plan_preview"
assert data["deployment_actions_executed"] is False
assert data["docker_command_executed"] is False
assert data["ssh_command_executed"] is False
@@ -1000,7 +1053,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_33_scheduler_plan_preview"
+ assert readiness["phase"] == "phase_34_match_review_plan_preview"
assert readiness["execute_requested"] is False
assert readiness["router_enabled"] is False
assert readiness["external_mcp_complete"] is False
@@ -1388,6 +1441,7 @@ def test_deployment_readiness_reports_app_only_release_gate():
assert readiness["checks"]["mcp_activation_runbook_preview_safe"] is True
assert readiness["checks"]["mcp_fetch_gate_preview_safe"] is True
assert readiness["checks"]["scheduler_plan_preview_safe"] is True
+ assert readiness["checks"]["match_review_plan_preview_safe"] is True
assert readiness["checks"]["writer_plan_dry_run_only"] is True
assert readiness["writer_plan_summary"]["writes_executed"] is False
assert "readiness_checks_not_all_passed" not in readiness["blocked_reasons"]
@@ -1409,6 +1463,7 @@ def test_deployment_readiness_reports_app_only_release_gate():
assert "/api/market_intel/mcp_activation_runbook" in readiness["production_smoke_targets"]
assert "/api/market_intel/mcp_fetch_gate" in readiness["production_smoke_targets"]
assert "/api/market_intel/scheduler_plan" in readiness["production_smoke_targets"]
+ assert "/api/market_intel/match_review_plan" in readiness["production_smoke_targets"]
assert readiness["write_approval_runbook"]["ready_for_real_write"] is False
assert readiness["write_approval_runbook"]["writes_executed"] is False
assert readiness["migration_blueprint"]["migration_executed"] is False
@@ -1430,6 +1485,9 @@ def test_deployment_readiness_reports_app_only_release_gate():
assert readiness["scheduler_plan"]["scheduler_registration_executed"] is False
assert readiness["scheduler_plan"]["crawler_job_started"] is False
assert readiness["scheduler_plan"]["database_write_executed"] is False
+ assert readiness["match_review_plan"]["review_queue_created"] is False
+ assert readiness["match_review_plan"]["auto_confirm_executed"] is False
+ assert readiness["match_review_plan"]["database_write_executed"] is False
def test_write_approval_runbook_is_read_only_and_blocks_real_write():