Skip to content

fix(package): exclude python test artifacts#243

Open
YOMXXX wants to merge 1 commit into
TencentCloud:mainfrom
YOMXXX:fix/package-publish-python-artifacts
Open

fix(package): exclude python test artifacts#243
YOMXXX wants to merge 1 commit into
TencentCloud:mainfrom
YOMXXX:fix/package-publish-python-artifacts

Conversation

@YOMXXX

@YOMXXX YOMXXX commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Summary

  • keep the Hermes runtime in the npm package while excluding Python __pycache__, .pyc, and test files
  • add a package files regression test so the exclusions stay in place
  • reduces the current dry-run package from 155 entries / 770,594 bytes to 145 entries / 703,237 bytes

Verification

  • npx vitest run src/package-files.test.ts (RED before fix, green after)
  • npm test
  • npm run build
  • npm pack --dry-run --json --silent
  • git diff --check

Signed-off-by: 李冠辰 <liguanchen@xiaomi.com>
@Maxwell-Code07

Copy link
Copy Markdown
Collaborator

Awesome work! 13 fixes in one batch covering Gateway hardening, embedding compatibility, Hermes memory injection, offload stability, and pipeline optimization — a very productive cleanup session 👏

We'll review and merge them one by one 🚀

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