conductor-oss / conductor-oss/javascript-sdk
HttpInputParameters.readTimeOut typed as string — server expects integer (ms)
- 主要言語
- 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