Skip to content

feat(cli): add log viewer modal to dashboard (F34)#76

Merged
wendeus0 merged 5 commits into
mainfrom
feature/F34-dashboard-logs
Mar 13, 2026
Merged

feat(cli): add log viewer modal to dashboard (F34)#76
wendeus0 merged 5 commits into
mainfrom
feature/F34-dashboard-logs

Conversation

@wendeus0
Copy link
Copy Markdown
Owner

This PR implements Feature F34 (Dashboard Logs), adding the ability to view step execution logs directly in the TUI dashboard.

Scope

  • Component: LogViewer (ModalScreen) in src/aignt_os/cli/dashboard.py
  • Interaction: Enter key on step list opens the modal
  • Logic: Reads logs from clean_output_path or raw_output_path

Validation

  • Unit tests: tests/unit/test_dashboard_logic.py (logic verification)
  • Full regression suite: 402 tests passed
  • Security review: File reading restricted to persisted paths

See RUN_REPORT.md for full delivery details.

GitHub Copilot and others added 5 commits March 13, 2026 04:00
- Implements 'aignt runs watch <run_id>' command.
- Adds Textual-based TUI for real-time run monitoring.
- Updates dependencies to include 'textual'.
- Adds unit tests for the watch command.
- Validated manually and via unit tests.
- Security review completed (see RUN_REPORT.md).

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Replaces simple DataTable with a split-pane layout (Sidebar + Content).
- Adds status icons and semantic colors.
- Implements interactive step selection for details.
- Updates RUN_REPORT.md with new features.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Implements LogViewer modal in TUI dashboard
- Adds action_show_logs bound to Enter key
- Adds unit tests for log viewer logic
- Updates RUN_REPORT.md with delivery details

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@wendeus0 wendeus0 merged commit 438aba0 into main Mar 13, 2026
5 of 6 checks passed
@wendeus0 wendeus0 deleted the feature/F34-dashboard-logs branch March 13, 2026 08:12
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