Repository navigation
Conversation
rix1337
marked this pull request as ready for review
October 19, 2026 09:21
Codex Review SummaryThis 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. |
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
The DOX hierarchy required agents to read and maintain nested instruction files, much of which repeated information available in the code. Replace its 14 child files with one root
AGENTS.mdand 25 focused Trail learnings that preserve architectural decisions, reasons, and constraints.Agents must use Trail first and stop with an installation command if it is missing. The root rules retain the ban on real source hostnames and require running pre-commit and accepting its edits. Those two rules are also learnings 01 and 02. Documentation policy moves into a learning, including native Trail headings and preserving committed decisions when a later note supersedes them.
Add a privacy check before pre-commit can stage files. It checks both working and staged Trail content for domains, credential assignments, personal author metadata, absolute paths, unsafe references, symlinks, and tracked caches. Shared notes use a generic author and repository-relative references. Generated maps stay gitignored and are built locally; collaboration shares the learnings through Git.
Update CONTRIBUTING with the setup and verification commands, without pinning the Trail version. Keep the dependency refresh and version bump to 4.7.6 produced by pre-commit. No user configuration or source-hostname changes are required.
Validation:
uv run python -X utf8 pre-commit.py: lint, formatting, and all 493 tests pass.uv run python check_trail.pyandtrail check --base origin/mainpass.