Skip to content

chore: periodic maintenance — spec, deps, fixes#35

Merged
chaliy merged 1 commit intomainfrom
claude/check-reqwest-version-aCxEX
Feb 12, 2026
Merged

chore: periodic maintenance — spec, deps, fixes#35
chaliy merged 1 commit intomainfrom
claude/check-reqwest-version-aCxEX

Conversation

@chaliy
Copy link
Contributor

@chaliy chaliy commented Feb 12, 2026

Summary

  • New spec: specs/maintenance.md — defines periodic maintenance checklist (deps, OpenAPI sync, feature parity, docs, cookbooks, security)
  • Security fix: Update bytes 1.11.0 → 1.11.1 (RUSTSEC-2026-0007, integer overflow in BytesMut::reserve)
  • Python fix: Replace deprecated class Config with model_config = ConfigDict(...) in Event model (Pydantic V2 migration)
  • OpenAPI sync: Update openapi/openapi.json from upstream — 14 new endpoints (tool-results, databases, storage, images CRUD, orgs, agent preview)
  • Spec update: specs/api-surface.md now documents all new endpoints with covered/not-covered categorization
  • TypeScript fix: Add missing session.delete() and session.cancel() methods for feature parity with Rust/Python

Test plan

  • just pre-pr passes (all lints + 128 tests across Rust/Python/TypeScript)
  • just check-cookbook passes (all cookbooks compile)
  • cargo audit clean after bytes update
  • No Python deprecation warnings remain

https://claude.ai/code/session_01G1Loc3aEWWej7D9RVQHPAh

- Add specs/maintenance.md defining the periodic maintenance checklist
- Fix RUSTSEC-2026-0007: update bytes 1.11.0 -> 1.11.1 (integer overflow)
- Fix Python Pydantic deprecation: class Config -> model_config = ConfigDict
- Update openapi/openapi.json from upstream (new endpoints: tool-results,
  databases, storage, images, orgs)
- Update specs/api-surface.md with new API endpoints
- Add missing session.delete() and session.cancel() to TypeScript SDK

All 128 tests pass. All lints clean. All cookbooks compile.

https://claude.ai/code/session_01G1Loc3aEWWej7D9RVQHPAh
Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 57e0993c74

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +57 to +61
### Tool Results
- `POST /v1/sessions/{id}/tool-results` - Submit tool results

### Session Databases
- `GET /v1/sessions/{id}/databases` - List databases

Choose a reason for hiding this comment

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

P2 Badge Reclassify unimplemented session endpoints as not covered

These newly added entries are listed under Covered Endpoints, but none of the SDK clients implement calls to /sessions/{id}/tool-results, /sessions/{id}/databases*, or /sessions/{id}/storage* (checked rust/src/client.rs, python/everruns_sdk/client.py, and typescript/src/client.ts, which contain no routes for those paths). Keeping them marked as covered creates a false feature-parity signal and will mislead maintenance checks and users about available SDK functionality.

Useful? React with 👍 / 👎.

@chaliy chaliy merged commit 71c3ac8 into main Feb 12, 2026
10 checks passed
@chaliy chaliy deleted the claude/check-reqwest-version-aCxEX branch February 12, 2026 04:43
@chaliy chaliy mentioned this pull request Feb 12, 2026
8 tasks
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