[BUG] sozo --typescript bindings errors

[rsodre]

Describe the bug

Source files generated by sozo --typescript have errors.

To Reproduce

• clone pistols, branch thai
• run: cd dojo && git checkout thai && sozo build --typescript
• inspect dojo/bindings/typescript/contracts.gen.ts and dojo/bindings/typescript/models.gen.ts

or just get the generated files here:

• contracts.gen.ts
• contracts.gen.ts

models.gen.ts

• big integer types (u64, u128, u256, felt252, ContractAddress) are being converted as number, while they should be bigint or BigNumberish.

// Type definition for `pistols::models::challenge::Challenge` struct
export interface Challenge {
  fieldOrder: string[];
  duel_id: number;        // u128 should be: bigint
  seed: number;           // u128 should be: bigint
  table_id: number;       // felt252 should be: bigint
  premise: Premise;
  quote: number;          // felt252 should be: bigint
  address_a: string;      // ContractAddress should be: bigint
  address_b: string;      // ContractAddress should be: bigint
  duelist_id_a: number;   // u128 should be: bigint
  duelist_id_b: number;   // u128 should be: bigint
  state: ChallengeState;
  winner: number;
  timestamp_start: number; // u64 should be: bigint
  timestamp_end: number;   // u64 should be: bigint
}

contracts.gen.ts

• calls to provider.execute have the wrong number of parameters (missing namespace)

• if I add the namespace manually, there's another error on entryPoint:

• any function with an argument named account generates a Duplicate identifier 'account'.ts(2300) error:

• interfaces that include snake_case and camelCase versions of the same functions have their functions duplicated, as the generator converts snake_casr to camelCase.

• some types are missing the namespace. ex: [Payment](payment: Payment) should be payment: models.Payment

const bank_charge = async (_account: Account, payer: string, payment: Payment) => {
const admin_setConfig = async (_account: Account, config: Config) => {
const admin_setTable = async (_account: Account, table: TableConfig) => {
const admin_setTableAdmittance = async (_account: Account, tableAdmittance: TableAdmittance) => {