Granular "feeling safe" control - #5045
Conversation
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request introduces a granular 'feeling safe' mechanism to allow enabling unsafe behaviors selectively. To improve the architecture, avoid using global state for tracking allowed functionalities. Make the DangerousFunctionality class and formatting functions context-aware by passing the Common object (or self) to retrieve options directly. Remove unused global variables, imports, and helper functions like allow_functionalities. Additionally, fix a typo in the TODO path for the local provision plugin and import Common directly under TYPE_CHECKING to avoid re-exports.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
![graphite-app[bot]](https://avatars.githubusercontent.com/in/158384?s=60&v=4){kind=small}
ed3d98f to
30c7fcc
Compare
35b7784 to
79d239e
Compare
79d239e to
482ae37
Compare
|
Caution Review failedAn error occurred during the review process. Please try again later. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
a5f5af4 to
125539c
Compare
LecrisUT
left a comment
LecrisUT
left a comment
There was a problem hiding this comment.
Other than having a more explicit none keyword, all LGTM.
125539c to
f402698
Compare
LecrisUT
left a comment
LecrisUT
left a comment
There was a problem hiding this comment.
Lgtm other than a couple of non-blocking implementation details
eaedae0 to
ee61f50
Compare
thrix
left a comment
thrix
left a comment
There was a problem hiding this comment.
Three findings could not be anchored to diff lines because the PR does not touch those files. All three confirmed present at ee61f50c:
| File | Problem |
|---|---|
docs/overview.rst:390 |
"Set this variable to 1" — TMT_FEELING_SAFE=1 now exits 2 |
agents/AGENTS.md:72 |
tmt --feeling-safe -vv run ... — the option swallows -vv and errors |
.claude/CLAUDE.md:72 |
Same command, same break |
Generated-by: Claude Code
38caa48 to
eba467d
Compare
4 participants
Instead of one big all-or-nothing switch, finer control over various dangerous action is needed. Users can still allow everything, with
--feeling-safe[=all], or allow individual dangerous functionalities.Pull Request Checklist