Skip to content

fix(app-tools): unpack native companion libraries - #149

Merged
snowzane merged 1 commit into
mainfrom
codex/app-tools-native-unpack
Sep 14, 2026
Merged

snowzane merged 1 commit into
mainfrom
codex/app-tools-native-unpack

Conversation

@snowzane

@snowzane snowzane commented Sep 14, 2026

Copy link
Copy Markdown
Contributor

Summary

The published Next AI Draw.io package failed before opening a window because the scaffold's ASAR rule unpacked Sharp's native add-on but left its libvips dylib inside the archive. App Tools 0.6.3 preserves the on-disk layout of .node, .dylib, and .dll files for the supported Mac/Windows targets. Existing App-owned packagers receive the same guidance through the release skill and README.

Scope

  • Component: app-tools.
  • Type: packaging bug fix; no API or admission-policy change.

Verification

  • App Tools suite: 135 passed, 11 platform-dependent skips.
  • Component package build and standalone scaffold smoke passed.
  • Package license matrix, SDK version matrix, and diff checks passed.
  • The exact installed 0.4.17 Sharp add-on reproduces the missing-dylib failure; the same archive fully extracted loads Sharp 0.35.4/libvips 8.18.6.
  • Real ASAR comparison with the corrected rule loads the Mac add-on; both actual Windows libvips DLLs are unpacked with matching bytes. Windows execution remains unverified.
  • Corrected App package launch and business acceptance; tracked separately in the Next AI Draw.io adaptation.
pnpm --filter @nimiplatform/app-tools test
pnpm release:component:pack -- --component app-tools --output-dir dist/component-release/app-tools-0.6.3
pnpm check:package-license-matrix
pnpm check:sdk-version-matrix
git diff --check

Breaking Changes

None. Existing App-owned packagers need the documented rule; sync does not overwrite their code.

Security Impact

Low: dependency libraries remain at their original relative paths outside ASAR so the OS loader can resolve them. Publisher signing/sealing and Runtime verification remain unchanged.

Rollback Plan

No data migration. A correction to a published package uses a new version; immutable releases remain unchanged.

Docs

Updated the package README, release skill, and starter README, including the stale asar: false description.

DCO

  • Commit is signed off.

Signed-off-by: snowzane <eric.3126@gmail.com>
@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying nimi-doc with  Cloudflare Pages  Cloudflare Pages

Latest commit: 8cbbd64
Status: ✅  Deploy successful!
Preview URL: https://adcce5fa.nimi-doc.pages.dev
Branch Preview URL: https://codex-app-tools-native-unpac.nimi-doc.pages.dev

View logs

@snowzane
snowzane merged commit e350606 into main Sep 14, 2026
22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant