Allow empty path in group operations

[Serjlee]

Hello! I recently found out the validation of the path parameter prevented the use of an empty string in the path.

In my use case I found myself in this situation:

• I wanted to create a /resource API Group, with CRUD for resource, plus some sets of endpoints for sub-resources (ie: /resource/:id/stuff)
• I got a StripSlashes middleware (required to handle legacy clients), which trims any / from incoming requests
• Since I have StripSlashes, If I want to add a  POST /resource/ I need to actually define the route as POST /resource
• The above is not possible because of the strict validation done in Register
  

  huma/huma.go

  Lines 625 to 626 in 6bfeeba

  	if op.Method == "" || op.Path == "" {
  	panic("method and path must be specified in operation")

With this PR I relaxed the validation of the Path parameter, allowing the empty string in the case of operations defined for a Group with at least a prefix defined

[Copilot]

Pull Request Overview

This PR relaxes the validation of the operation path parameter to allow an empty string when the operation is defined on a Group with at least one prefix.

• Splits the validation checks for the method and path parameters.
• Allows empty path in group operations by verifying that the API is a Group with a defined prefix.
• Adds a new test to verify the behavior of an empty path in group routes.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
huma.go	Updates the operation registration to relax the empty path validation for group endpoints.
group_test.go	Introduces a test case to ensure that an empty path is handled correctly in group operations.

[codecov]

Codecov Report

Attention: Patch coverage is 20.00000% with 4 lines in your changes missing coverage. Please review.

Project coverage is 93.08%. Comparing base (6bfeeba) to head (ffe4da5).
Report is 2 commits behind head on main.

Files with missing lines	Patch %	Lines
huma.go	20.00%	2 Missing and 2 partials ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #807      +/-   ##
==========================================
- Coverage   93.11%   93.08%   -0.04%     
==========================================
  Files          23       23              
  Lines        5304     5307       +3     
==========================================
+ Hits         4939     4940       +1     
- Misses        313      314       +1     
- Partials       52       53       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

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

[danielgtaylor]

Thanks 👍