Skip to content

fix: publish owox declaration files#1336

Open
Pugsin wants to merge 1 commit into
OWOX:mainfrom
Pugsin:codex/publish-owox-types
Open

fix: publish owox declaration files#1336
Pugsin wants to merge 1 commit into
OWOX:mainfrom
Pugsin:codex/publish-owox-types

Conversation

@Pugsin

@Pugsin Pugsin commented Jun 17, 2026

Copy link
Copy Markdown

Summary

  • include generated dist/**/*.d.ts files in the owox package publish allowlist
  • keep existing JS, bin, manifest, and shrinkwrap publish entries unchanged

Why

The package advertises types: "dist/index.d.ts" and the TypeScript config emits declarations, but the published owox@0.27.1 tarball only includes dist/**/*.js because the package files allowlist excludes .d.ts files. TypeScript consumers resolve a missing declaration entrypoint.

Validation

  • npm pack owox@0.27.1 --silent confirmed package/dist/index.js exists and package/dist/index.d.ts is absent
  • node -e "const p=require('./apps/owox/package.json'); if(!p.files.includes('./dist/**/*.d.ts')) process.exit(1)"
  • git diff --check

Signed-off-by: Pugsin <sergio.icdlf@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant