fix(reporter): suppress --report-markdown hint when markdown report is already enabled

[taiman724]

Summary

The hint at pkg/reporter/summary.go:556 unconditionally prints Run vet with --report-markdown=... even when the user has already passed --report-markdown. This adds a MarkdownReportEnabled field to SummaryReporterConfig and gates the println on it so the hint is shown only when markdown output is not enabled. scan and query subcommands both pass the flag state through.

Fixes #725.

Change

• pkg/reporter/summary.go: Add MarkdownReportEnabled bool field with a godoc comment, wrap the existing fmt.Println hint in an if !r.config.MarkdownReportEnabled guard.
• scan.go: pass MarkdownReportEnabled: !utils.IsEmptyString(markdownReportPath) when constructing the SummaryReporterConfig.
• query.go: same as above.

The new field is a bool whose zero value is false, preserving backward-compatible behavior for any existing SummaryReporterConfig{} literal (only pkg/reporter/markdown.go:66 has another instantiation, and its nested summary never calls Finish() — confirmed by tracing).

The mirror predicate !utils.IsEmptyString(markdownReportPath) is the same one used at scan.go:644 and query.go:256 to decide whether to instantiate the markdown reporter at all.

---

Prepared with AI assistance (Claude Sonnet 4.6 via Claude Code).

[safedep]

SafeDep Report Summary

No dependency changes detected. Nothing to scan.

View complete scan results →

_{This report is generated by SafeDep Github App}