Skip to content

Add vigilante logs command for session log viewing#281

Merged
nicobistolfi merged 1 commit intoaliengiraffe:mainfrom
mvanhorn:osc/feat-vigilante-logs-command
Mar 24, 2026
Merged

Add vigilante logs command for session log viewing#281
nicobistolfi merged 1 commit intoaliengiraffe:mainfrom
mvanhorn:osc/feat-vigilante-logs-command

Conversation

@mvanhorn
Copy link
Copy Markdown
Contributor

Summary

Adds a vigilante logs command that lists session log files and shows individual session logs by repo and issue number.

Why this matters

Session logs are written to ~/.vigilante/logs/ but there's no built-in way to view them. After installing Vigilante and running vigilante setup + vigilante daemon run --once, I had to manually cat ~/.vigilante/logs/vigilante.log to see what happened. This adds the missing read path.

The project has invested in log quality over multiple PRs:

Source Evidence
#179 Scoped session log filenames by repository
#26 Local timezone in logs
#117 Reduced daemon log noise
#267 Expanded status with session health and rate limits

All that log infrastructure exists but has no viewer. vigilante logs completes the observability surface alongside vigilante list and vigilante status.

Changes

  • internal/app/app.go: Added runLogsCommand function with --repo and --issue flags, plus the case "logs" dispatch in runCommand and the usage string entry
  • internal/app/app_test.go: 3 test functions covering list, show, and missing-log error cases

Testing

  • go test ./... passes (all packages)
  • gofmt -l . clean
  • go vet ./... clean
  • Built locally and tested all 3 modes:

vigilante logs (list mode)

list

vigilante logs --help

help

vigilante logs --repo owner/repo --issue 999 (error case)

error

This contribution was developed with AI assistance (Claude Code).

Add a `logs` subcommand that lists session log files or shows a
specific session log filtered by --repo and --issue flags.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
@nicobistolfi nicobistolfi merged commit f32ee5b into aliengiraffe:main Mar 24, 2026
1 check passed
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.

2 participants