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
Documentation-style tooling only; no runtime, auth, or application logic changes.
Overview
Adds a root justfile so contributors can run standard OpenEnv workflows via just instead of remembering long shell commands.
It sets PYTHONPATH=src:envs, defines just install (uv sync --all-extras), just test (pytest with the same env ignores and -m "not integration and not network and not docker" filters used for local/CI-friendly runs), and just test-one <target> for a single file or pattern. The default recipe lists available commands.
Reviewed by Cursor Bugbot for commit 88f6121. 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.
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
Documentation-style tooling only; no runtime, auth, or application logic changes.
Overview
Adds a root
justfileso contributors can run standard OpenEnv workflows viajustinstead of remembering long shell commands.It sets
PYTHONPATH=src:envs, definesjust install(uv sync --all-extras),just test(pytest with the same env ignores and-m "not integration and not network and not docker"filters used for local/CI-friendly runs), andjust test-one <target>for a single file or pattern. The default recipe lists available commands.Reviewed by Cursor Bugbot for commit 88f6121. Bugbot is set up for automated code reviews on this repo. Configure here.