Skip to content

Conversation

@aniket-okta
Copy link
Contributor

Summary

Fixes #90 - Prevents module GUID from regenerating on each OpenAPI code generation and syncs SDK with latest OpenAPI specification.

Changes Made

Critical Fixes

  • Added static packageGuid to openapi3/config.json - Prevents dynamic GUID generation that breaks module versioning
  • Updated Build.ps1.mustache template - Uses mustache delimiter syntax to preserve GUID curly braces in generated Build.ps1

Schema Updates

  • Regenerated SDK - Synced with latest OpenAPI spec (management.yaml)
  • Added optional fields to InlineHook OAuth models - New fields: authType, clientId, scope, tokenUrl (non-breaking additions)

Impact

  • Non-breaking - All changes are additive or cosmetic
  • Module GUID remains stable - E2BB23D9-1EB2-4082-B5E3-198D6F3FE65E
  • All 448 tests passing - Full test suite verified
  • Previous fixes preserved - ApiKey authentication fix (commit fbc202a) intact

Testing

  • Built module successfully
  • Validated module manifest with correct static GUID
  • Ran full Pester test suite: 448 passed, 0 failed
  • Verified module imports with 1,629 functions

Root Cause

Issue #90 occurred because fixes were applied to generated files instead of mustache templates, causing regeneration to overwrite previous commits.

Closes #90

@BinoyOza-okta
Copy link

LGTM!

@aniket-okta aniket-okta merged commit 4de7f44 into main Nov 7, 2025
6 checks passed
@aniket-okta aniket-okta deleted the OKTA-1051839 branch November 7, 2025 06:27
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.

Running OpenAPI3 CodeGen removes some previous commits

3 participants