[PR #384] [CLOSED] docs(api): clarify inclusive cursor semantics for sandbox logs

bsakaguchi commented

2026-10-29 14:22:57 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/kvcache-ai/AgentENV/pull/384
Author: @zlzGithub-0801
Created: 10/14/2026
Status: ❌ Closed

Base: main ← Head: fix/issue-353-desc

📝 Commits (1)

fab5a38 docs(api): clarify inclusive cursor semantics for sandbox logs

📊 Changes

2 files changed (+5 additions, -2 deletions)

View changed files

📝 src/api/generated/src/models.rs (+1 -1)
📝 src/api/openapi.yml (+4 -1)

📄 Description

What

Document inclusive cursor semantics and possible boundary log repetition in the logs API.

Why

Clients need to know that reusing the last log’s timestamp may return boundary logs again.

Closes #353

Scope and non-goals

Documentation and generated comments only; pagination behavior remains unchanged.

Design and behavior changes

N/A

Compatibility and operations

Public API or generated protocol: Description only; no contract changes.
Configuration or defaults: N/A
Snapshot manifest, artifact layout, or storage format: N/A
Upgrade and rollback: N/A
Host requirements, permissions, ports, or dependencies: N/A

Validation

make fmt
make clippy
make test-unit
Relevant Rust integration tests
make -C services test (required when services/ changes)
Generated clients/server regenerated with the documented make target
Documentation updated
Benchmarks or performance comparison completed

Risks and reviewer notes

Existing E2B-compatible behavior is preserved; duplicate or missing logs during pagination are not fixed.

Checklist

The PR contains one coherent change and no unrelated formatting or refactoring.
New behavior is covered by tests, or I explained why testing is impractical.
Logs and examples contain no credentials, tokens, or private registry information.
I did not manually edit generated code without updating its source and regenerating it.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/kvcache-ai/AgentENV/pull/384 **Author:** [@zlzGithub-0801](https://github.com/zlzGithub-0801) **Created:** 10/14/2026 **Status:** ❌ Closed **Base:** `main` ← **Head:** `fix/issue-353-desc` --- ### 📝 Commits (1) - [`fab5a38`](https://github.com/kvcache-ai/AgentENV/commit/fab5a3857c3aa70317ff0e0f6125eb8a4e3f1d56) docs(api): clarify inclusive cursor semantics for sandbox logs ### 📊 Changes **2 files changed** (+5 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `src/api/generated/src/models.rs` (+1 -1) 📝 `src/api/openapi.yml` (+4 -1) </details> ### 📄 Description ## What Document inclusive cursor semantics and possible boundary log repetition in the logs API. ## Why Clients need to know that reusing the last log’s timestamp may return boundary logs again. ## Related issue Closes #353 ## Scope and non-goals Documentation and generated comments only; pagination behavior remains unchanged. ## Design and behavior changes N/A ## Compatibility and operations - Public API or generated protocol: Description only; no contract changes. - Configuration or defaults: N/A - Snapshot manifest, artifact layout, or storage format: N/A - Upgrade and rollback: N/A - Host requirements, permissions, ports, or dependencies: N/A ## Validation - [ ] `make fmt` - [ ] `make clippy` - [ ] `make test-unit` - [ ] Relevant Rust integration tests - [ ] `make -C services test` (required when `services/` changes) - [x] Generated clients/server regenerated with the documented `make` target - [x] Documentation updated - [ ] Benchmarks or performance comparison completed ## Risks and reviewer notes Existing E2B-compatible behavior is preserved; duplicate or missing logs during pagination are not fixed. ## Checklist - [x] The PR contains one coherent change and no unrelated formatting or refactoring. - [x] New behavior is covered by tests, or I explained why testing is impractical. - [x] Logs and examples contain no credentials, tokens, or private registry information. - [x] I did not manually edit generated code without updating its source and regenerating it. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

bsakaguchi