Skip to content

fix(scripts): fix broken root scripts and missing contract scaffold - #1413

Merged
Hexstar-labs merged 1 commit into
BrainTease:mainfrom
ola196:fix/root-scripts-broken-paths-1382
Oct 21, 2026
Merged

Hexstar-labs merged 1 commit into
BrainTease:mainfrom
ola196:fix/root-scripts-broken-paths-1382

Conversation

@ola196

@ola196 ola196 commented Oct 21, 2026

Copy link
Copy Markdown
Contributor

Summary

Fixes all root-level scripts that referenced non-existent paths or workspaces, and adds missing scaffolding so every script can execute.

Closes #1382


Changes

Broken scripts fixed

Script Problem Fix
scripts/build-wasm.sh Hard-coded paths stellar-contract/ and packages/contracts/contracts/market/ do not exist Rewrote to loop over contracts/*/ guarded by Cargo.toml presence
scripts/publish-registry.sh Used deprecated soroban contract install (CLI is now stellar); searched for .wasm in repo root where none are committed Updated to stellar contract install; resolves WASMs from target/wasm32-unknown-unknown/release/
scripts/seed-test-data.sh / scripts/seed-staging-data.sh Called npm run seed:basic / seed:performance whose target files (src/test/seeds/*.seed.ts) were missing Created apps/backend/src/test/seeds/{basic,performance,cleanup}.seed.ts

Missing scaffold added

  • contracts/escrow/Cargo.toml — the crate had source code but no manifest, so cargo build skipped it; added Cargo.toml and registered the crate in the root workspace
  • apps/backend/src/test/seeds/ — three entry-point files created for the seed:basic, seed:performance, and seed:cleanup npm scripts

Execute bit (+x) restored

13 scripts were non-executable and would fail with Permission denied on a fresh clone:
audit-eslint-disables.sh, audit-imports.sh, build-wasm.sh, ci-local.sh, consolidate-types.sh, find-duplicate-types.sh, find-unused-exports.sh, fix-imports.sh, generate-contract-errors.js, generate-trend-report.js, migrate-error-enums.sh, publish-registry.sh, remove-unused-exports.sh


Testing

  • Verified all referenced paths now exist in the repository
  • All scripts/*.sh and scripts/*.js files are executable (chmod +x applied)
  • contracts/escrow is now a valid workspace member; cargo check --workspace will include it
  • Seed entry-point files follow the same pattern as src/database/seed.ts

- fix(scripts/build-wasm.sh): replace hard-coded non-existent paths
  (stellar-contract/, packages/contracts/contracts/market/) with a loop
  over contracts/*/ using the actual Cargo.toml presence as a guard

- fix(scripts/publish-registry.sh): replace deprecated 'soroban contract
  install' with 'stellar contract install'; resolve WASM files from
  target/wasm32-unknown-unknown/release/ instead of the repo root

- feat(backend/seeds): create missing seed entry-point files that
  seed:basic, seed:performance and seed:cleanup npm scripts resolve to
  (apps/backend/src/test/seeds/{basic,performance,cleanup}.seed.ts)

- fix(contracts/escrow): add missing Cargo.toml so the crate compiles;
  add contracts/escrow to workspace members in root Cargo.toml

- fix(scripts): add execute bit (+x) to 13 scripts that were missing it:
  audit-eslint-disables.sh, audit-imports.sh, ci-local.sh,
  consolidate-types.sh, find-duplicate-types.sh, find-unused-exports.sh,
  fix-imports.sh, generate-contract-errors.js, generate-trend-report.js,
  migrate-error-enums.sh, remove-unused-exports.sh
  (build-wasm.sh and publish-registry.sh also fixed above)

Closes BrainTease#1382
@drips-wave

drips-wave Bot commented Oct 21, 2026

Copy link
Copy Markdown

@ola196 Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@Hexstar-labs
Hexstar-labs merged commit 59d2640 into BrainTease:main Oct 21, 2026
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.

[Code Quality] #125 Remove unused npm scripts from root package.json

2 participants