Add build URLs to get-job output (#17)

* Initial plan

* Add build URLs to get-job command output

Co-authored-by: alexec <1142830+alexec@users.noreply.github.com>

* Update MCP server to also include build URLs

Co-authored-by: alexec <1142830+alexec@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: alexec <1142830+alexec@users.noreply.github.com>

Author: Copilot

README.md

@@ -143,8 +143,8 @@ jenkins get-job my-application-build
 # URL:                 https://jenkins.example.com/job/my-application-build/
 # Status:              SUCCESS
 # Description:         Builds the main application
-# Last Build:          #42 - SUCCESS
-# Last Success:        #42
+# Last Build:          #42 - SUCCESS (https://jenkins.example.com/job/my-application-build/42/)
+# Last Success:        #42 (https://jenkins.example.com/job/my-application-build/42/)
 ```
 
 **Working with Inner Jobs (Folders and Multi-branch Pipelines):**

duration.go

@@ -6,30 +6,30 @@ import "fmt"
 // Returns the largest unit that makes sense: N days, N hours, N minutes, or N seconds
 func formatDuration(milliseconds float64) string {
 	seconds := int64(milliseconds / 1000)
-	
+
 	if seconds < 60 {
 		if seconds == 1 {
 			return "1 second"
 		}
 		return fmt.Sprintf("%d seconds", seconds)
 	}
-	
+
 	minutes := seconds / 60
 	if minutes < 60 {
 		if minutes == 1 {
 			return "1 minute"
 		}
 		return fmt.Sprintf("%d minutes", minutes)
 	}
-	
+
 	hours := minutes / 60
 	if hours < 24 {
 		if hours == 1 {
 			return "1 hour"
 		}
 		return fmt.Sprintf("%d hours", hours)
 	}
-	
+
 	days := hours / 24
 	if days == 1 {
 		return "1 day"

duration_test.go

@@ -16,19 +16,19 @@ func TestFormatDuration(t *testing.T) {
 		{5000, "5 seconds"},
 		{30000, "30 seconds"},
 		{59000, "59 seconds"},
-		
+
 		// Minutes
 		{60000, "1 minute"},
 		{120000, "2 minutes"},
 		{1800000, "30 minutes"},
 		{3599000, "59 minutes"},
-		
+
 		// Hours
 		{3600000, "1 hour"},
 		{7200000, "2 hours"},
 		{43200000, "12 hours"},
 		{86399000, "23 hours"},
-		
+
 		// Days
 		{86400000, "1 day"},
 		{172800000, "2 days"},

main.go

@@ -239,17 +239,17 @@ func getJob(ctx context.Context, jobName string) error {
 
 	lastBuild, err := job.GetLastBuild(ctx)
 	if err == nil && lastBuild != nil {
-		printField("Last Build", fmt.Sprintf("#%d - %s", lastBuild.GetBuildNumber(), lastBuild.GetResult()))
+		printField("Last Build", fmt.Sprintf("#%d - %s (%s)", lastBuild.GetBuildNumber(), lastBuild.GetResult(), lastBuild.GetUrl()))
 	}
 
 	lastSuccess, err := job.GetLastSuccessfulBuild(ctx)
 	if err == nil && lastSuccess != nil {
-		printField("Last Success", fmt.Sprintf("#%d", lastSuccess.GetBuildNumber()))
+		printField("Last Success", fmt.Sprintf("#%d (%s)", lastSuccess.GetBuildNumber(), lastSuccess.GetUrl()))
 	}
 
 	lastFailed, err := job.GetLastFailedBuild(ctx)
 	if err == nil && lastFailed != nil {
-		printField("Last Failed", fmt.Sprintf("#%d", lastFailed.GetBuildNumber()))
+		printField("Last Failed", fmt.Sprintf("#%d (%s)", lastFailed.GetBuildNumber(), lastFailed.GetUrl()))
 	}
 
 	// Print inner jobs if they exist (for folders and multi-branch pipelines)

mcp.go

@@ -153,17 +153,17 @@ func getJobHandler(ctx context.Context, client *gojenkins.Jenkins, request mcp.C
 
 	lastBuild, err := job.GetLastBuild(ctx)
 	if err == nil && lastBuild != nil {
-		result += fmt.Sprintf("\nLast Build: #%d - %s", lastBuild.GetBuildNumber(), lastBuild.GetResult())
+		result += fmt.Sprintf("\nLast Build: #%d - %s (%s)", lastBuild.GetBuildNumber(), lastBuild.GetResult(), lastBuild.GetUrl())
 	}
 
 	lastSuccess, err := job.GetLastSuccessfulBuild(ctx)
 	if err == nil && lastSuccess != nil {
-		result += fmt.Sprintf("\nLast Success: #%d", lastSuccess.GetBuildNumber())
+		result += fmt.Sprintf("\nLast Success: #%d (%s)", lastSuccess.GetBuildNumber(), lastSuccess.GetUrl())
 	}
 
 	lastFailed, err := job.GetLastFailedBuild(ctx)
 	if err == nil && lastFailed != nil {
-		result += fmt.Sprintf("\nLast Failed: #%d", lastFailed.GetBuildNumber())
+		result += fmt.Sprintf("\nLast Failed: #%d (%s)", lastFailed.GetBuildNumber(), lastFailed.GetUrl())
 	}
 
 	// Add inner jobs if they exist (for folders and multi-branch pipelines)