Skip to content

Commit

Permalink
deps(elasticsearch): TypeScript fixes for ElasticSearch 8.0 update
Browse files Browse the repository at this point in the history
  • Loading branch information
nellh committed Dec 6, 2023
1 parent 5f70128 commit 8e2aea5
Show file tree
Hide file tree
Showing 12 changed files with 30 additions and 36 deletions.
2 changes: 1 addition & 1 deletion packages/openneuro-app/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -70,5 +70,5 @@
"publishConfig": {
"access": "public"
},
"gitHead": "fc6317d28f5ccb4e61d41d032640efc59607e7fe"
"gitHead": "5f7012815a4edd0be48ee30d1de2bad7d1268f63"
}
2 changes: 1 addition & 1 deletion packages/openneuro-cli/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -48,5 +48,5 @@
"publishConfig": {
"access": "public"
},
"gitHead": "fc6317d28f5ccb4e61d41d032640efc59607e7fe"
"gitHead": "5f7012815a4edd0be48ee30d1de2bad7d1268f63"
}
2 changes: 1 addition & 1 deletion packages/openneuro-client/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -35,5 +35,5 @@
"publishConfig": {
"access": "public"
},
"gitHead": "fc6317d28f5ccb4e61d41d032640efc59607e7fe"
"gitHead": "5f7012815a4edd0be48ee30d1de2bad7d1268f63"
}
2 changes: 1 addition & 1 deletion packages/openneuro-components/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -51,5 +51,5 @@
"publishConfig": {
"access": "public"
},
"gitHead": "fc6317d28f5ccb4e61d41d032640efc59607e7fe"
"gitHead": "5f7012815a4edd0be48ee30d1de2bad7d1268f63"
}
2 changes: 1 addition & 1 deletion packages/openneuro-indexer/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -34,5 +34,5 @@
"publishConfig": {
"access": "public"
},
"gitHead": "fc6317d28f5ccb4e61d41d032640efc59607e7fe"
"gitHead": "5f7012815a4edd0be48ee30d1de2bad7d1268f63"
}
2 changes: 1 addition & 1 deletion packages/openneuro-indexer/src/createIndices.ts
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ export const createIndices = async (elasticClient: Client): Promise<void> => {
const exists = await elasticClient.indices.exists({
index: DatasetsIndex.name,
})
if (exists.statusCode !== 200) {
if (exists) {
await elasticClient.indices.create({
index: DatasetsIndex.name,
body: {
Expand Down
8 changes: 2 additions & 6 deletions packages/openneuro-indexer/src/indexDatasets.ts
Original file line number Diff line number Diff line change
@@ -1,8 +1,4 @@
import {
// Interface of the generic API response
ApiResponse,
Client as ElasticClient,
} from "@elastic/elasticsearch"
import { Client as ElasticClient } from "@elastic/elasticsearch"
import {
DatasetQueryResult,
DatasetsIndex,
Expand All @@ -14,7 +10,7 @@ import {
*/
export const aliasDatasetsIndex = (
elasticClient: ElasticClient,
): Promise<ApiResponse> =>
) =>
elasticClient.indices.updateAliases({
body: {
actions: [{ add: { index: DatasetsIndex.name, alias: "datasets" } }],
Expand Down
2 changes: 1 addition & 1 deletion packages/openneuro-search/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -29,5 +29,5 @@
"publishConfig": {
"access": "public"
},
"gitHead": "fc6317d28f5ccb4e61d41d032640efc59607e7fe"
"gitHead": "5f7012815a4edd0be48ee30d1de2bad7d1268f63"
}
16 changes: 5 additions & 11 deletions packages/openneuro-search/src/query.ts
Original file line number Diff line number Diff line change
@@ -1,10 +1,4 @@
import {
// Interface of the generic API response
ApiResponse,
Client as ElasticClient,
// Object that contains the type definitions of every API method
RequestParams,
} from "@elastic/elasticsearch"
import { Client as ElasticClient } from "@elastic/elasticsearch"
import { ApolloClient, gql, NormalizedCacheObject } from "@apollo/client"
import { DatasetsIndex } from "./indexes/datasets"

Expand Down Expand Up @@ -130,8 +124,8 @@ export interface DatasetQueryResult {
*/
export function extractDatasetDocument(
datasetObj: DatasetQueryResult,
): RequestParams.Index {
const dataset: RequestParams.Index = {
) {
const dataset = {
index: DatasetsIndex.name,
id: datasetObj.id,
body: {
Expand Down Expand Up @@ -164,10 +158,10 @@ export function queryForIndex(
export async function indexDataset(
elasticClient: ElasticClient,
datasetObj: DatasetQueryResult,
): Promise<ApiResponse> {
) {
try {
console.log(`Indexing "${datasetObj.id}"`)
const response: ApiResponse = await elasticClient.index(
const response = await elasticClient.index(
extractDatasetDocument(datasetObj),
)
return response
Expand Down
2 changes: 1 addition & 1 deletion packages/openneuro-server/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -85,5 +85,5 @@
"publishConfig": {
"access": "public"
},
"gitHead": "fc6317d28f5ccb4e61d41d032640efc59607e7fe"
"gitHead": "5f7012815a4edd0be48ee30d1de2bad7d1268f63"
}
Original file line number Diff line number Diff line change
Expand Up @@ -102,8 +102,10 @@ export const datasetSearchConnection = async (
index: elasticIndex,
size: first,
q: `${q} AND public:true`,
// @ts-expect-error Limitations of TypeScript definitions from @elastic/elasticsearch client
body: requestBody,
})
// @ts-expect-error Limitations of TypeScript definitions from @elastic/elasticsearch client
return elasticRelayConnection(result, searchId, first)
}

Expand Down Expand Up @@ -241,9 +243,11 @@ export const advancedDatasetSearchConnection = async (
const result = await elasticClient.search({
index: elasticIndex,
size: first,
// @ts-expect-error Limitations of TypeScript definitions from @elastic/elasticsearch client
body: requestBody,
})
return elasticRelayConnection(
// @ts-expect-error Limitations of TypeScript definitions from @elastic/elasticsearch client
result,
searchId,
first,
Expand Down
22 changes: 11 additions & 11 deletions yarn.lock
Original file line number Diff line number Diff line change
Expand Up @@ -5405,8 +5405,8 @@ __metadata:
"@emotion/react": 11.11.1
"@emotion/styled": 11.11.0
"@niivue/niivue": 0.34.0
"@openneuro/client": ^4.21.0-alpha.4
"@openneuro/components": ^4.21.0-alpha.4
"@openneuro/client": ^4.21.0-alpha.5
"@openneuro/components": ^4.21.0-alpha.5
"@tanstack/react-table": ^8.9.3
"@testing-library/jest-dom": 6.1.3
"@testing-library/react": ^11.1.0
Expand Down Expand Up @@ -5457,7 +5457,7 @@ __metadata:
resolution: "@openneuro/cli@workspace:packages/openneuro-cli"
dependencies:
"@apollo/client": 3.7.2
"@openneuro/client": ^4.21.0-alpha.4
"@openneuro/client": ^4.21.0-alpha.5
"@types/mkdirp": 1.0.2
"@types/node": 18.11.9
bids-validator: 1.10.0
Expand All @@ -5482,14 +5482,14 @@ __metadata:
languageName: unknown
linkType: soft

"@openneuro/client@^4.21.0-alpha.4, @openneuro/client@workspace:packages/openneuro-client":
"@openneuro/client@^4.21.0-alpha.5, @openneuro/client@workspace:packages/openneuro-client":
version: 0.0.0-use.local
resolution: "@openneuro/client@workspace:packages/openneuro-client"
dependencies:
"@apollo/client": 3.7.2
"@graphql-tools/mock": ^9.0.0
"@graphql-tools/schema": ^10.0.0
"@openneuro/server": ^4.21.0-alpha.4
"@openneuro/server": ^4.21.0-alpha.5
core-js: ^3.10.1
crypto-hash: ^1.3.0
graphql: 16.8.1
Expand All @@ -5501,7 +5501,7 @@ __metadata:
languageName: unknown
linkType: soft

"@openneuro/components@^4.21.0-alpha.4, @openneuro/components@workspace:packages/openneuro-components":
"@openneuro/components@^4.21.0-alpha.5, @openneuro/components@workspace:packages/openneuro-components":
version: 0.0.0-use.local
resolution: "@openneuro/components@workspace:packages/openneuro-components"
dependencies:
Expand Down Expand Up @@ -5543,8 +5543,8 @@ __metadata:
dependencies:
"@apollo/client": 3.7.2
"@elastic/elasticsearch": ^8.10.0
"@openneuro/client": ^4.21.0-alpha.4
"@openneuro/search": ^4.21.0-alpha.4
"@openneuro/client": ^4.21.0-alpha.5
"@openneuro/search": ^4.21.0-alpha.5
"@types/jsonwebtoken": ^8
"@types/node": 18.11.9
"@types/tsc-watch": ^4
Expand All @@ -5558,7 +5558,7 @@ __metadata:
languageName: unknown
linkType: soft

"@openneuro/search@^4.21.0-alpha.4, @openneuro/search@workspace:packages/openneuro-search":
"@openneuro/search@^4.21.0-alpha.5, @openneuro/search@workspace:packages/openneuro-search":
version: 0.0.0-use.local
resolution: "@openneuro/search@workspace:packages/openneuro-search"
dependencies:
Expand All @@ -5573,7 +5573,7 @@ __metadata:
languageName: unknown
linkType: soft

"@openneuro/server@^4.21.0-alpha.4, @openneuro/server@workspace:packages/openneuro-server":
"@openneuro/server@^4.21.0-alpha.5, @openneuro/server@workspace:packages/openneuro-server":
version: 0.0.0-use.local
resolution: "@openneuro/server@workspace:packages/openneuro-server"
dependencies:
Expand All @@ -5583,7 +5583,7 @@ __metadata:
"@elastic/elasticsearch": ^8.10.0
"@graphql-tools/schema": ^10.0.0
"@keyv/redis": ^2.7.0
"@openneuro/search": ^4.21.0-alpha.4
"@openneuro/search": ^4.21.0-alpha.5
"@passport-next/passport-google-oauth2": ^1.0.0
"@sentry/node": ^4.5.3
"@types/cors": ^2
Expand Down

0 comments on commit 8e2aea5

Please sign in to comment.