fix: v0.3.7 quick wins (9 issues)

[ewega]

Summary

Independent bug fixes and improvements for v0.3.7. No architectural changes — those come in PR2 (scope/project separation).

Changes

Issue	Fix	File
#31	Validate empty resourceGroup before Azure cleanup	cmd/cleanup.go
#32	Remove duplicate PAT scope printing in connection creation	cmd/configure_connections.go
#33	Add "Testing connection..." message before test API call	cmd/connection_types.go
#34	Increase HTTP client timeout from 30s to 90s	internal/devlake/client.go
#36	Remove auto-connection pickup, always show interactive picker	cmd/configure_projects.go
#43	Warn when state file URL found but backend is unreachable	internal/devlake/discovery.go
#44	Replace duplicate stateFile struct with LoadState()	internal/devlake/discovery.go
#47	Fix state dedup to use plugin+connectionID (not plugin alone)	cmd/configure_connections.go
#48	Remove duplicate PAT scope printing in configure full	cmd/configure_full.go

Testing

• go build — clean
• go test ./... — only pre-existing failure in TestRunConfigureScopes_PluginFlag (configure_scopes_test.go only tests helpers and mutates globals without cleanup #45, global state mutation). No new failures.

Closes #31, #32, #33, #34, #36, #43, #44, #47, #48

[Copilot]

Pull request overview

This PR addresses 9 independent bug fixes and UX improvements for v0.3.7 without any architectural changes. The changes improve error handling, remove redundant output, fix state file management, and increase timeout for remote DevLake instances.

Changes:

• Improved error handling with validation for empty Azure resource groups and warnings for unreachable DevLake URLs in state files
• Enhanced UX by removing duplicate PAT scope printing and adding connection test feedback messages
• Fixed state file management to properly track multiple connections per plugin using plugin+connectionID as the dedup key
• Increased HTTP client timeout from 30s to 90s to support remote DevLake instances with higher latency
• Removed auto-connection pickup behavior to require explicit user selection

Reviewed changes

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

Show a summary per file

File	Description
cmd/cleanup.go	Adds validation to ensure resourceGroup is not empty before attempting Azure cleanup
cmd/configure_connections.go	Removes duplicate PAT scope printing and fixes state dedup to use plugin+connectionID instead of plugin alone
cmd/configure_full.go	Removes duplicate PAT scope printing during connection creation
cmd/configure_projects.go	Removes auto-connection pickup logic to always show interactive picker
cmd/connection_types.go	Adds "Testing connection..." message before connection test API call
internal/devlake/client.go	Increases HTTP client timeout from 30s to 90s for remote instance support
internal/devlake/discovery.go	Replaces duplicate stateFile struct with LoadState() and adds warning when state file URL is unreachable

[Copilot]

The emoji should be 🔌 (plug) not 🔑 (key) for "Testing connection..." to match the established pattern in the codebase. The 🔌 emoji is used for connection testing operations (see configure_connection_test_cmd.go line 140 and configure_connection_update.go line 177), while 🔑 is reserved for authentication/token operations.

Suggested change

	fmt.Println(" 🔑 Testing connection...")
	fmt.Println(" 🔌 Testing connection...")