Skip to content

SK-2812: Public interface cleanup#306

Open
aadarsh-st wants to merge 42 commits into
release/26.1.4from
aadarsh-st/SK-2812-pulic-interface-cleanup
Open

SK-2812: Public interface cleanup#306
aadarsh-st wants to merge 42 commits into
release/26.1.4from
aadarsh-st/SK-2812-pulic-interface-cleanup

Conversation

@aadarsh-st
Copy link
Copy Markdown
Collaborator

Summary

  • Renamed public fields to camelCase: clientIDclientId, keyIDkeyId,
    tokenURItokenUri, roleIDsroleIds, request_IDrequestId,
    downloadURLdownloadUrl, start_padding/stop_paddingstartPadding/stopPadding
  • FileUploadRequest: removed skyflowId from constructor; set via FileUploadOptions.setSkyflowId()
  • insertedFields always returns as array (never null)
  • DeleteResponse.deletedIds is now always Array<string> (non-optional)
  • Fail-fast SkyflowError thrown for null/undefined/"" field values in insert records before any API call; 0, false, 0.0 remain valid
  • All response objects (Insert, Update, Get, Delete, Query, Tokenize, DeidentifyText, DeidentifyFile) always include errors field — null when no errors, never absent
  • CHANGELOG updated with migration notes and upgrade diff examples

skyflow-himanshu and others added 30 commits February 17, 2026 13:58
…for-handling-deidentify-file-response

SK-1964: add sample for handling deidentify file response in async
… Access

The Node SDK already supports ctx as string | Record<string, any> for
bearer tokens and signed data tokens (shipped in SK-2391/PR #266), but
the README and samples only showed string usage. Add documentation and
examples for JSON object context to support Conditional Data Access
CEL expressions like request.context.role == 'admin'.

Refs DOCU-1441
Co-Authored-By: Claude <noreply@anthropic.com>
SK-2771: fix deidentifyFile crash when waitTime is exceeded before processing completes
…hanges

SK-2770: publish v1.x releases under the v1 dist-tag to preserve latest as V2
…alisation

SK-2766 reduce client initialisation
…-data-access-docs

DOCU-1441: Document JSON object context support for Conditional Data Access
@github-actions
Copy link
Copy Markdown

Gitleaks Findings: No secrets detected. Safe to proceed!

@github-actions
Copy link
Copy Markdown

Semgrep Findings: Issues with Error level severity are found (Error is Highest severity in Semgrep), Please resolve the issues before merging.

@github-actions
Copy link
Copy Markdown

Gitleaks Findings: No secrets detected. Safe to proceed!

@github-actions
Copy link
Copy Markdown

Semgrep Findings: Issues with Error level severity are found (Error is Highest severity in Semgrep), Please resolve the issues before merging.

@github-actions
Copy link
Copy Markdown

Gitleaks Findings: No secrets detected. Safe to proceed!

@github-actions
Copy link
Copy Markdown

Semgrep Findings: Issues with Error level severity are found (Error is Highest severity in Semgrep), Please resolve the issues before merging.

@github-actions
Copy link
Copy Markdown

Gitleaks Findings: No secrets detected. Safe to proceed!

@github-actions
Copy link
Copy Markdown

Semgrep Findings: Issues with Error level severity are found (Error is Highest severity in Semgrep), Please resolve the issues before merging.

@aadarsh-st aadarsh-st changed the base branch from main to release/26.1.4 May 13, 2026 06:48
@aadarsh-st aadarsh-st changed the base branch from release/26.1.4 to main May 13, 2026 06:56
@aadarsh-st aadarsh-st changed the base branch from main to release/26.1.4 May 13, 2026 08:06
aadarsh-st and others added 7 commits May 13, 2026 13:36
…package-lock.json

75 resolved URLs pointed to prekarilabs.jfrog.io causing E401 in CI npm install.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…o aadarsh-st/SK-2812-pulic-interface-cleanup
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.

6 participants