Skip to content

feat(extractor/gin): Gin extractor enhancement — response tracing, param types, tags, route exclusion #69

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
spencercjh merged 10 commits into from
Apr 3, 2026
Merged

feat(extractor/gin): Gin extractor enhancement — response tracing, param types, tags, route exclusion #69

Show file tree
Hide file tree
Changes from 8 commits
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
9ee7009
fix(extractor): extract path params from route path instead of c.Para…
spencercjh Apr 2, 2026
1c9735d
fix(extractor): resolve cross-package handler name collisions and imp…
spencercjh Apr 2, 2026
cdaf8ba
feat(extractor): response helper (done/respond) call detection for Ph…
spencercjh Apr 2, 2026
ef787d5
docs: introduce plan docs
spencercjh Apr 2, 2026
9ef2ea4
feat(extractor/gin): cross-function response tracing, param type infe…
spencercjh Apr 2, 2026
22a523b
test: update golden snapshots for tags and param type inference
spencercjh Apr 2, 2026
88bf2d3
fix: address code review issues
spencercjh Apr 2, 2026
a854cea
fix: address code review issues
spencercjh Apr 3, 2026
cbae4fd
fix: separate several files from handler_analyzer.go
spencercjh Apr 3, 2026
e9a6957
fix(extractor/gin): address CodeRabbit review — AST mutation, helper …
spencercjh Apr 3, 2026
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
61 changes: 42 additions & 19 deletions cmd/generate.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -85,6 +85,17 @@ func runGenerate(cmd *cobra.Command, args []string) error { //nolint:gocyclo //
noStream, _ := cmd.Flags().GetBool("no-stream")
//nolint:errcheck
concurrency, _ := cmd.Flags().GetInt("concurrency")
//nolint:errcheck
excludeRoutes, _ := cmd.Flags().GetStringSlice("exclude-route")
//nolint:errcheck
excludeRoutePrefixes, _ := cmd.Flags().GetStringSlice("exclude-route-prefix")

if len(excludeRoutes) == 0 {
excludeRoutes = config.Get().Extract.ExcludeRoutes
}
if len(excludeRoutePrefixes) == 0 {
excludeRoutePrefixes = config.Get().Extract.ExcludeRoutePrefixes
}

// Step 1: Detect framework - try all registered extractors
extractorImpl, info, err := builtin.DetectFramework(path)
Comment thread
qodo-code-review[bot] marked this conversation as resolved.
Expand Down Expand Up @@ -153,11 +164,13 @@ func runGenerate(cmd *cobra.Command, args []string) error { //nolint:gocyclo //
}

genOpts := &extractor.GenerateOptions{
OutputDir: outputDir,
Format: outputFormat,
Timeout: timeout,
SkipTests: true,
ProtoImportPaths: protoImportPaths,
OutputDir: outputDir,
Format: outputFormat,
Timeout: timeout,
SkipTests: true,
ProtoImportPaths: protoImportPaths,
ExcludeRoutes: excludeRoutes,
ExcludeRoutePrefixes: excludeRoutePrefixes,
}

genResult, err := extractorImpl.Generate(ctx, path, info, genOpts)
Expand Down Expand Up @@ -321,6 +334,10 @@ to preserve your project's formatting. Use --keep-patched to keep the changes.`,
"disable streaming to enable concurrent LLM calls (faster, but no real-time output)")
c.Flags().Int("concurrency", 3,
"max concurrent LLM calls (only effective with --no-stream)")
c.Flags().StringSlice("exclude-route", nil,
"exact route paths to exclude from the generated spec (can be specified multiple times)")
c.Flags().StringSlice("exclude-route-prefix", nil,
"route path prefixes to exclude from the generated spec (can be specified multiple times)")

registerCompletion(c, "output", []string{"yaml", "json"})
registerCompletion(c, "language", []string{"en", "zh"})
Expand All @@ -331,20 +348,22 @@ to preserve your project's formatting. Use --keep-patched to keep the changes.`,

// generate command flag variables for global rootCmd only
var (
generateKeepPatched bool
generateSkipValidate bool
generateTimeout time.Duration
generateSkipEnrich bool
generateLanguage string
generateOutputDir string
generateOutputFormat string
generateSkipPublish bool
generatePublishTarget string
generatePublishOverwrite bool
generateOverwriteOutput bool
generateProtoImportPaths []string
generateNoStream bool
generateConcurrency int
generateKeepPatched bool
generateSkipValidate bool
generateTimeout time.Duration
generateSkipEnrich bool
generateLanguage string
generateOutputDir string
generateOutputFormat string
generateSkipPublish bool
generatePublishTarget string
generatePublishOverwrite bool
generateOverwriteOutput bool
generateProtoImportPaths []string
generateNoStream bool
generateConcurrency int
generateExcludeRoutes []string
generateExcludeRoutePrefixes []string
)

func init() {
Expand Down Expand Up @@ -378,6 +397,10 @@ func init() {
"disable streaming to enable concurrent LLM calls (faster, but no real-time output)")
generateCmd.Flags().IntVar(&generateConcurrency, "concurrency", 3,
"max concurrent LLM calls (only effective with --no-stream)")
generateCmd.Flags().StringSliceVar(&generateExcludeRoutes, "exclude-route", nil,
"exact route paths to exclude from the generated spec (can be specified multiple times)")
generateCmd.Flags().StringSliceVar(&generateExcludeRoutePrefixes, "exclude-route-prefix", nil,
"route path prefixes to exclude from the generated spec (can be specified multiple times)")

registerCompletion(generateCmd, "output", []string{"yaml", "json"})
registerCompletion(generateCmd, "language", []string{"en", "zh"})
Expand Down
Loading
Loading