Enum with Root Types Filtering Recursive Reference

[ParkerVR]

Changes

Using the --enum and --root-types in combination causes recursive reference to enum types used in schemas, breaking the file.
A few issues related:

#2099
#2341

How to Review

A specific combination of flags causes recursive imports, marked as patch since it seems incredibly unlikely anyone could successfully use this output in a production system. Had a custom in my project and was trying to use the flags instead, but the combined behavior broke me. https://github.com/Equal-IQ/api-model/blob/e99585b6244ac814ea1044f62333474a5857c3b8/Containers/generate.js

Checklist

• Unit tests updated (CJS Test is broken before and after my change, unrelated)
• [] docs/ updated (if necessary) don't think any change is required, as this is a reasonable expected behavior
• [] pnpm run update:examples run (only applicable for openapi-typescript) no changes required to these, since the combination is not used for them

[netlify]

✅ Deploy Preview for openapi-ts ready!

Name	Link
🔨 Latest commit	3fabebe
🔍 Latest deploy log	https://app.netlify.com/projects/openapi-ts/deploys/6876f465d299a00008c1e265
😎 Deploy Preview	https://deploy-preview-2375--openapi-ts.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify project configuration.

[changeset-bot]

🦋 Changeset detected

Latest commit: 3fabebe

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

This PR includes changesets to release 2 packages

Name	Type
openapi-typescript	Patch
swr-openapi	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

[ParkerVR]

I added some snapshot tests here since this change is a bit particular to CLI Flag Combination, though I guess it could be inlined instead like some of the other partial api spec examples. I'll leave it to maintainers to decide on that

[drwpow]

I added some snapshot tests here since this change is a bit particular to CLI Flag Combination, though I guess it could be inlined instead like some of the other partial api spec examples. I'll leave it to maintainers to decide on that

All tests are great! My take is this is a maintainer concern shuffling tests around. Them just existing in a PR is all that matters, thank you!

[drwpow]

Great fix, and great tests! Thank you!