Repository navigation
Clarify the judgment-layer narrative and shorten README - #262
Conversation
Codex Review Summary: ✅ CompletedThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 3a59ee2935
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
|
||
| Want a managed Hosted Proofpress session and to work with us as a design | ||
| partner? [Tell us about your workflow](https://ancient-ball-940.notion.site/eacf21eef9b54c3287f72892cd024a1c?pvs=105). | ||
| ## Explore |
There was a problem hiding this comment.
Keep the README rewrite in sync with the CI contract
The repository's CI runs python -m unittest discover -s tests -v, and this rewrite now fails tests/test_quickstart.py::test_readme_keeps_legacy_discoverable_without_version_churn: line 145 still requires the ## Compatibility section, while lines 154–159 also require both architecture image references. Either retain those README elements or update the test in the same change so every PR is not blocked by a deterministic failure.
Useful? React with 👍 / 👎.
| uv tool install --with "mcp>=2,<3" "git+https://github.com/chenmingtang830/proofpress.git" | ||
| uv tool update-shell | ||
| ``` | ||
|
|
||
| Open a new terminal if `proofpress` is not found, then run: | ||
|
|
||
| ```sh | ||
| proofpress quickstart --ui |
There was a problem hiding this comment.
Restart the shell before invoking the installed command
On a first-time installation where uv's tool directory is not already on PATH, this code block reaches proofpress quickstart --ui in the same shell and fails with command-not-found. uv tool update-shell writes shell startup files and explicitly reports “Restart your shell to apply changes”; restore the block break and restart instruction before running proofpress, or invoke the installed executable through a path that works immediately.
Useful? React with 👍 / 👎.
Update the repository introduction to “the judgment layer for agents” and reduce the README to a concise product overview, one consequential judgment example, quick start, and documentation links. Move the reading path to existing detailed guides; preserve all existing history metadata and comment anchors.
Validation: local documentation links, preserved history anchors, and git diff --check. No runtime code changes.