@follenfang/fupload 0.0.22 → 0.0.24

package/fupload/SKILL.md

@@ -2,7 +2,7 @@
 name: fupload
 description: Explicit author-publishing workflow for World of Warcraft plugins, configuration shares, and WA/strings on NewBeeBox, NetEase DD, CurseForge, Heybox Workshop, and ModUs.Creator, including local Creator login reuse and plugin ZIP publishing. Use only when the user explicitly invokes `$fupload`, explicitly asks to use the Fupload Skill, or loads this Skill by path. Do not trigger from ordinary mentions of publishing, NewBeeBox, DD, CurseForge, Heybox, ModUs, plugins, configurations, or WA.
 metadata:
-  version: "0.0.22"
+  version: "0.0.24"
 ---
 
 # Fupload
@@ -22,7 +22,7 @@ Map natural language consistently:
 
 - `create`: create the main record. If first content is a separate platform action, plan it as a following atomic step.
 - `update`: publish a plugin version, change selected configuration-backup content, or publish a WA/string version.
-- `edit`: change fields allowed by the target platform's action allowlist. For DD plugins, this includes `description`, `html_desc`, and existing-record commercial, association, room/channel, and creation-statement settings; other first-publication metadata, categories, media, and version fields are create/update-only. Description edits read both detail and author-list projections; either may confirm the value, but if neither matches, report verification-required.
+- `edit`: change fields allowed by the target platform's action allowlist. For DD plugins, this includes `description`, `html_desc`, `update_desc`, and existing-record commercial, association, room/channel, and creation-statement settings; other first-publication metadata, categories, media, and version fields are create/update-only. Announcement edits preserve the published version and package; this edits the current plugin announcement, not an arbitrary historical version. Edits read both detail and author-list projections; either may confirm the value, but if neither matches, report verification-required.
 - `delete`: delete one main record only. It never deletes an individual version, media object, or associated record.
 
 Never offer bulk delete, drafts, guides, messages, or GUI control automation. Heybox supports individual version deletion with asynchronous readback; whole-module deletion is not supported. The DD task-session command may close verified official GUI processes only after the consent flow below.

package/fupload/references/dd.md

@@ -73,7 +73,7 @@ A published version's `update_desc` cannot be corrected through the CLI: `plugin
 
 Every path in `detail_img_files` must exist as a regular local file. Validation reports the exact indexed path, such as `$.detail_img_files[0]`, before any upload or mutation.
 
-`game_type` and the outer free/paid state are create-only. `plugin update` requires `sn`, `game_versions`, `version`, and `update_desc`; optional `file`, `detail_url`, and `release_type` publish a new version. `plugin edit` requires `sn` and accepts `description`, `html_desc`, plus the official existing-record commercial and association controls (`scope`, payment/lifetime/VIP fields, room/channel linkage, `creation_statement`, and associated content). Other first-publication metadata (`addon_type`, `name`, logo, detail images, and categories) and version fields are not edit fields. The CLI verifies description edits through both detail and author-list readback; if DD accepts the request but leaves the remote value unchanged, it reports a verification-required error rather than false success.
+`game_type` and the outer free/paid state are create-only. `plugin update` requires `sn`, `game_versions`, `version`, and `update_desc`; optional `file`, `detail_url`, and `release_type` publish a new version. `plugin edit` requires `sn` and accepts `description`, `html_desc`, `update_desc`, plus the official existing-record commercial and association controls (`scope`, payment/lifetime/VIP fields, room/channel linkage, `creation_statement`, and associated content). Other first-publication metadata (`addon_type`, `name`, logo, detail images, and categories) and package/version fields are not edit fields. To repair the current announcement, supply only `sn` and HTML `update_desc` (at most 1000 characters including tags) to `plugin edit`; the existing published version and package URL are preserved. This does not edit arbitrary rows in version history. The CLI verifies description edits through both detail and author-list readback; if DD accepts the request but leaves the remote value unchanged, it reports a verification-required error rather than false success.
 
 When `assign_user_sn` is present, the official UI exposes only the public scope option. Python validates the final rebuilt form and rejects a private scope before upload or mutation.
 

package/fupload/references/modus.md

