Skip to content

Roll up integration into dev#254

Merged
loning merged 2 commits into
devfrom
integration
Jul 6, 2026
Merged

Roll up integration into dev#254
loning merged 2 commits into
devfrom
integration

Conversation

@loning

@loning loning commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

Summary

This rollup includes PR #253 for issue #252, completing production coverage for the Lua SDK toml.decode surface. The change keeps the API decode-only and locks the Lua-facing contract with focused regression tests.

Changes

  • Added tests for invalid TOML fail-closed behavior with toml.decode invalid-toml: error text.
  • Covered nested tables, strings, booleans, integers, floats, arrays, and Lua sequence indexing through toml.decode.
  • Covered empty TOML input returning an empty Lua table.
  • Confirmed toml.encode remains absent.

Verification

PR metadata reports pre-merge CI passing, and Codecov reported all modified coverable lines covered.

中文摘要

本次 rollup 合入 #253,完成 #252toml.decode 的生产级行为覆盖。变更保持 decode-only surface,重点补齐 Lua 边界测试、错误分类、数组/表转换和 toml.encode 缺席约束。
⟦AI:FKST⟧

loning added 2 commits July 6, 2026 16:41
…oject/fkst-substrate/252/ready-github-devloop-issue-ChronoAIProject-fkst-substrate-252-intake-1093953665-3603307873

github-devloop implementation for #252
@codecov

codecov Bot commented Jul 6, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@loning loning merged commit 4d5a822 into dev Jul 6, 2026
8 checks passed
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