Skip to content

feat: license unification + TLV->ILV rename + WASM real SPZ test#39

Merged
gugu23456789 merged 4 commits into
mainfrom
fix/pages-deploy-on-tag
Jul 4, 2026
Merged

feat: license unification + TLV->ILV rename + WASM real SPZ test#39
gugu23456789 merged 4 commits into
mainfrom
fix/pages-deploy-on-tag

Conversation

@gugu23456789

Copy link
Copy Markdown
Member

Summary

This PR includes three groups of changes:

1. License Unification (MulanPSL-2.0)

  • Changed all @license MIT to @license MulanPSL-2.0 across all source files
  • Removed full MIT license text from report.h
  • Aligns with existing SPDX-License-Identifier headers (MulanPSL-2.0)

2. TLV -> ILV Naming (Indexed-Length-Value)

  • Renamed all TLV references to ILV in comments, error messages, and help text
  • Code was already using ILV naming (IlvRecord, ilv.h, etc.) - only docs/comments were outdated
  • Updated CLI help output and guide text

3. WASM CI Test Enhancement

  • Added real SPZ file testing using classroom_anime dataset from tests/fixtures/
  • Tests inspectSpz, dumpTrailer, listRegisteredExtensions, getCompatibilityBoard APIs
  • Tests UI panel rendering (no n/a in summary/extensions/trailer panels)
  • Covers both SPZ v3 and v4 formats, both synthetic and real files
  • 4 test files: synthetic_v3, synthetic_v4, real_classroom_anime_v3, real_classroom_anime_v4

4. Bug Fixes

  • createUnavailableMethod: returns Promise.reject() instead of sync throw
  • Fallback methods: return Promise.resolve() for consistency

Files Changed

  • 21 files modified, ~200 insertions, ~120 deletions
  • +2 binary fixtures: tests/fixtures/classroom_anime_v3.spz, classroom_anime_v4.spz

Test Plan

  • All CI checks pass (build + test on all platforms)
  • WASM smoke test passes (bundle audit + real SPZ file tests)
  • C++ unit tests pass

… on tag events)\n\nTag push events trigger GITHUB_TOKEN restrictions that prevent deploy_pages from assigning a runner. Switch to workflow_dispatch only.
…Z file WASM test

- License: change all @license MIT to @license MulanPSL-2.0
- License: remove full MIT license text from report.h
- Naming: rename all TLV references to ILV (Indexed-Length-Value) in comments
- Naming: update error messages and help text from TLV to ILV
- CI/WASM: add real SPZ file testing using spz-anime-text2scene-bench dataset
- CI/WASM: test inspectSpz, dumpTrailer, listRegisteredExtensions, getCompatibilityBoard APIs
- CI/WASM: test UI rendering (no n/a in panels) for v3 and v4 SPZ files
- Fix: createUnavailableMethod returns Promise.reject instead of sync throw
- Fix: listRegisteredExtensions/getCompatibilityBoard fallbacks return Promise.resolve
…ynthetic test coverage

- Add tests/fixtures/ with classroom_anime v3+v4 real SPZ files (smallest dataset: 8.47/8.66 MB)
- Remove remote git clone of spz-anime-text2scene-bench (unreliable + path typos)
- Copy real fixtures from tests/fixtures/ to build-wasm/site/ in CI
- Test all 4 files: synthetic_v3, synthetic_v4, real_v3, real_v4
- Each file tests: inspectSpz, dumpTrailer, listRegisteredExtensions, getCompatibilityBoard
- UI upload test uses real_v4.spz (classroom_anime v4)
@gugu23456789
gugu23456789 merged commit 3b1128c into main Jul 4, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants