[log] Add debug logging to spinner, import processor, and utility packages

[github-actions]

Summary

• pkg/console/spinner.go: Added spinnerLog logger (console:spinner) with logging for TTY detection, accessibility mode detection, and lifecycle events (spinner created, started, already-running guard, stopped)
• pkg/parser/import_processor.go: Added log calls to the public ProcessImportsFromFrontmatterWithSource entry point using the pre-declared importLog — logs workflow file path, base dir, errors, and result counts (imported files, merged tools size)
• pkg/stringutil/identifiers.go: Added identifiersLog logger (stringutil:identifiers) with logging for MarkdownToLockFile and LockFileToMarkdown path conversions
• pkg/stringutil/urls.go: Added urlsLog logger (stringutil:urls) with logging for ExtractDomainFromURL calls and URL parse failures
• pkg/cli/fix_codemods.go: Added fixCodemodsLog logger (cli:fix_codemods) with a log call in GetAllCodemods reporting the registry size on load

Test plan

• make build — no compilation errors
• make fmt — code properly formatted
• go build ./pkg/console/... ./pkg/stringutil/... ./pkg/parser/... ./pkg/cli/... — all packages build cleanly
• Logging only activates when DEBUG=console:spinner (or similar) is set, so no runtime behavior changes

Generated by Go Logger Enhancement · ◷

• expires on Mar 8, 2026, 8:30 PM UTC

[Copilot]

Pull request overview

Adds debug logging across several packages to improve observability of spinner lifecycle, import processing, and common string/path helpers.

Changes:

• Added namespace loggers and debug statements for spinner creation/start/stop behavior.
• Added debug logs around import processing entrypoint (inputs, errors, result sizes).
• Added debug logs for identifier path conversions and URL domain extraction; logged codemod registry load size.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
pkg/console/spinner.go	Adds console:spinner debug logs for TTY/accessibility detection and lifecycle events.
pkg/parser/import_processor.go	Adds parser:import_processor logs for import processing inputs, errors, and summary counts.
pkg/stringutil/identifiers.go	Adds stringutil:identifiers logs for markdown/lockfile path conversions.
pkg/stringutil/urls.go	Adds stringutil:urls logs for domain extraction and parse failures.
pkg/cli/fix_codemods.go	Adds cli:fix_codemods logger and logs registry size on load.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

[Copilot]

ExtractDomainFromURL logs the full input URL. This helper is used on access log entries and workflow tool URLs (e.g., Squid logs), which may include sensitive data in query strings, fragments, or userinfo. Consider logging only the extracted hostname (or a redacted form that strips query/fragment/userinfo) to avoid leaking secrets when DEBUG is enabled; similarly avoid printing the raw parse error if it may echo the original URL.

Suggested change

	urlsLog.Printf("Extracting domain from URL: %s", urlStr)
	// Handle full URLs with protocols (http://, https://)
	if strings.HasPrefix(urlStr, "http://") || strings.HasPrefix(urlStr, "https://") {
	// Parse full URL
	parsedURL, err := url.Parse(urlStr)
	if err != nil {
	// Fall back to string manipulation if parsing fails
	urlsLog.Printf("URL parse failed, using fallback: %v", err)
	urlsLog.Printf("Extracting domain from URL")
	// Handle full URLs with protocols (http://, https://)
	if strings.HasPrefix(urlStr, "http://") || strings.HasPrefix(urlStr, "https://") {
	// Parse full URL
	parsedURL, err := url.Parse(urlStr)
	if err != nil {
	// Fall back to string manipulation if parsing fails
	urlsLog.Printf("URL parse failed, using fallback")