Rename asset 'type' arg to 'asset_type' (Pylint W0622 redefined-builtin)

Author: JE-Chen

docs/source/Eng/doc/new_features/v48_features_doc.rst

@@ -20,9 +20,9 @@ Headless API
     from je_auto_control import AssetStore, active_environment
 
     store = AssetStore("assets.json")
-    store.set("max_retries", 3, type="int", environment="prod")
+    store.set("max_retries", 3, asset_type="int", environment="prod")
     store.set("api_base", "https://prod.example.com", environment="prod")
-    store.set("db_password", "vault_db_pw", type="credential")  # value = a ref
+    store.set("db_password", "vault_db_pw", asset_type="credential")  # value = a ref
 
     store.get("max_retries", environment="prod").value     # -> 3 (typed)
     store.get("api_base", environment="staging").value     # -> falls back to default

docs/source/Zh/doc/new_features/v48_features_doc.rst

@@ -17,9 +17,9 @@ JSON 後端(或記憶體內);純標準函式庫;不匯入 ``PySide6``。
     from je_auto_control import AssetStore, active_environment
 
     store = AssetStore("assets.json")
-    store.set("max_retries", 3, type="int", environment="prod")
+    store.set("max_retries", 3, asset_type="int", environment="prod")
     store.set("api_base", "https://prod.example.com", environment="prod")
-    store.set("db_password", "vault_db_pw", type="credential")  # value = 參照
+    store.set("db_password", "vault_db_pw", asset_type="credential")  # value = 參照
 
     store.get("max_retries", environment="prod").value     # -> 3(具型別)
     store.get("api_base", environment="staging").value     # -> 退回 default

je_auto_control/gui/script_builder/command_schema.py

