feature: Metadata contract

[Siel]

This PR aims to define the metadata contract between the different runtimes in pharmsol. The different runtimes now expose metadata directly so downstream crates can read it independently on the runtime used

[Copilot]

Pull request overview

This PR expands the compiled-runtime metadata contract so DSL runtimes can expose richer model metadata consistently across JIT, AoT, and WASM backends.

Changes:

• Adds state, covariate interpolation, and route destination/feature metadata to NativeModelInfo.
• Builds and exposes ValidatedModelMetadata from compiled native/runtime models.
• Tightens WASM metadata loading to require the compiled metadata envelope and kernel availability agreement.

Reviewed changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
src/parameter_order.rs	Updates test metadata construction for the expanded NativeModelInfo shape.
src/dsl/wasm.rs	Requires compiled metadata envelopes and updates WASM loader/tests for richer metadata.
src/dsl/runtime.rs	Exposes runtime metadata accessors and propagates model-construction errors.
src/dsl/native.rs	Constructs validated runtime metadata and exposes it from compiled native models.
src/dsl/model_info.rs	Extends compiled model info with states, interpolation, and route metadata.
src/dsl/mod.rs	Re-exports the new runtime state metadata type.
src/dsl/jit.rs	Propagates metadata validation errors during JIT compilation.
src/dsl/compiled_backend_abi.rs	Updates ABI tests for the expanded metadata fields.
src/dsl/aot.rs	Propagates metadata validation errors while loading AoT artifacts.
pharmsol-dsl/src/ir.rs	Makes covariate interpolation serializable for metadata transport.

Comments suppressed due to low confidence (2)

src/dsl/native.rs:401

• Defaulting legacy structured-block routes to Bolus makes the exposed validated metadata disagree with runtime behavior: RouteInputSemantics still treats kind == None routes as supporting both bolus and infusion inputs, while metadata().route(...).kind() will report bolus-only. Downstream crates relying on the new metadata contract can incorrectly reject infusion use for those models.

        // Structured-block DSL routes still lower without an explicit kind.
        // Treat them as declaration-ordered bolus routes for the shared
        // metadata surface while preserving the original runtime semantics
        // from `info.routes` below.
        let kind = route.kind.unwrap_or(RouteKind::Bolus);

src/dsl/model_info.rs:66

• The state metadata omits the declared state's length even though execution states can span multiple buffer slots (ExecutionState has len). Without preserving that length (or expanding one entry per slot), downstream metadata consumers cannot reconstruct vector/array state layouts or map offsets back to state elements correctly.

/// Metadata for one compiled state.
#[derive(Clone, Debug, PartialEq, Eq, Serialize, Deserialize)]
pub struct NativeStateInfo {
    /// Public state name.
    pub name: String,
    /// Dense runtime state offset.
    pub offset: usize,

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[github-actions]

Bencher Report

Branch	fix/missing_metadata
Testbed	mhovd-pgx

⚠️ WARNING: Truncated view!

The full continuous benchmarking report exceeds the maximum length allowed on this platform.

🐰 View full continuous benchmarking report in Bencher