Skip to content

openclaw adapter#5

Merged
rcholic merged 4 commits intomainfrom
openclaw_adapter
Feb 26, 2026
Merged

openclaw adapter#5
rcholic merged 4 commits intomainfrom
openclaw_adapter

Conversation

@rcholic
Copy link
Contributor

@rcholic rcholic commented Feb 26, 2026

No description provided.

rcholic and others added 4 commits February 25, 2026 18:25
The pyproject.toml already has B404 (subprocess import) in the skips
list, but the GitHub Actions workflow wasn't using the config file.
Added -c pyproject.toml flag to the bandit command.

This resolves the B404 warning which is a low-severity issue about
importing the subprocess module. The import is safe and necessary for
managing OpenClaw CLI subprocess lifecycle.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
- Fixed mypy error in openclaw_adapter.py:125 by casting dict value to str
- Fixed flake8 F541 in openclaw_browser_automation.py (removed f-string without placeholder)
- Fixed flake8 F841 in openclaw_browser_automation.py (unused result variable)
- Applied isort formatting

All pre-commit hooks now pass:
- isort, flake8, mypy, bandit, black, pyupgrade
- All 127 tests passing

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@rcholic rcholic merged commit df84aa3 into main Feb 26, 2026
3 checks passed
@rcholic rcholic deleted the openclaw_adapter branch February 26, 2026 02:40
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.

1 participant