Skip to content

fix: a stray log name no longer holds a dead job at FRESH - #20

Merged
jimy-r merged 1 commit into
mainfrom
audit/2026-10-10-run25
Oct 11, 2026
Merged

jimy-r merged 1 commit into
mainfrom
audit/2026-10-10-run25

Conversation

@jimy-r

@jimy-r jimy-r commented Oct 10, 2026

Copy link
Copy Markdown
Owner

One commit from the 2026-10-10 audit of this tool.

  • The checker took the last matching filename in sort order as the newest run. A name stamped in the future, or one with a date that does not exist, sorted last and passed a dead task. latest_log() now counts a name only when its stamp is a real time no more than five minutes ahead of the clock, and the detail line lists the names it set aside.
  • A task whose log names were all set aside reports LOG_UNREADABLE.
  • {time} in log_pattern accepts HHMMSS as well as HHMM.
  • A new optional task key, line_stamp, allows one ISO 8601 time in front of a sentinel.
  • 39 new tests and five selftest checks.

This changes what counts as a finding. A task that passed on such a name now reports STALE or LOG_UNREADABLE. The changelog entry is prepared as 0.3.1, and no tag or release is part of this PR.

🤖 Generated with Claude Code

….3.1

The checker took the last matching filename in sort order as the newest
run. A name stamped in the future, or one with a date that does not exist,
sorted last, gave a negative or missing age, skipped the staleness test and
passed the task on the success line inside it.

- latest_log() counts a name only when its stamp is a real time no more
  than five minutes ahead of the clock, picks the newest by that time, and
  hands back the names it set aside so the detail line can list them.
- A task whose log names were all set aside reports LOG_UNREADABLE instead
  of NEVER_RAN, or MANUAL_OK on a manual task.
- {time} in log_pattern accepts HHMMSS as well as HHMM.
- New optional task key line_stamp: one ISO 8601 time may sit in front of
  a sentinel, for a job whose logger stamps every line.
- 39 new tests and five selftest checks. README, SECURITY.md and CHANGELOG
  updated, and the example config carries the new key.

This changes what counts as a finding. A task that passed on such a name
now reports STALE or LOG_UNREADABLE.

Audit finding 0dce1ce0.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@jimy-r
jimy-r marked this pull request as ready for review October 11, 2026 02:45
@jimy-r
jimy-r merged commit 68bde85 into main Oct 11, 2026
8 checks passed
@jimy-r
jimy-r deleted the audit/2026-10-10-run25 branch October 11, 2026 02:45
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