fix type in umd module export

[fs-eire]

import Long from "../index.js";
export = Long;

previous code generates errors complaining about cannot import from ESmodule to commonJS. a workaround is to duplicate the content of index.d.ts

node_modules/long/umd/index.d.ts:1:18 - error TS1479: The current file is a CommonJS module whose imports will produce 'require' calls; however, the referenced file is an ECMAScript module and cannot be imported with 'require'. Consider writing a dynamic 'import("../index.js")' call instead.

1 import Long from "../index.js";
~~~~~~~~~~~~~

Found 1 error in node_modules/long/umd/index.d.ts:1

[fs-eire]

this may not be the best solution because it duplicates the index.d.ts. However I tried a few ways and this is the only one that works to me.

[Silventino]

This fixed the problem for me.

[tzvc]

This fixes my build too. What I am struggling to understand is why this is posing a problem only now, i've been running this code fine the past months

[pumano]

any news about it? it should be merged and populated to npm

[henriquecarv]

Maybe this can work instead?

Suggested change

	declare class Long {
	/**
	* Constructs a 64 bit two's-complement integer, given its low and high 32 bit values as signed integers. See the from* functions below for more convenient ways of constructing Longs.
	*/
	constructor(low: number, high?: number, unsigned?: boolean);
	/**
	* Maximum unsigned value.
	*/
	static MAX_UNSIGNED_VALUE: Long;
	/**
	* Maximum signed value.
	*/
	static MAX_VALUE: Long;
	/**
	* Minimum signed value.
	*/
	static MIN_VALUE: Long;
	/**
	* Signed negative one.
	*/
	static NEG_ONE: Long;
	/**
	* Signed one.
	*/
	static ONE: Long;
	/**
	* Unsigned one.
	*/
	static UONE: Long;
	/**
	* Unsigned zero.
	*/
	static UZERO: Long;
	/**
	* Signed zero
	*/
	static ZERO: Long;
	/**
	* The high 32 bits as a signed value.
	*/
	high: number;
	/**
	* The low 32 bits as a signed value.
	*/
	low: number;
	/**
	* Whether unsigned or not.
	*/
	unsigned: boolean;
	/**
	* Returns a Long representing the 64 bit integer that comes by concatenating the given low and high bits. Each is assumed to use 32 bits.
	*/
	static fromBits(lowBits: number, highBits: number, unsigned?: boolean): Long;
	/**
	* Returns a Long representing the given 32 bit integer value.
	*/
	static fromInt(value: number, unsigned?: boolean): Long;
	/**
	* Returns a Long representing the given value, provided that it is a finite number. Otherwise, zero is returned.
	*/
	static fromNumber(value: number, unsigned?: boolean): Long;
	/**
	* Returns a Long representation of the given string, written using the specified radix.
	*/
	static fromString(
	str: string,
	unsigned?: boolean | number,
	radix?: number
	): Long;
	/**
	* Creates a Long from its byte representation.
	*/
	static fromBytes(bytes: number[], unsigned?: boolean, le?: boolean): Long;
	/**
	* Creates a Long from its little endian byte representation.
	*/
	static fromBytesLE(bytes: number[], unsigned?: boolean): Long;
	/**
	* Creates a Long from its big endian byte representation.
	*/
	static fromBytesBE(bytes: number[], unsigned?: boolean): Long;
	/**
	* Tests if the specified object is a Long.
	*/
	static isLong(obj: any): obj is Long;
	/**
	* Converts the specified value to a Long.
	*/
	static fromValue(
	val:
	| Long
	| number
	| string
	| { low: number; high: number; unsigned: boolean },
	unsigned?: boolean
	): Long;
	/**
	* Returns the sum of this and the specified Long.
	*/
	add(addend: number | Long | string): Long;
	/**
	* Returns the bitwise AND of this Long and the specified.
	*/
	and(other: Long | number | string): Long;
	/**
	* Compares this Long's value with the specified's.
	*/
	compare(other: Long | number | string): number;
	/**
	* Compares this Long's value with the specified's.
	*/
	comp(other: Long | number | string): number;
	/**
	* Returns this Long divided by the specified.
	*/
	divide(divisor: Long | number | string): Long;
	/**
	* Returns this Long divided by the specified.
	*/
	div(divisor: Long | number | string): Long;
	/**
	* Tests if this Long's value equals the specified's.
	*/
	equals(other: Long | number | string): boolean;
	/**
	* Tests if this Long's value equals the specified's.
	*/
	eq(other: Long | number | string): boolean;
	/**
	* Gets the high 32 bits as a signed integer.
	*/
	getHighBits(): number;
	/**
	* Gets the high 32 bits as an unsigned integer.
	*/
	getHighBitsUnsigned(): number;
	/**
	* Gets the low 32 bits as a signed integer.
	*/
	getLowBits(): number;
	/**
	* Gets the low 32 bits as an unsigned integer.
	*/
	getLowBitsUnsigned(): number;
	/**
	* Gets the number of bits needed to represent the absolute value of this Long.
	*/
	getNumBitsAbs(): number;
	/**
	* Tests if this Long's value is greater than the specified's.
	*/
	greaterThan(other: Long | number | string): boolean;
	/**
	* Tests if this Long's value is greater than the specified's.
	*/
	gt(other: Long | number | string): boolean;
	/**
	* Tests if this Long's value is greater than or equal the specified's.
	*/
	greaterThanOrEqual(other: Long | number | string): boolean;
	/**
	* Tests if this Long's value is greater than or equal the specified's.
	*/
	gte(other: Long | number | string): boolean;
	/**
	* Tests if this Long's value is greater than or equal the specified's.
	*/
	ge(other: Long | number | string): boolean;
	/**
	* Tests if this Long's value is even.
	*/
	isEven(): boolean;
	/**
	* Tests if this Long's value is negative.
	*/
	isNegative(): boolean;
	/**
	* Tests if this Long's value is odd.
	*/
	isOdd(): boolean;
	/**
	* Tests if this Long's value is positive or zero.
	*/
	isPositive(): boolean;
	/**
	* Tests if this Long's value equals zero.
	*/
	isZero(): boolean;
	/**
	* Tests if this Long's value equals zero.
	*/
	eqz(): boolean;
	/**
	* Tests if this Long's value is less than the specified's.
	*/
	lessThan(other: Long | number | string): boolean;
	/**
	* Tests if this Long's value is less than the specified's.
	*/
	lt(other: Long | number | string): boolean;
	/**
	* Tests if this Long's value is less than or equal the specified's.
	*/
	lessThanOrEqual(other: Long | number | string): boolean;
	/**
	* Tests if this Long's value is less than or equal the specified's.
	*/
	lte(other: Long | number | string): boolean;
	/**
	* Tests if this Long's value is less than or equal the specified's.
	*/
	le(other: Long | number | string): boolean;
	/**
	* Returns this Long modulo the specified.
	*/
	modulo(other: Long | number | string): Long;
	/**
	* Returns this Long modulo the specified.
	*/
	mod(other: Long | number | string): Long;
	/**
	* Returns this Long modulo the specified.
	*/
	rem(other: Long | number | string): Long;
	/**
	* Returns the product of this and the specified Long.
	*/
	multiply(multiplier: Long | number | string): Long;
	/**
	* Returns the product of this and the specified Long.
	*/
	mul(multiplier: Long | number | string): Long;
	/**
	* Negates this Long's value.
	*/
	negate(): Long;
	/**
	* Negates this Long's value.
	*/
	neg(): Long;
	/**
	* Returns the bitwise NOT of this Long.
	*/
	not(): Long;
	/**
	* Returns count leading zeros of this Long.
	*/
	countLeadingZeros(): number;
	/**
	* Returns count leading zeros of this Long.
	*/
	clz(): number;
	/**
	* Returns count trailing zeros of this Long.
	*/
	countTrailingZeros(): number;
	/**
	* Returns count trailing zeros of this Long.
	*/
	ctz(): number;
	/**
	* Tests if this Long's value differs from the specified's.
	*/
	notEquals(other: Long | number | string): boolean;
	/**
	* Tests if this Long's value differs from the specified's.
	*/
	neq(other: Long | number | string): boolean;
	/**
	* Tests if this Long's value differs from the specified's.
	*/
	ne(other: Long | number | string): boolean;
	/**
	* Returns the bitwise OR of this Long and the specified.
	*/
	or(other: Long | number | string): Long;
	/**
	* Returns this Long with bits shifted to the left by the given amount.
	*/
	shiftLeft(numBits: number | Long): Long;
	/**
	* Returns this Long with bits shifted to the left by the given amount.
	*/
	shl(numBits: number | Long): Long;
	/**
	* Returns this Long with bits arithmetically shifted to the right by the given amount.
	*/
	shiftRight(numBits: number | Long): Long;
	/**
	* Returns this Long with bits arithmetically shifted to the right by the given amount.
	*/
	shr(numBits: number | Long): Long;
	/**
	* Returns this Long with bits logically shifted to the right by the given amount.
	*/
	shiftRightUnsigned(numBits: number | Long): Long;
	/**
	* Returns this Long with bits logically shifted to the right by the given amount.
	*/
	shru(numBits: number | Long): Long;
	/**
	* Returns this Long with bits logically shifted to the right by the given amount.
	*/
	shr_u(numBits: number | Long): Long;
	/**
	* Returns this Long with bits rotated to the left by the given amount.
	*/
	rotateLeft(numBits: number | Long): Long;
	/**
	* Returns this Long with bits rotated to the left by the given amount.
	*/
	rotl(numBits: number | Long): Long;
	/**
	* Returns this Long with bits rotated to the right by the given amount.
	*/
	rotateRight(numBits: number | Long): Long;
	/**
	* Returns this Long with bits rotated to the right by the given amount.
	*/
	rotr(numBits: number | Long): Long;
	/**
	* Returns the difference of this and the specified Long.
	*/
	subtract(subtrahend: number | Long | string): Long;
	/**
	* Returns the difference of this and the specified Long.
	*/
	sub(subtrahend: number | Long | string): Long;
	/**
	* Converts the Long to a 32 bit integer, assuming it is a 32 bit integer.
	*/
	toInt(): number;
	/**
	* Converts the Long to a the nearest floating-point representation of this value (double, 53 bit mantissa).
	*/
	toNumber(): number;
	/**
	* Converts this Long to its byte representation.
	*/
	toBytes(le?: boolean): number[];
	/**
	* Converts this Long to its little endian byte representation.
	*/
	toBytesLE(): number[];
	/**
	* Converts this Long to its big endian byte representation.
	*/
	toBytesBE(): number[];
	/**
	* Converts this Long to signed.
	*/
	toSigned(): Long;
	/**
	* Converts the Long to a string written in the specified radix.
	*/
	toString(radix?: number): string;
	/**
	* Converts this Long to unsigned.
	*/
	toUnsigned(): Long;
	/**
	* Returns the bitwise XOR of this Long and the given one.
	*/
	xor(other: Long | number | string): Long;
	}
	export default Long; // compatible with `import Long from "long"`
	export * from "long"

[fs-eire]

No. This does not work for me.

error TS2709: Cannot use namespace 'Long' as a type.

39     i?: (number|Long|null);
                   ~~~~

[fs-eire]

It has been about 4 months since this PR is out and as you can see we really need this change to be merged and publish to NPM so that a lot of users (#124, #125) will be happy.

I understand that as the code owner you may have some concerns about this PR (eg. duplicated source file for the .d.ts). How about we apply this fix first and then try to improve it later? A not-so-good solution is better than none.

[alecgibson]

@fs-eire @henriquecarv I think if you invert the import dependencies this should work without duplication:

// umd/index.d.ts

declare class Long {
  // ...
}

export = Long;

// index.d.ts

import Long from './umd/index.js';

export default Long;

[alecgibson]

Oh never mind I see that's what they've proposed in #130

Would recommend closing this PR in favour of that one.