You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This PR adds a justfile for dependency installation, the test suite, and individual test targets.
Note
Low Risk
Developer tooling only; no runtime, auth, or production code paths change.
Overview
Adds a root justfile so local dev can use just install, just test, and just test-one <path> instead of copying long shell commands from CI or agent docs.
It sets PYTHONPATH=src:envs for recipes and aligns just test with the default CI/local hook behavior: same pytest ignores for browsergym, dipg, and websearch env tests, plus marker filter not integration and not network and not docker. just with no args lists available recipes.
Reviewed by Cursor Bugbot for commit 0a116c8. Bugbot is set up for automated code reviews on this repo. Configure here.
Wraps the commands already documented in CLAUDE.md (install, test, lint,
format, docs-sync) so they are discoverable via `just --list` and harder
to get subtly wrong. `just test` mirrors the CI invocation, including the
same --ignore set and marker filter.
Independent of the RFC 005 work; split out so it can land or be dropped
on its own.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
The reason will be displayed to describe this comment to others. Learn more.
Stale comment
Reviewed at c3c04f884500e775512795bc08f5655da7038074. The test recipe currently matches the repository workflow, but the PR body promises lint, format, and docs-sync recipes that are absent from the 23-line justfile, and the file lacks a final newline. Please either add the claimed recipes or narrow the description, add the newline, refresh onto current main, and run exact-head CI (only Bugbot has run). No runtime or alignment concern otherwise.
The reason will be displayed to describe this comment to others. Learn more.
APPROVE at 508d8ef1
Useful ergonomic justfile. The test recipe matches the CLAUDE.md CI-safe invocation (same three --ignore flags; file-level PYTHONPATH propagates). Not wheel cargo.
The reason will be displayed to describe this comment to others. Learn more.
APPROVE at 3fc52f15 (merge-main only)
Rebased onto main after #1178. Feature tip unchanged from prior review at 58f92f7f (justfile install/test/test-one; final newline fixed). Non-blocking nit still: test-one comment names a stale test_agentic_harness_types.py path.
Root tooling only; not release cargo. Still needs exact-head repository CI.
The reason will be displayed to describe this comment to others. Learn more.
APPROVE @ ac5a7ec5
Addresses the prior nit: test-one example now points at the real tests/core/test_state_schema_subclass.py. Recipes still match the narrowed install/test/test-one scope.
Tooling-only; not 0.6.0 wheel cargo. Exact-head repository CI still needed before merge if workflows are gated.
The reason will be displayed to describe this comment to others. Learn more.
Re-APPROVE at 31bdf238 (merge-main only)
Ben merged main (26c9465e / #1179) into this fork PR. Stable feature patch-id is unchanged vs prior approved tip fd2eea6b (ef1dafed…); justfile blob identical.
Prior technical approval stands. Still needs maintainer Approve and run. Not 0.6.0 wheel cargo.
The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
This PR adds a justfile for dependency installation, the test suite, and individual test targets.
Note
Low Risk
Developer tooling only; no runtime, auth, or production code paths change.
Overview
Adds a root
justfileso local dev can usejust install,just test, andjust test-one <path>instead of copying long shell commands from CI or agent docs.It sets
PYTHONPATH=src:envsfor recipes and alignsjust testwith the default CI/local hook behavior: same pytest ignores for browsergym, dipg, and websearch env tests, plus marker filternot integration and not network and not docker.justwith no args lists available recipes.Reviewed by Cursor Bugbot for commit 0a116c8. Bugbot is set up for automated code reviews on this repo. Configure here.