spring-projects / spring-projects/spring-framework

Identify invalid factory methods AOT when possible

Open
#32,794 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

in: core theme: aot type: enhancement
Dominant language
Java
Stars
60.2k
Forks
38.8k
Avg merge
5d 2h
Merged PRs (30d)
27

Description

As discussed with @snicoll and @jhoeller while working on #32719, InstanceSupplierCodeGenerator#generateCode currently does not perform the same checks than org.springframework.beans.factory.support.ConstructorResolver#instantiateUsingFactoryMethod except the recent suspending function check added recently.

It would be nice from a DevXP and consistency perspective if it could perform AOT some of those checks (like the void return type check for example).

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by comparing InstanceSupplierCodeGenerator#generateCode with ConstructorResolver#instantiateUsingFactoryMethod, using the existing suspending function check as the reference point. Identify which factory-method validations can be performed during AOT generation, including the void return type check, and verify that the generated diagnostics are consistent with runtime behavior.

Written by the indexing model from the issue text.

Assessment

Tech stack
java, spring
Domain
backend, developer-experience
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
42/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.