rtk init --codex --uninstall says "not installed (nothing to remove)" and then "the hook may still be registered"
Maintainers usually reply within 1 day
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 72/100
Research direction
The contradictory report comes from codex_uninstall_report in src/hooks/init/codex.rs. Run the shell reproduction in the issue, which uses a symlinked .codex pointing outside the project, and compare the first line with the unchecked-hook note below it. Done when the first line no longer says RTK was not installed while the hook was left unchecked, and the note about the containment guard still appears. The exit code question belongs to #4164 and can stay out of this change.
Written by the indexing model from the issue text.
Description
When the containment guard refuses .codex/hooks.json and nothing else was removed, the report opens with a line that says there is nothing to do and then says the opposite:
W=$(mktemp -d); mkdir -p "$W/proj" "$W/out"; cd "$W/proj"; git init -q .
ln -s "$W/out" .codex; printf '{"hooks":{}}\n' > "$W/out/hooks.json"
rtk init --codex --uninstall; echo "exit=$?"
RTK was not installed for Codex CLI (nothing to remove)
Not checked: the Codex hook in .codex/hooks.json, which may still be registered:
.codex/hooks.json resolves to <W>/out/hooks.json, outside the project at <W>/proj.
Remove the symlink, or use --global to configure Codex outside the project.
exit=0
A reader who stops at the first line concludes there is nothing to do. When the hook was not checked, the first line should not claim RTK was not installed. codex_uninstall_report in src/hooks/init/codex.rs. Related: #4164 (exit code for the same state).
- Dominant language
- Rust
- Stars
- 83.5k
- Forks
- 5.3k
- Avg merge
- 5d 15h
- Merged PRs (30d)
- 63
Getting set up
- No Dockerfile or Docker Compose file
- Has a pull request template
- Read the contributing guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from rtk-ai/rtk
-
`rtk rewrite` runs the daily hook reminder and telemetry ping on the hook path, so shell hooks spend the reminder into /dev/nullPossibly taken A pull request linked to this issue is open or already merged. Openarea:cli bug good first issue priority:medium
Difficulty 2/5 1-3 hours Newbie friendliness 74/100
Maintainers usually reply within 1 day
-
area:cli bug good first issue priority:medium
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
Maintainers usually reply within 1 day
-
discover: -p/--project filter is ignored when --all is setPossibly taken @Nacl-HCl claimed this 2 days ago. Openarea:cli bug help wanted priority:high
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
Maintainers usually reply within 1 day
-
docs(openclaw): RTK_REWRITE_HOST relaxes every default ask, not only commands no rule matchedPossibly taken @Arslan-TR claimed this 7 days ago. Openarea:docs documentation good first issue priority:low
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
Maintainers usually reply within 1 day
-
rtk cargo test -- --list prints nothing: the filter reads the test listing as an empty runPossibly taken @yeahjack claimed this 12 days ago. Openarea:cli area:testing bug good first issue priority:medium
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
Maintainers usually reply within 1 day
Similar issues
-
Difficulty 1/5 Under an hour Newbie friendliness 88/100
pop-os/cosmic-comp#2956 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
Maintainers usually reply within 1 day
-
area:release criticality:p3 documentation
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
registrystack/registry-stack#2085 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 85/100
libp2p/rust-libp2p#6694 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
EmbarkStudios/cargo-deny#910 ·