fix(compile): resolve barrel/type-only default-import link wall — nestjs reaches runtime (#4872)

[proggeramlug]

Fixes the native-link wall from #4872: undefined …__default / __perry_wrap_perry_fn_…__default symbols for import barrels and type-only re-export surfaces. The nestjs-hello fixture now compiles and links end-to-end; it proceeds to a new (different) runtime wall documented in its WALLS.md.

Root causes & fixes

The CJS wrap lowers every require('X') to import _req_N from 'X' (a default import). Four gaps compounded behind that:

1. Default import of a module with no default export (crates/perry/src/commands/compile.rs) — rxjs resolves to its TS source src/index.ts, a barrel with only named exports; uid ships dist/index.mjs, same shape. The classifier unconditionally registered the local as a callable function import, so value reads emitted __perry_wrap_perry_fn_<src>__default and direct reads emitted perry_fn_<src>__default — symbols nothing defines. Now, when the resolved origin module has no default in its (propagated) export map, the local routes through the existing namespace-import machinery — Node's require(esm) semantics: member calls (rxjs_1.lastValueFrom(...)) resolve per-export to origin symbols, and whole-value reads materialize the namespace object.

2. __exportStar(require("./x"), exports) barrels (cjs_wrap/wrap.rs, extract_requires.rs) — tsc's CJS lowering of export * from './x' was invisible to the wrap, so multi-level barrels (@nestjs/common/index.js → decorators/index.js → core/index.js → controller.decorator.js) surfaced no static re-exports and import { Controller } from '@nestjs/common' link-failed on perry_fn_<common_index_js>__Controller. The wrap now also emits a real export * from '<spec>' for each __exportStar call (bare, tslib_1.-member, and (0, …)-sequenced forms), letting the existing transitive re-export propagation resolve names to their defining module. Specs covered by the star re-export are excluded from the recursive named-export pull so the static origin binding isn't shadowed by an export const X = _cjs.X runtime read.

3. Type-only interface modules (cjs_wrap/detect.rs) — nestjs dist *.interface.js files contain only Object.defineProperty(exports, "__esModule", { value: true }); — no exports.X =, no require( — so is_commonjs missed them and they compiled as zero-export ES modules, throwing ReferenceError: exports is not defined at init once the link succeeded. defineProperty(exports, is now a CJS marker.

4. export * leaked default across hops (compile.rs propagation loop) — spec-incorrect, and it would poison the has-default probe in (1) for barrels whose star sources have defaults.

Drive-by: perry-hir/lower_decl/class_validation.rs counted TS constructor overload signatures (body-less constructor(…);) as real constructors, rejecting rxjs's Notification (3 signatures + 1 implementation) with SyntaxError: class may only have one constructor. Only constructors with bodies count now.

Fixture status

tests/release/packages/nestjs-hello now reaches Wrote executable (~41 MB). It still SKIPs (WALLS.md retained): the boot dies in @nestjs/common/services/logger.service.js init because .prototype of a capturing class expression (ClassExprFresh lowering — trigger is a getter capturing an outer variable) reads undefined, so tslib's __decorate throws Cannot convert undefined or null to object. WALLS.md Wall 2 has a 10-line minimal repro for that follow-up.

Validation

• New e2e test crates/perry/tests/issue_4872_barrel_default_reexports.rs covers all three shapes (type-only surface, 2-level __exportStar chain, no-default ESM member call) — compile, link, run, byte-exact output.
• Full cargo test --release -p perry green, including the Stripe SDK: stripe.products.create throws 'replace is not a function' (request dispatch, flat body) after #4831 #4841 namespace-CJS classifier regression test (closest neighbor to the changed code).
• Release fixtures: hono-basic PASS, node-http-serve PASS. axios-get / ws-echo fail identically with the pre-change baseline binary (pre-existing, unrelated).
• cargo fmt --all -- --check and scripts/check_file_size.sh clean.

No version bump / changelog per contributor convention — maintainer folds in at merge.

Refs #4872 (link wall — resolved here; runtime wall remains tracked in the fixture's WALLS.md).