Skip to content

Commit cfc0dda

Browse files
committed
fix(types): add explicit type parameters to handleApiCall calls
Add explicit operation type parameters to handleApiCall calls across all fetch functions to resolve TypeScript type inference issues with SDK v3. Changes: - Analytics: getRepoAnalytics, getOrgAnalytics - Organization: listOrganizations, getOrgLicensePolicy, getQuota, getOrgSecurityPolicy, searchDependencies - Package: batchPackageFetch - Repository: createRepository, deleteRepository, updateRepository, getRepository - Scan: deleteFullScan, getFullScanMetadata, getReportSupportedFiles, getOrgFullScan (for streamFullScan)
1 parent a0fc5a0 commit cfc0dda

16 files changed

+74
-39
lines changed

packages/cli/src/commands/analytics/fetch-org-analytics.mts

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,10 @@ export async function fetchOrgAnalyticsData(
2424
}
2525
const sockSdk = sockSdkCResult.data
2626

27-
return await handleApiCall(sockSdk.getOrgAnalytics(time.toString()), {
28-
description: 'analytics data',
29-
})
27+
return await handleApiCall<'getOrgAnalytics'>(
28+
sockSdk.getOrgAnalytics(time.toString()),
29+
{
30+
description: 'analytics data',
31+
},
32+
)
3033
}

packages/cli/src/commands/analytics/fetch-repo-analytics.mts

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,10 @@ export async function fetchRepoAnalyticsData(
2525
}
2626
const sockSdk = sockSdkCResult.data
2727

28-
return await handleApiCall(sockSdk.getRepoAnalytics(repo, time.toString()), {
29-
description: 'analytics data',
30-
})
28+
return await handleApiCall<'getRepoAnalytics'>(
29+
sockSdk.getRepoAnalytics(repo, time.toString()),
30+
{
31+
description: 'analytics data',
32+
},
33+
)
3134
}

packages/cli/src/commands/organization/fetch-dependencies.mts

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,10 @@ export async function fetchDependencies(
3434
...config,
3535
} as FetchDependenciesConfig
3636

37-
return await handleApiCall(sockSdk.searchDependencies({ limit, offset }), {
38-
description: 'organization dependencies',
39-
})
37+
return await handleApiCall<'searchDependencies'>(
38+
sockSdk.searchDependencies({ limit, offset }),
39+
{
40+
description: 'organization dependencies',
41+
},
42+
)
4043
}

packages/cli/src/commands/organization/fetch-license-policy.mts

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,10 @@ export async function fetchLicensePolicy(
2424
}
2525
const sockSdk = sockSdkCResult.data
2626

27-
return await handleApiCall(sockSdk.getOrgLicensePolicy(orgSlug), {
28-
description: 'organization license policy',
29-
})
27+
return await handleApiCall<'getOrgLicensePolicy'>(
28+
sockSdk.getOrgLicensePolicy(orgSlug),
29+
{
30+
description: 'organization license policy',
31+
},
32+
)
3033
}

packages/cli/src/commands/organization/fetch-organization-list.mts

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -47,9 +47,12 @@ export async function fetchOrganization(
4747
sockSdk = sockSdkCResult.data
4848
}
4949

50-
const orgsCResult = await handleApiCall(sockSdk.listOrganizations(), {
51-
description,
52-
})
50+
const orgsCResult = await handleApiCall<'listOrganizations'>(
51+
sockSdk.listOrganizations(),
52+
{
53+
description,
54+
},
55+
)
5356
if (!orgsCResult.ok) {
5457
return orgsCResult
5558
}

packages/cli/src/commands/organization/fetch-quota.mts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,5 +20,7 @@ export async function fetchQuota(
2020
}
2121
const sockSdk = sockSdkCResult.data
2222

23-
return await handleApiCall(sockSdk.getQuota(), { description: 'token quota' })
23+
return await handleApiCall<'getQuota'>(sockSdk.getQuota(), {
24+
description: 'token quota',
25+
})
2426
}

packages/cli/src/commands/organization/fetch-security-policy.mts

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,10 @@ export async function fetchSecurityPolicy(
2424
}
2525
const sockSdk = sockSdkCResult.data
2626

27-
return await handleApiCall(sockSdk.getOrgSecurityPolicy(orgSlug), {
28-
description: 'organization security policy',
29-
})
27+
return await handleApiCall<'getOrgSecurityPolicy'>(
28+
sockSdk.getOrgSecurityPolicy(orgSlug),
29+
{
30+
description: 'organization security policy',
31+
},
32+
)
3033
}

packages/cli/src/commands/package/fetch-purls-shallow-score.mts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ export async function fetchPurlsShallowScore(
3535
`Requesting shallow score data for ${purls.length} package urls (purl): ${displayPurls}`,
3636
)
3737

38-
const batchPackageCResult = await handleApiCall(
38+
const batchPackageCResult = await handleApiCall<'batchPackageFetch'>(
3939
sockSdk.batchPackageFetch(
4040
{ components: purls.map(purl => ({ purl })) },
4141
{

packages/cli/src/commands/repository/fetch-create-repo.mts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@ export async function fetchCreateRepo(
4242
}
4343
const sockSdk = sockSdkCResult.data
4444

45-
return await handleApiCall(
45+
return await handleApiCall<'createRepository'>(
4646
sockSdk.createRepository(orgSlug, {
4747
default_branch: defaultBranch,
4848
description,

packages/cli/src/commands/repository/fetch-delete-repo.mts

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,10 @@ export async function fetchDeleteRepo(
2525
}
2626
const sockSdk = sockSdkCResult.data
2727

28-
return await handleApiCall(sockSdk.deleteRepository(orgSlug, repoName), {
29-
description: 'to delete a repository',
30-
})
28+
return await handleApiCall<'deleteRepository'>(
29+
sockSdk.deleteRepository(orgSlug, repoName),
30+
{
31+
description: 'to delete a repository',
32+
},
33+
)
3134
}

0 commit comments

Comments
 (0)