@bitget-ai/getagent-skill 0.2.1 → 0.2.2

package/skills/getagent/references/sdk/data/wikipedia.md

@@ -14,7 +14,7 @@ are callable through the DataSDK wrapper.
 ### `wikipedia.content`
 
 ```python
-data.wikipedia.content(title=None, lang='en', intro_only=False, provider='wikipedia')
+data.wikipedia.content(title=None, lang='en', intro_only=False)
 ```
 
 Summary: Content
@@ -24,7 +24,6 @@ Summary: Content
 | Endpoint ID | `wikipedia.content` |
 | HTTP | `GET` |
 | Path | `/inner/v1/agent-data/wikipedia/content` |
-| Default provider | `wikipedia` |
 | SDK | `supported` |
 | Host | `supported` |
 | Notes | - |
@@ -33,17 +32,16 @@ Summary: Content
 
 | Param | Required | Type | Default | Notes |
 |---|---|---|---|---|
-| `title` | `no` | `string | null` | `-` | Wikipedia article title to retrieve. (provider: wikipedia) |
-| `lang` | `no` | `string` | `en` | Wikipedia language edition. (provider: wikipedia) |
-| `intro_only` | `no` | `boolean` | `false` | Return only the introductory section instead of the full article. (provider: wikipedia) |
-| `provider` | `no` | `string` | `wikipedia` | - |
+| `title` | `no` | `string | null` | `-` | Wikipedia article title to retrieve. |
+| `lang` | `no` | `string` | `en` | Wikipedia language edition. |
+| `intro_only` | `no` | `boolean` | `false` | Return only the introductory section instead of the full article. |
 
 ---
 
 ### `wikipedia.search`
 
 ```python
-data.wikipedia.search(query=None, max_results=10, page=1, lang='en', provider='wikipedia')
+data.wikipedia.search(query=None, max_results=10, page=1, lang='en')
 ```
 
 Summary: Search
@@ -53,7 +51,6 @@ Summary: Search
 | Endpoint ID | `wikipedia.search` |
 | HTTP | `GET` |
 | Path | `/inner/v1/agent-data/wikipedia/search` |
-| Default provider | `wikipedia` |
 | SDK | `supported` |
 | Host | `supported` |
 | Notes | - |
@@ -62,18 +59,17 @@ Summary: Search
 
 | Param | Required | Type | Default | Notes |
 |---|---|---|---|---|
-| `query` | `no` | `string | null` | `-` | Search query string. (provider: wikipedia) |
-| `max_results` | `no` | `integer` | `10` | Maximum number of results. (provider: wikipedia) |
-| `page` | `no` | `integer` | `1` | Page number, starting at 1. (provider: wikipedia) |
-| `lang` | `no` | `string` | `en` | Wikipedia language edition. (provider: wikipedia) |
-| `provider` | `no` | `string` | `wikipedia` | - |
+| `query` | `no` | `string | null` | `-` | Search query string. |
+| `max_results` | `no` | `integer` | `10` | Maximum number of results. |
+| `page` | `no` | `integer` | `1` | Page number, starting at 1. |
+| `lang` | `no` | `string` | `en` | Wikipedia language edition. |
 
 ---
 
 ### `wikipedia.summary`
 
 ```python
-data.wikipedia.summary(title=None, lang='en', provider='wikipedia')
+data.wikipedia.summary(title=None, lang='en')
 ```
 
 Summary: Summary
@@ -83,7 +79,6 @@ Summary: Summary
 | Endpoint ID | `wikipedia.summary` |
 | HTTP | `GET` |
 | Path | `/inner/v1/agent-data/wikipedia/summary` |
-| Default provider | `wikipedia` |
 | SDK | `supported` |
 | Host | `supported` |
 | Notes | - |
@@ -92,6 +87,5 @@ Summary: Summary
 
 | Param | Required | Type | Default | Notes |
 |---|---|---|---|---|
-| `title` | `no` | `string | null` | `-` | Wikipedia article title to retrieve. (provider: wikipedia) |
-| `lang` | `no` | `string` | `en` | Wikipedia language edition. (provider: wikipedia) |
-| `provider` | `no` | `string` | `wikipedia` | - |
+| `title` | `no` | `string | null` | `-` | Wikipedia article title to retrieve. |
+| `lang` | `no` | `string` | `en` | Wikipedia language edition. |

package/skills/getagent/scripts/validate.py

@@ -19,6 +19,7 @@ from __future__ import annotations
 import ast
 import re
 import sys
+from datetime import datetime
 from pathlib import Path, PurePosixPath
 from zoneinfo import ZoneInfo, ZoneInfoNotFoundError
 
