Skip to content

Fix Reconcile history - #253

Merged
cs-raj merged 392 commits into
developmentfrom
fix/reconcile-history
Aug 11, 2026
Merged

Fix Reconcile history#253
cs-raj merged 392 commits into
developmentfrom
fix/reconcile-history

Conversation

@cs-raj

@cs-raj cs-raj commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

No description provided.

harshithad0703 and others added 30 commits January 17, 2025 14:17
…erage

improved code coverage by adding test case
DX -2025 License updated and version bump
fix: url change with host for live preview
DX | 24-01-2025 | Release | Hotfix
* Update node version to 22

* updated publish workflow file
reeshika-h and others added 23 commits July 31, 2026 11:01
…andling

fix: classify request timeouts distinctly instead of unknown error
Merging development (with the timeout classification fix) surfaced an
interaction: the classification test didn't set retryCondition, so it now
picks up the new default retryCondition (which retries ECONNABORTED),
taking the retry path instead of the classification path it was meant to
test. Same fix as already applied to the other pre-existing ECONNABORTED
test - explicitly disable retryCondition to isolate the behavior under test.

DX-9991
…rrors

fix: retry transient network errors by default
…ction-pooling

Default to keep-alive agents in Node environments for better performance
Bundles the three DX-9991 fixes: timeout classification, default retry
for transient network errors, and default keep-alive connection agents.

DX-9991
Three environments broke when keep-alive agents were introduced using
require('http') and require('https'):

1. Native ESM (ReferenceError: require is not defined)
   - Added typeof require === 'undefined' guard in createKeepAliveAgent()
   - Falls back to false (pre-1.5.0 behaviour) in ESM environments

2. Turbopack/webpack (expression is too dynamic)
   - Replaced require(moduleName) variable arg with static string literals
   - require('http') / require('https') are statically analysable by bundlers

3. UMD webpack build (Can't resolve 'http')
   - Added externals for http and https in config/webpack.config.js
   - UMD wrapper resolves them at runtime in Node; isNodeEnvironment guard
     prevents execution in browser where they are not available

Tests added:
- test/esm-compat.mjs: native ESM integration test (run after build)
- test/contentstack-core.node-agent.spec.ts: Jest wrapper spawning esm-compat.mjs
- test/esm-exports.spec.ts: static analysis asserting no dynamic require(variable)

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
… fix

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
fix: Resolve require() incompatibility in ESM and UMD builds
fix false positive hardcoded secrets issues
@cs-raj
cs-raj requested a review from a team as a code owner August 11, 2026 10:27
@snyk-io

snyk-io Bot commented Aug 11, 2026

Copy link
Copy Markdown

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues
Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@github-actions

Copy link
Copy Markdown

🔒 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

@cs-raj
cs-raj merged commit c99f813 into development Aug 11, 2026
10 checks passed
@cs-raj
cs-raj deleted the fix/reconcile-history branch August 11, 2026 10:59
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.

9 participants