v1 prep

[alex-omophub]

Summary by cubic

Prepares the v1.0.0 release of @omophub/omophub-node with a typed OMOPHub client (concepts, search, mappings, relationships, hierarchy, domains, vocabularies) and a FHIR resolver. Adds docs, examples, tests, and improved CI/publish workflows; hardens auth, retry/backoff (RFC 9110 Retry-After), and pagination.

• New Features

  • OMOPHub client: concepts, search (basic/advanced/autocomplete), mappings, relationships, hierarchy, domains, vocabularies, and FHIR resolve (single, batch, CodeableConcept) with omophubFhirUrl.
  • HTTP layer: per-call headers/query/AbortSignal/timeout, idempotency key (POST), and retries on 429/502/503/504 or network errors with full‑jitter backoff honoring Retry-After. POSTs are safely retried on 429 even without an idempotency key because the server declines without side effects.
  • Errors + pagination: methods return { data, error, meta, headers }; misuse throws OMOPHubError; async iterators throw OMOPHubIteratorError. Async pagination helpers and fetch‑all; request keys auto snake_cased; normalized search responses.
  • Tooling/CI/release: strict TypeScript, Biome, Vitest with coverage, e2e suite and runnable examples, CI on Node 22/24, npm publish with provenance. CI integration job runs only on pushes or manual dispatches to main/develop (branch‑gated to protect API keys).
  • Hardening: trim whitespace in setApiKey; strict RFC 9110 Retry-After parsing; pagination helpers handle outer envelopes and bare arrays.

• Migration

  • Requires Node 22+.
  • Set OMOPHUB_API_KEY or pass the key to new OMOPHub('oh_...').

^{Written for commit f040834. Summary will update on new commits.}

[cubic-dev-ai]

5 issues found across 147 files

<sub>Note: This PR contains a large number of files. cubic only reviews up to 100 files per PR, so some files may not have been reviewed. cubic prioritizes the most important files to review.
On a pro plan you can use ultrareview for larger PRs.

Re-trigger cubic</sub>

[cubic-dev-ai]

1 issue found across 8 files (changes from recent commits).

<sub>Reply with feedback, questions, or to request a fix.

Re-trigger cubic</sub>

[cubic-dev-ai]

1 issue found across 1 file (changes from recent commits).

<sub>Tip: Review your code locally with the cubic CLI to iterate faster.

Re-trigger cubic</sub>

[cubic-dev-ai]

1 issue found across 3 files (changes from recent commits).

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="src/client.ts">

<violation number="1" location="src/client.ts:200">
P1: This retry condition now allows POST/PATCH retries without an `Idempotency-Key` on 429 responses, which breaks the SDK’s documented retry contract for non-idempotent requests.</violation>
</file>

<sub>Tip: Review your code locally with the cubic CLI to iterate faster.

Re-trigger cubic</sub>

[cubic-dev-ai]

P1: This retry condition now allows POST/PATCH retries without an Idempotency-Key on 429 responses, which breaks the SDK’s documented retry contract for non-idempotent requests.

Prompt for AI agents

Check if this issue is valid — if so, understand the root cause and fix it. At src/client.ts, line 200:

<comment>This retry condition now allows POST/PATCH retries without an `Idempotency-Key` on 429 responses, which breaks the SDK’s documented retry contract for non-idempotent requests.</comment>

<file context>
@@ -197,7 +197,7 @@ export class OMOPHub {
           isRetryableStatus(response.status) &&
           attempt < this.maxRetries &&
-          isRetryableRequest(method, headers)
+          (isNoSideEffectStatus(response.status) || isRetryableRequest(method, headers))
         ) {
           const retryAfter = response.headers.get('retry-after');
</file context>