@@ -131,12 +131,12 @@ PUT are separate stages.
 | `file_id` | `fileId` | positive integer | update/edit/delete required; upload allocates | `GET /game/data/author/project/fileId/{projectId}` | allocated ID is used by signature; create metadata omits it | update/delete/signature | `id`/`fileId` in release detail |
 | `version` | `version` | non-empty string, max 120 | create/upload required after ZIP preflight | ZIP `.toc` plus caller label | caller value preserved | metadata create/update | `version` |
 | `type` | `type` | non-empty string, max 40 | create/upload required | Creator release-type choices/API | must be a server-supported value | metadata create/update | `type` |
-| `supported_game_versions` | `supportedGameVersionsReqs[]` | non-empty object array | required; ZIP-derived | installed `.toc` `Interface` values and game config | each entry requires string `gameVersion` and `server`; explicit caller value must match ZIP | metadata create/update | `supportedGameVersionsReqs`/version compatibility rows |
+| `supported_game_versions` | `supportedGameVersionsReqs[]` | non-empty object array | required; ZIP-derived | installed `.toc` `Interface` values and game config | each entry requires string `gameVersion` and `server`; explicit caller value must be a nonempty subset of ZIP and live `wow_builds` choices | metadata create/update | `supportedGameVersionsReqs`/version compatibility rows |
 | `md5` | `md5` | hex string, max 64 | derived from exact ZIP bytes | local ZIP preflight | must describe bytes later PUT | metadata create/update | `md5` |
 | `zip_size` | `zipSize` | non-negative integer | derived | local ZIP stat | must describe exact ZIP | metadata create/update | `zipSize` |
 | `unzip_size` | `unzipSize` | non-negative integer | derived | ZIP central directory | bounded by ZIP preflight | metadata create/update | `unzipSize` |
 | `path` | `path` | string, max 500 | metadata optional | Creator/object response | local path is never binary upload | metadata create/update | `path` |
-| `toc_version` | `tocVersion` | string, max 80 | derived | every ZIP `.toc` `Interface` | incompatible/missing/mixed values fail before write | metadata create/update | `tocVersion` |
+| `toc_version` | `tocVersion` | string, max 80 | derived | each addon `.toc` `Interface`; TOCs under a `Libs` directory are ignored | recognized flavor TOCs form an addon union; missing values and conflicting independent addons fail before write | metadata create/update | `tocVersion` |
 | `changelog` | `changelog` | string/null, max 10000 | optional | free text | omitted/empty behavior is action-specific | metadata create/update | `changelog` |
 | `file` | not in JSON; raw bytes | local path | create/upload required | local filesystem | valid ZIP, max 200 MiB; bytes PUT only after metadata/signature | signed URL from `GET .../upload/signature/{projectId}/{fileId}`, then HTTP `PUT` | object upload status plus release detail hashes/sizes |
 | `transaction_log` | not sent | local path string | optional | caller | redacted durable stage record | local record only | transaction JSON |
@@ -147,9 +147,9 @@ binary upload stages that ran; failed records include `failed_stage`, a redacted
 error, and whether the ZIP was retained. The transaction record is established
 before file-ID allocation and ZIP preflight, so those early failures are also
 durable. Binary-upload errors expose only a query-free endpoint identifier,
-status when available, and a bounded redacted response summary. The provider parses every ZIP `.toc`
+status when available, and a bounded redacted response summary. The provider parses each addon `.toc`
 `Interface` value to derive
-`toc_version` and `supported_game_versions`; explicit caller values must match.
+`toc_version` and `supported_game_versions`, and ignores TOCs under a `Libs` directory. Explicit `toc_version` must match. Known clients absent from live `wow_builds` are excluded from automatic selection; unknown Interface codes still fail. Explicit `supported_game_versions` may narrow selection to a product's supported scope, but must be a nonempty subset of both ZIP and live choices. Creator `plugin get/versions` uses the Creator session; main-client addon reads retain their separate session.
 The provider validates the ZIP, computes `md5`, `zip_size`, and `unzip_size`,
 registers metadata, obtains a signed URL, then uploads the ZIP bytes with HTTP
 `PUT` and `Content-Type: application/zip`. A local path is metadata only; it is

package/fupload/scripts/fupload_cli/__init__.py

@@ -1,3 +1,3 @@
 """Fupload Python CLI."""
 
