Java Spring duplicate parameter extraction

[EvtDanya]

Describe the bug
Noir incorrectly duplicates path parameters and mishandles @RequestParam in OpenAPI output

To Reproduce
Steps to reproduce the behavior:

1. Create controller:

package com.test;
import javax.servlet.http.HttpServletRequest;
import org.springframework.web.bind.annotation.DeleteMapping;
import org.springframework.web.bind.annotation.RestController;

@RestController
@RequestMapping("/api/v1/test")
public class MyController {

    @DeleteMapping(value = "{token}/test")
    public String Test(
        @Test @PathVariable(name = "token") String token,
    ) {
        return "hello!";
    }
}

2. Run noir ./bin/noir -b spec/functional_test/fixtures/java/spring/ -f oas3
3. Observe output

{"openapi":"3.0.0","info":{"title":"Generated by Noir","version":""},"paths":{"/api/v1/test/{token}/test":{"delete":{"responses":{"200":{"description":"Successful response"}},"parameters":[{"name":"token","in":"query"},{"name":"token","in":"query"}]}}...

Expected behavior
Correct OpenAPI output should:

{"openapi":"3.0.0","info":{"title":"Generated by Noir","version":""},"paths":{"/api/v1/test/{token}/test":{"delete":{"responses":{"200":{"description":"Successful response"}},"parameters":[{"name":"token","in":"query"}]}}...

1 token param instead of 2

Screenshots

Versions

• OS: linux
• Version v0.22.0

[ksg97031]

@EvtDanya cc @hahwul
Issues #628, #629, #630, and #631 will be patched in the next release.
Thank you for reporting these bugs.

[EvtDanya]

Thanks for the updates! Looking forward to the next release