Repository navigation
Conversation
Signed-off-by: nonam3e <timur@moonmath.ai>
Signed-off-by: nonam3e <timur@moonmath.ai>
Signed-off-by: nonam3e <timur@moonmath.ai>
Signed-off-by: nonam3e <timur@moonmath.ai>
Signed-off-by: nonam3e <timur@moonmath.ai>
Signed-off-by: nonam3e <timur@moonmath.ai>
Signed-off-by: nonam3e <timur@moonmath.ai>
zro-inference
left a comment
There was a problem hiding this comment.
Adds single-executable release binaries: a binaries matrix job in publish.yml (linux x64/musl/arm64, darwin arm64/x64 with --version smoke tests), scripts/build-sea.mjs plus release-manifest and shared helpers, a bundling/install.sh path, and a new src/selfupdate.ts for in-place self-update, with corresponding tests (args, install, release-manifest, selfupdate). Review coverage caveat: all four analysis specialists (correctness, security, test-coverage, adversarial) failed on this run — three with upstream 524 timeouts and one with a hard 1200s timeout — so no substantive findings were produced and this approval reflects lack of reported issues, not a verified clean pass. Notably unexamined by any specialist: checksum/signature verification and download-source trust in install.sh and src/selfupdate.ts, atomicity and rollback when replacing a running binary, and shell-quoting/uname-mapping edge cases in the installer. A re-review is recommended before relying on self-update in production.
JenClaude command reference
Reply with @MoonMathJen on this PR to trigger commands:
| Command | Example | What it does |
|---|---|---|
| Re-review | @MoonMathJen rereview |
Re-review after changes (full or scoped — see below) |
| Stop | @MoonMathJen stop |
Cancel in-progress review |
| Stop + summarize | @MoonMathJen stop and summarize |
Post partial findings so far |
| Focus specialty | @MoonMathJen only security |
Review only the named specialty |
| Skip specialty | @MoonMathJen skip test-coverage |
Review everything except the named specialty |
| Change model | @MoonMathJen pr-review-model: kimi-k2.7-coder |
Run this review with a specific model |
| Add context | @MoonMathJen focus on auth edge cases |
Free-text guidance passed to reviewers |
Specialties: correctness, security, test-coverage, adversarial
Any text in the comment beyond a recognized command is passed to the reviewers as context. Use it to point them at areas that matter, explain what changed since the last review, or call out constraints (e.g. "skip generated files", "the auth flow is the riskiest part here"). Hard review rules still win over reviewer guidance.
JenClaude metrics: cost $0.000, time 39m 20s, tokens 43,691.
Ships zro as self-contained binaries alongside the npm package: a release pipeline,
a
curl | bashinstaller, and in-binary self-update. The npm distribution is unchanged.scripts/build-sea.mjsbundles the CLI into the official (pinned,sha256-verified) Node 22 tarballs via postject for five platforms, inlining the version and platform at build time.
leg attaches a partial set instead of an empty release.
install.shreads the manifest, prefers the zstd asset, verifiessha256, installs to
~/.local/bin, and test-runs the binary before declaring success.zro updatefetches the latest manifest, verifies the download,and atomically replaces the running executable; failures never touch the old binary.
Up-to-date/newer-than-latest are no-ops;
ZRO_UPDATE_BASEoverrides the source.