Skip to content

feat: e2e event-driven code-review agent (TLS + NATS)#33

Merged
tgarciai merged 1 commit intomainfrom
feat/e2e-event-driven-agent
Mar 18, 2026
Merged

feat: e2e event-driven code-review agent (TLS + NATS)#33
tgarciai merged 1 commit intomainfrom
feat/e2e-event-driven-agent

Conversation

@tgarciai
Copy link
Member

Summary

New e2e test proving the core underpass-ai reactive loop:

NATS event → agent wakes up → runtime session (HTTPS/TLS) →
  tool discovery (70 tools) → write code → analyze → write review →
    NATS completion event

Use case: A code-review-agent receives task.assigned via NATS. It creates a session on the runtime over TLS, writes Go source code with a real bug (unchecked os.ReadFile error), reads and analyzes it, writes a REVIEW.md with 3 findings and REQUEST CHANGES verdict, then publishes task.completed back to NATS.

  • 5 tool invocations over HTTPS (fs.write_file, fs.read_file, fs.list)
  • NATS pub/sub round-trip verified
  • 10 steps, all passed

Cluster evidence

OK Subscribed to workspace.task.completed
OK Published task pr-review-6c71cdcd
OK Session: session-938fbe150433a867
OK Discovered 70 tools
OK handlers.go written (752 bytes)
OK Code analyzed: 752 bytes, unchecked-error bug=found
OK REVIEW.md written (1341 bytes)
OK Workspace: 2 files — handlers.go=yes, REVIEW.md=yes
OK Review verified: 1341 bytes, verdict=REQUEST CHANGES
OK NATS round-trip verified: task_id=pr-review-6c71cdcd
OK Event-Driven Agent E2E PASSED — 5 invocations over TLS

Test plan

  • Ran on live cluster with tls.mode=server (rev 23)
  • NATS running in namespace
  • Pod Completed, job succeeded
  • No sensitive data in test code

🤖 Generated with Claude Code

New e2e test (12) proving the core underpass-ai loop:
NATS event → agent activates → runtime session (HTTPS/TLS) →
tool discovery → write code → analyze → write review → NATS result.

Use case: code-review-agent receives task.assigned, writes Go source
with a known bug (unchecked os.ReadFile error), produces REVIEW.md
with 3 findings and REQUEST CHANGES verdict, publishes task.completed.

5 tool invocations over TLS, NATS pub/sub round-trip verified.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@sonarqubecloud
Copy link

@tgarciai tgarciai merged commit 3e8f989 into main Mar 18, 2026
9 checks 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.

1 participant