Skip to content

fix(worma): isolate schema workers per generator - #187

Merged
JOU-amjs merged 1 commit into
alovajs:mainfrom
LittleTurtle2333:fix/worma-generator-worker-isolation
Aug 11, 2026
Merged

fix(worma): isolate schema workers per generator#187
JOU-amjs merged 1 commit into
alovajs:mainfrom
LittleTurtle2333:fix/worma-generator-worker-isolation

Conversation

@LittleTurtle2333

Copy link
Copy Markdown
Contributor

Summary

  • isolate schema worker pools by each generator output directory
  • prevent concurrent OpenAPI generators from resolving refs against another document
  • add a two-generator regression test using distinct MallActivity and AdminUser schemas
  • document the isolation contract and add a patch changeset

Root cause

Schema workers were cached only by projectPath even though workerData permanently captures one OpenAPI document. Multiple generators in the same project could therefore reuse the wrong document context, causing timing-dependent missing-ref failures or false success when processBatch calls overlapped.

Validation

  • pnpm --filter wormajs test: 63 files passed, 777 tests passed, 6 skipped
  • pnpm --filter wormajs build
  • ESLint on the changed source and regression test

@changeset-bot

changeset-bot Bot commented Aug 11, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: e2d9728

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
wormajs Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions github-actions Bot added the worma label Aug 11, 2026
@JOU-amjs
JOU-amjs merged commit 0215b15 into alovajs:main Aug 11, 2026
2 checks passed
@JOU-amjs JOU-amjs mentioned this pull request Aug 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants