Skip to content

[Upstream PR #116] fix: accept stringified ambient cycle counts #96

@quangdang46

Description

@quangdang46

Mirrored from upstream 1jehuang/jcodePull Request #116 by @zerone0x
Original state: open
Created: 2026-05-03T20:09:55Z · Updated: 2026-05-03T20:09:57Z
Diff: https://github.com/1jehuang/jcode/pull/116.diff
This issue is an auto-mirrored copy. Comments and edits here are local to quangdang46/jcode — do not expect them to propagate upstream.


Summary

  • allow the ambient cycle tool to deserialize numeric counters from either JSON numbers or integer strings
  • apply the same lenient handling to next_schedule.wake_in_minutes
  • add regression coverage for stringified counts

Fixes #106

Test Plan

  • cargo fmt --all
  • cargo test -q test_end_cycle_input (blocked locally: missing pkg-config/OpenSSL development metadata; sudo requires a password on this host)

View in Codesmith
Need help on this PR? Tag @codesmith with what you need.

  • Let Codesmith autofix CI failures and bot reviews

Metadata

Metadata

Assignees

No one assigned

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions