Skip to content

Program and connection events in the audit trail - #58

Merged
mayur-tolexo merged 1 commit into
mainfrom
feat/audit-trace-events
Nov 2, 2026
Merged

mayur-tolexo merged 1 commit into
mainfrom
feat/audit-trace-events

Conversation

@mayur-tolexo

Copy link
Copy Markdown
Collaborator

Why

The backend audit trail now records every program a sandbox runs and every connection it opens, not just the calls made to it (NeevCloudAI/neevaiplatform-backend#1822). Those arrive as OCSF Process Activity (1007) and Network Activity (4001) events with no api. Recording is on by default, so with the current models sandbox.audit() raises a validation error on the first program event.

What

  • Regenerated aiagent.py from the backend spec: class_uid and category_uid accept the new classes, api and src_endpoint are optional, and process, exit_code, dst_endpoint and actor.process are new.
  • The vendored spec keeps snapshot trigger optional as before, so responses from older API versions still parse.
  • The audit examples and README handle all three event kinds, the api inventory documents which fields each kind carries, and update documents execution_trace.
  • Tests validate a program and a connection event, plus execution_trace alone on update.

Merge after the backend PR.

The audit trail now carries every program a sandbox runs and every
connection it opens, not only the calls made to it. They arrive as OCSF
Process Activity (class 1007) and Network Activity (class 4001) events
beside the existing API Activity ones, so api and src_endpoint are now
absent on some events, and process, exit_code, dst_endpoint and
actor.process are new.

Regenerated from the aiagent spec. The examples and README no longer
assume every event has api, and sandbox update documents execution_trace,
which turns program recording off. The vendored spec keeps snapshot
trigger optional, so responses from older API versions still parse.

Signed-off-by: Mayur Das <mayur.das@neevcloud.com>
@coderabbitai

coderabbitai Bot commented Oct 19, 2026 •

Copy link
Copy Markdown

Important

  • Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 895a163e-64d7-43a0-8701-fa6302f5bc3c
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

@mayur-tolexo
mayur-tolexo marked this pull request as ready for review November 2, 2026 13:05
@mayur-tolexo
mayur-tolexo merged commit 0ec51be into main Nov 2, 2026
5 checks passed
mayur-tolexo added a commit that referenced this pull request Nov 2, 2026
Conflicts with #58 resolved to this branch, which matches the current
aiagent API (working_directory, device, OCSF actor captions). Keeps the
auto-resume examples from #60, and updates the step-by-step README's
expected output for the two audit examples to what they now print.

Signed-off-by: Mayur Das <mayur.das@neevcloud.com>
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