-
Notifications
You must be signed in to change notification settings - Fork 33
dev v1.16.0 #51
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
SihaoLiu
wants to merge
61
commits into
main
Choose a base branch
from
dev
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
dev v1.16.0 #51
Changes from all commits
Commits
Show all changes
61 commits
Select commit
Hold shift + click to select a range
29a528f
Add methodology analysis phase for RLCR loop exit
SihaoLiu d3ded24
Fix methodology analysis completion: require report and fail closed
SihaoLiu 7282f6e
Fix read validator to allow historical round access during methodolog…
SihaoLiu 50e8bbb
Harden methodology analysis read bypass with path canonicalization an…
SihaoLiu f34d45f
Enforce read-only project files during methodology analysis phase
SihaoLiu 87687a2
Fix spawned agent bypass and broaden bash restriction coverage
SihaoLiu 9888044
Fix realpath for non-existent files and narrow gh allowlist
SihaoLiu 4c48209
Block project-root reads and interpreters during methodology analysis
SihaoLiu 786ed32
Block shell entry points and fix concurrent session binding in method…
SihaoLiu f2a12c6
Allow cancel script during methodology analysis, remove gh issue earl…
SihaoLiu 7b91c47
Scope methodology fallback to spawned agents only, add missing git su…
SihaoLiu 3c85332
Remove cross-session methodology fallback, add realpath raw path fall…
SihaoLiu 5c123c2
Expand methodology bash blocklist, require non-empty report, fix moni…
SihaoLiu 03ac7d0
Tighten cancel allowlist, document spawned agent limitation, remove d…
SihaoLiu 5a08329
Remove raw records from read allowlist, document bash read limitation
SihaoLiu 2cba950
Fix stale error messages in read validator methodology analysis block
SihaoLiu 3eed7d6
Add missing template for methodology analysis state file block message
SihaoLiu 0edbe3f
Merge dev: resolve conflicts with yolo/skip-quiz flags
SihaoLiu 13a47fb
Fix cancel allowlist regex and move methodology handler before git-clean
SihaoLiu 9b961a8
Harden RLCR against mainline drift
tastynoob cfc46d6
Use portable shebang across all shell scripts
Emin017 4303249
Bump version to 1.14.1 for branch CI validation
Emin017 4deb1db
Harden template-loader error handling and reduce code duplication
zevorn 0844648
Add native Codex hook support and harden install/test flows
shinezyy dbb88bc
fixed tests for bitlesson-select-routing
shinezyy 9ed3e1a
bump version of humanize
SihaoLiu 03e159c
Merge PR #49: Use portable shebang across all shell scripts
SihaoLiu 1862329
Merge PR #50: Harden template-loader error handling and reduce duplic…
SihaoLiu fe1832d
Merge PR #42: Add methodology analysis for RLCR loop exit
SihaoLiu a8f85e8
Merge PR #48: Harden RLCR against mainline drift
SihaoLiu 611a437
Fix codex_hooks flag probe, managed-hook regex, and legacy compat
SihaoLiu 2a0faf6
Fix BSD sed portability, awk field splitting, and strict parser defaults
SihaoLiu d122c82
Fix path injection in shim generation, JSON escaping, and legacy trac…
SihaoLiu 9cf2544
Merge branch 'main' into dev
SihaoLiu febbc00
Fix relative path bypass in methodology analysis read guard
SihaoLiu 17fabf7
Revert version back to 1.16.0 on dev branch
SihaoLiu 2ab5361
Fix relative path bypass in write validator and add Codex flag probes…
SihaoLiu 4326860
Fix mainline verdict parser picking last keyword instead of first
SihaoLiu 016caca
Add ask-gemini skill and tool-filtered monitor subcommands
SihaoLiu 883e3f5
Revert version back to 1.16.0 to match dev branch
SihaoLiu 916fc50
Reject path traversal segments in methodology analysis fallback guards
SihaoLiu 515f90a
Merge pull request #52 from humania-org/ask-gemini
SihaoLiu 95a6b7b
Fix edit validator path fallback and reject whitespace-only completio…
SihaoLiu bd506f2
Reject ambiguous multi-keyword verdict lines and fix Gemini CLI insta…
SihaoLiu 4561bc4
Shell-quote hook command paths and reject ambiguous verdict lines
SihaoLiu 338b4dd
Remove PR loop feature entirely
SihaoLiu 3374acb
fix minor display bug of monitor script
SihaoLiu a1f3614
Be more specific about what is a round
SihaoLiu c045dff
fix: normalize path slashes in PostToolUse hook to prevent session_id…
gyy0592 707097c
revert: remove version bump, keep only hook fix
gyy0592 79714b7
Add support for multiple comment formats in refine-plan
Lyken17 228d6aa
Update README.md
Lyken17 28813b6
Merge pull request #68 from gyy0592/fix/session-id-double-slash
SihaoLiu 690fb27
fix: use pipe delimiter in find_comment_markers to avoid splitting CM…
SihaoLiu 027bf0b
fix: update test assertions to match renamed comment terminology
SihaoLiu 3bef3ef
Merge pull request #78 from Lyken17/main
SihaoLiu 722eb25
[docs] clarify the place to use monitor script
SihaoLiu e61b570
Merge branch 'main' into dev
SihaoLiu 7c4356c
fix: correct managed-hook dedup regex and block unresolvable symlinks
SihaoLiu c4389bb
feat: add integral context to RLCR review prompts (PID complete)
zevorn 50ed1bc
Merge pull request #81 from zevorn/feat/rlcr-integral-context
SihaoLiu File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
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
This file was deleted.
Oops, something went wrong.
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
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
This file was deleted.
Oops, something went wrong.
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
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,17 @@ | ||
| { | ||
| "description": "Humanize Codex Hooks - Native Stop hooks for RLCR loops", | ||
| "hooks": { | ||
| "Stop": [ | ||
| { | ||
| "hooks": [ | ||
| { | ||
| "type": "command", | ||
| "command": "{{HUMANIZE_RUNTIME_ROOT}}/hooks/loop-codex-stop-hook.sh", | ||
| "timeout": 7200, | ||
| "statusMessage": "humanize RLCR stop hook" | ||
| } | ||
| ] | ||
| } | ||
| ] | ||
| } | ||
| } | ||
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
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
Oops, something went wrong.
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The template emits the hook command as a bare path, so when
{{HUMANIZE_RUNTIME_ROOT}}contains spaces (for example a customCODEX_HOMEpath like/Users/John Doe/.codex), the command is split at runtime and Codex cannot execute the stop hook script. That breaks RLCR/PR exit gating in those installations; the command value should be shell-quoted or otherwise encoded as a single executable path.Useful? React with 👍 / 👎.