conductor-oss / conductor-oss/javascript-sdk

HttpInputParameters.readTimeOut typed as string — server expects integer (ms)

オープン 初心者向け
#138 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る
bug
主要言語
TypeScript
スター
58
フォーク
20
平均マージ
1日 13時間
マージ済み PR(30日)
7

説明

## Summary

`HttpInputParameters.readTimeOut` is typed as `string` in `src/open-api/types.ts:156`.
The server's `HttpTask.Input.readTimeOut` is `Integer` (milliseconds). The sibling field
`connectionTimeOut` is correctly typed as `number`. The inconsistency is a bug.

## Server baseline

Conductor **3.32.0-rc.9**

## Evidence

**Static:**
```typescript
// src/open-api/types.ts:155-156
connectionTimeOut?: number; // ✓ correct
readTimeOut?: string; // ✗ should be number
```

**Server:**
```java
// http-task/.../HttpTask.java:275
private Integer readTimeOut = 3000;
```

**Live (2026-07-16):** Server accepts both `readTimeOut: 3000` (integer) and
`readTimeOut: "3000"` (string) due to Jackson coercion. However, the TypeScript type
is still wrong — passing a number for `readTimeOut` is a compile-time error with the
current type definition.

## Impact

TypeScript users who pass `readTimeOut: 5000` (the natural form) get a type error and
must write `readTimeOut: "5000"` (string), which is semantically confusing and diverges
from `connectionTimeOut: 5000` (number).

## Proposed fix

```typescript
// src/open-api/types.ts:156
readTimeOut?: number; // was: string
```

## Related

Discovered during systematic SDK audit against Conductor OSS 3.32.0-rc.9.

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

調査の方向性

src/open-api/types.ts:156 から始め、直前の行にある正しく型付けされた connectionTimeOut フィールドと readTimeOut を比較してください。数値のミリ秒値を受け入れられるように宣言を変更し、その後 TypeScript のチェックが通ること、およびパブリック型がサーバーの Integer フィールドと一致することを確認してください。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
typescript
領域
api
issue の種類
バグ
難易度
1/5
見積もり時間
1時間未満
活発さ
静か
明瞭さ
明確に書かれている
初心者へのやさしさ
91/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。