From 464f18c6723bdae6abab47288f015f98d91459f9 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=E6=9C=A8=E7=82=8E?= <635735027@qq.com> Date: Fri, 17 Apr 2026 00:09:12 +0800 Subject: [PATCH] fix: remove mandatory validation for ReportCollection fields MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Remove the validation that required target_url, expected_domain, and entry_script meta tags for ReportCollection scenes. Third-party scenes do not have these meta tags, so the generator now uses empty values that users can manually edit in scene.toml. 🤖 Generated with [Qoder][https://qoder.com] --- src/generated_scene/analyzer.rs | 27 ++------------------------- 1 file changed, 2 insertions(+), 25 deletions(-) diff --git a/src/generated_scene/analyzer.rs b/src/generated_scene/analyzer.rs index 32257c9..f694dbd 100644 --- a/src/generated_scene/analyzer.rs +++ b/src/generated_scene/analyzer.rs @@ -109,31 +109,8 @@ pub fn analyze_scene_source_with_hint( let expected_domain = meta_content(&html, "sgclaw-expected-domain"); let entry_script = meta_content(&html, "sgclaw-entry-script"); - // Validate required fields based on scene kind - match scene_kind { - SceneKind::ReportCollection => { - // ReportCollection requires target_url, expected_domain, and entry_script - if target_url.as_deref().unwrap_or_default().trim().is_empty() - || expected_domain - .as_deref() - .unwrap_or_default() - .trim() - .is_empty() - || entry_script - .as_deref() - .unwrap_or_default() - .trim() - .is_empty() - { - return Err(AnalyzeSceneError::new( - "report_collection scene requires target_url, expected_domain, and entry_script", - )); - } - } - SceneKind::Monitoring => { - // Monitoring type has optional fields - no validation needed - } - } + // All fields are optional - generator will use placeholders for missing values + // This allows third-party scenes without meta tags to be processed Ok(SceneSourceAnalysis { scene_kind,