@@ -1072,7 +1072,8 @@ def _add_misc_specs(specs: List[CommandSpec]) -> None:
         fields=(
             FieldSpec("name", FieldType.STRING),
             FieldSpec("value", FieldType.STRING),
-            FieldSpec("type", FieldType.ENUM, optional=True, default="text",
+            FieldSpec("asset_type", FieldType.ENUM, optional=True,
+                      default="text",
                       choices=("text", "int", "bool", "credential")),
             FieldSpec("environment", FieldType.STRING, optional=True,
                       default="default"),

je_auto_control/utils/assets/assets.py

@@ -75,11 +75,11 @@ def _flush(self) -> None:
         if self._path is not None:
             write_json_dict(self._path, self._data)
 
-    def set(self, name: str, value: Any, *, type: str = TYPE_TEXT,
+    def set(self, name: str, value: Any, *, asset_type: str = TYPE_TEXT,
             environment: str = DEFAULT_ENV) -> None:
         """Store ``value`` for ``name`` under ``environment`` with a type tag."""
         self._data.setdefault(environment, {})[name] = {
-            "type": type, "value": value}
+            "type": asset_type, "value": value}
         self._flush()
 
     def _lookup(self, name: str, environment: str,
@@ -125,11 +125,12 @@ def list(self, *, environment: Optional[str] = None) -> List[Asset]:
         ]
 
 
-def store_set(name: str, value: Any, *, type: str = TYPE_TEXT,
+def store_set(name: str, value: Any, *, asset_type: str = TYPE_TEXT,
               environment: str = DEFAULT_ENV,
               db: Optional[str] = None) -> Dict[str, Any]:
     """Set an asset and return a result dict (shared by executor/MCP layers)."""
-    AssetStore(db).set(name, value, type=type, environment=environment)
+    AssetStore(db).set(name, value, asset_type=asset_type,
+                       environment=environment)
     return {"ok": True, "name": name, "environment": environment}
 
 

je_auto_control/utils/executor/action_executor.py

@@ -3245,12 +3245,13 @@ def _mine_actions(actions: Any, min_len: int = 2, max_len: int = 5,
     }
 
 
-def _set_asset(name: str, value: Any, type: str = "text",
+def _set_asset(name: str, value: Any, asset_type: str = "text",
                environment: str = "default",
                db: Optional[str] = None) -> Dict[str, Any]:
     """Adapter: store a typed, environment-scoped asset."""
     from je_auto_control.utils.assets.assets import store_set
-    return store_set(name, value, type=type, environment=environment, db=db)
+    return store_set(name, value, asset_type=asset_type,
+                     environment=environment, db=db)
 
 
 def _get_asset(name: str, environment: str = "default",

je_auto_control/utils/mcp_server/tools/_factories.py

@@ -3126,13 +3126,14 @@ def asset_tools() -> List[MCPTool]:
     return [
         MCPTool(
             name="ac_set_asset",
-            description=("Store a typed, environment-scoped asset. 'type' is "
-                         "text/int/bool/credential (credential 'value' is a "
+            description=("Store a typed, environment-scoped asset. 'asset_type' "
+                         "is text/int/bool/credential (credential 'value' is a "
                          "secret name, not the secret). Returns {ok}."),
             input_schema=schema(
                 {"name": {"type": "string"}, "value": {},
-                 "type": {"type": "string",
-                          "enum": ["text", "int", "bool", "credential"]},
+                 "asset_type": {"type": "string",
+                                "enum": ["text", "int", "bool",
+                                         "credential"]},
                  **_ENV}, ["name", "value"]),
             handler=h.set_asset,
             annotations=SIDE_EFFECT_ONLY,

je_auto_control/utils/mcp_server/tools/_handlers.py

@@ -1512,9 +1512,10 @@ def mine_actions(actions, min_len=2, max_len=5, min_count=3):
     }
 
 
-def set_asset(name, value, type="text", environment="default", db=None):
+def set_asset(name, value, asset_type="text", environment="default", db=None):
     from je_auto_control.utils.assets.assets import store_set
-    return store_set(name, value, type=type, environment=environment, db=db)
+    return store_set(name, value, asset_type=asset_type,
+                     environment=environment, db=db)
 
 
 def get_asset(name, environment="default", db=None):

test/unit_test/headless/test_assets_batch.py

@@ -9,9 +9,9 @@
 
 def test_typed_coercion():
     store = AssetStore()
-    store.set("retries", "3", type="int")
-    store.set("enabled", "yes", type="bool")
-    store.set("title", "Hi", type="text")
+    store.set("retries", "3", asset_type="int")
+    store.set("enabled", "yes", asset_type="bool")
+    store.set("title", "Hi", asset_type="text")
     assert store.get("retries").value == 3
     assert store.get("enabled").value is True
     assert store.get("title").value == "Hi"
@@ -31,23 +31,23 @@ def test_environment_override_and_fallback():
 
 def test_credential_get_returns_reference_not_secret():
     store = AssetStore()
-    store.set("db_pw", "vault_db_ref", type="credential")
+    store.set("db_pw", "vault_db_ref", asset_type="credential")
     asset = store.get("db_pw")
     assert asset.type == "credential"
     assert asset.value == "vault_db_ref"     # the reference, not a secret
 
 
 def test_credential_resolve_uses_injected_resolver():
     store = AssetStore(secret_resolver={"vault_db_ref": "s3cr3t"}.get)
-    store.set("db_pw", "vault_db_ref", type="credential")
+    store.set("db_pw", "vault_db_ref", asset_type="credential")
     assert store.resolve("db_pw") == "s3cr3t"
-    store.set("plain", "visible", type="text")
+    store.set("plain", "visible", asset_type="text")
     assert store.resolve("plain") == "visible"    # non-credential returns value
 
 
 def test_resolve_without_resolver_raises():
     store = AssetStore()
-    store.set("db_pw", "ref", type="credential")
+    store.set("db_pw", "ref", asset_type="credential")
     with pytest.raises(RuntimeError):
         store.resolve("db_pw")
 
@@ -64,7 +64,7 @@ def test_list_and_delete():
 
 def test_persists_across_instances(tmp_path):
     db = str(tmp_path / "assets.json")
-    AssetStore(db).set("token", "42", type="int", environment="prod")
+    AssetStore(db).set("token", "42", asset_type="int", environment="prod")
     assert AssetStore(db).get("token", environment="prod").value == 42
 
 
@@ -80,7 +80,7 @@ def test_active_environment(monkeypatch):
 def test_executor_round_trip(tmp_path):
     db = str(tmp_path / "a.json")
     ac.execute_action([["AC_set_asset",
-                        {"name": "n", "value": "5", "type": "int",
+                        {"name": "n", "value": "5", "asset_type": "int",
                          "environment": "prod", "db": db}]])
     rec = ac.execute_action([["AC_get_asset",
                               {"name": "n", "environment": "prod", "db": db}]])