Release v1.16.5

.github/workflows/docker-release.yml

@@ -29,18 +29,18 @@ jobs:
           echo "Version: $VERSION"
 
       - name: Login to GitHub Container Registry
-        uses: docker/login-action@v3
+        uses: docker/login-action@v4
         with:
           registry: ghcr.io
           username: ${{ github.actor }}
           password: ${{ secrets.GITHUB_TOKEN }}
 
-      - uses: depot/use-action@v1
+      - uses: depot/use-action@v1.3.1
         with:
           project: xnsnw3m20t
 
       - name: Build and push container
-        uses: depot/build-push-action@v1
+        uses: depot/build-push-action@v1.17.0
         id: build
         with:
           project: xnsnw3m20t

.github/workflows/finalize-copybara-sync.yml

@@ -45,11 +45,13 @@ jobs:
         run: go mod tidy
 
       - name: Commit and push go mod tidy changes
+        env:
+          HEAD_BRANCH: ${{ github.ref_name }}
         run: |
           if ! git diff --quiet go.mod go.sum; then
             git add go.mod go.sum
             git commit -m "Run go mod tidy"
-            git push origin ${{ github.ref_name }}
+            git push origin "$HEAD_BRANCH"
           else
             echo "No changes from go mod tidy"
           fi
@@ -81,6 +83,7 @@ jobs:
           AUTHOR_NAME: ${{ steps.author.outputs.name }}
           AUTHOR_EMAIL: ${{ steps.author.outputs.email }}
           GITHUB_USER: ${{ steps.author.outputs.github_user }}
+          HEAD_BRANCH: ${{ github.ref_name }}
         run: |
           # Build PR body
           PR_BODY="## Copybara Sync - Release ${VERSION}
@@ -106,7 +109,7 @@ jobs:
           # Create the PR
           PR_URL=$(gh pr create \
             --base main \
-            --head "${{ github.ref_name }}" \
+            --head "$HEAD_BRANCH" \
             --title "Release ${VERSION}" \
             --body "$PR_BODY")
 

.github/workflows/release.yml

@@ -50,7 +50,7 @@ jobs:
           cache: true
 
       - name: Run GoReleaser (publish)
-        uses: goreleaser/goreleaser-action@v6
+        uses: goreleaser/goreleaser-action@v7
         with:
           version: latest
           args: release --clean

.github/workflows/tag-on-merge.yml

@@ -18,9 +18,10 @@ jobs:
     steps:
       - name: Extract version from branch name
         id: version
+        env:
+          BRANCH: ${{ github.event.pull_request.head.ref }}
         run: |
           # Extract v1.2.3 from copybara/v1.2.3
-          BRANCH="${{ github.event.pull_request.head.ref }}"
           VERSION=$(echo "$BRANCH" | sed 's|copybara/||')
           echo "version=$VERSION" >> $GITHUB_OUTPUT
           echo "Extracted version: $VERSION"
@@ -48,8 +49,8 @@ jobs:
       - name: Delete copybara branch
         env:
           GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+          BRANCH: ${{ github.event.pull_request.head.ref }}
         run: |
-          BRANCH="${{ github.event.pull_request.head.ref }}"
           echo "Deleting branch: $BRANCH"
           git push origin --delete "$BRANCH" || echo "Branch may have already been deleted"
 

.goreleaser.yaml

@@ -9,15 +9,15 @@ builds:
       - linux
       - windows
     ldflags:
-      - -s -w -X github.com/overmindtech/cli/tracing.version={{.Version}}
+      - -s -w -X github.com/overmindtech/cli/go/tracing.version={{.Version}}
   - binary: overmind
     id: overmind-macos
     env:
       - CGO_ENABLED=0
     goos:
       - darwin
     ldflags:
-      - -s -w -X github.com/overmindtech/cli/tracing.version={{.Version}}
+      - -s -w -X github.com/overmindtech/cli/go/tracing.version={{.Version}}
 
 # For now we are going to disable signing MacOS packages. This works on Dylan's
 # person laptop, but we haven't worked out a way to get this set up in a github

README.md

@@ -230,6 +230,14 @@ overmind --version
   changes you make with `overmind terraform apply`, so that you can be sure
   that your changes haven't had any unexpected downstream impact.
 
+- `overmind knowledge list`
+
+  View which knowledge files Overmind would discover from your current location.
+  Knowledge files in `.overmind/knowledge/` teach the AI investigator about your
+  infrastructure context, standards, and approved patterns. This command shows the
+  resolved knowledge directory path, valid files with their metadata, and any
+  validation warnings for invalid files.
+
 ## Cloud Provider Support
 
 The CLI automatically discovers AWS and GCP providers from your Terraform configuration.

aws-source/adapters/adapterhelpers_get_list_adapter_v2.go

@@ -4,6 +4,7 @@ import (
 	"context"
 	"errors"
 	"fmt"
+	"slices"
 	"time"
 
 	"github.com/overmindtech/cli/go/discovery"
@@ -135,13 +136,7 @@ func (s *GetListAdapterV2[ListInput, ListOutput, AWSItem, ClientStruct, Options]
 		return true
 	}
 
-	for _, s := range s.Scopes() {
-		if s == scope {
-			return true
-		}
-	}
-
-	return false
+	return slices.Contains(s.Scopes(), scope)
 }
 
 // Get retrieves an item from the adapter based on the provided scope, query, and

