[kernel 1116] browser logging: cdp pipeline by archandatta · Pull Request #194 · kernel/kernel-images

archandatta · 2026-04-01T13:26:28Z

Scaffolding of the CDP Pipeline implementation on the functions are in -> #202

Note

Medium Risk
Adds new capture-session lifecycle endpoints and introduces a new in-process event pipeline with file/ring-buffer writers, plus new shutdown/constructor wiring; concurrency and lifecycle handling are the main risk areas. CDP monitor is currently scaffolded, but the new API surface and persistence paths still warrant careful review.

Overview
Adds an event capture session feature, exposing GET/POST/PATCH/DELETE /events/capture_session to start/stop and reconfigure capture (including category filtering) and report session id, status, seq, and created_at.

Wires a new events.CaptureSession and cdpmonitor.Monitor into ApiService/main, including lifecycle cancellation and shutdown cleanup, and refactors the events pipeline to be reusable (Start/Stop), category-filtered, and backed by a validated/resettable ring buffer plus a filename-based JSONL writer.

Regenerates OpenAPI (oapi.go, openapi.yaml) and updates/extends tests to cover the new capture session handlers and the updated events/ring buffer/file writer behaviors.

^{Reviewed by Cursor Bugbot for commit 33c07d3. Bugbot is set up for automated code reviews on this repo. Configure here.}

…ies and fix delimiter in CategoryFor

