Skip to content

merge: integrate main (storage_session_id contract alignment) into feat/agent-skills-runtime#7

Merged
djuillard merged 9 commits into
feat/agent-skills-runtimefrom
merge/main-into-feat-skills-2026-05-14
May 14, 2026
Merged

merge: integrate main (storage_session_id contract alignment) into feat/agent-skills-runtime#7
djuillard merged 9 commits into
feat/agent-skills-runtimefrom
merge/main-into-feat-skills-2026-05-14

Conversation

@djuillard
Copy link
Copy Markdown

Summary

Brings the freshly-synced main (post PR #6) into our long-lived feature branch feat/agent-skills-runtime. The merge direction is main → feat-branch, not the other way around — feat/agent-skills-runtime is the production branch (built into code-interpreter:agent-skills) and continues to live independently.

Closes the contract divergence with LibreChat dev (post-2026-05-12) reported in our 422 incident earlier today (see SECURITY-OPERATIONS.md §8.5 to be updated post-merge).

What's coming in (4 upstream commits via PR #6)

Conflict resolution

None. git merge origin/main ran clean. feat/agent-skills-runtime hadn't touched src/api/files.py, src/models/exec.py, or tests/integration/test_librechat_compat.py since the previous merge (PR #5 on 2026-05-08) — so the 22 files / +259 / -101 from PR #6 land here as a pure merge with no manual resolution.

Stats

  • 22 files changed, +259 insertions, -101 deletions
  • Merge commit: 6877fde

Context — superseded local WIP

We had started writing our own compat patch on fix/librechat-dev-2026-05-12-compat (4 commits, 13 integration tests) earlier today before realizing upstream PR usnavy13#110 had been merged 30 min before our commits were ready, covering the same ground with a wider scope (PTCFileInput, FileRef, GET /files query params on top of what we'd planned). That local branch is now abandoned in favour of this sync. The wider upstream patch is in this merge.

Test plan

The merged code requires a rebuild of code-interpreter:agent-skills to take effect. After rebuild:

  • code-interpreter-api starts cleanly, logs show Sandbox network access ENABLED and /opt/skill-deps created
  • Python DOCX/PPTX/XLSX skills still work (soffice invocation from Python sandbox — guard from our feat-branch /proc + bind syscall config)
  • Bash sandbox can also invoke soffice (LibreChat bash_tool compat)
  • Functional end-to-end: upload a file via LibreChat UI in a code-interpreter agent chat at https://chat-dev.onbehalf.ai — should no longer return the 422 we hit this morning
  • Optional: pytest tests/integration/test_librechat_compat.py (with -e API_KEY=test-api-key-for-testing-12345 env override) passes

Follow-up

Once merged, rebuild + restart code-interpreter-api and validate via the LibreChat UI. Then update SECURITY-OPERATIONS.md §8.5 with today's incident timeline + resolution.

🤖 Generated with Claude Code
via Happy

usnavy13 and others added 9 commits May 8, 2026 22:15
docs: Add Star History section to README.md
Updated the API responses and models to replace all instances of session_id with storage_session_id for consistency. This change includes updates to the upload, list, and batch file operations, as well as adjustments in the corresponding tests and documentation to reflect the new naming convention.
…support

Updated the upload_files_batch and list_files functions to include support for resource kind and version parameters. The upload logic now marks files as agent files based on the kind provided. Additionally, the RequestFile and FileRef models have been updated to accept resource_id, kind, and version fields, with corresponding tests added to validate these changes.
fix: Align API contract with LibreChat client (storage_session_id, kind, resource fields)
fix: Align API contract with LibreChat client (storage_session_id, kind, resource fields)
sync: merge usnavy13/main (4 commits, storage_session_id contract alignment)
@djuillard djuillard merged commit 936281d into feat/agent-skills-runtime May 14, 2026
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