Skip to content

ci(spec-check): sparse-checkout @devhelm/openapi-tools from mono#15

Merged
caballeto merged 1 commit into
mainfrom
fix/spec-check-openapi-tools-sparse
May 1, 2026
Merged

ci(spec-check): sparse-checkout @devhelm/openapi-tools from mono#15
caballeto merged 1 commit into
mainfrom
fix/spec-check-openapi-tools-sparse

Conversation

@caballeto
Copy link
Copy Markdown
Contributor

Summary

  • First real dispatch fan-out of spec_updated surfaced that typegen.sh falls through to npx --package=@devhelm/openapi-tools in CI, which 404s (the package is an internal shared Springdoc preprocessor, not published to npm).
  • Sparse-checkout packages/openapi-tools from mono using the existing MONOREPO_DISPATCH_TOKEN, build it in place, and export OPENAPI_TOOLS so typegen.sh uses the explicit override instead of the npm fallback.
  • No new secret, no public npm publish, no change to typegen.sh (local dev still uses the sibling-monorepo path).

Test plan

  • Merge, then trigger spec-propagate on mono (or wait for next API spec change).
  • Spec Check job on this repo completes green; no PR opened when types are already in sync with mono's spec.

Made with Cursor

The shared Springdoc preprocessor used by typegen.sh is an internal-only
package that is intentionally not published to npm. The previous fallback
to `npx --package=@devhelm/openapi-tools` 404'd on every spec-check run
once the repository_dispatch fan-out started actually firing.

Sparse-checkout packages/openapi-tools from mono with the existing
MONOREPO_DISPATCH_TOKEN, build it in-place, and export OPENAPI_TOOLS so
typegen.sh picks the explicit-override path instead of the npm fallback.

No new secrets, no public npm publish.
@caballeto caballeto merged commit 94fd080 into main May 1, 2026
4 checks passed
@caballeto caballeto deleted the fix/spec-check-openapi-tools-sparse branch May 1, 2026 13:39
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