hiroTamada · 2026-04-07T13:45:54Z

 }

 func (s *ApiService) Shutdown(ctx context.Context) error {
+	s.monitorMu.Lock()


Is this going to add a latency in delete browsers request? If so, how much?

Shouldn't add any latency, shutdown holds the lock briefly to stop the CDP monitor and close the capture session i think both are fast synchronous teardown calls

rgarcia · 2026-04-07T15:09:28Z

+
+	s.captureSession.Start(uuid.New().String())
+
+	if err := s.cdpMonitor.Start(context.Background()); err != nil {


the comment says this restarts an existing monitor, but the handler never checks/stops an already-running one. might be worth making that behavior explicit here instead of relying on Start() side effects.

Ah yeah this was just a scaffolding I did for the actual implementation. I should've mentioned that there is a chained PR #202 that is capturing this https://github.com/kernel/kernel-images/pull/202/changes#diff-0ddb00e75e5f21a1deb16fb852c7f352f234bbf0b9ddcc3d11639b1d3293c572R90

rgarcia · 2026-04-07T15:09:28Z

+// Generates a new capture session ID, seeds the pipeline, then starts the
+// CDP monitor. If already running, the monitor is stopped and
+// restarted with a fresh session ID
+func (s *ApiService) StartCapture(w http.ResponseWriter, r *http.Request) {


question: is a no-arg /events/start intentional? i'd expect callers to want some capture config here over time (categories, detail level, screenshot/network verbosity, etc.), or at least an explicit note that this is a fixed server-defined profile for now.

yeah good point, added a body field to be able to send something like

{ "categories": ["console", "network"], "detailLevel": "verbose" }

rgarcia · 2026-04-07T15:09:28Z

+}
+
+// New creates a Monitor. displayNum is the X display for ffmpeg screenshots.
+func New(_ UpstreamProvider, _ PublishFunc, _ int) *Monitor {


i don't think we'll need displayNum for a pure CDP monitor, so i'd remove this until actual implementation lands.

this is handled in #202

…structor

… and category validation

rgarcia

api binary needs to be removed

This binary is tracked on main and was incidentally deleted earlier on this branch. Restoring it keeps the 13.4MB binary out of this PR's diff. Removing the tracked binary from main should be done in a separate PR.

rgarcia

one small thing but otherwise i think this is good to go!

rgarcia · 2026-04-09T19:20:09Z

+		return oapi.StartCaptureSession400JSONResponse{BadRequestErrorJSONResponse: oapi.BadRequestErrorJSONResponse{Message: err.Error()}}, nil
+	}
+
+	id := uuid.New().String()


let's use github.com/nrednav/cuid2 to be consistent with the rest of the api (and remove format: uuid for this id field in the openapi.yaml spec)

Sayan-

I think the last bugbot comment is worth resolving. Thanks for iterating - interfaces, components, and integration are looking great!

cursor

Cursor Bugbot has reviewed your changes and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit a4fd0d6. Configure here.}

archandatta commented Apr 2, 2026

View reviewed changes

Comment thread server/cmd/api/api/api.go Outdated

archandatta marked this pull request as ready for review April 2, 2026 15:19

cursor Bot reviewed Apr 2, 2026

View reviewed changes

Comment thread server/cmd/api/api/events.go Outdated

Comment thread server/lib/events/event.go Outdated

Base automatically changed from archand/kernel-1116/browser-logging to main April 2, 2026 15:34

archandatta force-pushed the archand/kernel-1116/cdp-pipeline branch from 1e544a7 to 6d929c8 Compare April 2, 2026 17:39

feat: add CDP pipeline, cdpmonitor, and wire into API service

76d837f

archandatta force-pushed the archand/kernel-1116/cdp-pipeline branch from 6d929c8 to 76d837f Compare April 2, 2026 17:44

cursor Bot reviewed Apr 2, 2026

View reviewed changes

Comment thread server/lib/events/event.go Outdated

archandatta added 2 commits April 2, 2026 18:23

refactor: rename Pipeline to CaptureSession and delete pipeline.go

5c0f7ae

review: fix request context leak in StartCapture, add missing categor…

ad2dd63

…ies and fix delimiter in CategoryFor

cursor Bot reviewed Apr 2, 2026

View reviewed changes

Comment thread server/cmd/api/api/events.go Outdated

Comment thread server/lib/events/ringbuffer.go Outdated

review: clean up syntax

365787c

archandatta requested review from Sayan-, hiroTamada and rgarcia April 6, 2026 12:43

hiroTamada reviewed Apr 7, 2026

View reviewed changes

Comment thread server/cmd/api/main.go Outdated

rgarcia requested changes Apr 7, 2026

View reviewed changes

archandatta added 5 commits April 7, 2026 18:00

review: move CategoryFor to cdpmonitor package

d086571

review: internalize ring buffer and file writer in CaptureSession con…

3339cac

…structor

review: write logs under /var/log/kernel and ensure dir exists

64e67b1

review: add capture config to /events/start and OpenAPI spec

048f9c6

review: fix lifecycle context, stop-before-reset ordering, seq reset,…

e57e58a

… and category validation

cursor Bot reviewed Apr 7, 2026

View reviewed changes

Comment thread server/lib/oapi/oapi.go Outdated

Comment thread server/lib/oapi/oapi.go

Comment thread server/cmd/api/api/api.go

archandatta added 2 commits April 7, 2026 18:39

fix: oapi version

886b893

fix: Shutdown cancels context outside lock, racing with StartCapture

06d2470

archandatta requested review from hiroTamada and rgarcia April 7, 2026 18:42

cursor Bot reviewed Apr 7, 2026

View reviewed changes

Comment thread server/cmd/api/api/events.go Outdated

review: validate DetailLevel with generated Valid

6bb5402

rgarcia requested changes Apr 9, 2026

View reviewed changes

archandatta added 6 commits April 9, 2026 15:03

review: update file writer to be internal to the package

c6dd362

review: tighten to Write(filename string, data []byte) error

ff8bddf

review: update panic -> error

a8bdeaf

review: update oapi and remove detail level

8f88ed0

review: remove url

3eaacb3

chore: restore server/api on branch

214858a

This binary is tracked on main and was incidentally deleted earlier on this branch. Restoring it keeps the 13.4MB binary out of this PR's diff. Removing the tracked binary from main should be done in a separate PR.

cursor Bot reviewed Apr 9, 2026

View reviewed changes

Comment thread server/lib/cdpmonitor/monitor.go

archandatta added 3 commits April 9, 2026 17:07

review: harden captureConfigFromOAPI and clarify stop comment

b06132a

review: unexport ringBuffer and drop AllCategories wrapper

8ebb5e3

review: replace event producers with cdp monitor in stop description

4bcba48

cursor Bot reviewed Apr 9, 2026

View reviewed changes

Comment thread server/cmd/api/api/api.go

remove test line

1295232

archandatta requested a review from rgarcia April 9, 2026 17:23

cursor Bot reviewed Apr 9, 2026

View reviewed changes

Comment thread server/cmd/api/main.go

rgarcia approved these changes Apr 9, 2026

View reviewed changes

Sayan- approved these changes Apr 9, 2026

View reviewed changes

review: update uuid to cuid2

a4fd0d6

cursor Bot reviewed Apr 10, 2026

View reviewed changes

Comment thread server/cmd/api/api/capture_session.go

archandatta added 3 commits April 10, 2026 15:22

Merge branch 'main' into archand/kernel-1116/cdp-pipeline

d6b348b

Merge branch 'main' into archand/kernel-1116/cdp-pipeline

3da65c3

fix naming

a7b2e54

archandatta removed the request for review from hiroTamada April 21, 2026 18:45

Merge branch 'main' into archand/kernel-1116/cdp-pipeline

33c07d3

archandatta requested review from Sayan-, hiroTamada and rgarcia April 21, 2026 20:50

Sayan- approved these changes Apr 21, 2026

View reviewed changes

archandatta merged commit 92fdcbd into main Apr 22, 2026
10 checks passed

archandatta deleted the archand/kernel-1116/cdp-pipeline branch April 22, 2026 17:41


		s.captureSession.Start(uuid.New().String())

		if err := s.cdpMonitor.Start(context.Background()); err != nil {

Conversation

archandatta commented Apr 1, 2026 • edited by cursor Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

rgarcia left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

rgarcia left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Sayan- left a comment

Choose a reason for hiding this comment

Uh oh!

cursor Bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

archandatta commented Apr 1, 2026 •

edited by cursor Bot

Loading