aws-source/adapters/adapterhelpers_get_list_adapter_v2_test.go

@@ -261,7 +261,7 @@ func TestListFuncPaginatorBuilder(t *testing.T) {
 			return output, nil
 		},
 		ItemMapper: func(query *string, scope string, awsItem string) (*sdp.Item, error) {
-			attrs, _ := sdp.ToAttributes(map[string]interface{}{
+			attrs, _ := sdp.ToAttributes(map[string]any{
 				"id": awsItem,
 			})
 			return &sdp.Item{

aws-source/adapters/adapterhelpers_get_list_source.go

@@ -4,6 +4,7 @@ import (
 	"context"
 	"errors"
 	"fmt"
+	"slices"
 	"time"
 
 	"buf.build/go/protovalidate"
@@ -108,13 +109,7 @@ func (s *GetListAdapter[AWSItem, ClientStruct, Options]) hasScope(scope string)
 		return true
 	}
 
-	for _, s := range s.Scopes() {
-		if s == scope {
-			return true
-		}
-	}
-
-	return false
+	return slices.Contains(s.Scopes(), scope)
 }
 
 // Get retrieves an item from the adapter based on the provided scope, query, and

aws-source/adapters/adapterhelpers_shared_tests.go

@@ -12,34 +12,6 @@ import (
 	"github.com/overmindtech/cli/go/sdp-go"
 )
 
-func PtrString(v string) *string {
-	return &v
-}
-
-func PtrInt32(v int32) *int32 {
-	return &v
-}
-
-func PtrInt64(v int64) *int64 {
-	return &v
-}
-
-func PtrFloat32(v float32) *float32 {
-	return &v
-}
-
-func PtrFloat64(v float64) *float64 {
-	return &v
-}
-
-func PtrTime(v time.Time) *time.Time {
-	return &v
-}
-
-func PtrBool(v bool) *bool {
-	return &v
-}
-
 type Subnet struct {
 	ID               *string
 	CIDR             string

aws-source/adapters/adapterhelpers_util.go

@@ -488,7 +488,7 @@ func GetAutoConfig(t *testing.T) (aws.Config, string, string) {
 // Converts an interface to SDP attributes using the `sdp.ToAttributesSorted`
 // function, and also allows the user to exclude certain top-level fields from
 // the resulting attributes
-func ToAttributesWithExclude(i interface{}, exclusions ...string) (*sdp.ItemAttributes, error) {
+func ToAttributesWithExclude(i any, exclusions ...string) (*sdp.ItemAttributes, error) {
 	attrs, err := sdp.ToAttributesViaJson(i)
 	if err != nil {
 		return nil, err

aws-source/adapters/apigateway-domain-name_test.go

@@ -43,21 +43,21 @@ import (
 
 func TestDomainNameOutputMapper(t *testing.T) {
 	domainName := &types.DomainName{
-		CertificateArn:                      PtrString("arn:aws:acm:region:account-id:certificate/certificate-id"),
-		CertificateName:                     PtrString("certificate-name"),
-		CertificateUploadDate:               PtrTime(time.Now()),
-		DistributionDomainName:              PtrString("distribution-domain-name"),
-		DistributionHostedZoneId:            PtrString("distribution-hosted-zone-id"),
-		DomainName:                          PtrString("domain-name"),
+		CertificateArn:                      new("arn:aws:acm:region:account-id:certificate/certificate-id"),
+		CertificateName:                     new("certificate-name"),
+		CertificateUploadDate:               new(time.Now()),
+		DistributionDomainName:              new("distribution-domain-name"),
+		DistributionHostedZoneId:            new("distribution-hosted-zone-id"),
+		DomainName:                          new("domain-name"),
 		DomainNameStatus:                    types.DomainNameStatusAvailable,
-		DomainNameStatusMessage:             PtrString("status-message"),
+		DomainNameStatusMessage:             new("status-message"),
 		EndpointConfiguration:               &types.EndpointConfiguration{Types: []types.EndpointType{types.EndpointTypeEdge}},
-		MutualTlsAuthentication:             &types.MutualTlsAuthentication{TruststoreUri: PtrString("truststore-uri")},
-		OwnershipVerificationCertificateArn: PtrString("arn:aws:acm:region:account-id:certificate/ownership-verification-certificate-id"),
-		RegionalCertificateArn:              PtrString("arn:aws:acm:region:account-id:certificate/regional-certificate-id"),
-		RegionalCertificateName:             PtrString("regional-certificate-name"),
-		RegionalDomainName:                  PtrString("regional-domain-name"),
-		RegionalHostedZoneId:                PtrString("regional-hosted-zone-id"),
+		MutualTlsAuthentication:             &types.MutualTlsAuthentication{TruststoreUri: new("truststore-uri")},
+		OwnershipVerificationCertificateArn: new("arn:aws:acm:region:account-id:certificate/ownership-verification-certificate-id"),
+		RegionalCertificateArn:              new("arn:aws:acm:region:account-id:certificate/regional-certificate-id"),
+		RegionalCertificateName:             new("regional-certificate-name"),
+		RegionalDomainName:                  new("regional-domain-name"),
+		RegionalHostedZoneId:                new("regional-hosted-zone-id"),
 		SecurityPolicy:                      types.SecurityPolicyTls12,
 		Tags:                                map[string]string{"key": "value"},
 	}