Repository navigation
Show thirdshift cron jobs in the Console - #962
Merged
Merged
Conversation
This was referenced Oct 21, 2026
Merged
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
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.
Summary
Shows active entries containing a literal thirdshift word or executable path. Reads
crontab -lwithout changing schedules or starting work. Empty crontabs and read failures produce distinct results. Other accounts, system crontabs and script contents are outside this inventory.Closes #953
Evidence
cargo test --test console cronjobs::cronjobs_pairs_cli_and_console_without_running_scheduled_commands -- --exactfailed becausecronjobswas not a command.After: all three cronjobs integration tests pass, covering CLI/Console parity, escaping, empty/read-failure states and cron's command/stdin separator.
[]for/usr/local/bin/thirdshift%stdin; its retained regression also failed.After: the same reproduction passes;
cronjobs::cronjobs_respects_crons_command_and_stdin_separatorincludes the job, excludes matches found only in stdin, and preserves the full displayed entry.cd console && npm test: all 36 browser tests pass, including Cron jobs link/keyboard navigation, reload and 390 px layout.cargo test: full Rust suite passes.cargo check --all-targets,cargo clippy --all-targets -- -D warnings, andcargo fmt --check: pass.Test seams
cronjobs [--json]and Console HTTP routes, replacingcrontab -lonly at the operating system process boundary.Merge Danger
Door: two-way
No persistent format or configuration changes.
Blast Radius: Console
Adds a read-only crontab inventory and its CLI equivalent; existing scheduling remains with the operating system.
Unaddressed findings
Standards
None.
Spec
None. The one P3 finding was reproduced, fixed in
0f75d1c, and verified by the reviewer on follow-up.Security review outcome
Review coverage
Reviewed against
main(7a9331c). Standards: 0 findings. Spec: 1 finding, resolved. Both independent reviewers read all 13 changed files and reviewed the follow-up. Unread changed files: none under either axis. Generated-output exclusions: none.Built with codex · gpt-6.1-sol · high