Skip to content

fix: seed command#55

Merged
maxy-shpfy merged 1 commit into
mainfrom
07-01-fix_seed_command
Jul 1, 2026
Merged

fix: seed command#55
maxy-shpfy merged 1 commit into
mainfrom
07-01-fix_seed_command

Conversation

@maxy-shpfy

@maxy-shpfy maxy-shpfy commented Jul 1, 2026

Copy link
Copy Markdown
Collaborator

TL;DR

pack-bundle.mjs now auto-discovers and packs all example bundles at once, and the seed script runs it automatically before seeding.

What changed?

  • pack-bundle.mjs can now operate without a sourceDir argument. When none is provided, it discovers every folder under examples/ that contains a tangent.yaml and packs each one into a .zip archive. Passing an explicit sourceDir still packs only that single folder.
  • The OUTPUT_DIR can be overridden via the PACK_BUNDLE_OUTPUT_DIR environment variable.
  • The seed script in package.json now runs pack-bundle.mjs first, ensuring all example bundles are freshly packed before the seed operation runs.

How to test?

  1. Run pnpm seed and verify that .zip archives are generated for all examples/<name>/ folders containing a tangent.yaml before the seed completes.
  2. Run node scripts/pack-bundle.mjs directly with no arguments and confirm all discoverable bundles are packed.
  3. Run node scripts/pack-bundle.mjs examples/tangle-oss and confirm only that single bundle is packed, matching previous behavior.

Why make this change?

Previously, pack-bundle.mjs only packed a single hardcoded bundle (examples/tangle-oss), requiring manual invocation for any additional bundles. This change removes that limitation so all example bundles stay in sync automatically, and wires the packing step into seed so the two operations can't fall out of sync.

maxy-shpfy commented Jul 1, 2026

Copy link
Copy Markdown
Collaborator Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@maxy-shpfy maxy-shpfy marked this pull request as ready for review July 1, 2026 23:27
@maxy-shpfy maxy-shpfy requested a review from a team July 1, 2026 23:27

maxy-shpfy commented Jul 1, 2026

Copy link
Copy Markdown
Collaborator Author

Merge activity

  • Jul 1, 11:52 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Jul 1, 11:52 PM UTC: @maxy-shpfy merged this pull request with Graphite.

@maxy-shpfy maxy-shpfy merged commit f369dc6 into main Jul 1, 2026
11 checks passed
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.

1 participant