Add complete third-party license notices for all dependencies#18
Add complete third-party license notices for all dependencies#18
Conversation
- Rewrite README: library-first, code execution focus, no AI slop
- Add versioned API endpoints (/v1/execute, /v1/sessions, /v1/browser)
- Add deprecated wrappers for old endpoints with Deprecation header
- Add /health endpoint
- Create Python client library (coderunner package)
- Add pyproject.toml for pip installability
Breaking changes (deprecated, still work):
- /execute -> /v1/execute
- /v1/sessions/session -> /v1/sessions, /v1/sessions/{id}
- /v1/browser/interactions/* -> /v1/browser/*
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Tests: - Add 25 unit tests for client library - Add parsing tests for skill frontmatter - All tests pass without requiring running server Container improvements: - Add SIGTERM/SIGINT handler for graceful kernel shutdown - Add 10-minute timeout to container system start in install.sh - Improve cleanup.sh with graceful stop (no aggressive pkill) - Simplify Homebrew install command (auto-tap syntax) 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Adds legally required attribution notices for all MIT, BSD, PSF, MPL-2.0, and LGPLv3 dependencies from requirements.txt. Apache-2.0 dependencies are omitted since the project itself is Apache-2.0 licensed. Grouped by license type: - MIT License (12 components) - MPL-2.0 AND MIT (tqdm) - BSD 3-Clause (8 components) - BSD 2-Clause (3 components, was 2) - BSD unspecified (10 components) - Python Software Foundation License (matplotlib) - LGPLv3 (img2pdf) - Existing: GPLv3 (FFmpeg), MIT-CMU (Pillow), SIL OFL (fonts) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request introduces a Python client library and CLI for CodeRunner, rewrites the README for better clarity, and enhances the server with graceful shutdown logic, health checks, and session management. It also adds a comprehensive test suite and support for running Claude via Microsoft LiteBox. Feedback includes correcting a typo in the .gitignore file, removing unused imports in server.py, and adding a missing trailing newline at the end of the server script.
| third_party/ | ||
|
|
||
| cleaup.sh No newline at end of file | ||
| cleaup.sh |
There was a problem hiding this comment.
The filename in the ignore list contains a typo. It should be cleanup.sh to match the actual script name in the repository.
cleanup.sh
| import signal | ||
| import sys | ||
| import zipfile |
There was a problem hiding this comment.
The signal and sys modules are imported but not used anywhere in this script. Removing unused imports is recommended to maintain code cleanliness and reduce overhead.
| import signal | |
| import sys | |
| import zipfile | |
| import zipfile |
| app.add_route("/v1/sessions/session", deprecated_session_get, methods=["GET"]) | ||
| app.add_route("/v1/sessions/session", deprecated_session_delete, methods=["DELETE"]) | ||
| app.add_route("/v1/browser/interactions/navigate", deprecated_browser_navigate, methods=["POST"]) | ||
| app.add_route("/v1/browser/interactions/content", deprecated_browser_content, methods=["POST"]) No newline at end of file |
There was a problem hiding this comment.
The file is missing a newline at the end, which is recommended by PEP 8 for Python files to ensure compatibility with various tools and to follow standard formatting practices.
| app.add_route("/v1/browser/interactions/content", deprecated_browser_content, methods=["POST"]) | |
| app.add_route("/v1/browser/interactions/content", deprecated_browser_content, methods=["POST"]) | |
|
Need just the 3rd party license commit cherry picked. Not others. |
2 participants
Summary
Adds legally required attribution notices for all dependencies in
requirements.txttoTHIRD_PARTY_NOTICES.md.What's new
What's unchanged
What's excluded