Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Research local incremental rebuilds speed improvements #1951

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft
Show file tree
Hide file tree
Changes from all commits
Commits
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 packages/blockchain-data/sdk/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@
"rollup": "^3.17.2",
"ts-mockito": "^2.6.1",
"typechain": "^8.1.1",
"typescript": "^4.9.5"
"typescript": "^5.5.2"
},
"homepage": "https://github.com/immutable/ts-immutable-sdk#readme",
"license": "Apache-2.0",
Expand Down
2 changes: 1 addition & 1 deletion packages/checkout/sdk-sample-app/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@
"react-app-rewired": "^2.2.1",
"react-scripts": "5.0.1",
"stream-browserify": "^3.0.0",
"typescript": "^4.9.5",
"typescript": "^5.5.2",
"web-vitals": "^2.1.4"
},
"private": true,
Expand Down
2 changes: 1 addition & 1 deletion packages/checkout/sdk/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@
"text-encoding": "^0.7.0",
"typedoc": "^0.24.8",
"typedoc-plugin-markdown": "^3.15.2",
"typescript": "^4.9.5"
"typescript": "^5.5.2"
},
"homepage": "https://github.com/immutable/ts-immutable-sdk",
"keywords": [
Expand Down
2 changes: 1 addition & 1 deletion packages/checkout/widgets-lib/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,7 @@
"rollup-plugin-svg": "^2.0.0",
"rollup-plugin-visualizer": "^5.12.0",
"ts-jest": "^29.1.0",
"typescript": "^4.9.5",
"typescript": "^5.5.2",
"web-vitals": "^2.1.4"
},
"homepage": "https://github.com/immutable/ts-immutable-sdk",
Expand Down
2 changes: 1 addition & 1 deletion packages/checkout/widgets-sample-app/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@
"@types/react": "^18.0.28",
"@types/react-dom": "^18.0.11",
"@vitejs/plugin-react": "^4.2.0",
"typescript": "^4.9.5",
"typescript": "^5.5.2",
"vite": "^5.0.12",
"vite-plugin-node-polyfills": "^0.16.0",
"web-vitals": "^2.1.4"
Expand Down
2 changes: 1 addition & 1 deletion packages/config/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@
"prettier": "^2.8.7",
"rollup": "^3.17.2",
"ts-node": "^10.9.1",
"typescript": "^4.9.5"
"typescript": "^5.5.2"
},
"engines": {
"node": ">=20.11.0"
Expand Down
2 changes: 1 addition & 1 deletion packages/internal/bridge/bridge-sample-app/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
"eslint": "^8.40.0",
"eslint-config-next": "13.3.1",
"ts-node": "^10.9.1",
"typescript": "^4.9.5"
"typescript": "^5.5.2"
},
"private": true,
"scripts": {
Expand Down
2 changes: 1 addition & 1 deletion packages/internal/bridge/sdk/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@
"rollup": "^3.17.2",
"ts-node": "^10.9.1",
"typechain": "^8.1.1",
"typescript": "^4.9.5"
"typescript": "^5.5.2"
},
"engines": {
"node": ">=20.11.0"
Expand Down
2 changes: 1 addition & 1 deletion packages/internal/cryptofiat/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@
"prettier": "^2.8.7",
"rollup": "^3.17.2",
"ts-node": "^10.9.1",
"typescript": "^4.9.5"
"typescript": "^5.5.2"
},
"engines": {
"node": ">=20.11.0"
Expand Down
2 changes: 1 addition & 1 deletion packages/internal/dex/sdk-sample-app/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@
"react": "^18.2.0",
"react-dom": "^18.2.0",
"tailwindcss": "3.3.2",
"typescript": "^4.9.5"
"typescript": "^5.5.2"
},
"private": true,
"scripts": {
Expand Down
2 changes: 1 addition & 1 deletion packages/internal/dex/sdk/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@
"rollup": "^3.17.2",
"ts-node": "^10.9.1",
"typechain": "^8.1.1",
"typescript": "^4.9.5"
"typescript": "^5.5.2"
},
"engines": {
"node": ">=20.11.0"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
"eslint": "^8.40.0",
"eslint-config-next": "13.3.1",
"ts-node": "^10.9.1",
"typescript": "^4.9.5"
"typescript": "^5.5.2"
},
"private": true,
"scripts": {
Expand Down
2 changes: 1 addition & 1 deletion packages/internal/factory/sdk/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@
"rollup": "^3.17.2",
"ts-node": "^10.9.1",
"typechain": "^8.1.1",
"typescript": "^4.9.5"
"typescript": "^5.5.2"
},
"engines": {
"node": ">=20.11.0"
Expand Down
7 changes: 3 additions & 4 deletions packages/internal/generated-clients/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -7,8 +7,7 @@
"devDependencies": {
"@openapitools/openapi-generator-cli": "^2.13.4",
"jest": "^29.4.3",
"rollup": "^3.17.2",
"typescript": "^4.9.5"
"typescript": "^5.5.2"
},
"engines": {
"node": ">=20.11.0"
Expand All @@ -22,8 +21,8 @@
"private": true,
"repository": "immutable/ts-immutable-sdk.git",
"scripts": {
"build": "NODE_ENV=production rollup --config rollup.config.js",
"dev": "rollup --config rollup.config.js -w",
"build": "tsc --build",
"dev": "tsc --watch",
"prepare": "wsrun -r build",
"test": "jest",
"typecheck": "tsc --noEmit --jsx preserve",
Expand Down
12 changes: 0 additions & 12 deletions packages/internal/generated-clients/rollup.config.js

This file was deleted.

Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ import type { Configuration } from './configuration';
import type { AxiosPromise, AxiosInstance, AxiosRequestConfig } from 'axios';
import globalAxios from 'axios';

export const BASE_PATH = "https://api.sandbox.immutable.com".replace(/\/+$/, "");
export const BASE_PATH: string = "https://api.sandbox.immutable.com".replace(/\/+$/, "");

/**
*
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ export const DUMMY_BASE_URL = 'https://example.com'
* @throws {RequiredError}
* @export
*/
export const assertParamExists = function (functionName: string, paramName: string, paramValue: unknown) {
export const assertParamExists = function (functionName: string, paramName: string, paramValue: unknown): void {
if (paramValue === null || paramValue === undefined) {
throw new RequiredError(paramName, `Required parameter ${paramName} was null or undefined when calling ${functionName}.`);
}
Expand All @@ -39,7 +39,7 @@ export const assertParamExists = function (functionName: string, paramName: stri
*
* @export
*/
export const setApiKeyToObject = async function (object: any, keyParamName: string, configuration?: Configuration) {
export const setApiKeyToObject = async function (object: any, keyParamName: string, configuration?: Configuration): Promise<void> {
if (configuration && configuration.apiKey) {
const localVarApiKeyValue = typeof configuration.apiKey === 'function'
? await configuration.apiKey(keyParamName)
Expand All @@ -52,7 +52,7 @@ export const setApiKeyToObject = async function (object: any, keyParamName: stri
*
* @export
*/
export const setBasicAuthToObject = function (object: any, configuration?: Configuration) {
export const setBasicAuthToObject = function (object: any, configuration?: Configuration): void {
if (configuration && (configuration.username || configuration.password)) {
object["auth"] = { username: configuration.username, password: configuration.password };
}
Expand All @@ -62,7 +62,7 @@ export const setBasicAuthToObject = function (object: any, configuration?: Confi
*
* @export
*/
export const setBearerAuthToObject = async function (object: any, configuration?: Configuration) {
export const setBearerAuthToObject = async function (object: any, configuration?: Configuration): Promise<void> {
if (configuration && configuration.accessToken) {
const accessToken = typeof configuration.accessToken === 'function'
? await configuration.accessToken()
Expand All @@ -75,7 +75,7 @@ export const setBearerAuthToObject = async function (object: any, configuration?
*
* @export
*/
export const setOAuthToObject = async function (object: any, name: string, scopes: string[], configuration?: Configuration) {
export const setOAuthToObject = async function (object: any, name: string, scopes: string[], configuration?: Configuration): Promise<void> {
if (configuration && configuration.accessToken) {
const localVarAccessTokenValue = typeof configuration.accessToken === 'function'
? await configuration.accessToken(name, scopes)
Expand Down Expand Up @@ -110,7 +110,7 @@ function setFlattenedQueryParams(urlSearchParams: URLSearchParams, parameter: an
*
* @export
*/
export const setSearchParams = function (url: URL, ...objects: any[]) {
export const setSearchParams = function (url: URL, ...objects: any[]): void {
const searchParams = new URLSearchParams(url.search);
setFlattenedQueryParams(searchParams, objects);
url.search = searchParams.toString();
Expand All @@ -120,7 +120,7 @@ export const setSearchParams = function (url: URL, ...objects: any[]) {
*
* @export
*/
export const serializeDataIfNeeded = function (value: any, requestOptions: any, configuration?: Configuration) {
export const serializeDataIfNeeded = function (value: any, requestOptions: any, configuration?: Configuration): any {
const nonString = typeof value !== 'string';
const needsSerialization = nonString && configuration && configuration.isJsonMime
? configuration.isJsonMime(requestOptions.headers['Content-Type'])
Expand All @@ -134,7 +134,7 @@ export const serializeDataIfNeeded = function (value: any, requestOptions: any,
*
* @export
*/
export const toPathString = function (url: URL) {
export const toPathString = function (url: URL): string {
return url.pathname + url.search + url.hash
}

Expand All @@ -143,7 +143,7 @@ export const toPathString = function (url: URL) {
* @export
*/
export const createRequestFunction = function (axiosArgs: RequestArgs, globalAxios: AxiosInstance, BASE_PATH: string, configuration?: Configuration) {
return <T = unknown, R = AxiosResponse<T>>(axios: AxiosInstance = globalAxios, basePath: string = BASE_PATH) => {
return <T = unknown, R = AxiosResponse<T>>(axios: AxiosInstance = globalAxios, basePath: string = BASE_PATH): Promise<R> => {
const axiosRequestArgs = {...axiosArgs.options, url: (configuration?.basePath || axios.defaults.baseURL || basePath) + axiosArgs.url};
return axios.request<T, R>(axiosRequestArgs);
};
Expand Down
4 changes: 2 additions & 2 deletions packages/internal/generated-clients/src/config/config.ts
Original file line number Diff line number Diff line change
Expand Up @@ -40,10 +40,10 @@ export const createConfig = ({
* @returns an ImmutableAPIConfiguration
*/
export const imxApiConfig = {
getProduction: () => createConfig({
getProduction: (): APIConfiguration => createConfig({
basePath: 'https://api.x.immutable.com',
}),
getSandbox: () => createConfig({
getSandbox: (): APIConfiguration => createConfig({
basePath: 'https://api.sandbox.x.immutable.com',
}),
};
Expand Down
2 changes: 1 addition & 1 deletion packages/internal/generated-clients/src/imx/base.ts
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ import { Configuration } from "./configuration";
// @ts-ignore
import globalAxios, { AxiosPromise, AxiosInstance, AxiosRequestConfig } from 'axios';

export const BASE_PATH = "https://api.sandbox.x.immutable.com".replace(/\/+$/, "");
export const BASE_PATH: string = "https://api.sandbox.x.immutable.com".replace(/\/+$/, "");

/**
*
Expand Down
18 changes: 9 additions & 9 deletions packages/internal/generated-clients/src/imx/common.ts
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ export const DUMMY_BASE_URL = 'https://example.com'
* @throws {RequiredError}
* @export
*/
export const assertParamExists = function (functionName: string, paramName: string, paramValue: unknown) {
export const assertParamExists = function (functionName: string, paramName: string, paramValue: unknown): void {
if (paramValue === null || paramValue === undefined) {
throw new RequiredError(paramName, `Required parameter ${paramName} was null or undefined when calling ${functionName}.`);
}
Expand All @@ -38,7 +38,7 @@ export const assertParamExists = function (functionName: string, paramName: stri
*
* @export
*/
export const setApiKeyToObject = async function (object: any, keyParamName: string, configuration?: Configuration) {
export const setApiKeyToObject = async function (object: any, keyParamName: string, configuration?: Configuration): Promise<void> {
if (configuration && configuration.apiKey) {
const localVarApiKeyValue = typeof configuration.apiKey === 'function'
? await configuration.apiKey(keyParamName)
Expand All @@ -51,7 +51,7 @@ export const setApiKeyToObject = async function (object: any, keyParamName: stri
*
* @export
*/
export const setBasicAuthToObject = function (object: any, configuration?: Configuration) {
export const setBasicAuthToObject = function (object: any, configuration?: Configuration): void {
if (configuration && (configuration.username || configuration.password)) {
object["auth"] = { username: configuration.username, password: configuration.password };
}
Expand All @@ -61,7 +61,7 @@ export const setBasicAuthToObject = function (object: any, configuration?: Confi
*
* @export
*/
export const setBearerAuthToObject = async function (object: any, configuration?: Configuration) {
export const setBearerAuthToObject = async function (object: any, configuration?: Configuration): Promise<void> {
if (configuration && configuration.accessToken) {
const accessToken = typeof configuration.accessToken === 'function'
? await configuration.accessToken()
Expand All @@ -74,7 +74,7 @@ export const setBearerAuthToObject = async function (object: any, configuration?
*
* @export
*/
export const setOAuthToObject = async function (object: any, name: string, scopes: string[], configuration?: Configuration) {
export const setOAuthToObject = async function (object: any, name: string, scopes: string[], configuration?: Configuration): Promise<void> {
if (configuration && configuration.accessToken) {
const localVarAccessTokenValue = typeof configuration.accessToken === 'function'
? await configuration.accessToken(name, scopes)
Expand Down Expand Up @@ -108,7 +108,7 @@ function setFlattenedQueryParams(urlSearchParams: URLSearchParams, parameter: an
*
* @export
*/
export const setSearchParams = function (url: URL, ...objects: any[]) {
export const setSearchParams = function (url: URL, ...objects: any[]): void {
const searchParams = new URLSearchParams(url.search);
setFlattenedQueryParams(searchParams, objects);
url.search = searchParams.toString();
Expand All @@ -118,7 +118,7 @@ export const setSearchParams = function (url: URL, ...objects: any[]) {
*
* @export
*/
export const serializeDataIfNeeded = function (value: any, requestOptions: any, configuration?: Configuration) {
export const serializeDataIfNeeded = function (value: any, requestOptions: any, configuration?: Configuration): any {
const nonString = typeof value !== 'string';
const needsSerialization = nonString && configuration && configuration.isJsonMime
? configuration.isJsonMime(requestOptions.headers['Content-Type'])
Expand All @@ -132,7 +132,7 @@ export const serializeDataIfNeeded = function (value: any, requestOptions: any,
*
* @export
*/
export const toPathString = function (url: URL) {
export const toPathString = function (url: URL): string {
return url.pathname + url.search + url.hash
}

Expand All @@ -141,7 +141,7 @@ export const toPathString = function (url: URL) {
* @export
*/
export const createRequestFunction = function (axiosArgs: RequestArgs, globalAxios: AxiosInstance, BASE_PATH: string, configuration?: Configuration) {
return <T = unknown, R = AxiosResponse<T>>(axios: AxiosInstance = globalAxios, basePath: string = BASE_PATH) => {
return <T = unknown, R = AxiosResponse<T>>(axios: AxiosInstance = globalAxios, basePath: string = BASE_PATH): Promise<R> => {
const axiosRequestArgs = {...axiosArgs.options, url: (configuration?.basePath || basePath) + axiosArgs.url};
return axios.request<T, R>(axiosRequestArgs);
};
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -446,7 +446,7 @@ export class AssetsApi extends BaseAPI {
* @throws {RequiredError}
* @memberof AssetsApi
*/
public getAsset(requestParameters: AssetsApiGetAssetRequest, options?: AxiosRequestConfig) {
public getAsset(requestParameters: AssetsApiGetAssetRequest, options?: AxiosRequestConfig): Promise<import("axios").AxiosResponse<Asset, any>> {
return AssetsApiFp(this.configuration).getAsset(requestParameters.tokenAddress, requestParameters.tokenId, requestParameters.includeFees, options).then((request) => request(this.axios, this.basePath));
}

Expand All @@ -458,7 +458,7 @@ export class AssetsApi extends BaseAPI {
* @throws {RequiredError}
* @memberof AssetsApi
*/
public listAssets(requestParameters: AssetsApiListAssetsRequest = {}, options?: AxiosRequestConfig) {
public listAssets(requestParameters: AssetsApiListAssetsRequest = {}, options?: AxiosRequestConfig): Promise<import("axios").AxiosResponse<ListAssetsResponse, any>> {
return AssetsApiFp(this.configuration).listAssets(requestParameters.pageSize, requestParameters.cursor, requestParameters.orderBy, requestParameters.direction, requestParameters.user, requestParameters.status, requestParameters.name, requestParameters.metadata, requestParameters.sellOrders, requestParameters.buyOrders, requestParameters.includeFees, requestParameters.collection, requestParameters.updatedMinTimestamp, requestParameters.updatedMaxTimestamp, requestParameters.auxiliaryFeePercentages, requestParameters.auxiliaryFeeRecipients, options).then((request) => request(this.axios, this.basePath));
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -277,7 +277,7 @@ export class BalancesApi extends BaseAPI {
* @throws {RequiredError}
* @memberof BalancesApi
*/
public getBalance(requestParameters: BalancesApiGetBalanceRequest, options?: AxiosRequestConfig) {
public getBalance(requestParameters: BalancesApiGetBalanceRequest, options?: AxiosRequestConfig): Promise<import("axios").AxiosResponse<Balance, any>> {
return BalancesApiFp(this.configuration).getBalance(requestParameters.owner, requestParameters.address, options).then((request) => request(this.axios, this.basePath));
}

Expand All @@ -289,7 +289,7 @@ export class BalancesApi extends BaseAPI {
* @throws {RequiredError}
* @memberof BalancesApi
*/
public listBalances(requestParameters: BalancesApiListBalancesRequest, options?: AxiosRequestConfig) {
public listBalances(requestParameters: BalancesApiListBalancesRequest, options?: AxiosRequestConfig): Promise<import("axios").AxiosResponse<ListBalancesResponse, any>> {
return BalancesApiFp(this.configuration).listBalances(requestParameters.owner, requestParameters.pageSize, requestParameters.cursor, requestParameters.orderBy, requestParameters.direction, options).then((request) => request(this.axios, this.basePath));
}
}
Loading
Loading