Skip to content

refactor: strip no_std from kiln-runtime — std-only interpreter per RFC #46#169

Merged
avrabe merged 1 commit intomainfrom
refactor/strip-nostd-interpreter
Mar 19, 2026
Merged

refactor: strip no_std from kiln-runtime — std-only interpreter per RFC #46#169
avrabe merged 1 commit intomainfrom
refactor/strip-nostd-interpreter

Conversation

@avrabe
Copy link
Collaborator

@avrabe avrabe commented Mar 19, 2026

Summary

  • RFC build(deps): bump wast from 228.0.0 to 229.0.0 #46 architecture docs: toolchain architecture, interface design, C API reference for the std-only interpreter / kiln-builtins split
  • Phase 1: Strip no_std from prelude.rs and lib.rs — remove 414 lines of cfg-gated polyfills, bounded collections, custom macros
  • Phase 2: Strip no_std from all 30 remaining source files — remove 1,251 lines of dead #[cfg(not(feature = "std"))] code paths, simplify compound guards (cfg(all(std, debugger))cfg(debugger))

Result: Zero cfg(feature = "std") / cfg(not(feature = "std")) guards remain in kiln-runtime. The interpreter is fully std-only. The no_std path is via kiln-builtins for synth-compiled code targeting gale/Zephyr.

Test plan

  • cargo check -p kiln-runtime — clean compile
  • cargo test -p kiln-runtime --lib — 95/95 tests pass
  • CI passes

🤖 Generated with Claude Code

…150)

Remove all #[cfg(feature = "std")] guards and dead #[cfg(not(feature = "std"))]
code paths from 30 kiln-runtime source files. The interpreter is std-only per
RFC #46 — the no_std path is via kiln-builtins for synth-compiled code.

- 1,251 lines of dead no_std code removed across 30 files
- Simplify compound guards: cfg(all(std, debugger)) → cfg(debugger), etc.
- All 95 kiln-runtime tests pass

Trace: skip
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@github-actions
Copy link

🔍 Build Diagnostics Report

Summary

Metric Base Branch This PR Change
Errors 0 0 0
Warnings 2 2 0

🎯 Impact Analysis

Issues in Files You Modified

  • 0 new errors introduced by your changes
  • 0 new warnings introduced by your changes
  • 0 total errors in modified files
  • 0 total warnings in modified files
  • 0 files you modified

Cascading Issues (Your Changes Breaking Other Files)

  • 0 new errors in unchanged files
  • 0 new warnings in unchanged files
  • 0 unchanged files now affected

Note: "Cascading issues" are errors in files you didn't modify, caused by your changes (e.g., breaking API changes, dependency issues).

✅ No Issues Detected

Perfect! Your changes don't introduce any new errors or warnings, and don't break any existing code.


📊 Full diagnostic data available in workflow artifacts

🔧 To reproduce locally:

# Install cargo-kiln
cargo install --path cargo-kiln

# Analyze your changes
cargo-kiln build --output json --filter-severity error
cargo-kiln check --output json --filter-severity warning

@avrabe avrabe merged commit 339267d into main Mar 19, 2026
16 checks passed
@avrabe avrabe deleted the refactor/strip-nostd-interpreter branch March 19, 2026 07:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant