Skip to content

feat: migrate Release, Workflow, PublishQueue & PublishRule modules to System.Text.Json#187

Merged
OMpawar-21 merged 6 commits into
enhc/betafrom
enhc/DX-7306-release-module
Jun 1, 2026
Merged

feat: migrate Release, Workflow, PublishQueue & PublishRule modules to System.Text.Json#187
OMpawar-21 merged 6 commits into
enhc/betafrom
enhc/DX-7306-release-module

Conversation

@OMpawar-21
Copy link
Copy Markdown
Contributor

@OMpawar-21 OMpawar-21 commented May 28, 2026

Summary

  • Replaced Newtonsoft.Json attributes with STJ equivalents in ReleaseModel, PublishRuleModel, and DeleteReleaseItemService
  • Fixed all service call sites in Workflow, PublishRule, PublishQueue, Release, and ReleaseItem to drop the legacy Newtonsoft serializer argument
  • Re-enabled Stack.Release(), Stack.Workflow(), and Stack.PublishQueue() methods (previously commented out pending migration)
  • Removed all <Compile Remove> exclusions across core, unit test, and integration test projects
  • Migrated unit and integration test assertion helpers from OpenJObjectResponse (Newtonsoft) to OpenJsonObjectResponse (STJ)
  • Restored workflow + publish rule setup in BulkOperationTest.ClassInitialize and implemented the previously stubbed EnsureBulkTestWorkflowAndPublishingRuleAsync

Test Plan

  • dotnet build Contentstack.Management.Core/contentstack.management.core.csproj — 0 errors
  • dotnet test Contentstack.Management.Core.Unit.Tests/ — all 991 tests pass
  • dotnet build Contentstack.Management.Core.Tests/ — 0 errors
  • Integration tests Contentstack015_BulkOperationTest, Contentstack004_ReleaseTest, Contentstack020_WorkflowTest run without workflow/Newtonsoft-related failures

🤖 Generated with Claude Code

@OMpawar-21
feat: migrate Release, Workflow, PublishQueue & PublishRule modules t…
c1d9aff 
…o System.Text.Json

- Replace Newtonsoft.Json attributes with System.Text.Json equivalents in
  ReleaseModel, PublishRuleModel, and DeleteReleaseItemService
- Fix FetchDeleteService/CreateUpdateService call sites in Workflow, PublishRule,
  PublishQueue, Release, and ReleaseItem (remove legacy serializer argument)
- Uncomment Stack.Release(), Stack.Workflow(), and Stack.PublishQueue() methods
- Remove <Compile Remove> exclusions from core, unit test, and integration test projects
- Update unit test assertions from OpenJObjectResponse (Newtonsoft) to OpenJsonObjectResponse (STJ)
- Migrate Contentstack004_ReleaseTest integration test from Newtonsoft.Json.Linq to System.Text.Json.Nodes
- Implement EnsureBulkTestWorkflowAndPublishingRuleAsync in BulkOperationTest
  to restore workflow/publish rule setup now that Stack.Workflow() is active
- Fix Contentstack020_WorkflowTest: replace .Value<bool>() with .GetValue<bool>() for STJ JsonNode
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 28, 2026

🔒 Security Scan Results

ℹ️ Note: Only vulnerabilities with available fixes (upgrades or patches) are counted toward thresholds.

Check Type Count (with fixes) Without fixes Threshold Result
🔴 Critical Severity 0 0 10 ✅ Passed
🟠 High Severity 0 0 25 ✅ Passed
🟡 Medium Severity 0 0 500 ✅ Passed
🔵 Low Severity 0 0 1000 ✅ Passed

⏱️ SLA Breach Summary

✅ No SLA breaches detected. All vulnerabilities are within acceptable time thresholds.

Severity Breaches (with fixes) Breaches (no fixes) SLA Threshold (with/no fixes) Status
🔴 Critical 0 0 15 / 30 days ✅ Passed
🟠 High 0 0 30 / 120 days ✅ Passed
🟡 Medium 0 0 90 / 365 days ✅ Passed
🔵 Low 0 0 180 / 365 days ✅ Passed

✅ BUILD PASSED - All security checks passed

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 1, 2026

🔒 Security Scan Results

ℹ️ Note: Only vulnerabilities with available fixes (upgrades or patches) are counted toward thresholds.

Check Type Count (with fixes) Without fixes Threshold Result
🔴 Critical Severity 0 0 10 ✅ Passed
🟠 High Severity 0 0 25 ✅ Passed
🟡 Medium Severity 0 0 500 ✅ Passed
🔵 Low Severity 0 0 1000 ✅ Passed

⏱️ SLA Breach Summary

✅ No SLA breaches detected. All vulnerabilities are within acceptable time thresholds.

