7 ms·
For all this "AI is the future," how in the world does OpenAI's own codebase still have "TODO" comments for the most trivial thing I can possibly imagine? <http
by mdaniel 1y ago
For all this "AI is the future," how in the world does OpenAI's own codebase still have "TODO" comments for the most trivial thing I can possibly imagine? <https://github.com/openai/codex/blob/rust-v0.0.2505302325/codex-rs/core/src/mcp_connection_manager.rs#L84 https://github.com/openai/codex/blob/rust-v0.0.2505302325/co...> made extra wtf by a comment at the top of the file laying out that requirement, so no "?" required <https://github.com/openai/codex/blob/rust-v0.0.2505302325/codex-rs/core/src/mcp_connection_manager.rs#L27 https://github.com/openai/codex/blob/rust-v0.0.2505302325/co...>
I would bet it took more wall-clock time to type out that comment than it would have for any number of AI agents to snap the required equivalent of `if not re.match(...): continue` into place
- quesera 1y agoSeems uncharitable to me. // TODO: Verify server name: require `^[a-zA-Z0-9_-]+$`? There may be several elements of server name verification to perform. That regex does not cover the complete range of possibilities for a server name. The author apparently decided to punt on the correctness of this low-risk test -- pending additional thought, research, consultation, or simply higher prioritization.