Go port - phase 1

Begin porting this CLI to go

- Justfile targets: `just go-build`, `go-test`, etc... `just ci-all` to run CI for both.
- Conventional `pkg/` and `cmd/` layout
- depends on the cobra CLI interface
- linter rules
- integrated go CI

The eventual result will be a -beta CLI version in parallel with the C# CLI

Author: offbyone

.github/workflows/go-ci.yml

@@ -0,0 +1,98 @@
+name: Go CI
+
+on:
+  push:
+    branches: [main]
+  pull_request:
+    branches: [main]
+  workflow_dispatch:
+
+permissions:
+  contents: read
+
+jobs:
+  go-build-and-test:
+    name: Go Build and Test
+    strategy:
+      fail-fast: false
+      matrix:
+        runner-os: [windows-latest, ubuntu-latest, macos-latest]
+
+    runs-on: ${{ matrix.runner-os }}
+
+    steps:
+      - uses: actions/checkout@v6
+        with:
+          persist-credentials: false
+
+      - name: Setup Go
+        uses: actions/setup-go@v6
+        with:
+          go-version-file: go.mod
+          cache: true
+
+      - name: Setup Just
+        uses: extractions/setup-just@e33e0265a09d6d736e2ee1e0eb685ef1de4669ff # v3
+
+      - name: Check Go formatting
+        if: matrix.runner-os == 'ubuntu-latest'
+        run: |
+          if [ -n "$(gofmt -l .)" ]; then
+            echo "Go files need formatting:"
+            gofmt -l .
+            exit 1
+          fi
+
+      - name: Go Vet
+        run: go vet ./...
+
+      - name: Build Go binaries
+        run: just go-build
+
+      - name: Create coverage directory
+        run: mkdir -p coverage
+        shell: bash
+
+      - name: Run Go tests
+        run: go test -v -race -coverprofile=./coverage/coverage.out ./...
+
+      - name: Generate coverage report
+        if: matrix.runner-os == 'ubuntu-latest'
+        run: |
+          go tool cover -html=./coverage/coverage.out -o coverage.html
+          go tool cover -func=./coverage/coverage.out
+
+      - name: Upload coverage artifact
+        if: matrix.runner-os == 'ubuntu-latest'
+        uses: actions/upload-artifact@v6
+        with:
+          name: go-coverage-report
+          path: coverage.html
+
+      - name: Test binaries
+        run: |
+          ./dist/gei --version
+          ./dist/ado2gh --version
+          ./dist/bbs2gh --version
+        shell: bash
+
+  go-lint:
+    name: Go Lint
+    runs-on: ubuntu-latest
+
+    steps:
+      - uses: actions/checkout@v6
+        with:
+          persist-credentials: false
+
+      - name: Setup Go
+        uses: actions/setup-go@v6
+        with:
+          go-version-file: go.mod
+          cache: true
+
+      - name: golangci-lint
+        uses: golangci/golangci-lint-action@1e7e51e771db61008b38414a730f564565cf7c20 # v9.2.0
+        with:
+          version: latest
+          args: --timeout=5m

.gitignore

@@ -359,3 +359,9 @@ MigrationBackup/
 /src/gei/Properties/launchSettings.json
 /src/OctoshiftCLI.IntegrationTests/Properties/launchSettings.json
 /src/ado2gh/Properties/launchSettings.json
+
+# Go coverage reports
+coverage/
+*.out
+coverage.html
+/coverage.out

.golangci.yml

@@ -0,0 +1,76 @@
+# golangci-lint configuration for gh-gei Go port
+# See https://golangci-lint.run/usage/configuration/
+
+run:
+  timeout: 5m
+  tests: true
+  modules-download-mode: readonly
+
+linters:
+  enable:
+    - gofmt         # Checks formatting
+    - goimports     # Checks imports
+    - govet         # Reports suspicious constructs
+    - errcheck      # Checks for unchecked errors
+    - staticcheck   # Static analysis
+    - unused        # Checks for unused code
+    - gosimple      # Simplify code
+    - ineffassign   # Detects ineffectual assignments
+    - typecheck     # Type-checks code
+    - bodyclose     # Checks for HTTP response body close
+    - noctx         # Finds HTTP requests without context
+    - misspell      # Finds commonly misspelled English words
+    - unconvert     # Removes unnecessary type conversions
+    - goconst       # Finds repeated strings that could be constants
+    - gocyclo       # Computes cyclomatic complexities
+    - gofumpt       # Stricter gofmt
+    - revive        # Fast, configurable, extensible linter
+    - gosec         # Security-focused linter
+    - errname       # Checks error naming
+    - errorlint     # Finds misuses of errors
+    - exportloopref # Checks for pointers to enclosing loop variables
+    - whitespace    # Detects leading/trailing whitespace
+
+linters-settings:
+  gocyclo:
+    min-complexity: 15
+  goconst:
+    min-len: 3
+    min-occurrences: 3
+  misspell:
+    locale: US
+  revive:
+    rules:
+      - name: exported
+        severity: warning
+        disabled: false
+      - name: package-comments
+        severity: warning
+        disabled: false
+  gosec:
+    excludes:
+      - G104  # Audit errors not checked (too noisy)
+
+issues:
+  exclude-use-default: false
+  max-issues-per-linter: 0
+  max-same-issues: 0
+  
+  exclude-rules:
+    # Exclude some linters from running on tests files
+    - path: _test\.go
+      linters:
+        - gocyclo
+        - errcheck
+        - gosec
+    
+    # Exclude error checks in main.go files (handled by cobra)
+    - path: cmd/.*/main\.go
+      text: "Error return value"
+      linters:
+        - errcheck
+
+output:
+  format: colored-line-number
+  print-issued-lines: true
+  print-linter-name: true