Severity Breaches (with fixes) Breaches (no fixes) SLA Threshold (with/no fixes) Status
🔴 Critical 0 0 15 / 30 days ✅ Passed
🟠 High 0 0 30 / 120 days ✅ Passed
🟡 Medium 0 0 90 / 365 days ✅ Passed
🔵 Low 0 0 180 / 365 days ✅ Passed

✅ BUILD PASSED - All security checks passed

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 1, 2026

🔒 Security Scan Results

ℹ️ Note: Only vulnerabilities with available fixes (upgrades or patches) are counted toward thresholds.

Check Type Count (with fixes) Without fixes Threshold Result
🔴 Critical Severity 0 0 10 ✅ Passed
🟠 High Severity 0 0 25 ✅ Passed
🟡 Medium Severity 0 0 500 ✅ Passed
🔵 Low Severity 0 0 1000 ✅ Passed

⏱️ SLA Breach Summary

✅ No SLA breaches detected. All vulnerabilities are within acceptable time thresholds.

Severity Breaches (with fixes) Breaches (no fixes) SLA Threshold (with/no fixes) Status
🔴 Critical 0 0 15 / 30 days ✅ Passed
🟠 High 0 0 30 / 120 days ✅ Passed
🟡 Medium 0 0 90 / 365 days ✅ Passed
🔵 Low 0 0 180 / 365 days ✅ Passed

✅ BUILD PASSED - All security checks passed

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 1, 2026

🔒 Security Scan Results

ℹ️ Note: Only vulnerabilities with available fixes (upgrades or patches) are counted toward thresholds.

Check Type Count (with fixes) Without fixes Threshold Result
🔴 Critical Severity 0 0 10 ✅ Passed
🟠 High Severity 0 0 25 ✅ Passed
🟡 Medium Severity 0 0 500 ✅ Passed
🔵 Low Severity 0 0 1000 ✅ Passed

⏱️ SLA Breach Summary

✅ No SLA breaches detected. All vulnerabilities are within acceptable time thresholds.

Severity Breaches (with fixes) Breaches (no fixes) SLA Threshold (with/no fixes) Status
🔴 Critical 0 0 15 / 30 days ✅ Passed
🟠 High 0 0 30 / 120 days ✅ Passed
🟡 Medium 0 0 90 / 365 days ✅ Passed
🔵 Low 0 0 180 / 365 days ✅ Passed

✅ BUILD PASSED - All security checks passed

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 1, 2026

🔒 Security Scan Results

ℹ️ Note: Only vulnerabilities with available fixes (upgrades or patches) are counted toward thresholds.

Check Type Count (with fixes) Without fixes Threshold Result
🔴 Critical Severity 0 0 10 ✅ Passed
🟠 High Severity 0 0 25 ✅ Passed
🟡 Medium Severity 0 0 500 ✅ Passed
🔵 Low Severity 0 0 1000 ✅ Passed

⏱️ SLA Breach Summary

✅ No SLA breaches detected. All vulnerabilities are within acceptable time thresholds.

Severity Breaches (with fixes) Breaches (no fixes) SLA Threshold (with/no fixes) Status
🔴 Critical 0 0 15 / 30 days ✅ Passed
🟠 High 0 0 30 / 120 days ✅ Passed
🟡 Medium 0 0 90 / 365 days ✅ Passed
🔵 Low 0 0 180 / 365 days ✅ Passed

✅ BUILD PASSED - All security checks passed

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 1, 2026

🔒 Security Scan Results

ℹ️ Note: Only vulnerabilities with available fixes (upgrades or patches) are counted toward thresholds.

Check Type Count (with fixes) Without fixes Threshold Result
🔴 Critical Severity 0 0 10 ✅ Passed
🟠 High Severity 0 0 25 ✅ Passed
🟡 Medium Severity 0 0 500 ✅ Passed
🔵 Low Severity 0 0 1000 ✅ Passed

⏱️ SLA Breach Summary

✅ No SLA breaches detected. All vulnerabilities are within acceptable time thresholds.

Severity Breaches (with fixes) Breaches (no fixes) SLA Threshold (with/no fixes) Status
🔴 Critical 0 0 15 / 30 days ✅ Passed
🟠 High 0 0 30 / 120 days ✅ Passed
🟡 Medium 0 0 90 / 365 days ✅ Passed
🔵 Low 0 0 180 / 365 days ✅ Passed

✅ BUILD PASSED - All security checks passed

@OMpawar-21 OMpawar-21 marked this pull request as ready for review June 1, 2026 07:29
@OMpawar-21 OMpawar-21 requested a review from a team as a code owner June 1, 2026 07:29
@OMpawar-21 OMpawar-21 merged commit 318c6d7 into enhc/beta Jun 1, 2026
7 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sunil-lakshman sunil-lakshman sunil-lakshman approved these changes

@reeshika-h reeshika-h reeshika-h approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@OMpawar-21 @sunil-lakshman @reeshika-h