-__version__ = "0.0.22"
+__version__ = "0.0.24"

package/fupload/scripts/fupload_cli/cli.py

@@ -400,7 +400,7 @@ def main(argv: Optional[Sequence[str]] = None) -> int:
             write_output(platform, operation, data)
             return 0
         modus_doctor = platform == "modus" and resource == "session" and action == "doctor"
-        modus_main_session = resource in ("addon", "plugin", "config", "wa", "builds") or (
+        modus_main_session = resource in ("addon", "config", "wa", "builds") or (
             modus_doctor and getattr(args, "client", "creator") == "main"
         )
         provider = NewBee() if platform == "newbee" else (DD() if platform == "dd" else (Blackbox() if platform == "blackbox" else (CurseForge() if platform == "curseforge" else _modus_provider(authenticate=not modus_doctor, main_session=modus_main_session))))

package/fupload/scripts/fupload_cli/dd.py

@@ -1469,7 +1469,7 @@ PLUGIN_CREATE_DEFAULTS = {
 # exceptions and are verified by detail and author-list readback after every
 # edit. Other first-publication metadata remains create-only.
 PLUGIN_EDIT_FIELDS = (
-    "description", "html_desc",
+    "description", "html_desc", "update_desc",
     "scope", "share_code_life_type", "need_buy", "price_fen", "buy_life_type",
     "jump_room", "room_id", "channel_id", "channel_type", "sync_room",
     "creation_statement", "with_associate", "associated_acts", "need_anchor_vip",

package/fupload/scripts/fupload_cli/modus.py

@@ -24,7 +24,7 @@ from typing import Any, Dict, Mapping, Optional
 
 from .errors import FuploadError, ValidationError, redact
 from .state_machine import COMPLETE, ProjectStateMachine
-from .modus_zip import parse_modus_zip
+from .modus_zip import parse_modus_zip, select_game_versions
 from .transport import json_request, multipart_request
 
 
@@ -1289,14 +1289,17 @@ class ModUs:
                 supplied_games = doc.get("supported_game_versions")
                 if supplied_toc is not None and str(supplied_toc) != derived["toc_version"]:
                     raise ValidationError("toc_version does not match the ZIP Interface field", path="$.toc_version")
-                if supplied_games is not None and _supported_game_versions(supplied_games) != derived["supported_game_versions"]:
-                    raise ValidationError("supported_game_versions does not match the ZIP Interface field", path="$.supported_game_versions")
+                selected_games = select_game_versions(
+                    derived["supported_game_versions"],
+                    self.options("game-versions", keys=["wow_builds"]),
+                    _supported_game_versions(supplied_games) if supplied_games is not None else None,
+                )
                 metadata.update({
                     "md5": md5,
                     "zip_size": size,
                     "unzip_size": unzip_size,
                     "toc_version": derived["toc_version"],
-                    "supported_game_versions": derived["supported_game_versions"],
+                    "supported_game_versions": selected_games,
                     "path": doc.get("path") or archive.name,
                 })
             current_stage = "release_metadata_update" if update else "release_metadata"

package/fupload/scripts/fupload_cli/modus_zip.py

@@ -14,6 +14,7 @@ of being silently classified.
 from __future__ import annotations
 
 import io
+import json
 import os
 import re
 import zipfile
@@ -38,14 +39,33 @@ INTERFACE_GAME_VERSION_MAP: Mapping[str, Mapping[str, str]] = {
     "11507": {"gameVersion": "Classic Era", "server": "wow_classic_era"},
     "11508": {"gameVersion": "Classic Era", "server": "wow_classic_era"},
     "40401": {"gameVersion": "Cataclysm Classic", "server": "wow_classic_cata"},
-    "40402": {"gameVersion": "Cataclysm Classic", "server": "wow_classic_cata"},
+    "11509": {"gameVersion": "1.15.9", "server": "wow_classic_era"},
+    "20506": {"gameVersion": "2.5.6", "server": "wow_anniversary"},
+    "38002": {"gameVersion": "3.80.2", "server": "wow_classic_titan"},
+    "40402": {"gameVersion": "4.4.2", "server": "wow_classic_cata"},
+    "50504": {"gameVersion": "5.5.4", "server": "wow_classic"},
 }
 
+# Recognized WoW clients in universal ZIPs which currently have no Creator
+# server mapping. Preserve their Interface codes, without inventing wire keys.
+KNOWN_UNMAPPED_INTERFACES = {"16001", "30405"}
+
 _INTERFACE_RE = re.compile(r"^\s*##\s*Interface\s*:\s*(.*?)\s*$", re.IGNORECASE)
 _INTERFACE_VALUE_RE = re.compile(r"^\d+$")
 _SOURCE = Union[str, os.PathLike[str], bytes, bytearray, memoryview, BinaryIO]
 
 
+def _is_library_toc(name: str) -> bool:
+    """Return whether ``name`` is an embedded library TOC.
+
+    Creator's release form reads the addon TOC.  Libraries such as Ace3 and
+    LibDeflate live under a ``Libs`` directory and declare their own Interface
+    values, so treating them as addons rejects packages the Creator accepts.
+    """
+    parts = name.replace("\\", "/").split("/")
+    return any(part.casefold() == "libs" for part in parts[:-1])
+
+
 def _read_source(source: _SOURCE) -> Tuple[bytes, str]:
     """Read a path, byte buffer, or seekable stream without leaking content."""
     if isinstance(source, (bytes, bytearray, memoryview)):
@@ -127,27 +147,43 @@ def _zip_entries(raw: bytes, source_name: str) -> Iterable[Tuple[str, bytes]]:
 def parse_modus_zip(source: _SOURCE) -> Dict[str, Any]:
     """Return ModUs metadata inferred from ``source``.
 
-    Every ``.toc`` in the archive must declare the same Interface set.  This
-    avoids choosing an arbitrary addon when a multi-addon archive contains
-    incompatible game versions.  Multiple Interface values in one TOC are
-    supported and become a deterministic comma-separated ``toc_version``.
+    Each addon's base and recognized flavor TOCs form one Interface union.
+    Independent addons must declare the same union. This avoids
+    choosing an arbitrary addon when a multi-addon archive contains
+    incompatible game versions.  TOCs under a ``Libs`` directory are ignored.
+    Multiple Interface values in one TOC are supported and become a
+    deterministic comma-separated ``toc_version``.
 
     Returned keys are JSON-ready and use the exact snake_case names accepted
     by the Fupload ModUs schema.  ``interface_values`` and ``toc_files`` are
     diagnostic fields for callers and can be omitted from the wire request.
     """
     raw, source_name = _read_source(source)
+    addon_entries = [
+        (name, toc_raw)
+        for name, toc_raw in _zip_entries(raw, source_name)
+        if not _is_library_toc(name)
+    ]
+    if not addon_entries:
+        raise ValidationError("ZIP contains no addon .toc file outside a Libs directory", path="$.file")
     signatures: List[Tuple[str, Tuple[str, ...]]] = []
-    for name, toc_raw in _zip_entries(raw, source_name):
+    for name, toc_raw in addon_entries:
         values = tuple(_interface_values(_decode_toc(toc_raw, name), name))
         signatures.append((name, values))
-    expected = signatures[0][1]
-    mismatches = [name for name, values in signatures[1:] if values != expected]
+    groups: Dict[str, set[str]] = {}
+    for name, values in signatures:
+        normalized = name.replace("\\", "/").casefold()
+        addon = re.sub(r"_(?:mainline|vanilla|classic|bcc|tbc|wrath|cata|mists)\.toc$", ".toc", normalized)
+        groups.setdefault(addon, set()).update(values)
+    first = next(iter(groups))
+    expected = tuple(sorted(groups[first], key=lambda item: (int(item), item)))
+    mismatches = [name for name, values in groups.items() if values != set(expected)]
     if mismatches:
-        names = ", ".join([signatures[0][0], *mismatches])
+        names = ", ".join([first, *mismatches])
         raise ValidationError("addon TOC Interface values are ambiguous across files: %s" % names, path="$.file")
 
-    unknown = [value for value in expected if value not in INTERFACE_GAME_VERSION_MAP]
+    unknown = [value for value in expected
+               if value not in INTERFACE_GAME_VERSION_MAP and value not in KNOWN_UNMAPPED_INTERFACES]
     if unknown:
         raise ValidationError(
             "unsupported addon TOC Interface value(s): %s" % ", ".join(unknown),
@@ -156,6 +192,8 @@ def parse_modus_zip(source: _SOURCE) -> Dict[str, Any]:
 
     games: List[Dict[str, str]] = []
     for interface in expected:
+        if interface in KNOWN_UNMAPPED_INTERFACES:
+            continue
         candidate = dict(INTERFACE_GAME_VERSION_MAP[interface])
         if candidate not in games:
             games.append(candidate)
@@ -164,6 +202,7 @@ def parse_modus_zip(source: _SOURCE) -> Dict[str, Any]:
         "supported_game_versions": games,
         "interface_values": list(expected),
         "toc_files": [name for name, _ in signatures],
+        "unmapped_interface_values": [value for value in expected if value in KNOWN_UNMAPPED_INTERFACES],
     }
 
 
@@ -172,4 +211,29 @@ def parse_modus_zip(source: _SOURCE) -> Dict[str, Any]:
 parse_zip_metadata = parse_modus_zip
 
 
-__all__ = ["INTERFACE_GAME_VERSION_MAP", "parse_modus_zip", "parse_zip_metadata"]
+def select_game_versions(derived: Sequence[Mapping[str, str]], config: Any,
+                         supplied: Any = None) -> List[Dict[str, str]]:
+    """Intersect known ZIP clients with live Creator choices and caller scope."""
+    rows = config if isinstance(config, list) else []
+    row = next((item for item in rows if isinstance(item, Mapping) and item.get("key") == "wow_builds"), None)
+    try:
+        builds = json.loads(row["value"]) if row and isinstance(row.get("value"), str) else None
+    except (ValueError, TypeError):
+        builds = None
+    if not isinstance(builds, Mapping) or not builds or any(
+        not isinstance(value, Mapping) or not isinstance(value.get("versions"), list)
+        or any(not isinstance(version, str) or not version.strip() for version in value["versions"])
+        for value in builds.values()
+    ):
+        raise ValidationError("live wow_builds config is missing or malformed", path="$.supported_game_versions")
+    available = [dict(item) for item in derived
+                 if item["server"] in builds and item["gameVersion"] in builds[item["server"]]["versions"]]
+    selected = available if supplied is None else supplied
+    if not isinstance(selected, list) or not selected or any(item not in available for item in selected):
+        raise ValidationError("supported_game_versions must be a nonempty subset of ZIP and live Creator choices", path="$.supported_game_versions")
+    if len({(item["server"], item["gameVersion"]) for item in selected}) != len(selected):
+        raise ValidationError("supported_game_versions must not contain duplicates", path="$.supported_game_versions")
+    return [dict(item) for item in selected]
+
+
+__all__ = ["INTERFACE_GAME_VERSION_MAP", "parse_modus_zip", "parse_zip_metadata", "select_game_versions"]

package/fupload/scripts/fupload_cli/schema.py

@@ -591,6 +591,9 @@ DD_PLUGIN_VERSION = {
     "version": f("string", nonempty=True, max_length=80),
     "update_desc": f("string", nonempty=True, max_length=1000),
 }
+# The official existing-plugin editor includes the update announcement in
+# its /addon/modify payload. Editing it does not require a new ZIP/version.
+DD_PLUGIN_EDIT_META["update_desc"] = DD_PLUGIN_VERSION["update_desc"]
 
 DD_CONFIG_META = {
     "scope": DD_COMMERCIAL["scope"], "title": f("string", nonempty=True, max_length=40),

package/npm/skill-manifest.json

@@ -1,9 +1,9 @@
 {
   "schema": "fupload.npm-skill-manifest.v1",
   "package_name": "@follenfang/fupload",
-  "package_version": "0.0.22",
-  "skill_version": "0.0.22",
-  "tree_sha256": "fffd84093cea75f44ca8ad836f4cfb643e692cf121162201892ad130f23fbc5a",
+  "package_version": "0.0.24",
+  "skill_version": "0.0.24",
+  "tree_sha256": "53bc8cdcaab317ddace896026d8668614dfaeb19f421cf0195fe4846f6e9933c",
   "files": [
     {
       "path": "agents/openai.yaml",
@@ -87,13 +87,13 @@
     },
     {
       "path": "references/dd.md",
-      "bytes": 20837,
-      "sha256": "4a174fa9470a23aee5c607b85123575c157559570a2d3fd4caa9a0874f9ae934"
+      "bytes": 21111,
+      "sha256": "18895d95e5221b3c2243922bbfc6f2e826f9084a0d5d11c37c3eb923a9dd7547"
     },
     {
       "path": "references/modus.md",
-      "bytes": 37142,
-      "sha256": "84e1524bc3404de3c3bc0582d85651b884c616ff009b1d01839fb84e348917b3"
+      "bytes": 37716,
+      "sha256": "f462731d05d786cdfd290eceaea9104b0f28814c9a5bd793358e4dc1a7abcdce"
     },
     {
       "path": "references/newbee-official-cli.md",
@@ -113,7 +113,7 @@
     {
       "path": "scripts/fupload_cli/__init__.py",
       "bytes": 50,
-      "sha256": "0cec28503481955d702020619b9cd9b3ec7032c4d20be0bbad2aa9f4f5dac6f8"
+      "sha256": "8903ed57cd646124d87b730c4c5923b97cacd906cef920de53c6b91acfb554f4"
     },
     {
       "path": "scripts/fupload_cli/blackbox_web.py",
@@ -127,8 +127,8 @@
     },
     {
       "path": "scripts/fupload_cli/cli.py",
-      "bytes": 37420,
-      "sha256": "3a052ec53c29ca95eafc9ba845d448ef45e4bd0231bf41ec1906e55281aa1650"
+      "bytes": 37410,
+      "sha256": "97ce8e53f916f619a16351c636d90b9ddf4a64ebb301a2cf28da81d65522863d"
     },
     {
       "path": "scripts/fupload_cli/curseforge.py",
@@ -147,8 +147,8 @@
     },
     {
       "path": "scripts/fupload_cli/dd.py",
-      "bytes": 117372,
-      "sha256": "19a303e84ce08dd3af9a905d04d24f7224c3302f83f0d62f899618cf45fe0433"
+      "bytes": 117387,
+      "sha256": "01051273f0e03010c88590305583f404de9588e60fc708999222aa80263ebb31"
     },
     {
       "path": "scripts/fupload_cli/errors.py",
@@ -162,13 +162,13 @@
     },
     {
       "path": "scripts/fupload_cli/modus_zip.py",
-      "bytes": 7833,
-      "sha256": "3b35b30266dd48874f679b40d12739534ede3dc58f0c34cdd35ac2deca4e07db"
+      "bytes": 11445,
+      "sha256": "ddf603cb24d5e9758be0428446f134e02aa1a44a8be39f697b8985ac34463e2f"
     },
     {
       "path": "scripts/fupload_cli/modus.py",
-      "bytes": 74134,
-      "sha256": "9841ce7188083b62c1c217d89436b3c3ddc3ecf726719d3c44c0b83027172fac"
+      "bytes": 74167,
+      "sha256": "f4ac5820476593f3db0a621faac61cbed319b0a046fb35945f9a684f292d0e5f"
     },
     {
       "path": "scripts/fupload_cli/newbee_auth.py",
@@ -182,8 +182,8 @@
     },
     {
       "path": "scripts/fupload_cli/schema.py",
-      "bytes": 54911,
-      "sha256": "bede6cc93de8a9d766e463cab0acb8725745da781aebabba73744394b594b93c"
+      "bytes": 55131,
+      "sha256": "b651963c504896cfef46f65b61d5fb638884aacb14a8e09593162526e239133f"
     },
     {
       "path": "scripts/fupload_cli/state_machine.py",
@@ -207,8 +207,8 @@
     },
     {
       "path": "SKILL.md",
-      "bytes": 28582,
-      "sha256": "925b6b480edae4272a6922200a7f046e95df96908edecd4660e1fc49bdb6688d"
+      "bytes": 28729,
+      "sha256": "9625fb102b1fcc347978d0b9fe73a51703ec8296d344550938e8a2dbe593565f"
     }
   ]
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@follenfang/fupload",
-  "version": "0.0.22",
+  "version": "0.0.24",
   "description": "Install and run the Fuploader Agent Skill and Python CLI.",
   "type": "module",
   "bin": {