Skip to content

ax: implement monitor command and conversations API#74

Merged
anj-s merged 3 commits into
mainfrom
u/anj/observability-pr4
Jun 17, 2026
Merged

ax: implement monitor command and conversations API#74
anj-s merged 3 commits into
mainfrom
u/anj/observability-pr4

Conversation

@anj-s

@anj-s anj-s commented Jun 10, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • AX Monitor CLI Command: Added the monitor command to the ax CLI, registered in cmd/ax/main.go.
  • Backend API: Implemented a local HTTP API endpoint /api/conversations that queries the SQLite database to fetch and summarize AX conversation histories.
  • SQLite Parsing Fixes: Handled sqlite timestamps by scanning them as strings and parsing them manually in Go to avoid scan errors when using SQLite aggregations.
  • Unit Tests: Added cmd/ax/monitor_test.go to verify correctness of conversation fetching logic.

Type of Change

  • New feature (non-breaking change which adds functionality)
  • Bug fix (non-breaking change which fixes an issue)
  • Refactor
  • Documentation
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Related Issues

Part 4 of the Observability & Monitoring epic. Stacked on top of PR #70.

Test Plan

  • Automated Tests: Unit tests pass:
    go test -v ./cmd/ax
  • Manual Verification: Verified that command compilation works both with and without the harness build tag:
    go build -o bin/ax ./cmd/ax
    go build -tags harness -o bin/ax_harness ./cmd/ax

Notes for Reviewer

This is the fourth of 5 planned modular PRs to roll out comprehensive AX observability.
It exposes a backend JSON API for conversation lists.

@anj-s anj-s mentioned this pull request Jun 10, 2026
5 tasks
Base automatically changed from u/anj/observability-pr3 to main June 10, 2026 21:26
@anj-s anj-s force-pushed the u/anj/observability-pr4 branch from e6911cb to d801b2e Compare June 13, 2026 04:29
@anj-s anj-s marked this pull request as ready for review June 13, 2026 04:41
@anj-s anj-s requested a review from wjjclaud June 13, 2026 04:42
@rakyll

rakyll commented Jun 15, 2026

Copy link
Copy Markdown
Member

See my comment at #75. We shouldn't build this in AX.

@anj-s anj-s requested a review from rakyll June 16, 2026 16:53

@rakyll rakyll left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We chatted offline that we need an intermediate solution for introspection when developing.

Let's call this "dashboard", otherwise LGTM.

@anj-s anj-s force-pushed the u/anj/observability-pr4 branch from 590a81d to 9f2530b Compare June 17, 2026 22:08
@anj-s anj-s merged commit 87ca760 into main Jun 17, 2026
7 checks passed
@anj-s anj-s deleted the u/anj/observability-pr4 branch June 17, 2026 22:21
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