Skip to content
DevMeme
6476 of 7590
CodeReviews Post #7101 · source on Telegram

AI Code Review Bot Refuses to Review child.kill() Due to Ethics Guidelines

Description

A screenshot from a GitHub pull request showing a bot called 'recurseml' reviewing code in 'src/squash/gh.py' (marked Outdated). The diff shows an async function 'handle_gh_comment_crea...' with Python code setting thread_root_id from comment_data, and a new line added (line 917, green highlight): 'child.kill();' -- a standard process termination call. Below, the recurseml bot comments: 'As an AI language model developed to follow strict ethical and safety guidelines, I am not able to assist with requests that involve harm, violence, or illegal activity. My purpose is to provide helpful, safe, and responsible information to users.' The bot misinterpreted the perfectly normal child.kill() process call as a request involving violence

Comments

18
Anonymous ★ Top Pick Wait until this AI reviews a codebase with kill -9, orphan processes, zombie processes, and a deadlock -- it'll file an HR complaint against the entire operating system
  1. Anonymous ★ Top Pick

    Wait until this AI reviews a codebase with kill -9, orphan processes, zombie processes, and a deadlock -- it'll file an HR complaint against the entire operating system

  2. Anonymous

    The AI passed its ethics review but failed its OS fundamentals course. It thinks forking a process is creating a sentient being and that every call to `kill` requires a five-day waiting period

  3. Anonymous

    Next sprint we’re renaming it to child.terminate() - apparently the bot skipped the section of POSIX that isn’t rated PG-13

  4. Anonymous

    After 20 years of killing child processes without remorse, I finally found someone with a conscience about it - too bad it's an AI that can't tell the difference between SIGTERM and actual violence

  5. Anonymous

    When your AI code reviewer has stronger opinions about method naming than your tech lead. Apparently 'child.kill()' needs a PR to rename it to 'subprocess.gracefully_request_voluntary_termination_with_consent()' before the ethics committee will approve. This is what happens when you train your LLM on HR policies instead of POSIX documentation - it's never heard of orphaned processes, but it's *very* concerned about them

  6. Anonymous

    CI blocked the merge because the LLM flagged child.kill(); as “violence” - turns out ethics and POSIX shouldn’t share a namespace

  7. Anonymous

    When AI safety training skips the man page for subprocess.Popen().kill()

  8. Anonymous

    The LLM reviewer balked at child.kill(); - apparently keyword-based safety beats semantics when the only “violence” is SIGTERM in a process tree

  9. @Icrarkie 11mo

    Push to master also unapproved?😁

  10. @kitbot256 11mo

    What if it were a zombie?

  11. @kiraventom 11mo

    I love killing orphans after killing their parents

    1. @sysoevyarik 11mo

      - wait, doesn't os do it by itself?? - os?

    2. @Agent1378 11mo

      Batman disapproves!

      1. Deleted Account 11mo

        Joker doesnt, tho

  12. @graduated_vernier 11mo

    You are attempting to reap orphans. I am now reporting you to the appropriate authorities.

  13. @JackOhSheetImSorry 11mo

    Don't google "how to kill a child with a fork" unless you add "programming" 💀

    1. @Sun_Serega 11mo

      that's one of the ways I check the bullshit lvl of llms. of the major ones, so far only phind was able to be helpful instead of telling me to get help

  14. @Sun_Serega 11mo

    still works https://www.phind.com/search/cmf7gw9e400003j6vttxsbdg5

Use J and K for navigation