burn dogfood fixes + accurate agy docs (v0.5.13)#21
Merged
Conversation
- burn: reject STALE artifacts (mtime guard via _clikae_mtime, GNU-stat-first), add --fresh, perl --timeout fallback, run-summary line, boundary docs - agy docs/i18n: state follows $HOME but login is one global Keychain entry; drop dead "can't rename" string; remove names a concrete other tank - tests: stale/fresh/overwrite + perl-fallback (308 green); bump to v0.5.13 Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
mixflavor
added a commit
that referenced
this pull request
Jun 13, 2026
- burn: reject STALE artifacts (mtime guard via _clikae_mtime, GNU-stat-first), add --fresh, perl --timeout fallback, run-summary line, boundary docs - agy docs/i18n: state follows $HOME but login is one global Keychain entry; drop dead "can't rename" string; remove names a concrete other tank - tests: stale/fresh/overwrite + perl-fallback (308 green); bump to v0.5.13 Co-authored-by: chodaict <x@cver.net> Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Hardens
clikae burn(from two independent dogfoods — tugtile + a live failed run) and corrects inaccurate agy docs/strings.burn
_clikae_mtimehelper. New--freshclears it first; a pre-existing artifact is warned.--timeoutperl fallback: whentimeout/gtimeout(coreutils) is absent (stock macOS), bound the run with aperlalarm. Only with none of the three is the run unbounded (still warned).agy docs/i18n accuracy
$HOME, but its login is one global Keychain entry — so different accounts can't be routed per-shell (switching is global). Corrected comments + i18n.T_RENAME_NO_AGY("can't be renamed") string — agy rename works and carries the per-tank Keychain login.remove agy <active-tank>now names a concrete other tank.Testing
batssuite green on macOS (+3 new burn cases: stale-reject,--fresh, overwrite→success).shellcheckclean.--timeoutfallback, and the summary line. Success path covered by the new bats cases.Platform note — help wanted 🙏
The one platform-sensitive change is artifact freshness (reads file mtime). It reuses the GNU-stat-first
_clikae_mtimehelper (correct on macOS BSDstatand Linux GNUstat) and a regression test (pre-existing → overwritten → success) runs in CI onubuntu-latest. That said, we develop and hand-test on macOS only — no real Linux/BSD hardware. Linux / WSL / BSD testers: please kick the tires onclikae burn --artifact(real success registers? any false "real task failure"?) and open an issue/PR.🤖 Generated with Claude Code