@tikomni/skills 1.0.5 → 1.0.6

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tikomni/skills",
-  "version": "1.0.5",
+  "version": "1.0.6",
   "description": "TikOmni skill installer CLI for structured social media crawling in Codex, Claude Code, and OpenClaw",
   "license": "MIT",
   "homepage": "https://github.com/mark-ly-wang/TikOmni-Skills#readme",

package/skills/social-media-crawl/scripts/pipelines/run_douyin_single_work.py

@@ -424,7 +424,6 @@ def _u1_fetch_one_video(
             fallback_reason="" if app_response.get("ok") else (
                 "primary_timeout_retry_exhausted" if app_response.get("timeout_retry_exhausted") else "primary_non_timeout_failure"
             ),
-            extra={"response": app_response},
         )
     )
     if app_response.get("ok"):
@@ -458,7 +457,6 @@ def _u1_fetch_one_video(
             fallback_reason="" if web_response.get("ok") else (
                 "fallback_timeout_retry_exhausted" if web_response.get("timeout_retry_exhausted") else "fallback_non_timeout_failure"
             ),
-            extra={"response": web_response},
         )
     )
     web_response["_attempts"] = attempts
@@ -891,23 +889,24 @@ def run_douyin_single_video(
         all_routes_failed=not bool(one_video_response.get("ok")),
     )
     for index, attempt in enumerate(attempts, start=1):
-        response = attempt.get("response") if isinstance(attempt, dict) else None
+        if not isinstance(attempt, dict):
+            continue
+        response = attempt.get("response") if isinstance(attempt.get("response"), dict) else attempt
         endpoint = attempt.get("endpoint") if isinstance(attempt, dict) else None
         label = attempt.get("route_label") if isinstance(attempt, dict) else None
-        if not isinstance(response, dict):
-            if attempt.get("skipped"):
-                trace.append(
-                    {
-                        "step": f"u1_fetch_one_video_attempt_{index}",
-                        "route_label": label,
-                        "endpoint": endpoint,
-                        "accept_reason": attempt.get("accept_reason"),
-                        "fallback_reason": attempt.get("fallback_reason"),
-                        "param_readiness": attempt.get("param_readiness"),
-                        "param_reason": attempt.get("param_reason"),
-                        "skipped": True,
-                    }
-                )
+        if attempt.get("skipped"):
+            trace.append(
+                {
+                    "step": f"u1_fetch_one_video_attempt_{index}",
+                    "route_label": label,
+                    "endpoint": endpoint,
+                    "accept_reason": attempt.get("accept_reason"),
+                    "fallback_reason": attempt.get("fallback_reason"),
+                    "param_readiness": attempt.get("param_readiness"),
+                    "param_reason": attempt.get("param_reason"),
+                    "skipped": True,
+                }
+            )
             continue
         _emit_http_progress(progress, stage="single_video.fetch", response=response, route_label=str(label or "route"))
         step = "u1_fetch_one_video_effective" if index == len(attempts) else f"u1_fetch_one_video_attempt_{index}"

package/skills/social-media-crawl/scripts/pipelines/run_xiaohongshu_single_work.py

@@ -1125,7 +1125,6 @@ def _fetch_note_info(
                 extra={
                     "method": str(route["method"]).upper(),
                     "field_completeness": response.get("_field_completeness"),
-                    "response": response,
                 },
             )
         )
@@ -1843,23 +1842,24 @@ def run_xiaohongshu_extract(
         all_routes_failed=not bool(note_response.get("ok")),
     )
     for index, attempt in enumerate(attempts, start=1):
-        response = attempt.get("response") if isinstance(attempt, dict) else None
+        if not isinstance(attempt, dict):
+            continue
+        response = attempt.get("response") if isinstance(attempt.get("response"), dict) else attempt
         endpoint = attempt.get("endpoint") if isinstance(attempt, dict) else None
         label = attempt.get("route_label") if isinstance(attempt, dict) else None
-        if not isinstance(response, dict):
-            if attempt.get("skipped"):
-                trace.append(
-                    {
-                        "step": f"u1_get_note_info_attempt_{index}",
-                        "route_label": label,
-                        "endpoint": endpoint,
-                        "accept_reason": attempt.get("accept_reason"),
-                        "fallback_reason": attempt.get("fallback_reason"),
-                        "param_readiness": attempt.get("param_readiness"),
-                        "param_reason": attempt.get("param_reason"),
-                        "skipped": True,
-                    }
-                )
+        if attempt.get("skipped"):
+            trace.append(
+                {
+                    "step": f"u1_get_note_info_attempt_{index}",
+                    "route_label": label,
+                    "endpoint": endpoint,
+                    "accept_reason": attempt.get("accept_reason"),
+                    "fallback_reason": attempt.get("fallback_reason"),
+                    "param_readiness": attempt.get("param_readiness"),
+                    "param_reason": attempt.get("param_reason"),
+                    "skipped": True,
+                }
+            )
             continue
         step = "u1_get_note_info_effective" if index == len(attempts) else f"u1_get_note_info_attempt_{index}"
         trace.append(

package/skills/social-media-crawl/tests/test_fixed_pipeline_fallback.py

@@ -2,6 +2,7 @@
 
 from __future__ import annotations
 
+import json
 import os
 import sys
 import unittest
@@ -14,10 +15,59 @@ if str(SKILL_ROOT) not in sys.path:
 
 from scripts.core.asr_pipeline import run_u2_asr_batch_with_timeout_retry
 from scripts.pipelines import homepage_collectors
+from scripts.pipelines import run_douyin_single_work
 from scripts.pipelines import run_xiaohongshu_single_work
 
 
 class FixedPipelineFallbackTest(unittest.TestCase):
+    def test_douyin_single_fetch_attempt_trace_is_json_serializable(self) -> None:
+        def fake_call_json_api(**_: object) -> dict:
+            return {
+                "ok": True,
+                "status_code": 200,
+                "request_id": "req-dy-single",
+                "error_reason": None,
+                "data": {"aweme_detail": {"aweme_id": "123"}},
+            }
+
+        with patch.object(run_douyin_single_work, "call_json_api", side_effect=fake_call_json_api):
+            response = run_douyin_single_work._u1_fetch_one_video(
+                base_url="https://api.tikomni.com",
+                token="test-token",
+                share_url="https://v.douyin.com/test-single/",
+                app_timeout_ms=1000,
+                web_timeout_ms=1000,
+            )
+
+        json.dumps(response, ensure_ascii=False)
+        self.assertNotIn("response", response["_attempts"][0])
+
+    def test_xhs_single_fetch_attempt_trace_is_json_serializable(self) -> None:
+        def fake_call_json_api(**_: object) -> dict:
+            return {
+                "ok": True,
+                "status_code": 200,
+                "request_id": "req-xhs-single",
+                "error_reason": None,
+                "data": {
+                    "title": "test title",
+                    "desc": "test content",
+                    "video": {"master_url": "https://example.com/video.mp4"},
+                },
+            }
+
+        with patch.object(run_xiaohongshu_single_work, "call_json_api", side_effect=fake_call_json_api):
+            response = run_xiaohongshu_single_work._fetch_note_info(
+                base_url="https://api.tikomni.com",
+                token="test-token",
+                timeout_ms=1000,
+                source_input={"share_text": "https://xhslink.com/test-single", "note_id": "note123"},
+                progress=None,
+            )
+
+        json.dumps(response, ensure_ascii=False)
+        self.assertNotIn("response", response["_attempts"][0])
+
     def test_xhs_single_route_plan_respects_version_priority_and_cookie_gate(self) -> None:
         source_input = {
             "share_text": (