@@ -52,6 +53,7 @@ MANIFEST_REQUIRED_FIELDS = [
 
 NAME_PATTERN = re.compile(r"^[a-z0-9]([a-z0-9-]{0,61}[a-z0-9])?$")
 BACKTEST_BAR_FIELD_PATTERN = re.compile(r"^[a-z][a-z0-9_]*$")
+PUBLIC_SYMBOL_TOKEN_PATTERN = re.compile(r"\b[A-Z0-9]{2,20}(?:USDT|USDC|USD|BTC|ETH)\b")
 CRON_EVERY_MINUTES_PATTERN = re.compile(r"^\*/(\d+)$")
 MIN_SCHEDULE_INTERVAL_MINUTES = 10
 DEFAULT_SCHEDULE_TZ = "Asia/Shanghai"
@@ -375,6 +377,21 @@ def validate_manifest(pkg_dir: Path, result: ValidationResult) -> dict:
     symbols = data.get("trading_symbols", [])
     if not isinstance(symbols, list) or not all(isinstance(item, str) and item.strip() for item in symbols):
         result.error("manifest.yaml: 'trading_symbols' must be a non-empty list of strings")
+    else:
+        normalized_symbols = {str(item).strip().upper() for item in symbols}
+        for field in ("display_name", "description"):
+            text = str(data.get(field, "") or "")
+            unknown_symbols = sorted(
+                token
+                for token in PUBLIC_SYMBOL_TOKEN_PATTERN.findall(text.upper())
+                if token not in normalized_symbols
+            )
+            if unknown_symbols:
+                result.error(
+                    f"manifest.yaml: '{field}' mentions symbols {unknown_symbols} "
+                    f"outside trading_symbols {sorted(normalized_symbols)}; "
+                    "if you corrected a typo or changed the fallback symbol, update all display text"
+                )
 
     decision_mode = data.get("decision_mode", "")
     if decision_mode and decision_mode not in DECISION_MODES:
@@ -465,6 +482,24 @@ def validate_backtest_yaml(pkg_dir: Path, manifest: dict, result: ValidationResu
         if not str(payload.get(field, "") or "").strip():
             result.error(f"{prefix}: missing '{field}'")
 
+    def _parse_backtest_datetime(value: object) -> datetime | None:
+        text = str(value or "").strip()
+        if not text:
+            return None
+        try:
+            return datetime.fromisoformat(text.replace("Z", "+00:00"))
+        except ValueError:
+            return None
+
+    def _instrument_symbol(item: dict) -> str:
+        raw = str(
+            item.get("raw_symbol")
+            or item.get("symbol")
+            or str(item.get("id", "") or "").split(".", 1)[0]
+            or ""
+        ).strip().upper()
+        return raw
+
     def _validate_instrument(item: dict, *, prefix: str) -> None:
         kind = str(item.get("kind", "") or "").strip().lower()
         if kind not in BACKTEST_INSTRUMENT_KINDS:
@@ -513,6 +548,24 @@ def validate_backtest_yaml(pkg_dir: Path, manifest: dict, result: ValidationResu
                 continue
             seen.add(field)
 
+    def _validate_no_provider(payload: object, *, prefix: str) -> None:
+        if isinstance(payload, dict):
+            if "provider" in payload:
+                result.error(
+                    f"{prefix}: 'provider' is not allowed; GetAgent routes historical "
+                    "data through the managed DataSDK provider"
+                )
+            for key, value in payload.items():
+                _validate_no_provider(value, prefix=f"{prefix}.{key}")
+        elif isinstance(payload, list):
+            for index, value in enumerate(payload):
+                _validate_no_provider(value, prefix=f"{prefix}[{index}]")
+
+    def _record_backtest_symbol(item: dict, *, prefix: str) -> None:
+        symbol = _instrument_symbol(item)
+        if symbol:
+            backtest_symbols.append((prefix, symbol))
+
     path = pkg_dir / "backtest.yaml"
     if not path.exists():
         return
@@ -522,6 +575,9 @@ def validate_backtest_yaml(pkg_dir: Path, manifest: dict, result: ValidationResu
         result.error("backtest.yaml: invalid YAML syntax")
         return
 
+    _validate_no_provider(data, prefix="backtest.yaml")
+    backtest_symbols: list[tuple[str, str]] = []
+
     if manifest.get("backtest_support") != "full":
         result.error("backtest.yaml is only allowed when manifest.yaml sets backtest_support: full")
 
@@ -572,6 +628,7 @@ def validate_backtest_yaml(pkg_dir: Path, manifest: dict, result: ValidationResu
         result.error("backtest.yaml: use either 'instrument' or 'instruments', not both")
     elif has_single:
         _validate_instrument(data["instrument"], prefix="backtest.yaml.instrument")
+        _record_backtest_symbol(data["instrument"], prefix="backtest.yaml.instrument")
     elif has_many:
         instruments = data.get("instruments") or []
         if not instruments:
@@ -581,18 +638,68 @@ def validate_backtest_yaml(pkg_dir: Path, manifest: dict, result: ValidationResu
                 result.error(f"backtest.yaml.instruments[{index}] must be a mapping")
                 continue
             _validate_instrument(item, prefix=f"backtest.yaml.instruments[{index}]")
+            _record_backtest_symbol(item, prefix=f"backtest.yaml.instruments[{index}]")
     else:
         result.error("backtest.yaml: missing 'instrument' or 'instruments'")
 
+    manifest_symbols = {
+        str(item or "").strip().upper()
+        for item in manifest.get("trading_symbols", [])
+        if str(item or "").strip()
+    }
+    if manifest_symbols and backtest_symbols:
+        actual_symbols = {symbol for _, symbol in backtest_symbols}
+        extra_symbols = [
+            f"{prefix}={symbol}"
+            for prefix, symbol in backtest_symbols
+            if symbol not in manifest_symbols
+        ]
+        missing_symbols = sorted(manifest_symbols - actual_symbols)
+        if extra_symbols or missing_symbols:
+            result.error(
+                "backtest.yaml: instruments must match manifest.yaml trading_symbols; "
+                f"unexpected backtest symbols={extra_symbols or []}, "
+                f"missing manifest symbols={missing_symbols or []}"
+            )
+
     execution = data.get("execution")
-    if execution is not None and not isinstance(execution, dict):
+    if not isinstance(execution, dict):
         result.error("backtest.yaml: 'execution' must be a mapping")
+    else:
+        start = str(execution.get("start") or "").strip()
+        end = str(execution.get("end") or "").strip()
+        if not start or not end:
+            result.error(
+                "backtest.yaml.execution: 'start' and 'end' are required so "
+                "the sandbox can prove the replay window has bars"
+            )
+        else:
+            parsed_start = _parse_backtest_datetime(start)
+            parsed_end = _parse_backtest_datetime(end)
+            if parsed_start is None or parsed_end is None:
+                result.error("backtest.yaml.execution: 'start' and 'end' must be ISO datetimes")
+            elif parsed_end <= parsed_start:
+                result.error("backtest.yaml.execution: 'end' must be after 'start'")
 
     data_requirements = data.get("data_requirements")
     if data_requirements is not None and not isinstance(data_requirements, dict):
         result.error("backtest.yaml: 'data_requirements' must be a mapping")
     elif isinstance(data_requirements, dict):
         _validate_required_bar_fields(data_requirements.get("required_bar_fields"))
+        required_fields = data_requirements.get("required_bar_fields")
+        if isinstance(required_fields, list):
+            source_text = "\n".join(
+                path.read_text(encoding="utf-8", errors="ignore")
+                for path in (pkg_dir / "src").rglob("*.py")
+            )
+            for field in required_fields:
+                if isinstance(field, str) and field.strip() and field.strip() not in source_text:
+                    result.error(
+                        "backtest.yaml.data_requirements.required_bar_fields: "
+                        f"declares '{field.strip()}' but src/** never references it; "
+                        "build the feature column with backtest.build_feature_frame(...) "
+                        "or remove the declaration"
+                    )
 
 
 def _local_import_roots(pkg_dir: Path) -> set[str]:
@@ -764,6 +871,23 @@ def _check_contract_tpsl_helper_call(tree: ast.AST, *, source_path: str) -> list
     return errors
 
 
+def _check_data_provider_keyword(tree: ast.AST, *, source_path: str) -> list[str]:
+    errors: list[str] = []
+    for node in ast.walk(tree):
+        if not isinstance(node, ast.Call):
+            continue
+        path = _attribute_path(node.func)
+        if not path or "data" not in path:
+            continue
+        for keyword in node.keywords:
+            if keyword.arg == "provider":
+                errors.append(
+                    f"{source_path}: do not pass provider=... to getagent.data calls "
+                    f"(line {keyword.value.lineno}); the managed DataSDK provider is selected by the platform"
+                )
+    return errors
+
+
 def _test_contains_follow_trade_guard(test: ast.AST) -> bool:
     if isinstance(test, ast.Call):
         return _attribute_path(test)[-2:] == ["runtime", "is_follow_trade"]
@@ -865,6 +989,8 @@ def validate_src_tree(pkg_dir: Path, result: ValidationResult) -> None:
             result.error(error)
         for error in _check_contract_tpsl_helper_call(tree, source_path=rel_path):
             result.error(error)
+        for error in _check_data_provider_keyword(tree, source_path=rel_path):
+            result.error(error)
         for error in _check_live_trade_mutation_guards(tree, source_path=rel_path):
             result.error(error)