Skip to content

Release: smoke test bundlů/binárek před publikací, sjednocení metadat, README - #2

Merged
maly merged 1 commit into
masterfrom
agent/issue-1-smoke-test
Oct 3, 2026
Merged

maly merged 1 commit into
masterfrom
agent/issue-1-smoke-test

Conversation

@maly

@maly maly commented Oct 3, 2026

Copy link
Copy Markdown
Contributor

Closes #1 (kromě rozhodnutí o licenci – viz níže)

Změny

  • Nový scripts/smoke.sh: bez argumentu přeloží ukázkový zdroj (node asm80.js → neprázdný .hex i .lst) a zavolá --help u asm80-link/ar/run (ověřeno: všechny končí exit kódem 0); s argumentem <dir> testuje linuxové pkg binárky (asm80-linux, …).
  • .github/workflows/release.yml: krok „Smoke test bundles“ před „Build binaries“, krok „Smoke test Linux binaries“ po sestavení; v jobu publish (needs: build-and-release) před npm publish smoke test bundlů a npm pack --dry-run.
  • Nový .github/workflows/ci.yml (PR + push do master): smoke test + npm pack --dry-run.
  • package.json: pkg.targets → node20-* (shodně s release workflow), scripts.test (smoke test), engines.node >= 18.
  • Nový README.md (instalace, příkazy, poznámka, že bundly jsou obfuskované a odkud pocházejí).
  • Smazány nepoužívané asm80*.mjs bundly (nikde se na ně neodkazuje, files je nezahrnuje). Pozor: build v asm80/IDEv2/asm80-cli/release.js je případně může znovu vygenerovat.

Ověření

  • npm test (smoke test bundlů) – OK; záměrně rozbitý asm80.js (throw new Error) → skript skončí s exit kódem 1.
  • Lokálně sestaveny pkg binárky (node20-linux-x64,node20-win-x64; pojmenování asm80-linux, asm80-ar-linux, …) a scripts/smoke.sh dist prošel.
  • npm pack --dry-run: 4 bundly + package.json + README.md, 586 kB.
  • Běh GitHub Actions (včetně testovacího tagu) jsem neověřoval.

Nedokončeno – vyžaduje rozhodnutí vlastníka

  • Licence: issue výslovně ponechává na vlastníkovi volbu MIT vs. BSD-2-Clause (a sjednocení s asm80/IDEv2/asm80-cli/package.json). Licenci jsem neměnil a soubor LICENSE nepřidal – po rozhodnutí stačí přidat LICENSE (npm ho do balíčku zahrne automaticky) a sjednotit license v obou repech.

@maly
maly merged commit 5620641 into master Oct 3, 2026
1 check passed
@maly
maly deleted the agent/issue-1-smoke-test branch October 3, 2026 07:25
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.

Release workflow: přidat smoke test sestavených balíčků/binárek před publikací a sjednotit metadata (licence, pkg.targets, README/LICENSE)

1 participant