Skip to content

fix: prune bundled Jarvis runtime payloads#1102

Draft
artemgetmann wants to merge 1 commit into
codex/jarvis-package-weight-20260704from
codex/jarvis-package-diet-prune-20260706
Draft

fix: prune bundled Jarvis runtime payloads#1102
artemgetmann wants to merge 1 commit into
codex/jarvis-package-weight-20260704from
codex/jarvis-package-diet-prune-20260706

Conversation

@artemgetmann

Copy link
Copy Markdown
Owner

Review Fast Path

  • User path fixed: Jarvis public packages stop carrying avoidable bundled-runtime weight from duplicate pre-sign assets and dependency docs/tests/examples.
  • Proof: bash scripts/test-macos-runtime-prune.sh; bash scripts/test-jarvis-release-size-report.sh; bash -n scripts/lib/macos-runtime-prune.sh scripts/package-mac-app.sh scripts/test-macos-runtime-prune.sh scripts/report-jarvis-release-size.sh scripts/test-jarvis-release-size-report.sh; git diff --check.
  • Shared-state footgun removed: pruning happens only inside the staged app bundle after production dependencies are materialized; repo node_modules, lockfiles, source deps, Intel Node, Intel uv, and macOS native twins stay intact.
  • Still hurts: stacked on chore: expand Jarvis size audit report #1094 until that report-tooling PR auto-merges; no full Jarvis app rebuild in this PR yet.

Why This Matters

  • Current Jarvis update artifact is large: existing report measured Jarvis.zip at 465,595,234 bytes and Jarvis.app at 1,615,323,136 bytes.
  • The safe first diet slice should remove generated/package auxiliary files, not reduce architecture coverage or runtime capability.

Scope Boundary

  • Changes packaging only.
  • Keeps Intel support.
  • Keeps bundled Node x64, bundled uv x64, macOS native addon twins, skills, templates, and extension runtime code.
  • Does not change notarization, Sparkle publication, release truth output, or runtime launch behavior.

Verification

  • bash scripts/test-macos-runtime-prune.sh
  • bash scripts/test-jarvis-release-size-report.sh
  • bash -n scripts/lib/macos-runtime-prune.sh scripts/package-mac-app.sh scripts/test-macos-runtime-prune.sh scripts/report-jarvis-release-size.sh scripts/test-jarvis-release-size-report.sh
  • git diff --check
  • Real artifact assumption check: source dist/assets and dist/plugin-sdk/assets in the warmed release lane are byte-identical before signing; already-signed app copies differ, so this patch dedupes during future staging before signing.
  • Real artifact estimate: node_modules docs/tests/examples/benchmarks under the current warmed Jarvis app account for about 35,454,976 bytes uncompressed.

AI Assistance

  • AI-assisted
  • Testing degree: targeted

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant