feat(jwt-templates): add permission, subject, and jti claim configuratio

[aviadl]

Fixes descope/etc#14107

View workflow run

Perfect! All tasks are completed. Let me create a summary of what was done:

Summary

I've successfully added the three missing JWT template configuration options to the Descope Terraform provider:

Changes Made

1. New JWT Template Attributes (in internal/models/project/jwttemplates/jwttemplate.go):

   • exclude_permission_claim (Boolean) - Excludes the permissions claim from JWT tokens
   • override_subject_claim (String) - Overrides the standard sub claim with a custom value
   • add_jti_claim (Boolean) - Adds a jti (JWT ID) claim for uniqueness tracking

2. Updated Model Structure:

   • Added fields to JWTTemplateModel struct
   • Updated Values() method to serialize new fields (using camelCase for API: excludePermissionClaim, overrideSubjectClaim, addJtiClaim)
   • Updated SetValues() method to deserialize new fields from API responses

3. Documentation:

   • Added descriptive documentation strings in internal/docs/docs.go
   • Regenerated Terraform registry documentation in docs/resources/project.md

4. Tests:

   • Updated test cases in internal/models/project/jwttemplates/jwttemplates_test.go to include the new attributes
   • All existing tests pass

5. Verification:

   • Code compiles successfully
   • Unit tests pass
   • Generated documentation includes the new fields

Commit Details

Created commit 8d0cd03 with message:

feat: add JWT template config options for permissions, subject, and JTI claims

The changes are now ready. The GitHub Actions workflow will handle pushing and creating the PR as per your configuration.

---

Created by Shuni 🐕

[Copilot]

Pull request overview

Adds missing JWT template configuration options to the Terraform provider so users can control permission, subject, and JTI claim behavior.

Changes:

• Extends JWT template schema/model with exclude_permission_claim, override_subject_claim, and add_jti_claim.
• Serializes/deserializes the new fields to/from API payloads.
• Regenerates docs and updates JWT template tests accordingly.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.

File	Description
internal/models/project/jwttemplates/jwttemplates_test.go	Adds assertions/config for the three new JWT template attributes.
internal/models/project/jwttemplates/jwttemplate.go	Adds schema + model fields and maps them in Values() / SetValues().
internal/docs/docs.go	Adds documentation strings for the new JWT template attributes.
docs/resources/project.md	Updates generated registry docs to include the new JWT template fields (but also removes unrelated connector-auth doc details).

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[aviadl]

@shuni please see my comments

[shuni-bot]

🐕 Done! Updated PR: #242

View workflow run

📋 Actions Taken

✅ Fix override_subject_claim to be boolean instead of string
✅ Add conditional serialization for override_subject_claim in Values()
✅ Update tests for boolean override_subject_claim
✅ Regenerate documentation
✅ Reply to PR threads explaining the changes
✅ Run tests and commit changes