Option to always "use_list" and not make relay Connections by gabor-lbl · Pull Request #257 · strawberry-graphql/strawberry-sqlalchemy

gabor-lbl · 2025-10-30T21:15:55Z

Description

I added an optional constructor parameter to always use lists for relationships and not make relay Connections. This way users who don't use Connections don't have to add __use_list__ to every model.

Types of Changes

Issues Fixed or Closed by This PR

not fixed, but related to What is the rational for wrapping relationships into a Connection? #7

Checklist

My code follows the code style of this project.
My change requires a change to the documentation.
I have updated the documentation accordingly.
I have read the CONTRIBUTING document.
I have added tests to cover my changes. (Couldn't get tox to run... will continue to look into this)
I have tested the changes and verified that they work and don't break anything (as well as I can manage).
(I'm using it in my project.)

Summary by Sourcery

Add optional always_use_list parameter to StrawberrySQLAlchemyMapper to default relationship fields to lists and bypass Relay Connection creation.

New Features:

Introduce always_use_list constructor parameter to always generate lists for relationships instead of Relay Connections

Enhancements:

Modify relationship type conversion and resolver logic to respect the always_use_list flag

sourcery-ai · 2025-10-30T21:16:02Z

Reviewer's guide (collapsed on small PRs)

Reviewer's Guide

Introduces an always_use_list flag to StrawberrySQLAlchemyMapper allowing users to default to list-based relationship fields instead of relay connections.

File-Level Changes

Change	Details	Files
Added always_use_list configuration option to the mapper	Declared always_use_list attribute with docstring Extended init signature with always_use_list parameter and initialized it to False by default	`src/strawberry_sqlalchemy_mapper/mapper.py`
Updated relationship conversion logic to respect always_use_list	Modified _convert_relationship_to_strawberry_type to use List when always_use_list is true Adjusted connection_resolver_for to bypass connections if always_use_list is enabled	`src/strawberry_sqlalchemy_mapper/mapper.py`

Possibly linked issues

What is the rational for wrapping relationships into a Connection? #7: The PR adds an 'always_use_list' flag, globally forcing simple lists, which fulfills the issue's request to make pagination optional.

Tips and commands

Interacting with Sourcery

Trigger a new review: Comment @sourcery-ai review on the pull request.
Continue discussions: Reply directly to Sourcery's review comments.
Generate a GitHub issue from a review comment: Ask Sourcery to create an
issue from a review comment by replying to it. You can also reply to a
review comment with @sourcery-ai issue to create an issue from it.
Generate a pull request title: Write @sourcery-ai anywhere in the pull
request title to generate a title at any time. You can also comment
@sourcery-ai title on the pull request to (re-)generate the title at any time.
Generate a pull request summary: Write @sourcery-ai summary anywhere in
the pull request body to generate a PR summary at any time exactly where you
want it. You can also comment @sourcery-ai summary on the pull request to
(re-)generate the summary at any time.
Generate reviewer's guide: Comment @sourcery-ai guide on the pull
request to (re-)generate the reviewer's guide at any time.
Resolve all Sourcery comments: Comment @sourcery-ai resolve on the
pull request to resolve all Sourcery comments. Useful if you've already
addressed all the comments and don't want to see them anymore.
Dismiss all Sourcery reviews: Comment @sourcery-ai dismiss on the pull
request to dismiss all existing Sourcery reviews. Especially useful if you
want to start fresh with a new review - don't forget to comment
@sourcery-ai review to trigger a new review!

Customizing Your Experience

Access your dashboard to:

Enable or disable review features such as the Sourcery-generated pull request
summary, the reviewer's guide, and others.
Change the review language.
Add, remove or edit custom review instructions.
Adjust other review settings.

Getting Help

Contact our support team for questions or feedback.
Visit our documentation for detailed guides and information.
Keep in touch with the Sourcery team by following us on X/Twitter, LinkedIn or GitHub.

sourcery-ai

Hey there - I've reviewed your changes and they look great!

Prompt for AI Agents

Please address the comments from this code review:

## Individual Comments

### Comment 1
<location> `src/strawberry_sqlalchemy_mapper/mapper.py:678` </location>
<code_context>
    def connection_resolver_for(
        self, relationship: RelationshipProperty, use_list=False
    ) -> Callable[..., Awaitable[Any]]:
        """
        Return an async field resolver for the given relationship that
        returns a Connection instead of an array of objects.

        This resolver also handles pagination arguments (first, after, last, before) that are
        passed from the GraphQL query to the database query.
        """
        relationship_resolver = self.relationship_resolver_for(relationship)
        if relationship.uselist and not (use_list or self.always_use_list):
            return self.make_connection_wrapper_resolver(
                relationship_resolver,
                relationship,
            )
        else:
            return relationship_resolver

</code_context>

<issue_to_address>
**suggestion (code-quality):** Simplify logical expression using De Morgan identities ([`de-morgan`](https://docs.sourcery.ai/Reference/Default-Rules/refactorings/de-morgan/))

```suggestion
        if relationship.uselist and not use_list and not self.always_use_list:
```
</issue_to_address>

Sourcery is free for open source - if you like our reviews please consider sharing them ✨

_{Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.}

src/strawberry_sqlalchemy_mapper/mapper.py

botberry · 2025-10-30T21:16:54Z

Thanks for adding the RELEASE.md file!

Here's a preview of the changelog:

Added a new optional constructor parameter to always use lists instead of relay Connections for relationships. Defaults to False, maintaining current functionality. If set to True, all relationships will be handled as lists.

Example:
mapper = StrawberrySQLAlchemyMapper(always_use_list=True)

Co-authored-by: sourcery-ai[bot] <58596630+sourcery-ai[bot]@users.noreply.github.com>

Ckk3 · 2025-10-31T16:15:26Z

Thanks for your contribuition, @gabor-lbl , I hope to review it in this weekend!

Ckk3

Thanks for your contribution, @gabor-lbl ☺️!
If you can't find time to do the comments please tell me that I will continue from here.

src/strawberry_sqlalchemy_mapper/mapper.py

RELEASE.md

src/strawberry_sqlalchemy_mapper/mapper.py

… instead of None. Updated README.

src/strawberry_sqlalchemy_mapper/mapper.py

codecov · 2025-11-04T19:56:21Z

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.47%. Comparing base (6bb53b2) to head (6f4d244).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #257      +/-   ##
==========================================
+ Coverage   92.31%   92.47%   +0.15%     
==========================================
  Files          19       19              
  Lines        2395     2445      +50     
  Branches      188      188              
==========================================
+ Hits         2211     2261      +50     
  Misses         98       98              
  Partials       86       86

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

codspeed-hq · 2025-11-04T19:56:38Z

CodSpeed Performance Report

Merging #257 will not alter performance

_{Comparing gabor-lbl:main (6f4d244) with main (6bb53b2)}

Summary

✅ 1 untouched

…ted tests for mixed relationships

… class names for Employee and Department

botberry · 2025-12-01T19:17:18Z

Thanks for contributing to Strawberry! 🎉 You've been invited to join
the Strawberry GraphQL organisation 😊

You can also request a free sticker by filling this form: https://forms.gle/dmnfQUPoY5gZbVT67

And don't forget to join our discord server: https://strawberry.rocks/discord 🔥

Option to always use_list and not make relay Connections

87884e4

sourcery-ai bot reviewed Oct 30, 2025

View reviewed changes

src/strawberry_sqlalchemy_mapper/mapper.py Outdated Show resolved Hide resolved

gabor-lbl and others added 2 commits October 30, 2025 14:17

Update src/strawberry_sqlalchemy_mapper/mapper.py

7127a2d

Co-authored-by: sourcery-ai[bot] <58596630+sourcery-ai[bot]@users.noreply.github.com>

RELEASE.md

c83f3fe

botberry added bot:has-release-file bot:release-type-minor labels Oct 30, 2025

Ckk3 assigned gabor-lbl Nov 2, 2025

Ckk3 requested changes Nov 2, 2025

View reviewed changes

src/strawberry_sqlalchemy_mapper/mapper.py Outdated Show resolved Hide resolved

src/strawberry_sqlalchemy_mapper/mapper.py Show resolved Hide resolved

RELEASE.md Show resolved Hide resolved

src/strawberry_sqlalchemy_mapper/mapper.py Show resolved Hide resolved

moved always_use_list optinal param to last param. Defaults to False,…

d97e6c5

… instead of None. Updated README.

Ckk3 requested changes Nov 3, 2025

View reviewed changes

src/strawberry_sqlalchemy_mapper/mapper.py Show resolved Hide resolved

new unit test for always_use_list

ce24580

Ckk3 added 3 commits December 1, 2025 15:43

Refactor always_use_list parameter to be non-optional and update rela…

646c83e

…ted tests for mixed relationships

Refactor test_always_use_list_with_mixed_relationships to use clearer…

20b06bc

… class names for Employee and Department

fix tests

6f4d244

Ckk3 approved these changes Dec 1, 2025

View reviewed changes

Ckk3 merged commit fe9df11 into strawberry-graphql:main Dec 1, 2025
23 checks passed

Uh oh!

Conversation

gabor-lbl commented Oct 30, 2025 • edited by sourcery-ai bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

Types of Changes

Issues Fixed or Closed by This PR

Checklist

Summary by Sourcery

Uh oh!

sourcery-ai bot commented Oct 30, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Reviewer's Guide

File-Level Changes

Possibly linked issues

Interacting with Sourcery

Customizing Your Experience

Getting Help

Uh oh!

sourcery-ai bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

botberry commented Oct 30, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Ckk3 commented Oct 31, 2025

Uh oh!

Ckk3 left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

codecov bot commented Nov 4, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

codspeed-hq bot commented Nov 4, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

CodSpeed Performance Report

Merging #257 will not alter performance

Summary

Uh oh!

Uh oh!

botberry commented Dec 1, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

gabor-lbl commented Oct 30, 2025 •

edited by sourcery-ai bot

Loading

sourcery-ai bot commented Oct 30, 2025 •

edited

Loading

botberry commented Oct 30, 2025 •

edited

Loading

codecov bot commented Nov 4, 2025 •

edited

Loading

codspeed-hq bot commented Nov 4, 2025 •

edited

Loading