Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
24 commits
Select commit Hold shift + click to select a range
601e177
SK-2841: Fixed the workflow
aadarsh-st May 25, 2026
62f8104
[AUTOMATED] Private Release 2.0.4-dev.601e177
aadarsh-st May 25, 2026
1cb4675
SK-2841: Changed code cov to true
aadarsh-st May 25, 2026
bd17593
Merge branch 'release/26.1.4' of https://github.com/skyflowapi/skyflo…
aadarsh-st May 25, 2026
659e6c4
[AUTOMATED] Private Release 2.0.4-dev.bd17593
aadarsh-st May 25, 2026
095f023
SK-2841: fixed workflow for codecov
aadarsh-st May 25, 2026
95575dd
Merge branch 'release/26.1.4' of https://github.com/skyflowapi/skyflo…
aadarsh-st May 25, 2026
1a22cc4
[AUTOMATED] Private Release 2.0.4-dev.95575dd
aadarsh-st May 25, 2026
f2de702
SK-2841: Added test cases
aadarsh-st May 25, 2026
28b08fc
Merge branch 'release/26.1.4' of https://github.com/skyflowapi/skyflo…
aadarsh-st May 25, 2026
17f8c34
[AUTOMATED] Private Release 2.0.4-dev.28b08fc
aadarsh-st May 25, 2026
dee6ce7
SK-2841: Updated coverage
aadarsh-st May 25, 2026
dff3e31
Merge branch 'release/26.1.4' of https://github.com/skyflowapi/skyflo…
aadarsh-st May 25, 2026
40170a0
[AUTOMATED] Private Release 2.0.4-dev.dff3e31
aadarsh-st May 25, 2026
c72a2c6
SK-2841: Updated detect test case
aadarsh-st May 25, 2026
9bc4bc1
[AUTOMATED] Private Release 2.0.4-dev.c72a2c6
aadarsh-st May 25, 2026
1346fd6
SK-2841: Increase coverage to 100%
aadarsh-st May 25, 2026
478d5b2
Merge branch 'release/26.1.4' of https://github.com/skyflowapi/skyflo…
aadarsh-st May 25, 2026
80cd85c
[AUTOMATED] Private Release 2.0.4-dev.478d5b2
aadarsh-st May 25, 2026
d7a2d9b
SK-2841: Updated coverage
aadarsh-st May 25, 2026
bd55be2
Merge branch 'release/26.1.4' of https://github.com/skyflowapi/skyflo…
aadarsh-st May 25, 2026
56cc918
[AUTOMATED] Private Release 2.0.4-dev.bd55be2
aadarsh-st May 25, 2026
d7aca66
SK-2841: Updated toekn coverage
aadarsh-st May 25, 2026
636dd41
[AUTOMATED] Private Release 2.0.4-dev.d7aca66
aadarsh-st May 25, 2026
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .github/workflows/CI.yml
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,6 @@ jobs:
ci-checks:
uses: ./.github/workflows/common-ci.yml
with:
upload-to-code-cov: false
upload-to-code-cov: true
secrets: inherit

2 changes: 1 addition & 1 deletion .github/workflows/common-ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ jobs:
run: npm run test

- name: Upload Code Coverage To Codecov
if: ${{ inputs.upload-to-code-cov == 'true' }}
if: ${{ inputs.upload-to-code-cov == true }}
uses: codecov/codecov-action@v2.1.0
with:
token: ${{ secrets.CODECOV_REPO_UPLOAD_TOKEN }}
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/common-release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ jobs:
- uses: actions/setup-node@v4
with:
node-version: '20.x'
# registry-url: "https://registry.npmjs.org"
registry-url: "https://registry.npmjs.org"

- name: Install Packages
run: npm install
Expand Down
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "skyflow-node",
"version": "2.1.0",
"version": "2.0.4-dev.d7aca66",
"description": "Skyflow SDK for Node.js",
"main": "./lib/index.js",
"module": "./lib/index.js",
Expand Down
2 changes: 1 addition & 1 deletion src/vault/controller/connections/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -143,7 +143,7 @@ class ConnectionController {

private async parseResponseBody(response: Response): Promise<any> {
const contentType =
response.headers.get(HTTP_HEADER.CONTENT_TYPE)?.toLowerCase() || "";
response.headers?.get(HTTP_HEADER.CONTENT_TYPE)?.toLowerCase() || "";

try {
if (contentType.includes(CONTENT_TYPE.APPLICATION_JSON)) {
Expand Down
8 changes: 4 additions & 4 deletions src/vault/controller/detect/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -434,8 +434,8 @@ class DetectController {

private parseDeidentifyTextResponse(records: DeidentifyStringResponse) {
return {
processedText: records.processed_text,
entities: records?.entities.map((entity: DetectedEntity) => ({
processedText: records?.processed_text,
entities: (records?.entities ?? []).map((entity: DetectedEntity) => ({
token: entity?.token,
value: entity?.value,
textIndex: {
Expand All @@ -449,8 +449,8 @@ class DetectController {
entity: entity?.entity_type,
scores: entity?.entity_scores,
})),
wordCount: records.word_count,
charCount: records.character_count,
wordCount: records?.word_count,
charCount: records?.character_count,
errors: null,
};
}
Expand Down
26 changes: 26 additions & 0 deletions test/service-account/client.test.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,26 @@
jest.mock('../../src/ _generated_/rest/api/resources/authentication/client/Client', () => ({
Authentication: jest.fn().mockImplementation((config) => ({ config })),
}));

const Client = require('../../src/service-account/client').default;
const {
Authentication,
} = require('../../src/ _generated_/rest/api/resources/authentication/client/Client');

describe('Service Account Client', () => {
beforeEach(() => {
Authentication.mockClear();
});

test('initializes Authentication with the token URI as baseUrl', () => {
const tokenUri = 'https://example.com/oauth/token';
const client = new Client(tokenUri);

expect(Authentication).toHaveBeenCalledWith({
baseUrl: tokenUri,
token: '',
});
expect(client.authApi).toBeDefined();
expect(client.authApi.config.baseUrl).toBe(tokenUri);
});
});
Loading
Loading