Skip to content

generative-ai: Add anti pattern example #1679

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.

Sign up for GitHub

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

Merged
corona10 merged 10 commits into from
Nov 9, 2025
Merged

generative-ai: Add anti pattern example #1679

Changes from 2 commits
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
55e0a9e
generative-ai: Add anti pattern example
corona10 Nov 1, 2025
2a102b8
reformat
corona10 Nov 1, 2025
7a7bec2
Address code review
corona10 Nov 1, 2025
f0e60d0
Add one more case
corona10 Nov 1, 2025
131f62f
reformat
corona10 Nov 1, 2025
f7d40f8
Address code review
corona10 Nov 8, 2025
3b1b75c
fix
corona10 Nov 8, 2025
fdc2c42
Update heading and use sentence case
willingc Nov 8, 2025
65f3bbe
Improve wording and spacing for principles
willingc Nov 8, 2025
cf1642e
Apply suggestions from code review
corona10 Nov 9, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 6 additions & 0 deletions getting-started/generative-ai.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -24,3 +24,9 @@ Unacceptable uses
Maintainers may close issues and PRs that are not useful or productive, including
those that are fully generated by AI. If a contributor repeatedly opens unproductive
issues or PRs, they may be blocked.

Anti-patterns
Copy link
Copy Markdown
Member

@nedbat nedbat Nov 1, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we have a second anti-pattern to include in the list? Other bad things: hallucinations, incorrectly changing tests instead of fixing code, ...

Copy link
Copy Markdown
Member

@gpshead gpshead Nov 2, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i think you covered the incorrectly changing tests instead of fixing one. Good call. I like having that in the list.

I don't personally see that so often anymore with Opus 4.1 or Sonnet 4.5, but older or lower end models tend to reward hack their way into that state more often - it's a thing we'll likely encounter coming from new contributors for a while as they can't be assumed to be using the latest and greatest.

Copy link
Copy Markdown
Collaborator

@willingc willingc Nov 4, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would use a different title than antipatterns. I would like to move toward "Considerations or Keys for Success".

I would start by linking to the section about submitting a quality PR:

  • consider whether the change is necessary
  • small, focused changes
  • tests that exercise the change
  • style follows the existing code

AI coding tools will compose code. The contributor, not the AI tool, is responsible for the quality and validity of code submitted with a PR.

Take the time to consider the code changes suggested by an AI tool. Respect the reviewer's time by validating your PR before submission.

=============
- While AI-assisted tools such as autocompletion can enhance productivity, they sometimes rewrite entire code blocks instead of making small, focused edits.
Copy link
Copy Markdown
Member

@gpshead gpshead Nov 2, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd actually tighten this phrasing up a bit, we don't need to explain what shape AI tools can take or what they can do (anyone reading this knows that).

I find this list of anti-patterns to not be AI specific. We already reject PRs coming from people who make larger than necessary changes that are difficult to review. So I'd suggest wording it as a "positive desired thing, not negative example" phrasing, starting something like this:

  • Make small focused edits, prefer not rewriting entire code blocks.
    This makes it easier to review changes and fully understand both the original intent of the code and the rationale behind the new modifications.
    ...

This can make it more difficult to review changes and to fully understand both the original intent of the code and the rationale behind the new modifications.
Please keep maintaining consistency with the original code helps preserve clarity, traceability, and meaningful review.
Copy link
Copy Markdown
Member

@nedbat nedbat Nov 1, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Please keep maintaining consistency with the original code helps preserve clarity, traceability, and meaningful review.
Maintaining consistency with the original code helps preserve clarity, traceability, and meaningful review.

We also might want to mention (and link to?) the idea that we want to avoid churn.

Loading