0xMiden / 0xMiden/web-sdk

`latest` on npm is a release candidate the public testnet rejects

Aberta
#349 1 comentário 0 reações 0 responsáveis Ver no GitHub
Linguagem predominante
TypeScript
Estrelas
1
Forks
21
Merge médio
12h 14min
PRs com merge (30d)
41

Descrição

Installing the SDK with the default tag gives a version that the public testnet will not accept, so the first call fails for anyone following the README.

```
npm dist-tags @miden-sdk/miden-sdk
latest -> 0.16.0-rc.5
next -> 0.16.0-alpha.2
```

Latest stable is 0.15.9.

### Repro

```bash
pnpm add @miden-sdk/miden-sdk # installs 0.16.0-rc.5
```

```js
const client = await MidenClient.createTestnet();
```

```
Failed to ensure genesis in place: RPC error: accept header validation failed:
server rejected request - please check your version and network settings
(client version: 0.16.0-rc.3, genesis commitment: none)
```

### The obvious fix leads into a second problem

Downgrading to 0.15.9 is the natural response, but the store left behind by the rc build then breaks the stable client:

```
failed to sync state: storage error: database-related non-query error:
failed to deserialize (failed to get sync height):
invalid type: unit value, expected struct SyncHeightIdxdbObject
```

`ensureClientVersion` in `crates/idxdb-store/src/js/schema.js` only resets the store when the incoming version is strictly greater, so a downgrade takes the early return and keeps the incompatible data:

```js
if (sameMajorMinor || !semver.gt(clientVersion, storedVersion)) {
await this.persistClientVersion(clientVersion);
return;
}
```

With `storedVersion` 0.16.0-rc.3 and `clientVersion` 0.15.9, `sameMajorMinor` is false and `semver.gt` is false, so the condition holds and no reset happens. The version marker is overwritten while the old data stays. Going the other way the reset does fire, so the protection is one directional.

This is the same function as #158 and #287, but a different aspect of it, so I filed it here rather than commenting there.

### Suggestion

Point `latest` at 0.15.9 and keep the rc under its own tag. That alone removes the path that leads into the store problem. Resetting on any version change rather than only on an increase would also cover the downgrade case, and the deserialization error could mention that the store may be stale.

Environment: macOS, Chrome 152, node 24, pnpm 10.18, testnet.

Guia de contribuição

Abrir o guia de contribuição

Direção de pesquisa

The issue is in the npm dist-tags and the store version handling. Start by checking the npm package tags via `npm dist-tags @miden-sdk/miden-sdk`. Look at the `ensureClientVersion` function in `crates/idxdb-store/src/js/schema.js` to understand the version comparison logic. The fix involves updating the npm tag and potentially modifying the version check to handle downgrades. Test by installing the SDK and verifying the testnet connection works with the stable version.

Escrita pelo modelo de indexação a partir do texto da issue.

Avaliação

Stack de tecnologia
nodejs, typescript
Domínio
cli, developer-experience, release
Tipo de issue
Bug
Dificuldade
3/5
Tempo estimado
1-2 dias
Status de atividade
Ativa
Clareza
Claramente especificada
Facilidade para iniciantes
65/100

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.